[Bug libc/2210] New: [PATCH] Move sys/io.h to sysdeps/{i386,x86_64}/

classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|

[Bug libc/2210] New: [PATCH] Move sys/io.h to sysdeps/{i386,x86_64}/

fweimer at redhat dot com
Former discussion in libc-alpha (see
http://sources.redhat.com/ml/libc-alpha/2002-09/msg00153.html) suggests that it
is not a problem if sys/io.h declares iopl() when the system doesn't have it.
Besides, iopl() can be implemented in userland when kernel offers an interface
with similar functionality (for kFreeBSD, we implemented it as a wrapper to
/dev/io).

If we count i386-pc-kfreebsd-gnu and x86_64-pc-kfreebsd-gnu, that'll be at least
5 platforms using a copy of the same sys/io.h file (only with cosmetical
differences).  I think it makes sense to move to sysdeps/i386 and sysdeps/x86_64
so that the file can be shared between them.

Please consider the attached patch.

--
           Summary: [PATCH] Move sys/io.h to sysdeps/{i386,x86_64}/
           Product: glibc
           Version: unspecified
            Status: NEW
          Severity: enhancement
          Priority: P2
         Component: libc
        AssignedTo: drepper at redhat dot com
        ReportedBy: rmh at aybabtu dot com
                CC: glibc-bugs at sources dot redhat dot com


http://sourceware.org/bugzilla/show_bug.cgi?id=2210

------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.
Reply | Threaded
Open this post in threaded view
|

[Bug libc/2210] [PATCH] Move sys/io.h to sysdeps/{i386,x86_64}/

fweimer at redhat dot com

------- Additional Comments From rmh at aybabtu dot com  2006-01-26 08:00 -------
Created an attachment (id=848)
 --> (http://sourceware.org/bugzilla/attachment.cgi?id=848&action=view)
move sys/io.h to sysdeps/{i386,x86_64}

2006-01-26  Robert Millan  <[hidden email]>

        * sysdeps/unix/sysv/linux/i386/sys/io.h:  Move to ...
        * sysdeps/i386/sys/io.h:  ... here.
        * sysdeps/mach/hurd/i386/sys/io.h:  Remove.

        * sysdeps/i386/Dist:  Add sys/io.h.
        * sysdeps/i386/Makefile:  Ditto.
        * sysdeps/mach/hurd/i386/Dist: Remove sys/io.h.
        * sysdeps/mach/hurd/i386/Makefile:  Ditto.
        * sysdeps/unix/sysv/linux/i386/Dist:  Ditto.
        * sysdeps/unix/sysv/linux/i386/Makefile:  Ditto.

        * sysdeps/unix/sysv/linux/x86_64/sys/io.h:  Move to ...
        * sysdeps/x86_64/sys/io.h:  ... here.

        * sysdeps/x86_64/Dist: Add sys/io.h.
        * sysdeps/x86_64/Makefile:  Ditto.
        * sysdeps/unix/sysv/linux/x86_64/Dist: Remove sys/io.h.
        * sysdeps/unix/sysv/linux/x86_64/Makefile:  Ditto.


--


http://sourceware.org/bugzilla/show_bug.cgi?id=2210

------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.
Reply | Threaded
Open this post in threaded view
|

[Bug libc/2210] [PATCH] Move sys/io.h to sysdeps/{i386,x86_64}/

fweimer at redhat dot com
In reply to this post by fweimer at redhat dot com

------- Additional Comments From drepper at redhat dot com  2006-02-03 22:16 -------
And use comments mentioning Linux on Hurd etc?  Right...

Beside, for these kind of system specific headers I have no interest in handling
the fallout of changes just because some platforms wants to be slightly different.

--
           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |WONTFIX


http://sourceware.org/bugzilla/show_bug.cgi?id=2210

------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.