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

2017-03-01 Thread Damien Le Moal
Martin, On 3/1/17 12:21, Martin K. Petersen wrote: >> "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

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 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..

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