[0/2] IB/ipoib fixes

Message

Daniel Vacek Dec. 11, 2023, 1:04 p.m. UTC

The first patch (hopefully) fixes a real issue while the second is an
unrelated cleanup. But it shares a context so sending as a series.

Daniel Vacek (2):
  IB/ipoib: Fix mcast list locking
  IB/ipoib: Clean up redundant netif_addr_lock

 drivers/infiniband/ulp/ipoib/ipoib_multicast.c | 9 ++++-----
 1 file changed, 4 insertions(+), 5 deletions(-)

Comments

Leon Romanovsky Dec. 12, 2023, 8:29 a.m. UTC | #1

On Mon, 11 Dec 2023 14:04:23 +0100, Daniel Vacek wrote:
> The first patch (hopefully) fixes a real issue while the second is an
> unrelated cleanup. But it shares a context so sending as a series.
> 
> Daniel Vacek (2):
>   IB/ipoib: Fix mcast list locking
>   IB/ipoib: Clean up redundant netif_addr_lock
> 
> [...]

Applied, thanks!

[1/1] IB/ipoib: Fix mcast list locking
      https://git.kernel.org/rdma/rdma/c/4f973e211b3b1c

Best regards,

Daniel Vacek Dec. 13, 2023, 12:18 p.m. UTC | #2

On Tue, Dec 12, 2023 at 9:29 AM Leon Romanovsky <[e-mail]> wrote:
>
>
> On Mon, 11 Dec 2023 14:04:23 +0100, Daniel Vacek wrote:
> > The first patch (hopefully) fixes a real issue while the second is an
> > unrelated cleanup. But it shares a context so sending as a series.
> >
> > Daniel Vacek (2):
> >   IB/ipoib: Fix mcast list locking
> >   IB/ipoib: Clean up redundant netif_addr_lock
> >
> > [...]
>
> Applied, thanks!

Thank you.

One small detail - I was asked by Yuya to change the "Reported-by" as follows:

---
Reported-by: Yuya Fujita <[e-mail]>
---

Would that be possible? And if yes, could you amend the commit
yourself or do you want me to send a v3?

--nX


> [1/1] IB/ipoib: Fix mcast list locking
>       https://git.kernel.org/rdma/rdma/c/4f973e211b3b1c
>
> Best regards,
> --
> Leon Romanovsky <[e-mail]>
>

Leon Romanovsky Dec. 17, 2023, 12:50 p.m. UTC | #3

On Wed, Dec 13, 2023 at 01:18:26PM +0100, Daniel Vacek wrote:
> On Tue, Dec 12, 2023 at 9:29 AM Leon Romanovsky <[e-mail]> wrote:
> >
> >
> > On Mon, 11 Dec 2023 14:04:23 +0100, Daniel Vacek wrote:
> > > The first patch (hopefully) fixes a real issue while the second is an
> > > unrelated cleanup. But it shares a context so sending as a series.
> > >
> > > Daniel Vacek (2):
> > >   IB/ipoib: Fix mcast list locking
> > >   IB/ipoib: Clean up redundant netif_addr_lock
> > >
> > > [...]
> >
> > Applied, thanks!
> 
> Thank you.
> 
> One small detail - I was asked by Yuya to change the "Reported-by" as follows:
> 
> ---
> Reported-by: Yuya Fujita <[e-mail]>
> ---
> 
> Would that be possible? And if yes, could you amend the commit
> yourself or do you want me to send a v3?

Unfortunately, it is already too late as we promoted my wip branch to be
official rdma/for-next.

Thanks

> 
> --nX
> 
> 
> > [1/1] IB/ipoib: Fix mcast list locking
> >       https://git.kernel.org/rdma/rdma/c/4f973e211b3b1c
> >
> > Best regards,
> > --
> > Leon Romanovsky <[e-mail]>
> >
>