[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <44A12092-5DA9-4A3C-ACBC-FF1AACB03BD3@oracle.com>
Date: Thu, 4 Sep 2025 14:22:02 +0000
From: Haakon Bugge <haakon.bugge@...cle.com>
To: Jakub Kicinski <kuba@...nel.org>
CC: Allison Henderson <allison.henderson@...cle.com>,
"David S. Miller"
<davem@...emloft.net>,
Eric Dumazet <edumazet@...gle.com>, Paolo Abeni
<pabeni@...hat.com>,
Simon Horman <horms@...nel.org>,
Santosh Shilimkar
<ssantosh@...nel.org>,
"stable@...r.kernel.org" <stable@...r.kernel.org>,
"netdev@...r.kernel.org" <netdev@...r.kernel.org>,
OFED mailing list
<linux-rdma@...r.kernel.org>,
"rds-devel@....oracle.com"
<rds-devel@....oracle.com>,
"linux-kernel@...r.kernel.org"
<linux-kernel@...r.kernel.org>
Subject: Re: [PATCH net v2] rds: ib: Remove unused extern definition
Hi Jakub,
> On 4 Sep 2025, at 15:55, Jakub Kicinski <kuba@...nel.org> wrote:
>
> On Thu, 4 Sep 2025 13:53:43 +0200 Håkon Bugge wrote:
>> Fixes: 2cb2912d6563 ("RDS: IB: add Fastreg MR (FRMR) detection support")
>> Cc: stable@...r.kernel.org
>
> Fixes: and cc: stable is for bug fixes which we care about so deeply we
> want them backported ASAP. Why do you think removing an unused
> declaration qualifies as such :\
Sorry if I have mis-interpreted the collateral. From [1], I quote:
"A Fixes: tag indicates that the patch fixes an issue in a previous commit." As such, it is an "issue" and I reference the offending commit.
As to "Cc: stable", you're quite right. My bad. You want a v3 or are you (and stable) able to handle it?
Thxs, Håkon
[1] https://www.kernel.org/doc/html/latest/process/submitting-patches.html#using-reported-by-tested-by-reviewed-by-suggested-by-and-fixes
Powered by blists - more mailing lists