[zaptel-commits] kpfleming: trunk r1307 - in /trunk: ./ zaptel.c

[zaptel-commits at lists.digium.com]

Author: kpfleming
Date: Mon Aug 14 15:42:35 2006
New Revision: 1307

URL: http://svn.digium.com/view/zaptel?rev=1307&view=rev
Log:
Merged revisions 1306 via svnmerge from 
https://origsvn.digium.com/svn/zaptel/branches/1.2

........
r1306 | kpfleming | 2006-08-14 15:40:55 -0500 (Mon, 14 Aug 2006) | 2 lines

revert this change until a version that compiles on 2.4 kernels and older 2.6 kernels is made available and tested

........

Modified:
    trunk/   (props changed)
    trunk/zaptel.c

Propchange: trunk/
------------------------------------------------------------------------------
--- branch-1.2-merged (original)
+++ branch-1.2-merged Mon Aug 14 15:42:35 2006
@@ -1,1 +1,1 @@
-/branches/1.2:1-916,918-936,938-949,958,962,970,990,1004,1011,1017,1031,1033,1060,1062,1064-1066,1069,1071,1076,1079,1081,1097,1101,1151,1185,1187,1205,1233,1235-1236,1243,1248,1259,1262,1298,1300,1304
+/branches/1.2:1-916,918-936,938-949,958,962,970,990,1004,1011,1017,1031,1033,1060,1062,1064-1066,1069,1071,1076,1079,1081,1097,1101,1151,1185,1187,1205,1233,1235-1236,1243,1248,1259,1262,1298,1300,1304,1306

Modified: trunk/zaptel.c
URL: http://svn.digium.com/view/zaptel/trunk/zaptel.c?rev=1307&r1=1306&r2=1307&view=diff
==============================================================================
--- trunk/zaptel.c (original)
+++ trunk/zaptel.c Mon Aug 14 15:42:35 2006
@@ -1135,14 +1135,8 @@
 	/* Assumes channel is already locked */
 	if ((zone >= ZT_TONE_ZONE_MAX) || (zone < -1))
 		return -EINVAL;
-
-	/* Since this routine is called both from IRQ as well as from userspace,
-	 * it is possible that we could be called during an IRQ while userspace
-	 * has locked this.  However unlikely, this could possibly cause a
-	 * deadlock. */
-	if (! read_trylock(&zone_lock))
-		return -EWOULDBLOCK;
-
+	
+	read_lock(&zone_lock);
 	if (zone == -1) {
 		zone = default_zone;
 	}
@@ -1153,7 +1147,7 @@
 	} else {
 		res = -ENODATA;
 	}
-
+	
 	read_unlock(&zone_lock);
 	return res;
 }