> -----Original Message----- > From: Lorenzo Pieralisi <lorenzo.pieralisi@xxxxxxx> > Sent: Wednesday, August 21, 2019 5:03 AM > To: Stephen Rothwell <sfr@xxxxxxxxxxxxxxxx> > Cc: Bjorn Helgaas <bhelgaas@xxxxxxxxxx>; Linux Next Mailing List <linux- > next@xxxxxxxxxxxxxxx>; Linux Kernel Mailing List <linux- > kernel@xxxxxxxxxxxxxxx>; Sasha Levin <sashal@xxxxxxxxxx>; Haiyang Zhang > <haiyangz@xxxxxxxxxxxxx> > Subject: Re: linux-next: Signed-off-by missing for commit in the pci tree > > On Wed, Aug 21, 2019 at 07:19:39AM +1000, Stephen Rothwell wrote: > > Hi all, > > > > Commit > > > > c4a29fbba415 ("PCI: hv: Use bytes 4 and 5 from instance ID as the PCI > domain numbers") > > > > is missing a Signed-off-by from its committer. > > > > Also, all the tags should be kept together, please. > > Fixed it up in my pci/hv branch, apologies. > > Lorenzo Hi thanks for fixing this, but I found the following commit now has the Subject line and commit message same as a previous patch. https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/commit/drivers?id=87b20a08dcc265959f5c59f18603ea0487fe609b The correct message for the patch should be the msg below. Any possibility to fix it again? Thanks. - Haiyang Subject: PCI: hv: Use bytes 4 and 5 from instance ID as the PCI domain numbers As recommended by Azure host team, the bytes 4, 5 have more uniqueness (info entropy) than bytes 8, 9. So now we use bytes 4, 5 as the PCI domain numbers. On older hosts, bytes 4, 5 can also be used -- no backward compatibility issues here. The chance of collision is greatly reduced. In the rare cases of collision, the driver code detects and finds another number that is not in use. Suggested-by: Michael Kelley <mikelley@xxxxxxxxxxxxx> Signed-off-by: Haiyang Zhang <haiyangz@xxxxxxxxxxxxx> Acked-by: Sasha Levin <sashal@xxxxxxxxxx> Signed-off-by: Lorenzo Pieralisi <lorenzo.pieralisi@xxxxxxx>