From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 9310ED5E13F for ; Fri, 8 Nov 2024 06:02:20 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:Content-Transfer-Encoding: Content-Type:In-Reply-To:From:References:Cc:To:Subject:MIME-Version:Date: Message-ID:Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=JfGAY+SgJN3XN/2DmcBwbwpAyE9Jg5zKh8nkpRpQOEw=; b=reeXdshLpjwkYI8ocANb+QW0K6 zV1x0tiXq+pvLjVAePCZR2cg1holbeCOvGFZG/MRUanzfXwieKUi2wmoxgUsD/JNlnUcg5g4Cbzwm W0i7XcLnUrIgzp0OqleCp0ewt6SORtM31Za+L+MZze/qurrVhKkdd6fq8k/BRAv5iRNYGP9My2yuw Qtnh8gHMgB6AqVyIDe+E3txaxUlk/KtjQsIoNG3bROZorPfh8puaVApztTTTZaXTfdgAoisLpm/03 fImwMCu5blj43f5CAI1cGNTqvUXz0SaRK6VFnwASTRXgCVFsa+4LrW4OShn4Mq8FKqhtzgrnnasMc 7pa1x8Cw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98 #2 (Red Hat Linux)) id 1t9I4P-00000009Rym-0LkK; Fri, 08 Nov 2024 06:02:09 +0000 Received: from linux.microsoft.com ([13.77.154.182]) by bombadil.infradead.org with esmtp (Exim 4.98 #2 (Red Hat Linux)) id 1t9I2d-00000009RiC-0S3d for linux-arm-kernel@lists.infradead.org; Fri, 08 Nov 2024 06:00:20 +0000 Received: from [192.168.0.103] (unknown [49.205.249.137]) by linux.microsoft.com (Postfix) with ESMTPSA id 47AD6212D516; Thu, 7 Nov 2024 22:00:02 -0800 (PST) DKIM-Filter: OpenDKIM Filter v2.11.0 linux.microsoft.com 47AD6212D516 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linux.microsoft.com; s=default; t=1731045617; bh=JfGAY+SgJN3XN/2DmcBwbwpAyE9Jg5zKh8nkpRpQOEw=; h=Date:Subject:To:Cc:References:From:In-Reply-To:From; b=GaPry9vZBpJ4Csbyy+XjXwBbb5PXQIRZpl+bL8WerrHnqdLQYdxQhcAh9bNdkoiwM 0rtOyqxosSzCXDs5w7d73GHL8PjRX14neKuGkeHWuPXg+GnCsZEBRD6wNTJ4ThdvID VDRS5dHBu8xaBb7Hc7Vqtd1vOwq2eibskIz/48fs= Message-ID: Date: Fri, 8 Nov 2024 11:29:58 +0530 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v2 3/4] hyperv: Add new Hyper-V headers in include/hyperv To: Nuno Das Neves , linux-hyperv@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, kvm@vger.kernel.org, iommu@lists.linux.dev, netdev@vger.kernel.org, linux-pci@vger.kernel.org, linux-arch@vger.kernel.org, virtualization@lists.linux.dev Cc: kys@microsoft.com, haiyangz@microsoft.com, wei.liu@kernel.org, mhklinux@outlook.com, decui@microsoft.com, catalin.marinas@arm.com, will@kernel.org, luto@kernel.org, tglx@linutronix.de, mingo@redhat.com, bp@alien8.de, dave.hansen@linux.intel.com, x86@kernel.org, hpa@zytor.com, seanjc@google.com, pbonzini@redhat.com, peterz@infradead.org, daniel.lezcano@linaro.org, joro@8bytes.org, robin.murphy@arm.com, davem@davemloft.net, edumazet@google.com, kuba@kernel.org, pabeni@redhat.com, lpieralisi@kernel.org, kw@linux.com, robh@kernel.org, bhelgaas@google.com, arnd@arndb.de, sgarzare@redhat.com, jinankjain@linux.microsoft.com, muminulrussell@gmail.com, skinsburskii@linux.microsoft.com, mukeshrathor@microsoft.com, vkuznets@redhat.com, ssengar@linux.microsoft.com, apais@linux.microsoft.com References: <1731018746-25914-1-git-send-email-nunodasneves@linux.microsoft.com> <1731018746-25914-4-git-send-email-nunodasneves@linux.microsoft.com> Content-Language: en-US From: Naman Jain In-Reply-To: <1731018746-25914-4-git-send-email-nunodasneves@linux.microsoft.com> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20241107_220019_241294_91F7E36B X-CRM114-Status: GOOD ( 19.87 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On 11/8/2024 4:02 AM, Nuno Das Neves wrote: > These headers contain definitions for regular Hyper-V guests (as in > hyperv-tlfs.h), as well as interfaces for more privileged guests like > Dom0. > > These files are derived from headers exported from Hyper-V, rather than > being derived from the TLFS document. (Although, to preserve > compatibility with existing Linux code, some definitions are copied > directly from hyperv-tlfs.h too). > > The new files follow a naming convention according to their original > use: > - hdk "host development kit" > - gdk "guest development kit" > With postfix "_mini" implying userspace-only headers, and "_ext" for > extended hypercalls. Naming convention for mini (which may have come from HyperV code) is a bit odd TBH. May be it has more to it than what is mentioned here or what I know. If more information helps, or this can be changed, please see. > > These names should be considered a rough guide only - since there are > many places already where both host and guest code are in the same > place, hvhdk.h (which includes everything) can be used most of the time. > > The original names are kept intact primarily to keep the provenance of > exactly where they came from in Hyper-V code, which is helpful for > manual maintenance and extension of these definitions. Microsoft > maintainers importing new definitions should take care to put them in > the right file. > > Note also that the files contain both arm64 and x86_64 code guarded by > \#ifdefs, which is how the definitions originally appear in Hyper-V. > Keeping this convention from Hyper-V code is another tactic for > simplying the process of importing new definitions. > > These headers are a step toward importing headers directly from Hyper-V > in the future, similar to Xen public files in include/xen/interface/. > > Signed-off-by: Nuno Das Neves While I understand the motivation behind this series that this is going to ease out the process of updating the header files with respect to HyperV, I think, we will need to pay attention to what we are bringing in with these headers, whether there are any users of it or not, and make sure that TLFS document is updated regularly, to avoid having bunch of code with no information of it. The code comments in these files are one step forward towards that. And from your cover letter it seems that some changes which actually make use of these additional interfaces are underway, so things will make more sense later. For now, this looks good to me. Regards, Naman