Thread (3 messages) 3 messages, 2 authors, 2008-10-28

Re: [PATCH] reintroduce accept4

flat view

From: Andrew Morton <hidden>
Date: 2008-10-28 04:54:08
Also in: linux-arch, lkml

(really does cc linux-arch this time)

On Mon, 27 Oct 2008 21:22:59 -0700 Ulrich Drepper [off-list ref] wrote:
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Andrew Morton wrote:
quoted
I'll confess to not having a clue what's going on here.
It has been discussed at length.
That's of little use to people trying to understand the git commit.

And it's of little use to people who write the documentation. 
Hopefully Michael has been following this closely enough to get by with
what we have here (ie: nothing).

And it's of little use to people who are trying to review this patch
and who haven't followed this alleged lengthy discussion. (ie: me)

And it's of little use to people who write for sites such as lwn.net,
http://kernelnewbies.org/LinuxChanges etc.

This is not some personal weekend hack project.  It is the Linux
kernel, used by millions around the world.
 accept created file descriptors and we
need flags for tis.
quoted
quoted
#elif __i386__
#define SYS_ACCEPT4     18
#define USE_SOCKETCALL 1
#define SOCK_CLOEXEC O_CLOEXEC
#else
Well.  This doesn't actually agree with the kernel patch.
What doesn't agree?

quoted
I'd suggest that i386 is sufficiently common to warrant its inclusion
in the initial patch.
The x86 code is included.  x86 uses socketcall instead of a syscall.
<wonders why arch/x86/include/asm/unistd_64.h defines
__ARCH_WANT_SYS_SOCKETCALL but doesn't wire up sys_socketcall()>
I changed all paccept occurrences in the tree, not just x86-64.
OK, that covered a lot of architectures.
--
To unsubscribe from this list: send the line "unsubscribe linux-api" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help