Re: [PATCH 0/24] make atomic_read() behave consistently across all architectures

2007-08-17 Thread Paul Mackerras
Satyam Sharma writes: I wonder if this'll generate smaller and better code than _both_ the other atomic_read_volatile() variants. Would need to build allyesconfig on lots of diff arch's etc to test the theory though. I'm sure it would be a tiny effect. This whole thread is arguing about

Re: [PATCH 0/24] make atomic_read() behave consistently across all architectures

2007-08-17 Thread Satyam Sharma
On Fri, 17 Aug 2007, Nick Piggin wrote: Satyam Sharma wrote: On Fri, 17 Aug 2007, Nick Piggin wrote: Sure, now that I learned of these properties I can start to audit code and insert barriers where I believe they are needed, but this simply means that almost all

Re: [PATCH 0/24] make atomic_read() behave consistently across all architectures

2007-08-17 Thread Satyam Sharma
On Fri, 17 Aug 2007, Andi Kleen wrote: On Friday 17 August 2007 05:42, Linus Torvalds wrote: On Fri, 17 Aug 2007, Paul Mackerras wrote: I'm really surprised it's as much as a few K. I tried it on powerpc and it only saved 40 bytes (10 instructions) for a G5 config. One of the

[PATCH] Fix check for return value of create_pid_namespace

2007-08-17 Thread Pavel Emelyanov
The function in question returns ERR_PTR-s to indicate the error, while the caller checks for return value to be NULL. Found during testing the OpenVZ kernel with pid namespaces. Signed-off-by: Pavel Emelyanov [EMAIL PROTECTED] Signed-off-by: Alexey Dobriyan [EMAIL PROTECTED] Cc: Sukadev

Re: [PATCH 0/24] make atomic_read() behave consistently across all architectures

2007-08-17 Thread Satyam Sharma
On Fri, 17 Aug 2007, Nick Piggin wrote: Satyam Sharma wrote: On Fri, 17 Aug 2007, Nick Piggin wrote: Also, why would you want to make these insane accessors for atomic_t types? Just make sure everybody knows the basics of barriers, and they can apply that knowledge to atomic_t

Re: [PATCH 0/24] make atomic_read() behave consistently across all architectures

2007-08-17 Thread Satyam Sharma
On Fri, 17 Aug 2007, Paul Mackerras wrote: Satyam Sharma writes: I wonder if this'll generate smaller and better code than _both_ the other atomic_read_volatile() variants. Would need to build allyesconfig on lots of diff arch's etc to test the theory though. I'm sure it would be a

Re: nf_conntrack_ipv4 must be loaded explicitly

2007-08-17 Thread Jan Engelhardt
On Aug 2 2007 20:33, Patrick McHardy wrote: End result: After loading nf_conntrack_ipv4.ko, everything works again (also with the bad ff09b7). But I have to load it explicitly, and I think that unfortunately breaks a lot of setups (such as mine) which assume ipv4 connection tracking is

Re: [PATCH 0/24] make atomic_read() behave consistently across all architectures

2007-08-17 Thread Satyam Sharma
On Fri, 17 Aug 2007, Nick Piggin wrote: Satyam Sharma wrote: [...] The point is about *author expecations*. If people do expect atomic_read() (or a variant thereof) to have volatile semantics, why not give them such a variant? Because they should be thinking about them in terms of

recursive use of bust_spinlocks()

2007-08-17 Thread Jan Beulich
Various architectures may call bust_spinlocks() recursively (when calling die() in the context do an unresolved page fault); the function itself, however, doesn't appear to be meant to be called in this manner. Nevertheless, this doesn't appear to be a problem as long as bust_spinlocks(0) doesn't

Re: [another git patch] move USB net drivers to drivers/net

2007-08-17 Thread Jan Engelhardt
Hi, On May 10 2007 18:12, Jan Engelhardt wrote: Subject: Re: [another git patch] move USB net drivers to drivers/net Hi Jeff, On May 9 2007 21:38, Jeff Garzik wrote: diff --git a/drivers/net/Makefile b/drivers/net/Makefile index 59c0459..c5d8423 100644 --- a/drivers/net/Makefile +++

Re: [patch] add kdump_after_notifier

2007-08-17 Thread Vivek Goyal
On Thu, Aug 16, 2007 at 06:26:35PM +0900, Takenori Nagano wrote: Vivek Goyal wrote: So for the time being I think we can put RAS tools on die notifier list and if it runs into issues we can always think of creating a separate list. Few things come to mind. - Why there is a separate

Re: [PATCH 0/24] make atomic_read() behave consistently across all architectures

2007-08-17 Thread Stefan Richter
I wrote: Nick Piggin wrote: You might find that these places that appear to need barriers are buggy for other reasons anyway. Can you point to some in-tree code we can have a look at? I could, or could not, if I were through with auditing the code. I remembered one case and posted it

Re: [PATCH] rtc: Make rtc-ds1742 driver hotplug-aware

2007-08-17 Thread Jan Engelhardt
On Aug 17 2007 01:06, Atsushi Nemoto wrote: Add an MODULE_ALIAS() to make this platform driver hotplug-aware. Signed-off-by: Atsushi Nemoto [EMAIL PROTECTED] --- diff --git a/drivers/rtc/rtc-ds1742.c b/drivers/rtc/rtc-ds1742.c index b2e5481..4bd22dc 100644 --- a/drivers/rtc/rtc-ds1742.c +++

Re: [PATCH]: proc: export a processes resource limits via proc/pid

2007-08-17 Thread Neil Horman
On Fri, Aug 17, 2007 at 04:09:26AM -0400, [EMAIL PROTECTED] wrote: On Thu, 16 Aug 2007 08:35:38 EDT, Neil Horman said: Hey again- Andrew requested that I repost this cleanly, after running the patch through checkpatch. As requested here it is with the changelog. Currently, there

Re: [PATCH 0/24] make atomic_read() behave consistently across all architectures

2007-08-17 Thread Stefan Richter
Nick Piggin wrote: Stefan Richter wrote: For architecture port authors, there is Documentation/atomic_ops.txt. Driver authors also can learn something from that document, as it indirectly documents the atomic_t and bitops APIs. Semantics and Behavior of Atomic and Bitmask Operations is

Re: nanosleep() accuracy

2007-08-17 Thread Michal Schmidt
GolovaSteek wrote: 2007/8/17, Michal Schmidt [EMAIL PROTECTED]: GolovaSteek skrev: Hello! I need use sleep with accurat timing. I use 2.6.21 with rt-prempt patch. with enabled rt_preempt, dyn_ticks, and local_apic But req.tv_nsec = 30; req.tv_sec = 0; nanosleep(req,NULL)

Re: [PATCH 0/24] make atomic_read() behave consistently across all architectures

2007-08-17 Thread Stefan Richter
Nick Piggin wrote: Satyam Sharma wrote: And we have driver / subsystem maintainers such as Stefan coming up and admitting that often a lot of code that's written to use atomic_read() does assume the read will not be elided by the compiler. So these are broken on i386 and x86-64? The

Unable to handle kernel NULL pointer dereference at 0000000000000001 RIP

2007-08-17 Thread Andreas Moroder
Hello, our SLES production system 2.6.5-7.252-smp #1 SMP Tue Feb 14 11:11:04 UTC 2006 x86_64 x86_64 x86_64 stopped with the following messages in the log Is this a known bug in this version and is it solved in the next versions. As I wrote it is a production server so I can't change away from

Re: Unable to handle kernel NULL pointer dereference at 0000000000000001 RIP

2007-08-17 Thread Jiri Kosina
On Fri, 17 Aug 2007, Andreas Moroder wrote: our SLES production system 2.6.5-7.252-smp #1 SMP Tue Feb 14 11:11:04 UTC 2006 x86_64 x86_64 x86_64 stopped with the following messages in the log Is this a known bug in this version and is it solved in the next versions. Hi, this is

Re: [2.6.23-rc3] NFSv4 client oops

2007-08-17 Thread Trond Myklebust
On Wed, 2007-08-15 at 18:08 -0400, Jeff Garzik wrote: While upgrading nfs-utils on my NFSv4 file server (F7 x86-64 2.6.23-rc3), I got an oops on the NFSv4 client (FC6 x86-64 2.6.23-rc3), and communications stopped. I rebooted the client, and everything was fine again. Then, on the

Re: [PATCH/RFC 4/4, second shot]KVM uses account_guest_time()

2007-08-17 Thread Laurent Vivier
KVM updates vtime in task_struct to allow account_guest_time() to modify user, system and guest time in cpustat accordingly. Index: kvm/drivers/kvm/Kconfig === --- kvm.orig/drivers/kvm/Kconfig2007-08-17 10:24:46.0

Re: [PATCH 0/24] make atomic_read() behave consistently across all architectures

2007-08-17 Thread Nick Piggin
Satyam Sharma wrote: On Fri, 17 Aug 2007, Nick Piggin wrote: Satyam Sharma wrote: It is very obvious. msleep calls schedule() (ie. sleeps), which is always a barrier. Probably you didn't mean that, but no, schedule() is not barrier because it sleeps. It's a barrier because it's

Re: [PATCH] NOMMU: Separate out VMAs

2007-08-17 Thread Bernd Schmidt
David Howells wrote: David Howells [EMAIL PROTECTED] wrote: Yes. I found the major leak this morning. There may be a minor leak, but I'm not convinced it's in the mmap stuff. See revised patch. Oops. That was the old patch. Try this one instead. Here are some changes to make it more

[PATCH/RFC 3/4, second shot]Introduce account_guest_time

2007-08-17 Thread Laurent Vivier
This is another way to compute guest time... I remove the account modifiers mechanism and call directly account_guest_time() from account_system_time(). account_system_time() computes user, system and guest times according value accumulated in vtime (a ktime_t) in task_struct by the virtual

Re: [PATCH] Make checkpatch rant about trailing ; at the end of if expr

2007-08-17 Thread Jan Engelhardt
On Aug 16 2007 10:21, Andy Whitcroft wrote: +if ($line =~ /\bif\s*\([^\)]*\)\s*\;/) { Heh, you are the second person to suggest this check today, do I detect some ripped out hair due to one of these! I've taken this idea and expanded it to cover if, for and while which can all

Re: [PATCH 0/24] make atomic_read() behave consistently across all architectures

2007-08-17 Thread Nick Piggin
Satyam Sharma wrote: On Fri, 17 Aug 2007, Nick Piggin wrote: I think they would both be equally ugly, You think both these are equivalent in terms of looks: | while (!atomic_read(v)) { | while (!atomic_read_xxx(v)) { ...

Re: RFHelp: Splitting MAINTAINERS into maintainers/* and Makefile/Kconfig support

2007-08-17 Thread Joe Perches
On Fri, 2007-08-17 at 02:25 -0400, Chris Snook wrote: The whole point of MAINTAINERS is to have one central repository for this information, instead of scattering it throughout the various source files. If that file is getting too unwieldy (and I don't think it is) then I could

Re: Corrupted filesystem with new Firewire stack

2007-08-17 Thread Martin K. Petersen
Stefan == Stefan Richter [EMAIL PROTECTED] writes: Stefan There were some similar reports involving that status write Stefan for unknown orb. I haven't found a way to reproduce it; I Stefan noticed it only once in the logs here so far. I get those all the time. Just do heavy ext3 I/O to the

Re: [PATCH 000 of 6] A few block-layer tidy-up patches.

2007-08-17 Thread Jens Axboe
On Fri, Aug 17 2007, Neil Brown wrote: On Friday August 17, [EMAIL PROTECTED] wrote: Please inspect the #block-2.6.24 branch to see the result. I don't know where to look for this. I checked http://git.kernel.org/?p=linux/kernel/git/axboe/linux-2.6-block.git but they don't

Re: [PATCH 0/24] make atomic_read() behave consistently across all architectures

2007-08-17 Thread Nick Piggin
Satyam Sharma wrote: On Fri, 17 Aug 2007, Nick Piggin wrote: Because they should be thinking about them in terms of barriers, over which the compiler / CPU is not to reorder accesses or cache memory operations, rather than special volatile accesses. This is obviously just a taste thing.

Re: [PATCH 0/24] make atomic_read() behave consistently across all architectures

2007-08-17 Thread Satyam Sharma
On Fri, 17 Aug 2007, Nick Piggin wrote: Satyam Sharma wrote: On Fri, 17 Aug 2007, Nick Piggin wrote: Satyam Sharma wrote: It is very obvious. msleep calls schedule() (ie. sleeps), which is always a barrier. Probably you didn't mean that, but no, schedule() is not barrier

Marvell 88E8056 gigabit ethernet controller

2007-08-17 Thread Kevin E
Hi all, I've read where the onboard Marvell lan controller on some Gigabyte boards don't work. I've got two systems using the same Gigabyte board, on one the LAN works on the other it dies like described by others. Here's the systems: Working system: Gigabyte 965P-DS3 rev 3.3 (BIOS

Re: [PATCH 0/24] make atomic_read() behave consistently across all architectures

2007-08-17 Thread Satyam Sharma
On Fri, 17 Aug 2007, Nick Piggin wrote: Satyam Sharma wrote: [...] You think both these are equivalent in terms of looks: | while (!atomic_read(v)) { | while (!atomic_read_xxx(v)) { ... |

Re: [kvm-devel] [PATCH/RFC 3/4]Introduce account modifiers mechanism

2007-08-17 Thread Avi Kivity
Laurent Vivier wrote: - remove PATCH 3, and add in task_struct a ktime vtime where we accumulate guest time (by calling something like guest_enter() and guest_exit() from the virtualization engine), and when in account_system_time() we have cputime vtime we substrate vtime from

Re: [PATCH 0/24] make atomic_read() behave consistently across all architectures

2007-08-17 Thread Nick Piggin
Satyam Sharma wrote: On Fri, 17 Aug 2007, Nick Piggin wrote: Satyam Sharma wrote: On Fri, 17 Aug 2007, Nick Piggin wrote: Satyam Sharma wrote: It is very obvious. msleep calls schedule() (ie. sleeps), which is always a barrier. Probably you didn't mean that, but no, schedule() is not

Re: [kvm-devel] [PATCH/RFC 4/4, second shot]KVM uses account_guest_time()

2007-08-17 Thread Avi Kivity
Laurent Vivier wrote: KVM updates vtime in task_struct to allow account_guest_time() to modify user, system and guest time in cpustat accordingly. --- kvm.orig/drivers/kvm/Kconfig 2007-08-17 10:24:46.0 +0200 +++ kvm/drivers/kvm/Kconfig 2007-08-17 10:25:25.0 +0200

Re: [kvm-devel] [PATCH/RFC 3/4]Introduce account modifiers mechanism

2007-08-17 Thread Laurent Vivier
Avi Kivity wrote: Laurent Vivier wrote: - remove PATCH 3, and add in task_struct a ktime vtime where we accumulate guest time (by calling something like guest_enter() and guest_exit() from the virtualization engine), and when in account_system_time() we have cputime vtime we

[PATCH -mm] x86_64: Save registers in saved_context during suspend and hibernation

2007-08-17 Thread Rafael J. Wysocki
From: Rafael J. Wysocki [EMAIL PROTECTED] During hibernation and suspend on x86_64 save CPU registers in the saved_context structure rather than in a handful of separate variables. Signed-off-by: Rafael J. Wysocki [EMAIL PROTECTED] Looks-ok-to: Pavel Machek [EMAIL PROTECTED] ---

[PATCH -mm] PNP: Make pnpacpi_suspend handle errors

2007-08-17 Thread Rafael J. Wysocki
From: Rafael J. Wysocki [EMAIL PROTECTED] pnpacpi_suspend() doesn't check the result returned by acpi_pm_device_sleep_state() before passing it to acpi_bus_set_power(), which may not be desirable.  Make it select the target power state of the device using its second argument if

Re: [kvm-devel] [PATCH/RFC 4/4, second shot]KVM uses account_guest_time()

2007-08-17 Thread Laurent Vivier
Avi Kivity wrote: Laurent Vivier wrote: KVM updates vtime in task_struct to allow account_guest_time() to modify user, system and guest time in cpustat accordingly. --- kvm.orig/drivers/kvm/Kconfig 2007-08-17 10:24:46.0 +0200 +++ kvm/drivers/kvm/Kconfig 2007-08-17

Re: [patch] encapsulate uevent()/add_uevent_var() buffer handling

2007-08-17 Thread Cornelia Huck
On Tue, 14 Aug 2007 15:15:12 +0200, Kay Sievers [EMAIL PROTECTED] wrote: From: Kay Sievers [EMAIL PROTECTED] Subject: Driver core: change add_uevent_var to use a struct This still needs some (trivial) s390 fixes: Signed-off-by: Cornelia Huck [EMAIL PROTECTED] --- drivers/s390/cio/device.c

Re: [kvm-devel] [PATCH/RFC 3/4, second shot]Introduce account_guest_time

2007-08-17 Thread Avi Kivity
Laurent Vivier wrote: This is another way to compute guest time... I remove the account modifiers mechanism and call directly account_guest_time() from account_system_time(). account_system_time() computes user, system and guest times according value accumulated in vtime (a ktime_t) in

Re: [PATCH 0/24] make atomic_read() behave consistently across all architectures

2007-08-17 Thread Satyam Sharma
On Fri, 17 Aug 2007, Nick Piggin wrote: Satyam Sharma wrote: On Fri, 17 Aug 2007, Nick Piggin wrote: Because they should be thinking about them in terms of barriers, over which the compiler / CPU is not to reorder accesses or cache memory operations, rather than special

Re: [PATCH] hostfs: Remove pointless if statement

2007-08-17 Thread Jeff Dike
On Fri, Aug 17, 2007 at 12:24:23PM +0530, Satyam Sharma wrote: We normally use comments for that, not dead code that a compiler then elids ;-) I'd argue that comments are for when you can't make the code self-explanatory. [PATCH] hostfs: Remove pointless if statement And replace with

Re: [kvm-devel] [PATCH/RFC 3/4]Introduce account modifiers mechanism

2007-08-17 Thread Christian Borntraeger
Am Freitag, 17. August 2007 schrieb Laurent Vivier: The normal user/system accounting has the same issue, no? Whereever we happen to land (kernel or user) gets the whole tick. Yes... but perhaps I should rewrite this too ;-) If you look further, you will see, that this was actually

Re: [kvm-devel] [PATCH/RFC 3/4]Introduce account modifiers mechanism

2007-08-17 Thread Laurent Vivier
Avi Kivity wrote: [...] The normal user/system accounting has the same issue, no? Whereever we happen to land (kernel or user) gets the whole tick. So I think it is okay to have the same limitation for guest time. So this is how it looks like. PATCH 1 and 2 are always a prerequisite.

[PATCH] Isolate some explicit usage of task-tgid

2007-08-17 Thread Pavel Emelyanov
With pid namespaces this field is now dangerous to use explicitly, so hide it behind the helpers. Also the pid and pgrp fields o task_struct and signal_struct are to be deprecated. Unfortunately this patch cannot be sent right now as this leads to tons of warnings, so start isolating them,

Re: System call interposition/unprotecting the table

2007-08-17 Thread Dave Jones
On Wed, Aug 15, 2007 at 12:48:35AM +0200, Andi Kleen wrote: In general the .data protection is only considered a debugging feature. I don't know why Fedora enables it in their production kernels. That would be because we think you are wrong 8) Well, it might at best buy

[PATCH] [SCSI] arcmsr: Fix error handling

2007-08-17 Thread Maik Hampel
Fixed error handling in queuecommand(), now all READ_ and WRITE_ commands are aborted in case of RAID is gone. Before only READ_6 and WRITE_6 commands were aborted. Signed-off-by: Maik Hampel [EMAIL PROTECTED] --- diff --git a/drivers/scsi/arcmsr/arcmsr_hba.c b/drivers/scsi/arcmsr/arcmsr_hba.c

Re: [PATCH 0/24] make atomic_read() behave consistently across all architectures

2007-08-17 Thread Paul E. McKenney
On Fri, Aug 17, 2007 at 01:09:08PM +0530, Satyam Sharma wrote: On Thu, 16 Aug 2007, Paul E. McKenney wrote: On Fri, Aug 17, 2007 at 07:59:02AM +0800, Herbert Xu wrote: On Thu, Aug 16, 2007 at 09:34:41AM -0700, Paul E. McKenney wrote: The compiler can also reorder non-volatile

Re: Testing the Current Upstream Kernel

2007-08-17 Thread Dave Jones
On Thu, Aug 16, 2007 at 10:04:21AM -0400, [EMAIL PROTECTED] wrote: On Thu, 16 Aug 2007 12:28:34 +0200, Susanne Oberhauser said: Jan Blunck [EMAIL PROTECTED] writes: With our openSUSE Build Service we build a daily kernel, where we take nightly snapshots of the current upstream

[PATCH][resend] i386: Fix a couple busy loops in mach_wakecpu.h:wait_for_init_deassert()

2007-08-17 Thread Satyam Sharma
On Fri, 17 Aug 2007, Satyam Sharma wrote: On Fri, 17 Aug 2007, Andi Kleen wrote: [...] [PATCH] i386, x86_64: __const_udelay() should not be marked inline Eek, this one wasn't quite right on both counts, (1) correctness of kernel/crash.c did not depend on __const_udelay() being uninlined

Re: [PATCH] Isolate some explicit usage of task-tgid

2007-08-17 Thread Oleg Nesterov
On 08/17, Pavel Emelyanov wrote: Actually the p-tgid == pid has to be changed to has_group_leader_pid(), but Oleg pointed out that this is the same and thread_group_leader() is more preferable. No, no, sorry for confusion! I was not clear. I meant that thread_group_leader() is imho better

Re: RFHelp: Splitting MAINTAINERS into maintainers/* and Makefile/Kconfig support

2007-08-17 Thread Sam Ravnborg
Hi Joe. Perhaps with a little automation it could be revived, Which is the help I'm looking for. Can someone please help here on ideas or implementation adding a makefile target for MAINTAINERS from files in a specific subdirectory? Adding a specific target på top-level Makefile is

Re: [linux-usb-devel] why was MODALIAS removed from usb kernel events? [u]

2007-08-17 Thread Alan Stern
On Fri, 17 Aug 2007, Andreas Jellinghaus [c] wrote: http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commitdiff;h=9f8b17e643fe6aa505629658445849397bda4e4f removes MODALIAS from one of the events, this breaks user space applications like openct - everything that depends on

Re: Need help with modules loading

2007-08-17 Thread Larry Finger
Kay Sievers wrote: On 8/17/07, Larry Finger [EMAIL PROTECTED] wrote: A new driver for the Broadcom BCM43xx devices has been written that uses mac80211, rather than softmac. The newest versions of the Broadcom firmware does not support all the BCM devices. Accordingly, a separate driver is

Re: [PATCH] Isolate some explicit usage of task-tgid

2007-08-17 Thread Pavel Emelyanov
Oleg Nesterov wrote: On 08/17, Pavel Emelyanov wrote: Actually the p-tgid == pid has to be changed to has_group_leader_pid(), but Oleg pointed out that this is the same and thread_group_leader() is more preferable. No, no, sorry for confusion! I was not clear. I meant that

why was MODALIAS removed from usb kernel events? [u]

2007-08-17 Thread Andreas Jellinghaus [c]
http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commitdiff;h=9f8b17e643fe6aa505629658445849397bda4e4f removes MODALIAS from one of the events, this breaks user space applications like openct - everything that depends on getting an event that has both the DEVICE= path to the

Re: [linux-usb-devel] why was MODALIAS removed from usb kernel events? [u]

2007-08-17 Thread Andreas Jellinghaus [c]
Am Freitag, 17. August 2007 schrieb Alan Stern: On Fri, 17 Aug 2007, Andreas Jellinghaus [c] wrote: http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit diff;h=9f8b17e643fe6aa505629658445849397bda4e4f removes MODALIAS from one of the events, this breaks user space

Re: RFHelp: Splitting MAINTAINERS into maintainers/* and Makefile/Kconfig support

2007-08-17 Thread Joe Perches
On Fri, 2007-08-17 at 16:48 +0200, Sam Ravnborg wrote: Adding a specific target på top-level Makefile is easy to do so I assume you are looking for something more advanced? No, I'm looking for something simple. Perhaps something like: Makefile contents: all: [0-9a-z]* install: sh cat

Re: Thinking outside the box on file systems

2007-08-17 Thread Phillip Susi
Kyle Moffett wrote: Problem 1: updating cached acls of descendent objects: How do you find out what a 'descendent object' is? Answer: You can't without recursing through the entire in-memory dentry tree. Such recursion is lock-intensive and has poor performance. Furthermore, you have to

Re: [PATCH] rtc: Make rtc-ds1742 driver hotplug-aware

2007-08-17 Thread David Brownell
On Friday 17 August 2007, Jan Engelhardt wrote: On Aug 17 2007 01:06, Atsushi Nemoto wrote: Add an MODULE_ALIAS() to make this platform driver hotplug-aware. ... +MODULE_ALIAS(ds1742); Why exactly is this needed? What script refers to the module as ds1742 instead of rtc-ds1742? Regular

Re: [Cluster-devel] Re: [gfs2][RFC] readdir caused ls process into D (uninterruptible) state, under testing with Samba 3.0.25

2007-08-17 Thread Steven Whitehouse
Hi, On Fri, 2007-08-17 at 15:43 +0800, rae l wrote: [some comments trimmed for brevity] then I start a simple ls command on the gfs2 mouting point: $ ls /mnt/gfs2 the ls process is also changed to D state, I think it's problems about readdir implementation in gfs2, and I want to

Re: Thinking outside the box on file systems

2007-08-17 Thread Valdis . Kletnieks
On Fri, 17 Aug 2007 11:19:21 EDT, Phillip Susi said: Kyle Moffett wrote: Problem 1: updating cached acls of descendent objects: How do you find out what a 'descendent object' is? Answer: You can't without recursing through the entire in-memory dentry tree. I suspect Kyle is not quite

Re: [PATCH 02/23] lib: percpu_counter_add

2007-08-17 Thread Josef Sipek
On Thu, Aug 16, 2007 at 09:45:27AM +0200, Peter Zijlstra wrote: ... Index: linux-2.6/fs/ext2/balloc.c === --- linux-2.6.orig/fs/ext2/balloc.c +++ linux-2.6/fs/ext2/balloc.c @@ -163,7 +163,7 @@ static int reserve_blocks(struct

Re: Testing the Current Upstream Kernel

2007-08-17 Thread Dave Jones
On Fri, Aug 17, 2007 at 10:31:19AM -0400, Dave Jones wrote: FWIW, you've been able to build the Fedora SRPMs with --with-vanilla for a while now, which generates an RPM with no patches at all other than the -rc/-git on which its based. We're trying to work out logistics of which

Re: [PATCH] [428/2many] MAINTAINERS - SERIAL ATA (SATA) SUBSYSTEM:

2007-08-17 Thread Joe Perches
On Fri, 2007-08-17 at 19:45 +0400, Sergei Shtylyov wrote: [EMAIL PROTECTED] wrote: Add file pattern to MAINTAINER entry +F: drivers/ata/sata_* The libata core doesn't match this pattern, ain't a a vital part of both SAT and PATA drivers? MBR, Sergei Improvements to the pattern would

Re: [patch 3/4] Linux Kernel Markers - Documentation

2007-08-17 Thread Mathieu Desnoyers
* Frank Ch. Eigler ([EMAIL PROTECTED]) wrote: Mathieu Desnoyers [EMAIL PROTECTED] writes: [...] +A marker placed in your code provides a hook to call a function (probe) that +you can provide at runtime. A marker can be on (a probe is connected to it) +or off (no probe is

Re: [-mm PATCH 0/9] Memory controller introduction (v6)

2007-08-17 Thread Dhaval Giani
On Fri, Aug 17, 2007 at 02:12:28PM +0530, Balbir Singh wrote: Hi Andrew, The code was also tested on a power box with regular machine usage scenarios, the config disabled and with a stress suite that touched all the memory in the system and was limited in a container. Dhaval ran several

Re: [PATCH] [428/2many] MAINTAINERS - SERIAL ATA (SATA) SUBSYSTEM:

2007-08-17 Thread Sergei Shtylyov
Hello. [EMAIL PROTECTED] wrote: Add file pattern to MAINTAINER entry Signed-off-by: Joe Perches [EMAIL PROTECTED] diff --git a/MAINTAINERS b/MAINTAINERS index 04c969c..016cd0c 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -4077,6 +4077,7 @@ M:[EMAIL PROTECTED] L: [EMAIL

Re: [PATCH] select: fix sys_select to not leak ERESTARTNOHAND to userspace

2007-08-17 Thread John Blackwood
Hi Neil, We've been having problems with this select patch change. Specifically -- previously, when a ptrace attach was done to a task blocked in a select() call and that task had a timeout value, the task would restart the select() call with an updated timeout value. With this patch in place,

[PATCH] wait_task_zombie: don't fight with non-existing race with a dying ptracee

2007-08-17 Thread Oleg Nesterov
(textually depends on wait_task_zombie-remove-unneeded-child-signal-check.patch) The p-exit_signal == -1 p-ptrace == 0 check and the comment are bogus. We already did exactly the same check in eligible_child(), we did not drop tasklist_lock since then, and both variables need

Re: [PATCH] rtc: Make rtc-ds1742 driver hotplug-aware

2007-08-17 Thread Jan Engelhardt
On Aug 17 2007 08:23, David Brownell wrote: On Friday 17 August 2007, Jan Engelhardt wrote: On Aug 17 2007 01:06, Atsushi Nemoto wrote: Add an MODULE_ALIAS() to make this platform driver hotplug-aware. ... +MODULE_ALIAS(ds1742); Why exactly is this needed? What script refers to the

Re: [PATCH] lockdep: annotate rcu_read_{,un}lock()

2007-08-17 Thread Paul E. McKenney
On Fri, Aug 17, 2007 at 09:56:45AM +0200, Peter Zijlstra wrote: On Thu, 2007-08-16 at 09:01 -0700, Paul E. McKenney wrote: On Thu, Aug 16, 2007 at 04:25:07PM +0200, Peter Zijlstra wrote: There seem to be some unbalanced rcu_read_{,un}lock() issues of late, how about doing something

Re: [PATCH] rtc: Make rtc-ds1742 driver hotplug-aware

2007-08-17 Thread Atsushi Nemoto
On Fri, 17 Aug 2007 17:37:01 +0200 (CEST), Jan Engelhardt [EMAIL PROTECTED] wrote: Most busses don't have managed device IDs like PCI, USB, or PNP. The platform, spi, and i2c busses use the driver name, which is obviously managed within the scope of all Linux drivers. Yeah but that does

Re: [PATCH 09/23] lib: percpu_counter_init error handling

2007-08-17 Thread Peter Zijlstra
On Fri, 2007-08-17 at 11:56 -0400, Josef Sipek wrote: On Thu, Aug 16, 2007 at 09:45:34AM +0200, Peter Zijlstra wrote: ) @@ -996,12 +997,16 @@ static int ext2_fill_super(struct super_ sbi-s_rsv_window_head.rsv_goal_size = 0; ext2_rsv_window_add(sb, sbi-s_rsv_window_head); -

Re: how to send a text format emai?

2007-08-17 Thread Jesper Juhl
On 17/08/07, Xu Yang [EMAIL PROTECTED] wrote: how to send a email of text format? Depends on your email program. But looking at your email you managed to do it: Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit -- Jesper Juhl [EMAIL PROTECTED] Don't top-post

[linux-acpi] No ACPI Sleep button events problem

2007-08-17 Thread Michał Majchrowicz
Hi. I have a problem with Sleep button events. dmesg shows this: ug 16 22:44:27 lapek ACPI: AC Adapter [ADP1] (off-line) Aug 16 22:44:27 lapek ACPI: Battery Slot [BAT0] (battery present) Aug 16 22:44:27 lapek ACPI: Power Button (FF) [PWRF] Aug 16 22:44:27 lapek ACPI: Lid Switch [LID0] Aug 16

Re: [PATCH 2/7] Simple Performance Counters: x86_64 support

2007-08-17 Thread Mathieu Desnoyers
* Andi Kleen ([EMAIL PROTECTED]) wrote: On Wed, Aug 01, 2007 at 11:58:44AM -0700, Christoph Lameter wrote: On Wed, 1 Aug 2007, Andi Kleen wrote: That might be on your systems, but for a mainline submission the standards are higher. Right that is why I asked for someone to take it

Re: [PATCH 11/23] mm: bdi init hooks

2007-08-17 Thread Josef Sipek
On Thu, Aug 16, 2007 at 09:45:36AM +0200, Peter Zijlstra wrote: provide BDI constructor/destructor hooks ... Index: linux-2.6/drivers/block/rd.c === --- linux-2.6.orig/drivers/block/rd.c +++ linux-2.6/drivers/block/rd.c ... @@

Re: [PATCH 0/3] x86_64 EFI runtime service support

2007-08-17 Thread H. Peter Anvin
Huang, Ying wrote: Has the zero page documentation and version numbering project made any progress? I think we cannot merge this without at least the version number More than that. You need to be able to boot a 32-bit kernel on a 64-bit system, so anything that breaks that is a nonstarter.

Re: [PATCH] rtc: Make rtc-ds1742 driver hotplug-aware

2007-08-17 Thread Atsushi Nemoto
On Fri, 17 Aug 2007 09:02:00 -0700, David Brownell [EMAIL PROTECTED] wrote: Because for some reason the driver name isn't rtc-ds1742 ... My preferred style for such patches puts the MODULE_ALIAS up near the strange driver name, so it's more clear what's going on. Putting all the MODULE_*()

Re: [PATCH 11/23] mm: bdi init hooks

2007-08-17 Thread Peter Zijlstra
On Fri, 2007-08-17 at 12:10 -0400, Josef Sipek wrote: On Thu, Aug 16, 2007 at 09:45:36AM +0200, Peter Zijlstra wrote: provide BDI constructor/destructor hooks Index: linux-2.6/drivers/block/rd.c === ---

Re: [PATCH 16/23] mm: scalable bdi statistics counters.

2007-08-17 Thread Peter Zijlstra
On Fri, 2007-08-17 at 12:20 -0400, Josef Sipek wrote: On Thu, Aug 16, 2007 at 09:45:41AM +0200, Peter Zijlstra wrote: Index: linux-2.6/include/linux/backing-dev.h === --- linux-2.6.orig/include/linux/backing-dev.h +++

Re: [PATCH 17/23] mm: count reclaimable pages per BDI

2007-08-17 Thread Josef Sipek
On Thu, Aug 16, 2007 at 09:45:42AM +0200, Peter Zijlstra wrote: ... Index: linux-2.6/include/linux/backing-dev.h === --- linux-2.6.orig/include/linux/backing-dev.h +++ linux-2.6/include/linux/backing-dev.h @@ -27,6 +27,7 @@ enum

kernel 2.6.19 porting

2007-08-17 Thread Xu Yang
Hi everyone, I have built up a software virtual prototype system with ARM11MPCORE. my system is similar with realview_eb_mpcore. but my system is simpler, which has only a mpcore , uart0, console,timer0_1, and memorys. I want to run kernel 2.6.19 on it. I have done some modification, and now I

Re: [PATCH] rtc: Make rtc-ds1742 driver hotplug-aware

2007-08-17 Thread David Brownell
On Friday 17 August 2007, Atsushi Nemoto wrote: My preferred style for such patches puts the MODULE_ALIAS up near the strange driver name, so it's more clear what's going on.  Putting all the MODULE_*() stuff at the end of the file gets confusing in this case. OK, then I should update

Re: [PATCH] rtc: Make rtc-ds1742 driver hotplug-aware

2007-08-17 Thread David Brownell
On Friday 17 August 2007, Jan Engelhardt wrote: Most busses don't have managed device IDs like PCI, USB, or PNP. The platform, spi, and i2c busses use the driver name, which is obviously managed within the scope of all Linux drivers. Yeah but that does not tell me why it needs the ds1742

RE: [-mm PATCH] DMA engine kconfig improvements (rev2)

2007-08-17 Thread Nelson, Shannon
From: Adrian Bunk [mailto:[EMAIL PROTECTED] Sent: Thursday, August 16, 2007 5:48 PM To: Nelson, Shannon Cc: [EMAIL PROTECTED]; linux-kernel@vger.kernel.org; Williams, Dan J; [EMAIL PROTECTED] Subject: Re: [-mm PATCH] DMA engine kconfig improvements (rev2) On Thu, Aug 16, 2007 at 05:10:25PM

Re: [PATCH 09/23] lib: percpu_counter_init error handling

2007-08-17 Thread Josef Sipek
On Thu, Aug 16, 2007 at 09:45:34AM +0200, Peter Zijlstra wrote: alloc_percpu can fail, propagate that error. Signed-off-by: Peter Zijlstra [EMAIL PROTECTED] --- fs/ext2/super.c| 11 --- fs/ext3/super.c| 11 --- fs/ext4/super.c

Re: [PATCH 16/23] mm: scalable bdi statistics counters.

2007-08-17 Thread Josef Sipek
On Thu, Aug 16, 2007 at 09:45:41AM +0200, Peter Zijlstra wrote: ... Index: linux-2.6/include/linux/backing-dev.h === --- linux-2.6.orig/include/linux/backing-dev.h +++ linux-2.6/include/linux/backing-dev.h ... @@ -24,6 +26,12 @@

Re: [PATCH 7/7] Simple Performance Counters: SLUB instrumentation

2007-08-17 Thread Mathieu Desnoyers
Hi Christoph, A few remarks on these tests: Why do you printk inside the timing period ? Filling the printk buffers or outputting on things such as serial console could really hurt your results. I hope you run your system with idle=poll and without frequency scaling at all, because otherwise

rtl8187: Invalid hwaddr! Using randomly generated MAC address

2007-08-17 Thread Alan Jenkins
I've just acquired this buggy piece of hardware otherwise known as a NetGear WG111v2. I googled and eventually found an explanation in the form of source code here: http://www.gelato.unsw.edu.au/lxr/source/include/linux/etherdevice.h#L93. It seems linux rejects the hardware MAC address because

Re: [-mm PATCH] DMA engine kconfig improvements (rev2)

2007-08-17 Thread Adrian Bunk
On Fri, Aug 17, 2007 at 08:59:29AM -0700, Nelson, Shannon wrote: From: Adrian Bunk [mailto:[EMAIL PROTECTED] Sent: Thursday, August 16, 2007 5:48 PM To: Nelson, Shannon Cc: [EMAIL PROTECTED]; linux-kernel@vger.kernel.org; Williams, Dan J; [EMAIL PROTECTED] Subject: Re: [-mm PATCH] DMA

Re: [PATCH 1/2] Add scaled time to taskstats based process accounting

2007-08-17 Thread Linas Vepstas
On Fri, Aug 17, 2007 at 08:22:40AM +1000, Paul Mackerras wrote: Linas Vepstas writes: My gut impression (maybe wrong?) is that the scaled time is, in a certain sense, more accurate than the unscaled time. The unscaled time is just time, as in how many seconds did this task spend on the

SVr4/SVID/SUS IPC conformance

2007-08-17 Thread Anton Arapov
Hi! * Please, gurus, who cares about standards conformance, do not ignore this message! 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. Create then drop 2 shmem segments, then

Re: [PATCH] rtc: Make rtc-ds1742 driver hotplug-aware

2007-08-17 Thread Kay Sievers
On Fri, 2007-08-17 at 09:55 -0700, David Brownell wrote: On Friday 17 August 2007, Kay Sievers wrote: Again, Again? We exchanges several mails a few weeks ago after the Debian bug caused by a modprobe loop. the only sane solution is to provide MODALIAS=platform:name from the platform

Re: [PATCH] rtc: Make rtc-ds1742 driver hotplug-aware

2007-08-17 Thread Kay Sievers
On 8/17/07, Atsushi Nemoto [EMAIL PROTECTED] wrote: On Fri, 17 Aug 2007 09:02:00 -0700, David Brownell [EMAIL PROTECTED] wrote: Because for some reason the driver name isn't rtc-ds1742 ... My preferred style for such patches puts the MODULE_ALIAS up near the strange driver name, so it's

[PATCH] __group_complete_signal: eliminate unneeded wakeup of -group_exit_task

2007-08-17 Thread Oleg Nesterov
Cleanup. __group_complete_signal() wakes up -group_exit_task twice. The second wakeup's state includes TASK_UNINTERRUPTIBLE, which is not very appropriate. Change the code to pass the correct argument to signal_wake_up() and kill now unneeded wake_up_process(). Signed-off-by: Oleg Nesterov

Re: [PATCH 1/7] Simple Performance Counters: Core Piece

2007-08-17 Thread Mathieu Desnoyers
Hi Christoph, Actually, get_cycles() at least on some AMD cpus, do not synchronize the core, which can skew the results. You might want to use get_cycles_sync() there. Mathieu * Christoph Lameter ([EMAIL PROTECTED]) wrote: Simple performance counters are a way to measure the performance on

<    1   2   3   4   5   6   7   >