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

From: Lorenzo Pieralisi <lorenzo.pieralisi@arm.com>
To: Sridhar Pitchai <Sridhar.Pitchai@microsoft.com>
Cc: Stephen Hemminger <sthemmin@microsoft.com>,
	"linux-pci@vger.kernel.org" <linux-pci@vger.kernel.org>,
	Haiyang Zhang <haiyangz@microsoft.com>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	"Michael Kelley \(EOSG\)" <Michael.H.Kelley@microsoft.com>,
	Bjorn Helgaas <helgaas@kernel.org>,
	Jake Oshins <jakeo@microsoft.com>,
	"devel@linuxdriverproject.org" <devel@linuxdriverproject.org>
Subject: Re: [PATCH v7] Revert "PCI: hv: Use device serial number as PCI domain"
Date: Thu, 12 Apr 2018 17:13:34 +0100	[thread overview]
Message-ID: <20180412161334.GA5803@e107981-ln.cambridge.arm.com> (raw)
In-Reply-To: <F8BC8B81-D3A4-40F7-95F4-CAF77A7917B7@microsoft.com>

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
_______________________________________________
devel mailing list
devel@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel