From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757987AbcBDRs3 (ORCPT ); Thu, 4 Feb 2016 12:48:29 -0500 Received: from mail-qg0-f65.google.com ([209.85.192.65]:35096 "EHLO mail-qg0-f65.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751097AbcBDRs1 (ORCPT ); Thu, 4 Feb 2016 12:48:27 -0500 Subject: Re: [PATCH 5/6] ibmvscsi: Remove unsupported host config MAD and sysfs interface To: Johannes Thumshirn , Tyrel Datwyler References: <1454542114-1797-1-git-send-email-tyreld@linux.vnet.ibm.com> <1454542114-1797-6-git-send-email-tyreld@linux.vnet.ibm.com> <20160204080317.GT27969@c203.arch.suse.de> Cc: martin.petersen@oracle.com, linux-scsi@vger.kernel.org, JBottomley@odin.com, linux-kernel@vger.kernel.org, hare@suse.de, brking@linux.vnet.ibm.com, nfont@linux.vnet.ibm.com, linuxppc-dev@lists.ozlabs.org From: Tyrel Datwyler Message-ID: <56B38EE7.6060102@gmail.com> Date: Thu, 4 Feb 2016 09:48:23 -0800 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.5.0 MIME-Version: 1.0 In-Reply-To: <20160204080317.GT27969@c203.arch.suse.de> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 02/04/2016 12:03 AM, Johannes Thumshirn wrote: > On Wed, Feb 03, 2016 at 05:28:33PM -0600, Tyrel Datwyler wrote: >> A VIOSRP_HOST_CONFIG_TYPE management datagram (MAD) has existed in >> the code for some time. From what information I've gathered from >> Brian King this was likely implemented on the host side in a SLES 9 >> based VIOS, which is no longer supported anywhere. Further, it is >> not defined in PAPR or supported by any AIX based VIOS. >> >> Treating as bit rot and removing the sysfs interface and associated >> host config code accordingly. > > Doesn't removing a sysfs interface potentially break userspace code? > In the general case yes, but I feel in this case no. First, Reading from this config attribute of a vscsi host adapter always returns nothing. Second, any userspace code using this attribute better be checking for the existence of config. Just a quick look for /sys/class/scsi_host/host*/config under other host adapters on my system I find that attribute doesn't exist for any of them. If there is truly enough concern that somebody may actually be accessing this useless attribute from userspace then we can still strip out the unsupported code, but leave the attribute and return nothing directly from the show function. -Tyrel