linux-kernel mailing list

FromSubjectsort iconDate
Mike Travis
Re: [PATCH 0/1] x86: Convert cpuinfo_x86 array to a per_cpu ...
Andrew Morton wrote: I'll take a look at this right away. Thanks, Mike -
Aug 3, 9:15 pm 2007
Mike Frysinger
custom baud rates with serial core
is there a method for setting custom baud rates that doesnt cause a "deprecated" message from showing up ? for example: # setserial -a /dev/ttyBF0 divisor 9 spd_cust setserial sets custom speed on ttyBF0. This is deprecated. my reading of the serial core code is that the only way to specify a baud rate that does not appear in the B* define list is by setting your baud to B38400 and utilizing UPF_SPD_CUST. this ends up in the deprecated warning code though in uart_set_info(). if it's deprecated...
Aug 3, 7:59 pm 2007
Jesper Juhl
[PATCH] UBI: Don't use signed int as array index before test...
Hi, I can't find anything guaranteeing that 'ubi_num' cannot be <0 in drivers/mtd/ubi/kapi.c::ubi_open_volume(), and in fact the code even tests for that and errors out if so. Unfortunately the test for "ubi_num < 0" happens after we've already used 'ubi_num' as an array index - bad thing to do if it is negative. This patch moves the test earlier in the function and then moves the indexing using that variable after the check. A bit safer :-) Signed-off-by: Jesper Juhl <jesper.ju...
Aug 3, 7:25 pm 2007
Roland McGrath
[PATCH] Add linux/elfcore-compat.h
This adds the linux/elfcore-compat.h header file, which is the CONFIG_COMPAT analog of the linux/elfcore.h header. Each arch that needs to fake out fs/binfmt_elf.c for its compat code can use this header to replace the hand-copied definitions of the compat variants of struct elf_prstatus et al. Only the pr_reg field varies by arch, so asm/{compat,elf}.h must define compat_elf_gregset_t before linux/elfcore-compat.h can be used. Signed-off-by: Roland McGrath <roland@redhat.com> --- incl...
Aug 3, 7:21 pm 2007
Roland McGrath
[PATCH] powerpc: Use linux/elfcore-compat.h
This makes powerpc64's compat code use the new linux/elfcore-compat.h, reducing some hand-copied duplication. Signed-off-by: Roland McGrath <roland@redhat.com> --- arch/powerpc/kernel/binfmt_elf32.c | 67 ++++++++++++++++------------------- include/asm-powerpc/elf.h | 38 +++++++++++---------- 2 files changed, 51 insertions(+), 54 deletions(-) diff --git a/arch/powerpc/kernel/binfmt_elf32.c b/arch/powerpc/kernel/binfmt_elf32.c index 5cb5875..1d45d77 100644 --- a/arch/power...
Aug 3, 7:23 pm 2007
Roland McGrath
[PATCH] x86_64: Use linux/elfcore-compat.h
This makes x86-64's ia32 code use the new linux/elfcore-compat.h, reducing some hand-copied duplication. Signed-off-by: Roland McGrath <roland@redhat.com> --- arch/x86_64/ia32/ia32_binfmt.c | 123 +++++++++++++++------------------------- include/asm-x86_64/compat.h | 6 ++ 2 files changed, 51 insertions(+), 78 deletions(-) diff --git a/arch/x86_64/ia32/ia32_binfmt.c b/arch/x86_64/ia32/ia32_binfmt.c index 90b37fc..57de329 100644 --- a/arch/x86_64/ia32/ia32_binfmt.c +++ b/arch/x8...
Aug 3, 7:22 pm 2007
Robert Hancock
Re: PROBLEM: snd-au8830: dead after software suspend
That driver will have to be updated to reinitialize the card properly after resume.. Have you reported this to the ALSA people? -- Robert Hancock Saskatoon, SK, Canada To email, remove "nospam" from hancockr@nospamshaw.ca Home Page: http://www.roberthancock.com/ -
Aug 3, 6:43 pm 2007
Joe Perches
Re: + remove-current-defines-and-uses-of-pr_err-add-pr_emerg...
Add new pr_<level> printk(KERN_<level> fmt "\n", ##arg) to kernel.h pr_info and pr_debug are unchanged Remove local pr_err #defines Convert current uses of pr_err Signed-off-by: Joe Perches <joe@perches.com> diff --git a/include/linux/kernel.h b/include/linux/kernel.h index 4300bb4..6447072 100644 --- a/include/linux/kernel.h +++ b/include/linux/kernel.h @@ -229,20 +229,24 @@ extern void print_hex_dump_bytes(const char *prefix_str, int prefix_type, void *buf, size_t le...
Aug 3, 6:16 pm 2007
Chuck Ebbert
Oops in 2.6.23-rc1-git9, arch/x86_64/pci/k8-bus.c::fill_mp_b...
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=250859 at line 74: muli@62829: muli@62829: sd = bus->sysdata; muli@62829: sd->node = node; <===== bus->sysdata is NULL. Last changed by this hunk of "x86-64: introduce struct pci_sysdata to facilitate sharing of ->sysdata": @@ -67,7 +69,9 @@ fill_mp_bus_to_cpumask(void) continue; if (!node_online(node)) node = 0; - bus->sysdata = (void *)node; + + sd = bus->sysdata; + ...
Aug 3, 6:10 pm 2007
Andrew Morton
Re: Oops in 2.6.23-rc1-git9, arch/x86_64/pci/k8-bus.c::fill_...
On Fri, 03 Aug 2007 18:10:03 -0400 Andy keeps trotting out a patch which will probably fix this, but for some reason it doesn't seem to make progress. -
Aug 3, 6:50 pm 2007
Rafael J. Wysocki
[PATCH] PM: Fix dependencies of CONFIG_SUSPEND and CONFIG_HI...
From: Rafael J. Wysocki <rjw@sisk.pl> Dependencies of CONFIG_SUSPEND and CONFIG_HIBERNATION introduced by commit 296699de6bdc717189a331ab6bbe90e05c94db06 "Introduce CONFIG_SUSPEND for suspend-to-Ram and standby" are incorrect, as they don't cover the facts that (1) not all architectures support suspend and (2) SMP hibernation is only possible on X86 and PPC64 (if PPC64_SWSUSP is set). Signed-off-by: Rafael J. Wysocki <rjw@sisk.pl> --- kernel/power/Kconfig | 9 ++++++--- 1 file c...
Aug 3, 5:59 pm 2007
Linus Torvalds
Re: [PATCH] PM: Fix dependencies of CONFIG_SUSPEND and CONFI...
Please, do that as SUSPEND_UP_POSSIBLE, and make it look like config SUSPEND_UP_POSSIBLE depends on BLACKFIN || MIPS || SUPERH || FRV || ((PPC32 && PPC_MPC52xx) depends on !SMP default y and now you can write the above thing as depends on SUSPEND_SMP_POSSIBLE || SUSPEND_UP_POSSIBLE and it's all much nicer. I would say that doing a similar setup for HIBERNATE would be nice too, but the SUSPEND case is more noticeable because the SMP case is so nice already. Li...
Aug 3, 6:01 pm 2007
Rafael J. Wysocki
Re: [PATCH] PM: Fix dependencies of CONFIG_SUSPEND and CONFI...
I did it symmetrically for suspend and hibernation and renamed SUSPEND_SMP to PM_SLEEP_SMP. Patch follows. Greetings, Rafael --- From: Rafael J. Wysocki <rjw@sisk.pl> Dependencies of CONFIG_SUSPEND and CONFIG_HIBERNATION introduced by commit 296699de6bdc717189a331ab6bbe90e05c94db06 "Introduce CONFIG_SUSPEND for suspend-to-Ram and standby" are incorrect, as they don't cover the facts that (1) not all architectures support suspend and (2) SMP hibernation is only possible on X86 an...
Aug 3, 6:47 pm 2007
Gert Robben
PROBLEM: snd-au8830: dead after software suspend
After a suspend-resume cycle (using Suspend2), my sound card doesn't work anymore. It works again after reloading the module. Thanks for any help! Gert Robben -
Aug 3, 5:58 pm 2007
Oleg Nesterov
[PATCH] kill an obsolete sub-thread-ptrace stuff
There is a couple of subtle checks which were needed to handle ptracing from the same thread group. This was deprecated a long ago, imho this code just complicates the understanding. And, the "->parent->signal->flags & SIGNAL_GROUP_EXIT" check in exit_notify() is not right. SIGNAL_GROUP_EXIT can mean exec(), not exit_group(). This means ptracer can lose a ptraced zombie on exec(). Minor problem, but still the bug. Signed-off-by: Oleg Nesterov <oleg@tv-sign.ru> --- t/kernel/ex...
Aug 3, 5:04 pm 2007
Roland McGrath
Re: [PATCH] kill an obsolete sub-thread-ptrace stuff
This bug never happens because this check is only in the now-impossible ptrace-same-group case. The code can certainly go. Thanks, Roland -
Aug 3, 5:51 pm 2007
Oleg Nesterov
Re: [PATCH] kill an obsolete sub-thread-ptrace stuff
This means that changelog is wrong and should be changed. However, I disagree. exit_notify() doesn't check the ptrace-same-group case. So, unless I missed something, we set EXIT_DEAD in any case, even if ptracer doesn't belong to our thread group. No? Oleg. -
Aug 3, 6:00 pm 2007
Roland McGrath
Re: [PATCH] kill an obsolete sub-thread-ptrace stuff
Oh, you're right. I was reading the other line you changed. Thanks, Roland -
Aug 3, 6:06 pm 2007
George G. Davis
[PATCH] Fix local_irq_* macro definition thinkos for the !TR...
From: George G. Davis <gdavis@mvista.com> Fix local_irq_* macro definition errors for the !TRACE_IRQFLAGS_SUPPORT case in which the macros are errantly "redefined" rather than defined. Signed-off-by: George G. Davis <gdavis@mvista.com> diff --git a/include/linux/irqflags.h b/include/linux/irqflags.h index 412e025..49a3df8 100644 --- a/include/linux/irqflags.h +++ b/include/linux/irqflags.h @@ -67,10 +67,10 @@ * The local_irq_*() APIs are equal to the raw_local_irq*() * if !TR...
Aug 3, 4:48 pm 2007
Meelis Roos
spurious NCQ completions from FUJITSU MHW2120BH
Todays git gave me some EH errors during fsck (30 times mounted and time to check), then recovered and works fine. Seems to be spurious NCQ completion. The disk is Device Model: FUJITSU MHW2120BH Firmware Version: 00000012 User Capacity: 120
Aug 3, 4:47 pm 2007
Milan Plzik
[PATCH] pxa2xx PCMCIA timing issue on iPAQ H5550
Good day, recently I've been trying to get working PCMCIA interface on H5000 ipaq series, using dual PCMCIA sleeve. So far things work correctly, but I had to do one modification to drivers/pcmcia/pxa2xx_base.c to get the interface working with orinoco gold PCMCIA card (wired pcnet_cs ethernet card worked even without this modification). Patch attached. The issue has something to do with assert time on PCMCIA bus, but I'm not really sure what -- I found the working value just by trial&e...
Aug 3, 4:22 pm 2007
Oleg Nesterov
Re: Processes spinning forever, apparently in lock_timer_bas...
I don't think there is an unfairness problem. We are looping only if timer->base changes in between. IOW, there is no "lock + unlock + lock(same_lock)" here, we take another lock on each iteration. All traces start from schedule_timeout()->mod_timer(). This timer is local, nobody can see it, its ->base can't be NULL or changed. This means that lock_timer_base() can't loop, but can't take the tvec_t_base_s.lock. But in that case the trace should different, strange. Oleg. -
Aug 3, 4:14 pm 2007
Miklos Szeredi
[patch v2] fuse: fix permission checking on sticky directories
From: Miklos Szeredi <mszeredi@suse.cz> The VFS checks sticky bits on the parent directory even if the filesystem defines it's own ->permission(). In some situations (sshfs, mountlo, etc) the user does have permission to delete a file even if the attribute based checking would not allow it. So work around this by storing the permission bits separately and returning them in stat(), but cutting the permission bits off from inode->i_mode. This is slightly hackish, but it's probably n...
Aug 3, 4:03 pm 2007
Adrian McMenamin
[PATCH] SH: add machine-ops and Dreamcast specific fix-up
(This code is closely modelled on the i386 machine-ops, though currently the only board specific fixup is for the Dreamcast and so there is not a separate set of mach fixups.) This needs to be tested against a wide range of SH boards and would be good to go in -mm if Paul acks it. Add machine-ops code to the SH code base, allowing board specific reboot and halt code. Currently only Dreamcast specific warm reboot fixup in code. Signed-off by: Adrian McMenamin <adrian@mcmen.demon.co.uk> ...
Aug 3, 3:26 pm 2007
Paul E. McKenney
[PATCH RFC -rt] synchronize_sched() without migration
The combination of CPU hotplug and PREEMPT_RCU has resulted in deadlocks due to the migration-based implementation of synchronize_sched() in -rt. This experimental patch maps synchronize_sched() back onto Classic RCU, eliminating the migration, thus hopefully also eliminating the deadlocks. It is not clear that this is a good long-term approach, but it will at least permit people doing CPU hotplug in -rt kernels additional wiggle room in their design and implementation. The basic approach is to cau...
Aug 3, 3:12 pm 2007
Juergen Kreileder
lvcreate on 2.6.22.1: kernel tried to execute NX-protected p...
Hi, I got the appended BUG from a 32-bit 2.6.22.1 kernel (with exec-shield patch and PAE enabled) on an Athlon64 with dmsetup 1.02.03 and lvm2 v2.02.02. (Note, the message comes from the vanilla kernel, not from the exec-shiled patch.) I wasn't able to reproduce the problem so far. The machine creates several snapshot volumes every 4 hours and worked fine with the new kernel for several days. It had 2.6.16.12+exec-shield before and ran flawlessy for over a year. kernel tried to execute N...
Aug 3, 3:10 pm 2007
Javier Pello
[PATCH] request_firmware: skip timeout if userspace was not ...
Hi, I have prepared a patch that makes request_firmware skip the usual grace period that it gives firmware images to show up, if it determines that userspace was not notified at all. When request_firmware is called, it sends an event to userspace to ask for the firmware image that is needed, and then it installs a timeout so that it will not wait forever for the image. However, if userspace did not receive the event at all (no /sbin/hotplug, for instance), then having the kernel wait is pointles...
Aug 3, 3:07 pm 2007
Paul E. McKenney
[PATCH RFC -rt] Allow rcutorture to handle synchronize_sched()
The rcutorture module uses a random-number generator that periodically calls get_random_bytes() to add entropy. This random-number generator is invoked from within the RCU read-side critical section, which, in the case of the synchronize_sched() variant of RCU, has preeemption disabled. Unfortunately, get_random_bytes() ends up acquiring normal spinlocks, which can block in -rt, resulting in very large numbers of "scheduling while atomic" messages. This patch takes a very crude approach, simply su...
Aug 3, 2:44 pm 2007
Josh Triplett
Re: [PATCH RFC -rt] Allow rcutorture to handle synchronize_s...
Acked-by: Josh Triplett <josh@kernel.org> -
Aug 3, 6:38 pm 2007
Robert Hancock
Asus A8N-SLI Deluxe latest BIOS wants acpi_use_timer_override
It appears that the latest BIOS version 1805 for the ASUS A8N-SLI Deluxe board fixes the broken ACPI timer override that was in earlier versions, and which the kernel ignores on nForce4 chipsets. Now the same override: ACPI: INT_SRC_OVR (bus 0 bus_irq 0 global_irq 2 dfl dfl) seems to be needed, since without acpi_use_timer_override we end up getting this (with apic=verbose): ..TIMER: vector=0x30 apic1=0 pin1=0 apic2=-1 pin2=-1 ..MP-BIOS bug: 8254 timer not connected to IO-APIC ...trying to set...
Aug 3, 2:38 pm 2007
Andi Kleen
Re: Asus A8N-SLI Deluxe latest BIOS wants acpi_use_timer_ove...
I have pending patches to key the NF5 case on the PCI-ID, but that obviously cannot be done for NF4. I cannot think of anything good but putting DMI BIOS version We don't really know which pins are connected where. This means they likely have some bits they tweaked to get the different behaviour, but they're unknown. There are also some northbridge bits that can be tweaked (e.g. some of the ATI BIOS workarounds used to use that) but it's still unknown what the Nvidia chipset actually does a...
Aug 3, 4:01 pm 2007
Len Brown
Re: Asus A8N-SLI Deluxe latest BIOS wants acpi_use_timer_ove...
Is it possible to open a bugzilla and drop the acpidump and dmesg from both the new and old BIOS in it? Assuming the override in both cases is identical, I'm really baffled by this failure. It seems that the new BIOS is playing some tricks that we have not seen before, as your hardware didn't change... thanks, -Len -
Aug 3, 4:27 pm 2007
Andi Kleen
Re: Asus A8N-SLI Deluxe latest BIOS wants acpi_use_timer_ove...
My guess is that they changed some bits outside the DSDT. But ok There is some flexibility in how the interrupt gets delivered, since it gets converted to a HT message and the Northbridge has a few tweaks for processing those. -Andi -
Aug 3, 4:36 pm 2007
Jonathan Corbet
[PATCH] msleep() with hrtimers
Here's the second (and probably final) posting of the msleep() with hrtimers patch. The problem being addressed here is that the current msleep() will stop for a minimum of two jiffies, meaning that, on a HZ=100 system, msleep(1) delays for for about 20ms. In a driver with one such delay for each of 150 or so register setting operations, the extra time adds up to a few seconds. This patch addresses the situation by using hrtimers. On tickless systems with working timers, msleep(1) now sleeps for...
Aug 3, 2:37 pm 2007
Roman Zippel
Re: [PATCH] msleep() with hrtimers
Hi, The main point is still that these are two _different_ APIs for different usages, so I still prefer to add a hrsleep() instead. bye, Roman -
Aug 3, 3:19 pm 2007
Arjan van de Ven
Re: [PATCH] msleep() with hrtimers
I would actually prefer it the other way around; call the not-so-accurate one "msleep_approx()" or somesuch, to make it explicit that the sleep is only approximate... -
Aug 3, 3:46 pm 2007
Roman Zippel
Re: [PATCH] msleep() with hrtimers
Hi, Actually the hrsleep() function would allow for submillisecond sleeps, which might be what some of the 450 users really want and they only use msleep(1) because it's the next best thing. A hrsleep() function is really what makes most sense from an API perspective. bye, Roman -
Aug 3, 3:58 pm 2007
Arjan van de Ven
Re: [PATCH] msleep() with hrtimers
I respectfully disagree. The power of msleep is that the unit of sleep time is in the name; so in your proposal it would be hr_msleep or somesuch. I much rather do the opposite in that case; make the "short" name be the best implementation of the requested behavior, and have qualifiers for allowing exceptions to that... least surprise and all that. -- if you want to mail me at work (you don't), use arjan (at) linux.intel.com Test the interaction between Linux and your BIOS via [ message continues ]
" title="http://www.linuxfir...">http://www.linuxfir...
Aug 3, 7:53 pm 2007
Ingo Molnar
Re: [PATCH] msleep() with hrtimers
still looks good to me. Acked-by: Ingo Molnar <mingo@elte.hu> Ingo -
Aug 3, 2:54 pm 2007
Robert Hancock
[PATCH -mm] libata: add human-readable error value decoding v3
This adds human-readable decoding of the ATA status and error registers (similar to what drivers/ide does) as well as the SATA Serror register to libata error handling output. This prevents the need to pore through standards documents to figure out the meaning of the bits in these registers when looking at error reports. Some bits that drivers/ide decoded are not decoded here, since the bits are either command-dependent or obsolete, and properly parsing them would add too much complexity. Signed-...
Aug 3, 2:00 pm 2007
Chuck Ebbert
Re: [PATCH -mm] libata: add human-readable error value decod...
This is really nice. Maybe make it configurable, though? -
Aug 3, 7:18 pm 2007
Alan Cox
Re: [PATCH -mm] libata: add human-readable error value decod...
On Fri, 03 Aug 2007 12:00:57 -0600 Not sure its really worth it. Does "ABRT IDNF" actually tell you any more than the hex code ? In fact I find the hex code clearer because its usually the combination of state (0x51) etc that tells the story. No objection to it going in and on a correctness basis Acked-by: Alan Cox <alan@redhat.com> -
Aug 3, 5:54 pm 2007
Anton Arapov
SysV IPC: shmctl/msgctl/semctl returns EIDRM instead of EINVAL
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. Create then drop 2 shmem segments, then create a third. 2. Try to shmctl(IPC_STAT) the two now-invalid shm IDs. 3. Note error codes returned. One call gives EINVAL, one gives EIDRM due to collision with the third shmem segment. Should both give EINVAL, this is what I've got on every other Unix I've tried it on. ...
Aug 3, 1:59 pm 2007
Miklos Szeredi
[patch 11/11] fuse: no ENOENT from fuse device read
From: Miklos Szeredi <mszeredi@suse.cz> Don't return -ENOENT for a read() on the fuse device when the request was aborted. Instead return -ENODEV, meaning the filesystem has been force-umounted or aborted. Previously ENOENT meant that the request was interrupted, but now the 'aborted' flag is not set in case of interrupts. Signed-off-by: Miklos Szeredi <mszeredi@suse.cz> --- Index: linux/fs/fuse/dev.c =================================================================== --- linu...
Aug 3, 1:44 pm 2007
Miklos Szeredi
[patch 10/11] fuse: no abort on interrupt
From: Miklos Szeredi <mszeredi@suse.cz> Don't set 'aborted' flag on a request if it's interrupted. We have to wait for the answer anyway, and this would only a very little time while copying the reply. This means, that write() on the fuse device will not return -ENOENT during normal operation, only if the filesystem is aborted by a forced umount or through the fusectl interface. This could simplify userspace code somewhat when backward compatibility with earlier kernel versions is not r...
Aug 3, 1:44 pm 2007
Miklos Szeredi
[patch 09/11] fuse: cleanup in release
From: Miklos Szeredi <mszeredi@suse.cz> Move dput/mntput pair from request_end() to fuse_release_end(), because there's no other place they are used. Signed-off-by: Miklos Szeredi <mszeredi@suse.cz> --- Index: linux/fs/fuse/dev.c =================================================================== --- linux.orig/fs/fuse/dev.c 2007-08-03 18:36:01.000000000 +0200 +++ linux/fs/fuse/dev.c 2007-08-03 18:37:24.000000000 +0200 @@ -231,8 +231,6 @@ static void request_end(struct fuse_conn ...
Aug 3, 1:44 pm 2007
Miklos Szeredi
[patch 08/11] fuse: fix permission checking on sticky direct...
From: Miklos Szeredi <mszeredi@suse.cz> The VFS checks sticky bits on the parent directory even if the filesystem defines it's own ->permission(). In some situations (sshfs, mountlo, etc) the user does have permission to delete a file even if the attribute based checking would not allow it. So work around this by storing the permission bits separately and returning them in stat(), but cutting the permission bits off from inode->i_mode. This is slightly hackish, but it's probably n...
Aug 3, 1:44 pm 2007
Jan Engelhardt
Re: [patch 08/11] fuse: fix permission checking on sticky di...
^ Although it does not matter in this case, | is safer (and perhaps faster?). Jan -- -
Aug 3, 2:20 pm 2007
Miklos Szeredi
[patch 07/11] fuse: refresh stale attributes in fuse_permiss...
From: Miklos Szeredi <mszeredi@suse.cz> fuse_permission() didn't refresh inode attributes before using them, even if the validity has already expired. Thanks to Junjiro Okajima for spotting this. Also remove some old code to unconditionally refresh the attributes on the root inode. Signed-off-by: Miklos Szeredi <mszeredi@suse.cz> --- Index: linux/fs/fuse/dir.c =================================================================== --- linux.orig/fs/fuse/dir.c 2007-08-03 18:36:01....
Aug 3, 1:44 pm 2007
Miklos Szeredi
[patch 06/11] fuse: set i_nlink to sane value after mount
From: Miklos Szeredi <mszeredi@suse.cz> Aufs seems to depend on a positive i_nlink value. So fill in a dummy but sane value for the root inode at mount time. The inode attributes are refreshed with the correct values at the first opportunity. Signed-off-by: Miklos Szeredi <mszeredi@suse.cz> --- Index: linux/fs/fuse/inode.c =================================================================== --- linux.orig/fs/fuse/inode.c 2007-08-03 18:36:13.000000000 +0200 +++ linux/fs/fuse/ino...
Aug 3, 1:44 pm 2007
previous daytodaynext day
August 2, 2007August 3, 2007August 4, 2007