Re: [PATCH rdma-next 0/3] "Remove 9 BUG_ON()s" Cont.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Jun 01, 2018 at 11:24:05AM -0400, Doug Ledford wrote:
> On Tue, 2018-05-29 at 15:14 +0300, Leon Romanovsky wrote:
> > From: Leon Romanovsky <leonro@xxxxxxxxxxxx>
> >
> > Hi,
> >
> > This is supplementary patch set to previously sent "Remove 9 BUG_ON()s"
> > series.
> >
> > Thanks
> >
> > Leon Romanovsky (3):
> >   RDMA/mlx5: Don't check return value of zap_vma_ptes()
> >   RDMA/mlx4: Don't crash machine if zap_vma_ptes() fails
> >   mm: Remove return value of zap_vma_ptes()
> >
> >  drivers/infiniband/hw/mlx4/main.c | 10 ++--------
> >  drivers/infiniband/hw/mlx5/main.c |  5 +----
> >  include/linux/mm.h                |  8 ++++----
> >  mm/memory.c                       |  7 +++----
> >  4 files changed, 10 insertions(+), 20 deletions(-)
> >
> > --
> > 2.14.3
> >
>
> This series was incomplete and would have caused compile errors on
> aarch64.  So, I wrote a patch for hns_roce that updated it as well, then
> included it in this series.  Using git grep when changing a core
> function like zap_vma_ptes is usually a good idea just to catch anything
> from other arches you don't normally see ;-).

Doug,

First of all thank you for taking time to review and to fix. I need to
be less hasty.

Regarding "git grep", I actually did it and hns patch was accepted
after I posted my series.
https://elixir.bootlin.com/linux/v4.17-rc7/ident/zap_vma_ptes

hns patch was accepted at 30/5/18
https://git.kernel.org/pub/scm/linux/kernel/git/rdma/rdma.git/commit/drivers/infiniband/hw/hns/hns_roce_main.c?h=wip/dl-for-next&id=fedc3abe7bd2dcc4c80bcf3cff8708a3908d8219

My patches were posted at 29/5/18
https://patchwork.kernel.org/patch/10435165/

Thanks

>
> --
> Doug Ledford <dledford@xxxxxxxxxx>
>     GPG KeyID: B826A3330E572FDD
>     Key fingerprint = AE6B 1BDA 122B 23B4 265B  1274 B826 A333 0E57 2FDD


Attachment: signature.asc
Description: PGP signature


[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Photo]     [Yosemite News]     [Yosemite Photos]     [Linux Kernel]     [Linux SCSI]     [XFree86]

  Powered by Linux