lkml.org 
[lkml]   [2022]   [Mar]   [16]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
SubjectRe: [PATCH RFC 2/2] libata: Use scsi cmnd budget token for qc tag for SAS host
From
Hi Damien,

>> - }
>> - return -1;
>> + return scmd->budget_token;
>> }
> Since this is now not actually allocating a tag, I would rename this
> something like ata_sas_get_tag(). Or even better, simply open code this
> in ata_qc_new_init() since that is the only caller.

ok, I think it might be better to open code in ata_qc_new_init(), as
suggested. That should avoid the need for the return -1 call.

>
>>
>> void ata_sas_free_tag(unsigned int tag, struct ata_port *ap)
>> {
>> - clear_bit(tag, &ap->sas_tag_allocated);
>> }
> This is called only in ata_qc_free(). With this change, the function is
> empty, so let's completely remove it.
>

ok

>>
>> /**
>> diff --git a/drivers/ata/libata-scsi.c b/drivers/ata/libata-scsi.c
>> index ed8be585a98f..45d63a2ba3ee 100644
>> --- a/drivers/ata/libata-scsi.c
>> +++ b/drivers/ata/libata-scsi.c
>> @@ -640,7 +640,7 @@ static struct ata_queued_cmd *ata_scsi_qc_new(struct ata_device *dev,
>> {

Thanks,
John

\
 
 \ /
  Last update: 2022-03-16 09:24    [W:0.115 / U:0.116 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site