* RE: [PATCH] aacraid: Fix 2T+ drives on SmartIOC-2000 [not found] <20170830222241.1560-1-david.carroll@microsemi.com> @ 2017-09-08 20:50 ` Raghava Aditya Renukunta 0 siblings, 0 replies; 4+ messages in thread From: Raghava Aditya Renukunta @ 2017-09-08 20:50 UTC (permalink / raw) To: Martin K . Petersen, James Bottomley Cc: Dave Carroll, linux-scsi, dl-esc-Aacraid Linux Driver, Scott Benesh, Brian King, stable@vger.kernel.org > -----Original Message----- > From: Dave Carroll [mailto:david.carroll@microsemi.com] > Sent: Wednesday, August 30, 2017 3:23 PM > To: Martin K . Petersen <martin.petersen@oracle.com>; James Bottomley > <jejb@linux.vnet.ibm.com> > Cc: Dave Carroll <david.carroll@microsemi.com>; linux-scsi <linux- > scsi@vger.kernel.org>; dl-esc-Aacraid Linux Driver > <aacraid@microsemi.com>; Scott Benesh <scott.benesh@microsemi.com>; > Brian King <brking@linux.vnet.ibm.com>; stable@vger.kernel.org > Subject: [PATCH] aacraid: Fix 2T+ drives on SmartIOC-2000 > > The logic for supporting large drives was previously tied to 4Kn support > for SmartIOC-2000. As SmartIOC-2000 does not support volumes using 4Kn > drives, use the intended option flag AAC_OPT_NEW_COMM_64 to > determine > support for volumes greater than 2T. > > cc: stable@vger.kernel.org > Signed-off-by: Dave Carroll <david.carroll@microsemi.com> > > --- > drivers/scsi/aacraid/aachba.c | 12 ++++++------ > drivers/scsi/aacraid/aacraid.h | 5 +++++ > 2 files changed, 11 insertions(+), 6 deletions(-) > > diff --git a/drivers/scsi/aacraid/aachba.c b/drivers/scsi/aacraid/aachba.c > index 4591113..18a1a1f 100644 > --- a/drivers/scsi/aacraid/aachba.c > +++ b/drivers/scsi/aacraid/aachba.c > @@ -695,13 +695,13 @@ static void _aac_probe_container1(void * context, > struct fib * fibptr) > int status; > > dresp = (struct aac_mount *) fib_data(fibptr); > - if (!(fibptr->dev->supplement_adapter_info.supported_options2 & > - AAC_OPTION_VARIABLE_BLOCK_SIZE)) > + if (!aac_supports_2T(fibptr->dev)) { > dresp->mnt[0].capacityhigh = 0; > - if ((le32_to_cpu(dresp->status) != ST_OK) || > - (le32_to_cpu(dresp->mnt[0].vol) != CT_NONE)) { > - _aac_probe_container2(context, fibptr); > - return; > + if ((le32_to_cpu(dresp->status) == ST_OK) && > + (le32_to_cpu(dresp->mnt[0].vol) != CT_NONE)) { > + _aac_probe_container2(context, fibptr); > + return; > + } > } > scsicmd = (struct scsi_cmnd *) context; > > diff --git a/drivers/scsi/aacraid/aacraid.h b/drivers/scsi/aacraid/aacraid.h > index 6981299..998fbad 100644 > --- a/drivers/scsi/aacraid/aacraid.h > +++ b/drivers/scsi/aacraid/aacraid.h > @@ -2701,6 +2701,11 @@ static inline int aac_is_src(struct aac_dev *dev) > return 0; > } > > +static inline int aac_supports_2T(struct aac_dev *dev) > +{ > + return (dev->adapter_info.options & AAC_OPT_NEW_COMM_64); > +} > + > char * get_container_type(unsigned type); > extern int numacb; > extern char aac_driver_version[]; > -- > 2.8.4 Reviewed-by: Raghava Aditya Renukunta <RaghavaAditya.Renukunta@microsemi.com> ^ permalink raw reply [flat|nested] 4+ messages in thread
* RE: [PATCH] aacraid: Fix 2T+ drives on SmartIOC-2000 @ 2017-09-08 20:50 ` Raghava Aditya Renukunta 0 siblings, 0 replies; 4+ messages in thread From: Raghava Aditya Renukunta @ 2017-09-08 20:50 UTC (permalink / raw) To: Dave Carroll, Martin K . Petersen, James Bottomley Cc: Dave Carroll, linux-scsi, dl-esc-Aacraid Linux Driver, Scott Benesh, Brian King, stable@vger.kernel.org > -----Original Message----- > From: Dave Carroll [mailto:david.carroll@microsemi.com] > Sent: Wednesday, August 30, 2017 3:23 PM > To: Martin K . Petersen <martin.petersen@oracle.com>; James Bottomley > <jejb@linux.vnet.ibm.com> > Cc: Dave Carroll <david.carroll@microsemi.com>; linux-scsi <linux- > scsi@vger.kernel.org>; dl-esc-Aacraid Linux Driver > <aacraid@microsemi.com>; Scott Benesh <scott.benesh@microsemi.com>; > Brian King <brking@linux.vnet.ibm.com>; stable@vger.kernel.org > Subject: [PATCH] aacraid: Fix 2T+ drives on SmartIOC-2000 > > The logic for supporting large drives was previously tied to 4Kn support > for SmartIOC-2000. As SmartIOC-2000 does not support volumes using 4Kn > drives, use the intended option flag AAC_OPT_NEW_COMM_64 to > determine > support for volumes greater than 2T. > > cc: stable@vger.kernel.org > Signed-off-by: Dave Carroll <david.carroll@microsemi.com> > > --- > drivers/scsi/aacraid/aachba.c | 12 ++++++------ > drivers/scsi/aacraid/aacraid.h | 5 +++++ > 2 files changed, 11 insertions(+), 6 deletions(-) > > diff --git a/drivers/scsi/aacraid/aachba.c b/drivers/scsi/aacraid/aachba.c > index 4591113..18a1a1f 100644 > --- a/drivers/scsi/aacraid/aachba.c > +++ b/drivers/scsi/aacraid/aachba.c > @@ -695,13 +695,13 @@ static void _aac_probe_container1(void * context, > struct fib * fibptr) > int status; > > dresp = (struct aac_mount *) fib_data(fibptr); > - if (!(fibptr->dev->supplement_adapter_info.supported_options2 & > - AAC_OPTION_VARIABLE_BLOCK_SIZE)) > + if (!aac_supports_2T(fibptr->dev)) { > dresp->mnt[0].capacityhigh = 0; > - if ((le32_to_cpu(dresp->status) != ST_OK) || > - (le32_to_cpu(dresp->mnt[0].vol) != CT_NONE)) { > - _aac_probe_container2(context, fibptr); > - return; > + if ((le32_to_cpu(dresp->status) == ST_OK) && > + (le32_to_cpu(dresp->mnt[0].vol) != CT_NONE)) { > + _aac_probe_container2(context, fibptr); > + return; > + } > } > scsicmd = (struct scsi_cmnd *) context; > > diff --git a/drivers/scsi/aacraid/aacraid.h b/drivers/scsi/aacraid/aacraid.h > index 6981299..998fbad 100644 > --- a/drivers/scsi/aacraid/aacraid.h > +++ b/drivers/scsi/aacraid/aacraid.h > @@ -2701,6 +2701,11 @@ static inline int aac_is_src(struct aac_dev *dev) > return 0; > } > > +static inline int aac_supports_2T(struct aac_dev *dev) > +{ > + return (dev->adapter_info.options & AAC_OPT_NEW_COMM_64); > +} > + > char * get_container_type(unsigned type); > extern int numacb; > extern char aac_driver_version[]; > -- > 2.8.4 Reviewed-by: Raghava Aditya Renukunta <RaghavaAditya.Renukunta@microsemi.com> ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] aacraid: Fix 2T+ drives on SmartIOC-2000 2017-09-08 20:50 ` Raghava Aditya Renukunta @ 2017-09-15 0:57 ` Martin K. Petersen -1 siblings, 0 replies; 4+ messages in thread From: Martin K. Petersen @ 2017-09-15 0:57 UTC (permalink / raw) To: Dave Carroll Cc: Raghava Aditya Renukunta, Martin K . Petersen, James Bottomley, linux-scsi, dl-esc-Aacraid Linux Driver, Scott Benesh, Brian King, stable@vger.kernel.org Dave, Not sure what happened. I do not have the original patch submission that Raghava replied to. Nor does patchwork or marc.info. Please resubmit. Thanks! -- Martin K. Petersen Oracle Linux Engineering ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] aacraid: Fix 2T+ drives on SmartIOC-2000 @ 2017-09-15 0:57 ` Martin K. Petersen 0 siblings, 0 replies; 4+ messages in thread From: Martin K. Petersen @ 2017-09-15 0:57 UTC (permalink / raw) To: Dave Carroll Cc: Raghava Aditya Renukunta, Martin K . Petersen, James Bottomley, linux-scsi, dl-esc-Aacraid Linux Driver, Scott Benesh, Brian King, stable@vger.kernel.org Dave, Not sure what happened. I do not have the original patch submission that Raghava replied to. Nor does patchwork or marc.info. Please resubmit. Thanks! -- Martin K. Petersen Oracle Linux Engineering ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2017-09-15 0:57 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <20170830222241.1560-1-david.carroll@microsemi.com>
2017-09-08 20:50 ` [PATCH] aacraid: Fix 2T+ drives on SmartIOC-2000 Raghava Aditya Renukunta
2017-09-08 20:50 ` Raghava Aditya Renukunta
2017-09-15 0:57 ` Martin K. Petersen
2017-09-15 0:57 ` Martin K. Petersen
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.