On 10-Jan-19 12:59, Leon Romanovsky wrote: >> @@ -624,7 +627,8 @@ int ib_register_device(struct ib_device *device, const char *name, >> >> list_for_each_entry(client, &client_list, list) >> if (!add_client_context(device, client) && client->add) >> - client->add(device); >> + if (device->kverbs_provider || client->no_kverbs_req) > > This check should be in add_client_context() to avoid duplication. Thanks, will do.