Re: [PATCH 4/6] libata: Add ata_host_flags

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Jeff Garzik wrote:
> Brian King wrote:
>> Since SATA devices attached to SAS HBAs will not have
>> a host_set, add a helper function to retrieve the host_set
>> flags which can check for the existence of a host_set.
>>
>> Signed-off-by: Brian King <brking@xxxxxxxxxx>
>> ---
>>
>>  libata-dev-bjking1/drivers/scsi/libata-core.c |    7 +++----
>>  libata-dev-bjking1/include/linux/libata.h     |    7 +++++++
>>  2 files changed, 10 insertions(+), 4 deletions(-)
>>
>> diff -puN drivers/scsi/libata-core.c~libata_sas_no_simplex drivers/scsi/libata-core.c
>> --- libata-dev/drivers/scsi/libata-core.c~libata_sas_no_simplex	2006-06-06 08:55:43.000000000 -0500
>> +++ libata-dev-bjking1/drivers/scsi/libata-core.c	2006-06-06 08:55:43.000000000 -0500
>> @@ -2161,7 +2161,7 @@ int ata_set_mode(struct ata_port *ap, st
>>  	/* Record simplex status. If we selected DMA then the other
>>  	 * host channels are not permitted to do so.
>>  	 */
>> -	if (used_dma && (ap->host_set->flags & ATA_HOST_SIMPLEX))
>> +	if (used_dma && (ata_host_flags(ap) & ATA_HOST_SIMPLEX))
> 
> I'm not sure I like this one.  Will have to think a bit more on this -- 
> there might be a need for ipr to carry some cross-port libata state.

I can't think of any need at this point. Would it be reasonable to go ahead
with this change and add in the support for a cross-port libata state
if and when such a need comes along? 


-- 
Brian King
eServer Storage I/O
IBM Linux Technology Center
-
: send the line "unsubscribe linux-ide" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Filesystems]     [Linux SCSI]     [Linux RAID]     [Git]     [Kernel Newbies]     [Linux Newbie]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Samba]     [Device Mapper]

  Powered by Linux