Re: [PATCH 0/5] UM: Fine-tuning for some function implementations

2017-01-19 Thread Jeff Dike
The kmalloc(sizeof(struct foo), ...) => kmalloc(sizeof(*foo), ...) ones are OK. The rest is cargo-cult programming. Jeff -- Jeff Dike AddToIt 978-254-0789 (o) 978-394-8986 (c)

Re: [PATCH 0/5] UM: Fine-tuning for some function implementations

2017-01-19 Thread Jeff Dike
The kmalloc(sizeof(struct foo), ...) => kmalloc(sizeof(*foo), ...) ones are OK. The rest is cargo-cult programming. Jeff -- Jeff Dike AddToIt 978-254-0789 (o) 978-394-8986 (c)

Re: [PATCH 1/5] um: port: Move an assignment for the variable "fd" in port_wait()

2017-01-18 Thread Jeff Dike
he inline fd = -ERESTARTSYS; is likely free, or close to it, and I believe that having anything besides a goto inside the if could make it significantly more expensive. Jeff -- Jeff Dike AddToIt 978-254-0789 (o) 978-394-8986 (c)

Re: [PATCH 1/5] um: port: Move an assignment for the variable "fd" in port_wait()

2017-01-18 Thread Jeff Dike
he inline fd = -ERESTARTSYS; is likely free, or close to it, and I believe that having anything besides a goto inside the if could make it significantly more expensive. Jeff -- Jeff Dike AddToIt 978-254-0789 (o) 978-394-8986 (c)

Re: [PATCH 2/5] um: port: Delete three error messages for a failed memory allocation

2017-01-18 Thread Jeff Dike
fd = fd, I don't see how this eliminates a possible error. It should behave exactly the same. To me, this is an expressiveness issue. !x is something you use with something that is conceptually a Boolean. x == NULL is a question about a pointer, which is the case here. Jeff -- Jeff Dike AddToIt 978-254-0789 (o) 978-394-8986 (c)

Re: [PATCH 2/5] um: port: Delete three error messages for a failed memory allocation

2017-01-18 Thread Jeff Dike
fd = fd, I don't see how this eliminates a possible error. It should behave exactly the same. To me, this is an expressiveness issue. !x is something you use with something that is conceptually a Boolean. x == NULL is a question about a pointer, which is the case here. Jeff -- Jeff Dike AddToIt 978-254-0789 (o) 978-394-8986 (c)

Re: Should a x86 gdb work under an x86_64 kernel?

2008-02-25 Thread Jeff Dike
On Mon, Feb 25, 2008 at 09:49:43AM -0500, Theodore Ts'o wrote: > I've noticed that if I try to use a 32-bit x86 gdb to debug a > 32-bit program under a 64-bit x86_64 kernel, gdb (version 6.6-debian, > from Ubuntu Gutsy) immediately core dumps as soon as I run the test > program under the

Re: Should a x86 gdb work under an x86_64 kernel?

2008-02-25 Thread Jeff Dike
On Mon, Feb 25, 2008 at 09:49:43AM -0500, Theodore Ts'o wrote: I've noticed that if I try to use a 32-bit x86 gdb to debug a 32-bit program under a 64-bit x86_64 kernel, gdb (version 6.6-debian, from Ubuntu Gutsy) immediately core dumps as soon as I run the test program under the

Re: [bug] uml doesn't boot under 2.6.25-rc1 host (was Re: 2.6.24-mm1 bugs)

2008-02-21 Thread Jeff Dike
On Thu, Feb 21, 2008 at 06:20:23PM +0100, Miklos Szeredi wrote: > Bisected it down to > > good e7b5e11eaaa8ef93a34e68016de51152d0d62911 > bad bde6f5f59c2b2b48a7a849c129d5b48838fe77ee > > I strongly suspect it's one of the ptrace cleanup patches. Roland, > could you please have a look? I

Re: [bug] uml doesn't boot under 2.6.25-rc1 host (was Re: 2.6.24-mm1 bugs)

2008-02-21 Thread Jeff Dike
On Thu, Feb 21, 2008 at 06:20:23PM +0100, Miklos Szeredi wrote: Bisected it down to good e7b5e11eaaa8ef93a34e68016de51152d0d62911 bad bde6f5f59c2b2b48a7a849c129d5b48838fe77ee I strongly suspect it's one of the ptrace cleanup patches. Roland, could you please have a look? I agree.

Re: [PATCH 09/16] um: use get_personality()

2008-02-20 Thread Jeff Dike
On Wed, Feb 20, 2008 at 07:19:13PM +0800, WANG Cong wrote: > Signed-off-by: WANG Cong <[EMAIL PROTECTED]> Looks good - you should add some sort of changelog though. Jeff -- Work email - jdike at linux dot intel dot com -- To unsubscribe from this list: send the

Re: [PATCH 09/16] um: use get_personality()

2008-02-20 Thread Jeff Dike
On Wed, Feb 20, 2008 at 07:19:13PM +0800, WANG Cong wrote: Signed-off-by: WANG Cong [EMAIL PROTECTED] Looks good - you should add some sort of changelog though. Jeff -- Work email - jdike at linux dot intel dot com -- To unsubscribe from this list: send the

Re: 2.6.24-mm1 bugs

2008-02-15 Thread Jeff Dike
On Fri, Feb 15, 2008 at 08:52:04PM +0100, Miklos Szeredi wrote: > > What does it do? > > See below. > [0.42] Checking host MADV_REMOVE support...<3>MADV_REMOVE failed, err > = -38 Where'd MADV_REMOVE go? I have it on 2.6.24. > [0.42] Failed to get registers from stub, errno =

Re: 2.6.24-mm1 bugs

2008-02-15 Thread Jeff Dike
On Fri, Feb 15, 2008 at 12:43:45PM +0100, Miklos Szeredi wrote: > - UML doesn't boot: guest is 2.6.24-mm1 also, haven't tried any >other. Same guest boots fine on 2.6.24 host. What does it do? Any chance you can bisect it? Jeff -- Work email - jdike at

[RFC PATCH] Address spaces as independent objects

2008-02-15 Thread Jeff Dike
These are against 2.6.24. Build both host and guest from this tree. Jeff -- Work email - jdike at linux dot intel dot com commit 8ebb7e2d1636f0fca44caaab936e9bfe21ae515b Author: Jeff Dike <[EMAIL PROTECTED]> Date: Mon Feb 4 15:38:02 2008 -0500 Host

Re: [uml-devel] [Patch] arch/um/kernel/um_arch.c: some small improvements

2008-02-15 Thread Jeff Dike
On Fri, Feb 15, 2008 at 10:07:42PM +0800, WANG Cong wrote: > > Make some small improvements for arch/um/kernel/um_arch.c. > > Signed-off-by: WANG Cong <[EMAIL PROTECTED]> > Cc: Jeff Dike <[EMAIL PROTECTED]> ACK Jeff -- Work email - jdike at linux dot inte

Re: [uml-devel] [Patch] arch/um/kernel/um_arch.c: some small improvements

2008-02-15 Thread Jeff Dike
On Fri, Feb 15, 2008 at 10:07:42PM +0800, WANG Cong wrote: Make some small improvements for arch/um/kernel/um_arch.c. Signed-off-by: WANG Cong [EMAIL PROTECTED] Cc: Jeff Dike [EMAIL PROTECTED] ACK Jeff -- Work email - jdike at linux dot intel dot com -- To unsubscribe from

Re: 2.6.24-mm1 bugs

2008-02-15 Thread Jeff Dike
On Fri, Feb 15, 2008 at 12:43:45PM +0100, Miklos Szeredi wrote: - UML doesn't boot: guest is 2.6.24-mm1 also, haven't tried any other. Same guest boots fine on 2.6.24 host. What does it do? Any chance you can bisect it? Jeff -- Work email - jdike at

Re: 2.6.24-mm1 bugs

2008-02-15 Thread Jeff Dike
On Fri, Feb 15, 2008 at 08:52:04PM +0100, Miklos Szeredi wrote: What does it do? See below. [0.42] Checking host MADV_REMOVE support...3MADV_REMOVE failed, err = -38 Where'd MADV_REMOVE go? I have it on 2.6.24. [0.42] Failed to get registers from stub, errno = 3 [

[RFC PATCH] Address spaces as independent objects

2008-02-15 Thread Jeff Dike
-linux-develm=120223005624238q=raw These are against 2.6.24. Build both host and guest from this tree. Jeff -- Work email - jdike at linux dot intel dot com commit 8ebb7e2d1636f0fca44caaab936e9bfe21ae515b Author: Jeff Dike [EMAIL PROTECTED] Date: Mon Feb 4 15

Re: 2.6.24.2 won't compile UML

2008-02-14 Thread Jeff Dike
On Thu, Feb 14, 2008 at 12:13:09PM +0100, Ph. Marek wrote: > make -C linux-2.6.24.2/ O=output_path/build ARCH=um bzImage > make[2]: *** No rule to make target `bzImage'. Stop. This seems pretty clear, no? bzImage is a x86-ism. Just leave it off, and you'll get linux and vmlinux.

Re: 2.6.24.2 won't compile UML

2008-02-14 Thread Jeff Dike
On Thu, Feb 14, 2008 at 12:13:09PM +0100, Ph. Marek wrote: make -C linux-2.6.24.2/ O=output_path/build ARCH=um bzImage make[2]: *** No rule to make target `bzImage'. Stop. This seems pretty clear, no? bzImage is a x86-ism. Just leave it off, and you'll get linux and vmlinux.

[PATCH] asm-*/futex.h should include linux/uaccess.h

2008-02-13 Thread Jeff Dike
, ia64, mips, and powerpc. Signed-off-by: Jeff Dike <[EMAIL PROTECTED]> --- include/asm-generic/futex.h |2 +- include/asm-ia64/futex.h|2 +- include/asm-mips/futex.h|2 +- include/asm-parisc/futex.h |2 +- include/asm-powerpc/futex.h |2 +- include/asm-sh/f

[PATCH 3/4] UML - Fix helper_wait calls in watchdog

2008-02-12 Thread Jeff Dike
rivers/harddog_user.c:82: error: too many arguments to function 'helper_wait' /arch/um/drivers/harddog_user.c:89: error: too many arguments to function 'helper_wait' Signed-off-by: Johann Felix Soden <[EMAIL PROTECTED]> Signed-off-by: Jeff Dike <[EMAIL PROTECTED]> --- arch/um/drivers/harddog_use

[PATCH 1/4] UML - Fix initrd printk

2008-02-12 Thread Jeff Dike
From: Johann Felix Soden <[EMAIL PROTECTED]> If the initrd file has zero-length, the error message should contain the filepath. Cc: WANG Cong <[EMAIL PROTECTED]> Signed-off-by: Johann Felix Soden <[EMAIL PROTECTED]> Signed-off-by: Jeff Dike <[EMAIL PROTECTED]> --- arch/u

[PATCH 4/4] UML - Fix FP register corruption

2008-02-12 Thread Jeff Dike
forever. This patch saves the FP state before entering the SIGSEGV handler and restores it afterwards. Signed-off-by: Jeff Dike <[EMAIL PROTECTED]> --- arch/um/include/registers.h |2 ++ arch/um/include/sysdep-i386/ptrace_user.h |3 +++ arch/um/include/sysdep-

[PATCH 2/4] UML - Remove unused sigcontext accessors

2008-02-12 Thread Jeff Dike
The macros which extract registers from a struct sigcontext are no longer needed and can be removed. They are starting not to build anyway, given the removal of the 'e' and 'r' from register names during the x86 merge. Cc: Jiri Olsa <[EMAIL PROTECTED]> Signed-off-by: Jeff Dike <[EMAIL

[PATCH 0/4] UML - Four for 2.6.25

2008-02-12 Thread Jeff Dike
These should go to Linus - there are three build fixes and a serious bug fix. Jeff -- Work email - jdike at linux dot intel dot com -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More

[PATCH 3/4] UML - Fix helper_wait calls in watchdog

2008-02-12 Thread Jeff Dike
:82: error: too many arguments to function 'helper_wait' /arch/um/drivers/harddog_user.c:89: error: too many arguments to function 'helper_wait' Signed-off-by: Johann Felix Soden [EMAIL PROTECTED] Signed-off-by: Jeff Dike [EMAIL PROTECTED] --- arch/um/drivers/harddog_user.c |4 ++-- 1 files

[PATCH 1/4] UML - Fix initrd printk

2008-02-12 Thread Jeff Dike
From: Johann Felix Soden [EMAIL PROTECTED] If the initrd file has zero-length, the error message should contain the filepath. Cc: WANG Cong [EMAIL PROTECTED] Signed-off-by: Johann Felix Soden [EMAIL PROTECTED] Signed-off-by: Jeff Dike [EMAIL PROTECTED] --- arch/um/kernel/initrd.c |2 +- 1

[PATCH 0/4] UML - Four for 2.6.25

2008-02-12 Thread Jeff Dike
These should go to Linus - there are three build fixes and a serious bug fix. Jeff -- Work email - jdike at linux dot intel dot com -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to [EMAIL PROTECTED] More

Re: [PATCH] slob: fix linking for user mode linux

2008-02-11 Thread Jeff Dike
On Mon, Feb 11, 2008 at 02:44:21PM -0800, Christoph Lameter wrote: > UML defined its own external __kmalloc and things. Isnt there some other > way to fix it? I guess including slab.h is not possible here? This is definitely dubious code on my part and I wouldn't support Pekka's patch unless

Re: [PATCH] slob: fix linking for user mode linux

2008-02-11 Thread Jeff Dike
On Mon, Feb 11, 2008 at 02:44:21PM -0800, Christoph Lameter wrote: UML defined its own external __kmalloc and things. Isnt there some other way to fix it? I guess including slab.h is not possible here? This is definitely dubious code on my part and I wouldn't support Pekka's patch unless

[PATCH] UML - update defconfig

2008-02-07 Thread Jeff Dike
[ This is 2.6.25 fodder ] Update defconfig. Cc: Christoph Hellwig <[EMAIL PROTECTED]> Signed-off-by: Jeff Dike <[EMAIL PROTECTED]> --- arch/um/defconfig | 280 -- 1 file changed, 187 insertions(+), 93 deletions(-) Index: linux-

[PATCH] UML - update defconfig

2008-02-07 Thread Jeff Dike
[ This is 2.6.25 fodder ] Update defconfig. Cc: Christoph Hellwig [EMAIL PROTECTED] Signed-off-by: Jeff Dike [EMAIL PROTECTED] --- arch/um/defconfig | 280 -- 1 file changed, 187 insertions(+), 93 deletions(-) Index: linux-2.6-git/arch/um

[PATCH 3/5] UML - Improved error handling while locating temp dir

2008-02-06 Thread Jeff Dike
eleted obsolete comment ] Signed-off-by: Jim Meyering <[EMAIL PROTECTED]> Signed-off-by: Jeff Dike <[EMAIL PROTECTED]> --- arch/um/os-Linux/mem.c | 15 ++- 1 file changed, 6 insertions(+), 9 deletions(-) Index: linux-2.6-git/arch

[PATCH 2/5] UML - Style fixes in arch/um/os-Linux

2008-02-06 Thread Jeff Dike
is no longer needed. create_tmp_file was also made static. checkpatch moans about an EXPORT_SYMBOL in user_syms.c which is part of a macro definition - this is copying a bit of kernel infrastructure into the libc side of UML because the kernel headers can't be included there. Signed-off-by: Jeff Dike

[PATCH 1/5] UML - Runtime host VMSPLIT detection

2008-02-06 Thread Jeff Dike
into. Signed-off-by: Jeff Dike <[EMAIL PROTECTED]> --- arch/um/Kconfig | 11 -- arch/um/Kconfig.i386| 37 - arch/um/Kconfig.x86_64 |4 - arch/um/Makefile| 11 -- arch/um/def

[PATCH 0/5] UML - Five for 2.6.25

2008-02-06 Thread Jeff Dike
This batch should go into the 2.6.25 merge window. There are a couple of patches which had been sitting in -mm waiting for this merge window, but got lost somehow: runtime host vmsplit detection - Miklos spotted a bug, since fixed, with a 32-bit UML on a 64-bit host style fixes in

[PATCH 5/5] UML - Fix mm_context memory leak

2008-02-06 Thread Jeff Dike
[ Spotted by Miklos ] Fix a memory leak in init_new_context. The struct page ** buffer allocated for install_special_mapping was never recorded, and thus leaked when the mm_struct was freed. Fix it by saving the pointer in mm_context_t and freeing it in arch_exit_mmap. Signed-off-by: Jeff Dike

[PATCH 4/5] UML - x86_64 should copy %fs during fork

2008-02-06 Thread Jeff Dike
%fs needs to be copied from parent to child during fork. Tidied up some whitespace while I was here. Signed-off-by: Jeff Dike <[EMAIL PROTECTED]> --- include/asm-um/processor-x86_64.h |3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) Index: linux-2.6-git/include/asm-um/pro

[PATCH 4/5] UML - x86_64 should copy %fs during fork

2008-02-06 Thread Jeff Dike
%fs needs to be copied from parent to child during fork. Tidied up some whitespace while I was here. Signed-off-by: Jeff Dike [EMAIL PROTECTED] --- include/asm-um/processor-x86_64.h |3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) Index: linux-2.6-git/include/asm-um/processor-x86_64

[PATCH 0/5] UML - Five for 2.6.25

2008-02-06 Thread Jeff Dike
This batch should go into the 2.6.25 merge window. There are a couple of patches which had been sitting in -mm waiting for this merge window, but got lost somehow: runtime host vmsplit detection - Miklos spotted a bug, since fixed, with a 32-bit UML on a 64-bit host style fixes in

[PATCH 5/5] UML - Fix mm_context memory leak

2008-02-06 Thread Jeff Dike
[ Spotted by Miklos ] Fix a memory leak in init_new_context. The struct page ** buffer allocated for install_special_mapping was never recorded, and thus leaked when the mm_struct was freed. Fix it by saving the pointer in mm_context_t and freeing it in arch_exit_mmap. Signed-off-by: Jeff Dike

[PATCH 2/5] UML - Style fixes in arch/um/os-Linux

2008-02-06 Thread Jeff Dike
is no longer needed. create_tmp_file was also made static. checkpatch moans about an EXPORT_SYMBOL in user_syms.c which is part of a macro definition - this is copying a bit of kernel infrastructure into the libc side of UML because the kernel headers can't be included there. Signed-off-by: Jeff Dike

[PATCH 1/5] UML - Runtime host VMSPLIT detection

2008-02-06 Thread Jeff Dike
into. Signed-off-by: Jeff Dike [EMAIL PROTECTED] --- arch/um/Kconfig | 11 -- arch/um/Kconfig.i386| 37 - arch/um/Kconfig.x86_64 |4 - arch/um/Makefile| 11 -- arch/um/defconfig

[PATCH 3/5] UML - Improved error handling while locating temp dir

2008-02-06 Thread Jeff Dike
comment ] Signed-off-by: Jim Meyering [EMAIL PROTECTED] Signed-off-by: Jeff Dike [EMAIL PROTECTED] --- arch/um/os-Linux/mem.c | 15 ++- 1 file changed, 6 insertions(+), 9 deletions(-) Index: linux-2.6-git/arch/um/os-Linux/mem.c

Re: [PATCH] uml: handle unusual results from find_tempdir.

2008-02-05 Thread Jeff Dike
On Tue, Feb 05, 2008 at 05:25:06PM +0100, Jim Meyering wrote: > > An alternative: make find_tempdir set tempdir to default_tempdir > upon malloc failure. > > * arch/um/os-Linux/mem.c (make_tempfile): Handle NULL tempdir. > Don't let a long tempdir (e.g., via TMPDIR) provoke heap corruption. > >

Re: [-mm Patch] arch/um/kernel/initrd.c: fix a missed conversion specifier

2008-02-05 Thread Jeff Dike
On Wed, Feb 06, 2008 at 12:25:57AM +0800, WANG Cong wrote: > > Fix a missed conversion specifier of a printk in > arch/um/kernel/initrd.c. > > Signed-off-by: WANG Cong <[EMAIL PROTECTED]> > Cc: Jeff Dike <[EMAIL PROTECTED]> ACK - 2.6.25 material. Sigh.

Re: [-mm Patch] arch/um/kernel/initrd.c: fix a missed conversion specifier

2008-02-05 Thread Jeff Dike
On Wed, Feb 06, 2008 at 12:25:57AM +0800, WANG Cong wrote: Fix a missed conversion specifier of a printk in arch/um/kernel/initrd.c. Signed-off-by: WANG Cong [EMAIL PROTECTED] Cc: Jeff Dike [EMAIL PROTECTED] ACK - 2.6.25 material. Sigh. Jeff -- To unsubscribe from

Re: [PATCH] uml: handle unusual results from find_tempdir.

2008-02-05 Thread Jeff Dike
On Tue, Feb 05, 2008 at 05:25:06PM +0100, Jim Meyering wrote: An alternative: make find_tempdir set tempdir to default_tempdir upon malloc failure. * arch/um/os-Linux/mem.c (make_tempfile): Handle NULL tempdir. Don't let a long tempdir (e.g., via TMPDIR) provoke heap corruption.

Re: [git Patch] UML: a build error fix

2008-02-01 Thread Jeff Dike
used on x86_64 is gone. arch/um/include/sysdep-i386/syscalls.h got some formatting since I was looking at it. Signed-off-by: Jeff Dike <[EMAIL PROTECTED]> Cc: WANG Cong <[EMAIL PROTECTED]> --- arch/um/include/sysdep-i386/syscalls.h |5 +++-- arch/um/include/sysdep-x86

Re: [Patch] arch/um/include/init.h: Fix missing macro definitions

2008-02-01 Thread Jeff Dike
ng now for some reason. It turns out that there wasn't a definition of __used. Fixed this by copying the relevant stuff from compiler.h in the userspace case, and including compiler.h in the kernel case. >From WANG Cong <[EMAIL PROTECTED]> - added definition of __section Signed-o

Re: [Patch] arch/um/include/init.h: Fix missing macro definitions

2008-02-01 Thread Jeff Dike
reason. It turns out that there wasn't a definition of __used. Fixed this by copying the relevant stuff from compiler.h in the userspace case, and including compiler.h in the kernel case. From WANG Cong [EMAIL PROTECTED] - added definition of __section Signed-off-by: Jeff Dike [EMAIL PROTECTED] Cc

Re: [git Patch] UML: a build error fix

2008-02-01 Thread Jeff Dike
is gone. arch/um/include/sysdep-i386/syscalls.h got some formatting since I was looking at it. Signed-off-by: Jeff Dike [EMAIL PROTECTED] Cc: WANG Cong [EMAIL PROTECTED] --- arch/um/include/sysdep-i386/syscalls.h |5 +++-- arch/um/include/sysdep-x86_64/kernel-offsets.h |9

[RFC PATCH] Address spaces as independent objects

2008-01-29 Thread Jeff Dike
Below is a patch which allows address spaces to be created, manipulated, and destroyed independently of processes. The additions are two system calls, new_mm and switch_mm /proc//mm PTRACE_SWITCH_MM new_mm() returns a file descriptor referencing a new address space which

[RFC PATCH] Address spaces as independent objects

2008-01-29 Thread Jeff Dike
Below is a patch which allows address spaces to be created, manipulated, and destroyed independently of processes. The additions are two system calls, new_mm and switch_mm /proc/pid/mm PTRACE_SWITCH_MM new_mm() returns a file descriptor referencing a new address space

Re: [PATCH] random - add async I/O support

2008-01-21 Thread Jeff Dike
On Mon, Jan 21, 2008 at 12:43:25PM -0600, Matt Mackall wrote: > This conflicts just about everywhere with my latest code, but I'll fix > that up. Great, thanks. Jeff -- Work email - jdike at linux dot intel dot com -- To unsubscribe from this list: send the line "unsubscribe

[PATCH] random - add async I/O support

2008-01-21 Thread Jeff Dike
if (fcntl(fd, F_SETOWN, getpid()) < 0) { perror("Setting SIGIO"); exit(1); } printf("Drained the pool\n"); read(fd, , sizeof(n)); printf("Found more randomness\n"); return(0); } Signed-off-by: Jeff

[PATCH] random - add async I/O support

2008-01-21 Thread Jeff Dike
(1); } printf(Drained the pool\n); read(fd, n, sizeof(n)); printf(Found more randomness\n); return(0); } Signed-off-by: Jeff Dike [EMAIL PROTECTED] --- drivers/char/random.c | 43 +-- 1 file changed, 33 insertions

Re: [PATCH] random - add async I/O support

2008-01-21 Thread Jeff Dike
On Mon, Jan 21, 2008 at 12:43:25PM -0600, Matt Mackall wrote: This conflicts just about everywhere with my latest code, but I'll fix that up. Great, thanks. Jeff -- Work email - jdike at linux dot intel dot com -- To unsubscribe from this list: send the line unsubscribe

Re: UML is leaking memory in arch_dup_mmap

2008-01-18 Thread Jeff Dike
On Fri, Jan 18, 2008 at 03:42:54PM +0100, Miklos Szeredi wrote: > uml:~# grep arch_dup_mmap /proc/slab_allocators > size-32: 10861 arch_dup_mmap+0x9f/0x130 Whoops - try the patch below. Jeff -- Work email - jdike at linux dot intel dot com Index:

Re: UML is leaking memory in arch_dup_mmap

2008-01-18 Thread Jeff Dike
On Fri, Jan 18, 2008 at 03:42:54PM +0100, Miklos Szeredi wrote: uml:~# grep arch_dup_mmap /proc/slab_allocators size-32: 10861 arch_dup_mmap+0x9f/0x130 Whoops - try the patch below. Jeff -- Work email - jdike at linux dot intel dot com Index:

Re: [-mm Patch] uml: fix a building error

2008-01-17 Thread Jeff Dike
On Thu, Jan 17, 2008 at 03:17:38PM -0800, Pallipadi, Venkatesh wrote: > >#define X X > > > >is a no-op, yes? > > > > Later there is code in generic.h which is doing > #ifndef ioremap_wc > #define ioremap_wc ioremap_nocache > #endif Ah, that makes a bit more sense. It'd be nice if there was less

Re: [-mm Patch] uml: fix a building error

2008-01-17 Thread Jeff Dike
On Thu, Jan 17, 2008 at 01:41:50PM -0800, Venki Pallipadi wrote: > > And while we're on the subject, what's the deal with these, in > > include/asm-x86/io.h? > > > > #define ioremap_wc ioremap_wc > > #define unxlate_dev_mem_ptr unxlate_dev_mem_ptr > > > > If archs want to override the defaults

[PATCH 19/20] UML - port mutex conversion

2008-01-17 Thread Jeff Dike
From: Daniel Walker <[EMAIL PROTECTED]> The port_sem is already used as a mutex since it's using DECLARE_MUTEX(), but the underlying construct is still a semaphore .. This patch switches it over to a struct mutex. Signed-off-by: Daniel Walker <[EMAIL PROTECTED]> Signed-off-by: Jeff

[PATCH 13/20] UML - Spelling fix

2008-01-17 Thread Jeff Dike
From: Joe Perches <[EMAIL PROTECTED]> Spelling fix Signed-off-by: Joe Perches <[EMAIL PROTECTED]> Signed-off-by: Jeff Dike <[EMAIL PROTECTED]> --- arch/um/sys-x86_64/signal.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) Index: linux-2.6-git/arch/um/s

[PATCH 1/20] UML - Runtime detection of host VMSPLIT on i386

2008-01-17 Thread Jeff Dike
into. Signed-off-by: Jeff Dike <[EMAIL PROTECTED]> --- arch/um/Kconfig | 11 -- arch/um/Kconfig.i386| 37 - arch/um/Kconfig.x86_64 |4 - arch/um/Makefile| 11 -- arch/um/def

[PATCH 15/20] UML - Fix infinite mconsole loop

2008-01-17 Thread Jeff Dike
minate the stopping loop and resume the kernel. This is a problem. A better solution is to make the loop infinite and don't leave it until we are explicitly told to. Signed-off-by: Karol Swietlicki <[EMAIL PROTECTED]> Signed-off-by: Jeff Dike <[EMAIL PROTECTED]> --- arch/um/drivers/mcon

[PATCH 10/20] UML - Move register initialization

2008-01-17 Thread Jeff Dike
Calling init_registers inside the skas3 checking causes mysterious crashes if it doesn't happen because the skas3 checking is bypassed. This patch moves it to os_early_checks. Signed-off-by: Jeff Dike <[EMAIL PROTECTED]> --- arch/um/os-Linux/start_up.c | 10 +++--- 1 file chan

[PATCH 2/20] UML - Style fixes in arch/um/os-LInux

2008-01-17 Thread Jeff Dike
is no longer needed. create_tmp_file was also made static. checkpatch moans about an EXPORT_SYMBOL in user_syms.c which is part of a macro definition - this is copying a bit of kernel infrastructure into the libc side of UML because the kernel headers can't be included there. Signed-off-by: Jeff Dike

[PATCH 7/20] UML - Add back CONFIG_HZ

2008-01-17 Thread Jeff Dike
Anvin" <[EMAIL PROTECTED]> Signed-off-by: Jeff Dike <[EMAIL PROTECTED]> --- arch/um/Kconfig|4 include/asm-um/param.h |2 +- 2 files changed, 5 insertions(+), 1 deletion(-) Index: linux-2.6.22/arch/um/Kconfig ===

[PATCH 4/20] UML - Implement O_APPEND

2008-01-17 Thread Jeff Dike
The .a flags in openflags never had an implementation. Signed-off-by: Jeff Dike <[EMAIL PROTECTED]> --- arch/um/os-Linux/file.c |2 ++ 1 file changed, 2 insertions(+) Index: linux-2.6.22/arch/um/os-Linux/file.c === ---

[PATCH 5/20] UML - Remove fakehd

2008-01-17 Thread Jeff Dike
The fakehd switch lost its implementation at some point. Since no one is screaming for it, we might as well remove it. Signed-off-by: Jeff Dike <[EMAIL PROTECTED]> --- arch/um/drivers/ubd_kern.c | 20 1 file changed, 4 insertions(+), 16 deletions(-) Index: linux-

[PATCH 6/20] UML - DEBUG_SHIRQ fixes

2008-01-17 Thread Jeff Dike
ome error cleanup in mconsole_init while I was there. Cc: "Karol Swietlicki" <[EMAIL PROTECTED]> Signed-off-by: Jeff Dike <[EMAIL PROTECTED]> --- arch/um/drivers/line.c |8 +--- arch/um/drivers/mconsole_kern.c |8 +++- 2 files changed, 12 insertions(

[PATCH 8/20] UML - Style fixes in arch/um/sys-x86_64

2008-01-17 Thread Jeff Dike
Style fixes in arch/um/sys-x86_64: updated copyrights CodingStyle fixes added severities to printks which needed them A bunch of functions in sys-*/ptrace_user.c turn out to be unused, so they and their declarations are gone. Signed-off-by: Jeff Dike <[EMAIL PROTEC

[PATCH 0/20] UML - Lots of patches for 2.6.25

2008-01-17 Thread Jeff Dike
This is a lot of cleanup. There are style fixes, printk fixes, spelling fixes, mutex conversions, and small bug fixes. This is all obviously for 2.6.25. Jeff -- Work email - jdike at linux dot intel dot com -- To unsubscribe from this list: send the line

[PATCH 17/20] UML - LDT mutex conversion

2008-01-17 Thread Jeff Dike
From: Daniel Walker <[EMAIL PROTECTED]> The ldt.semaphore conforms to the new struct mutex requirments, so I converted it to use the new API and changed the name. Signed-off-by: Daniel Walker <[EMAIL PROTECTED]> Signed-off-by: Jeff Dike <[EMAIL PROTECTED]> --- arch/um/sys

[PATCH 12/20] UML - Remove TOPDIR

2008-01-17 Thread Jeff Dike
From: WANG Cong <[EMAIL PROTECTED]> TOPDIR is obsolete, use srctree instead. This patch removes TOPDIR from all UML Makefiles. Cc: Sam Ravnborg <[EMAIL PROTECTED]> Signed-off-by: WANG Cong <[EMAIL PROTECTED]> Signed-off-by: Jeff Dike <[EMAIL PROTECTED]> --- arch/um

[PATCH 18/20] UML - mconsole mutex conversion

2008-01-17 Thread Jeff Dike
From: Daniel Walker <[EMAIL PROTECTED]> The plug_mem_mutex is already used as a mutex since it's using DECLARE_MUTEX(), but the underlying construct is still a semaphore .. This patch switches it over to a struct mutex. Signed-off-by: Daniel Walker <[EMAIL PROTECTED]> Signed-off-b

[PATCH 14/20] UML - Remove map_cb

2008-01-17 Thread Jeff Dike
John Reiser noticed that a physical memory region was being mapped twice. This patch fixes that, and it inlines the responsible function, as that had only one caller. Cc: John Reiser <[EMAIL PROTECTED]> Signed-off-by: Jeff Dike <[EMAIL PROTECTED]> --- arch/um/kernel/mem.c |8

[PATCH 11/20] UML - Remove unused fields from mm_context

2008-01-17 Thread Jeff Dike
The 3-level page table fixes forgot to remove a couple now-unused fields from struct mm_context. Signed-off-by: Jeff Dike <[EMAIL PROTECTED]> --- arch/um/include/um_mmu.h |4 1 file changed, 4 deletions(-) Index: linux-2.6.22/arch/um/include/um

[PATCH 20/20] UML - defconfig tweaks

2008-01-17 Thread Jeff Dike
Tweak the UML defconfig - we probably don't need 256 old-style ptys - this slows down udev noticably enable hostfs disable slab debugging - another noticable performance hit Signed-off-by: Jeff Dike <[EMAIL PROTECTED]> --- arch/um/defconfig |6 +++--- 1 file chan

[PATCH 16/20] UML - Use of a public MAC is a warning, not an error

2008-01-17 Thread Jeff Dike
Downgrade one of the MAC validity checks. If it's one that could be possibly assigned to a physical NIC, then nothing will break. So, emit a warning in this case, but keep the requested MAC. Signed-off-by: Jeff Dike <[EMAIL PROTECTED]> --- arch/um/drivers/net_kern.c |9 -

[PATCH 9/20] UML - Add newlines to printks

2008-01-17 Thread Jeff Dike
Some printks were missing newlines. Signed-off-by: Jeff Dike <[EMAIL PROTECTED]> --- arch/um/os-Linux/skas/process.c |8 1 file changed, 4 insertions(+), 4 deletions(-) Index: linux-2.6.22/arch/um/os-Linux/skas/pro

[PATCH 3/20] UML - SMP locking commentary

2008-01-17 Thread Jeff Dike
Add some more commentary about various pieces of global data not needing locking. Also got rid of unmap_physmem since that is no longer used. Signed-off-by: Jeff Dike <[EMAIL PROTECTED]> --- arch/um/drivers/net_kern.c |7 ++- arch/um/include/mem_user.h |1 - arch/um/kernel

Re: [-mm Patch] uml: fix a building error

2008-01-17 Thread Jeff Dike
On Thu, Jan 17, 2008 at 11:38:53AM -0800, Pallipadi, Venkatesh wrote: > Apart from unxlate, there is also ioremap_wc which is defined in the > same way. And while we're on the subject, what's the deal with these, in include/asm-x86/io.h? #define ioremap_wc ioremap_wc #define unxlate_dev_mem_ptr

Re: [-mm Patch] uml: fix a building error

2008-01-17 Thread Jeff Dike
On Thu, Jan 17, 2008 at 07:11:13PM +0100, Mariusz Kozlowski wrote: > I see this on sparc64 as well: > > CC drivers/char/mem.o > drivers/char/mem.c: In function 'read_mem': > drivers/char/mem.c:136: error: implicit declaration of function > 'unxlate_dev_mem_ptr' > make[2]: ***

Re: [-mm Patch] UML: fix a building error

2008-01-17 Thread Jeff Dike
On Thu, Jan 17, 2008 at 09:21:08PM +0800, WANG Cong wrote: > Building uml failed in current -mm tree. ;( > > The below patch fixes this building error: > ... > include/asm/arch/system.h:8:22: error: asm/nops.h: No such file or directory > ... > > Cc: Jeff Dike <[EMA

Re: [-mm Patch] uml: fix a building error

2008-01-17 Thread Jeff Dike
On Thu, Jan 17, 2008 at 09:56:41PM +0800, WANG Cong wrote: > > This patch fixes this building error: > ... > drivers/char/mem.c: In function ‘read_mem’: > drivers/char/mem.c:136: error: implicit declaration of function > ‘unxlate_dev_mem_ptr’ > ... > > Cc: Jeff Dike &

Re: [-mm Patch] uml: fix a building error

2008-01-17 Thread Jeff Dike
On Thu, Jan 17, 2008 at 07:11:13PM +0100, Mariusz Kozlowski wrote: I see this on sparc64 as well: CC drivers/char/mem.o drivers/char/mem.c: In function 'read_mem': drivers/char/mem.c:136: error: implicit declaration of function 'unxlate_dev_mem_ptr' make[2]: ***

Re: [-mm Patch] UML: fix a building error

2008-01-17 Thread Jeff Dike
On Thu, Jan 17, 2008 at 09:21:08PM +0800, WANG Cong wrote: Building uml failed in current -mm tree. ;( The below patch fixes this building error: ... include/asm/arch/system.h:8:22: error: asm/nops.h: No such file or directory ... Cc: Jeff Dike [EMAIL PROTECTED] Signed-off-by: WANG Cong

Re: [-mm Patch] uml: fix a building error

2008-01-17 Thread Jeff Dike
On Thu, Jan 17, 2008 at 09:56:41PM +0800, WANG Cong wrote: This patch fixes this building error: ... drivers/char/mem.c: In function ‘read_mem’: drivers/char/mem.c:136: error: implicit declaration of function ‘unxlate_dev_mem_ptr’ ... Cc: Jeff Dike [EMAIL PROTECTED] Signed-off

[PATCH 3/20] UML - SMP locking commentary

2008-01-17 Thread Jeff Dike
Add some more commentary about various pieces of global data not needing locking. Also got rid of unmap_physmem since that is no longer used. Signed-off-by: Jeff Dike [EMAIL PROTECTED] --- arch/um/drivers/net_kern.c |7 ++- arch/um/include/mem_user.h |1 - arch/um/kernel/mem.c

Re: [-mm Patch] uml: fix a building error

2008-01-17 Thread Jeff Dike
On Thu, Jan 17, 2008 at 11:38:53AM -0800, Pallipadi, Venkatesh wrote: Apart from unxlate, there is also ioremap_wc which is defined in the same way. And while we're on the subject, what's the deal with these, in include/asm-x86/io.h? #define ioremap_wc ioremap_wc #define unxlate_dev_mem_ptr

[PATCH 13/20] UML - Spelling fix

2008-01-17 Thread Jeff Dike
From: Joe Perches [EMAIL PROTECTED] Spelling fix Signed-off-by: Joe Perches [EMAIL PROTECTED] Signed-off-by: Jeff Dike [EMAIL PROTECTED] --- arch/um/sys-x86_64/signal.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) Index: linux-2.6-git/arch/um/sys-x86_64/signal.c

[PATCH 9/20] UML - Add newlines to printks

2008-01-17 Thread Jeff Dike
Some printks were missing newlines. Signed-off-by: Jeff Dike [EMAIL PROTECTED] --- arch/um/os-Linux/skas/process.c |8 1 file changed, 4 insertions(+), 4 deletions(-) Index: linux-2.6.22/arch/um/os-Linux/skas/process.c

[PATCH 11/20] UML - Remove unused fields from mm_context

2008-01-17 Thread Jeff Dike
The 3-level page table fixes forgot to remove a couple now-unused fields from struct mm_context. Signed-off-by: Jeff Dike [EMAIL PROTECTED] --- arch/um/include/um_mmu.h |4 1 file changed, 4 deletions(-) Index: linux-2.6.22/arch/um/include/um_mmu.h

[PATCH 20/20] UML - defconfig tweaks

2008-01-17 Thread Jeff Dike
Tweak the UML defconfig - we probably don't need 256 old-style ptys - this slows down udev noticably enable hostfs disable slab debugging - another noticable performance hit Signed-off-by: Jeff Dike [EMAIL PROTECTED] --- arch/um/defconfig |6 +++--- 1 file changed, 3

[PATCH 17/20] UML - LDT mutex conversion

2008-01-17 Thread Jeff Dike
From: Daniel Walker [EMAIL PROTECTED] The ldt.semaphore conforms to the new struct mutex requirments, so I converted it to use the new API and changed the name. Signed-off-by: Daniel Walker [EMAIL PROTECTED] Signed-off-by: Jeff Dike [EMAIL PROTECTED] --- arch/um/sys-i386/ldt.c | 14

  1   2   3   4   5   6   7   8   9   10   >