Re: [PATCH 1/1] PCI/bwctrl: Remove also pcie_bwctrl_lbms_rwsem

[Krzysztof Wilczy??ski <kwilczynski@kernel.org>]

Hello,

[...]
> > Done.  Squashed with the first commit from Ilpo, see:
> > 
> >   https://git.kernel.org/pub/scm/linux/kernel/git/pci/pci.git/commit/?h=bwctrl&id=2389d8dc38fee18176c49e9c4804f5ecc55807fa
> 
> Awesome, thank you!
> 
> > Let me know if there is anything else needed.
> 
> Actually, two small things:
> 
> - That patch on the pci/bwctrl topic branch is still marked "New"
>   in patchwork, even though it's been applied:
>   https://patchwork.kernel.org/project/linux-pci/patch/20250422115548.1483-1-ilpo.jarvinen@linux.intel.com/
> 
> - Version 1 of the same patch is likewise marked "New", even though
>   it's been superseded:
>   https://patchwork.kernel.org/project/linux-pci/patch/20250417124633.11470-1-ilpo.jarvinen@linux.intel.com/
> 
> Unfortunately I can't update it myself because I'm not the submitter.
> (Ilpo could do it if he has a patchwork.kernel.org account.)

I had a look and I think Ilpo beat me to it.  Thank you Ilpo!

Also, thank you for chasing after this.  Much appreciated.  Good to know
people use Patchwork to track what is up with our tree. :)

> Something unrelated (only if you feel like doing it):
> 
> On the pci/enumeration branch, Bjorn queued up a revert which was
> waiting to be ack'ed by AMD IOMMU maintainers:
> https://lore.kernel.org/r/20250425163259.GA546441@bhelgaas/
> 
> In the meantime the ack has arrived:
> https://lore.kernel.org/r/aCLv7cN_s1Z4abEl@8bytes.org/
> 
> So the remaining housekeeping items are:
> 
> - Add Jörg's Acked-by to commit e86c7278eba8 on pci/enumeration
> - Remove the "XXX" marker from the subject line
> - Remove "Needs AMD IOMMU ack" from the commit message

Done.  Have a look to make sure that things look correct now:

  https://git.kernel.org/pub/scm/linux/kernel/git/pci/pci.git/commit/?h=enumeration&id=3be5fa236649da6404f1bca1491bf02d4b0d5cce

> Again, only if you feel like doing it.  No urgency.

No problem!  Any time. :)

Thank you!

	Krzysztof