From mboxrd@z Thu Jan 1 00:00:00 1970 From: Hannes Reinecke Date: Mon, 02 Apr 2012 13:35:09 +0000 Subject: Re: [PATCH] path_id: Re-introduce SAS phy enumeration of devices Message-Id: <4F79AB0D.9000100@suse.de> List-Id: References: <1333371414-412-1-git-send-email-nils.carlson@ericsson.com> In-Reply-To: <1333371414-412-1-git-send-email-nils.carlson@ericsson.com> MIME-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable To: linux-hotplug@vger.kernel.org Hi Nils, Thanks a lot for this. It definitely a step in the correct direction. However, I have some comments to make: On 04/02/2012 02:56 PM, Nils Carlson wrote: > When path_id was converted to C code the enumeration of SAS > devices by phy disappeared. This patch reintroduces enumeration > of the form >=20 > pci-0000:05:00.0-sas-phy0:1-0x500000e114de2b42:0-lun0 >=20 > where phy0:1 is the reintroduced substring where 0 corresponds > to the lowest phy identifier on the port to which the device > is connected and 1 is the number of phys on the port. >=20 I would rather not do this. First of all, I doubt we need the overall number of phys here. Secondly, James B. assured me that the phy enumeration is pretty much stable, so we should be able to use that number as-is. Also, there is a 1:1 match between the 'phy_identifier' sysfs attribute and the second number of the phy name itself (the first number is the SCSI host number), so we could as well just parse the phy name and get the number from there. But, of course, reading the phy_identifier is okay, too. So I would propose just to insert a 'phy1' there. But again, thanks for doing it. Cheers, Hannes --=20 Dr. Hannes Reinecke zSeries & Storage hare@suse.de +49 911 74053 688 SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 N=FCrnberg GF: J. Hawn, J. Guild, F. Imend=F6rffer, HRB 16746 (AG N=FCrnberg)