Re: [PATCH v5 2/4] fusion: remove iopriority handling

2016-10-14 Thread Adam Manzanares
Th 10/13/2016 22:34, Christoph Hellwig wrote: > > Style wise you can further remove the extra parens around > > SCpnt->device->tagged_supported > > As well as the now redundant braces. > > I did send a patch looking just like that earlier :) I'll remove the patch from the patchset. I must have

Re: [PATCH v5 2/4] fusion: remove iopriority handling

2016-10-13 Thread Christoph Hellwig
> Style wise you can further remove the extra parens around > SCpnt->device->tagged_supported > As well as the now redundant braces. I did send a patch looking just like that earlier :) -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to

Re: [PATCH v5 2/4] fusion: remove iopriority handling

2016-10-13 Thread Shaun Tancheff
On Thu, Oct 13, 2016 at 6:00 PM, Adam Manzanares wrote: > The request priority is now by default coming from the ioc. It was not > clear what this code was trying to do based upon the iopriority class or > data. The driver should check that a device supports priorities

[PATCH v5 2/4] fusion: remove iopriority handling

2016-10-13 Thread Adam Manzanares
The request priority is now by default coming from the ioc. It was not clear what this code was trying to do based upon the iopriority class or data. The driver should check that a device supports priorities and use them according to the specificiations of ioprio. Signed-off-by: Adam Manzanares