From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1761531AbcLUSAW (ORCPT ); Wed, 21 Dec 2016 13:00:22 -0500 Received: from mail-bl2nam02on0106.outbound.protection.outlook.com ([104.47.38.106]:55488 "EHLO NAM02-BL2-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1758955AbcLUSAU (ORCPT ); Wed, 21 Dec 2016 13:00:20 -0500 X-Greylist: delayed 74886 seconds by postgrey-1.27 at vger.kernel.org; Wed, 21 Dec 2016 13:00:20 EST From: KY Srinivasan To: Roman Kagan , Paolo Bonzini , =?iso-8859-2?Q?Radim_Kr=E8m=E1=F8?= , Vitaly Kuznetsov CC: Thomas Gleixner , Ingo Molnar , "H. Peter Anvin" , "x86@kernel.org" , "Haiyang Zhang" , "kvm@vger.kernel.org" , "linux-kernel@vger.kernel.org" , "devel@linuxdriverproject.org" , "Denis V . Lunev" Subject: RE: [PATCH 00/15] hyperv: more stuff to uapi + cleanup Thread-Topic: [PATCH 00/15] hyperv: more stuff to uapi + cleanup Thread-Index: AQHSWtmpZQNaB0U3UECcydngdVz18KESrd7A Date: Wed, 21 Dec 2016 18:00:17 +0000 Message-ID: References: <20161220155602.6298-1-rkagan@virtuozzo.com> In-Reply-To: <20161220155602.6298-1-rkagan@virtuozzo.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: authentication-results: spf=none (sender IP is ) smtp.mailfrom=kys@microsoft.com; x-originating-ip: [2601:600:8c00:1040:b076:3b2a:409c:285e] x-ms-office365-filtering-correlation-id: f7630c3c-bb3b-4861-83fe-08d429cb3930 x-microsoft-antispam: UriScan:;BCL:0;PCL:0;RULEID:(22001);SRVR:BLUPR03MB1409; x-microsoft-exchange-diagnostics: 1;BLUPR03MB1409;7:r8lZ67rcfJcvKKFFUi1XY1aNDcpM7GmHH9LpSdCzm74oxmNg0f7P99306c/mwz0IuA193Dz5s2SNBRWu7ZOtYeEMk7/00jmWoLReyVdEVKCT3L7Zq5LLE4WXqBpZhfCJglhVrTSsTttRVeu0eKkRi3NmaSb1OJqMjOyHgi3mYx2HsentAodX7M9ZlIO1CuAeX3/WsbkIjhuagSrOKis6T2l8WUxAmxEP+Nf4moA9I4TqcEmfkUfTpPzgu+xP93oDEjvk+IlzEMbQwigacq5ZIBpodXsCl2jYmqhCBo0TRUs6pg/+TdcVOdCBtg/NyJWy053PR9wYzJ8l8BISLD34Liopg+wXIVXvvPHAvndIk/RqL4U3bh+SHULIpOOteU5RRpvij26p0SE+n/2vTYj9TBtZ1UQ7IWrilMkFIiA6pdS0/Z9Mhf/d1g1Jfe7KxCjFUHw3H+7ql/n0OCTst8oOzG8iH1PyBJ6UDV3oFaLTK/o= x-microsoft-antispam-prvs: x-exchange-antispam-report-test: UriScan:(278428928389397)(9452136761055); x-exchange-antispam-report-cfa-test: BCL:0;PCL:0;RULEID:(61425038)(6040375)(601004)(2401047)(8121501046)(5005006)(3002001)(10201501046)(6055026)(61426038)(61427038)(6041248)(20161123555025)(20161123560025)(20161123564025)(20161123562025)(6072148)(6047074);SRVR:BLUPR03MB1409;BCL:0;PCL:0;RULEID:;SRVR:BLUPR03MB1409; x-forefront-prvs: 01630974C0 x-forefront-antispam-report: SFV:NSPM;SFS:(10019020)(6009001)(7916002)(39860400002)(39850400002)(39450400003)(39840400002)(39410400002)(13464003)(377454003)(189002)(199003)(8990500004)(68736007)(106356001)(99286002)(105586002)(2950100002)(7416002)(5660300001)(81156014)(106116001)(74316002)(38730400001)(54356999)(229853002)(7696004)(76176999)(81166006)(9686002)(50986999)(8676002)(8936002)(101416001)(2906002)(33656002)(5005710100001)(10090500001)(3660700001)(76576001)(3280700002)(6116002)(102836003)(97736004)(122556002)(189998001)(2900100001)(10290500002)(4326007)(7736002)(77096006)(6436002)(92566002)(305945005)(86612001)(5001770100001)(6506006)(86362001)(25786008);DIR:OUT;SFP:1102;SCL:1;SRVR:BLUPR03MB1409;H:DM5PR03MB2490.namprd03.prod.outlook.com;FPR:;SPF:None;PTR:InfoNoRecords;MX:1;A:1;LANG:en; spamdiagnosticoutput: 1:99 spamdiagnosticmetadata: NSPM Content-Type: text/plain; charset="iso-8859-2" MIME-Version: 1.0 X-OriginatorOrg: microsoft.com X-MS-Exchange-CrossTenant-originalarrivaltime: 21 Dec 2016 18:00:17.7510 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 72f988bf-86f1-41af-91ab-2d7cd011db47 X-MS-Exchange-Transport-CrossTenantHeadersStamped: BLUPR03MB1409 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: 8bit X-MIME-Autoconverted: from quoted-printable to 8bit by mail.home.local id uBLI0RKC017311 > -----Original Message----- > From: Roman Kagan [mailto:rkagan@virtuozzo.com] > Sent: Tuesday, December 20, 2016 7:56 AM > To: Paolo Bonzini ; Radim Krèmáø > ; KY Srinivasan ; Vitaly > Kuznetsov > Cc: Thomas Gleixner ; Ingo Molnar > ; H. Peter Anvin ; x86@kernel.org; > Haiyang Zhang ; kvm@vger.kernel.org; linux- > kernel@vger.kernel.org; devel@linuxdriverproject.org; Denis V . Lunev > ; Roman Kagan > Subject: [PATCH 00/15] hyperv: more stuff to uapi + cleanup > > Expose more Hyper-V-related definitions in the uapi header for > consumption by userspace. > > While doing so, get rid of a number of duplications between the KVM and > the guest driver code. Also a few other cleanups are made which are not > strictly necessary for the main purpose of the series but appear > reasonable to do at the same time. > > The most controversial is the last patch which modifies the stuff > already published in the uapi header, in the hope that no userspace > applications have started relying on it; I'm ok dropping it if this is > unacceptable. Roman, First, let me thank you. Broadly, this patch-set can be broken into 1. Moving existing definitions around - (to make it possible to share these between Hyper-V guest drivers and KVM) 2. Cleanup of the existing code in the VMBUS driver (under drivers/hv). To the extent possible, I want to take all non-KVM code through Greg's tree. We can then modify the KVM code to use these common definitions. Currently, I too am working on restructuring VMBUS driver code to fully isolate all x86 dependencies. I can work with you on integration as I too am moving things around. Regards, K. Y > > Roman Kagan (15): > hyperv: consolidate TSC ref page definitions > hyperv: uapi-fy synic event flags definitions > hyperv: use standard bitops > hyperv: define VMBus message type > hyperv: GFP_ATOMIC -> GFP_KERNEL > hyperv: avoid unnecessary vmalloc > hyperv: dedup cpuid definitions > hyperv: dedup crash msr related definitions > hyperv: unify Hyper-V msr definitions > hyperv: uapi-fy PostMessage and SignalEvent hypercall structures > hyperv: uapi-fy monitored notification structures > hyperv: move VMBus connection ids to uapi > hyperv: move function close to its only callsite > hyperv_vmbus: drop unused definitions > hyperv: redefine hv_message without bitfields > > arch/x86/include/asm/kvm_host.h | 2 +- > arch/x86/include/uapi/asm/hyperv.h | 101 +++++++--- > drivers/hv/hyperv_vmbus.h | 399 +------------------------------------ > include/linux/hyperv.h | 24 +-- > arch/x86/kvm/hyperv.c | 14 +- > drivers/hv/channel.c | 8 +- > drivers/hv/channel_mgmt.c | 30 +-- > drivers/hv/connection.c | 65 ++---- > drivers/hv/hv.c | 300 +++++++++++++--------------- > drivers/hv/vmbus_drv.c | 67 +++---- > 10 files changed, 288 insertions(+), 722 deletions(-) > > -- > 2.9.3