Re: [PATCH 0/3] Separate zone requests from medium access requests

2017-02-28 Thread Martin K. Petersen
> "Damien" == Damien Le Moal writes: Damien, Damien> The problem remains that the mpt3sas driver needs fixing. As you Damien> suggest, we can do that in sd, or directly in mpt3sas. I tried Damien> to do a clean fix in sd, but always end up consuming a lot of Damien> aspirin because of all t

Re: [PATCH 0/3] Separate zone requests from medium access requests

2017-02-28 Thread Damien Le Moal
Martin, On 3/1/17 11:52, Martin K. Petersen wrote: >> "Christoph" == Christoph Hellwig writes: > > Christoph> I don't really like this too much - this is too many SCSI > Christoph> specifics for the block layer to care. Maybe using bios for > Christoph> the zone ops was a mistake after all,

Re: [PATCH 0/3] Separate zone requests from medium access requests

2017-02-28 Thread Martin K. Petersen
> "Christoph" == Christoph Hellwig writes: Christoph> I don't really like this too much - this is too many SCSI Christoph> specifics for the block layer to care. Maybe using bios for Christoph> the zone ops was a mistake after all, and we should just have Christoph> operations in struct bloc

Re: [PATCH 0/3] Separate zone requests from medium access requests

2017-02-28 Thread Bart Van Assche
On Tue, 2017-02-28 at 17:02 +0100, Christoph Hellwig wrote: > I don't really like this too much - this is too many SCSI specifics > for the block layer to care. Maybe using bios for the zone ops was a > mistake after all, and we should just have operations in struct block_device > instead.. blk_r

Re: [PATCH 0/3] Separate zone requests from medium access requests

2017-02-28 Thread Christoph Hellwig
I don't really like this too much - this is too many SCSI specifics for the block layer to care. Maybe using bios for the zone ops was a mistake after all, and we should just have operations in struct block_device instead..

[PATCH 0/3] Separate zone requests from medium access requests

2017-02-28 Thread Damien Le Moal
This series introduces blk_rq_accesses_medium(), which is equivalent to !blk_rq_is_passthrough() minus the zone request operations REQ_OP_ZONE_REPORT and REQ_OP_ZONE_RESET. This new helper allows avoiding problems due to the non-standard nature of these commands (report zones does no operate on log