Re: [PATCH 5/7] libata: move and reduce locking to the pio data xfer functions

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

 



Alan Cox wrote:
>> This case is where I fail to understand how it's supposed to work.  If
>> IRQ beats the clearing of the ignore irq flag && execution of completion
>> routine from wq, it ignores the IRQ, right?  The IRQ line remains
> 
> I was assuming your polling handler would be polling and clearing the IRQ
> status otherwise it doens't work anyway - cable noise or drive error and
> an early DRQ de-assert would do the same thing during a transfer with IRQ
> blocked only on the last dword.

Yeah, it's not a water-tight solution.  I don't think we can do that
with IRQ enabled during transfer.  So, we can have either highly jerky
but reliable system if PIO is used or usually better behaving PIO with
some chance of getting nobody-cared.  Also, we can flag known buggy
devices/controllers such that IRQ is disabled over PIO by default.  I
think it's worth doing but this definitely should be debated better.

-- 
tejun
-
To unsubscribe from this list: 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