Here are a few driver core fixes against your current git tree that fix some more problems that have cropped up: - shutdown problem due to logic problem with cpufreq usage of kobjects. - build fix for powerpc due to previous kobject changes. - runtime errors when CONFIG_SYSFS=n - UIO code now works properly from my previous messups - proper encoding of the ja_JP stable_kernel_rules.txt file - updates to the stable_kernel_rules.txt file - mark ide=reverse as obsolete in preparation of 2.6....

From: Stephen Rothwell <sfr@canb.auug.org.au> Introduced by commit 79393fc46ede43451a500a132e5de9856f5a4c83 ("kobject: convert pseries/power.c to kobj_attr interface"). sys_create_file takes a "struct attrbute *" not a "struct kobj_addribute *". arch/powerpc/platforms/pseries/power.c: In function 'apo_pm_init': arch/powerpc/platforms/pseries/power.c:78: warning: passing argument 2 of 'sysfs_create_file' from incompatible pointer type Signed-off-by: Stephen Rothwell <sfr@canb.auug.org....

From: Brandon Philips <brandon@ifup.org> This fixes two bugs with UIO that cropped up recently in -rc1 1) WARNING: at fs/sysfs/file.c:334 sysfs_open_file when trying to open a map addr/size file - complaining about missing sysfs_ops for ktype 2) Permission denied when reading uio/uio0/maps/map0/{addr,size} when files are mode S_IRUGO Also fix a typo: attr_attribute -> addr_attribute Signed-off-by: Brandon Philips <bphilips@suse.de> Signed-off-by: Hans J. Koch <hjk@l...

From: Bartlomiej Zolnierkiewicz <bzolnier@gmail.com> - it is valid only if "Probe IDE PCI devices in the PCI bus order (DEPRECATED)" config option is used - Greg needs to remove pci_get_device_reverse() for PCI core changes Cc: Alan Cox <alan@lxorguk.ukuu.org.uk> Acked-by: Sergei Shtylyov <sshtylyov@ru.mvista.com> Signed-off-by: Bartlomiej Zolnierkiewicz <bzolnier@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de> --- drivers/ide/ide.c | 2 +- 1...

From: Randy Dunlap <randy.dunlap@oracle.com> Fix drivers/base/ missing kernel-doc parameters: Warning(linux-2.6.24-git12//drivers/base/driver.c:133): No description found for parameter 'drv' Warning(linux-2.6.24-git12//drivers/base/driver.c:133): No description found for parameter 'kobj' Warning(linux-2.6.24-git12//drivers/base/driver.c:133): No description found for parameter 'fmt' Warning(linux-2.6.24-git12//drivers/base/power/main.c:530): No description found for parameter 'state' Signed-...

On Thu, Feb 21, 2008 at 03:47:59PM -0800, Greg Kroah-Hartman wrote: > From: Balaji Rao <balajirrao@gmail.com> > > The cpufreq core should not take an extra kobject reference count for no > reason, and then refuse to release it. This has been reported as > keeping machines from properly powering down all the way. > > > Signed-off-by: Balaji Rao <balajirrao@gmail.com> > Cc: Dave Jones <davej@codemonkey.org.uk> > Cc: Yi Yang <yi.y.yang@...

From: Tsugikazu Shibata <tshibata@ab.jp.nec.com> Fix the UTF-8 broken encodings in the ja_JP version of stable_kernel_rules.txt From: Tsugikazu Shibata <tshibata@ab.jp.nec.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de> --- Documentation/ja_JP/stable_kernel_rules.txt | 106 +++++++++++++------------- 1 files changed, 53 insertions(+), 53 deletions(-) diff --git a/Documentation/ja_JP/stable_kernel_rules.txt b/Documentation/ja_JP/stable_kernel_rules.txt index 17d8751....

On Fri, 22 Feb 2008 00:34:29 +1100 Stephen Rothwell <sfr@canb.auug.org.au> = Just to be clear, the "name" of a tree comes from the Name field in the Next/Trees file. --=20 Cheers, Stephen Rothwell sfr@canb.auug.org.au http://www.canb.auug.org.au/~sfr/

>>> On Thu, Feb 21, 2008 at 4:42 PM, in message <20080221214219.GA27209@elte.hu>, Note that the "3%" figure being thrown around was from a single patch within the series. We are actually getting a net average gain of 443% in dbench. And note that the number goes *up* when you remove the ticketlocks. The ticketlocks are there to prevent latency spikes, not improve throughput. Its very subtle to get this concept to work. The first few weeks, we were getting 90% regressions ;) ...

Remove the seq_file boilerplate used to construct the memcontrol stats map, and instead use the new map representation for cgroup control files Signed-off-by: Paul Menage <menage@google.com> --- mm/memcontrol.c | 30 ++++++------------------------ 1 file changed, 6 insertions(+), 24 deletions(-) Index: cgroupmap-2.6.25-rc2-mm1/mm/memcontrol.c =================================================================== --- cgroupmap-2.6.25-rc2-mm1.orig/mm/memcontrol.c +++ cgroupmap-2.6.25-rc2...

Adds a new type of supported control file representation, a map from strings to u64 values. The map type is printed in a similar format to /proc/meminfo or /proc/<pid>/status, i.e. "$key: $value\n" Signed-off-by: Paul Menage <menage@google.com> --- include/linux/cgroup.h | 19 +++++++++++++++ kernel/cgroup.c | 59 ++++++++++++++++++++++++++++++++++++++++++++++++- 2 files changed, 77 insertions(+), 1 deletion(-) Index: cgroupmap-2.6.25-rc2-mm1/include/linux/cgroup.h ...

Sorry, I should have added: v0.13 has no disk format changes since v0.12. -chris --

This is just wrong. The From: goes at the *top*, and if it's not there, my scripts won't pick it up as the author. Linus --

I'll fix it, but I want people to know so that I don't have to fix things like this in the future (*). Linus (*) I keed, I keed. Of *course* I'll have to fix things like this in the future too. But hopefully not quite as often. --

In addition to "From: ", you can place "Subject: " and "Date: " at the beginning to make sure Linus or whoever applies patches would not get the authorship information from the e-mail itself you send (this is especially true when you are forwarding somebody else's patch). Mnemonic is that the equivalents to E-mail headers go at the top. You will never have Signed-off-by:, Tested-by:, nor Acked-by: in your e-mail headers. They go at the bottom. This is just a toolsmith speaking, I am not in ...

These patches simplify the code required to report values from a res_counter object in a cgroups control file. The first patch adds res_counter_read_uint, which simply reports the current value for a res_counter member. The second replaces the existing mem_cgroup_read() with a simpler version that calls res_counter_read_uint(). Signed-off-by: Paul Menage <menage@google.com> -- --

Hi, While testing with hotplugging cpus today, I've came across a stack trace generated by lockdep. The reason for that is that do_boot_cpu() in smpboot_64.c ends up initializing a struct work_struct variable in the stack. These patches fix this by making it static. a per-cpu variable is choosen, since, to the best of my knowledge, nothing prevents two cpus going up at the same time, and so a single static c_idle won't do. I separated it in two patches to better isolate the change. The first p...

Sure. To make it clearer where the problem is, I also added the attached patch to my testing. Trace is: (note the call to do_fork_idle) INFO: trying to register non-static key. the code is fine but needs lockdep annotation. turning off the locking correctness validator. key: ffff81003bdf9d18, name: c_idle.work Pid: 11, comm: events/1 Not tainted 2.6.25-rc2 #129 Call Trace: [<ffffffff8104f757>] static_obj+0x5d/0x74 [<ffffffff81052218>] __lock_acquire+0x8b5/0xc3e [<fff...

This patch makes the work field in create_idle have a static address. Otherwise, being a stack variable, it can boild down the lockdep system. Signed-off-by: Glauber Costa <gcosta@redhat.com> --- arch/x86/kernel/smpboot_64.c | 13 +++++++------ 1 files changed, 7 insertions(+), 6 deletions(-) diff --git a/arch/x86/kernel/smpboot_64.c b/arch/x86/kernel/smpboot_64.c index f15b774..d79b7a8 100644 --- a/arch/x86/kernel/smpboot_64.c +++ b/arch/x86/kernel/smpboot_64.c @@ -545,6 +545,7 @@ s...

Indeed. I missed the static lock_class_key definition inside INIT_WORK macro. It does the job neatly. -- Glauber Costa. "Free as in Freedom" http://glommer.net "The less confident you are, the more serious you have to act." --

->busproc method is used by HDIO_SET_BUSSTATE ioctl but it has no chance of working as intended (in 2.4.x days) because to issue an ioctl there is a device node needed and: - for BUSSTATE_TRISTATE+OFF it is too late (devices are already gone) - for BUSSTATE_TRISTATE+ON it is too early (devices are not registered yet) Just remove ->busproc method for now (it was only implemented by hpt366, siimage and tc86c001 host drivers). Cc: Sergei Shtylyov <sshtylyov@ru.mvista.com> Signed-off...

NAK. This is NOT PAGE_SIZE (even though the two values are coincidentally the same, for historical reasons.) -hpa --

Historical reasons... it used to be a "magic page" -- that's long since gone. -hpa --

No problem; this is why we do reviews :) -hpa --

Hm. Perhaps it would be better to move whatever definition you need into a header in a common place (or move xen-ops.h entirely). J --