All of lore.kernel.org
 help / color / mirror / Atom feed
From: Leon Romanovsky <leon@kernel.org>
To: "Dr. David Alan Gilbert" <linux@treblig.org>
Cc: Allison Henderson <allison.henderson@oracle.com>,
	"edumazet@google.com" <edumazet@google.com>,
	"davem@davemloft.net" <davem@davemloft.net>,
	"pabeni@redhat.com" <pabeni@redhat.com>,
	"kuba@kernel.org" <kuba@kernel.org>,
	"rds-devel@oss.oracle.com" <rds-devel@oss.oracle.com>,
	"netdev@vger.kernel.org" <netdev@vger.kernel.org>,
	"linux-rdma@vger.kernel.org" <linux-rdma@vger.kernel.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH] RDMA/rds: remove unused struct 'rds_ib_dereg_odp_mr'
Date: Sun, 21 Jul 2024 10:05:57 +0300	[thread overview]
Message-ID: <20240721070557.GE1265781@unreal> (raw)
In-Reply-To: <ZpsEof3hxKGQBmqF@gallifrey>

On Sat, Jul 20, 2024 at 12:28:17AM +0000, Dr. David Alan Gilbert wrote:
> * Allison Henderson (allison.henderson@oracle.com) wrote:
> > On Sat, 2024-06-01 at 00:33 +0100, linux@treblig.org wrote:
> > > From: "Dr. David Alan Gilbert" <linux@treblig.org>
> > > 
> > > 'rds_ib_dereg_odp_mr' has been unused since the original
> > > commit 2eafa1746f17 ("net/rds: Handle ODP mr
> > > registration/unregistration").
> > > 
> > > Remove it.
> > > 
> > > Signed-off-by: Dr. David Alan Gilbert <linux@treblig.org>
> > 
> > This patch looks fine to me, the struct is indeed unused at this point.
> > Thanks for the clean up!
> > 
> > Reviewed-by: Allison Henderson <allison.henderson@oracle.com>
> 
> Hi,
>   Does anyone know who might pick this one up - I don't think
> it's in -next yet?

1. We are in merge window and this patch is not a bug fix, so it should
   wait until the next merge window.
2. Title should be net/rds ... and not RDMA/rds ...
3. netdev is closed right now, so it should be resubmitted after next merge
   window ends.

Thanks

  reply	other threads:[~2024-07-21  7:06 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-05-31 23:33 [PATCH] RDMA/rds: remove unused struct 'rds_ib_dereg_odp_mr' linux
2024-06-04 17:08 ` Simon Horman
2024-06-04 17:32   ` Dr. David Alan Gilbert
2024-06-04 17:51 ` Allison Henderson
2024-07-20  0:28   ` Dr. David Alan Gilbert
2024-07-21  7:05     ` Leon Romanovsky [this message]
2024-07-21 11:02       ` Dr. David Alan Gilbert
2024-07-21 12:06         ` Leon Romanovsky
2024-07-29 21:01           ` Dr. David Alan Gilbert
2024-07-21  6:41 ` Zhu Yanjun
2024-07-21 10:59   ` Dr. David Alan Gilbert

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=20240721070557.GE1265781@unreal \
    --to=leon@kernel.org \
    --cc=allison.henderson@oracle.com \
    --cc=davem@davemloft.net \
    --cc=edumazet@google.com \
    --cc=kuba@kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-rdma@vger.kernel.org \
    --cc=linux@treblig.org \
    --cc=netdev@vger.kernel.org \
    --cc=pabeni@redhat.com \
    --cc=rds-devel@oss.oracle.com \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.