Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751491AbaK1Iyv (ORCPT ); Fri, 28 Nov 2014 03:54:51 -0500 Received: from e06smtp16.uk.ibm.com ([195.75.94.112]:41934 "EHLO e06smtp16.uk.ibm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751447AbaK1Iyu (ORCPT ); Fri, 28 Nov 2014 03:54:50 -0500 Date: Fri, 28 Nov 2014 09:54:36 +0100 From: David Hildenbrand To: "Michael S. Tsirkin" Cc: linux-kernel@vger.kernel.org, David Miller , cornelia.huck@de.ibm.com, rusty@au1.ibm.com, nab@linux-iscsi.org, pbonzini@redhat.com, thuth@linux.vnet.ibm.com, Rusty Russell , Ohad Ben-Cohen , Christian Borntraeger , linux390@de.ibm.com, Martin Schwidefsky , Heiko Carstens , Sudeep Dutt , Siva Yerramreddy , lguest@lists.ozlabs.org, linux-s390@vger.kernel.org, virtualization@lists.linux-foundation.org Subject: Re: [PATCH v6 05/46] virtio: assert 32 bit features in transports Message-ID: <20141128095436.20c59e84@thinkpad-w530> In-Reply-To: <1417118789-18231-6-git-send-email-mst@redhat.com> References: <1417118789-18231-1-git-send-email-mst@redhat.com> <1417118789-18231-6-git-send-email-mst@redhat.com> Organization: IBM Deutschland GmbH X-Mailer: Claws Mail 3.10.1 (GTK+ 2.24.24; x86_64-redhat-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit X-TM-AS-MML: disable X-Content-Scanned: Fidelis XPS MAILER x-cbid: 14112808-0025-0000-0000-000002A1BEAA Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org > At this point, no transports set any of the high 32 feature bits. > Since transports generally can't (yet) cope with such bits, add BUG_ON > checks to make sure they are not set by mistake. > > Based on rproc patch by Rusty. > > Signed-off-by: Rusty Russell > Signed-off-by: Michael S. Tsirkin > --- > drivers/lguest/lguest_device.c | 3 +++ > drivers/misc/mic/card/mic_virtio.c | 3 +++ > drivers/remoteproc/remoteproc_virtio.c | 3 +++ > drivers/s390/kvm/kvm_virtio.c | 3 +++ > drivers/s390/kvm/virtio_ccw.c | 3 +++ > drivers/virtio/virtio_mmio.c | 3 +++ > drivers/virtio/virtio_pci.c | 3 +++ > 7 files changed, 21 insertions(+) > Looks sane to me. Reviewed-by: David Hildenbrand -- 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/