Great, thanks
-----Original Message----- From: Jason Gunthorpe <j...@ziepe.ca> Sent: Thursday, December 20, 2018 11:02 PM To: Leon Romanovsky <l...@kernel.org> Cc: Doug Ledford <dledf...@redhat.com>; Yishai Hadas <yish...@mellanox.com>; RDMA mailing list <linux-r...@vger.kernel.org>; Achiad Shochat <ach...@mellanox.com>; Saeed Mahameed <sae...@mellanox.com>; linux-netdev <netdev@vger.kernel.org>; Leon Romanovsky <leo...@mellanox.com> Subject: Re: [PATCH mlx5-next] IB/mlx5: Prevent allocating UMEM and UCTX as some general object On Wed, Dec 19, 2018 at 04:28:15PM +0200, Leon Romanovsky wrote: > From: Yishai Hadas <yish...@mellanox.com> > > The driver needs to prevent a user space application to create a UMEM > and UCTX via the general object command. > > The UMEM must go through the kernel UMEM_REG method to prevent the > user from setting physical addresses by himself. The UCTX is some > internal kernel object and shouldn't be exposed. > > As of not being any more part of the general object the caps bits were > moved to be some log_xxx indication in the general HCA caps, 0 means > not supported. > > The firmware code was adapted as well to match the above. > > Signed-off-by: Yishai Hadas <yish...@mellanox.com> > Reviewed-by: Achiad Shochat <ach...@mellanox.com> > Signed-off-by: Leon Romanovsky <leo...@mellanox.com> > --- > drivers/infiniband/hw/mlx5/devx.c | 34 ++++++++--------- > drivers/infiniband/hw/mlx5/main.c | 3 +- > include/linux/mlx5/mlx5_ifc.h | 62 +++++++++++++++++++++---------- > 3 files changed, 58 insertions(+), 41 deletions(-) Applied to for-next Thanks, Jason
