[dahdi-commits] sruffell: branch linux/sruffell/chan_list_refactoring r9257 - /linux/team/sru...

SVN commits to the DAHDI project dahdi-commits at lists.digium.com
Thu Sep 2 12:40:59 CDT 2010


Author: sruffell
Date: Thu Sep  2 12:40:47 2010
New Revision: 9257

URL: http://svnview.digium.com/svn/dahdi?view=rev&rev=9257
Log:
dahdi: Refactor some of the access to the 'spans' array.

It is still used at span registration / unregistration to assign the span
number, and when iterating through all spans. Otherwise, moves the code base a
step closer to removal of the 'spans' array.

Modified:
    linux/team/sruffell/chan_list_refactoring/drivers/dahdi/dahdi-base.c

Modified: linux/team/sruffell/chan_list_refactoring/drivers/dahdi/dahdi-base.c
URL: http://svnview.digium.com/svn/dahdi/linux/team/sruffell/chan_list_refactoring/drivers/dahdi/dahdi-base.c?view=diff&rev=9257&r1=9256&r2=9257
==============================================================================
--- linux/team/sruffell/chan_list_refactoring/drivers/dahdi/dahdi-base.c (original)
+++ linux/team/sruffell/chan_list_refactoring/drivers/dahdi/dahdi-base.c Thu Sep  2 12:40:47 2010
@@ -402,37 +402,49 @@
 	struct dahdi_tone mfr2_rev_continuous[16];	/* MFR2 REV tones for this zone, continuous play */
 };
 
-static struct dahdi_span *spans[DAHDI_MAX_SPANS];
-static struct dahdi_chan *chans[DAHDI_MAX_CHANNELS];
-
-static int maxspans = 0;
-static int maxchans = 0;
-static int maxconfs = 0;
-static int maxlinks = 0;
-
-static int default_zone = -1;
-
-short __dahdi_mulaw[256];
-short __dahdi_alaw[256];
-
-#ifndef CONFIG_CALC_XLAW
-u_char __dahdi_lin2mu[16384];
-
-u_char __dahdi_lin2a[16384];
-#endif
-
-static u_char defgain[256];
-
-#if LINUX_VERSION_CODE < KERNEL_VERSION(2, 6, 10)
-#define __RW_LOCK_UNLOCKED() RW_LOCK_UNLOCKED
-#endif
-
 #ifdef DEFINE_RWLOCK
 static DEFINE_RWLOCK(zone_lock);
 static DEFINE_RWLOCK(chan_lock);
 #else
 static rwlock_t zone_lock = RW_LOCK_UNLOCKED;
 static rwlock_t chan_lock = RW_LOCK_UNLOCKED;
+#endif
+
+/* Protected by chan_lock. */
+static struct dahdi_span *spans[DAHDI_MAX_SPANS];
+static struct dahdi_chan *chans[DAHDI_MAX_CHANNELS];
+
+static int maxspans = 0;
+
+static struct dahdi_span *find_span(int spanno)
+{
+	return (spanno > 0 && spanno < DAHDI_MAX_SPANS) ? spans[spanno] : NULL;
+}
+
+static inline unsigned int span_count(void)
+{
+	return maxspans;
+}
+
+static int maxchans = 0;
+static int maxconfs = 0;
+static int maxlinks = 0;
+
+static int default_zone = -1;
+
+short __dahdi_mulaw[256];
+short __dahdi_alaw[256];
+
+#ifndef CONFIG_CALC_XLAW
+u_char __dahdi_lin2mu[16384];
+
+u_char __dahdi_lin2a[16384];
+#endif
+
+static u_char defgain[256];
+
+#if LINUX_VERSION_CODE < KERNEL_VERSION(2, 6, 10)
+#define __RW_LOCK_UNLOCKED() RW_LOCK_UNLOCKED
 #endif
 
 static struct dahdi_zone *tone_zones[DAHDI_TONE_ZONE_MAX];
@@ -672,7 +684,9 @@
 static int dahdi_proc_read(char *page, char **start, off_t off, int count, int *eof, void *data)
 {
 	int x, len = 0, real_count;
-	long span;
+	long spanno;
+	struct dahdi_span *s;
+
 
 	/* In Linux 2.6, page is always PROC_BLOCK_SIZE=(PAGE_SIZE-1024) bytes.
 	 * 0<count<=PROC_BLOCK_SIZE . count=1 will produce an error in
@@ -680,77 +694,81 @@
 	 * An ugly hack. Good way: seq_printf (seq_file.c). */
         real_count = count;
 	count = PAGE_SIZE-1024;
-	span = (long)data;
-	if (!span)
+	spanno = (long)data;
+	if (!spanno)
 		return 0;
 
-	if (spans[span]->name)
-		len += snprintf(page + len, count - len, "Span %ld: %s ",
-				span, spans[span]->name);
-	if (spans[span]->desc)
+	s = find_span(spanno);
+	if (!s)
+		return -ENODEV;
+
+	if (s->name)
+		len += snprintf(page + len, count - len, "Span %d: %s ",
+				s->spanno, s->name);
+	if (s->desc)
 		len += snprintf(page + len, count - len, "\"%s\"",
-				spans[span]->desc);
+				s->desc);
 	else
 		len += snprintf(page + len, count - len, "\"\"");
 
-	if (spans[span] == master)
+	if (s == master)
 		len += snprintf(page + len, count - len, " (MASTER)");
 
-	if (spans[span]->lineconfig) {
+	if (s->lineconfig) {
 		/* framing first */
-		if (spans[span]->lineconfig & DAHDI_CONFIG_B8ZS)
+		if (s->lineconfig & DAHDI_CONFIG_B8ZS)
 			len += snprintf(page + len, count - len, " B8ZS/");
-		else if (spans[span]->lineconfig & DAHDI_CONFIG_AMI)
+		else if (s->lineconfig & DAHDI_CONFIG_AMI)
 			len += snprintf(page + len, count - len, " AMI/");
-		else if (spans[span]->lineconfig & DAHDI_CONFIG_HDB3)
+		else if (s->lineconfig & DAHDI_CONFIG_HDB3)
 			len += snprintf(page + len, count - len, " HDB3/");
 		/* then coding */
-		if (spans[span]->lineconfig & DAHDI_CONFIG_ESF)
+		if (s->lineconfig & DAHDI_CONFIG_ESF)
 			len += snprintf(page + len, count - len, "ESF");
-		else if (spans[span]->lineconfig & DAHDI_CONFIG_D4)
+		else if (s->lineconfig & DAHDI_CONFIG_D4)
 			len += snprintf(page + len, count - len, "D4");
-		else if (spans[span]->lineconfig & DAHDI_CONFIG_CCS)
+		else if (s->lineconfig & DAHDI_CONFIG_CCS)
 			len += snprintf(page + len, count - len, "CCS");
 		/* E1's can enable CRC checking */
-		if (spans[span]->lineconfig & DAHDI_CONFIG_CRC4)
+		if (s->lineconfig & DAHDI_CONFIG_CRC4)
 			len += snprintf(page + len, count - len, "/CRC4");
 	}
 
 	len += snprintf(page + len, count - len, " ");
 
 	/* list alarms */
-	len += fill_alarm_string(page + len, count - len, spans[span]->alarms);
-	if (spans[span]->syncsrc &&
-		(spans[span]->syncsrc == spans[span]->spanno))
+	len += fill_alarm_string(page + len, count - len, s->alarms);
+	if (s->syncsrc &&
+		(s->syncsrc == s->spanno))
 		len += snprintf(page + len, count - len, "ClockSource ");
 	len += snprintf(page + len, count - len, "\n");
-	if (spans[span]->count.bpv)
+	if (s->count.bpv)
 		len += snprintf(page + len, count - len, "\tBPV count: %d\n",
-				spans[span]->count.bpv);
-	if (spans[span]->count.crc4)
+				s->count.bpv);
+	if (s->count.crc4)
 		len += snprintf(page + len, count - len,
 				"\tCRC4 error count: %d\n",
-				spans[span]->count.crc4);
-	if (spans[span]->count.ebit)
+				s->count.crc4);
+	if (s->count.ebit)
 		len += snprintf(page + len, count - len,
 				"\tE-bit error count: %d\n",
-				spans[span]->count.ebit);
-	if (spans[span]->count.fas)
+				s->count.ebit);
+	if (s->count.fas)
 		len += snprintf(page + len, count - len,
 				"\tFAS error count: %d\n",
-				spans[span]->count.fas);
-	if (spans[span]->irqmisses)
+				s->count.fas);
+	if (s->irqmisses)
 		len += snprintf(page + len, count - len,
 				"\tIRQ misses: %d\n",
-				spans[span]->irqmisses);
-	if (spans[span]->timingslips)
+				s->irqmisses);
+	if (s->timingslips)
 		len += snprintf(page + len, count - len,
 				"\tTiming slips: %d\n",
-				spans[span]->timingslips);
+				s->timingslips);
 	len += snprintf(page + len, count - len, "\n");
 
-	for (x = 0; x < spans[span]->channels; x++) {
-		struct dahdi_chan *chan = spans[span]->chans[x];
+	for (x = 0; x < s->channels; x++) {
+		struct dahdi_chan *chan = s->chans[x];
 
 		if (chan->name)
 			len += snprintf(page + len, count - len,
@@ -3562,19 +3580,20 @@
 		span->lastalarms = span->alarms;
 		for (x = 0; x < span->channels; x++)
 			dahdi_alarm_channel(span->chans[x], span->alarms);
+
 		/* Switch to other master if current master in alarm */
 		for (x=1; x<maxspans; x++) {
-			if (spans[x] && !spans[x]->alarms && (spans[x]->flags & DAHDI_FLAG_RUNNING)) {
-				if (master != spans[x]) {
+			struct dahdi_span *const s = spans[x];
+			if (s && !s->alarms && (s->flags & DAHDI_FLAG_RUNNING)) {
+				if (master != s) {
 					module_printk(KERN_NOTICE,
 						"Master changed to %s\n",
-						spans[x]->name);
+						s->name);
 				}
-				master = spans[x];
+				master = s;
 				break;
 			}
 		}
-
 		/* Report more detailed alarms */
 		if (debug) {
 			if (span->alarms & DAHDI_ALARM_LOS) {
@@ -3595,20 +3614,6 @@
 		}
 	}
 }
-
-#define VALID_SPAN(j) do { \
-	if ((j >= DAHDI_MAX_SPANS) || (j < 1)) \
-		return -EINVAL; \
-	if (!spans[j]) \
-		return -ENXIO; \
-} while(0)
-
-#define CHECK_VALID_SPAN(j) do { \
-	/* Start a given span */ \
-	if (get_user(j, (int __user *)data)) \
-		return -EFAULT; \
-	VALID_SPAN(j); \
-} while(0)
 
 #define VALID_CHANNEL(j) do { \
 	if ((j >= DAHDI_MAX_CHANNELS) || (j < 1)) \
@@ -3798,6 +3803,7 @@
 		struct dahdi_params param;
 	} stack;
 
+	struct dahdi_span *s;
 	struct dahdi_chan *chan;
 	unsigned long flags;
 	int i,j;
@@ -3935,55 +3941,54 @@
 		if (copy_from_user(&stack.spaninfo, user_data, size_to_copy))
 			return -EFAULT;
 		i = stack.spaninfo.spanno; /* get specified span number */
-		if ((i < 0) || (i >= maxspans)) return(-EINVAL);  /* if bad span no */
 		if (i == 0) {
 			/* if to figure it out for this chan */
 			if (!chans[unit])
 				return -EINVAL;
 			i = chans[unit]->span->spanno;
 		}
-		if (!spans[i])
+		s = find_span(i);
+		if (!s)
 			return -EINVAL;
 		stack.spaninfo.spanno = i; /* put the span # in here */
-		stack.spaninfo.totalspans = 0;
-		if (maxspans) stack.spaninfo.totalspans = maxspans - 1; /* put total number of spans here */
-		dahdi_copy_string(stack.spaninfo.desc, spans[i]->desc, sizeof(stack.spaninfo.desc));
-		dahdi_copy_string(stack.spaninfo.name, spans[i]->name, sizeof(stack.spaninfo.name));
-		stack.spaninfo.alarms = spans[i]->alarms;		/* get alarm status */
-		stack.spaninfo.rxlevel = spans[i]->rxlevel;	/* get rx level */
-		stack.spaninfo.txlevel = spans[i]->txlevel;	/* get tx level */
-
-		stack.spaninfo.bpvcount = spans[i]->count.bpv;
-		stack.spaninfo.crc4count = spans[i]->count.crc4;
-		stack.spaninfo.ebitcount = spans[i]->count.ebit;
-		stack.spaninfo.fascount = spans[i]->count.fas;
-		stack.spaninfo.fecount = spans[i]->count.fe;
-		stack.spaninfo.cvcount = spans[i]->count.cv;
-		stack.spaninfo.becount = spans[i]->count.be;
-		stack.spaninfo.prbs = spans[i]->count.prbs;
-		stack.spaninfo.errsec = spans[i]->count.errsec;
-
-		stack.spaninfo.irqmisses = spans[i]->irqmisses;	/* get IRQ miss count */
-		stack.spaninfo.syncsrc = spans[i]->syncsrc;	/* get active sync source */
-		stack.spaninfo.totalchans = spans[i]->channels;
+		stack.spaninfo.totalspans = span_count();
+		dahdi_copy_string(stack.spaninfo.desc, s->desc, sizeof(stack.spaninfo.desc));
+		dahdi_copy_string(stack.spaninfo.name, s->name, sizeof(stack.spaninfo.name));
+		stack.spaninfo.alarms = s->alarms;		/* get alarm status */
+		stack.spaninfo.rxlevel = s->rxlevel;	/* get rx level */
+		stack.spaninfo.txlevel = s->txlevel;	/* get tx level */
+
+		stack.spaninfo.bpvcount = s->count.bpv;
+		stack.spaninfo.crc4count = s->count.crc4;
+		stack.spaninfo.ebitcount = s->count.ebit;
+		stack.spaninfo.fascount = s->count.fas;
+		stack.spaninfo.fecount = s->count.fe;
+		stack.spaninfo.cvcount = s->count.cv;
+		stack.spaninfo.becount = s->count.be;
+		stack.spaninfo.prbs = s->count.prbs;
+		stack.spaninfo.errsec = s->count.errsec;
+
+		stack.spaninfo.irqmisses = s->irqmisses;	/* get IRQ miss count */
+		stack.spaninfo.syncsrc = s->syncsrc;	/* get active sync source */
+		stack.spaninfo.totalchans = s->channels;
 		stack.spaninfo.numchans = 0;
-		for (j = 0; j < spans[i]->channels; j++) {
-			if (spans[i]->chans[j]->sig)
+		for (j = 0; j < s->channels; j++) {
+			if (s->chans[j]->sig)
 				stack.spaninfo.numchans++;
 		}
-		stack.spaninfo.lbo = spans[i]->lbo;
-		stack.spaninfo.lineconfig = spans[i]->lineconfig;
-		stack.spaninfo.irq = spans[i]->irq;
-		stack.spaninfo.linecompat = spans[i]->linecompat;
-		dahdi_copy_string(stack.spaninfo.lboname, dahdi_lboname(spans[i]->lbo), sizeof(stack.spaninfo.lboname));
-		if (spans[i]->manufacturer)
-			dahdi_copy_string(stack.spaninfo.manufacturer, spans[i]->manufacturer,
+		stack.spaninfo.lbo = s->lbo;
+		stack.spaninfo.lineconfig = s->lineconfig;
+		stack.spaninfo.irq = s->irq;
+		stack.spaninfo.linecompat = s->linecompat;
+		dahdi_copy_string(stack.spaninfo.lboname, dahdi_lboname(s->lbo), sizeof(stack.spaninfo.lboname));
+		if (s->manufacturer)
+			dahdi_copy_string(stack.spaninfo.manufacturer, s->manufacturer,
 				sizeof(stack.spaninfo.manufacturer));
-		if (spans[i]->devicetype)
-			dahdi_copy_string(stack.spaninfo.devicetype, spans[i]->devicetype, sizeof(stack.spaninfo.devicetype));
-		dahdi_copy_string(stack.spaninfo.location, spans[i]->location, sizeof(stack.spaninfo.location));
-		if (spans[i]->spantype)
-			dahdi_copy_string(stack.spaninfo.spantype, spans[i]->spantype, sizeof(stack.spaninfo.spantype));
+		if (s->devicetype)
+			dahdi_copy_string(stack.spaninfo.devicetype, s->devicetype, sizeof(stack.spaninfo.devicetype));
+		dahdi_copy_string(stack.spaninfo.location, s->location, sizeof(stack.spaninfo.location));
+		if (s->spantype)
+			dahdi_copy_string(stack.spaninfo.spantype, s->spantype, sizeof(stack.spaninfo.spantype));
 
 		if (copy_to_user(user_data, &stack.spaninfo, size_to_copy))
 			return -EFAULT;
@@ -3995,62 +4000,60 @@
 				   size_to_copy))
 			return -EFAULT;
 		i = stack.spaninfo_v1.spanno; /* get specified span number */
-		if ((i < 0) || (i >= maxspans))
-			return -EINVAL;  /* if bad span no */
 		if (i == 0) {
 			/* if to figure it out for this chan */
 			if (!chans[unit])
 				return -EINVAL;
 			i = chans[unit]->span->spanno;
 		}
-		if (!spans[i])
+		s = find_span(i);
+		if (!s)
 			return -EINVAL;
 		stack.spaninfo_v1.spanno = i; /* put the span # in here */
 		stack.spaninfo_v1.totalspans = 0;
-		if (maxspans) /* put total number of spans here */
-			stack.spaninfo_v1.totalspans = maxspans - 1;
+		stack.spaninfo_v1.totalspans = span_count();
 		dahdi_copy_string(stack.spaninfo_v1.desc,
-				  spans[i]->desc,
+				  s->desc,
 				  sizeof(stack.spaninfo_v1.desc));
 		dahdi_copy_string(stack.spaninfo_v1.name,
-				  spans[i]->name,
+				  s->name,
 				  sizeof(stack.spaninfo_v1.name));
-		stack.spaninfo_v1.alarms = spans[i]->alarms;
-		stack.spaninfo_v1.bpvcount = spans[i]->count.bpv;
-		stack.spaninfo_v1.rxlevel = spans[i]->rxlevel;
-		stack.spaninfo_v1.txlevel = spans[i]->txlevel;
-		stack.spaninfo_v1.crc4count = spans[i]->count.crc4;
-		stack.spaninfo_v1.ebitcount = spans[i]->count.ebit;
-		stack.spaninfo_v1.fascount = spans[i]->count.fas;
-		stack.spaninfo_v1.irqmisses = spans[i]->irqmisses;
-		stack.spaninfo_v1.syncsrc = spans[i]->syncsrc;
-		stack.spaninfo_v1.totalchans = spans[i]->channels;
+		stack.spaninfo_v1.alarms = s->alarms;
+		stack.spaninfo_v1.bpvcount = s->count.bpv;
+		stack.spaninfo_v1.rxlevel = s->rxlevel;
+		stack.spaninfo_v1.txlevel = s->txlevel;
+		stack.spaninfo_v1.crc4count = s->count.crc4;
+		stack.spaninfo_v1.ebitcount = s->count.ebit;
+		stack.spaninfo_v1.fascount = s->count.fas;
+		stack.spaninfo_v1.irqmisses = s->irqmisses;
+		stack.spaninfo_v1.syncsrc = s->syncsrc;
+		stack.spaninfo_v1.totalchans = s->channels;
 		stack.spaninfo_v1.numchans = 0;
-		for (j = 0; j < spans[i]->channels; j++) {
-			if (spans[i]->chans[j]->sig)
+		for (j = 0; j < s->channels; j++) {
+			if (s->chans[j]->sig)
 				stack.spaninfo_v1.numchans++;
 		}
-		stack.spaninfo_v1.lbo = spans[i]->lbo;
-		stack.spaninfo_v1.lineconfig = spans[i]->lineconfig;
-		stack.spaninfo_v1.irq = spans[i]->irq;
-		stack.spaninfo_v1.linecompat = spans[i]->linecompat;
+		stack.spaninfo_v1.lbo = s->lbo;
+		stack.spaninfo_v1.lineconfig = s->lineconfig;
+		stack.spaninfo_v1.irq = s->irq;
+		stack.spaninfo_v1.linecompat = s->linecompat;
 		dahdi_copy_string(stack.spaninfo_v1.lboname,
-				  dahdi_lboname(spans[i]->lbo),
+				  dahdi_lboname(s->lbo),
 				  sizeof(stack.spaninfo_v1.lboname));
-		if (spans[i]->manufacturer)
+		if (s->manufacturer)
 			dahdi_copy_string(stack.spaninfo_v1.manufacturer,
-				spans[i]->manufacturer,
+				s->manufacturer,
 				sizeof(stack.spaninfo_v1.manufacturer));
-		if (spans[i]->devicetype)
+		if (s->devicetype)
 			dahdi_copy_string(stack.spaninfo_v1.devicetype,
-					  spans[i]->devicetype,
+					  s->devicetype,
 					  sizeof(stack.spaninfo_v1.devicetype));
 		dahdi_copy_string(stack.spaninfo_v1.location,
-				  spans[i]->location,
+				  s->location,
 				  sizeof(stack.spaninfo_v1.location));
-		if (spans[i]->spantype)
+		if (s->spantype)
 			dahdi_copy_string(stack.spaninfo_v1.spantype,
-					  spans[i]->spantype,
+					  s->spantype,
 					  sizeof(stack.spaninfo_v1.spantype));
 
 		if (copy_to_user((__user struct dahdi_spaninfo_v1 *) data,
@@ -4191,6 +4194,7 @@
 	unsigned long flags;
 	void __user * const user_data = (void __user *)data;
 	int rv;
+	struct dahdi_span *s;
 	switch(cmd) {
 	case DAHDI_INDIRECT:
 	{
@@ -4204,40 +4208,49 @@
 	case DAHDI_SPANCONFIG:
 	{
 		struct dahdi_lineconfig lc;
+		struct dahdi_span *s;
 
 		if (copy_from_user(&lc, user_data, sizeof(lc)))
 			return -EFAULT;
-		VALID_SPAN(lc.span);
-		if ((lc.lineconfig & 0x1ff0 & spans[lc.span]->linecompat) !=
+		s = find_span(lc.span);
+		if (!s)
+			return -ENXIO;
+		if ((lc.lineconfig & 0x1ff0 & s->linecompat) !=
 		    (lc.lineconfig & 0x1ff0))
 			return -EINVAL;
-		if (spans[lc.span]->ops->spanconfig) {
-			spans[lc.span]->lineconfig = lc.lineconfig;
-			spans[lc.span]->lbo = lc.lbo;
-			spans[lc.span]->txlevel = lc.lbo;
-			spans[lc.span]->rxlevel = 0;
-
-			return spans[lc.span]->ops->spanconfig(spans[lc.span], &lc);
+		if (s->ops->spanconfig) {
+			s->lineconfig = lc.lineconfig;
+			s->lbo = lc.lbo;
+			s->txlevel = lc.lbo;
+			s->rxlevel = 0;
+
+			return s->ops->spanconfig(s, &lc);
 		}
 		return 0;
 	}
 	case DAHDI_STARTUP:
-		CHECK_VALID_SPAN(j);
-		if (spans[j]->flags & DAHDI_FLAG_RUNNING)
+		if (get_user(j, (int __user *)data))
+			return -EFAULT;
+		s = find_span(j);
+		if (!s)
+			return -ENXIO;
+
+		if (s->flags & DAHDI_FLAG_RUNNING)
 			return 0;
 
-		if (spans[j]->ops->startup)
-			res = spans[j]->ops->startup(spans[j]);
+		if (s->ops->startup)
+			res = s->ops->startup(s);
 
 		if (!res) {
 			/* Mark as running and hangup any channels */
-			spans[j]->flags |= DAHDI_FLAG_RUNNING;
-			for (x=0;x<spans[j]->channels;x++) {
-				y = dahdi_q_sig(spans[j]->chans[x]) & 0xff;
-				if (y >= 0) spans[j]->chans[x]->rxsig = (unsigned char)y;
-				spin_lock_irqsave(&spans[j]->chans[x]->lock, flags);
-				dahdi_hangup(spans[j]->chans[x]);
-				spin_unlock_irqrestore(&spans[j]->chans[x]->lock, flags);
+			s->flags |= DAHDI_FLAG_RUNNING;
+			for (x = 0; x < s->channels; x++) {
+				y = dahdi_q_sig(s->chans[x]) & 0xff;
+				if (y >= 0)
+					s->chans[x]->rxsig = (unsigned char)y;
+				spin_lock_irqsave(&s->chans[x]->lock, flags);
+				dahdi_hangup(s->chans[x]);
+				spin_unlock_irqrestore(&s->chans[x]->lock, flags);
 				/*
 				 * Set the rxhooksig back to
 				 * DAHDI_RXSIG_INITIAL so that new events are
@@ -4245,15 +4258,20 @@
 				 * received hook state.
 				 * 
 				 */
-				spans[j]->chans[x]->rxhooksig = DAHDI_RXSIG_INITIAL;
+				s->chans[x]->rxhooksig = DAHDI_RXSIG_INITIAL;
 			}
 		}
 		return 0;
 	case DAHDI_SHUTDOWN:
-		CHECK_VALID_SPAN(j);
-		if (spans[j]->ops->shutdown)
-			res =  spans[j]->ops->shutdown(spans[j]);
-		spans[j]->flags &= ~DAHDI_FLAG_RUNNING;
+		if (get_user(j, (int __user *)data))
+			return -EFAULT;
+		s = find_span(j);
+		if (!s)
+			return -ENXIO;
+
+		if (s->ops->shutdown)
+			res =  s->ops->shutdown(s);
+		s->flags &= ~DAHDI_FLAG_RUNNING;
 		return 0;
 	case DAHDI_ATTACH_ECHOCAN:
 	{
@@ -4612,16 +4630,16 @@
 		  /* get struct from user */
 		if (copy_from_user(&maint, user_data, sizeof(maint)))
 			return -EFAULT;
-		/* must be valid span number */
-		if ((maint.spanno < 1) || (maint.spanno > DAHDI_MAX_SPANS) || (!spans[maint.spanno]))
+		s = find_span(maint.spanno);
+		if (!s)
 			return -EINVAL;
-		if (!spans[maint.spanno]->ops->maint)
+		if (!s->ops->maint)
 			return -ENOSYS;
-		spin_lock_irqsave(&spans[maint.spanno]->lock, flags);
+		spin_lock_irqsave(&s->lock, flags);
 		  /* save current maint state */
-		i = spans[maint.spanno]->maintstat;
+		i = s->maintstat;
 		  /* set maint mode */
-		spans[maint.spanno]->maintstat = maint.command;
+		s->maintstat = maint.command;
 		switch(maint.command) {
 		case DAHDI_MAINT_NONE:
 		case DAHDI_MAINT_LOCALLOOP:
@@ -4630,23 +4648,23 @@
 			/* if same, ignore it */
 			if (i == maint.command)
 				break;
-			rv = spans[maint.spanno]->ops->maint(spans[maint.spanno], maint.command);
-			spin_unlock_irqrestore(&spans[maint.spanno]->lock, flags);
+			rv = s->ops->maint(s, maint.command);
+			spin_unlock_irqrestore(&s->lock, flags);
 			if (rv)
 				return rv;
-			spin_lock_irqsave(&spans[maint.spanno]->lock, flags);
+			spin_lock_irqsave(&s->lock, flags);
 			break;
 		case DAHDI_MAINT_LOOPUP:
 		case DAHDI_MAINT_LOOPDOWN:
-			spans[maint.spanno]->mainttimer = DAHDI_LOOPCODE_TIME * DAHDI_CHUNKSIZE;
-			rv = spans[maint.spanno]->ops->maint(spans[maint.spanno], maint.command);
-			spin_unlock_irqrestore(&spans[maint.spanno]->lock, flags);
+			s->mainttimer = DAHDI_LOOPCODE_TIME * DAHDI_CHUNKSIZE;
+			rv = s->ops->maint(s, maint.command);
+			spin_unlock_irqrestore(&s->lock, flags);
 			if (rv)
 				return rv;
-			rv = schluffen(&spans[maint.spanno]->maintq);
+			rv = schluffen(&s->maintq);
 			if (rv)
 				return rv;
-			spin_lock_irqsave(&spans[maint.spanno]->lock, flags);
+			spin_lock_irqsave(&s->lock, flags);
 			break;
 		case DAHDI_MAINT_FAS_DEFECT:
 		case DAHDI_MAINT_MULTI_DEFECT:
@@ -4661,26 +4679,23 @@
 			   maintenance functions, unless the driver is
 			   already in a maint state */
 			if(!i)
-				spans[maint.spanno]->maintstat = 0;
-
-			rv = spans[maint.spanno]->ops->maint(spans[maint.spanno],
-							     maint.command);
-			spin_unlock_irqrestore(&spans[maint.spanno]->lock,
-					       flags);
+				s->maintstat = 0;
+
+			rv = s->ops->maint(s, maint.command);
+			spin_unlock_irqrestore(&s->lock, flags);
 			if (rv)
 				return rv;
-			spin_lock_irqsave(&spans[maint.spanno]->lock, flags);
+			spin_lock_irqsave(&s->lock, flags);
 			break;
 		default:
-			spin_unlock_irqrestore(&spans[maint.spanno]->lock,
-					       flags);
+			spin_unlock_irqrestore(&s->lock, flags);
 			module_printk(KERN_NOTICE,
 				      "Unknown maintenance event: %d\n",
 				      maint.command);
 			return -ENOSYS;
 		}
-		dahdi_alarm_notify(spans[maint.spanno]);  /* process alarm-related events */
-		spin_unlock_irqrestore(&spans[maint.spanno]->lock, flags);
+		dahdi_alarm_notify(s);  /* process alarm-related events */
+		spin_unlock_irqrestore(&s->lock, flags);
 		break;
 	}
 	case DAHDI_DYNAMIC_CREATE:
@@ -8678,27 +8693,28 @@
 
 	local_irq_save(flags);
 	for (x=0;x<maxspans;x++) {
-		if (spans[x] && (spans[x]->flags & DAHDI_FLAG_RUNNING)) {
-			if (spans[x]->watchcounter == DAHDI_WATCHDOG_INIT) {
+		s = spans[x];
+		if (s && (s->flags & DAHDI_FLAG_RUNNING)) {
+			if (s->watchcounter == DAHDI_WATCHDOG_INIT) {
 				/* Whoops, dead card */
-				if ((spans[x]->watchstate == DAHDI_WATCHSTATE_OK) ||
-					(spans[x]->watchstate == DAHDI_WATCHSTATE_UNKNOWN)) {
-					spans[x]->watchstate = DAHDI_WATCHSTATE_RECOVERING;
-					if (spans[x]->watchdog) {
-						module_printk(KERN_NOTICE, "Kicking span %s\n", spans[x]->name);
-						spans[x]->watchdog(spans[x], DAHDI_WATCHDOG_NOINTS);
+				if ((s->watchstate == DAHDI_WATCHSTATE_OK) ||
+					(s->watchstate == DAHDI_WATCHSTATE_UNKNOWN)) {
+					s->watchstate = DAHDI_WATCHSTATE_RECOVERING;
+					if (s->watchdog) {
+						module_printk(KERN_NOTICE, "Kicking span %s\n", s->name);
+						s->watchdog(spans[x], DAHDI_WATCHDOG_NOINTS);
 					} else {
-						module_printk(KERN_NOTICE, "Span %s is dead with no revival\n", spans[x]->name);
-						spans[x]->watchstate = DAHDI_WATCHSTATE_FAILED;
+						module_printk(KERN_NOTICE, "Span %s is dead with no revival\n", s->name);
+						s->watchstate = DAHDI_WATCHSTATE_FAILED;
 					}
 				}
 			} else {
-				if ((spans[x]->watchstate != DAHDI_WATCHSTATE_OK) &&
-					(spans[x]->watchstate != DAHDI_WATCHSTATE_UNKNOWN))
-						module_printk(KERN_NOTICE, "Span %s is alive!\n", spans[x]->name);
-				spans[x]->watchstate = DAHDI_WATCHSTATE_OK;
-			}
-			spans[x]->watchcounter = DAHDI_WATCHDOG_INIT;
+				if ((s->watchstate != DAHDI_WATCHSTATE_OK) &&
+					(s->watchstate != DAHDI_WATCHSTATE_UNKNOWN))
+						module_printk(KERN_NOTICE, "Span %s is alive!\n", s->name);
+				s->watchstate = DAHDI_WATCHSTATE_OK;
+			}
+			s->watchcounter = DAHDI_WATCHDOG_INIT;
 		}
 	}
 	local_irq_restore(flags);




More information about the dahdi-commits mailing list