Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752592AbbKKNFo (ORCPT ); Wed, 11 Nov 2015 08:05:44 -0500 Received: from mail-lf0-f50.google.com ([209.85.215.50]:33349 "EHLO mail-lf0-f50.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751959AbbKKNFl convert rfc822-to-8bit (ORCPT ); Wed, 11 Nov 2015 08:05:41 -0500 From: Kashyap Desai References: <1447243241-10912-1-git-send-email-Sreekanth.Reddy@avagotech.com> <1447243241-10912-10-git-send-email-Sreekanth.Reddy@avagotech.com> <56433B12.4080405@suse.de> In-Reply-To: <56433B12.4080405@suse.de> MIME-Version: 1.0 X-Mailer: Microsoft Outlook 14.0 Thread-Index: AQEald+GEjKJiX6etB9hw3Yx/hPNIgFcE2P7AcjollOf6q6eEA== Date: Wed, 11 Nov 2015 18:35:37 +0530 Message-ID: <55868cb7dfd5f144d82f1983c8b9331a@mail.gmail.com> Subject: RE: [PATCH RESEND 09/25] mpt3sas: Don't send PHYDISK_HIDDEN Raid Action request on SAS2 HBA's To: Hannes Reinecke , Sreekanth Reddy , jejb@kernel.org Cc: martin.petersen@oracle.com, linux-scsi@vger.kernel.org, JBottomley@parallels.com, Sathya Prakash , linux-kernel@vger.kernel.org, hch@infradead.org, Chaitra Basappa , Suganath Prabu Subramani Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8BIT Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 6232 Lines: 185 > -----Original Message----- > From: Hannes Reinecke [mailto:hare@suse.de] > Sent: Wednesday, November 11, 2015 6:27 PM > To: Sreekanth Reddy; jejb@kernel.org > Cc: martin.petersen@oracle.com; linux-scsi@vger.kernel.org; > JBottomley@Parallels.com; Sathya.Prakash@avagotech.com; > kashyap.desai@avagotech.com; linux-kernel@vger.kernel.org; > hch@infradead.org; chaitra.basappa@avagotech.com; suganath- > prabu.subramani@avagotech.com > Subject: Re: [PATCH RESEND 09/25] mpt3sas: Don't send PHYDISK_HIDDEN > Raid Action request on SAS2 HBA's > > On 11/11/2015 01:00 PM, Sreekanth Reddy wrote: > > From: Sreekanth Reddy > > > > Don't send PHYDISK_HIDDEN Raid Action request for SAS2 HBA's. > > Since these HBA's doesn't support this Raid Action. > > > > Also enable fast_path only for SAS3 HBA's. > > > > Signed-off-by: Sreekanth Reddy > > --- > > drivers/scsi/mpt3sas/mpt3sas_scsih.c | 19 +++++++++++++++++-- > > 1 file changed, 17 insertions(+), 2 deletions(-) > > > > diff --git a/drivers/scsi/mpt3sas/mpt3sas_scsih.c > > b/drivers/scsi/mpt3sas/mpt3sas_scsih.c > > index a638920..80469d0 100644 > > --- a/drivers/scsi/mpt3sas/mpt3sas_scsih.c > > +++ b/drivers/scsi/mpt3sas/mpt3sas_scsih.c > > @@ -1165,8 +1165,10 @@ scsih_target_alloc(struct scsi_target *starget) > > if (test_bit(sas_device->handle, ioc->pd_handles)) > > sas_target_priv_data->flags |= > > MPT_TARGET_FLAGS_RAID_COMPONENT; > > +#ifndef SCSI_MPT2SAS > > if (sas_device->fast_path) > > sas_target_priv_data->flags |= > MPT_TARGET_FASTPATH_IO; > > +#endif > > } > > spin_unlock_irqrestore(&ioc->sas_device_lock, flags); > > > > @@ -3719,11 +3721,13 @@ scsih_qcmd(struct Scsi_Host *shost, struct > scsi_cmnd *scmd) > > ioc->build_zero_len_sge(ioc, &mpi_request->SGL); > > > > if (likely(mpi_request->Function == > MPI2_FUNCTION_SCSI_IO_REQUEST)) > > { > > +#ifndef SCSI_MPT2SAS > > if (sas_target_priv_data->flags & > MPT_TARGET_FASTPATH_IO) { > > mpi_request->IoFlags = cpu_to_le16(scmd- > >cmd_len | > > MPI25_SCSIIO_IOFLAGS_FAST_PATH); > > mpt3sas_base_put_smid_fast_path(ioc, smid, > handle); > > } else > > +#endif > > mpt3sas_base_put_smid_scsi_io(ioc, smid, handle); > > } else > > mpt3sas_base_put_smid_default(ioc, smid); @@ -5031,8 > +5035,10 @@ > > _scsih_add_device(struct MPT3SAS_ADAPTER *ioc, u16 handle, u8 > phy_num, > > sas_device->device_info = device_info; > > sas_device->sas_address = sas_address; > > sas_device->phy = sas_device_pg0.PhyNum; > > +#ifndef SCSI_MPT2SAS > > sas_device->fast_path = (le16_to_cpu(sas_device_pg0.Flags) & > > MPI25_SAS_DEVICE0_FLAGS_FAST_PATH_CAPABLE) ? 1 : 0; > > +#endif > > > > if (sas_device_pg0.Flags & > MPI2_SAS_DEVICE0_FLAGS_ENCL_LEVEL_VALID) { > > sas_device->enclosure_level = > > @@ -5731,6 +5737,7 @@ _scsih_sas_discovery_event(struct > MPT3SAS_ADAPTER *ioc, > > } > > } > > > > +#ifndef SCSI_MPT2SAS > > /** > > * _scsih_ir_fastpath - turn on fastpath for IR physdisk > > * @ioc: per adapter object > > @@ -5750,7 +5757,6 @@ _scsih_ir_fastpath(struct MPT3SAS_ADAPTER > *ioc, u16 handle, u8 phys_disk_num) > > u16 ioc_status; > > u32 log_info; > > > > - > > mutex_lock(&ioc->scsih_cmds.mutex); > > > > if (ioc->scsih_cmds.status != MPT3_CMD_NOT_USED) { @@ - > 5825,6 > > +5831,8 @@ _scsih_ir_fastpath(struct MPT3SAS_ADAPTER *ioc, u16 > handle, u8 phys_disk_num) > > FORCE_BIG_HAMMER); > > return rc; > > } > > +/* End of not defined SCSI_MPT2SAS */ #endif > > > > /** > > * _scsih_reprobe_lun - reprobing lun @@ -6017,8 +6025,10 @@ > > _scsih_sas_pd_hide(struct MPT3SAS_ADAPTER *ioc, > > if (!sas_device) > > return; > > > > +#ifndef SCSI_MPT2SAS > > /* hiding raid component */ > > _scsih_ir_fastpath(ioc, handle, element->PhysDiskNum); > > +#endif > > if (starget) > > starget_for_each_device(starget, (void *)1, > _scsih_reprobe_lun); } > > @@ -6067,7 +6077,9 @@ _scsih_sas_pd_add(struct MPT3SAS_ADAPTER > *ioc, > > sas_device = _scsih_sas_device_find_by_handle(ioc, handle); > > spin_unlock_irqrestore(&ioc->sas_device_lock, flags); > > if (sas_device) { > > +#ifndef SCSI_MPT2SAS > > _scsih_ir_fastpath(ioc, handle, element->PhysDiskNum); > > +#endif > > return; > > } > > > > @@ -6091,7 +6103,9 @@ _scsih_sas_pd_add(struct MPT3SAS_ADAPTER > *ioc, > > mpt3sas_transport_update_links(ioc, sas_address, handle, > > sas_device_pg0.PhyNum, > MPI2_SAS_NEG_LINK_RATE_1_5); > > > > +#ifndef SCSI_MPT2SAS > > _scsih_ir_fastpath(ioc, handle, element->PhysDiskNum); > > +#endif > > _scsih_add_device(ioc, handle, 0, 1); } > > > > @@ -6202,13 +6216,14 @@ _scsih_sas_ir_config_change_event(struct > > MPT3SAS_ADAPTER *ioc, > > > > element = (Mpi2EventIrConfigElement_t *)&event_data- > >ConfigElement[0]; > > if (ioc->shost_recovery) { > > - > > +#ifndef SCSI_MPT2SAS > > for (i = 0; i < event_data->NumElements; i++, element++) { > > if (element->ReasonCode == > MPI2_EVENT_IR_CHANGE_RC_HIDE) > > _scsih_ir_fastpath(ioc, > > le16_to_cpu(element- > >PhysDiskDevHandle), > > element->PhysDiskNum); > > } > > +#endif > > return; > > } > > for (i = 0; i < event_data->NumElements; i++, element++) { > > > Why didn't you use 'hba_mpi_version_belonged' here? Hannes, what have requested is there in "[PATCH 19/25] mpt3sas: Single driver module which supports both SAS 2.0 & SAS 3.0 HBA's" Actually patch series till 0-18 are resend by Sreekanth just for quick reference and/or to avoid any confusion. New changes are from 19-25. From 0-18 is already picked by Martin. We need review by tag for patch series from 19 to 25. ` Kashyap > > Cheers, > > Hannes > -- > Dr. Hannes Reinecke zSeries & Storage > hare@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) -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/