Re: [PATCH] mfd: menelaus: use NULL instead of 0

2013-08-01 Thread Jingoo Han
On Friday, August 02, 2013 2:53 PM, Sachin Kamat wrote: > On 2 August 2013 11:20, Jingoo Han wrote: [...] > > > > mutex_lock(_menelaus->lock); > > - if (vtg == 0) > > + if (vtg == NULL) > > Instead you could just make this: > if (!vtg) Thank you for your

Re: [PATCH] mfd: menelaus: use NULL instead of 0

2013-08-01 Thread Sachin Kamat
On 2 August 2013 11:20, Jingoo Han wrote: > 'mmc_callback_data' and 'vtg' are pointers. > Fix the following sparse warning: > > drivers/mfd/menelaus.c:445:43: warning: Using plain integer as NULL pointer > drivers/mfd/menelaus.c:469:20: warning: Using plain integer as NULL pointer > >

Re: [PATCH v2 13/18] x86, numa, mem_hotplug: Skip all the regions the kernel resides in.

2013-08-01 Thread Tang Chen
On 08/01/2013 09:42 PM, Tejun Heo wrote: On Thu, Aug 01, 2013 at 03:06:35PM +0800, Tang Chen wrote: At early time, memblock will reserve some memory for the kernel, such as the kernel code and data segments, initrd file, and so on=EF=BC=8C which means the kernel resides in these memory regions.

Re: [PATCH v2 10/18] x86, acpi: Try to find if SRAT is overrided earlier.

2013-08-01 Thread Tang Chen
On 08/02/2013 09:19 AM, Toshi Kani wrote: .. +phys_addr_t __init early_acpi_override_srat(void) +{ + int i; + u32 length; + long offset; + void *ramdisk_vaddr; + struct acpi_table_header *table; + struct cpio_data file; + unsigned long map_step =

[PATCH] staging/lustre: lloop depends on BLOCK

2013-08-01 Thread Xiong Zhou
From: Xiong Zhou Add a config option for llite/lloop in lustre driver, making it depends on BLOCK to fix this better: drivers/staging/lustre/lustre/fid/../include/linux/lustre_compat25.h:117:2: error: implicit declaration of function ‘unregister_blkdev' Also, remove the wrapper

[PATCH] mfd: menelaus: use NULL instead of 0

2013-08-01 Thread Jingoo Han
'mmc_callback_data' and 'vtg' are pointers. Fix the following sparse warning: drivers/mfd/menelaus.c:445:43: warning: Using plain integer as NULL pointer drivers/mfd/menelaus.c:469:20: warning: Using plain integer as NULL pointer Signed-off-by: Jingoo Han --- drivers/mfd/menelaus.c |4 ++--

[PATCH] MFD: ucb1x00-core: add CONFIG_PM_SLEEP to suspend/resume

2013-08-01 Thread Jingoo Han
Add CONFIG_PM_SLEEP to suspend/resume functions to fix the build warnings when CONFIG_PM_SLEEP is not selected. This is because sleep PM callbacks defined by SET_SYSTEM_SLEEP_PM_OPS are only used when the CONFIG_PM_SLEEP is enabled. drivers/mfd/ucb1x00-core.c:672:12: warning: 'ucb1x00_suspend'

Re: [PATCH v2 1/1] mfd: palmas: Add power off control

2013-08-01 Thread Bill Huang
On Thu, 2013-08-01 at 21:08 +0800, Nishanth Menon wrote: > On 04:08-20130801, Bill Huang wrote: > > On Wed, 2013-07-31 at 19:57 +0800, Nishanth Menon wrote: > > > > > > If you notice the reference code I send, atleast on TWL6035/37 variants > > > of Palmas, U

[PATCH 2/2] mfd: ab8500-gpadc: add CONFIG_PM_SLEEP to suspend/resume

2013-08-01 Thread Jingoo Han
Add CONFIG_PM_SLEEP to suspend/resume functions to fix the build warnings when CONFIG_PM_SLEEP is not selected. This is because sleep PM callbacks defined by SET_SYSTEM_SLEEP_PM_OPS are only used when the CONFIG_PM_SLEEP is enabled. drivers/mfd/ab8500-gpadc.c:891:12: warning:

[PATCH 1/2] mfd: ab8500-gpadc: add CONFIG_PM_RUNTIME to runtime_suspend/runtime_resume

2013-08-01 Thread Jingoo Han
Add CONFIG_PM_RUNTIME to runtime_suspend/runtime_resume functions to fix the build warnings when CONFIG_PM_RUNTIME is not selected. This is because runtime PM callbacks defined by SET_RUNTIME_PM_OPS are only used when the CONFIG_PM_RUNTIME is enabled. drivers/mfd/ab8500-gpadc.c:870:12: warning:

Re: [PATCH v8 10/12] iommu/exynos: add bus notifier for registering System MMU

2013-08-01 Thread Rahul Sharma
Hi Cho, On Fri, Jul 26, 2013 at 5:00 PM, Cho KyongHo wrote: > When a device driver is registered, all constructs to handle System MMU > is prepared by bus notifier call. > > Signed-off-by: Cho KyongHo > --- > drivers/iommu/exynos-iommu.c | 708 - > 1

[PATCH v2] acpi: video: remove unused code

2013-08-01 Thread Felipe Contreras
_BCM_use_index and _BCL_use_index are never used and probably never will. Signed-off-by: Felipe Contreras --- drivers/acpi/video.c | 12 1 file changed, 12 deletions(-) diff --git a/drivers/acpi/video.c b/drivers/acpi/video.c index 0ec434d..1a04dfe 100644 ---

Re: RFC: vfio-pci API for PCI bus/slot (hot) resets

2013-08-01 Thread Benjamin Herrenschmidt
On Thu, 2013-08-01 at 16:18 -0600, Alex Williamson wrote: > vfio-pci needs to support an interface to do hot resets (PCI parent > bridge secondary bus reset). We need this to support reset of > co-assigned devices where one or more of the devices does not support > function level reset. In

Re: [ANNOUNCE] 3.10.4-rt1

2013-08-01 Thread Benjamin Herrenschmidt
On Thu, 2013-08-01 at 20:03 -0400, Paul Gortmaker wrote: > I've added Ben to the CC in case he has a suggestion on > how best to fix this, even though it is not yet mainline. Can you exchange with a TIF_ that isn't used in asm ? For example TIF_PERFMON_* ? Keep all the asm ones below 16 and move

Re: [PATCH] ARM: dts: exynos5420: add ADC device tree node

2013-08-01 Thread Naveen Krishna Ch
On 2 August 2013 10:20, sunil joshi wrote: > Hi Naveen, > exynos5250.dtsi also needs this entry. > Pls consider adding adc node in exynos5250.dtsi as well, and move the > common content of adc node to exynos5.dtsi. Sure, sunil will run a boot test and submit the code > > Regards > Sunil > > > On

Re: Re: [RFC][PATCH 3/4] tracing/kprobes: Fail to unregister if probe event files are open

2013-08-01 Thread Masami Hiramatsu
(2013/08/01 22:34), Oleg Nesterov wrote: > Just one off-topic note, > >> > @@ -632,7 +635,9 @@ static int release_all_trace_probes(void) >> >/* TODO: Use batch unregistration */ >> >while (!list_empty(_list)) { >> >tp = list_entry(probe_list.next, struct trace_probe, list); >>

Re: [PATCH] acpi: video: fix reversed indexed BQC

2013-08-01 Thread Aaron Lu
On 08/02/2013 12:50 PM, Felipe Contreras wrote: > On Thu, Aug 1, 2013 at 11:30 PM, Aaron Lu wrote: >> On 08/02/2013 12:11 PM, Felipe Contreras wrote: >>> On Thu, Aug 1, 2013 at 9:03 PM, Aaron Lu wrote: On 08/02/2013 07:34 AM, Felipe Contreras wrote: > Commit 1a7c618 (ACPI video: support

[PATCH] acpi: video: remove unused code

2013-08-01 Thread Felipe Contreras
_BCM_use_index is never used and probably never will. Signed-off-by: Felipe Contreras --- drivers/acpi/video.c | 7 --- 1 file changed, 7 deletions(-) diff --git a/drivers/acpi/video.c b/drivers/acpi/video.c index 0ec434d..54e2d4d 100644 --- a/drivers/acpi/video.c +++

Re: __ftrace_hash_rec_update FTRACE_WARN_ON.

2013-08-01 Thread Steve Hodgson
On Tue, Jul 30, 2013 at 5:51 PM, Steven Rostedt wrote: > On Sat, 2013-07-27 at 12:16 -0700, Steve Hodgson wrote: > >> This patch fixes ftrace across module removal/reinsertion on our 3.6.11 >> kernel. > > I found a few corner cases that this patch doesn't work with, and the > solution is just

Re: [PATCH] ARM: dts: exynos5420: add ADC device tree node

2013-08-01 Thread sunil joshi
Hi Naveen, exynos5250.dtsi also needs this entry. Pls consider adding adc node in exynos5250.dtsi as well, and move the common content of adc node to exynos5.dtsi. Regards Sunil On Thu, Aug 1, 2013 at 2:58 PM, Naveen Krishna Chatradhi wrote: > From: Jaehoon Kim > > Add device tree node for

Re: [PATCH v2 02/18] earlycpio.c: Fix the confusing comment of find_cpio_data().

2013-08-01 Thread Tang Chen
On 08/02/2013 05:57 AM, Toshi Kani wrote: .. struct cpio_data __cpuinit find_cpio_data(const char *path, void *data, This patch does not apply cleanly. It seems that your branch does not have 0db0628d90125193280eabb501c94feaf48fa9ab. I have rebased the patch-set to linux 3.11-rc3.

Re: [PATCH] acpi: video: fix reversed indexed BQC

2013-08-01 Thread Felipe Contreras
On Thu, Aug 1, 2013 at 11:30 PM, Aaron Lu wrote: > On 08/02/2013 12:11 PM, Felipe Contreras wrote: >> On Thu, Aug 1, 2013 at 9:03 PM, Aaron Lu wrote: >>> On 08/02/2013 07:34 AM, Felipe Contreras wrote: Commit 1a7c618 (ACPI video: support _BQC/_BCL/_BCM methods that use index values)

[PATCH] fs: Allow unprivileged linkat(..., AT_EMPTY_PATH) aka flink

2013-08-01 Thread Andy Lutomirski
Every now and then someone proposes a new flink syscall, and this spawns a long discussion of whether it would be a security problem. I think that this is missing the point: flink is *already* allowed without privilege as long as /proc is mounted -- it's called AT_SYMLINK_FOLLOW. Now that

Re: [PATCH v2 3/3] sched: clean-up struct sd_lb_stat

2013-08-01 Thread Preeti U Murthy
Hi Joonsoo, On 08/02/2013 07:20 AM, Joonsoo Kim wrote: > There is no reason to maintain separate variables for this_group > and busiest_group in sd_lb_stat, except saving some space. > But this structure is always allocated in stack, so this saving > isn't really benificial. > > This patch unify

Re: [Update][PATCH] cpufreq: Do not hold driver module references for additional policy CPUs

2013-08-01 Thread Viresh Kumar
Wow!! Lot of stuff happened while I was asleep.. @Srivatsa: Thanks for answering what I would have answered to Rafael :) And you should really get some sleep, I would suggest :) On 2 August 2013 02:23, Rafael J. Wysocki wrote: > From: Rafael J. Wysocki > Subject: cpufreq: Do not hold driver

Re: [PATCH 04/23] hugetlb_cgroup: pass around @hugetlb_cgroup instead of @cgroup

2013-08-01 Thread Aneesh Kumar K.V
Tejun Heo writes: > cgroup controller API will be converted to primarily use struct > cgroup_subsys_state instead of struct cgroup. In preparation, make > hugetlb_cgroup functions pass around struct hugetlb_cgroup instead of > struct cgroup. > > This patch shouldn't cause any behavior

Re: [PATCH] acpi: video: fix reversed indexed BQC

2013-08-01 Thread Aaron Lu
On 08/02/2013 12:11 PM, Felipe Contreras wrote: > On Thu, Aug 1, 2013 at 9:03 PM, Aaron Lu wrote: >> On 08/02/2013 07:34 AM, Felipe Contreras wrote: >>> Commit 1a7c618 (ACPI video: support _BQC/_BCL/_BCM methods that use >>> index values) assumed that bl->levels were not reverted, but at this >>>

Re: [Ksummit-2013-discuss] DT bindings as ABI [was: Do we have people interested in device tree janitoring / cleanup?]

2013-08-01 Thread David Gibson
On Wed, Jul 31, 2013 at 05:26:47PM -0400, jonsm...@gmail.com wrote: > On Wed, Jul 31, 2013 at 4:48 PM, Russell King - ARM Linux > wrote: > > On Wed, Jul 31, 2013 at 04:37:36PM -0400, jonsm...@gmail.com > wrote: [snip] > Alternatively you may be of the belief that it is impossible to get > rid of

Re: [PATCH v2 2/3] sched: factor out code to should_we_balance()

2013-08-01 Thread Preeti U Murthy
Hi Joonsoo, On 08/02/2013 07:20 AM, Joonsoo Kim wrote: > Now checking whether this cpu is appropriate to balance or not > is embedded into update_sg_lb_stats() and this checking has no direct > relationship to this function. There is not enough reason to place > this checking at

Re: [RESEND PATCH 1/2] ARM: msm: Add support for MSM8974

2013-08-01 Thread David Brown
On Thu, Aug 01, 2013 at 08:36:38PM -0700, Rohit Vaswani wrote: On 8/1/2013 7:29 PM, Stephen Warren wrote: On 08/01/2013 07:23 PM, Rohit Vaswani wrote: This patch adds basic board support for MSM8974 which belongs to the Snapdragon 800 family. For now, just support a basic machine with device

Re: [PATCH 2/3] acpi: video: trivial style cleanups

2013-08-01 Thread Felipe Contreras
On Thu, Aug 1, 2013 at 8:55 PM, Aaron Lu wrote: > On 08/02/2013 07:44 AM, Felipe Contreras wrote: >> Signed-off-by: Felipe Contreras > > Change log please. You mean a commit message? That's what it's called in Git lingo, and it's right there: acpi: video: trivial style cleanups >> --- >>

Re: [PATCH 1/3] acpi: video: trivial costmetic cleanups

2013-08-01 Thread Felipe Contreras
On Thu, Aug 1, 2013 at 8:50 PM, Aaron Lu wrote: > On 08/02/2013 07:43 AM, Felipe Contreras wrote: >> Signed-off-by: Felipe Contreras > > Please add change log explaining what you have changed. > It seems that the patch modify comment style only, some add a space and > some change spaces to tab,

[PATCH] h8300: include: asm: add "vga.h" in Kbuild

2013-08-01 Thread Chen Gang
Need add "vga.h" in Kbuild, or can not pass compiling, the related error is: CC drivers/tty/tty_io.o In file included from include/linux/selection.h:11:0, from drivers/tty/tty_io.c:103: include/linux/vt_buffer.h:18:21: fatal error: asm/vga.h: No such file or

Re: [PATCH 21/23] cgroup: make cftype->[un]register_event() deal with cgroup_subsys_state instead of cgroup

2013-08-01 Thread Li Zefan
> @@ -506,15 +506,17 @@ struct cftype { >* you want to provide this functionality. Use eventfd_signal() >* on eventfd to send notification to userspace. >*/ > - int (*register_event)(struct cgroup *cgrp, struct cftype *cft, > - struct eventfd_ctx

Re: [PATCH] acpi: video: fix reversed indexed BQC

2013-08-01 Thread Felipe Contreras
On Thu, Aug 1, 2013 at 9:03 PM, Aaron Lu wrote: > On 08/02/2013 07:34 AM, Felipe Contreras wrote: >> Commit 1a7c618 (ACPI video: support _BQC/_BCL/_BCM methods that use >> index values) assumed that bl->levels were not reverted, but at this >> point they already are, so there's no need to revert

[PATCH] h8300: Makefile: remove the hard code CROSS_COMPILE

2013-08-01 Thread Chen Gang
Current Linux can set CROSS_CMPILE from command line or config file, so need remove the hard code CROSS_COMPILE from Makefile, or can not work with cross-compiler with additional prefix (e.g. "h8300-gchen-elf-"). Signed-off-by: Chen Gang --- arch/h8300/Makefile |1 - 1 files changed, 0

[PATCH] fs: Fix file mode for O_TMPFILE

2013-08-01 Thread Andy Lutomirski
O_TMPFILE, like O_CREAT, should respect the requested mode and should create regular files. This fixes two bugs: O_TMPFILE required privilege (because the mode ended up as 000) and it produced bogus inodes with no type. Signed-off-by: Andy Lutomirski --- fs/open.c | 2 +- 1 file changed, 1

[PATCH resend] MAINTAINERS: Add separate section for USB NETWORKING DRIVERS

2013-08-01 Thread Joe Perches
There are several drivers in drivers/net/usb/ that do not have specific MAINTAINERS that should have emails forwarded to the linux-usb mailing list. Add a section for those drivers. Signed-off-by: Joe Perches --- > In the future, you do not need to send drivers/net/usb/ patches to me, > netdev

Re: [PATCH 08/23] cgroup: pass around cgroup_subsys_state instead of cgroup in subsystem methods

2013-08-01 Thread Li Zefan
> @@ -4199,12 +4208,13 @@ static void init_cgroup_css(struct > cgroup_subsys_state *css, > /* invoke ->css_online() on a new CSS and mark it online if successful */ > static int online_css(struct cgroup_subsys *ss, struct cgroup *cgrp) > { > + struct cgroup_subsys_state *css =

Re: [ipc,shm] BUG: lock held when returning to user space!

2013-08-01 Thread Davidlohr Bueso
On Sat, 2013-07-27 at 21:34 +0800, Fengguang Wu wrote: > On Sat, Jul 20, 2013 at 09:46:45AM -0700, Davidlohr Bueso wrote: > > On Sun, 2013-07-21 at 00:02 +0800, Xiaotian Feng wrote: > > > On Sat, Jul 20, 2013 at 9:13 PM, Fengguang Wu > > > wrote: > > > > Greetings, > > > > > > > > I got the

Re: [PATCH] scripts/mod/modpost.c: permit '.cranges' secton for sh64 architecture.

2013-08-01 Thread Chen Gang
On 08/02/2013 10:27 AM, Rusty Russell wrote: > Chen Gang writes: >> Need permit '.cranges' section for sh64 architecture, or modpost will >> report warning: >> >> LD init/built-in.o >> WARNING: init/built-in.o (.cranges): unexpected non-allocatable section. >> Did you forget to use

Re: [PATCH] MAINTAINERS: Add separate section for USB NETWORKING DRIVERS

2013-08-01 Thread Greg KH
On Thu, Aug 01, 2013 at 08:50:10PM -0700, Joe Perches wrote: > On Fri, 2013-08-02 at 11:30 +0800, Greg KH wrote: > > On Thu, Aug 01, 2013 at 01:09:55AM -0700, Joe Perches wrote: > > > Lots of "maintainers" don't apply patches. > > I have no idea what you are meaning here. > > Just that

Re: WARNING: CPU: 26 PID: 93793 at fs/ext4/inode.c:230 ext4_evict_inode+0x4c9/0x500 [ext4]() still in 3.11-rc3

2013-08-01 Thread Davidlohr Bueso
On Thu, 2013-08-01 at 22:33 +0200, Jan Kara wrote: > Hi, > > On Thu 01-08-13 13:14:19, Davidlohr Bueso wrote: > > FYI I'm seeing loads of the following messages with Linus' latest > > 3.11-rc3 (which includes 822dbba33458cd6ad) > Thanks for notice. I see you are running reaim to trigger this.

Re: [PATCH] scripts/mod/modpost.c: permit '.cranges' secton for sh64 architecture.

2013-08-01 Thread Rusty Russell
Chen Gang writes: > Need permit '.cranges' section for sh64 architecture, or modpost will > report warning: > > LD init/built-in.o > WARNING: init/built-in.o (.cranges): unexpected non-allocatable section. > Did you forget to use "ax"/"aw" in a .S file? > Note that for example

Re: [PATCH 08/23] cgroup: pass around cgroup_subsys_state instead of cgroup in subsystem methods

2013-08-01 Thread Li Zefan
> @@ -4298,7 +4308,7 @@ static long cgroup_create(struct cgroup *parent, struct > dentry *dentry, > for_each_root_subsys(root, ss) { > struct cgroup_subsys_state *css; > > - css = ss->css_alloc(cgrp); > + css =

[PATCH] perf: sample after exit loses thread correlation - v3

2013-08-01 Thread David Ahern
Occassionally events (e.g., context-switch, sched tracepoints) are losing the conversion of sample data associated with a thread. For example: $ perf record -e sched:sched_switch -c 1 -a -- sleep 5 $ perf script ls 30482 [000] 1379727.583037: sched:sched_switch: prev_comm=ls prev_pid=30482

Re: [PATCH] MAINTAINERS: Add separate section for USB NETWORKING DRIVERS

2013-08-01 Thread Joe Perches
On Fri, 2013-08-02 at 11:30 +0800, Greg KH wrote: > On Thu, Aug 01, 2013 at 01:09:55AM -0700, Joe Perches wrote: > > Lots of "maintainers" don't apply patches. > I have no idea what you are meaning here. Just that maintainers aren't necessarily appliers. In fact, most aren't. > Add netdev to the

Re: [PATCH 2/3] xfs: Implement FALLOC_FL_COLLAPSE_RANGE

2013-08-01 Thread Dave Chinner
On Thu, Aug 01, 2013 at 02:33:09PM +0900, Namjae Jeon wrote: > 2013/8/1, Dave Chinner : > > On Wed, Jul 31, 2013 at 11:42:14PM +0900, Namjae Jeon wrote: > >> From: Namjae Jeon > >> > >> New fallocate flag FALLOC_FL_COLLAPSE_RANGE implementation for XFS. > > > Hi Dave. > > A good start, but

Re: [PATCH] i2c: Fix Kontron PLD prescaler calculation

2013-08-01 Thread Guenter Roeck
On Fri, Jul 26, 2013 at 02:04:55PM +0200, Michael Brunner wrote: > Add some necessary braces that have been removed during driver cleanup. > This fixes the I2C prescaler calculation. > > Signed-off-by: Michael Brunner Tested-by: Guenter Roeck Reviewed-by: Guenter Roeck Guenter -- To

Re: [PATCH] tty: Only hangup once

2013-08-01 Thread Greg Kroah-Hartman
On Wed, Jul 31, 2013 at 02:05:45PM -0400, Peter Hurley wrote: > Instrumented testing shows a tty can be hungup multiple times [1]. > Although concurrent hangups are properly serialized, multiple > hangups for the same tty should be prevented. > > If tty has already been HUPPED, abort hangup. Note

Re: [RESEND PATCH 1/2] ARM: msm: Add support for MSM8974

2013-08-01 Thread Rohit Vaswani
On 8/1/2013 7:29 PM, Stephen Warren wrote: On 08/01/2013 07:23 PM, Rohit Vaswani wrote: This patch adds basic board support for MSM8974 which belongs to the Snapdragon 800 family. For now, just support a basic machine with device tree. arch/arm/boot/dts/msm8974.dts | 26

Re: [PATCH] MAINTAINERS: Add separate section for USB NETWORKING DRIVERS

2013-08-01 Thread Greg KH
On Thu, Aug 01, 2013 at 01:09:55AM -0700, Joe Perches wrote: > On Thu, 2013-08-01 at 15:56 +0800, Greg KH wrote: > > On Thu, Aug 01, 2013 at 12:48:48AM -0700, Joe Perches wrote: > > > There are several drivers in drivers/net/usb/ that > > > do not have specific MAINTAINERS that should have > > >

Re: [PATCH 16/23] cgroup: relocate cgroup_advance_iter()

2013-08-01 Thread Li Zefan
On 2013/8/2 5:49, Tejun Heo wrote: > For some reason, cgroup_advance_iter() is standing lonely all away > from its iter comrades. Relocate it. > There're some other functions that are in the same situation. Do you think it's better to relocate them, or just leave it as it is? -- To unsubscribe

Re: [PATCHSET cgroup/for-3.12] cgroup: use cgroup_subsys_state as the primary subsystem interface handle

2013-08-01 Thread Li Zefan
On 2013/8/2 5:49, Tejun Heo wrote: > Hello, > > Currently, struct cgroup * is used as the main interface handle > between cgroup core and its subsystems, which works but is a bit > clunky because subsystems usually care much more about css's > (cgroup_subsys_state) a lot more than cgroups, which

Re: [LINUX PATCH RFC v2] trafgen: xilinx: add axi traffic generator driver

2013-08-01 Thread Greg KH
On Sat, Jul 20, 2013 at 08:31:14PM +0530, Srikanth Thokala wrote: > This is the driver for AXI Traffic Generator IP. The AXI > Traffic Generator IP is a core that stresses the AXI4 > interconnect and other AXI4 peripherals in the system. > It generates a wide variety of AXI4 transactions based on

Re: [PATCH RFC V11 15/18] kvm : Paravirtual ticketlocks support for linux guests running on KVM hypervisor

2013-08-01 Thread Raghavendra K T
On 08/01/2013 02:34 PM, Raghavendra K T wrote: On 08/01/2013 01:15 PM, Gleb Natapov wrote: Shall I consider this as an ack for kvm part? For everything except 18/18. For that I still want to see numbers. But 18/18 is pretty independent from the reset of the series so it should not stop the

Re: [PATCH] Tools: hv: in kvp_set_ip_info free mac_addr right after usage

2013-08-01 Thread gre...@linuxfoundation.org
On Thu, Aug 01, 2013 at 02:35:36PM +, KY Srinivasan wrote: > > > > -Original Message- > > From: Olaf Hering [mailto:o...@aepfle.de] > > Sent: Thursday, August 01, 2013 8:58 AM > > To: KY Srinivasan; gre...@linuxfoundation.org > > Cc: linux-kernel@vger.kernel.org; Olaf Hering > >

Re: [PATCH] Tools: hv: check return value of system in hv_kvp_daemon

2013-08-01 Thread gre...@linuxfoundation.org
On Thu, Aug 01, 2013 at 02:36:10PM +, KY Srinivasan wrote: > > > > -Original Message- > > From: Olaf Hering [mailto:o...@aepfle.de] > > Sent: Thursday, August 01, 2013 9:04 AM > > To: KY Srinivasan; gre...@linuxfoundation.org > > Cc: linux-kernel@vger.kernel.org; Olaf Hering > >

Re: [PATCH v2] staging/lustre: lloop depends on BLOCK

2013-08-01 Thread Greg Kroah-Hartman
On Fri, Aug 02, 2013 at 10:20:28AM +0800, Xiong Zhou wrote: > > > On Wed, 31 Jul 2013, Greg Kroah-Hartman wrote: > > > On Wed, Jul 31, 2013 at 10:30:41AM +0800, Xiong Zhou wrote: > > > From: Xiong Zhou > > > > > > First version of this patch makes LUSTRE_FS depends on BLOCK. Second > > >

Re: [PATCH] scsi disk: Use its own buffer for the vpd request

2013-08-01 Thread Martin K. Petersen
> "Bernd" == Bernd Schubert writes: Bernd, Bernd> Once I noticed that scsi_get_vpd_page() works fine from other Bernd> function calls and that it is not 0x89, but already 0x0 that Bernd> fails fixing it became easy. Bernd> Nix, any chance you could verify it also works for you? Do we get

[for-next-3.11][PATCH 4/5] tracing/kprobes: Fail to unregister if probe event files are in use

2013-08-01 Thread Steven Rostedt
From: "Steven Rostedt (Red Hat)" When a probe is being removed, it cleans up the event files that correspond to the probe. But there is a race between writing to one of these files and deleting the probe. This is especially true for the "enable" file. CPU 0 CPU

[for-next-3.11][PATCH 2/5] tracing: trace_remove_event_call() should fail if call/file is in use

2013-08-01 Thread Steven Rostedt
From: Oleg Nesterov Change trace_remove_event_call(call) to return the error if this call is active. This is what the callers assume but can't verify outside of the tracing locks. Both trace_kprobe.c/trace_uprobe.c need the additional changes, unregister_trace_probe() should abort if

[for-next-3.11][PATCH 1/5] debugfs: debugfs_remove_recursive() must not rely on list_empty(d_subdirs)

2013-08-01 Thread Steven Rostedt
From: Oleg Nesterov debugfs_remove_recursive() is wrong, 1. it wrongly assumes that !list_empty(d_subdirs) means that this dir should be removed. This is not that bad by itself, but: 2. if d_subdirs does not becomes empty after __debugfs_remove() it gives up and silently fails, it

[for-next-3.11][PATCH 3/5] tracing: Add comment to describe special break case in probe_remove_event_call()

2013-08-01 Thread Steven Rostedt
From: "Steven Rostedt (Red Hat)" The "break" used in the do_for_each_event_file() is used as an optimization as the loop is really a double loop. The loop searches all event files for each trace_array. There's only one matching event file per trace_array and after we find the event file for the

[for-next-3.11][PATCH 5/5] tracing/uprobes: Fail to unregister if probe event files are in use

2013-08-01 Thread Steven Rostedt
From: "Steven Rostedt (Red Hat)" Uprobes suffer the same problem that kprobes have. There's a race between writing to the "enable" file and removing the probe. The probe checks for it being in use and if it is not, goes about deleting the probe and the event that represents it. But the problem

[for-next-3.11][PATCH 0/5] tracing: Final fixes for the race between open event file and deleting event

2013-08-01 Thread Steven Rostedt
Oleg, Is all the races that we are aware of between accessing event files and deleting events covered by these patches? I think I have them all. Was there any patches that I missed, as there were a lot of threads and lots of patches sent out, but not all were considered final. I think I got the

Re: [PATCH RFC 1/2] qspinlock: Introducing a 4-byte queue spinlock implementation

2013-08-01 Thread Raghavendra K T
On 08/02/2013 02:39 AM, Waiman Long wrote: On 08/01/2013 04:23 PM, Raghavendra K T wrote: On 08/01/2013 08:07 AM, Waiman Long wrote: [..] Though I could see some gains in overcommit, but it hurted undercommit in some workloads :(. The gcc 4.4.7 compiler that I used in my test machine has

Re: [PATCH] ACPI: Do not fail acpi_bind_one() if device is already bound correctly

2013-08-01 Thread Lan Tianyu
2013/8/2 Rafael J. Wysocki : > From: Rafael J. Wysocki > > Modify acpi_bind_one() so that it doesn't fail if the device > represented by its first argument has already been bound to the > given ACPI handle (second argument), because that is not a good > enough reason for returning an error code.

Re: PROBLEM: 3.11.0-rc2+ lost of connectivity

2013-08-01 Thread Cong Wang
On Thu, 01 Aug 2013 at 14:34 GMT, Sebastiano Spicuglia wrote: > > The machine suddenly stops to be connected to the network. > Running "ifdown eth0" "ifup eth0" the network starts to work > again for few minutes and then the failure happens again. > It is not even possible to ping the machine. >

Re: [PATCH RFC 1/2] qspinlock: Introducing a 4-byte queue spinlock implementation

2013-08-01 Thread Raghavendra K T
On 08/02/2013 02:17 AM, Peter Zijlstra wrote: On Fri, Aug 02, 2013 at 01:53:22AM +0530, Raghavendra K T wrote: You need to learn to trim your replies.. I already stopped reading that paravirt thread because of it. Soon I'll introduce you to my /dev/null mail reader. will be more careful next

Re: [PATCH 2/3] xfs: Implement FALLOC_FL_COLLAPSE_RANGE

2013-08-01 Thread Dave Chinner
On Thu, Aug 01, 2013 at 01:24:02AM -0700, Christoph Hellwig wrote: > Instead of adding more mess to change_file_space it might be a good idea > to pull my > > "refactor the preallocation and hole punching code" > > series from December in first. You mean this one:

[PATCH 3/4 v2] rcu: Simplify RCU_STATE_INITIALIZER() macro

2013-08-01 Thread Steven Rostedt
From: "Steven Rostedt (Red Hat)" The RCU_STATE_INITIALIZER() macro is used only in the rcutree.c file as well as the rcutree_plugin.h file. It is passed as a rvalue to a variable of a similar name. A per_cpu variable is also created with a similar name as well. The uses of

Re: [PATCH 8/8] prepare to remove /proc/sys/vm/hugepages_treat_as_movable

2013-08-01 Thread Naoya Horiguchi
On Thu, Aug 01, 2013 at 11:29:39AM +0530, Aneesh Kumar K.V wrote: > Naoya Horiguchi writes: > > > On Wed, Jul 31, 2013 at 12:02:30AM +0530, Aneesh Kumar K.V wrote: > >> Naoya Horiguchi writes: > >> > >> > Now hugepages are definitely movable. So allocating hugepages from > >> > ZONE_MOVABLE is

[PATCH 4/4 v2] rcu: Have the RCU tracepoints use the tracepoint_string infrastructure

2013-08-01 Thread Steven Rostedt
From: "Steven Rostedt (Red Hat)" Currently, RCU tracepoints save only a pointer to strings in the ring buffer. When displayed via the /sys/kernel/debug/tracing/trace file they are referenced like the printf "%s" that looks at the address in the ring buffer and prints out the string it points

[PATCH 2/4 v2] rcu: Add const annotation to char * for RCU tracepoints and functions

2013-08-01 Thread Steven Rostedt
From: "Steven Rostedt (Red Hat)" All the RCU tracepoints and functions that reference char pointers do so with just 'char *' even though they do not modify the contents of the string itself. This will cause warnings if a const char * is used in one of these functions. The RCU tracepoints store

[PATCH 1/4 v2] tracing: Add __tracepoint_string() to export string pointers

2013-08-01 Thread Steven Rostedt
From: "Steven Rostedt (Red Hat)" There are several tracepoints (mostly in RCU), that reference a string pointer and uses the print format of "%s" to display the string that exists in the kernel, instead of copying the actual string to the ring buffer (saves time and ring buffer space). But this

[PATCH 0/4 v2] [GIT PULL][RCU][3.12] tracing/rcu: Export strings to userspace

2013-08-01 Thread Steven Rostedt
Paul, As we talked about earlier, I set up a branch based off of v3.11-rc2 which the first patch has my tracing patch I need for other updates, and the other three are RCU specific patches that you should take. You can either just pull this branch from my tree, or you can pull just the first

Re: [PATCH 1/3] fs: Introduce new flag FALLOC_FL_COLLAPSE_RANGE

2013-08-01 Thread Dave Chinner
On Thu, Aug 01, 2013 at 02:07:39PM +0900, Namjae Jeon wrote: > 2013/8/1, Dave Chinner : > > On Wed, Jul 31, 2013 at 11:42:00PM +0900, Namjae Jeon wrote: > >> From: Namjae Jeon > >> > >> Fallocate now supports new FALLOC_FL_COLLAPSE_RANGE flag. > >> The semantics of this flag are following: > >>

Re: [PATCH 3/8] Add all memory via sysfs probe interface at once

2013-08-01 Thread Michael Ellerman
On Wed, Jul 24, 2013 at 01:37:47PM -0500, Nathan Fontenot wrote: > When doing memory hot add via the 'probe' interface in sysfs we do not > need to loop through and add memory one section at a time. I think this > was originally done for powerpc, but is not needed. This patch removes > the loop

Re: [PATCH] mm/hotplug: fix a drain pcp bug when offline pages

2013-08-01 Thread Xishi Qiu
On 2013/8/2 2:59, Cody P Schafer wrote: > On 08/01/2013 02:18 AM, Xishi Qiu wrote: >> __offline_pages() >> start_isolate_page_range() >>set_migratetype_isolate() >> set_pageblock_migratetype() -> this pageblock will be marked as >> MIGRATE_ISOLATE >>

Re: [RESEND PATCH 1/2] ARM: msm: Add support for MSM8974

2013-08-01 Thread Stephen Warren
On 08/01/2013 07:23 PM, Rohit Vaswani wrote: > This patch adds basic board support for MSM8974 which > belongs to the Snapdragon 800 family. > For now, just support a basic machine with device tree. > arch/arm/boot/dts/msm8974.dts | 26 ++ That's odd. If this is

Re: [PATCH 2/8] Mark powerpc memory resources as busy

2013-08-01 Thread Michael Ellerman
On Wed, Jul 24, 2013 at 01:36:34PM -0500, Nathan Fontenot wrote: > Memory I/O resources need to be marked as busy or else we cannot remove > them when doing memory hot remove. I would have thought it was the opposite? cheers -- To unsubscribe from this list: send the line "unsubscribe

Re: [PATCH 1/8] register bootmem pages for powerpc when sparse vmemmap is not defined

2013-08-01 Thread Michael Ellerman
On Wed, Jul 24, 2013 at 01:35:11PM -0500, Nathan Fontenot wrote: > Previous commit 46723bfa540... introduced a new config option > HAVE_BOOTMEM_INFO_NODE that ended up breaking memory hot-remove for powerpc > when sparse vmemmap is not defined. So that's a bug fix that should go into 3.10 stable?

Re: [PATCH v2] staging/lustre: lloop depends on BLOCK

2013-08-01 Thread Xiong Zhou
On Wed, 31 Jul 2013, Greg Kroah-Hartman wrote: > On Wed, Jul 31, 2013 at 10:30:41AM +0800, Xiong Zhou wrote: > > From: Xiong Zhou > > > > First version of this patch makes LUSTRE_FS depends on BLOCK. Second > > version makes only lloop depends on BLOCK with a config option for this > >

[RESEND PATCH 1/4] ARM: msm: Remove pen_release usage

2013-08-01 Thread Rohit Vaswani
pen_release is no longer required as the synchronization is now managed by generic arm code. This is done as suggested in https://lkml.org/lkml/2013/6/4/184 Signed-off-by: Rohit Vaswani --- arch/arm/mach-msm/Makefile | 2 +- arch/arm/mach-msm/headsmp.S | 39

[PATCH 3/4] ARM: msm: Add SMP support for 8960

2013-08-01 Thread Rohit Vaswani
Add the cpus bindings and the Krait release sequence to make SMP work for MSM8960 Signed-off-by: Rohit Vaswani --- Documentation/devicetree/bindings/arm/cpus.txt | 2 + Documentation/devicetree/bindings/arm/msm/kpss.txt | 16 ++ arch/arm/boot/dts/msm8960-cdp.dts | 22

[RESEND PATCH 2/4] ARM: msm: Re-organize platsmp to make it extensible

2013-08-01 Thread Rohit Vaswani
This makes it easy to add SMP support for new targets by adding cpus property and the release sequence. We add the enable-method property for the cpus property to specify which release sequence to use. While at it, add the 8660 cpus bindings to make SMP work. Signed-off-by: Rohit Vaswani ---

[RESEND PATCH 4/4] ARM: msm: Add support for 8974 SMP

2013-08-01 Thread Rohit Vaswani
Add the cpus bindings and the Kraitv2 release sequence to make SMP work for 2 cores on MSM8974. Signed-off-by: Rohit Vaswani --- Documentation/devicetree/bindings/arm/cpus.txt | 1 + arch/arm/boot/dts/msm8974.dts | 23 arch/arm/mach-msm/board-dt-8974.c |

[RESEND PATCH 0/4]Add SMP support for MSM8660, MSM8960 and MSM8974

2013-08-01 Thread Rohit Vaswani
This series re-organizes the platsmp.c and adds SMP support for MSM8660, MSM8960 and MSM8974. This is based on 3.11-rc3. We convert to using the cpus property in device tree and add a "enable-method" property for arm32. This helps select the appropriate release sequence for the secondary cores

Re: /sys/module/pcie_aspm/parameters/policy not writable?

2013-08-01 Thread Pavel Machek
Hi! > >> > But since the problem also occurs with Windows, it's pretty likely > >> > that there's a BIOS update to fix it. I notice on the X60 support > >> > page that there are several versions newer than what you're running. > >> > >> Do you have any interest in trying a newer BIOS to see if

[PATCH 2/4] mm, migrate: allocation new page lazyily in unmap_and_move()

2013-08-01 Thread Joonsoo Kim
We don't need a new page and then go out immediately if some condition is met. Allocation has overhead in comparison with some condition check, so allocating lazyily is preferable solution. Signed-off-by: Joonsoo Kim diff --git a/mm/migrate.c b/mm/migrate.c index 6f0c244..86db87e 100644 ---

[PATCH 1/4] mm, page_alloc: add likely macro to help compiler optimization

2013-08-01 Thread Joonsoo Kim
We rarely allocate a page with ALLOC_NO_WATERMARKS and it is used in slow path. For making fast path more faster, add likely macro to help compiler optimization. Signed-off-by: Joonsoo Kim diff --git a/mm/page_alloc.c b/mm/page_alloc.c index b100255..86ad44b 100644 --- a/mm/page_alloc.c +++

[PATCH 4/4] swap: clean-up #ifdef in page_mapping()

2013-08-01 Thread Joonsoo Kim
PageSwapCache() is always false when !CONFIG_SWAP, so compiler properly discard related code. Therefore, we don't need #ifdef explicitly. Signed-off-by: Joonsoo Kim diff --git a/include/linux/swap.h b/include/linux/swap.h index d95cde5..c638a71 100644 --- a/include/linux/swap.h +++

[PATCH 3/4] mm: move pgtable related functions to right place

2013-08-01 Thread Joonsoo Kim
pgtable related functions are mostly in pgtable-generic.c. So move remaining functions from memory.c to pgtable-generic.c. Signed-off-by: Joonsoo Kim diff --git a/mm/memory.c b/mm/memory.c index 1ce2e2a..26bce51 100644 --- a/mm/memory.c +++ b/mm/memory.c @@ -374,30 +374,6 @@ void

Re: [QUERY] lguest64

2013-08-01 Thread Rusty Russell
Mike Rapoport writes: > On Wed, Jul 31, 2013 at 3:17 PM, H. Peter Anvin wrote: >> On 07/31/2013 02:39 AM, Mike Rapoport wrote: >>> >>> The use case I had in mind is to use lguest as a nested hypervisor in >>> public clouds. As of today, major public clouds do not support nested >>>

Re: [PATCH] acpi: video: fix reversed indexed BQC

2013-08-01 Thread Aaron Lu
On 08/02/2013 07:34 AM, Felipe Contreras wrote: > Commit 1a7c618 (ACPI video: support _BQC/_BCL/_BCM methods that use > index values) assumed that bl->levels were not reverted, but at this > point they already are, so there's no need to revert them yet again. When acpi_video_bqc_value_to_level is

[PATCH] mm, slab_common: add 'unlikely' to size check of kmalloc_slab()

2013-08-01 Thread Joonsoo Kim
Size is usually below than KMALLOC_MAX_SIZE. If we add a 'unlikely' macro, compiler can make better code. Signed-off-by: Joonsoo Kim diff --git a/mm/slab_common.c b/mm/slab_common.c index 538bade..f0410eb 100644 --- a/mm/slab_common.c +++ b/mm/slab_common.c @@ -373,7 +373,7 @@ struct kmem_cache

Re: [PATCH] regulator: pfuze100: Fix off-by-one for max_register setting

2013-08-01 Thread Robin Gong
On Thu, Aug 01, 2013 at 07:59:56PM +0800, Axel Lin wrote: > max_register should be register count - 1. > > Signed-off-by: Axel Lin > --- > Hi Robin, > I don't have the datasheet, can you check if this patch is correct or not? > If PFUZE_NUMREGS really means register count, I think this fix is

[PATCH] DMA: let filter functions of of_dma_simple_xlate possible check of_node

2013-08-01 Thread Richard Zhao
pass of_phandle_args dma_spec to dma_request_channel in of_dma_simple_xlate, so the filter function could access of_node in of_phandle_args. It also remove restriction of #dma-cells has to be one. Signed-off-by: Richard Zhao --- drivers/dma/edma.c | 7 +-- drivers/dma/of-dma.c | 10

Re: [PATCH 3/3] acpi: video: remove unnecessary casting

2013-08-01 Thread Aaron Lu
On 08/02/2013 07:44 AM, Felipe Contreras wrote: > Signed-off-by: Felipe Contreras Looks good to me, except that no change log feels a little weird. Reviewed-by: Aaron Lu Thanks, Aaron > --- > drivers/acpi/video.c | 6 ++ > 1 file changed, 2 insertions(+), 4 deletions(-) > > diff --git

  1   2   3   4   5   6   7   8   9   10   >