On Sun, Jan 06, 2019 at 11:23:03AM +0200, Gal Pressman wrote:
> Lack of mandatory verbs no longer fail device registration, the device
> will be marked as a non-kverbs provider.
>
> Signed-off-by: Gal Pressman <galpress@xxxxxxxxxx>
> ---
> drivers/infiniband/hw/usnic/usnic_ib_main.c | 7 ----
> drivers/infiniband/hw/usnic/usnic_ib_verbs.c | 53 ----------------------------
> drivers/infiniband/hw/usnic/usnic_ib_verbs.h | 15 --------
> 3 files changed, 75 deletions(-)
You need to cc the driver maintainers on patches like this. They
should probably ack this..
> diff --git a/drivers/infiniband/hw/usnic/usnic_ib_main.c b/drivers/infiniband/hw/usnic/usnic_ib_main.c
> index b2323a52a0dd..cc68f3d757eb 100644
> --- a/drivers/infiniband/hw/usnic/usnic_ib_main.c
> +++ b/drivers/infiniband/hw/usnic/usnic_ib_main.c
> @@ -333,32 +333,25 @@ static void usnic_get_dev_fw_str(struct ib_device *device, char *str)
> static const struct ib_device_ops usnic_dev_ops = {
> .alloc_pd = usnic_ib_alloc_pd,
> .alloc_ucontext = usnic_ib_alloc_ucontext,
> - .create_ah = usnic_ib_create_ah,
> .create_cq = usnic_ib_create_cq,
> .create_qp = usnic_ib_create_qp,
> .dealloc_pd = usnic_ib_dealloc_pd,
> .dealloc_ucontext = usnic_ib_dealloc_ucontext,
> .dereg_mr = usnic_ib_dereg_mr,
> - .destroy_ah = usnic_ib_destroy_ah,
> .destroy_cq = usnic_ib_destroy_cq,
> .destroy_qp = usnic_ib_destroy_qp,
> .get_dev_fw_str = usnic_get_dev_fw_str,
> - .get_dma_mr = usnic_ib_get_dma_mr,
> .get_link_layer = usnic_ib_port_link_layer,
> .get_netdev = usnic_get_netdev,
> .get_port_immutable = usnic_port_immutable,
> .mmap = usnic_ib_mmap,
> .modify_qp = usnic_ib_modify_qp,
> - .poll_cq = usnic_ib_poll_cq,
> - .post_recv = usnic_ib_post_recv,
> - .post_send = usnic_ib_post_send,
> .query_device = usnic_ib_query_device,
> .query_gid = usnic_ib_query_gid,
> .query_pkey = usnic_ib_query_pkey,
> .query_port = usnic_ib_query_port,
> .query_qp = usnic_ib_query_qp,
> .reg_user_mr = usnic_ib_reg_mr,
> - .req_notify_cq = usnic_ib_req_notify_cq,
> };
Did you audit that uverbs doesn't crash if these are made NULL,
including ioctl?
Jason
