On 06/23/2015 09:46 AM, Stefan Hajnoczi wrote: > On Mon, Jun 22, 2015 at 10:08:22AM -0400, John Snow wrote: >> (sector_count == 0 means 64K sectors, though, like it does in regular >> ATA.) > > I don't see a case to handle this. >
I apologize: Corrected in [PATCH 09/16] ahci: correct ncq sector count, in the "part 2" series. Sorry for the confusing order, I played a lot of patch-reshuffling and due to the refactors to get NCQ migrating, sometimes it was literally just a sin of convenience to leave certain patches after the shuffle. I tried to leave each patch fixing just one very small issue at a time so they were easier to digest, instead of batching them together like "Fix (everything wrong about) sector count." Maybe that wasn't the right approach, you can let me know. Thank you, --John