mailing list of musl libc
 help / color / mirror / code / Atom feed
From: Rich Felker <dalias@libc.org>
To: musl@lists.openwall.com
Subject: Re: [PATCH 00/16] sys/ioctl.h and termios.h cleanups
Date: Sun, 3 Jul 2016 15:10:48 -0400	[thread overview]
Message-ID: <20160703191048.GG15995@brightrain.aerifal.cx> (raw)
In-Reply-To: <cover.1467552244.git.nsz@port70.net>

On Sun, Jul 03, 2016 at 05:01:24PM +0200, Szabolcs Nagy wrote:
> the contents of sys/ioctl.h is linux specific and not defined by any
> standard, the defacto standard is whatever glibc sys/ioctl.h does.
> 
> termios.h is defined by posix, but there are various extensions.
> glibc does not include linux uapi headers, but contains a copy of
> most asm/termbits.h definitions etc.
> 
> this is v2 of the patchset, changes:
> - add powerpc64 and mipsn32 changes
> - add *speed members to mips termios struct
> 
> Szabolcs Nagy (16):
>   use the generic ioctl.h for x86_64, x32 and aarch64
>   remove linux asm/sockios.h macros from ioctl.h
>   remove ioctl macros that were removed from linux uapi
>   add missing SIOCSIFNAME from linux/sockios.h to ioctl.h
>   add missing TIOC* macros to ioctl.h
>   remove mips and powerpc ioctls that are missing from linux uapi
>   fix mips, mips64, mipsn32 TIOCM_* macros in ioctl.h
>   fix TIOCMSET in mips ioctl.h
>   remove termios2 related ioctls from sh ioctl.h
>   fix FIOQSIZE in arm ioctl.h
>   fix generic termios.h to be more consistent with glibc
>   fix powerpc termios.h to be more consistent with glibc
>   fix mips termios.h to be more consistent with glibc
>   fix TCS* definitions in mips termios.h
>   fix CBAUDEX in powerpc termios.h
>   fix struct termios in mips64 and mipsn32 termios.h

Thanks! I'm merging all of these except #2, which broke the build of
musl itself by eliminating SIOCATMARK, and #16, for which I already
had my own version with a much more detailed commit message discussing
the motivation.

#11-13 also break musl itself (CBAUD needed) but I'm introducing an
extra commit before them to use the _BSD_SOURCE FTM. I also renamed
them (as discussed on irc) to make it clear that the goal is namespace
correctness and not just matching glibc (which would only be correct
if glibc is more right, which it is in this case).

#2 could still be considered at some point, but if we do that we need
to find a way to expose to musl the arch-specific SIOCATMARK it needs
for internal use.

Rich


  parent reply	other threads:[~2016-07-03 19:10 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-07-03 15:01 Szabolcs Nagy
2016-07-03 15:02 ` [PATCH 01/16] use the generic ioctl.h for x86_64, x32 and aarch64 Szabolcs Nagy
2016-07-03 15:02 ` [PATCH 02/16] remove linux asm/sockios.h macros from ioctl.h Szabolcs Nagy
2016-07-03 15:03 ` [PATCH 03/16] remove ioctl macros that were removed from linux uapi Szabolcs Nagy
2016-07-03 15:03 ` [PATCH 04/16] add missing SIOCSIFNAME from linux/sockios.h to ioctl.h Szabolcs Nagy
2016-07-03 15:04 ` [PATCH 05/16] add missing TIOC* macros " Szabolcs Nagy
2016-07-03 15:05 ` [PATCH 06/16] remove mips and powerpc ioctls that are missing from linux uapi Szabolcs Nagy
2016-07-03 15:05 ` [PATCH 07/16] fix mips, mips64, mipsn32 TIOCM_* macros in ioctl.h Szabolcs Nagy
2016-07-03 15:06 ` [PATCH 08/16] fix TIOCMSET in mips ioctl.h Szabolcs Nagy
2016-07-03 15:06 ` [PATCH 09/16] remove termios2 related ioctls from sh ioctl.h Szabolcs Nagy
2016-07-03 15:07 ` [PATCH 10/16] fix FIOQSIZE in arm ioctl.h Szabolcs Nagy
2016-07-03 15:07 ` [PATCH 11/16] fix generic termios.h to be more consistent with glibc Szabolcs Nagy
2016-07-03 15:08 ` [PATCH 12/16] fix powerpc " Szabolcs Nagy
2016-07-03 15:08 ` [PATCH 13/16] fix mips " Szabolcs Nagy
2016-07-03 15:09 ` [PATCH 14/16] fix TCS* definitions in mips termios.h Szabolcs Nagy
2016-07-03 15:09 ` [PATCH 15/16] fix CBAUDEX in powerpc termios.h Szabolcs Nagy
2016-07-03 15:10 ` [PATCH 16/16] fix struct termios in mips64 and mipsn32 termios.h Szabolcs Nagy
2016-07-03 19:10 ` Rich Felker [this message]
  -- strict thread matches above, loose matches on Subject: below --
2016-04-10 12:09 [PATCH 00/16] sys/ioctl.h and termios.h cleanups Szabolcs Nagy

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20160703191048.GG15995@brightrain.aerifal.cx \
    --to=dalias@libc.org \
    --cc=musl@lists.openwall.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://git.vuxu.org/mirror/musl/

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).