Re: [PATCH v4 2/4] fusion: remove iopriority handling
On 10/13/2016 09:53 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 and use > them according to the specificiations of ioprio. > > Signed-off-by: Adam Manzanares > --- > drivers/message/fusion/mptscsih.c | 5 - > 1 file changed, 5 deletions(-) > > diff --git a/drivers/message/fusion/mptscsih.c > b/drivers/message/fusion/mptscsih.c > index 6c9fc11..4740bb6 100644 > --- a/drivers/message/fusion/mptscsih.c > +++ b/drivers/message/fusion/mptscsih.c > @@ -1369,11 +1369,6 @@ mptscsih_qcmd(struct scsi_cmnd *SCpnt) > if ((vdevice->vtarget->tflags & MPT_TARGET_FLAGS_Q_YES) > && (SCpnt->device->tagged_supported)) { > scsictl = scsidir | MPI_SCSIIO_CONTROL_SIMPLEQ; > - if (SCpnt->request && SCpnt->request->ioprio) { > - if (((SCpnt->request->ioprio & 0x7) == 1) || > - !(SCpnt->request->ioprio & 0x7)) > - scsictl |= MPI_SCSIIO_CONTROL_HEADOFQ; > - } > } else > scsictl = scsidir | MPI_SCSIIO_CONTROL_UNTAGGED; > > Reviewed-by: Hannes Reinecke Cheers, Hannes -- Dr. Hannes ReineckeTeamlead Storage & Networking h...@suse.de +49 911 74053 688 SUSE LINUX GmbH, Maxfeldstr. 5, 90409 Nürnberg GF: F. Imendörffer, J. Smithard, J. Guild, D. Upmanyu, G. Norton HRB 21284 (AG Nürnberg)
Re: [PATCH v4 2/4] fusion: remove iopriority handling
The 10/13/2016 15:05, Sathya Prakash Veerichetty wrote: > By removing the code below, we put all the commands for all the types of > devices (SAS/SATA) as simple-Q (requeue as the device require) and I am > not sure whether it is the intention of this change. > This is the intention of the change. I don't think the iopriority of the request is being used correctly. What does it mean to use 0x7 as an indicator that a command should be put at the head of the queue? This would be clearer if it was using some of the macros from ioprio. If 0x7 means something special I think this should be some #define in the includes of the fusion driver with some documentation. > -Original Message- > From: Adam Manzanares [mailto:adam.manzana...@hgst.com] > Sent: Thursday, October 13, 2016 1:54 PM > To: ax...@kernel.dk; t...@kernel.org; dan.j.willi...@intel.com; > h...@suse.de; martin.peter...@oracle.com; mchri...@redhat.com; > toshi.k...@hpe.com; ming@canonical.com; sathya.prak...@broadcom.com; > chaitra.basa...@broadcom.com; suganath-prabu.subram...@broadcom.com > Cc: linux-bl...@vger.kernel.org; linux-...@vger.kernel.org; > linux-kernel@vger.kernel.org; mpt-fusionlinux@broadcom.com; > linux-s...@vger.kernel.org; Adam Manzanares; Adam Manzanares > Subject: [PATCH v4 2/4] fusion: remove iopriority handling > > 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 > --- > drivers/message/fusion/mptscsih.c | 5 - > 1 file changed, 5 deletions(-) > > diff --git a/drivers/message/fusion/mptscsih.c > b/drivers/message/fusion/mptscsih.c > index 6c9fc11..4740bb6 100644 > --- a/drivers/message/fusion/mptscsih.c > +++ b/drivers/message/fusion/mptscsih.c > @@ -1369,11 +1369,6 @@ mptscsih_qcmd(struct scsi_cmnd *SCpnt) > if ((vdevice->vtarget->tflags & MPT_TARGET_FLAGS_Q_YES) > && (SCpnt->device->tagged_supported)) { > scsictl = scsidir | MPI_SCSIIO_CONTROL_SIMPLEQ; > - if (SCpnt->request && SCpnt->request->ioprio) { > - if (((SCpnt->request->ioprio & 0x7) == 1) || > - !(SCpnt->request->ioprio & 0x7)) > - scsictl |= MPI_SCSIIO_CONTROL_HEADOFQ; > - } > } else > scsictl = scsidir | MPI_SCSIIO_CONTROL_UNTAGGED; > > -- > 2.1.4 > > Western Digital Corporation (and its subsidiaries) E-mail Confidentiality > Notice & Disclaimer: > > This e-mail and any files transmitted with it may contain confidential or > legally privileged information of WDC and/or its affiliates, and are > intended solely for the use of the individual or entity to which they are > addressed. If you are not the intended recipient, any disclosure, copying, > distribution or any action taken or omitted to be taken in reliance on it, > is prohibited. If you have received this e-mail in error, please notify > the sender immediately and delete the e-mail in its entirety from your > system. > -- > To unsubscribe from this list: send the line "unsubscribe linux-ide" in > the body of a message to majord...@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html Take care, Adam
RE: [PATCH v4 2/4] fusion: remove iopriority handling
By removing the code below, we put all the commands for all the types of devices (SAS/SATA) as simple-Q (requeue as the device require) and I am not sure whether it is the intention of this change. -Original Message- From: Adam Manzanares [mailto:adam.manzana...@hgst.com] Sent: Thursday, October 13, 2016 1:54 PM To: ax...@kernel.dk; t...@kernel.org; dan.j.willi...@intel.com; h...@suse.de; martin.peter...@oracle.com; mchri...@redhat.com; toshi.k...@hpe.com; ming@canonical.com; sathya.prak...@broadcom.com; chaitra.basa...@broadcom.com; suganath-prabu.subram...@broadcom.com Cc: linux-bl...@vger.kernel.org; linux-...@vger.kernel.org; linux-kernel@vger.kernel.org; mpt-fusionlinux@broadcom.com; linux-s...@vger.kernel.org; Adam Manzanares; Adam Manzanares Subject: [PATCH v4 2/4] fusion: remove iopriority handling 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 --- drivers/message/fusion/mptscsih.c | 5 - 1 file changed, 5 deletions(-) diff --git a/drivers/message/fusion/mptscsih.c b/drivers/message/fusion/mptscsih.c index 6c9fc11..4740bb6 100644 --- a/drivers/message/fusion/mptscsih.c +++ b/drivers/message/fusion/mptscsih.c @@ -1369,11 +1369,6 @@ mptscsih_qcmd(struct scsi_cmnd *SCpnt) if ((vdevice->vtarget->tflags & MPT_TARGET_FLAGS_Q_YES) && (SCpnt->device->tagged_supported)) { scsictl = scsidir | MPI_SCSIIO_CONTROL_SIMPLEQ; - if (SCpnt->request && SCpnt->request->ioprio) { - if (((SCpnt->request->ioprio & 0x7) == 1) || - !(SCpnt->request->ioprio & 0x7)) - scsictl |= MPI_SCSIIO_CONTROL_HEADOFQ; - } } else scsictl = scsidir | MPI_SCSIIO_CONTROL_UNTAGGED; -- 2.1.4 Western Digital Corporation (and its subsidiaries) E-mail Confidentiality Notice & Disclaimer: This e-mail and any files transmitted with it may contain confidential or legally privileged information of WDC and/or its affiliates, and are intended solely for the use of the individual or entity to which they are addressed. If you are not the intended recipient, any disclosure, copying, distribution or any action taken or omitted to be taken in reliance on it, is prohibited. If you have received this e-mail in error, please notify the sender immediately and delete the e-mail in its entirety from your system.