[lkml]   [2006]   [May]   [18]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: [PATCH 4/9] namespaces: utsname: switch to using uts namespaces
Serge E. Hallyn wrote:

>I suppose I could insert a separate patchset fixing up the spacing in
>those blocks but making no real changes at all, then apply my patch on
>top of that...?

While you're fixing whitespace, patch 1 and 6 add trailing whitespace.
Use a current version of git to see the warnings.


>>>--- a/arch/mips/kernel/syscall.c
>>>+++ b/arch/mips/kernel/syscall.c
>>>@@ -232,7 +232,7 @@ out:
>>> */
>>> asmlinkage int sys_uname(struct old_utsname __user * name)
>>> {
>>>- if (name && !copy_to_user(name, &system_utsname, sizeof (*name)))
>>>+ if (name && !copy_to_user(name, utsname(), sizeof (*name)))
>>OK, here's my big comment/question. I want to see <nodename> increased to
>>256 bytes (per current POSIX), so each field of struct <variant>_utsname
>>needs be copied individually (I think) instead of doing a single
>>struct copy.
>>I've been working on this for the past few weeks (among other
>>things). Sorry about the timing.
>>I could send patches for this against mainline in a few days,
>>but I'll be glad to listen to how it would be easiest for all of us
>>to handle.
>>I'm probably a little over half done with my patches.
>>They will end up adding a lib/utsname.c that has functions for:
>> put_oldold_unmame() // to user
>> put_old_uname() // to user
>> put_new_uname() // to user
>> put_posix_uname() // to user
>Ok, so long as these functions accept a utsname, we should be able to
>just change what we pass in to these functions to being the namespace's
>utsname, right? Or am I missing the really nasty part?

To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to
More majordomo info at
Please read the FAQ at

 \ /
  Last update: 2006-05-19 05:15    [W:0.065 / U:10.860 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site