If something is worth doing, it's worth doing right.  

This patch set cleans up a bunch of problems with the use of
errstring.  Gone are the strcmps between errstrs, the irritating
conversions of errno string values to errstr (and the resulting double
messages in userspace), and the networking stack's weird reluctance to
throw errors.  All error() calls must have an actual errstring.

Oh, and the kernel now builds with -Werror.

If you spot anything wrong with the patches, let me know.

Barret


The following changes since commit 9f97f2af901210a851a35b03d166c42ad423c728:

  Remove the static_link_warning from glibc (XCC) (2016-02-12 15:02:52 -0500)

are available in the git repository at:

  [email protected]:brho/akaros.git errstr

for you to fetch changes up to 23616f789424da373b764cbd07c8b72051a489df:

  Remove kernel errno string processing (2016-02-14 09:03:22 -0500)

----------------------------------------------------------------
Barret Rhoden (13):
      Build the kernel with -Werror
      Remove bootp
      Throw errors from within tcpctl()
      Throw errors from within ipifcctl()
      Have #ip's protocol ctl()s throw errors
      Have #ip protocol's connect()s throw errors
      Have #ip protocol's announce()s throw errors
      Have #ip protocol's bind()s throw errors
      qio: Use an empty string to mark a closed queue
      Remove the printk format %e
      Outlaw the setting of NULL errstrs
      Remove uses of errno_to_string()
      Remove kernel errno string processing

 Makefile              |   2 +-
 kern/include/error.h  |  16 +----
 kern/include/ip.h     |  24 +++-----
 kern/src/Kbuild       |  12 +---
 kern/src/net/devip.c  |  87 ++++++++------------------
 kern/src/net/dial.c   |   2 +-
 kern/src/net/icmp.c   |  22 ++-----
 kern/src/net/icmp6.c  |  21 +++----
 kern/src/net/ipifc.c  | 168 +++++++++++++++++++++++---------------------------
 kern/src/net/tcp.c    |  72 ++++++++--------------
 kern/src/net/udp.c    |  42 +++++--------
 kern/src/ns/chan.c    |   5 +-
 kern/src/ns/parse.c   |   8 +--
 kern/src/ns/qio.c     |  21 ++++---
 kern/src/ns/sysfile.c |   2 +-
 kern/src/printfmt.c   |  11 ----
 kern/src/process.c    |   5 +-
 kern/src/readline.c   |   2 +-
 kern/src/syscall.c    |   4 +-
 19 files changed, 193 insertions(+), 333 deletions(-)

-- 
You received this message because you are subscribed to the Google Groups 
"Akaros" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To post to this group, send email to [email protected].
For more options, visit https://groups.google.com/d/optout.

Reply via email to