git://git.onelab.eu
/
linux-2.6.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
fedora core 6 1.2949 + vserver 2.2.0
[linux-2.6.git]
/
drivers
/
isdn
/
capi
/
kcapi.c
diff --git
a/drivers/isdn/capi/kcapi.c
b/drivers/isdn/capi/kcapi.c
index
8d58cfd
..
783a255
100644
(file)
--- a/
drivers/isdn/capi/kcapi.c
+++ b/
drivers/isdn/capi/kcapi.c
@@
-24,12
+24,15
@@
#include <linux/capi.h>
#include <linux/kernelcapi.h>
#include <linux/init.h>
#include <linux/capi.h>
#include <linux/kernelcapi.h>
#include <linux/init.h>
+#include <linux/moduleparam.h>
+#include <linux/delay.h>
#include <asm/uaccess.h>
#include <linux/isdn/capicmd.h>
#include <linux/isdn/capiutil.h>
#ifdef CONFIG_AVMB1_COMPAT
#include <linux/b1lli.h>
#endif
#include <asm/uaccess.h>
#include <linux/isdn/capicmd.h>
#include <linux/isdn/capiutil.h>
#ifdef CONFIG_AVMB1_COMPAT
#include <linux/b1lli.h>
#endif
+#include <linux/mutex.h>
static char *revision = "$Revision: 1.1.2.8 $";
static char *revision = "$Revision: 1.1.2.8 $";
@@
-40,7
+43,7
@@
static int showcapimsgs = 0;
MODULE_DESCRIPTION("CAPI4Linux: kernel CAPI layer");
MODULE_AUTHOR("Carsten Paeth");
MODULE_LICENSE("GPL");
MODULE_DESCRIPTION("CAPI4Linux: kernel CAPI layer");
MODULE_AUTHOR("Carsten Paeth");
MODULE_LICENSE("GPL");
-
MODULE_PARM(showcapimsgs, "i"
);
+
module_param(showcapimsgs, uint, 0
);
/* ------------------------------------------------------------- */
/* ------------------------------------------------------------- */
@@
-61,10
+64,10
@@
static char capi_manufakturer[64] = "AVM Berlin";
#define NCCI2CTRL(ncci) (((ncci) >> 24) & 0x7f)
LIST_HEAD(capi_drivers);
#define NCCI2CTRL(ncci) (((ncci) >> 24) & 0x7f)
LIST_HEAD(capi_drivers);
-
rwlock_t capi_drivers_list_lock = RW_LOCK_UNLOCKED
;
+
DEFINE_RWLOCK(capi_drivers_list_lock)
;
-static
rwlock_t application_lock = RW_LOCK_UNLOCKED
;
-static DE
CLARE_MUTEX(controller_sem
);
+static
DEFINE_RWLOCK(application_lock)
;
+static DE
FINE_MUTEX(controller_mutex
);
struct capi20_appl *capi_applications[CAPI_MAXAPPL];
struct capi_ctr *capi_cards[CAPI_MAXCONTR];
struct capi20_appl *capi_applications[CAPI_MAXAPPL];
struct capi_ctr *capi_cards[CAPI_MAXCONTR];
@@
-148,7
+151,10
@@
static void register_appl(struct capi_ctr *card, u16 applid, capi_register_param
{
card = capi_ctr_get(card);
{
card = capi_ctr_get(card);
- card->register_appl(card, applid, rparam);
+ if (card)
+ card->register_appl(card, applid, rparam);
+ else
+ printk(KERN_WARNING "%s: cannot get card resources\n", __FUNCTION__);
}
}
@@
-171,10
+177,15
@@
static void notify_up(u32 contr)
if (showcapimsgs & 1) {
printk(KERN_DEBUG "kcapi: notify up contr %d\n", contr);
}
if (showcapimsgs & 1) {
printk(KERN_DEBUG "kcapi: notify up contr %d\n", contr);
}
-
+ if (!card) {
+ printk(KERN_WARNING "%s: invalid contr %d\n", __FUNCTION__, contr);
+ return;
+ }
for (applid = 1; applid <= CAPI_MAXAPPL; applid++) {
ap = get_capi_appl_by_nr(applid);
for (applid = 1; applid <= CAPI_MAXAPPL; applid++) {
ap = get_capi_appl_by_nr(applid);
- if (ap && ap->callback && !ap->release_in_progress)
+ if (!ap || ap->release_in_progress) continue;
+ register_appl(card, applid, &ap->rparam);
+ if (ap->callback && !ap->release_in_progress)
ap->callback(KCI_CONTRUP, contr, &card->profile);
}
}
ap->callback(KCI_CONTRUP, contr, &card->profile);
}
}
@@
-197,9
+208,10
@@
static void notify_down(u32 contr)
}
}
}
}
-static void notify_handler(
void *data
)
+static void notify_handler(
struct work_struct *work
)
{
{
- struct capi_notifier *np = data;
+ struct capi_notifier *np =
+ container_of(work, struct capi_notifier, work);
switch (np->cmd) {
case KCI_CONTRUP:
switch (np->cmd) {
case KCI_CONTRUP:
@@
-224,7
+236,7
@@
static int notify_push(unsigned int cmd, u32 controller, u16 applid, u32 ncci)
if (!np)
return -ENOMEM;
if (!np)
return -ENOMEM;
- INIT_WORK(&np->work, notify_handler
, np
);
+ INIT_WORK(&np->work, notify_handler);
np->cmd = cmd;
np->controller = controller;
np->applid = applid;
np->cmd = cmd;
np->controller = controller;
np->applid = applid;
@@
-237,10
+249,11
@@
static int notify_push(unsigned int cmd, u32 controller, u16 applid, u32 ncci)
/* -------- Receiver ------------------------------------------ */
/* -------- Receiver ------------------------------------------ */
-static void recv_handler(
void *_ap
)
+static void recv_handler(
struct work_struct *work
)
{
struct sk_buff *skb;
{
struct sk_buff *skb;
- struct capi20_appl *ap = (struct capi20_appl *) _ap;
+ struct capi20_appl *ap =
+ container_of(work, struct capi20_appl, recv_work);
if ((!ap) || (ap->release_in_progress))
return;
if ((!ap) || (ap->release_in_progress))
return;
@@
-317,19
+330,8
@@
EXPORT_SYMBOL(capi_ctr_handle_message);
void capi_ctr_ready(struct capi_ctr * card)
{
void capi_ctr_ready(struct capi_ctr * card)
{
- u16 appl;
- struct capi20_appl *ap;
-
card->cardstate = CARD_RUNNING;
card->cardstate = CARD_RUNNING;
- down(&controller_sem);
- for (appl = 1; appl <= CAPI_MAXAPPL; appl++) {
- ap = get_capi_appl_by_nr(appl);
- if (!ap || ap->release_in_progress) continue;
- register_appl(card, appl, &ap->rparam);
- }
- up(&controller_sem);
-
printk(KERN_NOTICE "kcapi: card %d \"%s\" ready.\n",
card->cnr, card->name);
printk(KERN_NOTICE "kcapi: card %d \"%s\" ready.\n",
card->cnr, card->name);
@@
-396,20
+398,20
@@
attach_capi_ctr(struct capi_ctr *card)
{
int i;
{
int i;
-
down(&controller_sem
);
+
mutex_lock(&controller_mutex
);
for (i = 0; i < CAPI_MAXCONTR; i++) {
if (capi_cards[i] == NULL)
break;
}
if (i == CAPI_MAXCONTR) {
for (i = 0; i < CAPI_MAXCONTR; i++) {
if (capi_cards[i] == NULL)
break;
}
if (i == CAPI_MAXCONTR) {
-
up(&controller_sem
);
+
mutex_unlock(&controller_mutex
);
printk(KERN_ERR "kcapi: out of controller slots\n");
return -EBUSY;
}
capi_cards[i] = card;
printk(KERN_ERR "kcapi: out of controller slots\n");
return -EBUSY;
}
capi_cards[i] = card;
-
up(&controller_sem
);
+
mutex_unlock(&controller_mutex
);
card->nrecvctlpkt = 0;
card->nrecvdatapkt = 0;
card->nrecvctlpkt = 0;
card->nrecvdatapkt = 0;
@@
-527,18
+529,18
@@
u16 capi20_register(struct capi20_appl *ap)
ap->callback = NULL;
init_MUTEX(&ap->recv_sem);
skb_queue_head_init(&ap->recv_queue);
ap->callback = NULL;
init_MUTEX(&ap->recv_sem);
skb_queue_head_init(&ap->recv_queue);
- INIT_WORK(&ap->recv_work, recv_handler
, (void *)ap
);
+ INIT_WORK(&ap->recv_work, recv_handler);
ap->release_in_progress = 0;
write_unlock_irqrestore(&application_lock, flags);
ap->release_in_progress = 0;
write_unlock_irqrestore(&application_lock, flags);
-
down(&controller_sem
);
+
mutex_lock(&controller_mutex
);
for (i = 0; i < CAPI_MAXCONTR; i++) {
if (!capi_cards[i] || capi_cards[i]->cardstate != CARD_RUNNING)
continue;
register_appl(capi_cards[i], applid, &ap->rparam);
}
for (i = 0; i < CAPI_MAXCONTR; i++) {
if (!capi_cards[i] || capi_cards[i]->cardstate != CARD_RUNNING)
continue;
register_appl(capi_cards[i], applid, &ap->rparam);
}
-
up(&controller_sem
);
+
mutex_unlock(&controller_mutex
);
if (showcapimsgs & 1) {
printk(KERN_DEBUG "kcapi: appl %d up\n", applid);
if (showcapimsgs & 1) {
printk(KERN_DEBUG "kcapi: appl %d up\n", applid);
@@
-561,13
+563,13
@@
u16 capi20_release(struct capi20_appl *ap)
capi_applications[ap->applid - 1] = NULL;
write_unlock_irqrestore(&application_lock, flags);
capi_applications[ap->applid - 1] = NULL;
write_unlock_irqrestore(&application_lock, flags);
-
down(&controller_sem
);
+
mutex_lock(&controller_mutex
);
for (i = 0; i < CAPI_MAXCONTR; i++) {
if (!capi_cards[i] || capi_cards[i]->cardstate != CARD_RUNNING)
continue;
release_appl(capi_cards[i], ap->applid);
}
for (i = 0; i < CAPI_MAXCONTR; i++) {
if (!capi_cards[i] || capi_cards[i]->cardstate != CARD_RUNNING)
continue;
release_appl(capi_cards[i], ap->applid);
}
-
up(&controller_sem
);
+
mutex_unlock(&controller_mutex
);
flush_scheduled_work();
skb_queue_purge(&ap->recv_queue);
flush_scheduled_work();
skb_queue_purge(&ap->recv_queue);
@@
-831,8
+833,7
@@
static int old_capi_manufacturer(unsigned int cmd, void __user *data)
while (card->cardstate != CARD_RUNNING) {
while (card->cardstate != CARD_RUNNING) {
- set_current_state(TASK_INTERRUPTIBLE);
- schedule_timeout(HZ/10); /* 0.1 sec */
+ msleep_interruptible(100); /* 0.1 sec */
if (signal_pending(current)) {
capi_ctr_put(card);
if (signal_pending(current)) {
capi_ctr_put(card);
@@
-856,8
+857,7
@@
static int old_capi_manufacturer(unsigned int cmd, void __user *data)
while (card->cardstate > CARD_DETECTED) {
while (card->cardstate > CARD_DETECTED) {
- set_current_state(TASK_INTERRUPTIBLE);
- schedule_timeout(HZ/10); /* 0.1 sec */
+ msleep_interruptible(100); /* 0.1 sec */
if (signal_pending(current))
return -EINTR;
if (signal_pending(current))
return -EINTR;