From mboxrd@z Thu Jan 1 00:00:00 1970 From: john cooper Subject: Re: [PATCH 2/2] Remove virtio_blk VBID ioctl Date: Mon, 21 Jun 2010 01:07:53 -0400 Message-ID: <4C1EF3A9.6080703@redhat.com> References: <1276886283-1571-1-git-send-email-ryanh@us.ibm.com> <1276886283-1571-2-git-send-email-ryanh@us.ibm.com> <201006211100.19719.rusty@rustcorp.com.au> Mime-Version: 1.0 Content-Type: text/plain; charset=iso-8859-15 Content-Transfer-Encoding: 7bit Cc: Ryan Harper , virtualization@lists.linux-foundation.org, kvm@vger.kernel.org, qemu-devel@nongnu.org, john.cooper@redhat.com To: Rusty Russell Return-path: Received: from mx1.redhat.com ([209.132.183.28]:36487 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753205Ab0FUF0S (ORCPT ); Mon, 21 Jun 2010 01:26:18 -0400 In-Reply-To: <201006211100.19719.rusty@rustcorp.com.au> Sender: kvm-owner@vger.kernel.org List-ID: Rusty Russell wrote: > On Sat, 19 Jun 2010 04:08:03 am Ryan Harper wrote: >> With the availablility of a sysfs device attribute for examining disk serial >> numbers the ioctl is no longer needed. The user-space changes for this aren't >> upstream yet so we don't have any users to worry about. > > If John Cooper acks this, I'll push it to Linus immediately. Actually I'm the one who suggested removing it. The code in question was only intended as example usage of accessing the s/n data in the driver, for the /sys interface under discussion back then. That effort subsequently stalled and Ryan had recently picked it up. As such I believe this overshadows the general need for an ioctl. Even if for some reason an ioctl would be justified going forward, a more usage friendly form would be better. So let's just drop it for now as the corresponding qemu-side code hasn't been merged. > Unfortunately we offered this interface in 2.6.34, and we're now removing it. > That's unpleasant. Indeed. This entire effort, aside from being an exercise in protracted agony, probably violates a Rube Goldberg patent. Thanks, -john -- john.cooper@redhat.com From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from [140.186.70.92] (port=55194 helo=eggs.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1OQZWO-0004D0-CD for qemu-devel@nongnu.org; Mon, 21 Jun 2010 01:26:13 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.69) (envelope-from ) id 1OQZWN-0004Mg-7e for qemu-devel@nongnu.org; Mon, 21 Jun 2010 01:26:12 -0400 Received: from mx1.redhat.com ([209.132.183.28]:54438) by eggs.gnu.org with esmtp (Exim 4.69) (envelope-from ) id 1OQZWN-0004MW-0y for qemu-devel@nongnu.org; Mon, 21 Jun 2010 01:26:11 -0400 Message-ID: <4C1EF3A9.6080703@redhat.com> Date: Mon, 21 Jun 2010 01:07:53 -0400 From: john cooper MIME-Version: 1.0 References: <1276886283-1571-1-git-send-email-ryanh@us.ibm.com> <1276886283-1571-2-git-send-email-ryanh@us.ibm.com> <201006211100.19719.rusty@rustcorp.com.au> In-Reply-To: <201006211100.19719.rusty@rustcorp.com.au> Content-Type: text/plain; charset=iso-8859-15 Content-Transfer-Encoding: 7bit Subject: [Qemu-devel] Re: [PATCH 2/2] Remove virtio_blk VBID ioctl List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Rusty Russell Cc: john.cooper@redhat.com, Ryan Harper , qemu-devel@nongnu.org, kvm@vger.kernel.org, virtualization@lists.linux-foundation.org Rusty Russell wrote: > On Sat, 19 Jun 2010 04:08:03 am Ryan Harper wrote: >> With the availablility of a sysfs device attribute for examining disk serial >> numbers the ioctl is no longer needed. The user-space changes for this aren't >> upstream yet so we don't have any users to worry about. > > If John Cooper acks this, I'll push it to Linus immediately. Actually I'm the one who suggested removing it. The code in question was only intended as example usage of accessing the s/n data in the driver, for the /sys interface under discussion back then. That effort subsequently stalled and Ryan had recently picked it up. As such I believe this overshadows the general need for an ioctl. Even if for some reason an ioctl would be justified going forward, a more usage friendly form would be better. So let's just drop it for now as the corresponding qemu-side code hasn't been merged. > Unfortunately we offered this interface in 2.6.34, and we're now removing it. > That's unpleasant. Indeed. This entire effort, aside from being an exercise in protracted agony, probably violates a Rube Goldberg patent. Thanks, -john -- john.cooper@redhat.com