From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:53963) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1cU1wP-0002Ng-2E for qemu-devel@nongnu.org; Wed, 18 Jan 2017 20:55:06 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1cU1wM-0001aC-0G for qemu-devel@nongnu.org; Wed, 18 Jan 2017 20:55:05 -0500 Received: from mx0b-001b2d01.pphosted.com ([148.163.158.5]:55158 helo=mx0a-001b2d01.pphosted.com) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1cU1wL-0001Zz-Rl for qemu-devel@nongnu.org; Wed, 18 Jan 2017 20:55:01 -0500 Received: from pps.filterd (m0098413.ppops.net [127.0.0.1]) by mx0b-001b2d01.pphosted.com (8.16.0.20/8.16.0.20) with SMTP id v0J1rdno031605 for ; Wed, 18 Jan 2017 20:55:00 -0500 Received: from e35.co.us.ibm.com (e35.co.us.ibm.com [32.97.110.153]) by mx0b-001b2d01.pphosted.com with ESMTP id 282etctrpu-1 (version=TLSv1.2 cipher=AES256-SHA bits=256 verify=NOT) for ; Wed, 18 Jan 2017 20:55:00 -0500 Received: from localhost by e35.co.us.ibm.com with IBM ESMTP SMTP Gateway: Authorized Use Only! Violators will be prosecuted for from ; Wed, 18 Jan 2017 18:55:00 -0700 Date: Thu, 19 Jan 2017 09:54:52 +0800 From: Dong Jia Shi References: <20170112072513.98411-1-bjsdjshi@linux.vnet.ibm.com> <20170112072513.98411-3-bjsdjshi@linux.vnet.ibm.com> <20170117145142.4954666f@t450s.home> <20170118025117.GN30301@bjsdjshi@linux.vnet.ibm.com> <20170118134147.5cf6beb0.cornelia.huck@de.ibm.com> <20170118134332.4210104e@t450s.home> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20170118134332.4210104e@t450s.home> Message-Id: <20170119015452.GS30301@bjsdjshi@linux.vnet.ibm.com> Subject: Re: [Qemu-devel] [PATCH RFC v2 02/12] vfio: linux-headers update for vfio-ccw List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Alex Williamson Cc: Cornelia Huck , Dong Jia Shi , kvm@vger.kernel.org, linux-s390@vger.kernel.org, qemu-devel@nongnu.org, renxiaof@linux.vnet.ibm.com, borntraeger@de.ibm.com, agraf@suse.com, pmorel@linux.vnet.ibm.com, pasic@linux.vnet.ibm.com, wkywang@linux.vnet.ibm.com * Alex Williamson [2017-01-18 13:43:32 -0700]: > On Wed, 18 Jan 2017 13:41:47 +0100 > Cornelia Huck wrote: > > > On Wed, 18 Jan 2017 10:51:17 +0800 > > Dong Jia Shi wrote: > > > > > * Alex Williamson [2017-01-17 14:51:42 -0700]: > > > > > > > On Thu, 12 Jan 2017 08:25:03 +0100 > > > > Dong Jia Shi wrote: > > > > > > > > > From: Xiao Feng Ren > > > > > > > > > > This is a placeholder for a linux-headers update. > > > > > > > > > > Signed-off-by: Xiao Feng Ren > > > > > --- > > > > > include/standard-headers/asm-s390/vfio_ccw.h | 28 ++++++++++++++++++++++++++++ > > > > > linux-headers/linux/vfio.h | 17 +++++++++++++++++ > > > > > 2 files changed, 45 insertions(+) > > > > > create mode 100644 include/standard-headers/asm-s390/vfio_ccw.h > > > > > > > > > > diff --git a/include/standard-headers/asm-s390/vfio_ccw.h b/include/standard-headers/asm-s390/vfio_ccw.h > > > > > new file mode 100644 > > > > > index 0000000..cddc09b > > > > > --- /dev/null > > > > > +++ b/include/standard-headers/asm-s390/vfio_ccw.h > > > > > @@ -0,0 +1,28 @@ > > > > > +/* > > > > > + * Interfaces for vfio-ccw > > > > > + * > > > > > + * Copyright IBM Corp. 2017 > > > > > + * > > > > > + * This program is free software; you can redistribute it and/or modify > > > > > + * it under the terms of the GNU General Public License (version 2 only) > > > > > + * as published by the Free Software Foundation. > > > > > + * > > > > > + * Author(s): Dong Jia Shi > > > > > + */ > > > > > + > > > > > +#ifndef _VFIO_CCW_H_ > > > > > +#define _VFIO_CCW_H_ > > > > > + > > > > > +#include "standard-headers/linux/types.h" > > > > > + > > > > > +struct ccw_io_region { > > > > > +#define ORB_AREA_SIZE 12 > > > > > + uint8_t orb_area[ORB_AREA_SIZE]; > > > > > +#define SCSW_AREA_SIZE 12 > > > > > + uint8_t scsw_area[SCSW_AREA_SIZE]; > > > > > +#define IRB_AREA_SIZE 96 > > > > > + uint8_t irb_area[IRB_AREA_SIZE]; > > > > > + uint32_t ret_code; > > > > > +} QEMU_PACKED; > > > > > + > > > > > +#endif > > > > > > > > This is really part of the uapi for the vfio-ccw mdev device, isn't it? > > > Yes, it is. > > > > > > > Should it really be buried in asm-s390 in the kernel? > > > > > > > We had an internal discussion on this question before, since we think > > > this interface is strongly s390 dependent, we put it here. What do you > > > suggest? Thanks, > > > > As there are already architecture-specific definitions in the common > > file, having everything in the same file is probably better. I'm > > certainly not against moving it out from asm-s390. > > As soon as we use it for the interface definition to the vfio region, I > think it makes sense for it to move to uapi. It's not really > architecture specific as much as it is device specific, for a device > which happens to only appear on s390 hardware. Putting it in uapi > should also make sure people think twice about changing it. Thanks, > > Alex > Fair enough. Let's move it to include/uapi/linux/vfio_ccw.h. -- Dong Jia