From: Greg KH <gregkh@linuxfoundation.org>
To: Haakon Bugge <haakon.bugge@oracle.com>
Cc: Allison Henderson <allison.henderson@oracle.com>,
"David S. Miller" <davem@davemloft.net>,
Eric Dumazet <edumazet@google.com>,
Jakub Kicinski <kuba@kernel.org>, Paolo Abeni <pabeni@redhat.com>,
Simon Horman <horms@kernel.org>,
"stable@vger.kernel.org" <stable@vger.kernel.org>,
"netdev@vger.kernel.org" <netdev@vger.kernel.org>,
OFED mailing list <linux-rdma@vger.kernel.org>,
"rds-devel@oss.oracle.com" <rds-devel@oss.oracle.com>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH net 2/2] rds: ib: Remove unused extern definition
Date: Wed, 3 Sep 2025 18:56:55 +0200 [thread overview]
Message-ID: <2025090314-corporate-yanking-e1b1@gregkh> (raw)
In-Reply-To: <1D680271-B2DF-4CAB-A91D-4577CA6861E8@oracle.com>
On Wed, Sep 03, 2025 at 04:51:01PM +0000, Haakon Bugge wrote:
>
>
> > On 3 Sep 2025, at 18:38, Greg KH <gregkh@linuxfoundation.org> wrote:
> >
> > On Wed, Sep 03, 2025 at 06:31:37PM +0200, Håkon Bugge wrote:
> >> Fixes: 2cb2912d6563 ("RDS: IB: add Fastreg MR (FRMR) detection support")
> >> Signed-off-by: Håkon Bugge <haakon.bugge@oracle.com>
> >> ---
> >> net/rds/ib_mr.h | 1 -
> >> 1 file changed, 1 deletion(-)
> >
> > I know I can't take patches without any changelog text, but maybe other
> > maintainer are more lax :)
>
> You mean the empty commit message? Did pass checkpatch.pl --strict though.
checkpatch is a hint. What would you want to see if you were a reviewer?
next prev parent reply other threads:[~2025-09-03 16:56 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-09-03 16:31 [PATCH net 1/2] rds: ib: Increment i_fastreg_wrs before bailing out Håkon Bugge
2025-09-03 16:31 ` [PATCH net 2/2] rds: ib: Remove unused extern definition Håkon Bugge
2025-09-03 16:38 ` Greg KH
2025-09-03 16:51 ` Haakon Bugge
2025-09-03 16:56 ` Greg KH [this message]
2025-09-03 17:22 ` Haakon Bugge
2025-09-09 10:54 ` [PATCH net 1/2] rds: ib: Increment i_fastreg_wrs before bailing out Paolo Abeni
2025-09-09 14:41 ` Haakon Bugge
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=2025090314-corporate-yanking-e1b1@gregkh \
--to=gregkh@linuxfoundation.org \
--cc=allison.henderson@oracle.com \
--cc=davem@davemloft.net \
--cc=edumazet@google.com \
--cc=haakon.bugge@oracle.com \
--cc=horms@kernel.org \
--cc=kuba@kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-rdma@vger.kernel.org \
--cc=netdev@vger.kernel.org \
--cc=pabeni@redhat.com \
--cc=rds-devel@oss.oracle.com \
--cc=stable@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox