From 218271adca2a3033d6ac9e3a71dcf81d5543afc6 Mon Sep 17 00:00:00 2001 From: Leon Romanovsky Date: Sat, 22 Apr 2017 17:28:49 +0300 Subject: [PATCH] Ib/core: Mark local uverbs_std_types functions to be static Functions declared in uverbs_std_types.c are local to that file, but they lack static declarations. This produces a lot of sparse warnings, like the one below: drivers/infiniband/core/uverbs_std_types.c:41:5: warning: symbol 'uverbs_free_ah' was not declared. Should it be static? So mark them as static. CC: Matan Barak Signed-off-by: Leon Romanovsky Signed-off-by: Doug Ledford --- drivers/infiniband/core/uverbs_std_types.c | 48 +++++++++++++++--------------- 1 file changed, 24 insertions(+), 24 deletions(-) diff --git a/drivers/infiniband/core/uverbs_std_types.c b/drivers/infiniband/core/uverbs_std_types.c index e3338b19d6a2..ad3caad40945 100644 --- a/drivers/infiniband/core/uverbs_std_types.c +++ b/drivers/infiniband/core/uverbs_std_types.c @@ -38,26 +38,26 @@ #include "rdma_core.h" #include "uverbs.h" -int uverbs_free_ah(struct ib_uobject *uobject, - enum rdma_remove_reason why) +static int uverbs_free_ah(struct ib_uobject *uobject, + enum rdma_remove_reason why) { return ib_destroy_ah((struct ib_ah *)uobject->object); } -int uverbs_free_flow(struct ib_uobject *uobject, - enum rdma_remove_reason why) +static int uverbs_free_flow(struct ib_uobject *uobject, + enum rdma_remove_reason why) { return ib_destroy_flow((struct ib_flow *)uobject->object); } -int uverbs_free_mw(struct ib_uobject *uobject, - enum rdma_remove_reason why) +static int uverbs_free_mw(struct ib_uobject *uobject, + enum rdma_remove_reason why) { return uverbs_dealloc_mw((struct ib_mw *)uobject->object); } -int uverbs_free_qp(struct ib_uobject *uobject, - enum rdma_remove_reason why) +static int uverbs_free_qp(struct ib_uobject *uobject, + enum rdma_remove_reason why) { struct ib_qp *qp = uobject->object; struct ib_uqp_object *uqp = @@ -82,8 +82,8 @@ int uverbs_free_qp(struct ib_uobject *uobject, return ret; } -int uverbs_free_rwq_ind_tbl(struct ib_uobject *uobject, - enum rdma_remove_reason why) +static int uverbs_free_rwq_ind_tbl(struct ib_uobject *uobject, + enum rdma_remove_reason why) { struct ib_rwq_ind_table *rwq_ind_tbl = uobject->object; struct ib_wq **ind_tbl = rwq_ind_tbl->ind_tbl; @@ -95,8 +95,8 @@ int uverbs_free_rwq_ind_tbl(struct ib_uobject *uobject, return ret; } -int uverbs_free_wq(struct ib_uobject *uobject, - enum rdma_remove_reason why) +static int uverbs_free_wq(struct ib_uobject *uobject, + enum rdma_remove_reason why) { struct ib_wq *wq = uobject->object; struct ib_uwq_object *uwq = @@ -109,8 +109,8 @@ int uverbs_free_wq(struct ib_uobject *uobject, return ret; } -int uverbs_free_srq(struct ib_uobject *uobject, - enum rdma_remove_reason why) +static int uverbs_free_srq(struct ib_uobject *uobject, + enum rdma_remove_reason why) { struct ib_srq *srq = uobject->object; struct ib_uevent_object *uevent = @@ -134,8 +134,8 @@ int uverbs_free_srq(struct ib_uobject *uobject, return ret; } -int uverbs_free_cq(struct ib_uobject *uobject, - enum rdma_remove_reason why) +static int uverbs_free_cq(struct ib_uobject *uobject, + enum rdma_remove_reason why) { struct ib_cq *cq = uobject->object; struct ib_uverbs_event_queue *ev_queue = cq->cq_context; @@ -153,14 +153,14 @@ int uverbs_free_cq(struct ib_uobject *uobject, return ret; } -int uverbs_free_mr(struct ib_uobject *uobject, - enum rdma_remove_reason why) +static int uverbs_free_mr(struct ib_uobject *uobject, + enum rdma_remove_reason why) { return ib_dereg_mr((struct ib_mr *)uobject->object); } -int uverbs_free_xrcd(struct ib_uobject *uobject, - enum rdma_remove_reason why) +static int uverbs_free_xrcd(struct ib_uobject *uobject, + enum rdma_remove_reason why) { struct ib_xrcd *xrcd = uobject->object; struct ib_uxrcd_object *uxrcd = @@ -178,8 +178,8 @@ int uverbs_free_xrcd(struct ib_uobject *uobject, return ret; } -int uverbs_free_pd(struct ib_uobject *uobject, - enum rdma_remove_reason why) +static int uverbs_free_pd(struct ib_uobject *uobject, + enum rdma_remove_reason why) { struct ib_pd *pd = uobject->object; @@ -190,8 +190,8 @@ int uverbs_free_pd(struct ib_uobject *uobject, return 0; } -int uverbs_hot_unplug_completion_event_file(struct ib_uobject_file *uobj_file, - enum rdma_remove_reason why) +static int uverbs_hot_unplug_completion_event_file(struct ib_uobject_file *uobj_file, + enum rdma_remove_reason why) { struct ib_uverbs_completion_event_file *comp_event_file = container_of(uobj_file, struct ib_uverbs_completion_event_file, -- 2.11.0