[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <98fa31e7-db86-35f0-a71c-a1ebf27f93f0@linux.intel.com>
Date: Thu, 15 May 2025 15:21:39 +0300 (EEST)
From: Ilpo Järvinen <ilpo.jarvinen@...ux.intel.com>
To: Lukas Wunner <lukas@...ner.de>
cc: Krzysztof Wilczy??ski <kwilczynski@...nel.org>,
Bjorn Helgaas <bhelgaas@...gle.com>,
"Paul E. McKenney" <paulmck@...nel.org>, linux-pci@...r.kernel.org,
LKML <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH 1/1] PCI/bwctrl: Remove also pcie_bwctrl_lbms_rwsem
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.
Powered by blists - more mailing lists