LWN.net Logo

Remove racy check_mem_region() call from pcbit/drv.c

From:  Bob Miller <rem@osdl.org>
To:  trivial@rustcorp.com.au
Subject:  [PATCH 2.5.73 TRIVIAL] Remove racy check_mem_region() call from pcbit/drv.c
Date:  Thu, 26 Jun 2003 13:27:40 -0700
Cc:  linux-kernel@vger.kernel.org

Removed the check_mem_region() call and replaced with request_mem_region().

--
Bob Miller                                      Email: rem@osdl.org
Open Source Development Lab                     Phone: 503.626.2455 Ext. 17

diff -Nru a/drivers/isdn/pcbit/drv.c b/drivers/isdn/pcbit/drv.c
--- a/drivers/isdn/pcbit/drv.c	Wed Jun 25 16:18:28 2003
+++ b/drivers/isdn/pcbit/drv.c	Wed Jun 25 16:18:28 2003
@@ -87,15 +87,13 @@
 
 	if (mem_base >= 0xA0000 && mem_base <= 0xFFFFF ) {
 		dev->ph_mem = mem_base;
-		if (check_mem_region(dev->ph_mem, 4096)) {
+		if (!request_mem_region(dev->ph_mem, 4096, "PCBIT mem")) {
 			printk(KERN_WARNING
 				"PCBIT: memory region %lx-%lx already in use\n",
 				dev->ph_mem, dev->ph_mem + 4096);
 			kfree(dev);
 			dev_pcbit[board] = NULL;
 			return -EACCES;
-		} else {
-			request_mem_region(dev->ph_mem, 4096, "PCBIT mem");
 		}
 		dev->sh_mem = (unsigned char*)ioremap(dev->ph_mem, 4096);
 	}
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Copyright © 2003, Eklektix, Inc.
Comments and public postings are copyrighted by their creators.
Linux is a registered trademark of Linus Torvalds