linux-kernel - RE: [EXTERNAL] Re: [PATCH v6 0/2] PCI: hv: Hyper-V vPCI for arm64

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <BN8PR21MB1140A6EFABF87C0FBAA0606AC0789@BN8PR21MB1140.namprd21.prod.outlook.com>
Date:   Fri, 17 Dec 2021 18:49:11 +0000
From:   Sunil Muthuswamy <sunilmut@...rosoft.com>
To:     Bjorn Helgaas <helgaas@...nel.org>,
        Sunil Muthuswamy <sunilmut@...ux.microsoft.com>
CC:     KY Srinivasan <kys@...rosoft.com>,
        Haiyang Zhang <haiyangz@...rosoft.com>,
        Stephen Hemminger <sthemmin@...rosoft.com>,
        "wei.liu@...nel.org" <wei.liu@...nel.org>,
        "maz@...nel.org" <maz@...nel.org>,
        Dexuan Cui <decui@...rosoft.com>,
        "tglx@...utronix.de" <tglx@...utronix.de>,
        "mingo@...hat.com" <mingo@...hat.com>,
        "bp@...en8.de" <bp@...en8.de>, "hpa@...or.com" <hpa@...or.com>,
        "lorenzo.pieralisi@....com" <lorenzo.pieralisi@....com>,
        "robh@...nel.org" <robh@...nel.org>, "kw@...ux.com" <kw@...ux.com>,
        "bhelgaas@...gle.com" <bhelgaas@...gle.com>,
        "arnd@...db.de" <arnd@...db.de>, "x86@...nel.org" <x86@...nel.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        "linux-hyperv@...r.kernel.org" <linux-hyperv@...r.kernel.org>,
        "linux-pci@...r.kernel.org" <linux-pci@...r.kernel.org>,
        "linux-arch@...r.kernel.org" <linux-arch@...r.kernel.org>
Subject: RE: [EXTERNAL] Re: [PATCH v6 0/2] PCI: hv: Hyper-V vPCI for arm64

On Wednesday, December 15, 2021 8:35 AM,
Bjorn Helgaas <helgaas@...nel.org> wrote:

>
> On Thu, Nov 18, 2021 at 12:51:28AM -0800, Sunil Muthuswamy wrote:
> 
> > Sunil Muthuswamy (2):
> >   PCI: hv: Make the code arch neutral by adding arch specific interfaces
> >   arm64: PCI: hv: Add support for Hyper-V vPCI
> 
> Both patches are primarily to drivers/pci/controller/pci-hyperv.c, so
> why do the subject lines look so different?
> 
> Instead of making up a new format from scratch, look at the previous
> history and copy it:
> 
>   $ git log --oneline drivers/pci/controller/pci-hyperv.c
>   f18312084300 ("PCI: hv: Remove unnecessary use of %hx")
>   41608b64b10b ("PCI: hv: Fix sleep while in non-sleep context when removing
> child devices from the bus")
>   88f94c7f8f40 ("PCI: hv: Turn on the host bridge probing on ARM64")
>   9e7f9178ab49 ("PCI: hv: Set up MSI domain at bridge probing time")
>   38c0d266dc80 ("PCI: hv: Set ->domain_nr of pci_host_bridge at probing time")
>   418cb6c8e051 ("PCI: hv: Generify PCI probing")
>   8f6a6b3c50ce ("PCI: hv: Support for create interrupt v3")
>   7d815f4afa87 ("PCI: hv: Add check for hyperv_initialized in init_hv_pci_drv()")
>   326dc2e1e59a ("PCI: hv: Remove bus device removal unused
> refcount/functions")
>   ...
> 
> The second patch adds arm64 support, so it *should* mention arm64, but
> it can be something like this:
> 
>   PCI: hv: Add arm64 Hyper-V vPCI support
> 
> >  arch/arm64/include/asm/hyperv-tlfs.h |   9 +
> >  arch/x86/include/asm/hyperv-tlfs.h   |  33 ++++
> >  arch/x86/include/asm/mshyperv.h      |   7 -
> >  drivers/pci/Kconfig                  |   2 +-
> >  drivers/pci/controller/Kconfig       |   2 +-
> >  drivers/pci/controller/pci-hyperv.c  | 281 ++++++++++++++++++++++++---
> >  include/asm-generic/hyperv-tlfs.h    |  33 ----
> >  7 files changed, 300 insertions(+), 67 deletions(-)

Thanks. If I am reading your feedback above correctly, the only correction
you are suggesting is to rename the subject line of the second patch to fix
the format. Otherwise, the subject lines of the patches describe what they
do. I will address this in v7.

- Sunil