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

["Ilpo Järvinen" <ilpo.jarvinen@linux.intel.com>]

[-- Attachment #1: Type: text/plain, Size: 2279 bytes --]

On Thu, 15 May 2025, Lukas Wunner wrote:

> On Thu, May 15, 2025 at 05:43:46PM +0900, Krzysztof Wilczy??ski wrote:
> > Done.  Squashed with the first commit from Ilpo, see:

Thanks Krzysztof for handling this, I should have put the note about 
squashing it to the resubmission but I forgot (this time I didn't do 
the diff against the previous version before sending it which I normally 
do).

> >   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'm a pdx86 maintainer so I do have an account, yes. I actually had the 
patchwork page listing my PCI patches already open, but I just hadn't hit 
"update" button yet.

I've done those two changes now.

I'm a bit hesitant to mark "Accepted" state though, I did it this time 
but in general I feel I might be overstepping my authority even if I know 
some patches have been accepted.

> 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
> 
> Again, only if you feel like doing it.  No urgency.


-- 
 i.