Re: [PATCH -mm -v3 1/2] i386/x86_64 boot: setup data

2007-09-27 Thread Huang, Ying
Hi, Peter, On Wed, 2007-09-19 at 09:04 -0700, H. Peter Anvin wrote: Huang, Ying wrote: Known Issues: 1. Where is safe to place the linked list of setup_data? Because the length of the linked list of setup_data is variable, it can not be copied into BSS segment of kernel as that of

Re: 2.6.23-rc7 - _random_ IRQ23 : nobody cared

2007-09-27 Thread Paul Rolland
Hi Tejun, On Thu, 27 Sep 2007 09:55:22 +0900 Tejun Heo [EMAIL PROTECTED] wrote: Paul Rolland wrote: Hi David, On Mon, 24 Sep 2007 23:56:59 +0930 David Newall [EMAIL PROTECTED] wrote: Paul Rolland (???) wrote: Hell, IRQ 23 is shared between libata and my modem !!!

Re: try_module_get code understanding

2007-09-27 Thread Heiko Carstens
I was going through try_module_get function in include/linux/module.h file (2.6.22 stock kernel) - which is like: - static inline int try_module_get(struct module *module){ int ret = 1; --- error case when !module if (module) { unsigned int cpu = get_cpu();

Re: sata_sil24 broken since 2.6.23-rc4-mm1

2007-09-27 Thread Torsten Kaiser
On 9/27/07, Tejun Heo [EMAIL PROTECTED] wrote: Tejun Heo wrote: Torsten Kaiser wrote: Comparing the driver/ata directory from rc3-mm1 and rc4-mm1 the following change looked the most suspicions to me:

Re: try_module_get code understanding

2007-09-27 Thread Valdis . Kletnieks
On Thu, 27 Sep 2007 05:19:06 -, Shreyansh Jain said: - static inline int try_module_get(struct module *module){ int ret = 1; --- error case when !module if (module) { unsigned int cpu = get_cpu(); if (likely(module_is_live(module)))

Re: sata_sil24 broken since 2.6.23-rc4-mm1

2007-09-27 Thread Jeff Garzik
Torsten Kaiser wrote: On 9/27/07, Tejun Heo [EMAIL PROTECTED] wrote: Tejun Heo wrote: Torsten Kaiser wrote: Comparing the driver/ata directory from rc3-mm1 and rc4-mm1 the following change looked the most suspicions to me:

Re: try_module_get code understanding

2007-09-27 Thread Shreyansh Jain
Thanks for you reply, please see inline. Heiko Carstens heiko.carstens at de.ibm.com writes: [snip] static inline int try_module_get(struct module *module){ int ret = 1; --- error case when !module if (module) { unsigned int cpu = get_cpu(); if

Re: 2.6.23-rc8: Known regressions from 2.6.22

2007-09-27 Thread Meelis Roos
RJW This message lists some known regressions from 2.6.22 for which there are RJW no fixes in the mainline that I know of. If any of them have been fixed RJW already, please let me know. RJW RJW If you know of any other unresolved regressions from 2.6.22, please let me know RJW either and I'll

Re: [RFC][PATCH 1/2 -mm] kexec based hibernation -v3: kexec jump

2007-09-27 Thread Huang, Ying
On Wed, 2007-09-26 at 16:30 -0400, Joseph Fannin wrote: But, in my ignorance, I'm not sure even fixing the ext3 bug will guarantee you consistent metadata so that you can handle a swap/hibernate file. You can do a sync(), but how do you make that not race against running processes without

Re: sys_chroot+sys_fchdir Fix

2007-09-27 Thread David Newall
Adrian Bunk wrote: You claimed: -- snip -- Look, when chroot was being designed, I think they intended that even root should be unable to get out. They went so far as to say that dot-dot wouldn't let you out; and it doesn't. -- snip -- You were clearly saying that whom you call they

Re: [RFC][PATCH 1/2 -mm] kexec based hibernation -v3: kexec jump

2007-09-27 Thread Nigel Cunningham
Hi. On Thursday 27 September 2007 16:33:54 Huang, Ying wrote: On Wed, 2007-09-26 at 16:30 -0400, Joseph Fannin wrote: But, in my ignorance, I'm not sure even fixing the ext3 bug will guarantee you consistent metadata so that you can handle a swap/hibernate file. You can do a sync(),

Re: sys_chroot+sys_fchdir Fix

2007-09-27 Thread Adrian Bunk
On Thu, Sep 27, 2007 at 04:12:53PM +0930, David Newall wrote: Adrian Bunk wrote: You claimed: -- snip -- Look, when chroot was being designed, I think they intended that even root should be unable to get out. They went so far as to say that dot-dot wouldn't let you out; and it doesn't.

Re: sched-devel feedback

2007-09-27 Thread Ingo Molnar
* Andr? Goddard Rosa [EMAIL PROTECTED] wrote: Hi, Ingo , Mike and Peter! Just passing around to say that 2.6.23-rc8-sched-dev is the best scheduler ever to me. It's great for 3D games. cool! :-) http://www.openarena.ws/?files is really great with this scheduler. I played a

Re: Warnings and Oops on 2.6.23-rc6 while activily using rfcomm links (mm/slab.c)

2007-09-27 Thread Cornelia Huck
On Wed, 26 Sep 2007 15:27:10 +0200, Pierre-Yves Paulus [EMAIL PROTECTED] wrote: Please use CONFIG_SLUB and turn on SLUB debugging. Something is very wrong somewhere... I did so, on 2.6.23-rc8. I also did include CONFIG_DEBUG_DRIVER and CONFIG_DEBUG_KOBJECT, following Cornelia Huck's

Re: try_module_get code understanding

2007-09-27 Thread Heiko Carstens
Then what is return value if my module tries to 'get' a module which does not exist (and is a module, not in-built)? . Is it '1' ? Or am I imagining a hypothetical scenario which would not exist? That is not supposed to happen. After a module got unloaded there shouldn't be any objects around

Re: [Announce] Linux-tiny project revival

2007-09-27 Thread Arnd Bergmann
On Thursday 20 September 2007, you wrote: So instead of:   printk(KERN_NOTICE Fruit=%d\n, banana); It would now be:   printk(KERN_NOTICE, Fruit=%d\n, banana); Change the header from:   #define KERN_NOTICE 5 to:   #define KERN_NOTICE 5 Then you can change the printk guts to do

[PATCH 000 of 4] Change prototype for bi_end_io

2007-09-27 Thread NeilBrown
Hi Jens, here are a few more patches from my set that makes various changes to bio submission and handling. These change the -bi_end_io prototype so that 1/ no 'size' is passed 2/ there is no return value. The 'size' is not really of interest to any bi_end_io handler in existance. It

[PATCH 001 of 4] Remove flush_dry_bio_endio

2007-09-27 Thread NeilBrown
The entire function of flush_dry_bio_endio is to undo the effects of bio_endio (when called on a barrier request). So remove the function and the call to bio_endio. This allows us to remove bi_size from struct request_queue. Signed-off-by: Neil Brown [EMAIL PROTECTED] ### Diffstat output

[PATCH 002 of 4] Only call bi_end_io once for any bio.

2007-09-27 Thread NeilBrown
Currently bi_end_io can be called multiple times as sub-requests complete. However no -bi_end_io function wants to know about that. So only call when the bio is complete. Signed-off-by: Neil Brown [EMAIL PROTECTED] ### Diffstat output ./fs/bio.c |4 +++- 1 file changed, 3 insertions(+), 1

[PATCH 003 of 4] Don't decrement bi_size in bio_endio.

2007-09-27 Thread NeilBrown
The only caller of bio_endio that does not pass the full bi_size is end_that_request_first. Also, no -bi_end_io method is really interested in bi_size being decremented. So move the decrement and related code into ll_rw_blk and merge it with order_bio_endio to form req_bio_endio which does

[PATCH 004 of 4] Drop 'size' argument from bio_endio and bi_end_io.

2007-09-27 Thread NeilBrown
As bi_end_io is only called once when the reqeust is complete, the 'size' argument is now redundant. Remove it. Now there is no need for bio_endio to subtract the size completed from bi_size. So don't do that either. While we are at it, change bi_end_io to return void. Signed-off-by: Neil

Re: State of the Linux PCI Subsystem for 2.6.23-rc8

2007-09-27 Thread Jeff Garzik
Greg KH wrote: On Wed, Sep 26, 2007 at 11:40:58PM +0200, Brice Goglin wrote: Greg KH wrote: Here's a summary of the current state of the Linux PCI subsystem, as of 2.6.23-rc8. If the information in here is incorrect, or anyone knows of any outstanding issues not listed here, please let me

Re: sys_chroot+sys_fchdir Fix

2007-09-27 Thread Christer Weinigel
On Thu, 27 Sep 2007 06:49:28 +0930 David Newall [EMAIL PROTECTED] wrote: For sure, a root user can get out of a chroot a million different ways. Young Alan said as much at the beginning of this conversation, and I have always agreed. I don't hope to secure Linux within chroot, simply to

[PATCH] hid: hidraw_connect memleak fix

2007-09-27 Thread Mariusz Kozlowski
It looks like hidraw_connect() is leaking memory in case of failure. Also it should return -ENOMEM when kzalloc fails. Signed-off-by: Mariusz Kozlowski [EMAIL PROTECTED] drivers/hid/hidraw.c | 14 ++ 1 file changed, 10 insertions(+), 4 deletions(-) ---

Re: [PATCH] Module use count must be updated as bridges are created/destroyed

2007-09-27 Thread Jan Beulich
Stephen Hemminger [EMAIL PROTECTED] 26.09.07 19:12 On Wed, 26 Sep 2007 17:08:19 +0100 Jan Beulich [EMAIL PROTECTED] wrote: Stephen Hemminger [EMAIL PROTECTED] 26.09.07 17:37 On Wed, 26 Sep 2007 08:53:27 +0100 Jan Beulich [EMAIL PROTECTED] wrote: Otherwise 'modprobe -r' on a module

[PATCH] sound: snd_register_device_for_dev fix

2007-09-27 Thread Mariusz Kozlowski
Now snd_register_device_for_dev() can oops when device_create() returns ERR_PTR(err). Scenario: preg-dev = device_create(...); /* fails */ if (preg-dev) /* contains ERR_PTR(err) */ dev_set_drvdata(preg-dev, private_data); and dev_set_drvdata() looks like this: static inline void

Re: [git] CFS-devel, latest code

2007-09-27 Thread Ingo Molnar
* Dmitry Adamushko [EMAIL PROTECTED] wrote: humm... I think, it'd be safer to have something like the following change in place. The thing is that __pick_next_entity() must never be called when first_fair(cfs_rq) == NULL. It wouldn't be a problem, should 'run_node' be the very first

Re: kbuild: LDFLAGS_MODULE unusable for external module builds (2.6.23-rc2)

2007-09-27 Thread Henry Nestler
Hello Sam, Henry Nestler wrote: Sam Ravnborg wrote: What macro should set for linker parameters of foo.o ? I'm not shure. Have you read: Documentation/kbuild/makfilefiles.txt? [...] If your example requires the LDFALGS_$@ I wil introduce it - for now it has not been required (except for

Re: [PATCH] Since we have counters in __u64 format we have to print them with %llu macros.

2007-09-27 Thread Balbir Singh
H. Peter Anvin wrote: Balbir Singh wrote: Andreas Schwab wrote: Maxim Uvarov [EMAIL PROTECTED] writes: diff --git a/Documentation/accounting/getdelays.c b/Documentation/accounting/getdelays.c index cbee3a2..73924df 100644 --- a/Documentation/accounting/getdelays.c +++

Re: kbuild: LDFLAGS_MODULE unusable for external module builds (2.6.23-rc2)

2007-09-27 Thread Sam Ravnborg
Hi Henry. Snipped a very detaild an good example... Thanks for your comprehensive feedback on this topic. Idea for future: An 'APPEND_LDFLAGS' would be nice to append flags behind the last .o object. Than can be close the group with --end-group. Exactly linker call would be with grouping

Re: [PATCH 1/5] The config option itself

2007-09-27 Thread Pavel Emelyanov
[EMAIL PROTECTED] wrote: Serge E. Hallyn [EMAIL PROTECTED] wrote: | Quoting Serge E. Hallyn ([EMAIL PROTECTED]): | Quoting Pavel Emelyanov ([EMAIL PROTECTED]): | The option is called NAMESPACES. It can be selectable only | if EMBEDDED is chosen (this was Eric's requisition). When |

Re: [PATCH 1/8] taskstats: fix indentation of long argument lists

2007-09-27 Thread Balbir Singh
Andrew Morton wrote: On Wed, 26 Sep 2007 19:08:18 +0200 Guillaume Chazarain [EMAIL PROTECTED] wrote: Align with the opening parenthesis. Changelog since V1 (http://lkml.org/lkml/2007/9/21/527): - renamed fill_threadgroup() and add_tsk() to respectively fill_threadgroup_stats() and

Re: Man page for revised timerfd API

2007-09-27 Thread Geoff Clare
Michael Kerrisk [EMAIL PROTECTED] wrote, on 26 Sep 2007: .TH TIMERFD_CREATE 2 2007-09-26 Linux Linux Programmer's Manual .SH NAME timerfd_create, timerfd_settime, timer_gettime \- s/timer_/timerfd_/ timers that notify via file descriptors .SH SYNOPSIS .\ FIXME . This header file may well

Re: CFS: new java yield graphs

2007-09-27 Thread Ingo Molnar
* Antoine Martin [EMAIL PROTECTED] wrote: -BEGIN PGP SIGNED MESSAGE- Hash: SHA512 These are pure cpu scheduling tests, not doing any I/O this time. All these tests are still pathological in the sense that they are only meant to show differences between schedulers rather than try

Re: [PATCH 1/8] taskstats: fix indentation of long argument lists

2007-09-27 Thread Guillaume Chazarain
Le Wed, 26 Sep 2007 13:01:44 -0700, Andrew Morton [EMAIL PROTECTED] a écrit : Also, I don't think we're seeing enough review and test from people on these patch series - I don't have time to do it all. (Well, apparently I do, but I don't think it's a good situation). I don't object to your

Re: 2.6.23-rc7 - _random_ IRQ23 : nobody cared

2007-09-27 Thread Benjamin Herrenschmidt
Let me guess... this is a T61 or X61 ? There's a problem with these that we don't fully understand yet, we're getting those stale interrupts all over the range. I wonder if it could be a bug with the ICH8 chipset... If yours is one of these, it's being dealt with (or attempted to deal with) at

[PATCH] pci: use pci=bfsort for HP DL385 G2, DL585 G2

2007-09-27 Thread Michal Schmidt
Hello, HP ProLiant systems DL385 G2 and DL585 G2 need pci=bfsort to enumerate PCI devices in the expected order. (John, can you please confirm and ACK this?) Signed-off-by: Michal Schmidt [EMAIL PROTECTED] diff --git a/arch/i386/pci/common.c b/arch/i386/pci/common.c index ebc6f3c..8737c53

2.6.23-rc8-mm2

2007-09-27 Thread Andrew Morton
ftp://ftp.kernel.org/pub/linux/kernel/people/akpm/patches/2.6/2.6.23-rc8/2.6.23-rc8-mm2/ - The scheduler devel tree has been restored - The driver tree is presently busted, so I reverted it to the 2..23-rc8-mm1 version. - It's now a nearly-32MB diff. Boilerplate: - See the `hot-fixes'

Updating include/asm symlink when $ARCH changes

2007-09-27 Thread Martin Michlmayr
I cross compile arm and mips kernels from the same kernel tree. When I build a kernel the first time with a fresh kernel tree, the include/asm symlink is set properly. However, when I compile for a different $ARCH, the include/asm is not changed and the build fails. Would it be possible for the

Re: Network slowdown due to CFS

2007-09-27 Thread Jarek Poplawski
On 26-09-2007 15:31, Ingo Molnar wrote: * David Schwartz [EMAIL PROTECTED] wrote: I think the real fix would be for iperf to use blocking network IO though, or maybe to use a POSIX mutex or POSIX semaphores. So it's definitely not a bug in the kernel, only in iperf? Martin: Actually, in

Re: [PATCH] hid: hidraw_connect memleak fix

2007-09-27 Thread Jiri Kosina
On Thu, 27 Sep 2007, Mariusz Kozlowski wrote: It looks like hidraw_connect() is leaking memory in case of failure. Also it should return -ENOMEM when kzalloc fails. Mariusz, good catch, will apply to me tree. Thanks, -- Jiri Kosina SUSE Labs - To unsubscribe from this list: send the line

Re: BUG: unable to handle kernel NULL pointer dereference at virtual address 00000008

2007-09-27 Thread D-Tick
Sorry, my mistake, the link should be http://lkml.org/lkml/2007/7/25/268 memtest was running for one week with no results, i also tried the ram from another box. I replaced all SATA kables half a year ago with no result :/ A friend told me he had problems with a VIA Board when using the hardware

Re: Network slowdown due to CFS

2007-09-27 Thread Ingo Molnar
* Jarek Poplawski [EMAIL PROTECTED] wrote: the (small) patch below fixes the iperf locking bug and removes the yield() use. There are numerous immediate benefits of this patch: ... sched_yield() is almost always the symptom of broken locking or other bug. In that sense CFS does the

Re: Network slowdown due to CFS

2007-09-27 Thread Ingo Molnar
* Martin Michlmayr [EMAIL PROTECTED] wrote: I think the real fix would be for iperf to use blocking network IO though, or maybe to use a POSIX mutex or POSIX semaphores. So it's definitely not a bug in the kernel, only in iperf? (CCing Stephen Hemminger who wrote the iperf patch.)

[PATCH -mm] task_struct: move -fpu_counter and -oomkilladj

2007-09-27 Thread Alexey Dobriyan
There is nice 2 byte hole after struct task_struct::ioprio field into which we can put two 1-byte fields: -fpu_counter and -oomkilladj. [cc'ing Arjan just in case -fpu_counter placement wasn't completely random :^)] Signed-off-by: Alexey Dobriyan [EMAIL PROTECTED] --- include/linux/sched.h |

[PATCH -mm] VT ioctl race fix

2007-09-27 Thread Samuel Ortiz
Hi, When calling the RELDISP VT ioctl, we are reading vt_newvt while the console workqueue could be messing with it (through change_console()). We fix this race by taking the console semaphore before reading vt_newvt. Andrew, would you please consider this patch for -mm inclusion ?

PROBLEM: kernel BUG at net/core/skbuff.c:1341!

2007-09-27 Thread Niccolo Rigacci
1. kernel BUG at net/core/skbuff.c:1341! 2. I got a kernel panic, the hardware is a good IBM 206m server with 2 Gb ECC memory, Memtest86+ run several times without problem. The message on the console is quite explicit, so is this really a kernel bug? The box is a web server with moderate

Re: 2.6.23-rc7 - _random_ IRQ23 : nobody cared

2007-09-27 Thread Paul Rolland
Hello, On Thu, 27 Sep 2007 19:04:11 +1000 Benjamin Herrenschmidt [EMAIL PROTECTED] wrote: Let me guess... this is a T61 or X61 ? Bad luck ;) This is an Asus P5W-DH Deluxe motherboard, with a Core2 6400 CPU, a bunch of disk (2 IDE, 3 SATA, 1 CDRW and 1 DVDRW-DL), and a damned Olitec PCI V92

Re: Updating include/asm symlink when $ARCH changes

2007-09-27 Thread Alexey Dobriyan
On 9/27/07, Martin Michlmayr [EMAIL PROTECTED] wrote: I cross compile arm and mips kernels from the same kernel tree. You can use O= feature and skip this issue completely. make ARCH=mips O=../build/mips ... make ARCH=arm O=../build/arm ... When I build a kernel the first time with a

Re: Man page for revised timerfd API

2007-09-27 Thread Michael Kerrisk
Davide, A further question: what is the expected behavior in the following scenario: 1. Create a timerfd and arm it. 2. Wait until M timer expirations have occurred 3. Modify the settings of the timer 4. Wait for N further timer expirations have occurred 5. read() from the timerfd Does the

Re: Man page for revised timerfd API

2007-09-27 Thread Michael Kerrisk
[various useful comments snipped] Thanks Geoff -- I will incorporate all of the points you mentioned. Cheers, Michael -- Michael Kerrisk maintainer of Linux man pages Sections 2, 3, 4, 5, and 7 Want to help with man page maintenance? Grab the latest tarball at

RE: [PATCH 1/2] [INPUT] Blackfin BF54x Input Keypad controller driver

2007-09-27 Thread Hennerich, Michael
Hi Dmitry, -Original Message- From: Dmitry Torokhov [mailto:[EMAIL PROTECTED] Sent: Mittwoch, 26. September 2007 15:38 To: [EMAIL PROTECTED]; Michael Hennerich Cc: [EMAIL PROTECTED]; Linux Kernel; uclinux-dist- [EMAIL PROTECTED] Subject: Re: [PATCH 1/2] [INPUT] Blackfin BF54x Input Keypad

Re: 2.6.23-rc8-mm2 - drivers/net/ibm_newemac/mal - broken

2007-09-27 Thread Kamalesh Babulal
Andrew Morton wrote: ftp://ftp.kernel.org/pub/linux/kernel/people/akpm/patches/2.6/2.6.23-rc8/2.6.23-rc8-mm2/ Hi Andrew, The drivers/net/ibm_newemac/mal seems to be broken with 2.6.23-rc8-mm2 also, it was reported on 2.6.23-rc8-mm1 (http://lkml.org/lkml/2007/9/25/173). -- Thanks Regards,

Re: Network slowdown due to CFS

2007-09-27 Thread Martin Michlmayr
* Ingo Molnar [EMAIL PROTECTED] [2007-09-27 11:49]: Martin, could you check the iperf patch below instead of the yield patch - does it solve the iperf performance problem equally well, and does CPU utilization drop for you too? Yes, it works and CPU goes down too. -- Martin Michlmayr

Re: Network slowdown due to CFS

2007-09-27 Thread Ingo Molnar
* Martin Michlmayr [EMAIL PROTECTED] wrote: * Ingo Molnar [EMAIL PROTECTED] [2007-09-27 11:49]: Martin, could you check the iperf patch below instead of the yield patch - does it solve the iperf performance problem equally well, and does CPU utilization drop for you too? Yes, it works

kernel Oops in ext3 code

2007-09-27 Thread Norbert Preining
Hi all! (Please Cc) kernel 2.6.23-rc6 Debian/sid kernel ooops: BUG: unable to handle kernel paging request at virtual address 104b printing eip: c0195bd3 *pde = Oops: [#1] PREEMPT SMP Modules linked in: vboxdrv binfmt_misc fuse coretemp hwmon gspca videodev

Re: [PATCH 000 of 4] Change prototype for bi_end_io

2007-09-27 Thread Jens Axboe
On Thu, Sep 27 2007, NeilBrown wrote: Hi Jens, here are a few more patches from my set that makes various changes to bio submission and handling. These change the -bi_end_io prototype so that 1/ no 'size' is passed 2/ there is no return value. The 'size' is not really of

Re: Network slowdown due to CFS

2007-09-27 Thread Martin Michlmayr
* Ingo Molnar [EMAIL PROTECTED] [2007-09-27 12:56]: i'm curious by how much does CPU go down, and what's the output of iperf? (does it saturate full 100mbit network bandwidth) I get about 94-95 Mbits/sec and CPU drops from 99% to about 82% (this is with a 600 MHz ARM CPU). -- Martin Michlmayr

Re: sys_chroot+sys_fchdir Fix

2007-09-27 Thread Theodore Tso
On Thu, Sep 27, 2007 at 09:28:08AM +0200, Christer Weinigel wrote: So the OpenBSD man page seems to be in the minority here. Any portable code can not assume that CWD changes. And changing the Linux behaviour now would be a rather big change which might break userspace. And yes, there are

[PATCH] Compile handle_percpu_irq even for uniprocessor kernels

2007-09-27 Thread Ralf Baechle
Compiling handle_percpu_irq only on uniprocessor generates an artificial special case so a typical use like: set_irq_chip_and_handler(irq, some_irq_type, handle_percpu_irq); needs to be conditionally compiled only on SMP systems as well and an alternative UP construct is usually needed - for

Re: Updating include/asm symlink when $ARCH changes

2007-09-27 Thread Sam Ravnborg
On Thu, Sep 27, 2007 at 11:25:21AM +0200, Martin Michlmayr wrote: I cross compile arm and mips kernels from the same kernel tree. When I build a kernel the first time with a fresh kernel tree, the include/asm symlink is set properly. However, when I compile for a different $ARCH, the

Re: [PATCH] Compile handle_percpu_irq even for uniprocessor kernels

2007-09-27 Thread Thomas Gleixner
On Thu, 2007-09-27 at 12:24 +0100, Ralf Baechle wrote: Compiling handle_percpu_irq only on uniprocessor generates an artificial special case so a typical use like: set_irq_chip_and_handler(irq, some_irq_type, handle_percpu_irq); needs to be conditionally compiled only on SMP systems as

Re: [PATCH 1/2] strip generated symbols from *.ko

2007-09-27 Thread Sam Ravnborg
(private reply) Being occupied by non-linux stuff lately but will review your patches soon. non-linux stuff includes too little sleep because my baby girl having yet another new teeth and it hurst... I hope to go over it during the weekend so Ican include it in next merge window. Sam

Re: [PATCH 3/5] ibmebus: Add device creation and bus probing based on of_device

2007-09-27 Thread Arnd Bergmann
On Wednesday 26 September 2007, Joachim Fenkes wrote: The devtree root is now searched for devices matching a built-in whitelist during boot, so these devices appear on the bus from the beginning. It is still possible to manually add/remove devices to/from the bus by using the probe/remove

Re: [PATCH 1/5] PowerPC: Move of_device allocation into of_device.[ch]

2007-09-27 Thread Arnd Bergmann
On Wednesday 26 September 2007, Joachim Fenkes wrote: Extract generic of_device allocation code from of_platform_device_create() and move it into of_device.[ch], called of_device_alloc(). Also, there's now of_device_free() which puts the device node. This way, bus drivers that build on

Re: [PATCH 4/5] ibmebus: Move to of_device and of_platform_driver, match eHCA and eHEA drivers

2007-09-27 Thread Arnd Bergmann
On Wednesday 26 September 2007, Joachim Fenkes wrote: Replace struct ibmebus_dev and struct ibmebus_driver with struct of_device and struct of_platform_driver, respectively. Match the external ibmebus interface and drivers using it. Signed-off-by: Joachim Fenkes [EMAIL PROTECTED] Acked-by:

Re: [PATCH 2/5] ibmebus: Remove bus match/probe/remove functions

2007-09-27 Thread Arnd Bergmann
On Wednesday 26 September 2007, Joachim Fenkes wrote: Remove old code that will be replaced by rewritten and shorter functions in the next patch. Keep struct ibmebus_dev and struct ibmebus_driver for now, but replace ibmebus_{,un}register_driver() by dummy functions. This way, the kernel will

Re: [PATCH 2/3] MIPS: Remove deprecated IRQ flags (SA_*)

2007-09-27 Thread Ralf Baechle
On Thu, Sep 27, 2007 at 01:35:43AM +0300, Ahmed S. Darwish wrote: A patch to stop using deprecated IRQ flags. The new IRQF_* macros are used instead. Thanks, queued for 2.6.24. Ralf - To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to [EMAIL

Re: [GIT PULL] Workaround for broken Geode E820 BIOS

2007-09-27 Thread Joerg Pommnitz
I just tried 2.6.23-rc8 with the patch applied. Works fine here, so my very first Acked-by: Joerg Pommnitz [EMAIL PROTECTED] for whatever it's worth, since it is already in Linus' tree. Thanks to Peter and Jordan for taking the interest and time to track this one down and fix it. --

Re: 2.6.23-rc8: Known regressions from 2.6.22

2007-09-27 Thread Rafael J. Wysocki
On Thursday, 27 September 2007 01:29, Chuck Ebbert wrote: On 09/26/2007 07:27 PM, Rafael J. Wysocki wrote: Subject:Regression in 2.6.23-pre Was: Problems with 2.6.23-rc6 on AMD Geode LX800 Submitter: Joerg Pommnitz [EMAIL PROTECTED] References: http://lkml.org/lkml/2007/9/26/91

Re: 2.6.23-rc8: Known regressions from 2.6.22

2007-09-27 Thread Rafael J. Wysocki
On Thursday, 27 September 2007 08:19, Meelis Roos wrote: RJW This message lists some known regressions from 2.6.22 for which there are RJW no fixes in the mainline that I know of. If any of them have been fixed RJW already, please let me know. RJW RJW If you know of any other unresolved

Re: why network devices don't do reference counting? (Re: [PATCH] Module use count must be updated as bridges are created/destroyed)

2007-09-27 Thread Helge Hafting
Stephen Hemminger wrote: On Thu, 27 Sep 2007 00:18:55 +0200 (CEST) Jan Engelhardt [EMAIL PROTECTED] wrote: On Sep 26 2007 14:06, Stephen Hemminger wrote: No, network devices don't do reference counting. Could you explain why, please? After `udevd` on boot loads lots of

Re: 2.6.23-rc8-mm1 -- powerpc link failure

2007-09-27 Thread Andy Whitcroft
actually, my first patch wasn't using weak symbols, but I have been convinced that it's the way to go(tm). Please see http://lkml.org/lkml/2007/9/1/131 and the ongoing thread. I am fine with replacing the brk randomization patch with the one that wasn't using weak symbols (posted in the

eventfd.2 man page

2007-09-27 Thread Michael Kerrisk
Hi Davide, I've slightly tweaked the eventfd.2 man page in preparation for adding it to the man-pages set. Could you please review the text below, and confirm that it correctly describes intended behavior. Thanks, Michael .\ Copyright (C) 2007 Michael Kerrisk [EMAIL PROTECTED] .\ starting

Re: kernel Oops in ext3 code

2007-09-27 Thread Rafael J. Wysocki
On Thursday, 27 September 2007 12:31, Norbert Preining wrote: Hi all! (Please Cc) kernel 2.6.23-rc6 Debian/sid Does it happen with 2.6.22? kernel ooops: BUG: unable to handle kernel paging request at virtual address 104b printing eip: c0195bd3 *pde = Oops:

Re: 2.6.23-rc8-mm1 -- powerpc link failure

2007-09-27 Thread Jiri Kosina
On Thu, 27 Sep 2007, Andy Whitcroft wrote: Ok, this problem seems to still persist in 2.6.23-rc8-mm2. It seems we have three options from here: 1) update the compiler support list to exclude these compilers, or 2) back this change out, or 3) switch to the version not using __weak. The

[git patches] umem driver cleanups

2007-09-27 Thread Jeff Garzik
Please pull from 'umem' branch of master.kernel.org:/pub/scm/linux/kernel/git/jgarzik/misc-2.6.git umem to receive the following updates: drivers/block/umem.c| 202 --- {include/linux = drivers/block}/umem.h | 19 +-- include/linux/pci_ids.h

Re: Warnings and Bug on 2.6.23-rc6 closing rfcomm links (device_move() API ?)

2007-09-27 Thread Rafael J. Wysocki
On Tuesday, 25 September 2007 17:13, Pierre-Yves Paulus wrote: Hello, Yet another report, once again while putting rfcomm system under load. Several USB adapters, several links. Is this a regression or does it happen with 2.6.22 too? Greetings, Rafael - To unsubscribe from this list: send

Revised signalfd man-page

2007-09-27 Thread Michael Kerrisk
Davide, Below is the current signalfd man-page. Could you please review to see whether the man page describes the intended implementation, and especially look at a few questions embedded in the page (look for Davide). Cheers, Michael .\ Copyright (C) 2007 Michael Kerrisk [EMAIL PROTECTED] .\

signalfd_siginfo field names -- revisited

2007-09-27 Thread Michael Kerrisk
Hi Davide, (We started discussing this quite a while back, and you then seemed positively disposed to my idea below, but then holidays intervened, so I'm resending this slightly revised version of my earlier mail.) The signalfd_siginfo structure is defined as: struct signalfd_siginfo { /*

[PATCH] sysfs: backport of sysfs_readdir fix from 2.6.22.y to 2.6.16.y (CVE-2007-3104)

2007-09-27 Thread Miloslav Semler
This patch solves CVE-2007-3104 - sysfs_readdir oops. More can be found here: http://git.kernel.org/?p=linux/kernel/git/stable/linux-2.6.22.y.git;a=commit;h=dc351252b33f8fede396d6173dba117bcb933607 Signed-off-by: Miloslav Semler --- diff -uprN linux-2.6.16.53/fs/sysfs/dir.c

Re: Network slowdown due to CFS

2007-09-27 Thread Jarek Poplawski
On Thu, Sep 27, 2007 at 11:46:03AM +0200, Ingo Molnar wrote: * Jarek Poplawski [EMAIL PROTECTED] wrote: the (small) patch below fixes the iperf locking bug and removes the yield() use. There are numerous immediate benefits of this patch: ... sched_yield() is almost always the

Re: [PATCH 1/5] The config option itself

2007-09-27 Thread Cedric Le Goater
Pavel Emelyanov wrote: The option is called NAMESPACES. It can be selectable only if EMBEDDED is chosen (this was Eric's requisition). When the EMBEDDED is off namespaces will be on automatically. One more option (NAMESPACES_EXPERIMENTAL) was added by Serge's request to move there all the

Re: [PATCH 2/5] Move the UST namespace under the option

2007-09-27 Thread Cedric Le Goater
Pavel Emelyanov wrote: Currently all the namespace management code is in the kernel/utsname.c file, so just compile it out and make stub in .h file. The init namespace itself is in init/version.c and is left in the kernel. Signed-off-by: Pavel Emelyanov [EMAIL PROTECTED] Acked-by:

Re: [PATCH 4/5] Move the user namespace under the option

2007-09-27 Thread Cedric Le Goater
Pavel Emelyanov wrote: We currently have a CONFIG_USER_NS option. Just rename it into CONFIG_NAMESPACES_EXPERIMANTAL and move the init_user_ns into user.c file to make the kernel compile and work without the namespaces support. Signed-off-by: Pavel Emelyanov [EMAIL PROTECTED] Acked-by:

Re: [PATCH 3/5] Move the IPC namespace under the option

2007-09-27 Thread Cedric Le Goater
Pavel Emelyanov wrote: Currently all the IPC namespace management code is in ipc/util.c. I moved this code into ipc/namespace.c file which is compiled out when needed. The linux/ipc_namespace.h file is used to store the prototypes of the functions in namespace.c and the stubs for

Re: Revised signalfd man-page

2007-09-27 Thread Heikki Orsila
On Thu, Sep 27, 2007 at 02:19:05PM +0200, Michael Kerrisk wrote: uint28_t pad[\fIX\fP]; /* Pad size to 128 bytes (allow space additional fields in the future) */ I think you mean uint8_t.. -- Heikki Orsila Barbie's law: [EMAIL PROTECTED]

Re: [PATCH 4/5] Move the user namespace under the option

2007-09-27 Thread Robert P. J. Day
On Thu, 27 Sep 2007, Cedric Le Goater wrote: diff --git a/include/linux/user_namespace.h b/include/linux/user_namespace.h index b5f41d4..dda160c 100644 --- a/include/linux/user_namespace.h +++ b/include/linux/user_namespace.h @@ -17,7 +17,7 @@ struct user_namespace { extern struct

Re: Revised signalfd man-page

2007-09-27 Thread Michael Kerrisk
On Thu, Sep 27, 2007 at 02:19:05PM +0200, Michael Kerrisk wrote: uint28_t pad[\fIX\fP]; /* Pad size to 128 bytes (allow space additional fields in the future) */ I think you mean uint8_t.. Yep -- I sure do thanks, Michael -- Michael Kerrisk

Re: [RFC] QoS params patch

2007-09-27 Thread roel
Mark Gross wrote: On Wed, Sep 26, 2007 at 04:41:59PM -0700, Randy Dunlap wrote: On Wed, 26 Sep 2007 15:40:26 -0700 Mark Gross wrote: The following is the qos_param patch that implements a genralization of latency.c. Just some general comments (as on irc): - use 'diffstat -p1 -w70' to

Re: kernel Oops in ext3 code

2007-09-27 Thread Norbert Preining
On Do, 27 Sep 2007, Rafael J. Wysocki wrote: Does it happen with 2.6.22? Hard to say. It didn't happen as long as I used -22, but it didn't happen for a long time (since I run -rc6), and it is not reproducible. What I did at this time is a: tar -cjf foo.tar.bz2

[PATCH] fs: Correct SuS compliance for open of large file without options

2007-09-27 Thread Alan Cox
The early LFS work that Linux uses favours EFBIG in various places. SuSv3 specifically uses EOVERFLOW for this as noted by Michael (Bug 7253) -- [EOVERFLOW] The named file is a regular file and the size of the file cannot be represented correctly in an object of type off_t. We should

Re: Network slowdown due to CFS

2007-09-27 Thread Ingo Molnar
* Jarek Poplawski [EMAIL PROTECTED] wrote: On Thu, Sep 27, 2007 at 11:46:03AM +0200, Ingo Molnar wrote: [...] What you missed is that there is no such thing as predictable yield behavior for anything but SCHED_FIFO/RR tasks (for which tasks CFS does keep the behavior). Please read this

[PATCH] JBD2: debug code cleanup.

2007-09-27 Thread Jose R. Santos
From: Jose R. Santos [EMAIL PROTECTED] JBD2: debug code cleanup. Mostly stolen from akpm's JBD cleanup patch. - use `#ifdef foo' instead of `#if defined(foo)' - Make journal_enable_debug __read_mostly just for the heck of it - Make jbd_debugfs_dir and jbd_debug static - debugfs_remove(NULL)

Re: Warnings and Bug on 2.6.23-rc6 closing rfcomm links (device_move() API ?)

2007-09-27 Thread Cornelia Huck
On Thu, 27 Sep 2007 14:40:07 +0200, Pierre-Yves Paulus [EMAIL PROTECTED] wrote: Hello, Yet another report, once again while putting rfcomm system under load. Several USB adapters, several links. Is this a regression or does it happen with 2.6.22 too? I've not tested with 2.6.22,

Re: Chroot bug

2007-09-27 Thread Jiri Kosina
On Wed, 26 Sep 2007, Miloslav Semler wrote: so there is no discussion about mount others. I think, if you have CAP_SYS_MOUNT/CAP_SYS_ADMIN, you need not solve chroot() and how to break it. CAP_SYS_PTRACE allows you to break out of chroot in a pretty trivial way too. -- Jiri Kosina - To

Re: lockdep wierdness...

2007-09-27 Thread Peter Zijlstra
On Mon, 2007-09-24 at 22:13 -0400, Steven Rostedt wrote: On Mon, Sep 24, 2007 at 06:07:38PM -0400, Trond Myklebust wrote: I'm seeing lockdep warning about a potential lock inversion between mm-mmap_sem and inode-i_mutex in NFS (see attachment). Unfortunately the basis for the warning

Re: kernel Oops in ext3 code

2007-09-27 Thread Rafael J. Wysocki
On Thursday, 27 September 2007 15:13, Norbert Preining wrote: On Do, 27 Sep 2007, Rafael J. Wysocki wrote: Does it happen with 2.6.22? Hard to say. It didn't happen as long as I used -22, but it didn't happen for a long time (since I run -rc6), and it is not reproducible. What I did at

Re: [PATCH -mm] task_struct: move -fpu_counter and -oomkilladj

2007-09-27 Thread Arjan van de Ven
On Thu, 27 Sep 2007 13:55:57 +0400 Alexey Dobriyan [EMAIL PROTECTED] wrote: There is nice 2 byte hole after struct task_struct::ioprio field into which we can put two 1-byte fields: -fpu_counter and -oomkilladj. [cc'ing Arjan just in case -fpu_counter placement wasn't completely random

Re: lockdep wierdness...

2007-09-27 Thread Christoph Hellwig
On Thu, Sep 27, 2007 at 03:51:07PM +0200, Peter Zijlstra wrote: Christoph, does Steve's story make sense? Yes. All that would need to be done is add an extra lock_class_key to file_system_type for i_mutex_dir_key, and extend alloc_inode to say something like: if (dir)

  1   2   3   4   5   6   7   8   >