Re: [PATCH 4/9] namespaces: utsname: switch to using uts namespaces
From: Sam Vilain
Date: Thu May 18 2006 - 23:12:23 EST
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.
>>>@@ -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
>>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
>>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 majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/