Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933012AbaLBNSY (ORCPT ); Tue, 2 Dec 2014 08:18:24 -0500 Received: from exprod7og127.obsmtp.com ([64.18.2.210]:47047 "EHLO exprod7og127.obsmtp.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754569AbaLBNSW (ORCPT ); Tue, 2 Dec 2014 08:18:22 -0500 From: Sreekanth Reddy References: <1375351291-27107-1-git-send-email-jvesely@redhat.com> MIME-Version: 1.0 X-Mailer: Microsoft Outlook 14.0 Thread-Index: Ac6OniBe6EkK5yWwQHuM4CDvYnurtQTp8z7wWvoY2BA= Date: Tue, 2 Dec 2014 18:48:18 +0530 Message-ID: <57e91c7f7bc1d3e0cd511f828ec8140b@mail.gmail.com> Subject: RE: [PATCH 0/2] mpt{2,3}sas remove disconnected phys on topology change To: Jan Vesely , linux-scsi@vger.kernel.org, linux-kernel@vger.kernel.org, "James E.J. Bottomley" Cc: Tomas Henzl , hch@infradead.org Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi James/Chris We are observing below issue due to this patch set code changes Issue Description: Drives connected Enclosure/Expander won't be visible to the OS if any one remove and add expander cable with in DMD (Device Missing Delay) time period (also if any power-off and power-on the Enclosure with in the DMD period). i.e. Due to this code changes; driver will unregister the Enclosure port or target PHY with SCSI transport layer, whenever driver receives "SAS TOPOLOGY CHANGE LIST EVENT" with reason code set to "PHY link status change" and phy link rate is Zero. Firmware usually send this type of "SAS TOPOLOGY CHANGE LIST EVENT" event whenever drive is missing. but still here DMD timer has not yet expired so normally driver should not unregister the target with the SCSI transport layer instead it will move the state of the drive to blocked state (so it won't receive any IOs from SML for this drive). Normally, Driver should only unregister the target with SCSI transport layer only when driver receives "SAS TOPOLOGY CHANGE LIST EVENT" with reason code set to "Target Remove". Firmware will send this event when the DMD timer expires for the missing drive. If drive comes back with in the DMD period then firmware won't send this target remove event and so driver won't unregister the target drive with the SCSI transport layer. Driver will just unblock the target drive to start IOs to this drive. So, can you please revert this patch set changes back in the latest upstream kernel and if possible on stable kernels. I checked with the Redhat team and there are fine to revert back these patch set changes. Regards, Sreekanth >-----Original Message----- >From: Jan Vesely [mailto:jvesely@redhat.com] >Sent: Thursday, August 01, 2013 3:31 PM >To: linux-scsi@vger.kernel.org; linux-kernel@vger.kernel.org >Cc: Jan Vesely; Nandigama, Nagalakshmi; Reddy, Sreekanth; Tomas Henzl >Subject: [PATCH 0/2] mpt{2,3}sas remove disconnected phys on topology >change > >From: Jan Vesely > >These two patches add phy removal on link loss. This change keeps sysfs >up- to-date with actually connected phys. Without these patches, >disconnected phys remain listed under their former ports. > >tested on both mpt2sas and mpt3sas hw. > >CC: Nagalakshmi Nandigama >CC: Sreekanth Reddy >CC: Tomas Henzl >Signed-off-by: Jan Vesely > >Jan Vesely (2): > mpt2sas: Remove phys on topology change. > mpt3sas: Remove phys on topology change > > drivers/scsi/mpt2sas/mpt2sas_transport.c | 5 ++++- > drivers/scsi/mpt3sas/mpt3sas_transport.c | 5 ++++- > 2 files changed, 8 insertions(+), 2 deletions(-) > -- 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/