diff options
author | Sjur Braendeland <sjur.brandeland@stericsson.com> | 2010-04-28 04:54:36 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2010-04-28 15:55:12 -0400 |
commit | 5b2086567503f9b55136642031ec0067319f58e0 (patch) | |
tree | 00c4264ed7a8a989b398166c2c5f98175f5c28a5 /net | |
parent | e539d83cc8a4fa581cbf8ed288fdadb19a692cb0 (diff) |
caif: Add reference counting to service layer
Changes:
o Added functions cfsrvl_get and cfsrvl_put.
o Added support release_client to use by socket and net device.
o Increase reference counting for in-flight packets from cfmuxl
Signed-off-by: Sjur Braendeland <sjur.brandeland@stericsson.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net')
-rw-r--r-- | net/caif/caif_dev.c | 6 | ||||
-rw-r--r-- | net/caif/cfcnfg.c | 7 | ||||
-rw-r--r-- | net/caif/cfmuxl.c | 7 | ||||
-rw-r--r-- | net/caif/cfsrvl.c | 7 |
4 files changed, 26 insertions, 1 deletions
diff --git a/net/caif/caif_dev.c b/net/caif/caif_dev.c index be1f674a3b67..0145bae0274f 100644 --- a/net/caif/caif_dev.c +++ b/net/caif/caif_dev.c | |||
@@ -346,6 +346,12 @@ int caif_disconnect_client(struct cflayer *adap_layer) | |||
346 | } | 346 | } |
347 | EXPORT_SYMBOL(caif_disconnect_client); | 347 | EXPORT_SYMBOL(caif_disconnect_client); |
348 | 348 | ||
349 | void caif_release_client(struct cflayer *adap_layer) | ||
350 | { | ||
351 | cfcnfg_release_adap_layer(adap_layer); | ||
352 | } | ||
353 | EXPORT_SYMBOL(caif_release_client); | ||
354 | |||
349 | /* Per-namespace Caif devices handling */ | 355 | /* Per-namespace Caif devices handling */ |
350 | static int caif_init_net(struct net *net) | 356 | static int caif_init_net(struct net *net) |
351 | { | 357 | { |
diff --git a/net/caif/cfcnfg.c b/net/caif/cfcnfg.c index d52f2566916e..f94f3dfe85c1 100644 --- a/net/caif/cfcnfg.c +++ b/net/caif/cfcnfg.c | |||
@@ -247,6 +247,13 @@ end: | |||
247 | } | 247 | } |
248 | EXPORT_SYMBOL(cfcnfg_disconn_adapt_layer); | 248 | EXPORT_SYMBOL(cfcnfg_disconn_adapt_layer); |
249 | 249 | ||
250 | void cfcnfg_release_adap_layer(struct cflayer *adap_layer) | ||
251 | { | ||
252 | if (adap_layer->dn) | ||
253 | cfsrvl_put(adap_layer->dn); | ||
254 | } | ||
255 | EXPORT_SYMBOL(cfcnfg_release_adap_layer); | ||
256 | |||
250 | static void cfcnfg_linkdestroy_rsp(struct cflayer *layer, u8 channel_id, | 257 | static void cfcnfg_linkdestroy_rsp(struct cflayer *layer, u8 channel_id, |
251 | struct cflayer *client_layer) | 258 | struct cflayer *client_layer) |
252 | { | 259 | { |
diff --git a/net/caif/cfmuxl.c b/net/caif/cfmuxl.c index 6fb9f9e96cf8..7372f27f1d32 100644 --- a/net/caif/cfmuxl.c +++ b/net/caif/cfmuxl.c | |||
@@ -62,6 +62,7 @@ int cfmuxl_set_uplayer(struct cflayer *layr, struct cflayer *up, u8 linkid) | |||
62 | { | 62 | { |
63 | struct cfmuxl *muxl = container_obj(layr); | 63 | struct cfmuxl *muxl = container_obj(layr); |
64 | spin_lock(&muxl->receive_lock); | 64 | spin_lock(&muxl->receive_lock); |
65 | cfsrvl_get(up); | ||
65 | list_add(&up->node, &muxl->srvl_list); | 66 | list_add(&up->node, &muxl->srvl_list); |
66 | spin_unlock(&muxl->receive_lock); | 67 | spin_unlock(&muxl->receive_lock); |
67 | return 0; | 68 | return 0; |
@@ -172,8 +173,11 @@ struct cflayer *cfmuxl_remove_uplayer(struct cflayer *layr, u8 id) | |||
172 | struct cfmuxl *muxl = container_obj(layr); | 173 | struct cfmuxl *muxl = container_obj(layr); |
173 | spin_lock(&muxl->receive_lock); | 174 | spin_lock(&muxl->receive_lock); |
174 | up = get_up(muxl, id); | 175 | up = get_up(muxl, id); |
176 | if (up == NULL) | ||
177 | return NULL; | ||
175 | memset(muxl->up_cache, 0, sizeof(muxl->up_cache)); | 178 | memset(muxl->up_cache, 0, sizeof(muxl->up_cache)); |
176 | list_del(&up->node); | 179 | list_del(&up->node); |
180 | cfsrvl_put(up); | ||
177 | spin_unlock(&muxl->receive_lock); | 181 | spin_unlock(&muxl->receive_lock); |
178 | return up; | 182 | return up; |
179 | } | 183 | } |
@@ -203,8 +207,9 @@ static int cfmuxl_receive(struct cflayer *layr, struct cfpkt *pkt) | |||
203 | */ | 207 | */ |
204 | return /* CFGLU_EPROT; */ 0; | 208 | return /* CFGLU_EPROT; */ 0; |
205 | } | 209 | } |
206 | 210 | cfsrvl_get(up); | |
207 | ret = up->receive(up, pkt); | 211 | ret = up->receive(up, pkt); |
212 | cfsrvl_put(up); | ||
208 | return ret; | 213 | return ret; |
209 | } | 214 | } |
210 | 215 | ||
diff --git a/net/caif/cfsrvl.c b/net/caif/cfsrvl.c index d470c51c6431..aff31f34528f 100644 --- a/net/caif/cfsrvl.c +++ b/net/caif/cfsrvl.c | |||
@@ -158,6 +158,13 @@ void cfsrvl_init(struct cfsrvl *service, | |||
158 | service->layer.ctrlcmd = cfservl_ctrlcmd; | 158 | service->layer.ctrlcmd = cfservl_ctrlcmd; |
159 | service->layer.modemcmd = cfservl_modemcmd; | 159 | service->layer.modemcmd = cfservl_modemcmd; |
160 | service->dev_info = *dev_info; | 160 | service->dev_info = *dev_info; |
161 | kref_init(&service->ref); | ||
162 | } | ||
163 | |||
164 | void cfsrvl_release(struct kref *kref) | ||
165 | { | ||
166 | struct cfsrvl *service = container_of(kref, struct cfsrvl, ref); | ||
167 | kfree(service); | ||
161 | } | 168 | } |
162 | 169 | ||
163 | bool cfsrvl_ready(struct cfsrvl *service, int *err) | 170 | bool cfsrvl_ready(struct cfsrvl *service, int *err) |