[PATCH 2/5][GFS2] Use macro instead of explicit check for mandatory locks

2007-09-12 Thread Pavel Emelyanov
The __MANDATORY_LOCK(inode) macro makes the same check, but makes the code more readable. Signed-off-by: Pavel Emelyanov <[EMAIL PROTECTED]> Cc: Steven Whitehouse <[EMAIL PROTECTED]> --- fs/gfs2/ops_file.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git

Re: [patch 1/5] Extended crashkernel command line

2007-09-12 Thread Vivek Goyal
On Tue, Sep 11, 2007 at 12:01:10PM +0200, Bernhard Walle wrote: > * Vivek Goyal <[EMAIL PROTECTED]> [2007-09-11 08:15]: > > > > "offset" seems to be optional in the new syntax. What happens if user does > > not specify offset. I think crash_base will be set to zero and system will > > try to

[PATCH 1/5] Use existing macros for distinguishing mandatory locks

2007-09-12 Thread Pavel Emelyanov
The combination of S_ISGID bit set and S_IXGRP bit unset is used to mark the inode as "mandatory lockable" and there's a macro for this check called MANDATORY_LOCK(inode). However, fs/locks.c and some filesystems still perform the explicit i_mode checking. Switch the fs/locks.c to macro

[PATCH] free_irq(): Actually handle DEBUG_SHIRQ

2007-09-12 Thread Maciej W. Rozycki
Code used for DEBUG_SHIRQ in free_irq() is unreachable -- the for() loop within never terminates otherwise than by return. This is an obvious fix. Signed-off-by: Maciej W. Rozycki <[EMAIL PROTECTED]> --- Please apply. While at it, I have a question about the complementary code within

Re: [PATCH] powerpc: add new required termio functions

2007-09-12 Thread Michael Neuling
In message <[EMAIL PROTECTED]> you wrote: > On Wed, 12 Sep 2007 12:20:32 +0200 Heiko Carstens <[EMAIL PROTECTED]> wrote: > > > On Wed, Sep 12, 2007 at 12:04:39PM +1000, Michael Neuling wrote: > > > The "tty: termios locking functions break with new termios type" patch > > >

Re: SYSFS: need a noncaching read

2007-09-12 Thread Heiko Schocher
Hello Greg Am Mittwoch, den 12.09.2007, 03:01 -0700 schrieb Greg KH: > On Wed, Sep 12, 2007 at 07:32:07AM +0200, Robert Schwebel wrote: > > On Tue, Sep 11, 2007 at 11:43:17AM +0200, Heiko Schocher wrote: > > > I have developed a device driver and use the sysFS to export some > > > registers to

Re: [PATCH] pci: fix unterminated pci_device_id lists

2007-09-12 Thread Jeff Garzik
Kees Cook wrote: This patch against 2.6.23-rc6 fixes a couple drivers that do not correctly terminate their pci_device_id lists. This results in garbage being spewed into modules.pcimap when the module happens to not have 28 NULL bytes following the table, and/or the last PCI ID is actually

Re: [PATCH 07/15] Don't return -ENOSYS as extra notes size if spufs is not loaded

2007-09-12 Thread Arnd Bergmann
On Wednesday 12 September 2007, Michael Ellerman wrote: > Because the SPU coredump code might be built as part of a module (spufs), > we have a stub which is called by the coredump code, this routine then calls > into spufs if it's loaded. > > Unfortunately the stub returns -ENOSYS if spufs is

[Resend][PATCH -mm] Hibernation: Enter platform hibernation state in a consistent way (rev. 4)

2007-09-12 Thread Rafael J. Wysocki
From: Rafael J. Wysocki <[EMAIL PROTECTED]> Make hibernation_platform_enter() execute the enter-a-sleep-state sequence instead of the mixed shutdown-with-entering-S4 thing. Replace the shutting down of devices done by kernel_shutdown_prepare(), before entering the ACPI S4 sleep state, with

Re: [PATCH] powerpc: add new required termio functions

2007-09-12 Thread Andrew Morton
On Wed, 12 Sep 2007 12:20:32 +0200 Heiko Carstens <[EMAIL PROTECTED]> wrote: > On Wed, Sep 12, 2007 at 12:04:39PM +1000, Michael Neuling wrote: > > The "tty: termios locking functions break with new termios type" patch > > (f629307c857c030d5a3dd777fee37c8bb395e171) breaks the powerpc compile. > >

Re: [PATCH -mm] ssb: Make pcmciahost depend on PCMCIA=y

2007-09-12 Thread Michael Buesch
On Wednesday 12 September 2007 12:17:45 Paul Mundt wrote: > On Wed, Sep 12, 2007 at 12:09:09PM +0200, Michael Buesch wrote: > > On Wednesday 12 September 2007 04:11:00 Paul Mundt wrote: > > > SSB uses a bool (SSB_PCMCIAHOST_POSSIBLE) to determine whether to > > > build in PCMCIA support or not, as

Re: [PATCH] Move the definition of pr_err() into kernel.h

2007-09-12 Thread Stephen Hemminger
On Tue, 11 Sep 2007 09:56:05 -0500 Emil Medve <[EMAIL PROTECTED]> wrote: > Other pr_*() macros are already defined in kernel.h, but pr_err() was defined > multiple times in several other places > > Signed-off-by: Emil Medve <[EMAIL PROTECTED]> pr_error seems better than pr_err Please add the

Re: [RFC+PATCH] RTC calibration

2007-09-12 Thread Dag-Erling Smørgrav
Arne Georg Gleditsch <[EMAIL PROTECTED]> writes: > Dag-Erling Smørgrav <[EMAIL PROTECTED]> writes: > > + case RTC_SPEED_UP: > > + err = rtc_speed_up(rtc); > > + break; > > + > > + case RTC_SLOW_DOWN: > > + err = rtc_speed_up(rtc); > > + break; > This

[Resend][PATCH -mm] Hibernation: Check if ACPI is enabled during restore in the right place

2007-09-12 Thread Rafael J. Wysocki
From: Rafael J. Wysocki <[EMAIL PROTECTED]> The following scenario leads to total confusion of the platform firmware on some boxes (eg. HPC nx6325): * Hibernate with ACPI enabled * Resume passing "acpi=off" to the boot kernel To prevent this from happening it's necessary to check if ACPI is

Re: [RFC 0/3] Recursive reclaim (on __PF_MEMALLOC)

2007-09-12 Thread Peter Zijlstra
On Wed, 2007-09-05 at 05:14 -0700, Christoph Lameter wrote: > Using the VM to throttle networking is a pretty bad thing because it > assumes single critical user of memory. There are other consumers of > memory and if you have a load that depends on other things than networking > then you

Re: [RFC+PATCH] RTC calibration

2007-09-12 Thread Arne Georg Gleditsch
Dag-Erling Smørgrav <[EMAIL PROTECTED]> writes: > + case RTC_SPEED_UP: > + err = rtc_speed_up(rtc); > + break; > + > + case RTC_SLOW_DOWN: > + err = rtc_speed_up(rtc); > + break; This doesn't look quite right. rtc_slow_down, surely? --

Re: [RFC] Union Mount: Readdir approaches

2007-09-12 Thread Al Boldi
[EMAIL PROTECTED] wrote: > But if you really want to read or try it, you can get all source files > from sourceforge. Read http://aufs.sf.net and try, > $ cvs -d:pserver:[EMAIL PROTECTED]:/cvsroot/aufs login > (CVS password is empty) > $ cvs -z3 -d:pserver:[EMAIL PROTECTED]:/cvsroot/aufs co > aufs

Re: Request for Linux Kernel Mailing List archives

2007-09-12 Thread Andreas Herrmann
On Wed, Jun 20, 2007 at 12:38:48PM -0400, Hank Leininger wrote: > On Wed, 20 Jun 2007, H. Peter Anvin wrote: > >Bj�rn Steinbrink wrote: > >> > >>marc.info supports that. > >>http://marc.info/[EMAIL PROTECTED] > >Excellent! Didn't see that documented anywhere. > > You're right, it's not really.

Re: [PATCH] powerpc: add new required termio functions

2007-09-12 Thread Heiko Carstens
On Wed, Sep 12, 2007 at 12:04:39PM +1000, Michael Neuling wrote: > The "tty: termios locking functions break with new termios type" patch > (f629307c857c030d5a3dd777fee37c8bb395e171) breaks the powerpc compile. > [...] > I'm guessing other architectures are broken too? FWIW, the above quoted

Re: [PATCH -mm] ssb: Make pcmciahost depend on PCMCIA=y

2007-09-12 Thread Paul Mundt
On Wed, Sep 12, 2007 at 12:09:09PM +0200, Michael Buesch wrote: > On Wednesday 12 September 2007 04:11:00 Paul Mundt wrote: > > SSB uses a bool (SSB_PCMCIAHOST_POSSIBLE) to determine whether to > > build in PCMCIA support or not, as the PCMCIA host code itself is > > also only a bool, make

Re: [PATCH -mm] ssb: Make pcmciahost depend on PCMCIA=y

2007-09-12 Thread Michael Buesch
On Wednesday 12 September 2007 04:11:00 Paul Mundt wrote: > SSB uses a bool (SSB_PCMCIAHOST_POSSIBLE) to determine whether to > build in PCMCIA support or not, as the PCMCIA host code itself is > also only a bool, make SSB_PCMCIAHOST_POSSIBLE depend on PCMCIA=y. > > Without this,

Re: [-mm PATCH 1/9] Memory controller resource counters (v6)

2007-09-12 Thread Balbir Singh
Paul Menage wrote: > Hi Balbir/Pavel, > > As I mentioned to you directly at the kernel summit, I think it might > be cleaner to integrate resource counters more closely with control > groups. So rather than controllers such as the memory controller > having to create their own boilerplate cf_type

Re: SYSFS: need a noncaching read

2007-09-12 Thread Greg KH
On Wed, Sep 12, 2007 at 07:32:07AM +0200, Robert Schwebel wrote: > On Tue, Sep 11, 2007 at 11:43:17AM +0200, Heiko Schocher wrote: > > I have developed a device driver and use the sysFS to export some > > registers to userspace. > > Uuuh, uggly. Don't do that. Device drivers are there to abstract

[PATCH] RDMA/CMA: Use neigh_event_send() to initiate neighbour discovery.

2007-09-12 Thread Steve Wise
RDMA/CMA: Use neigh_event_send() to initiate neighbour discovery. Calling arp_send() to initiate neighbour discovery (ND) doesn't do the full ND protocol. Namely, it doesn't handle retransmitting the arp request if it is dropped. The function neigh_event_send() does all this. Without doing full

2.6.23-rc4-mm1: kgdboe link errors

2007-09-12 Thread Jiri Slaby
Hi, randconfig [1] causes this link errors: ERROR: "netpoll_cleanup" [drivers/net/kgdboe.ko] undefined! ERROR: "netpoll_setup" [drivers/net/kgdboe.ko] undefined! ERROR: "netpoll_parse_options" [drivers/net/kgdboe.ko] undefined! ERROR: "netpoll_poll" [drivers/net/kgdboe.ko] undefined! ERROR:

hci_sock.c build failure

2007-09-12 Thread Andre Haupt
Hi, I did a make randconfig and then make C=1. hci_sock.c fails to build because it doesnt know compat_timeval (i guess). find the error message and the config attached. regards, Andre LD net/ax25/ax25.o LD net/ax25/built-in.o CHECK net/bluetooth/af_bluetooth.c CC

Re: [PATCH -mm] fs: define file_fsync() even for CONFIG_BLOCK=n

2007-09-12 Thread Christoph Hellwig
On Wed, Sep 12, 2007 at 11:06:10AM +0900, Paul Mundt wrote: > There's nothing that is problematic for file_fsync() with CONFIG_BLOCK=n, > and it's built in unconditionally anyways, so move the prototype out to > reflect that. Without this, the unionfs build bails out. Unionfs should stop using it

Re: x86 merge - a little feedback

2007-09-12 Thread Christoph Hellwig
On Tue, Sep 11, 2007 at 10:34:13PM +0200, Adrian Bunk wrote: > As an example, visws.c is as much non-64bit as it is pre-i686. Bullshit. visws were shipped with P3s. - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More

Re: clockevents: fix resume logic

2007-09-12 Thread Andrew Morton
On Tue, 11 Sep 2007 23:49:47 +0200 Thomas Gleixner <[EMAIL PROTECTED]> wrote: > On Tue, 2007-09-11 at 21:52 +0200, Thomas Gleixner wrote: > > > C1: type[C1] promotion[C2] demotion[--] latency[001] > > > usage[0010] duration[] > > >*C2:

Re: clockevents: fix resume logic

2007-09-12 Thread Andrew Morton
On Tue, 11 Sep 2007 21:52:01 +0200 Thomas Gleixner <[EMAIL PROTECTED]> wrote: > On Tue, 2007-09-11 at 11:44 -0700, Andrew Morton wrote: > > > > there doesn't seem to be a lot of difference in the time handling, > > > > except > > > > there are large changes in when things happen in the bootup

[PATCH] Fix hci_sock build with CONFIG_COMPAT=n

2007-09-12 Thread S.Çağlar Onur
Hi; Commit 66eb50d5c972cc16df2be730497b7f06d75d8132 introduces runtime check of CONFIG_COMPAT but without following patch compiliation failed with following error; ... CC [M] net/bluetooth/hci_event.o CC [M] net/bluetooth/hci_sock.o net/bluetooth/hci_sock.c: In function `hci_sock_cmsg':

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

2007-09-12 Thread Peter Zijlstra
On Tue, 2007-09-11 at 22:31 -0400, John Stoffel wrote: I hope the snipped questions were sufficiently answered in the other mail. If not, holler :-) > Peter> 3 is done by also scaling the dirty limit proportional to the > Peter> current task's recent dirty rate. > > Do you mean task or device

Re: [PATCH 08/15] Use spufs_coredump_num_notes everywhere, and don't NULL terminate

2007-09-12 Thread Arnd Bergmann
On Wednesday 12 September 2007, Michael Ellerman wrote: > The spufs_coredump_read array is NULL terminated, and we also store the size. > We only need one or the other, and storing the size should save a teensy bit > of memory vs NULL terminating, so do that. Given that we have another array in

Re: [PATCH 09/15] Internal __spufs_get_foo() routines should take a spu_context *

2007-09-12 Thread Arnd Bergmann
On Wednesday 12 September 2007, Michael Ellerman wrote: > The SPUFS attribute get routines take a void * because the generic attribute > code doesn't know what sort of data it's passing around. > > However our internal __spufs_get_foo() routines can take a spu_context * > directly, which saves

Re: [PATCH 11/15] Combine spufs_coredump_calls with spufs_calls

2007-09-12 Thread Arnd Bergmann
On Wednesday 12 September 2007, Michael Ellerman wrote: > Because spufs might be built as a module, we can't have other parts of the > kernel calling directly into it, we need stub routines that check first if the > module is loaded. > > Currently we have two structures which hold callbacks for

Re: [PATCH 10/15] Add contents of npc file to SPU coredumps

2007-09-12 Thread Arnd Bergmann
On Wednesday 12 September 2007, Michael Ellerman wrote: > Signed-off-by: Michael Ellerman <[EMAIL PROTECTED]> Acked-by: Arnd Bergmann <[EMAIL PROTECTED]> - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at

Re: [RFC]selinux: Improving SELinux read/write performance

2007-09-12 Thread Yuichi Nakamura
Hi. Stephen Smalley pointed out possibility of race condition in off-list discussion. Stephen Smalley said: > One other observation about the patch: it presently leaves open a > (small) race window in which the file could get relabeled or policy gets > reloaded between the time of the normal

Re: [PATCH 15/15] Add DEFINE_SPUFS_ATTRIBUTE()

2007-09-12 Thread Arnd Bergmann
On Wednesday 12 September 2007, Michael Ellerman wrote: > On Wed, 2007-09-12 at 17:43 +1000, Michael Ellerman wrote: > > This patch adds DEFINE_SPUFS_ATTRIBUTE(), a wraper around > > DEFINE_SIMPLE_ATTRIBUTE which does the specified locking for the get > > routine for us. > > > > Unfortunately we

Re: [PATCH 21/23] mm: per device dirty threshold

2007-09-12 Thread Peter Zijlstra
On Tue, 2007-09-11 at 22:36 -0400, John Stoffel wrote: > Peter> Scale writeback cache per backing device, proportional to its > Peter> writeout speed. By decoupling the BDI dirty thresholds a > Peter> number of problems we currently have will go away, namely: > > Ah, this clarifies my questions!

Re: [PATCH 01/15] Extract the file descriptor search logic in SPU coredump code

2007-09-12 Thread Andrew Morton
On Wed, 12 Sep 2007 18:17:42 +1000 Jeremy Kerr <[EMAIL PROTECTED]> wrote: > This series looks good to me, thanks for the fixes. I'll do some testing > tomorrow but it looks like it'll be fine as-is. > > Andrew - almost all of these are for spufs, the notable exception being: > > [PATCH 12/15]

Re: [PATCH 4/6] Embed zone_id information within the zonelist->zones pointer

2007-09-12 Thread Mel Gorman
On (12/09/07 16:51), KAMEZAWA Hiroyuki didst pronounce: > On Tue, 11 Sep 2007 22:31:27 +0100 (IST) > Mel Gorman <[EMAIL PROTECTED]> wrote: > > > Using two zonelists per node requires very frequent use of zone_idx(). This > > is costly as it involves a lookup of another structure and a

Re: [PATCH] Moxa: Fix tiny compiler warning when building withoug CONFIG_PCI

2007-09-12 Thread Jiri Slaby
Andrew Morton napsal(a): > On Fri, 17 Aug 2007 00:08:58 +0200 Jesper Juhl <[EMAIL PROTECTED]> wrote: > >> Fix this tiny compiler warning in Moxa driver : >> drivers/char/mxser.c:386: warning: 'mxser_get_PCI_conf' declared 'static' >> but never defined >> when building without CONFIG_PCI. >>

Re: [00/41] Large Blocksize Support V7 (adds memmap support)

2007-09-12 Thread Andrea Arcangeli
On Tue, Sep 11, 2007 at 05:04:41PM -0700, Christoph Lameter wrote: > I would think that your approach would be slower since you always have to > populate 1 << N ptes when mmapping a file? Plus there is a lot of wastage I don't have to populate them, I could just map one at time. The only reason

Re: [PATCH 01/15] Extract the file descriptor search logic in SPU coredump code

2007-09-12 Thread Jeremy Kerr
Hi Michael, This series looks good to me, thanks for the fixes. I'll do some testing tomorrow but it looks like it'll be fine as-is. Andrew - almost all of these are for spufs, the notable exception being: [PATCH 12/15] Cleanup ELF coredump extra notes logic which touches the generic

Re: [Sparc64 BUG] hangup under booting 2.6.22.X whith Sparc64

2007-09-12 Thread David Miller
From: "Kövedi_Krisztián" <[EMAIL PROTECTED]> Date: Wed, 12 Sep 2007 09:47:45 +0200 > The patch work fine the kernel booting up without error messages. Thanks for testing. - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More

Re: [PATCH 15/15] Add DEFINE_SPUFS_ATTRIBUTE()

2007-09-12 Thread Michael Ellerman
On Wed, 2007-09-12 at 17:43 +1000, Michael Ellerman wrote: > This patch adds DEFINE_SPUFS_ATTRIBUTE(), a wraper around > DEFINE_SIMPLE_ATTRIBUTE which does the specified locking for the get > routine for us. > > Unfortunately we need two get routines (a locked and unlocked version) to > support

stripping down the kernel-parameters.txt file

2007-09-12 Thread Robert P. J. Day
while killing some time between compiles and ridding the kernel-parameters.txt file of out-of-date or incorrect cruft, it occurs to me that that file is borderline valueless since it apparently tries to document every possible boot-time parameter, including those associated with individual

Can LVM block I/O and hang a system?

2007-09-12 Thread Maurice Volaski
A working system begins hanging and it seems to be stuck on I/O processes that use ext3 partitions that are running on top of LVM. The system is AMD 64-bit running Gentoo. Kernel is Gentoo 2.6.22-r3 and LVM lvm2-2.02.27. Here is the disk setup: Boot disk, attached to motherboard via SATA 1)

Re: [Sparc64 BUG] hangup under booting 2.6.22.X whith Sparc64

2007-09-12 Thread Kövedi Krisztián
2007/9/11, David Miller <[EMAIL PROTECTED]>: > > I think this patch should fix the problem. > > Please give it a try, thanks. > > diff --git a/arch/sparc64/kernel/pci.c b/arch/sparc64/kernel/pci.c > index 139b4cf..e8dac81 100644 > --- a/arch/sparc64/kernel/pci.c > +++ b/arch/sparc64/kernel/pci.c >

Re: [PATCH 4/6] Embed zone_id information within the zonelist->zones pointer

2007-09-12 Thread KAMEZAWA Hiroyuki
On Tue, 11 Sep 2007 22:31:27 +0100 (IST) Mel Gorman <[EMAIL PROTECTED]> wrote: > Using two zonelists per node requires very frequent use of zone_idx(). This > is costly as it involves a lookup of another structure and a substraction > operation. As struct zone is always word aligned and normally

[PATCH 14/15] Respect RLIMIT_CORE in spu coredump code

2007-09-12 Thread Michael Ellerman
Currently the spu coredump code doesn't respect the ulimit, it should. Signed-off-by: Michael Ellerman <[EMAIL PROTECTED]> --- arch/powerpc/platforms/cell/spufs/coredump.c |4 1 files changed, 4 insertions(+), 0 deletions(-) diff --git a/arch/powerpc/platforms/cell/spufs/coredump.c

[PATCH 15/15] Add DEFINE_SPUFS_ATTRIBUTE()

2007-09-12 Thread Michael Ellerman
This patch adds DEFINE_SPUFS_ATTRIBUTE(), a wraper around DEFINE_SIMPLE_ATTRIBUTE which does the specified locking for the get routine for us. Unfortunately we need two get routines (a locked and unlocked version) to support the coredump code. This patch hides one of those (the locked version)

[PATCH 12/15] Cleanup ELF coredump extra notes logic

2007-09-12 Thread Michael Ellerman
To start with, arch_notes_size() etc. is a little too ambiguous a name for my liking, so change the function names to be more explicit. Calling through macros is ugly, especially with hidden parameters, so don't do that, call the routines directly. Use ARCH_HAVE_EXTRA_ELF_NOTES as the only flag,

[PATCH 13/15] Handle errors in SPU coredump code, and support coredump to a pipe

2007-09-12 Thread Michael Ellerman
Rework spufs_coredump_extra_notes_write() to check for and return errors. If we're coredumping to a pipe we can't trust file->f_pos, we need to maintain the foffset value passed to us. The cleanest way to do this is to have the low level write routine increment foffset when we've successfully

[PATCH 10/15] Add contents of npc file to SPU coredumps

2007-09-12 Thread Michael Ellerman
Signed-off-by: Michael Ellerman <[EMAIL PROTECTED]> --- arch/powerpc/platforms/cell/spufs/file.c |8 +++- 1 files changed, 7 insertions(+), 1 deletions(-) diff --git a/arch/powerpc/platforms/cell/spufs/file.c b/arch/powerpc/platforms/cell/spufs/file.c index d19220f..52f020a 100644 ---

[PATCH 11/15] Combine spufs_coredump_calls with spufs_calls

2007-09-12 Thread Michael Ellerman
Because spufs might be built as a module, we can't have other parts of the kernel calling directly into it, we need stub routines that check first if the module is loaded. Currently we have two structures which hold callbacks for these stubs, the syscalls are in spufs_calls and the coredump calls

[PATCH 08/15] Use spufs_coredump_num_notes everywhere, and don't NULL terminate

2007-09-12 Thread Michael Ellerman
The spufs_coredump_read array is NULL terminated, and we also store the size. We only need one or the other, and storing the size should save a teensy bit of memory vs NULL terminating, so do that. Signed-off-by: Michael Ellerman <[EMAIL PROTECTED]> ---

[PATCH 09/15] Internal __spufs_get_foo() routines should take a spu_context *

2007-09-12 Thread Michael Ellerman
The SPUFS attribute get routines take a void * because the generic attribute code doesn't know what sort of data it's passing around. However our internal __spufs_get_foo() routines can take a spu_context * directly, which saves plonking it in and out of a void * again. Signed-off-by: Michael

[PATCH 06/15] Correctly calculate the size of the local-store to dump

2007-09-12 Thread Michael Ellerman
The routine to dump the local store, __spufs_mem_read(), does not take the spu_lslr_RW value into account - so we shouldn't check it when we're calculating the size either. Signed-off-by: Michael Ellerman <[EMAIL PROTECTED]> --- arch/powerpc/platforms/cell/spufs/coredump.c | 16

[PATCH 07/15] Don't return -ENOSYS as extra notes size if spufs is not loaded

2007-09-12 Thread Michael Ellerman
Because the SPU coredump code might be built as part of a module (spufs), we have a stub which is called by the coredump code, this routine then calls into spufs if it's loaded. Unfortunately the stub returns -ENOSYS if spufs is not loaded, which is interpreted by the coredump code as an extra

[PATCH 05/15] Write some SPU coredump values as ASCII

2007-09-12 Thread Michael Ellerman
Unfortunately GDB expects some of the SPU coredump values to be identical in format to what is found in spufs. This means we need to dump some of the values as ASCII strings, not the actual values. Because we don't know what the values will be, we always print the values with the format

[PATCH 04/15] Use computed sizes/#defines rather than literals in SPU coredump code

2007-09-12 Thread Michael Ellerman
The spufs_coredump_reader array contains the size of the data that will be returned by the read routine. Currently these are specified as literals, and though some are obvious, sizeof(u32) == 4, others are not, 69 * 8 == ??? Instead, use sizeof() whatever type is returned by each routine, or in

[PATCH 01/15] Extract the file descriptor search logic in SPU coredump code

2007-09-12 Thread Michael Ellerman
Extract the logic for searching through the file descriptors for spu contexts into a separate routine, coredump_next_context(), so we can use it elsewhere in future. In the process we flatten the for loop, and move the NOSCHED test into coredump_next_context(). Signed-off-by: Michael Ellerman

[PATCH 02/15] Remove ctx_info and ctx_info_list

2007-09-12 Thread Michael Ellerman
Remove the ctx_info struct entirely, and also the ctx_info_list. This fixes a race where two processes can clobber each other's ctx_info structs. Instead of using the list, we just repeat the search through the file descriptor table. Signed-off-by: Michael Ellerman <[EMAIL PROTECTED]> ---

[PATCH 03/15] Call spu_acquire_saved() before calculating the SPU note sizes

2007-09-12 Thread Michael Ellerman
It makes sense to stop the SPU processes as soon as possible. Also if we dont acquire_saved() I think there's a possibility that the value in csa.priv2.spu_lslr_RW won't be accurate. Signed-off-by: Michael Ellerman <[EMAIL PROTECTED]> --- arch/powerpc/platforms/cell/spufs/coredump.c |2 ++ 1

[PATCH 1/2] dgrs: remove from build, config, and maintainer list

2007-09-12 Thread Nathanael Nerode
From: Nathanael Nerode Stop building and configuring driver for Digi RightSwitch, which was never actually sold to anyone, and remove it from MAINTAINERS. In response to an investigation into the firmware of the "Digi Rightswitch" driver, Andres Salomon discovered: > > Dear Andres: > > After

Re: [-mm patch] kernel/kexec.c: make code static

2007-09-12 Thread Ken'ichi Ohmichi
Hi Adrian, Maneesh, Maneesh Soni wrote: > On Mon, Sep 10, 2007 at 02:20:40PM +0200, Adrian Bunk wrote: >> On Mon, Sep 10, 2007 at 11:55:49AM +0900, Ken'ichi Ohmichi wrote: >>> Hi Adrian, >>> >>> >>> 2007/09/09 22:25:16 +0200, Adrian Bunk <[EMAIL PROTECTED]> wrote: On Fri, Aug 31, 2007 at

[PATCH] pci: fix unterminated pci_device_id lists

2007-09-12 Thread Kees Cook
This patch against 2.6.23-rc6 fixes a couple drivers that do not correctly terminate their pci_device_id lists. This results in garbage being spewed into modules.pcimap when the module happens to not have 28 NULL bytes following the table, and/or the last PCI ID is actually truncated from the

Re: Kernel Panic on 2.6.23-rc5 (solved)

2007-09-12 Thread Daniel Exner
Hi! Michal Piotrowski: > On 06/09/07, Daniel Exner <[EMAIL PROTECTED]> wrote: > > I'm not really sure if this is a regression or if I simply hit a hardware > > problem. > > After some time of work (mostly hours sometimes minutes) my system will > > freeze including Blinking LED's and

Re: [00/41] Large Blocksize Support V7 (adds memmap support)

2007-09-12 Thread Nick Piggin
On Wednesday 12 September 2007 11:49, David Chinner wrote: > On Tue, Sep 11, 2007 at 04:00:17PM +1000, Nick Piggin wrote: > > > > OTOH, I'm not sure how much buy-in there was from the filesystems > > > > guys. Particularly Christoph H and XFS (which is strange because they > > > > already do

Re: Socket-related problem in x86_64 Kernel (2.6.16.53-0.8-smp)?

2007-09-12 Thread Ulrich Windl
On 11 Sep 2007 at 17:04, Al Viro wrote: > On Tue, Sep 11, 2007 at 05:54:38PM +0200, Ulrich Windl wrote: > > > If not, any clues on debugging/tracing? There's a > > /usr/src/linux/Documentation/oops-tracing.txt, but no "segfault-tracing". > > That would be because it has fsck-all to do with the

Re: [announce] CFS-devel, performance improvements

2007-09-12 Thread Mike Galbraith
On Tue, 2007-09-11 at 22:04 +0200, Ingo Molnar wrote: > fresh back from the Kernel Summit, Peter Zijlstra and me are pleased to > announce the latest iteration of the CFS scheduler development tree. Our > main focus has been on simplifications and performance - and as part of > that we've also

Re: [BUGFIX] x86_64: NX bit handling in change_page_attr

2007-09-12 Thread Huang, Ying
On Tue, 2007-09-11 at 20:23 -0700, Andrew Morton wrote: > On Fri, 17 Aug 2007 13:28:38 +0800 "Huang, Ying" <[EMAIL PROTECTED]> wrote: > > > This patch fixes a bug of change_page_attr/change_page_attr_addr on > > Intel x86_64 CPU. After changing page attribute to be executable with > > these

Re: Problem charging blackberry 8700c with berry_charge (2.6.22.6)

2007-09-12 Thread Matt LaPlante
On Mon, 10 Sep 2007 23:35:02 -0700 Greg KH <[EMAIL PROTECTED]> wrote: > > > Sep 9 13:49:01 prizm kernel: [ 584.407498] drivers/usb/core/inode.c: > > creating file '003' > > Sep 9 13:49:01 prizm kernel: [ 584.407509] hub 5-0:1.0: state 7 ports 8 > > chg evt 0004 > > Sep 9 13:49:01

Re: hci_sock.c build failure

2007-09-12 Thread Coly Li
Coly Li 写道: David Miller 写道: From: Greg KH [EMAIL PROTECTED] Date: Wed, 12 Sep 2007 05:02:53 -0700 It's not even a randconfig issue, my build dies too (this is Linus's current tree.) Time to poke through the 10 bluetooth patches that were just added... I'll push the following fix to

Re: [00/41] Large Blocksize Support V7 (adds memmap support)

2007-09-12 Thread Martin J. Bligh
Christoph Lameter wrote: On Tue, 11 Sep 2007, Nick Piggin wrote: But that's not my place to say, and I'm actually not arguing that high order pagecache does not have uses (especially as a practical, shorter-term solution which is unintrusive to filesystems). So no, I don't think I'm really

Re: [PATCH] Scheduler Profiling - Use Immediate Values

2007-09-12 Thread Andi Kleen
The idea is not to hide the unlikely, but to leave the opportunity to make this primitive evolve in something that won't depend on a load immediate and only require patching of a jump, given the appropriate gcc support (yet to come). If that ever happens the code can be still changed. But i

Re: 2.6.23-rc6-git1 -- termios *_1 compile failures on powerpc

2007-09-12 Thread Josh Boyer
On 9/12/07, Andy Whitcroft [EMAIL PROTECTED] wrote: The following commit just hit mainline and all my powerpc test boxes are failing during compilation: commit f629307c857c030d5a3dd777fee37c8bb395e171 tty: termios locking functions break with new termios type I think this is

Re: 2.6.23-rc5: possible irq lock inversion dependency detected

2007-09-12 Thread David Miller
From: Herbert Xu [EMAIL PROTECTED] Date: Tue, 11 Sep 2007 20:43:27 +0800 On Tue, Sep 11, 2007 at 08:01:46AM -0400, jamal wrote: [NET_SCHED] protect action config/dump from irqs Looks good! Thanks Jamal. Applied, I'll try to push this in some time soon. - To unsubscribe from this list:

Re: stripping down the kernel-parameters.txt file

2007-09-12 Thread Rogan Dawes
Robert P. J. Day wrote: p.s. by basic, i mean those boot-time parms defined by either __setup() or early_param(). which means that module writers should, as much as possible, stop using those macros to define command-line parameters for their modules. that would go a long way to restoring

Re: [PATCH 08/12] IB/ehca: Replace get_paca()-paca_index by the more portable smp_processor_id()

2007-09-12 Thread Joachim Fenkes
On Tuesday 11 September 2007 16:51, Nathan Lynch wrote: - get_paca()-paca_index, __FUNCTION__, \ + smp_processor_id(), __FUNCTION__, \ I think I see these macros used in preemptible code (e.g. ehca_probe), where smp_processor_id()

[PATCH 08/12] IB/ehca: Replace get_paca()-paca_index by the more portable raw_smp_processor_id()

2007-09-12 Thread Joachim Fenkes
We can use raw_smp_processor_id() here because the processor ID is only used for debug output and may therefore be preemption-unsafe. Signed-off-by: Joachim Fenkes [EMAIL PROTECTED] --- This is the same patch, but with smp_processor_id() replaced by raw_smp_processor_id(), as kindly pointed out

Re: stripping down the kernel-parameters.txt file

2007-09-12 Thread Robert P. J. Day
On Wed, 12 Sep 2007, Rogan Dawes wrote: Robert P. J. Day wrote: p.s. by basic, i mean those boot-time parms defined by either __setup() or early_param(). which means that module writers should, as much as possible, stop using those macros to define command-line parameters for their

Re: [PATCH net-2.6.24] cxgb3 - napi update

2007-09-12 Thread David Miller
From: Divy Le Ray [EMAIL PROTECTED] Date: Sun, 09 Sep 2007 00:09:17 -0700 From: Divy Le Ray [EMAIL PROTECTED] Embed napi_struct directly into sge_qset. Use napi_schedule/napi_complete. Signed-off-by: Divy Le Ray [EMAIL PROTECTED] Patch applied, thanks a lot! - To unsubscribe from this

Re: [NFS] [patch] sunrpc: make closing of old temporary sockets work (was: problems with lockd in 2.6.22.6)

2007-09-12 Thread Wolfgang Walter
Am Mittwoch, 12. September 2007 15:37 schrieb J. Bruce Fields: On Wed, Sep 12, 2007 at 02:07:10PM +0200, Wolfgang Walter wrote: as already described old temporary sockets (client is gone) of lockd aren't closed after some time. So, with enough clients and some time gone, there are 80 open

Re: [patch 2/6] Linux Kernel Markers - Add kconfig menus for the marker code

2007-09-12 Thread Mathieu Desnoyers
* Randy Dunlap ([EMAIL PROTECTED]) wrote: On Thu, 06 Sep 2007 16:07:35 -0400 Mathieu Desnoyers wrote: [snip] This patch: Add Kconfig menus for the marker code. [EMAIL PROTECTED]: Never ever select MODULES] Signed-off-by: Mathieu Desnoyers [EMAIL PROTECTED] Reviewed-by:

[PATCH] update checkpatch.pl to version 0.10

2007-09-12 Thread Andy Whitcroft
This version brings a number of new checks, and a number of bug fixes. Of note: - better categorisation and space checks for dual use unary/binary operators - warn on deprecated use of {SPIN,RW}_LOCK_UNLOCKED - check if/for/while with trailing ';' for hanging statements - detect DOS

Define termios_1 functions for powerpc, s390, avr32 and frv

2007-09-12 Thread Paul Mackerras
Commit f629307c857c030d5a3dd777fee37c8bb395e171 introduced uses of kernel_termios_to_user_termios_1 and user_termios_to_kernel_termios_1 on all architectures. However, powerpc, s390, avr32 and frv don't currently define those functions since their termios struct didn't need to be changed when the

Re: Possible hang inside interrupt handler on sata_promise.

2007-09-12 Thread Mikael Pettersson
Robin Holt writes: I have been experiencing hangs on a newly setup machine. Unfortunately, it appears to be hanging inside the interrupt handler as sysrq and caps-lock led seem to stop working when the event occurs. I am guessing it is related to the sata_promise driver, but that is

Lossy interrupts on x86_64

2007-09-12 Thread Jesse Barnes
I just narrowed down a weird problem where I was losing more than 50% of my vblank interrupts to what seems to be the hires timers patch. Stock 2.6.23-rc5 works fine, but the latest (171) kernel from rawhide drops most of my interrupts unless I also have another interrupt source running (e.g.

Re: Kernel Panic - 2.6.23-rc4-mm1 ia64 - was Re: Update: [Automatic] NUMA replicated pagecache ...

2007-09-12 Thread Andy Whitcroft
On Wed, Sep 12, 2007 at 11:09:47AM -0400, Lee Schermerhorn wrote: Interesting, I don't see a memory controller function in the stack trace, but I'll double check to see if I can find some silly race condition in there. right. I noticed that after I sent the mail. Also, config

Re: [PATCH/RFC] doc: about email clients for Linux kernel patches

2007-09-12 Thread Adrian Bunk
On Wed, Sep 12, 2007 at 08:02:29AM -0700, Randy Dunlap wrote: Adrian Bunk wrote: On Wed, Sep 12, 2007 at 01:24:13PM +0800, WANG Cong wrote: On Tue, Sep 11, 2007 at 08:29:26PM +0200, Adrian Bunk wrote: On Tue, Sep 11, 2007 at 10:16:44AM -0700, Randy Dunlap wrote: ...

Re: [PATCH] ehca_irq: Misc cpuinit section annotations and #ifdef cleanups

2007-09-12 Thread Joachim Fenkes
On Thursday 23 August 2007 01:28, Satyam Sharma wrote: * Replace {un}register_cpu_notifier with {un}register_hotcpu_notifier thereby losing a couple of #ifdef HOTPLUG_CPU pairs. * Move comp_pool_callback_nb declaration to below that of callback function so that initialization of

Re: OOPS in __copy_user_zeroing_intel

2007-09-12 Thread Michal Piotrowski
Hi Chris, On 11/09/2007, Chris Friesen [EMAIL PROTECTED] wrote: Hi all, We're running a modified 2.6.10 on a dual-Xeon system. We've had a number of instances where we've seen oopses in the pipe code. I've included the most recent one below. This bug left us with a hung process as the

2.6.22.6: still seeing section mismatch warnings

2007-09-12 Thread Frank van Maarseveen
I'm still seeing the warnings below (2.6.22 started off with lots of section mismatch warning) but I have no idea if it is safe to ignore these: WARNING: arch/i386/kernel/built-in.o(.text+0xea62): Section mismatch: reference to .init.data:trampoline_end (between 'setup_trampoline' and

Re: hci_sock.c build failure

2007-09-12 Thread Marcel Holtmann
Hi Dave, It's not even a randconfig issue, my build dies too (this is Linus's current tree.) Time to poke through the 10 bluetooth patches that were just added... I'll push the following fix to Linus. the patch looks absolutely fine to me. You can put an Acked-by line to it. Sorry

Building a kernel-source RPM (not a kernel RPM)?

2007-09-12 Thread Dan Stromberg
I sent this to kernel newbies first, and while I got one response there, it answered a different question than the one I was asking... I'm on a SuSE system. I'm working on automating the install of said system, but it needs a Linus kernel - 2.6.21.7 specifically, and it needs kernel source too

2.6.23-rc6-git1 i386/allmodconfig broke

2007-09-12 Thread Jan Dittmer
Since -rc6: - i386/allmodconfig: broke CC [M] net/bluetooth/hci_core.o CC [M] net/bluetooth/hci_conn.o CC [M] net/bluetooth/hci_event.o CC [M] net/bluetooth/hci_sock.o net/bluetooth/hci_sock.c: In function 'hci_sock_cmsg': net/bluetooth/hci_sock.c:352: error: storage size

Re: Lossy interrupts on x86_64

2007-09-12 Thread Jesse Barnes
On Wednesday, September 12, 2007, Randy Dunlap wrote: On Wed, 12 Sep 2007 08:33:15 -0700 Jesse Barnes wrote: I just narrowed down a weird problem where I was losing more than 50% of my vblank interrupts to what seems to be the hires timers patch. Stock 2.6.23-rc5 works fine, but the latest

[2.6.23-rc6] i386 build breakage on current git

2007-09-12 Thread Jeff Garzik
Platform i386 seems to be missing struct compat_timeval, which causes the i386 allmodconfig build to break in current -git: net/bluetooth/hci_sock.c: In function ‘hci_sock_cmsg’: net/bluetooth/hci_sock.c:352: error: storage size of ‘ctv’ isn’t known net/bluetooth/hci_sock.c:352: warning: unused

<    1   2   3   4   5   6   7   8   >