aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMike Frysinger <vapier@gentoo.org>2013-12-24 05:33:28 -0500
committerMike Frysinger <vapier@gentoo.org>2013-12-24 05:34:52 -0500
commitfe1a83aa1cbe166f8534925103c8d3a4f047f26c (patch)
tree0cf90d375e894f4afa949e86819fd9e7565eaf22
parent41b1792698a335d3a85381921a84a16e9635f36a (diff)
downloadglibc-fe1a83aa1cbe166f8534925103c8d3a4f047f26c.tar
glibc-fe1a83aa1cbe166f8534925103c8d3a4f047f26c.tar.gz
glibc-fe1a83aa1cbe166f8534925103c8d3a4f047f26c.tar.bz2
glibc-fe1a83aa1cbe166f8534925103c8d3a4f047f26c.zip
ia64: ioperm: clean up long dead code
This file has a few #if 0 code paths which cause a build time warning: ports/sysdeps/unix/sysv/linux/ia64/ioperm.c:66:7: warning: variable 'prot' set but not used [-Wunused-but-set-variable] Rather than add more #if 0 around that variable, just delete the code altogether. Not like it's going to ever be implemented. Signed-off-by: Mike Frysinger <vapier@gentoo.org>
-rw-r--r--ports/ChangeLog.ia644
-rw-r--r--ports/sysdeps/unix/sysv/linux/ia64/ioperm.c23
2 files changed, 8 insertions, 19 deletions
diff --git a/ports/ChangeLog.ia64 b/ports/ChangeLog.ia64
index 756664c97a..03380477bd 100644
--- a/ports/ChangeLog.ia64
+++ b/ports/ChangeLog.ia64
@@ -1,5 +1,9 @@
2013-12-24 Mike Frysinger <vapier@gentoo.org>
+ * sysdeps/unix/sysv/linux/ia64/ioperm.c: Delete code behind #if 0.
+
+2013-12-24 Mike Frysinger <vapier@gentoo.org>
+
* sysdeps/ia64/sotruss-lib.c: New file.
2013-12-24 Mike Frysinger <vapier@gentoo.org>
diff --git a/ports/sysdeps/unix/sysv/linux/ia64/ioperm.c b/ports/sysdeps/unix/sysv/linux/ia64/ioperm.c
index 83fc557f44..6a8d17c0f4 100644
--- a/ports/sysdeps/unix/sysv/linux/ia64/ioperm.c
+++ b/ports/sysdeps/unix/sysv/linux/ia64/ioperm.c
@@ -59,11 +59,7 @@ io_offset (unsigned long int port)
int
_ioperm (unsigned long int from, unsigned long int num, int turn_on)
{
-#if 0
- unsigned long int addr, len, base;
-#endif
unsigned long int base;
- int prot;
/* this test isn't as silly as it may look like; consider overflows! */
if (from >= MAX_PORT || from + num > MAX_PORT)
@@ -91,14 +87,9 @@ _ioperm (unsigned long int from, unsigned long int num, int turn_on)
return -1;
len = io_offset (MAX_PORT);
-#if 1
/* see comment below */
base = (unsigned long int) __mmap (0, len, PROT_READ | PROT_WRITE, MAP_SHARED,
fd, phys_io_base);
-#else
- base = (unsigned long int) __mmap (0, len, PROT_NONE, MAP_SHARED,
- fd, phys_io_base);
-#endif
__close (fd);
if ((long) base == -1)
@@ -106,25 +97,19 @@ _ioperm (unsigned long int from, unsigned long int num, int turn_on)
io.base = base;
}
- prot = PROT_READ | PROT_WRITE;
}
else
{
if (!io.base)
return 0; /* never was turned on... */
-
- prot = PROT_NONE;
}
-#if 0
+
/* We can't do mprotect because that would cause us to lose the
uncached flag that the /dev/mem driver turned on. A MAP_UNCACHED
- flag seems so much cleaner... */
- addr = (io.base + io_offset (from)) & io.page_mask;
- len = io.base + io_offset (from + num) - addr;
- return mprotect ((void *) addr, len, prot);
-#else
+ flag seems so much cleaner...
+
+ See the history of this file for a version that tried mprotect. */
return 0;
-#endif
}
int