Commit 19ae89d3 authored by NeilBrown's avatar NeilBrown Committed by Greg Kroah-Hartman

staging: lustre: libcfs: remove wi_data from cfs_workitem

In every case, the value passed via wi_data can be determined
from the cfs_workitem pointer using container_of().

So use container_of(), and discard wi_data.
Signed-off-by: default avatarNeilBrown <neilb@suse.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 0aa211e3
...@@ -75,8 +75,6 @@ struct cfs_workitem { ...@@ -75,8 +75,6 @@ struct cfs_workitem {
struct list_head wi_list; struct list_head wi_list;
/** working function */ /** working function */
cfs_wi_action_t wi_action; cfs_wi_action_t wi_action;
/** arg for working function */
void *wi_data;
/** in running */ /** in running */
unsigned short wi_running:1; unsigned short wi_running:1;
/** scheduled */ /** scheduled */
...@@ -84,13 +82,12 @@ struct cfs_workitem { ...@@ -84,13 +82,12 @@ struct cfs_workitem {
}; };
static inline void static inline void
cfs_wi_init(struct cfs_workitem *wi, void *data, cfs_wi_action_t action) cfs_wi_init(struct cfs_workitem *wi, cfs_wi_action_t action)
{ {
INIT_LIST_HEAD(&wi->wi_list); INIT_LIST_HEAD(&wi->wi_list);
wi->wi_running = 0; wi->wi_running = 0;
wi->wi_scheduled = 0; wi->wi_scheduled = 0;
wi->wi_data = data;
wi->wi_action = action; wi->wi_action = action;
} }
......
...@@ -944,7 +944,7 @@ sfw_create_test_rpc(struct sfw_test_unit *tsu, struct lnet_process_id peer, ...@@ -944,7 +944,7 @@ sfw_create_test_rpc(struct sfw_test_unit *tsu, struct lnet_process_id peer,
static int static int
sfw_run_test(struct swi_workitem *wi) sfw_run_test(struct swi_workitem *wi)
{ {
struct sfw_test_unit *tsu = wi->swi_workitem.wi_data; struct sfw_test_unit *tsu = container_of(wi, struct sfw_test_unit, tsu_worker);
struct sfw_test_instance *tsi = tsu->tsu_instance; struct sfw_test_instance *tsi = tsu->tsu_instance;
struct srpc_client_rpc *rpc = NULL; struct srpc_client_rpc *rpc = NULL;
...@@ -1016,7 +1016,7 @@ sfw_run_batch(struct sfw_batch *tsb) ...@@ -1016,7 +1016,7 @@ sfw_run_batch(struct sfw_batch *tsb)
atomic_inc(&tsi->tsi_nactive); atomic_inc(&tsi->tsi_nactive);
tsu->tsu_loop = tsi->tsi_loop; tsu->tsu_loop = tsi->tsi_loop;
wi = &tsu->tsu_worker; wi = &tsu->tsu_worker;
swi_init_workitem(wi, tsu, sfw_run_test, swi_init_workitem(wi, sfw_run_test,
lst_sched_test[lnet_cpt_of_nid(tsu->tsu_dest.nid)]); lst_sched_test[lnet_cpt_of_nid(tsu->tsu_dest.nid)]);
swi_schedule_workitem(wi); swi_schedule_workitem(wi);
} }
......
...@@ -176,7 +176,7 @@ srpc_init_server_rpc(struct srpc_server_rpc *rpc, ...@@ -176,7 +176,7 @@ srpc_init_server_rpc(struct srpc_server_rpc *rpc,
struct srpc_buffer *buffer) struct srpc_buffer *buffer)
{ {
memset(rpc, 0, sizeof(*rpc)); memset(rpc, 0, sizeof(*rpc));
swi_init_workitem(&rpc->srpc_wi, rpc, srpc_handle_rpc, swi_init_workitem(&rpc->srpc_wi, srpc_handle_rpc,
srpc_serv_is_framework(scd->scd_svc) ? srpc_serv_is_framework(scd->scd_svc) ?
lst_sched_serial : lst_sched_test[scd->scd_cpt]); lst_sched_serial : lst_sched_test[scd->scd_cpt]);
...@@ -280,7 +280,7 @@ srpc_service_init(struct srpc_service *svc) ...@@ -280,7 +280,7 @@ srpc_service_init(struct srpc_service *svc)
* NB: don't use lst_sched_serial for adding buffer, * NB: don't use lst_sched_serial for adding buffer,
* see details in srpc_service_add_buffers() * see details in srpc_service_add_buffers()
*/ */
swi_init_workitem(&scd->scd_buf_wi, scd, swi_init_workitem(&scd->scd_buf_wi,
srpc_add_buffer, lst_sched_test[i]); srpc_add_buffer, lst_sched_test[i]);
if (i && srpc_serv_is_framework(svc)) { if (i && srpc_serv_is_framework(svc)) {
...@@ -517,7 +517,7 @@ __must_hold(&scd->scd_lock) ...@@ -517,7 +517,7 @@ __must_hold(&scd->scd_lock)
int int
srpc_add_buffer(struct swi_workitem *wi) srpc_add_buffer(struct swi_workitem *wi)
{ {
struct srpc_service_cd *scd = wi->swi_workitem.wi_data; struct srpc_service_cd *scd = container_of(wi, struct srpc_service_cd, scd_buf_wi);
struct srpc_buffer *buf; struct srpc_buffer *buf;
int rc = 0; int rc = 0;
...@@ -968,7 +968,7 @@ srpc_server_rpc_done(struct srpc_server_rpc *rpc, int status) ...@@ -968,7 +968,7 @@ srpc_server_rpc_done(struct srpc_server_rpc *rpc, int status)
int int
srpc_handle_rpc(struct swi_workitem *wi) srpc_handle_rpc(struct swi_workitem *wi)
{ {
struct srpc_server_rpc *rpc = wi->swi_workitem.wi_data; struct srpc_server_rpc *rpc = container_of(wi, struct srpc_server_rpc, srpc_wi);
struct srpc_service_cd *scd = rpc->srpc_scd; struct srpc_service_cd *scd = rpc->srpc_scd;
struct srpc_service *sv = scd->scd_svc; struct srpc_service *sv = scd->scd_svc;
struct srpc_event *ev = &rpc->srpc_ev; struct srpc_event *ev = &rpc->srpc_ev;
...@@ -1188,7 +1188,7 @@ srpc_send_rpc(struct swi_workitem *wi) ...@@ -1188,7 +1188,7 @@ srpc_send_rpc(struct swi_workitem *wi)
LASSERT(wi); LASSERT(wi);
rpc = wi->swi_workitem.wi_data; rpc = container_of(wi, struct srpc_client_rpc, crpc_wi);
LASSERT(rpc); LASSERT(rpc);
LASSERT(wi == &rpc->crpc_wi); LASSERT(wi == &rpc->crpc_wi);
......
...@@ -476,13 +476,13 @@ swi_wi_action(struct cfs_workitem *wi) ...@@ -476,13 +476,13 @@ swi_wi_action(struct cfs_workitem *wi)
} }
static inline void static inline void
swi_init_workitem(struct swi_workitem *swi, void *data, swi_init_workitem(struct swi_workitem *swi,
swi_action_t action, struct cfs_wi_sched *sched) swi_action_t action, struct cfs_wi_sched *sched)
{ {
swi->swi_sched = sched; swi->swi_sched = sched;
swi->swi_action = action; swi->swi_action = action;
swi->swi_state = SWI_STATE_NEWBORN; swi->swi_state = SWI_STATE_NEWBORN;
cfs_wi_init(&swi->swi_workitem, data, swi_wi_action); cfs_wi_init(&swi->swi_workitem, swi_wi_action);
} }
static inline void static inline void
...@@ -533,7 +533,7 @@ srpc_init_client_rpc(struct srpc_client_rpc *rpc, struct lnet_process_id peer, ...@@ -533,7 +533,7 @@ srpc_init_client_rpc(struct srpc_client_rpc *rpc, struct lnet_process_id peer,
crpc_bulk.bk_iovs[nbulkiov])); crpc_bulk.bk_iovs[nbulkiov]));
INIT_LIST_HEAD(&rpc->crpc_list); INIT_LIST_HEAD(&rpc->crpc_list);
swi_init_workitem(&rpc->crpc_wi, rpc, srpc_send_rpc, swi_init_workitem(&rpc->crpc_wi, srpc_send_rpc,
lst_sched_test[lnet_cpt_of_nid(peer.nid)]); lst_sched_test[lnet_cpt_of_nid(peer.nid)]);
spin_lock_init(&rpc->crpc_lock); spin_lock_init(&rpc->crpc_lock);
atomic_set(&rpc->crpc_refcount, 1); /* 1 ref for caller */ atomic_set(&rpc->crpc_refcount, 1); /* 1 ref for caller */
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment