linux-kernel - Re: [PATCH v7] Revert "PCI: hv: Use device serial number as PCI domain"

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]

Date:   Thu, 12 Apr 2018 17:13:34 +0100
From:   Lorenzo Pieralisi <lorenzo.pieralisi@....com>
To:     Sridhar Pitchai <Sridhar.Pitchai@...rosoft.com>
Cc:     Haiyang Zhang <haiyangz@...rosoft.com>,
        Bjorn Helgaas <helgaas@...nel.org>,
        Dexuan Cui <decui@...rosoft.com>,
        Stephen Hemminger <sthemmin@...rosoft.com>,
        Jake Oshins <jakeo@...rosoft.com>,
        KY Srinivasan <kys@...rosoft.com>,
        "Michael Kelley (EOSG)" <Michael.H.Kelley@...rosoft.com>,
        "devel@...uxdriverproject.org" <devel@...uxdriverproject.org>,
        "linux-pci@...r.kernel.org" <linux-pci@...r.kernel.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH v7] Revert "PCI: hv: Use device serial number as PCI
 domain"

On Thu, Apr 12, 2018 at 03:56:42PM +0000, Sridhar Pitchai wrote:
>     > 
>     >     >> I am still not happy with this patch.
>     >     >>
>     >     >> -  You do not explain at all the dependency on commit 0c195567a8f6 and
>     >     >>    you should because that's fundamental, if that patch is not present
>     >     >>    this revert breaks the kernel as per previous discussions[1].
>     >     >> -  You are sending this patch to all stable kernels that contain the
>     >     >>    commit you are fixing - some that may not contain the commit above
>     >     >>    (that was merged in v4.14), you are breaking those kernels, if not
>     >     >>    explain me why please
>     > 
>     >     >If there's a dependency on 0c195567a8f6, I totally agree that
>     >     >needs to be cleared up.  I was assuming that turned out to be
>     >     >irrelevant.
>     > That is right. There is no dependency on 0c195567a8f6. We just need to revert
>     > 4a9b0933bdfc.
>     
>     > This patch should only be applied to later versions after  0c195567a8f6" (transparent VF).
>     > Otherwise it causes long & random names of VF NICs for bonding. That will make bonding
>     > config difficult, especially for auto config.
>     
>     >Thanks,
>     >- Haiyang
> Ok. I will update the patch, as Lorenzo suggested. 

Please update your email client quoting policy so that it does not add
unnecessary indentation spaces, for long threads this gets messy.

If you send a patch to a specific stable version you have to test it
first thing on that stable version before adding a stable tag, reverting
a patch from stable kernels is the last thing we want to do - that's the
reason why I keep complaining.

Thanks,
Lorenzo