Re: [RFC 16/26] union-mount: Introduce union_mount structure

2007-08-05 Thread Bharata B Rao
On Mon, Jul 30, 2007 at 06:13:39PM +0200, Jan Blunck wrote: > + > +int append_to_union(struct vfsmount *mnt, struct dentry *dentry, > + struct vfsmount *dest_mnt, struct dentry *dest_dentry) > +{ > + struct union_mount *this, *um; > + > + BUG_ON(!IS_MNT_UNION(mnt)); > + > +

Re: Possible error in 2.6.23-rc2-rt1 series

2007-08-05 Thread Ingo Molnar
* Peter Williams <[EMAIL PROTECTED]> wrote: > I've just been reviewing these patches and have spotted a possible > error in the file arch/ia64/kernel/time.c in that the scope of the > #ifdef on CONFIG_TIME_INTERPOLATION seems to have grown quite a lot > since 2.2.23-rc1-rt7. It used to chop

Re: [PATCH] msleep() with hrtimers

2007-08-05 Thread Arjan van de Ven
On Mon, 2007-08-06 at 03:03 +0200, Roman Zippel wrote: > There's no problem to provide a high resolution sleep, but there is also > no reason to mess with msleep, don't fix what ain't broken... John Corbet provided the patch because he had a problem with the current msleep... in that it didn't

Re: [PATCH 0/5] x86_64 EFI support -v3

2007-08-05 Thread Huang, Ying
On Tue, 2007-07-31 at 12:47 +0800, Eric W. Biederman wrote: > Using efi_set_virtual means kdump doesn't work which means that no > one is going to use this in a prebuilt kernel. It is possible to make kexec/kdump work with EFI virtual mode, in following ways: 1. Do not turn on EFI in kexeced

Re: [patch] implement smarter atime updates support, v2

2007-08-05 Thread Ingo Molnar
* Theodore Tso <[EMAIL PROTECTED]> wrote: > On Sun, Aug 05, 2007 at 09:28:38PM +0200, Ingo Molnar wrote: > > > > added the relatime_interval sysctl that allows the changing of the > > atime update frequency. (default: 1 day / 86400 seconds) > > What if you specify the interval as a per-mount

Re: [PATCH] Off-by-one in /sys/module/*/refcnt

2007-08-05 Thread Tejun Heo
Kay Sievers wrote: >> @@ -785,7 +785,7 @@ static ssize_t show_refcnt(struct module_attribute >> *mattr, >>struct module *mod, char *buffer) >> { >> /* sysfs holds a reference */ >> - return sprintf(buffer, "%u\n", module_refcount(mod)-1); >> +

Re: Disk spin down issue on shut down/suspend to disk

2007-08-05 Thread Tejun Heo
Cc'ing Henrique. Any ideas? Michał sed wrote: > Greetings > > I'm experiencing double disk spin down issue on my HP nx6310 laptop > during shut down and suspend to disk. The drive is power down on "Will > now halt message" then turned back on and off again with the laptop > itself. I'm using

Re: [PATCH] Fix /proc/pid/pagemap return length calculation

2007-08-05 Thread Dave Boutcher
On Sun, 5 Aug 2007 22:34:46 -0500, Matt Mackall <[EMAIL PROTECTED]> said: > > On Sun, Aug 05, 2007 at 09:03:23PM -0500, Dave Boutcher wrote: >> >> /proc/pid/pagemap has a header (usually 8 bytes) the length >> of which needs to be compensated for when converting from >> proc file offset to page

Re: why are some atomic_t's not volatile, while most are?

2007-08-05 Thread Jerry Jiang
Is there some feedback on this point ? Thank you ./Jerry On Sun, 1 Jul 2007 08:49:37 -0400 (EDT) "Robert P. J. Day" <[EMAIL PROTECTED]> wrote: > > prompted by the earlier post on "volatile"s, is there a reason that > most atomic_t typedefs use volatile int's, while the rest don't? > > $

Re: Memory leaking behaviour in 2.6.20.11, reiserfs related?

2007-08-05 Thread Rob Mueller
This is pretty much a vanilla kernel, with just one patch to work around a deadlock problem in the reiserfs_file_write code that I think isn't fixed. http://lists.linuxcoding.com/kernel/2006-q1/msg32508.html So that sounds like a reiserfs bug. Yes, and it was definitely there still in

Possible error in 2.6.23-rc2-rt1 series

2007-08-05 Thread Peter Williams
I've just been reviewing these patches and have spotted a possible error in the file arch/ia64/kernel/time.c in that the scope of the #ifdef on CONFIG_TIME_INTERPOLATION seems to have grown quite a lot since 2.2.23-rc1-rt7. It used to chop out one if statement and now it chops out half the file.

Re: [PATCH] Fix /proc/pid/pagemap return length calculation

2007-08-05 Thread Matt Mackall
On Sun, Aug 05, 2007 at 09:03:23PM -0500, Dave Boutcher wrote: > > /proc/pid/pagemap has a header (usually 8 bytes) the length > of which needs to be compensated for when converting from > proc file offset to page number. The calculation of the > starting page number (svpfn) compensates for

Re: lmbench ctxsw regression with CFS

2007-08-05 Thread Nick Piggin
On Sat, Aug 04, 2007 at 08:50:37AM +0200, Ingo Molnar wrote: > > * Nick Piggin <[EMAIL PROTECTED]> wrote: > > > Oh good. Thanks for getting to the bottom of it. We have normally > > disliked too much runtime tunables in the scheduler, so I assume these > > are mostly going away or under a

[PATCH] take sched_debug.c out of nasal demon territory

2007-08-05 Thread Al Viro
C99 6.10.3[11]: preprocessing directive within the argument list of macro invocation => undefined behaviour. Don't do that... Signed-off-by: Al Viro <[EMAIL PROTECTED]> --- diff --git a/kernel/sched_debug.c b/kernel/sched_debug.c index 1c61e53..8421b93 100644 --- a/kernel/sched_debug.c

[RFC] VFS: mnotify (was: [PATCH 00/23] per device dirty throttling -v8)

2007-08-05 Thread Al Boldi
Jakob Oestergaard wrote: > Why on earth would you cripple the kernel defaults for ext3 (which is a > fine FS for boot/root filesystems), when the *fundamental* problem you > really want to solve lie much deeper in the implementation of the > filesystem? Noatime doesn't solve the problem, it just

Re: [PATCH RT] put in a relatively high number for rcu read lock upper limit.

2007-08-05 Thread Paul E. McKenney
On Sun, Aug 05, 2007 at 07:53:10PM +0200, Ingo Molnar wrote: > > * Steven Rostedt <[EMAIL PROTECTED]> wrote: > > > Paul and Ingo, > > > > Should we just remove the upper limit check, or is something like this > > patch sound? > > i've changed the limit to 30 (the same depth limit is used by

[PATCH] Fix /proc/pid/pagemap return length calculation

2007-08-05 Thread Dave Boutcher
/proc/pid/pagemap has a header (usually 8 bytes) the length of which needs to be compensated for when converting from proc file offset to page number. The calculation of the starting page number (svpfn) compensates for this, but the calculation of the ending page number (evpfn) does not,

Re: e1000 doesn't resume properly from standby (2.6.23-rc2)

2007-08-05 Thread Kok, Auke
Simon Arlott wrote: 00:0a.0 Ethernet controller: Intel Corp. 82546EB Gigabit Ethernet Controller (Copper) (rev 01) Subsystem: Intel Corp.: Unknown device 1012 Flags: bus master, 66Mhz, medium devsel, latency 32, IRQ 5 Memory at e302 (64-bit, non-prefetchable)

Re: [PATCH] remove hugetlb_instantiation_mutex

2007-08-05 Thread Zhang, Yanmin
On Fri, 2007-08-03 at 09:53 -0700, Nish Aravamudan wrote: > On 8/3/07, Adam Litke <[EMAIL PROTECTED]> wrote: > > On Mon, 2007-07-30 at 15:15 +0800, Zhang, Yanmin wrote: > > > On Fri, 2007-07-27 at 11:37 -0500, Adam Litke wrote: > > > > Hey... I am amazed at how quickly you came back with a patch

Re: [PATCH] Fix /proc/pid/pagemap end address calculation

2007-08-05 Thread Matt Mackall
On Sun, Aug 05, 2007 at 09:03:28PM -0500, Dave Boutcher wrote: > > When dumping vma information the pagemap_read routine calculates > the minimum of what the user asks for and the end of the vma. > Unfortunately the code uses vma->vm_start rather than vma->vm_end > which can result in the end

Re: [GIT PULL] Blackfin arch update for 2.6.23

2007-08-05 Thread Bryan Wu
On Sun, 2007-08-05 at 22:26 -0400, Mike Frysinger wrote: > On 8/5/07, Bryan Wu <[EMAIL PROTECTED]> wrote: > > On Sun, 2007-08-05 at 22:04 -0400, Mike Frysinger wrote: > > > On 8/5/07, Bryan Wu <[EMAIL PROTECTED]> wrote: > > > > Bryan Wu (4): > > > > Blackfin SPI driver: Initial supporting

[PATCH] Fix /proc/pid/pagemap end address calculation

2007-08-05 Thread Dave Boutcher
When dumping vma information the pagemap_read routine calculates the minimum of what the user asks for and the end of the vma. Unfortunately the code uses vma->vm_start rather than vma->vm_end which can result in the end address being before the start, and a nasty never-ending loop in the kernel.

Re: [PATH 0/1] Kexec jump - v2 - the first step to kexec based hibernation

2007-08-05 Thread Huang, Ying
On Sun, 2007-08-05 at 20:55 +0200, Pavel Machek wrote: > Did the trick, I got the kernel to load, and it even attempted > exec... but I got doublefault (or what is it?) > > Int 6: ... EIP: c4739906. Address is in reserve_bootmem_core. > > Do I have to disable ACPI completely? I tried with

Re: [GIT PULL] Blackfin arch update for 2.6.23

2007-08-05 Thread Mike Frysinger
On 8/5/07, Bryan Wu <[EMAIL PROTECTED]> wrote: > On Sun, 2007-08-05 at 22:04 -0400, Mike Frysinger wrote: > > On 8/5/07, Bryan Wu <[EMAIL PROTECTED]> wrote: > > > Bryan Wu (4): > > > Blackfin SPI driver: Initial supporting BF54x in SPI driver > > > > > > Michael Hennerich (11): > > >

Re: RFT: updatedb "morning after" problem [was: Re: -mm merge plans for 2.6.23]

2007-08-05 Thread david
On Mon, 6 Aug 2007, Nick Piggin wrote: [EMAIL PROTECTED] wrote: On Sun, 29 Jul 2007, Rene Herman wrote: > On 07/29/2007 01:41 PM, [EMAIL PROTECTED] wrote: > > > I agree that tinkering with the core VM code should not be done > > lightly, > > but this has been put through the proper

suspend-to-disk using a SAS drive

2007-08-05 Thread Nevine AbouGhazaleh
I am trying to suspend-to-disk using the Suspend2 modules. I am using an SSD drive connected through a SAS bus interface. The SSD acts as a boot disk. All read and write transactions to the drive works well during normal operation. when I hibernate (suspend-to-disk). I get the following

Re: RFT: updatedb "morning after" problem [was: Re: -mm merge plans for 2.6.23]

2007-08-05 Thread Nick Piggin
[EMAIL PROTECTED] wrote: On Sun, 29 Jul 2007, Rene Herman wrote: On 07/29/2007 01:41 PM, [EMAIL PROTECTED] wrote: I agree that tinkering with the core VM code should not be done lightly, but this has been put through the proper process and is stalled with no hints on how to move forward.

Re: [ck] Re: -mm merge plans for 2.6.23

2007-08-05 Thread Nick Piggin
Matthew Hawkins wrote: On 7/25/07, Nick Piggin <[EMAIL PROTECTED]> wrote: I guess /proc/meminfo, /proc/zoneinfo, /proc/vmstat, /proc/slabinfo before and after the updatedb run with the latest kernel would be a first step. top and vmstat output during the run wouldn't hurt either. Hi Nick,

Re: [GIT PULL] Blackfin arch update for 2.6.23

2007-08-05 Thread Bryan Wu
On Sun, 2007-08-05 at 22:04 -0400, Mike Frysinger wrote: > On 8/5/07, Bryan Wu <[EMAIL PROTECTED]> wrote: > > Bryan Wu (4): > > Blackfin SPI driver: Initial supporting BF54x in SPI driver > > > > Michael Hennerich (11): > > Blackfin arch: store labels so we later know who allocated >

Re: [GIT PULL] Blackfin arch update for 2.6.23

2007-08-05 Thread Mike Frysinger
On 8/5/07, Bryan Wu <[EMAIL PROTECTED]> wrote: > Bryan Wu (4): > Blackfin SPI driver: Initial supporting BF54x in SPI driver > > Michael Hennerich (11): > Blackfin arch: store labels so we later know who allocated > GPIO/Peripheral resources > Blackfin arch: add peripheral

Re: high system cpu load during intense disk i/o

2007-08-05 Thread Andrew Morton
On Sun, 5 Aug 2007 19:03:12 +0300 Dimitrios Apostolou <[EMAIL PROTECTED]> wrote: > was my report so complicated? We're bad. Seems that your context switch rate when running two instances of badblocks against two different disks went batshit insane. It doesn't happen here. Please capture the

Re: [rfc] balance-on-fork NUMA placement

2007-08-05 Thread Nick Piggin
On Fri, Aug 03, 2007 at 01:10:13PM -0700, Suresh B wrote: > On Fri, Aug 03, 2007 at 02:20:10AM +0200, Nick Piggin wrote: > > On Thu, Aug 02, 2007 at 11:33:39AM -0700, Martin Bligh wrote: > > > Nick Piggin wrote: > > > >On Wed, Aug 01, 2007 at 03:52:11PM -0700, Martin Bligh wrote: > > > >>>And so

Re: [PATCH] msleep() with hrtimers

2007-08-05 Thread Roman Zippel
Hi, On Sun, 5 Aug 2007, Arjan van de Ven wrote: > Timers are course resolution that is highly HZ-value dependent. For > cases where you want a finer resolution, the kernel now has a way to > provide that functionality... so why not use the quality of service this > provides.. We're going in

Re: [PATCH][RESEND] fix a potential NULL pointer deref in XFS on failed mount.

2007-08-05 Thread David Chinner
On Sat, Aug 04, 2007 at 08:30:21PM +0200, Jesper Juhl wrote: > Back in 2006 (2006-10-31 to be specific, reposted on 2006-11-16), I > submitted a patch to fix a potential NULL pointer deref in XFS on > failed mount. Already checked into xfs-dev tree. Will go to next mainline merge.

[PATCH] Enable lguest drivers in Kconfig

2007-08-05 Thread Rusty Russell
Lguest drivers need to default to "Y" otherwise they're never selected for new builds. (We don't bother prompting, because they're less than 4k combined, and implied by selecting lguest support). Signed-off-by: Rusty Russell <[EMAIL PROTECTED]> --- drivers/lguest/Kconfig |2 ++ 1 file

Re: [PATCH] msleep() with hrtimers

2007-08-05 Thread Arjan van de Ven
> > because a lot of parts of the kernel think and work in milliseconds, > > it's logical and USEFUL to at least provide an interface that works on > > milliseconds. > > If the millisecond resolution is enough for these users, that means the > current msleep will work fine for them. except that

Re: Linux 2.6.23-rc2

2007-08-05 Thread Jeff Chua
On 8/6/07, Rafael J. Wysocki <[EMAIL PROTECTED]> wrote: > What does 's2ram -i' say about your machine? This machine can be identified by: sys_vendor = "LENOVO" sys_product = "1702E7A" sys_version = "ThinkPad X60s" bios_version = "7BETD0WW (2.11 )" Thanks, Jeff. - To

Re: [PATCH 00/23] per device dirty throttling -v8

2007-08-05 Thread David Chinner
On Sat, Aug 04, 2007 at 09:16:35PM +0200, Florian Weimer wrote: > * Andrew Morton: > > > The easy preventive is to mount with data=writeback. Maybe that should > > have been the default. > > The documentation I could find suggests that this may lead to a > security weakness (old data in blocks

Re: [PATCH] msleep() with hrtimers

2007-08-05 Thread Roman Zippel
Hi, On Sat, 4 Aug 2007, Arjan van de Ven wrote: > > hr_msleep makes no sense. Why should we tie this interface to millisecond > > resolution? > > because a lot of parts of the kernel think and work in milliseconds, > it's logical and USEFUL to at least provide an interface that works on >

Re: [PATCH 00/23] per device dirty throttling -v8

2007-08-05 Thread David Chinner
On Sun, Aug 05, 2007 at 06:42:30AM -0400, Jeff Garzik wrote: > Jakob Oestergaard wrote: > >Oh dear. > > > >Why not just make ext3 fsync() a no-op while you're at it? > > > >Distros can turn it back on if it's needed... > > > >Of course I'm not serious, but like atime, fsync() is something one > >

Re: rtc max frequency setting

2007-08-05 Thread Michael Chang
On 8/4/07, H. Peter Anvin <[EMAIL PROTECTED]> wrote: > Jan Engelhardt wrote: > > Hi, > > > > with the old rtc.ko module, there was a /proc/sys/dev/rtc/max-user-freq > > that could be set. With rtc_cmos.ko (or the new rtc infrastructure in > > general), I am missing this file. Where can I set the

Re: [PATCH 00/23] per device dirty throttling -v8

2007-08-05 Thread Theodore Tso
On Sat, Aug 04, 2007 at 09:16:35PM +0200, Florian Weimer wrote: > * Andrew Morton: > > > The easy preventive is to mount with data=writeback. Maybe that should > > have been the default. > > The documentation I could find suggests that this may lead to a > security weakness (old data in blocks

Re: [BUG RT] WARNING: at kernel/sched.c:5071 2.6.23-rc1-rt7

2007-08-05 Thread Steven Rostedt
On Sun, 2007-08-05 at 08:56 +0200, Ingo Molnar wrote: > * Steven Rostedt <[EMAIL PROTECTED]> wrote: > > > P.S. I really found out that the system becomes VERY non-responsive > > when you run with both hard and softirqs as threads, but with > > PREEMPT_NONE ;-) > > hm. That's not supposed to

Re: suspend/hibernation regression between 2.6.19 and 2.6.20 w/ Thinkpad T41

2007-08-05 Thread Pavel Machek
Hi! > It is a small - but IMHO nagging - regression between these 2 kernel versions. > > To make a "software suspend" at this notebook ("suspend to RAM") you have > to press + . Pressing the -Key after that wakes up the notenbook. > > If you hibernated the system ("suspend to disc"), you have

Re: [PATCH 2.6.22.y] ieee1394: revert "sbp2: enforce 32bit DMA mapping"

2007-08-05 Thread Benjamin Herrenschmidt
On Sun, 2007-08-05 at 09:54 +0200, Stefan Richter wrote: > Benjamin Herrenschmidt wrote: > >>> If setting 32-bit DMA mask fails on ppc64, that sounds like a problem > >>> with the DMA implementation on that architecture. There are enough cards > >>> out there that only support 32-bit DMA that this

Re: "Badness at kernel/irq/resend.c:70" on boot - via-pmu?

2007-08-05 Thread Benjamin Herrenschmidt
On Sat, 2007-08-04 at 21:41 -0700, Linus Torvalds wrote: > > On Sun, 5 Aug 2007, Paul Collins wrote: > > > > I got the message below on boot with 2.6.23-rc2 on my PowerBook. > > It's a debug message, I think we need to remove it. It's trying to figure > out what goes wrong with one particular

Re: Possible bug in realtek 8169 ethernet driver

2007-08-05 Thread Francois Romieu
Bram <[EMAIL PROTECTED]> : [...] > The router attached to it indicates a 100mbit link. But that's about it. > I cannot get any data over it. I can manually configure it to have an IP > address and netmask, but it won't see anything on the local net. DHCP > doesn't work either. Nothing out of the

Re: SysV IPC: shmctl/msgctl/semctl returns EIDRM instead of EINVAL

2007-08-05 Thread Anton Arapov
Please, fellas, take a look on my post! Thanks in advance. Anton Arapov <[EMAIL PROTECTED]> writes: > Hi! > > SysV code returns EIDRM for collision of IDs. I sure it should return > EINVAL. > > Steps to reproduce: (this for shared memory code, for msg/sem it is the > same) >1.

Re: Distributed storage.

2007-08-05 Thread Daniel Phillips
On Sunday 05 August 2007 08:01, Evgeniy Polyakov wrote: > On Sun, Aug 05, 2007 at 01:06:58AM -0700, Daniel Phillips wrote: > > > DST original code worked as device mapper plugin too, but its two > > > additional allocations (io and clone) per block request ended up > > > for me as a show stopper.

Re: Distributed storage.

2007-08-05 Thread Daniel Phillips
On Sunday 05 August 2007 08:08, Evgeniy Polyakov wrote: > If we are sleeping in memory pool, then we already do not have memory > to complete previous requests, so we are in trouble. Not at all. Any requests in flight are guaranteed to get the resources they need to complete. This is

Re: [PATCH 00/23] per device dirty throttling -v8

2007-08-05 Thread Andi Kleen
Brice Figureau <[EMAIL PROTECTED]> writes: > > 2) I _still_ don't get the "performances" of 2.6.17, but since that's the > better combination I could get, I think there is IMHO progress in the right > direction (to be compared to no progress since 2.6.18, that's better :-)). If you could

Re: [PATCH 1/2] wait_task_zombie: remove unneeded child->signal check

2007-08-05 Thread Oleg Nesterov
On 08/05, Roland McGrath wrote: > > > A zombie must have a valid ->signal, we are going to release it and > > __exit_signal() starts with BUG_ON(!sig). > > Yes, this is safe because it's after the EXIT_DEAD check under tasklist_lock. Yes thanks, the changelog could be better. We "own" this

Re: [PATCH 4/5] UML - Simplify helper stack handling

2007-08-05 Thread Andrew Morton
On Sun, 5 Aug 2007 22:41:14 +0200 Luca Tettamanti <[EMAIL PROTECTED]> wrote: > Il Wed, Jun 27, 2007 at 11:37:01PM -0700, Andrew Morton ha scritto: > > > > So I'm running the generic version of this on i386 with 8k stacks (below), > > with a quick LTP run. > > > > Holy cow, either we use a

Re: [PATCH 1/2] wait_task_zombie: remove unneeded child->signal check

2007-08-05 Thread Roland McGrath
> A zombie must have a valid ->signal, we are going to release it and > __exit_signal() starts with BUG_ON(!sig). Yes, this is safe because it's after the EXIT_DEAD check under tasklist_lock. Thanks, Roland - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of

Re: [PATCH] exit_notify: don't take tasklist for TIF_SIGPENDING re-targeting

2007-08-05 Thread Roland McGrath
Looks fine to me. Thanks, Roland - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/

Re: [PATCH] zap_other_threads: don't optimize thread_group_empty() case

2007-08-05 Thread Roland McGrath
Looks fine to me. Thanks, Roland - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/

Re: [1/3] 2.6.23-rc2: known regressions

2007-08-05 Thread Henrique de Moraes Holschuh
On Sun, 05 Aug 2007, Michal Piotrowski wrote: > Subject : T60 ACPI issues/THINKPAD_ACPI_INPUT_ENABLED seems regressive > References : http://lkml.org/lkml/2007/8/1/198 > http://lkml.org/lkml/2007/8/1/176 > Last known good : ? 2.6.22 > Submitter :

Re: [patch] implement smarter atime updates support, v2

2007-08-05 Thread Theodore Tso
On Sun, Aug 05, 2007 at 09:28:38PM +0200, Ingo Molnar wrote: > > added the relatime_interval sysctl that allows the changing of the atime > update frequency. (default: 1 day / 86400 seconds) What if you specify the interval as a per-mount option? i.e., mount -o relatime=86400

Re: [PATCH 00/23] per device dirty throttling -v8

2007-08-05 Thread Christoph Hellwig
On Sun, Aug 05, 2007 at 09:57:02AM +0200, Florian Weimer wrote: > For instance, some editors don't perform fsync-then-rename, but simply > truncate the file when saving (because they want to preserve hard > links). With XFS, this tends to cause null bytes on crashes. Since > ext3 has got a much

Re: [PATCH 00/23] per device dirty throttling -v8

2007-08-05 Thread Christoph Hellwig
On Sun, Aug 05, 2007 at 02:26:53AM +0200, Andi Kleen wrote: > I always thought the right solution would be to just sync atime only > very very lazily. This means if a inode is only dirty because of an > atime update put it on a "only write out when there is nothing to do > or the memory is really

Re: [PATCH/RFT] finish i386 and x86-64 sysdata conversion

2007-08-05 Thread Yinghai Lu
On 8/5/07, Jeff Garzik <[EMAIL PROTECTED]> wrote: > Yinghai Lu wrote: > > pci_scan_bus_on_node(int bus, struct pci_ops *ops, int node) > > x86_pci_scan_root_bus(int bus) > > { > > pci_scan_bus_on_node(bus, _root_ops, -1); > > } > > > > i need node as one param for my patch later in irq.c and

Re: [PATCH 4/5] UML - Simplify helper stack handling

2007-08-05 Thread Luca Tettamanti
Il Wed, Jun 27, 2007 at 11:37:01PM -0700, Andrew Morton ha scritto: > > So I'm running the generic version of this on i386 with 8k stacks (below), > with a quick LTP run. > > Holy cow, either we use a _lot_ of stack or these numbers are off: > > vmm:/home/akpm> dmesg -s 100|grep 'bytes

Re: [PATCH 00/23] per device dirty throttling -v8

2007-08-05 Thread Andrew Morton
On Sun, 5 Aug 2007 22:21:12 +0200 Jörn Engel <[EMAIL PROTECTED]> wrote: > On Sun, 5 August 2007 20:37:14 +0200, Jörn Engel wrote: > > > > Guess I should throw in a kernel compile test as well, just to get a > > feel for the performance. > > Three runs each of noatime, relatime and atime, both

Re: [PATCH 00/23] per device dirty throttling -v8

2007-08-05 Thread Christoph Hellwig
On Sat, Aug 04, 2007 at 09:42:59PM +0200, J??rn Engel wrote: > On Sat, 4 August 2007 21:26:15 +0200, J??rn Engel wrote: > > > > Given the choice between only "atime" and "noatime" I'd agree with you. > > Heck, I use it myself. But "relatime" seems to combine the best of both > > worlds. It

Re: [PATCH 00/23] per device dirty throttling -v8

2007-08-05 Thread Christoph Hellwig
On Sun, Aug 05, 2007 at 11:01:18AM -0700, Arjan van de Ven wrote: > > on the journalling side this would be one transaction (not 5 milion) > and... since inodes are grouped on disk, you can even get some better > coalescing this way... > > Wonder if we could do inode-grouping smartly; eg if we

Re: [PATCH 00/23] per device dirty throttling -v8

2007-08-05 Thread Jörn Engel
On Sun, 5 August 2007 20:37:14 +0200, Jörn Engel wrote: > > Guess I should throw in a kernel compile test as well, just to get a > feel for the performance. Three runs each of noatime, relatime and atime, both with cold caches and with warm caches. Scripts below. Run on a Thinkpad T40, 1.5GHz,

Re: [patch] implement smarter atime updates support

2007-08-05 Thread Arjan van de Ven
On Sun, 2007-08-05 at 21:04 +0100, Alan Cox wrote: > O> you might want to add > > > > /* > > * if the inode is dirty already, do the atime update since > > * we'll be doing the disk IO anyway to clean the inode. > > */ > > if (inode->i_state & I_DIRTY) > >

Re: [2.6.23 regression fix] fix thinkpad_acpi without hardware

2007-08-05 Thread Henrique de Moraes Holschuh
On Sun, 05 Aug 2007, Adrian Bunk wrote: > René Treffer reported that booting a CONFIG_THINKPAD_ACPI=y kernel on a > machine without the hardware results in an Oops. > > The trace is thinkpad_acpi_module_init -> thinkpad_acpi_module_exit -> > driver_remove_file -> sysfs_hash_and_remove. > > The

Re: [Patch] panic.c

2007-08-05 Thread Heiko Carstens
> >The idea behind this is to keep the power usage on panic’d machines > >(without auto-reboot) low. Another point is in an Virtual Machine > >environment the process of the VM is using 100% of the host-cpu. This > >would stuck other programs or VMs. This patch brings the VM to stop and > >keeps

Re: suspend/hibernation regression between 2.6.19 and 2.6.20 w/ Thinkpad T41

2007-08-05 Thread Henrique de Moraes Holschuh
On Sun, 05 Aug 2007, Toralf Förster wrote: > It is a small - but IMHO nagging - regression between these 2 kernel versions. > > To make a "software suspend" at this notebook ("suspend to RAM") you have > to press + . Pressing the -Key after that wakes up the notenbook. > > If you hibernated the

Re: [PATCH] ufs: move non-layout parts of ufs_fs.h to fs/ufs/

2007-08-05 Thread Christoph Hellwig
On Sat, Aug 04, 2007 at 08:36:49PM +0100, Al Viro wrote: > On Sat, Aug 04, 2007 at 11:24:31PM +0400, Evgeniy Dushistov wrote: > > Move prototypes and in-core structures to fs/ufs/ similar to what most > > other filesystems already do. > > > > I made little modifications: move also ufs debug

Re: high system cpu load during intense disk i/o

2007-08-05 Thread Rafał Bilski
Hello and thanks for your reply. Hi, The cron job that is running every 10 min on my system is mpop (a fetchmail-like program) and another running every 5 min is mrtg. Both normally finish within 1-2 seconds. The fact that these simple cron jobs don't finish ever is certainly because of the

Re: [patch] implement smarter atime updates support

2007-08-05 Thread Alan Cox
O> you might want to add > > /* >* if the inode is dirty already, do the atime update since >* we'll be doing the disk IO anyway to clean the inode. >*/ > if (inode->i_state & I_DIRTY) > return 1; This makes the actual result somewhat less

Re: [PATCH] sonypi: Fix initialization warning

2007-08-05 Thread Richard Knutsson
Thomas Renninger wrote: On Sun, 2007-08-05 at 21:05 +0200, Richard Knutsson wrote: Signed-off-by: Richard Knutsson <[EMAIL PROTECTED]> --- Got this from the compiler (gcc (GCC) 4.1.2 20070626 (Red Hat 4.1.2-13)): drivers/char/sonypi.c:1153: warning: initialization from incompatible pointer

encrypted hibernation (was Re: Hibernation considerations)

2007-08-05 Thread Pavel Machek
Hi! > > > Two things which I think would be nice to consider are: > > >1) Encryption - I'd actually prefer if my luks device did not > > >remember the key accross a hibernation; I want to be forced to > > >reenter the phrase. However I don't know what the best thing > > >

Re: [patch] implement smarter atime updates support

2007-08-05 Thread Arjan van de Ven
> +static int relatime_need_update(struct inode *inode, struct timespec now) > +{ > + /* > + * Is mtime younger than atime? If yes, update atime: > + */ > + if (timespec_compare(>i_mtime, >i_atime) >= 0) > + return 1; > + /* > + * Is ctime younger than

[2.6.23 regression fix] fix thinkpad_acpi without hardware

2007-08-05 Thread Adrian Bunk
René Treffer reported that booting a CONFIG_THINKPAD_ACPI=y kernel on a machine without the hardware results in an Oops. The trace is thinkpad_acpi_module_init -> thinkpad_acpi_module_exit -> driver_remove_file -> sysfs_hash_and_remove. The error handling if thinkpad_acpi_module_init() fails

Re: lvcreate on 2.6.22.1: kernel tried to execute NX-protected page

2007-08-05 Thread Juergen Kreileder
Arjan van de Ven wrote: > On Sun, 2007-08-05 at 21:03 +0200, Juergen Kreileder wrote: >> I've upgraded devmapper to 1.02.20 and lvm2 to 2.02.26. Didn't help much, >> I just got a the same BUG again: >> >> kernel tried to execute NX-protected page - exploit attempt? (uid: 0) >> BUG: unable to

Re: [2/3] 2.6.23-rc2: known regressions

2007-08-05 Thread Michal Piotrowski
Rafael J. Wysocki pisze: > On Sunday, 5 August 2007 18:26, Michal Piotrowski wrote: >> Hi all, >> >> Here is a list of some known regressions in 2.6.23-rc2. >> >> Feel free to add new regressions/remove fixed etc. >> http://kernelnewbies.org/known_regressions >> > >> Power management >> >>

Re: [PATCH] sonypi: Fix initialization warning

2007-08-05 Thread Thomas Renninger
On Sun, 2007-08-05 at 21:05 +0200, Richard Knutsson wrote: > Signed-off-by: Richard Knutsson <[EMAIL PROTECTED]> > --- > Got this from the compiler (gcc (GCC) 4.1.2 20070626 (Red Hat 4.1.2-13)): > drivers/char/sonypi.c:1153: warning: initialization from incompatible pointer > type > > > diff

Re: lvcreate on 2.6.22.1: kernel tried to execute NX-protected page

2007-08-05 Thread Arjan van de Ven
On Sun, 2007-08-05 at 21:03 +0200, Juergen Kreileder wrote: > I've upgraded devmapper to 1.02.20 and lvm2 to 2.02.26. Didn't help much, > I just got a the same BUG again: > > kernel tried to execute NX-protected page - exploit attempt? (uid: 0) > BUG: unable to handle kernel paging request at

Re: [PATCH 00/23] per device dirty throttling -v8

2007-08-05 Thread Ingo Molnar
* Alan Cox <[EMAIL PROTECTED]> wrote: > > also add the CONFIG_DEFAULT_RELATIME kernel option, which makes > > "norelatime" the default for all mounts without an extra kernel boot > > option. > > Should be a mount option. it is already a mount option too. > > + relatime[FS] default

[patch] implement smarter atime updates support, v2

2007-08-05 Thread Ingo Molnar
new version: added the relatime_interval sysctl that allows the changing of the atime update frequency. (default: 1 day / 86400 seconds) Ingo --> Subject: [patch] [patch] implement smarter atime updates support From: Ingo Molnar <[EMAIL PROTECTED]> change

Re: [PATCH 00/23] per device dirty throttling -v8

2007-08-05 Thread Alan Cox
> change relatime updates to be performed once per day. This makes > relatime a compatible solution for HSM, mailer-notification and > tmpwatch applications too. Sweet > > also add the CONFIG_DEFAULT_RELATIME kernel option, which makes > "norelatime" the default for all mounts without an extra

[patch] implement smarter atime updates support

2007-08-05 Thread Ingo Molnar
* Ingo Molnar <[EMAIL PROTECTED]> wrote: > tested it by moving the date forward: > > # date > Sun Aug 5 22:55:14 CEST 2007 > # date -s "Tue Aug 7 22:55:14 CEST 2007" > Tue Aug 7 22:55:14 CEST 2007 > > access to a file did not generate disk IO before the date was set, and > it

Re: Kernel Bug in 2.4.35 when compiled gcc>=4.2.0 and -march=c3

2007-08-05 Thread Willy Tarreau
On Sun, Aug 05, 2007 at 05:43:37PM +0200, Willy Tarreau wrote: > On Sun, Aug 05, 2007 at 10:56:04AM +0200, Axel Reinhold wrote: > > i found a bug in linux-2.4.35. > > > > the bug produces a crashing kernel when compiled > > with gcc >=4.2.0 and VIA C3 optimized -march=c3 > > (CONFIG_MCYRIXIII=y)

Re: [PATCH 00/23] per device dirty throttling -v8

2007-08-05 Thread Alan Cox
On Sun, 5 Aug 2007 20:08:26 +0200 Ingo Molnar <[EMAIL PROTECTED]> wrote: > > * Alan Cox <[EMAIL PROTECTED]> wrote: > > > And you honestly think that putting it in Kconfig as well as allowing > > users to screw up horribly and creating incompatible defaults you > > So far you've not offered

Re: 2.6.23-rc1: USB hard disk broken

2007-08-05 Thread David Brownell
On Sunday 05 August 2007, Oliver Neukum wrote: > > > > 2007-08-05_10:30:27.75572 kern.err: > > ehci_hcd :00:1d.7: dev 6 ep1in scatterlist error 0/-121 That's rather strange since it means a *success* (urb->status 0) was reported after a short read (scatterlist status -120,

Re: [PATCH 00/23] per device dirty throttling -v8

2007-08-05 Thread david
On Sun, 5 Aug 2007, Diego Calleja wrote: El Sun, 5 Aug 2007 09:13:20 +0200, Ingo Molnar <[EMAIL PROTECTED]> escribió: Measurements show that noatime helps 20-30% on regular desktop workloads, easily 50% for kernel builds and much more than that (in excess of 100%) for file-read-intense

Re: [PATCH 00/23] per device dirty throttling -v8

2007-08-05 Thread Ingo Molnar
* Linus Torvalds <[EMAIL PROTECTED]> wrote: > On Sun, 5 Aug 2007, Ingo Molnar wrote: > > > > you mean tmpwatch? The trivial change below fixes this. And with that > > we've come to the end of an extremely short list of atime dependencies. > > You wouldn't even need these kinds of games. > >

[PATCH] sonypi: Fix initialization warning

2007-08-05 Thread Richard Knutsson
Signed-off-by: Richard Knutsson <[EMAIL PROTECTED]> --- Got this from the compiler (gcc (GCC) 4.1.2 20070626 (Red Hat 4.1.2-13)): drivers/char/sonypi.c:1153: warning: initialization from incompatible pointer type diff --git a/drivers/char/sonypi.c b/drivers/char/sonypi.c index 73037a4..2dcd519

[RFC][PATCH] uli526x: Add suspend and resume routines

2007-08-05 Thread Rafael J. Wysocki
[Sorry for the excessive CCs, but I don't know who's the maintainer. ;-)] --- From: Rafael J. Wysocki <[EMAIL PROTECTED]> Add suspend/resume support to the uli526x network driver (tested on x86_64, with "Ethernet controller: ALi Corporation M5263 Ethernet Controller, rev 40"). This patch is

Re: lvcreate on 2.6.22.1: kernel tried to execute NX-protected page

2007-08-05 Thread Juergen Kreileder
I've upgraded devmapper to 1.02.20 and lvm2 to 2.02.26. Didn't help much, I just got a the same BUG again: kernel tried to execute NX-protected page - exploit attempt? (uid: 0) BUG: unable to handle kernel paging request at virtual address f492c1f8 printing eip: f492c1f8 *pdpt =

Re: [PATCH 00/23] per device dirty throttling -v8

2007-08-05 Thread adi
On Sun, Aug 05, 2007 at 02:44:08PM -0400, Dave Jones wrote: > It still fails miserably for me. > > If I hit 'C' and '?' I get a list of my mail folders, with some of them > marked 'N' if they have new mail. Without atime, those N's never show > up and every mbox looks like it has no new mail.

Re: [PATH 0/1] Kexec jump - v2 - the first step to kexec based hibernation

2007-08-05 Thread Pavel Machek
Hi! > > [EMAIL PROTECTED]:~# kexec -p /data/l/linux/arch/i386/boot/bzImage > > --append="init=/bin/bash kexec_jump_buf_pfn=`cat > > /sys/kernel/kexec_jump_buf_pfn`" > > Could not find a free area of memory of 9000 bytes... > > locate_hole failed > > [EMAIL PROTECTED]:~# > > > > What am I doing

Re: [PATCH 00/23] per device dirty throttling -v8

2007-08-05 Thread Dave Jones
On Sun, Aug 05, 2007 at 09:21:41AM +0200, Ingo Molnar wrote: > * Alan Cox <[EMAIL PROTECTED]> wrote: > > > With a Red Hat on if we can move from /dev/hda to /dev/sda in FC7 then > > we can move from atime to noatime by default on FC8 with appropriate > > release note warnings and having a

Re: [PATH 1/1] Kexec jump - v2 - kexec jump

2007-08-05 Thread Pavel Machek
Hi! > > > This patch implement the functionality of jumping from kexeced kernel > > > to original kernel. > > > > > > A new reboot command named LINUX_REBOOT_CMD_KJUMP is defined to > > > trigger the jumping to (executing) the new kernel or jumping back to > > > the original kernel. > > > >

Re: high system cpu load during intense disk i/o

2007-08-05 Thread Dimitrios Apostolou
On Sunday 05 August 2007 20:58:15 Rafał Bilski wrote: > > Hello again, > > Hi! > > > was my report so complicated? Perhaps I shouldn't have included so many > > oprofile outputs. Anyway, if anyone wants to have a look, the most > > important is two_discs_bad.txt oprofile output, attached on my

Re: [PATCH 00/23] per device dirty throttling -v8

2007-08-05 Thread Jörn Engel
On Sun, 5 August 2007 11:02:33 -0700, Arjan van de Ven wrote: > > but does it work with relatime ? Like a greased penguin. I had to reboot with my ugly patch posted earlier in the patch to actually test it, though. Relatime suffers from a distribution problem, nothing else. Guess I should

Re: [PATCH] create CONFIG_SUSPEND_UP_POSSIBLE

2007-08-05 Thread Pavel Machek
On Fri 2007-08-03 15:23:19, Len Brown wrote: > On Tuesday 31 July 2007 02:38, Pavel Machek wrote: > > Hi! > > > > > Without this change, it is possible to build CONFIG_HIBERNATE > > > on all !SMP architectures, but not necessarily their SMP versions. > > > > Did you want to say "CONFIG_SUSPEND"?

  1   2   3   4   5   >