[PATCH 08/28] proc: Move PDE_NET() to fs/proc/proc_net.c [RFC]

2013-04-16 Thread David Howells
Move PDE_NET() to fs/proc/proc_net.c as that's where the only user is. Signed-off-by: David Howells --- fs/proc/proc_net.c |4 include/linux/proc_fs.h |5 - 2 files changed, 4 insertions(+), 5 deletions(-) diff --git a/fs/proc/proc_net.c b/fs/proc/proc_net.c index b4ac657

[PATCH 10/28] proc: Add proc_mkdir_data() [RFC]

2013-04-16 Thread David Howells
Add proc_mkdir_data() to allow procfs directories to be created that are annotated at the time of creation with private data rather than doing this post-creation. This means no access is then required to the proc_dir_entry struct to set this. Signed-off-by: David Howells cc: Neela Syam Kolli cc

[PATCH 15/28] reiserfs: Don't access the proc_dir_entry in r_open(), r_start() r_show() [RFC]

2013-04-16 Thread David Howells
Don't access the proc_dir_entry in ReiserFS's r_open(), r_start() r_show() procfs interface functions. ReiserFS stores the ->show() method pointer in PDE->data and the super_block pointer in PDE->parent->data. This isn't changing. Currently, ReiserFS passes the PDE pointer into seq_file::private

[PATCH 19/28] drm: proc: Use remove_proc_subtree() [RFC]

2013-04-16 Thread David Howells
Use remove_proc_subtree() rather than remove_proc_entry() to remove a minor-specific drm proc directory and all its children. Things could theoretically be improved by storing the drm_minor pointer in the minor-specific dir proc_dir_entry struct data and then scrapping the list of proc files - but

[PATCH 09/28] proc: Move some bits from linux/proc_fs.h to linux/{of.h, signal.h, tty.h} [RFC]

2013-04-16 Thread David Howells
Move some bits from linux/proc_fs.h to linux/of.h, signal.h and tty.h. Also move proc_tty_init() and proc_device_tree_init() to fs/proc/internal.h as they're internal to procfs. Signed-off-by: David Howells cc: devicetree-disc...@lists.ozlabs.org cc: linux-a...@vger.kernel.org cc: Greg Kroah-Har

[PATCH 23/28] ppc: Clean up scanlog [RFC]

2013-04-16 Thread David Howells
Clean up the pseries scanlog driver's use of procfs: (1) Don't need to save the proc_dir_entry pointer as we have the filename to remove with. (2) Save the scan log buffer pointer in a static variable (there is only one of it) and don't save it in the PDE (which doesn't have a destruc

[PATCH 24/28] proc: Supply an accessor to get the name in a proc_dir_entry struct [RFC]

2013-04-16 Thread David Howells
Supply an accessor to get access to the name in a proc_dir_entry struct, just returning a const pointer to it. This is only needed by the xt_hashlimit netfilter module as that appears to use the name in the pde to save a label in the xt_hashlimit_htable struct - which will be a problem if CONFIG_P

[PATCH 25/28] proc: Supply an accessor to get the process ID associated with some proc files [RFC]

2013-04-16 Thread David Howells
Supply an accessor to get the process ID associated with some proc files and directories (get_proc_pid()). Signed-off-by: David Howells cc: Tejun Heo cc: Li Zefan cc: contain...@lists.linux-foundation.org cc: cgro...@vger.kernel.org --- fs/proc/base.c |5 + include/linux/proc

[PATCH 28/28] proc: Move non-public stuff from linux/proc_fs.h to fs/proc/internal.h [RFC]

2013-04-16 Thread David Howells
Move non-public declarations and definitions from linux/proc_fs.h to fs/proc/internal.h. Signed-off-by: David Howells --- fs/proc/internal.h | 307 ++- fs/proc/kcore.c |1 include/linux/proc_fs.h | 151 ++- 3 fi

[PATCH 26/28] proc: Supply a function to remove a proc entry by PDE [RFC]

2013-04-16 Thread David Howells
Supply a function (proc_remove()) to remove a proc entry (and any subtree rooted there) by proc_dir_entry pointer rather than by name and (optionally) root dir entry pointer. This allows us to eliminate all remaining pde->name accesses outside of procfs. Signed-off-by: David Howells cc: linux-a.

[PATCH 27/28] proc: Make the PROC_I() and PDE() macros internal to procfs [RFC]

2013-04-16 Thread David Howells
Make the PROC_I() and PDE() macros internal to procfs. This means making PDE_DATA() out of line. This could be made more optimal by storing PDE()->data into inode->i_private. Also provide a __PDE_DATA() that is inline and internal to procfs. Signed-off-by: David Howells --- fs/proc/generic.c

[PATCH 17/28] drm: Constify drm_proc_list[] [RFC]

2013-04-16 Thread David Howells
Constify drm_proc_list[] and related pointers. Signed-off-by: David Howells cc: dri-de...@lists.freedesktop.org --- drivers/gpu/drm/drm_proc.c |6 +++--- include/drm/drmP.h |2 +- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/gpu/drm/drm_proc.c b/drivers

[PATCH 20/28] hostap: proc: Use remove_proc_subtree() [RFC]

2013-04-16 Thread David Howells
Use remove_proc_subtree() rather than remove_proc_entry() to remove a device-specific proc directory and all its children. Signed-off-by: David Howells cc: Jouni Malinen cc: Johannes Berg cc: linux-wirel...@vger.kernel.org cc: de...@driverdev.osuosl.org --- drivers/net/wireless/hostap/hostap_

[PATCH 22/28] ppc: Clean up rtas_flash driver somewhat [RFC]

2013-04-16 Thread David Howells
Clean up some of the problems with the rtas_flash driver: (1) It shouldn't fiddle with the internals of the procfs filesystem (altering pde->count). (2) If pid namespaces are in effect, then you can get multiple inodes connected to a single pde, thereby rendering the pde->count > 2 te

[PATCH 21/28] dgrp: Clean up the use of procfs [RFC]

2013-04-16 Thread David Howells
Clean up the use of procfs by the dgrp driver: (1) Use remove_proc_subtree() for the mass slaughter of a subdir full of proc files rather than doing it manually. (2) When creating files, only call ID_TO_CHAR() once to generate the name. Signed-off-by : David Howells cc: Bill Pemberton c

Re: [PATCH] fuse: use kernel headers when __KERNEL__ is set

2013-04-16 Thread Colin Cross
On Tue, Apr 16, 2013 at 11:21 AM, Linus Torvalds wrote: >> What I meant is IF is included by userspace (it sure is >> meant to be included and *is* included by libfuse and other stuff) >> THEN using instead of is fine regardless of >> whether __linux__ is defined or not. > > That's probably tru

[PATCH 18/28] drm: proc: Use minor->index to label things, not PDE->name [RFC]

2013-04-16 Thread David Howells
Use minor->index to label things, not the name field from the proc_dir_entry of the /proc/dwm// directory. Also, use "%u" not "%d" to render the value and use a 12-byte buffer in which to render the integer, not a 16-byte buffer. The longest string an unsigned int can give you is 10 chars (429496

[PATCH 14/28] proc: Supply an accessor for getting the data from a PDE's parent [RFC]

2013-04-16 Thread David Howells
Supply an accessor function for getting the private data from the parent proc_dir_entry struct of the proc_dir_entry struct associated with an inode. ReiserFS, for instance, stores the super_block pointer in the proc directory it makes for that super_block, and a pointer to the respective seq_file

[PATCH 13/28] airo: Use remove_proc_subtree() [RFC]

2013-04-16 Thread David Howells
Use remove_proc_subtree() to remove the airo device subdir and all its children instead of doing it manually. Signed-off-by: David Howells cc: linux-wirel...@vger.kernel.org --- drivers/net/wireless/airo.c | 49 +++ 1 file changed, 13 insertions(+), 36

[PATCH net] hyperv: Fix a compiler warning in netvsc_send()

2013-04-16 Thread Haiyang Zhang
Fixed: warning: cast from pointer to integer of different size Reported-by: kbuild test robot Signed-off-by: Haiyang Zhang Reviewed-by: K. Y. Srinivasan --- drivers/net/hyperv/netvsc.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/net/hyperv/netvsc.c b/dr

Re: [PATCH] PCI / PM: Fix fallback to PCI_D0 in pci_platform_power_transition()

2013-04-16 Thread Bjorn Helgaas
On Tue, Apr 16, 2013 at 12:01 PM, Rafael J. Wysocki wrote: > On Monday, April 15, 2013 02:18:18 PM Bjorn Helgaas wrote: >> On Fri, Apr 12, 2013 at 11:50 PM, Yinghai Lu wrote: >> > On Fri, Apr 12, 2013 at 4:58 PM, Rafael J. Wysocki wrote: >> >> From: Rafael J. Wysocki >> >> >> >> Commit b51306c

[PATCH 16/28] zoran: Don't print proc_dir_entry data in debug [RFC]

2013-04-16 Thread David Howells
Don't print proc_dir_entry data in debug as we're soon to have no direct access to the contents of the PDE. Print what was put in there instead. Signed-off-by: David Howells cc: mjpeg-us...@lists.sourceforge.net cc: linux-me...@vger.kernel.org --- drivers/media/pci/zoran/zoran_procfs.c |2

[PATCH 00/28] Privatise procfs internals [RFC]

2013-04-16 Thread David Howells
Here is a series of patches to make the procfs internals private to the procfs filesystem. This is built on top of the patches to eliminate create_proc_read_entry() after the kill-read_proc_t tag. These patches include fixes for the places that are attempting to abuse proc_dir_entry->count and d

[PATCH 06/28] proc: Move proc_fd() to fs/proc/fd.h [RFC]

2013-04-16 Thread David Howells
Move proc_fd() to fs/proc/fd.h. Signed-off-by: David Howells --- fs/proc/fd.h |5 + fs/proc/internal.h |5 - 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/fs/proc/fd.h b/fs/proc/fd.h index cbb1d47..7c047f2 100644 --- a/fs/proc/fd.h +++ b/fs/proc/fd.h @@ -1

[PATCH 07/28] proc: Split the namespace stuff out into linux/proc_ns.h [RFC]

2013-04-16 Thread David Howells
Split the proc namespace stuff out into linux/proc_ns.h. Signed-off-by: David Howells cc: net...@vger.kernel.org cc: Serge E. Hallyn cc: Eric W. Biederman --- fs/namespace.c |6 ++-- fs/proc/inode.c |8 ++--- fs/proc/namespaces.c | 17 +++ include/linu

[PATCH 11/28] rtl8187se: Use a dir under /proc/net/r8180/ [RFC]

2013-04-16 Thread David Howells
Create a dir under /proc/net/r8180/ named for the device and create that device's files under there. This means that there won't be a problem for multiple devices in the system (if such is possible) and it means we don't need to save the 'device directory' PDE any more as we can just do a proc sub

[PATCH 12/28] rtl8192u: Don't need to save device proc dir PDE [RFC]

2013-04-16 Thread David Howells
Don't need to save the PDE of a directory created under /proc/net/rtl8192/ as we can use proc subtree deletion to get rid of it and all its children. Signed-off-by: David Howells cc: Jerry Chuang cc: Mauro Carvalho Chehab cc: linux-wirel...@vger.kernel.org cc: de...@driverdev.osuosl.org --- d

[PATCH 04/28] proc: Supply PDE attribute setting accessor functions [RFC]

2013-04-16 Thread David Howells
Supply accessor functions to set attributes in proc_dir_entry structs. The following are supplied: proc_set_size() and proc_set_user(). Signed-off-by: David Howells cc: linuxppc-...@lists.ozlabs.org cc: linux-me...@vger.kernel.org cc: net...@vger.kernel.org cc: linux-wirel...@vger.kernel.org cc:

Re: [PATCH v3 linux-next] cpufreq: ondemand: Calculate gradient of CPU load to early increase frequency

2013-04-16 Thread Stratos Karafotis
On 04/10/2013 06:22 AM, Viresh Kumar wrote: On 9 April 2013 22:26, Stratos Karafotis wrote: On 04/05/2013 10:50 PM, Stratos Karafotis wrote: Hi Viresh, On 04/04/2013 07:54 AM, Viresh Kumar wrote: Hi Stratos, Yes, your results show some improvements. BUT if performance is the only thing we

[PATCH 03/28] proc: Split kcore bits from linux/procfs.h into linux/kcore.h [RFC]

2013-04-16 Thread David Howells
Split kcore bits from linux/procfs.h into linux/kcore.h. Signed-off-by: David Howells cc: linux-m...@linux-mips.org cc: sparcli...@vger.kernel.org cc: x...@kernel.org cc: linux...@kvack.org --- arch/mips/mm/init.c |1 + arch/score/mm/init.c|2 +- arch/x86/mm/init_64.c |1 +

[PATCH] ia64: Make sure interrupts enabled when we "safe_halt()"

2013-04-16 Thread Luck, Tony
In commit d166991234347215dc23fc9dc15a63a83a1a54e1 idle: Implement generic idle function Thomas Gleixner cleaned up many things but perturbed some fragile code that was keeping ia64 alive. So we started seeing: WARNING: at kernel/cpu/idle.c:94 cpu_idle_loop+0x360/0x380() and other unpleasantn

Re: [PATCH] futex: bugfix for futex-key conflict when futex use hugepage

2013-04-16 Thread Dave Hansen
Instead of bothering to store the index, why not just calculate it, like: On 04/15/2013 08:37 PM, zhang.y...@zte.com.cn wrote: > +static inline int get_page_compound_index(struct page *page) > +{ > + if (PageHead(page)) > + return 0; > + return compound_head(page) - page;

Re: [PATCH V2] gpio: palmas: add dt support

2013-04-16 Thread Stephen Warren
On 04/16/2013 08:31 AM, Laxman Dewangan wrote: > Add of_device_id table for Palma GPIO to be enable the > driver from DT file. > > The driver can be registered from DT file as: > palmas: tps65913@58 { > ::: > palmas_gpio: palmas_gpio { >

Re: [PATCH] futex: bugfix for futex-key conflict when futex use hugepage

2013-04-16 Thread Darren Hart
On 04/16/2013 11:37 AM, Dave Hansen wrote: > Instead of bothering to store the index, why not just calculate it, like: > > On 04/15/2013 08:37 PM, zhang.y...@zte.com.cn wrote: >> +static inline int get_page_compound_index(struct page *page) >> +{ >> + if (PageHead(page)) >> +

Re: [PATCH 19/26] sh: Don't use create_proc_read_entry() [RFC]

2013-04-16 Thread Paul Mundt
On Tue, Apr 16, 2013 at 07:28:42AM +0100, Al Viro wrote: > On Tue, Apr 16, 2013 at 03:11:13PM +0900, Simon Horman wrote: > > On Thu, Apr 11, 2013 at 02:30:09PM +0100, David Howells wrote: > > > Don't use create_proc_read_entry() as that is deprecated, but rather use > > > proc_create_data() and seq

Re: Build regressions/improvements in v3.9-rc7

2013-04-16 Thread Geert Uytterhoeven
On Tue, 16 Apr 2013, Geert Uytterhoeven wrote: > JFYI, when comparing v3.9-rc7 to v3.9-rc6[3], the summaries are: > - build errors: +8/-10 + drivers/pcmcia/electra_cf.c: error: implicit declaration of function '__ioremap_at' [-Werror=implicit-function-declaration]: => 225:2 + drivers/pcmci

Re: [PATCH] fuse: use kernel headers when __KERNEL__ is set

2013-04-16 Thread Miklos Szeredi
On Tue, Apr 16, 2013 at 8:29 PM, Colin Cross wrote: > Dropping __linux__ causes a make headers_check warning, which the > kbuild test robot reported this morning: > usr/include/linux/fuse.h:99: found __[us]{8,16,32,64} type without > #include > Using my patch without modification does not cause t

Re: [PATCH net] hyperv: Fix a compiler warning in netvsc_send()

2013-04-16 Thread David Miller
From: Haiyang Zhang Date: Tue, 16 Apr 2013 12:03:51 -0700 > Fixed: warning: cast from pointer to integer of different size > > Reported-by: kbuild test robot > Signed-off-by: Haiyang Zhang > Reviewed-by: K. Y. Srinivasan If you're coding pointers into request IDs you better make that explici

Re: linux-next: manual merge of the mfd tree with the v4l-dvb tree

2013-04-16 Thread Samuel Ortiz
On Tue, Apr 16, 2013 at 09:25:45AM -0700, Andrey Smirnov wrote: > On Tue, Apr 16, 2013 at 6:51 AM, Samuel Ortiz wrote: > > Hi Mauro, > > > > On Wed, Apr 10, 2013 at 06:48:28AM -0300, Mauro Carvalho Chehab wrote: > >> Em Wed, 10 Apr 2013 08:42:53 +0200 > >> Samuel Ortiz escreveu: > >> > >> > Hi St

Re: [PATCH] clk: exynos5250: Fix divider values for sclk_mmc{0,1,2,3}

2013-04-16 Thread Doug Anderson
Hi, On Mon, Apr 8, 2013 at 12:22 AM, Kukjin Kim wrote: > Mike Turquette wrote: >> >> Quoting Tushar Behera (2013-04-02 01:20:40) >> > In legacy setup, sclk_mmc{0,1,2,3} used PRE_RATIO bit-field (8-bit wide) >> > instead of RATIO bit-field (4-bit wide) for dividing clock rate. >> > >> > With curre

RE: [PATCH net] hyperv: Fix a compiler warning in netvsc_send()

2013-04-16 Thread Haiyang Zhang
> -Original Message- > From: David Miller [mailto:da...@davemloft.net] > Sent: Tuesday, April 16, 2013 3:12 PM > To: Haiyang Zhang > Cc: net...@vger.kernel.org; KY Srinivasan; o...@aepfle.de; > jasow...@redhat.com; linux-kernel@vger.kernel.org; > de...@linuxdriverproject.org > Subject: R

Re: [PATCH] [BZ905179] audit: omit check for uid and gid validity in audit rules and data

2013-04-16 Thread Richard Guy Briggs
On Tue, Apr 09, 2013 at 02:39:32AM -0700, Eric W. Biederman wrote: > Andrew Morton writes: > > On Wed, 20 Mar 2013 15:18:17 -0400 Richard Guy Briggs > > wrote: > >> audit rule additions containing "-F auid!=4294967295" were failing with > >> EINVAL. > > The only case where this appears to make

Re: [PATCH v2] make dm and dm-crypt forward cgroup context (was: dm-crypt parallelization patches)

2013-04-16 Thread Mikulas Patocka
On Tue, 16 Apr 2013, Tejun Heo wrote: > Hey, > > On Mon, Apr 15, 2013 at 09:02:06AM -0400, Mikulas Patocka wrote: > > The patch is not bug-prone, because we already must make sure that the > > cloned bio has shorter lifetime than the master bio - so the patch doesn't > > introduce any new pos

Re: [PATCH net] hyperv: Fix a compiler warning in netvsc_send()

2013-04-16 Thread David Miller
From: Haiyang Zhang Date: Tue, 16 Apr 2013 19:34:52 + > The Hyper-V hosts always use 64 bit request id. The guests can have > 32 or 64 bit pointers which equal to the ulong type size. So we cast > it to ulong type. And, assigning 32bit integer to 64 bit variable > works fine. It works fine,

[PATCH] kvm: Allow build-time configuration of KVM device assignment

2013-04-16 Thread Alex Williamson
We hope to at some point deprecate KVM legacy device assignment in favor of VFIO-based assignment. Towards that end, allow legacy device assignment to be deconfigured. Signed-off-by: Alex Williamson --- This depends on Alex Graf's irqfd generalization series to remove IRQ routing code from assi

[PATCH 7/5] mem-soft-dirty: Reshuffle CONFIG_ options to be more Arch-friendly

2013-04-16 Thread Pavel Emelyanov
As Stephen Rothwell pointed out, config options, that depend on architecture support, are better to be wrapped into a select + depends on scheme. Do this for CONFIG_MEM_SOFT_DIRTY, as it currently works only for X86. Signed-off-by: Pavel Emelyanov Cc: Stephen Rothwell --- diff --git a/arch/Kc

Re: [PATCH 5/5] mm: Soft-dirty bits for user memory changes tracking

2013-04-16 Thread Pavel Emelyanov
>>> >From that perspective, the dependency on X86 is awful. What's the >>> problem here and what do other architectures need to do to be able to >>> support the feature? >> >> The problem here is that I don't know what free bits are available on >> page table entries on other architectures. I was

Re: [PATCH] lowmemorykiller: prevent multiple instances of low memory killer

2013-04-16 Thread David Rientjes
On Tue, 16 Apr 2013, Oskar Andero wrote: > > > The comment in shrinker.h is misleading, not the source code. > > > do_shrinker_shrink() will fail for anything negative and 0. > > > > The comment is correct. The only acceptable negative return is -1. > > Look at the second time do_shrinker_shrink

Re: [PATCH] fuse: use kernel headers when __KERNEL__ is set

2013-04-16 Thread Colin Cross
On Tue, Apr 16, 2013 at 12:11 PM, Miklos Szeredi wrote: > On Tue, Apr 16, 2013 at 8:29 PM, Colin Cross wrote: >> Dropping __linux__ causes a make headers_check warning, which the >> kbuild test robot reported this morning: >> usr/include/linux/fuse.h:99: found __[us]{8,16,32,64} type without >> #

[PATCH 2/9] xen/smp/spinlock: Fix leakage of the spinlock interrupt line for every CPU online/offline

2013-04-16 Thread Konrad Rzeszutek Wilk
While we don't use the spinlock interrupt line (see for details commit f10cd522c5fbfec9ae3cc01967868c9c2401ed23 - xen: disable PV spinlocks on HVM) - we should still do the proper init / deinit sequence. We did not do that correctly and for the CPU init for PVHVM guest we would allocate an interrup

[PATCH 7/9] xen/spinlock: Disable IRQ spinlock (PV) allocation on PVHVM

2013-04-16 Thread Konrad Rzeszutek Wilk
See git commit f10cd522c5fbfec9ae3cc01967868c9c2401ed23 (xen: disable PV spinlocks on HVM) for details. But we did not disable it everywhere - which means that when we boot as PVHVM we end up allocating per-CPU irq line for spinlock. This fixes that. Signed-off-by: Konrad Rzeszutek Wilk --- arc

[PATCH 5/9] xen/time: Add default value of -1 for IRQ and check for that.

2013-04-16 Thread Konrad Rzeszutek Wilk
If the timer interrupt has been de-init or is just now being initialized, the default value of -1 should be preset as interrupt line. Check for that and if something is odd WARN us. Signed-off-by: Konrad Rzeszutek Wilk --- arch/x86/xen/time.c | 7 +-- 1 file changed, 5 insertions(+), 2 delet

[PATCH 3/9] xen/time: Fix kasprintf splat when allocating timer%d IRQ line.

2013-04-16 Thread Konrad Rzeszutek Wilk
When we online the CPU, we get this splat: smpboot: Booting Node 0 Processor 1 APIC 0x2 installing Xen timer for CPU 1 BUG: sleeping function called from invalid context at /home/konrad/ssd/konrad/linux/mm/slab.c:3179 in_atomic(): 1, irqs_disabled(): 0, pid: 0, name: swapper/1 Pid: 0, comm: swapp

[PATCH 9/9] xen/smp: Unifiy some of the PVs and PVHVM offline CPU path

2013-04-16 Thread Konrad Rzeszutek Wilk
The "xen_cpu_die" and "xen_hvm_cpu_die" are very similar. Lets coalesce them. Signed-off-by: Konrad Rzeszutek Wilk --- arch/x86/xen/smp.c | 14 -- 1 file changed, 4 insertions(+), 10 deletions(-) diff --git a/arch/x86/xen/smp.c b/arch/x86/xen/smp.c index 415694c..0d466d7 100644 ---

[PATCH 1/9] xen/smp: Fix leakage of timer interrupt line for every CPU online/offline.

2013-04-16 Thread Konrad Rzeszutek Wilk
In the PVHVM path when we do CPU online/offline path we would leak the timer%d IRQ line everytime we do a offline event. The online path (xen_hvm_setup_cpu_clockevents via x86_cpuinit.setup_percpu_clockev) would allocate a new interrupt line for the timer%d. But we would still use the old interrup

[PATCH 8/9] xen/smp/pvhvm: Don't initialize IRQ_WORKER as we are using the native one.

2013-04-16 Thread Konrad Rzeszutek Wilk
There is no need to use the PV version of the IRQ_WORKER mechanism as under PVHVM we are using the native version. The native version is using the SMP API. They just sit around unused: 69: 0 0 xen-percpu-ipi irqwork0 83: 0 0 xen-percpu-ipi irq

[PATCH 6/9] xen/spinlock: Check against default value of -1 for IRQ line.

2013-04-16 Thread Konrad Rzeszutek Wilk
The default (uninitialized) value of the IRQ line is -1. Check if we already have allocated an spinlock interrupt line and if somebody is trying to do it again. Also set it to -1 when we offline the CPU. Signed-off-by: Konrad Rzeszutek Wilk --- arch/x86/xen/spinlock.c | 4 1 file changed, 4

[PATCH 4/9] xen/events: Check that IRQ value passed in is valid.

2013-04-16 Thread Konrad Rzeszutek Wilk
We naively assume that the IRQ value passed in is correct. If it is not, then any dereference operation for the 'info' structure will result in crash - so might as well guard ourselves and sprinkle copious amounts of WARN_ON. Signed-off-by: Konrad Rzeszutek Wilk --- drivers/xen/events.c | 20 +++

[PATCH] fixes for v3.10 in the CPU hotplug patch (v1).

2013-04-16 Thread Konrad Rzeszutek Wilk
The first three patches fix outstanding issues with v3.9 (and earlier) kernels where the simple sequence of: echo 0 > /sys/devices/system/cpu/cpu1/online echo 1 > /sys/devices/system/cpu/cpu1/online would embarrassingly not work. As such they also have the sta...@vger.kernel.org on them. [PATCH

Re: [PATCH] clk: Always notify whole subtree when reparenting

2013-04-16 Thread Sören Brinkmann
Hi Mike, On Tue, Apr 16, 2013 at 12:58:26PM -0700, Mike Turquette wrote: > Quoting Soren Brinkmann (2013-04-16 10:06:50) > > A clock's notifier count only reflects notifiers which are registered > > directly for that clock. A reparent operation though affects the whole > > subtree because of a pot

Re: OOM-killer and strange RSS value in 3.9-rc7

2013-04-16 Thread David Rientjes
On Tue, 16 Apr 2013, Han Pingtian wrote: > Hi list, > > On a power7 system, we have installed 3.9-rc7 and crash 6.1.6. If I run > something like "make -j 64" to compile linux kernel from source, sooner > or latter, oom-killer will be triggered. Before that, when I trying to > analyse the live sys

[RFC] Pinctrl: driver design supporting gpiolib

2013-04-16 Thread hanumant
Hi I am trying to implement a pinctrl driver. I have given a brief description of the relevant pinctrl hw features as well as a the use cases and my proposed solution for them. Please advise/comment. HW: The pin controller has the following properties 1) Each pin has its own 32 bit register a

RE: [PATCH net] hyperv: Fix a compiler warning in netvsc_send()

2013-04-16 Thread Haiyang Zhang
> -Original Message- > From: David Miller [mailto:da...@davemloft.net] > Sent: Tuesday, April 16, 2013 3:49 PM > To: Haiyang Zhang > Cc: net...@vger.kernel.org; KY Srinivasan; o...@aepfle.de; > jasow...@redhat.com; linux-kernel@vger.kernel.org; > de...@linuxdriverproject.org > Subject: R

[PATCH] devcg: remove parent_cgroup.

2013-04-16 Thread Rami Rosen
In devcgroup_css_alloc(), there is no longer need for parent_cgroup. bd2953ebbb("devcg: propagate local changes down the hierarchy") made the variable parent_cgroup redundant. This patch removes parent_cgroup from devcgroup_css_alloc(). Signed-off-by: Rami Rosen --- security/device_cgroup.c | 2

Re: [PATCH v9 09/16] PCI, x86: implement pcibios_{add|remove}_bus() hooks

2013-04-16 Thread David Rientjes
On Fri, 12 Apr 2013, Jiang Liu wrote: > diff --git a/arch/x86/pci/common.c b/arch/x86/pci/common.c > index 901177d..305c68b 100644 > --- a/arch/x86/pci/common.c > +++ b/arch/x86/pci/common.c > @@ -6,6 +6,7 @@ > > #include > #include > +#include > #include > #include > #include > @@ -1

Re: [PATCH net] hyperv: Fix a compiler warning in netvsc_send()

2013-04-16 Thread Joe Perches
On Tue, 2013-04-16 at 20:18 +, Haiyang Zhang wrote: > > From: David Miller [mailto:da...@davemloft.net] > > It works fine, but it's unclean. > > Keep the req_id type as "u64", because that's what it is, a 64-bit request > > ID. > I will make this update and submit a new patch, with the added

Re: [PROBLEM] perf requires python-devel to compile

2013-04-16 Thread David Ahern
On 4/16/13 10:08 AM, Michael Witten wrote: You should probably disable python support more directly: make NO_LIBPYTHON=1 sure, but I should not have to do anything. The intent of the existing auto-probing code is to figure out what is installed and build a binary with those capabilities.

Re: [PATCH] devcg: remove parent_cgroup.

2013-04-16 Thread Aristeu Rozanski
On Tue, Apr 16, 2013 at 11:24:55PM +0300, Rami Rosen wrote: > In devcgroup_css_alloc(), there is no longer need for parent_cgroup. > bd2953ebbb("devcg: propagate local changes down the hierarchy") made > the variable parent_cgroup redundant. This patch removes parent_cgroup > from devcgroup_css_all

[PATCH v10] irq: add quirk for broken interrupt remapping on 55XX chipsets

2013-04-16 Thread Neil Horman
A few years back intel published a spec update: http://www.intel.com/content/dam/doc/specification-update/5520-and-5500-chipset-ioh-specification-update.pdf For the 5520 and 5500 chipsets which contained an errata (specificially errata 53), which noted that these chipsets can't properly do interru

Re: Bonding driver has bad load balancing for forwarded traffic, 3.7+

2013-04-16 Thread David Miller
From: Eric Dumazet Date: Tue, 16 Apr 2013 07:25:17 -0700 > On Tue, 2013-04-16 at 06:51 -0700, Eric Dumazet wrote: > >> Perfect, thanks a lot for all this ! >> >> Tested-by: Vitaly V. Bursov >> >> > > By the way, we probably should use skb_flow_dissect() to get proper > hashing for tunnels u

Re: [PATCH] neighbour: Convert NEIGH_PRINTK to neigh_dbg

2013-04-16 Thread David Miller
From: Joe Perches Date: Mon, 15 Apr 2013 18:17:19 -0700 > Update debugging messages to a more current style. > > Emit these debugging messages at KERN_DEBUG instead > of KERN_DEFAULT. > > Add and use neigh_dbg(level, fmt, ...) macro > Add dynamic_debug capability via pr_debug > Convert embedded

RE: [PATCH net] hyperv: Fix a compiler warning in netvsc_send()

2013-04-16 Thread Haiyang Zhang
> -Original Message- > From: Joe Perches [mailto:j...@perches.com] > Sent: Tuesday, April 16, 2013 4:31 PM > To: Haiyang Zhang > Cc: David Miller; net...@vger.kernel.org; KY Srinivasan; o...@aepfle.de; > jasow...@redhat.com; linux-kernel@vger.kernel.org; > de...@linuxdriverproject.org > Sub

Re: [PROBLEM] perf requires python-devel to compile

2013-04-16 Thread Michael Witten
On Tue, 16 Apr 2013 13:32:08 -0700, David Ahern wrote: > On 4/16/13 10:08 AM, Michael Witten wrote: >> You should probably disable python support more directly: >> >>make NO_LIBPYTHON=1 > > sure, but I should not have to do anything. The intent of the existing > auto-probing code is to figur

Re: sched/cputime: sig->prev_stime underflow

2013-04-16 Thread Dave Hansen
On 04/16/2013 04:06 AM, Stanislaw Gruszka wrote: > On Thu, Apr 11, 2013 at 11:47:37AM -0700, Dave Hansen wrote: >> On 04/11/2013 12:45 AM, Stanislaw Gruszka wrote: >>> On Mon, Apr 08, 2013 at 08:57:16AM -0700, Dave Hansen wrote: On 04/04/2013 04:41 PM, Frederic Weisbecker wrote: > Does thi

Re: [PATCH] devcg: remove parent_cgroup.

2013-04-16 Thread Serge Hallyn
Quoting Aristeu Rozanski (a...@redhat.com): > On Tue, Apr 16, 2013 at 11:24:55PM +0300, Rami Rosen wrote: > > In devcgroup_css_alloc(), there is no longer need for parent_cgroup. > > bd2953ebbb("devcg: propagate local changes down the hierarchy") made > > the variable parent_cgroup redundant. This

Re: [PATCH 2/3] mem-hotplug: Put kernel_physical_mapping_remove() declaration in CONFIG_MEMORY_HOTREMOVE.

2013-04-16 Thread Andrew Morton
On Mon, 15 Apr 2013 17:46:46 +0800 Tang Chen wrote: > kernel_physical_mapping_remove() is only called by arch_remove_memory() in > init_64.c, which is enclosed in CONFIG_MEMORY_HOTREMOVE. So when we don't > configure CONFIG_MEMORY_HOTREMOVE, the compiler will give a warning: > > warning: _

Re: [PATCH] kvm: Allow build-time configuration of KVM device assignment

2013-04-16 Thread Alexander Graf
On 16.04.2013, at 21:49, Alex Williamson wrote: > We hope to at some point deprecate KVM legacy device assignment in > favor of VFIO-based assignment. Towards that end, allow legacy > device assignment to be deconfigured. > > Signed-off-by: Alex Williamson Definitely a step into the right dir

Re: [PATCH v2] kvm: nVMX: check vmcs12 for valid activity state

2013-04-16 Thread Marcelo Tosatti
On Mon, Apr 15, 2013 at 03:00:27PM +0200, Paolo Bonzini wrote: > KVM does not use the activity state VMCS field, and does not support > it in nested VMX either (the corresponding bits in the misc VMX feature > MSR are zero). Fail entry if the activity state is set to anything but > "active". > >

Re: [PATCH] devcg: remove parent_cgroup.

2013-04-16 Thread Aristeu Rozanski
On Tue, Apr 16, 2013 at 04:07:25PM -0500, Serge Hallyn wrote: > Quoting Aristeu Rozanski (a...@redhat.com): > > On Tue, Apr 16, 2013 at 11:24:55PM +0300, Rami Rosen wrote: > > > In devcgroup_css_alloc(), there is no longer need for parent_cgroup. > > > bd2953ebbb("devcg: propagate local changes dow

Re: [PATCH 16/28] zoran: Don't print proc_dir_entry data in debug [RFC]

2013-04-16 Thread Mauro Carvalho Chehab
Em 16-04-2013 15:26, David Howells escreveu: Don't print proc_dir_entry data in debug as we're soon to have no direct access to the contents of the PDE. Print what was put in there instead. Let me just apply this simple one, as it doesn't depend on the rest of the patches in this series. Si

Re: [PATCH 03/28] proc: Split kcore bits from linux/procfs.h into linux/kcore.h [RFC]

2013-04-16 Thread KOSAKI Motohiro
On Tue, Apr 16, 2013 at 11:26 AM, David Howells wrote: > Split kcore bits from linux/procfs.h into linux/kcore.h. > > Signed-off-by: David Howells > cc: linux-m...@linux-mips.org > cc: sparcli...@vger.kernel.org > cc: x...@kernel.org > cc: linux...@kvack.org I have no seen any issue in this chan

Re: [PATCH 04/28] proc: Supply PDE attribute setting accessor functions [RFC]

2013-04-16 Thread Mauro Carvalho Chehab
Em 16-04-2013 15:26, David Howells escreveu: Supply accessor functions to set attributes in proc_dir_entry structs. The following are supplied: proc_set_size() and proc_set_user(). Signed-off-by: David Howells cc: linuxppc-...@lists.ozlabs.org cc: linux-me...@vger.kernel.org cc: net...@vger.ke

Re: [PATCH 10/28] proc: Add proc_mkdir_data() [RFC]

2013-04-16 Thread Mauro Carvalho Chehab
Em 16-04-2013 15:26, David Howells escreveu: Add proc_mkdir_data() to allow procfs directories to be created that are annotated at the time of creation with private data rather than doing this post-creation. This means no access is then required to the proc_dir_entry struct to set this. Signed-

[PATCH v2 0/6] Chainsaw efivars.c

2013-04-16 Thread Matt Fleming
From: Matt Fleming drivers/firmware/efivars.c has grown pretty large and is ~2K lines. Inside efivars.c there's currently, o code for handling EFI variables at the firmware-level o sysfs code for exposing EFI variables o a new EFI variable filesystem o a persistent storage backend all

[PATCH v2 2/6] efivars: Keep a private global pointer to efivars

2013-04-16 Thread Matt Fleming
From: Matt Fleming Some machines have an EFI variable interface that does not conform to the UEFI specification, e.g. CONFIG_GOOGLE_SMI. Add the necessary code so that it's only possible to use one implementation of EFI variable operations at runtime. This allows us to keep a single (file-scope)

[PATCH v2 1/6] efi: move utf16 string functions to efi.h

2013-04-16 Thread Matt Fleming
From: Matt Fleming There are currently two implementations of the utf16 string functions. Somewhat confusingly, they've got different names. Centralise the functions in efi.h. Cc: Tom Gundersen Reviewed-by: Mike Waychison Signed-off-by: Matt Fleming --- drivers/firmware/efivars.c | 17

[PATCH 4/6] efivars: Move pstore code into the new EFI directory

2013-04-16 Thread Matt Fleming
From: Matt Fleming efivars.c has grown far too large and needs to be divided up. Create a new directory and move the persistence storage code to efi-pstore.c now that it uses the new efivar API. This helps us to greatly reduce the size of efivars.c and paves the way for moving other code out of e

[PATCH 5/6] efivarfs: Move to fs/efivarfs

2013-04-16 Thread Matt Fleming
From: Matt Fleming Now that efivarfs uses the efivar API, move it out of efivars.c and into fs/efivarfs where it belongs. This move will eventually allow us to enable the efivarfs code without having to also enable CONFIG_EFI_VARS built, and vice versa. Furthermore, things like, mount -t ef

[PATCH V2 1/2] regulator: core: return err value for regulator_get if there is no DT binding

2013-04-16 Thread Nishanth Menon
commit 6d191a5fc7a969d972f1681e1c23781aecb06a61 (regulator: core: Don't defer probe if there's no DT binding for a supply) Attempted to differentiate between regulator_get() with an actual DT binding for the supply and when there is none to avoid unnecessary deferal. However, ret value supplied by

[PATCH] um: Use generic idle loop

2013-04-16 Thread Richard Weinberger
Signed-off-by: Richard Weinberger --- arch/um/Kconfig.common | 1 + arch/um/kernel/process.c | 27 --- 2 files changed, 5 insertions(+), 23 deletions(-) diff --git a/arch/um/Kconfig.common b/arch/um/Kconfig.common index bceee66..5783406 100644 --- a/arch/um/Kconfig.com

[PATCH net] hyperv: Fix a compiler warning in netvsc_send()

2013-04-16 Thread Haiyang Zhang
Fixed: warning: cast from pointer to integer of different size The Hyper-V hosts always use 64 bit request id. The guests can have 32 or 64 bit pointers which equal to the ulong type size. So we cast it to ulong type. And, assigning 32bit integer to 64 bit variable works fine. The VMBus returns t

Re: [RFC 0/7] Update REPORTING-BUGS

2013-04-16 Thread Sarah Sharp
Thanks Ted! On Mon, Apr 15, 2013 at 09:47:42PM -0400, Theodore Ts'o wrote: > Sarah, > > Thanks so much for improving the REPORTING-BUGS file. With your > changes it looks way better! > > > - Ted -- To unsubscribe from this list: send the line "unsubscribe linux-ke

Re: [PATCH 2/2] ptrace/x86: dont delay perf_event_disable() till second pass in ptrace_write_dr7()

2013-04-16 Thread Frederic Weisbecker
2013/4/16 Oleg Nesterov : > On 04/16, Frederic Weisbecker wrote: >> > rc = ptrace_modify_breakpoint(bp, len, type, tsk, disabled); >> > if (rc) >> > break; >> >> It would be nice to warn here: >> >>WARN_ON_ONCE(rc && second_pass); > > Well, I disagree

Re: [Patch 1/1] cciss: bug fix, prevent cciss from loading in kdump kernel

2013-04-16 Thread Andrew Morton
On Mon, 15 Apr 2013 12:59:06 -0500 Mike Miller wrote: > Patch 1/1 > > If hpsa is selected as the Smart Array driver cciss may try to load in the > kdump kernel. When this happens kdump fails and a core file cannot be created. > This patch prevents cciss from trying to load in this scenario. This

Re: [PATCH 03/28] proc: Split kcore bits from linux/procfs.h into linux/kcore.h [RFC]

2013-04-16 Thread David Howells
KOSAKI Motohiro wrote: > I have no seen any issue in this change. but why? Is there any > motivation rather than cleanup? Stopping stuff mucking about with the internals of procfs incorrectly (sometimes because the internals of procfs have changed, but the drivers haven't). David -- To unsubsc

Re: [PATCH v10] irq: add quirk for broken interrupt remapping on 55XX chipsets

2013-04-16 Thread Don Dutile
On 04/16/2013 04:38 PM, Neil Horman wrote: A few years back intel published a spec update: http://www.intel.com/content/dam/doc/specification-update/5520-and-5500-chipset-ioh-specification-update.pdf For the 5520 and 5500 chipsets which contained an errata (specificially errata 53), which noted

Re: [PATCH v2 0/4] remove GENERIC_GPIO

2013-04-16 Thread Alexandre Courbot
Hi Grant, Stephen, On Tue, Apr 16, 2013 at 2:45 AM, Grant Likely wrote: >>> Grant, if this is ok with you, how shall we have this integrated into your >>> branch? Half of this has been tested in my -next branch, and the present >>> patches >>> make the next half, should I resend you the whole se

Re: [PATCH 03/28] proc: Split kcore bits from linux/procfs.h into linux/kcore.h [RFC]

2013-04-16 Thread KOSAKI Motohiro
On Tue, Apr 16, 2013 at 3:07 PM, David Howells wrote: > > KOSAKI Motohiro wrote: > >> I have no seen any issue in this change. but why? Is there any >> motivation rather than cleanup? > > Stopping stuff mucking about with the internals of procfs incorrectly > (sometimes because the internals of p

Re: [PATCH 4/6] x86: relocs: build separate 32/64-bit tools

2013-04-16 Thread H. Peter Anvin
On 04/12/2013 01:13 PM, Kees Cook wrote: > Since the ELF structures and access macros change size based on 32 vs > 64 bits, build a separate 32-bit relocs tool (for handling realmode > and 32-bit relocations), and a 64-bit relocs tool (for handling 64-bit > kernel relocations). > > Signed-off-by:

Re: RE: [PATCH v3] clk: add PowerPC corenet clock driver support

2013-04-16 Thread Mike Turquette
Quoting Tang Yuantian-B29983 (2013-04-15 23:59:34) > Hi Mike, > > I really appreciate if you can spend some times to review this patch. > Yauntian, Thanks for submitting this patch. I have frozen the changes I plan to submit for 3.10, with the exception of any last-minute fixes. I'll take a c

  1   2   3   4   5   6   >