From mboxrd@z Thu Jan 1 00:00:00 1970 From: Rusty Russell Subject: Re: [PATCH] virtio_pci: fix macro exported in uapi Date: Fri, 17 May 2013 09:39:07 +0930 Message-ID: <87wqqy8o2k.fsf@rustcorp.com.au> References: <20130516164446.GA32007@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Cc: linux-kernel@vger.kernel.org, virtualization@lists.linux-foundation.org, David Howells , Dave Jones , Paolo Bonzini , Stephen Hemminger , "Paul E. McKenney" , Thomas Gleixner To: "Michael S. Tsirkin" , kvm@vger.kernel.org Return-path: In-Reply-To: <20130516164446.GA32007@redhat.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: virtualization-bounces@lists.linux-foundation.org Errors-To: virtualization-bounces@lists.linux-foundation.org List-Id: kvm.vger.kernel.org "Michael S. Tsirkin" writes: > macro VIRTIO_PCI_CONFIG except in the unlikely event userspace > actually has a structure with a field named msix_enabled. > Get the msix_enabled by value instead, to make it useful > for userspace. > > Signed-off-by: Michael S. Tsirkin Macro still isn't usable, because userspace can't know whether it's the new or old. We need to either remove it from UAPI, or rename it to VIRTIO_PCI_CONFIG_OFF. Cheers, Rusty.