Re: [PATCH v2] mtd: cmdlinepart: fix the wrong partitions number when truncating occurs

2012-08-25 Thread Huang Shijie
On Sun, Aug 26, 2012 at 2:06 AM, Shmulik Ladkani wrote: > Hi, > > On Sat, 25 Aug 2012 05:26:51 -0400 Huang Shijie wrote: >> On Sat, Aug 25, 2012 at 5:02 AM, Shmulik Ladkani >> wrote: >> > Your analysis seems right, but let me offer an alternative approach. >> > >> > I would simply: >> > >> > -

Re: [Patch] block: remove the deprecated ub driver

2012-08-25 Thread Cong Wang
On Sun, Aug 26, 2012 at 1:37 AM, Sebastian Andrzej Siewior wrote: > On Sat, Aug 25, 2012 at 11:51:23PM +0800, Cong Wang wrote: >> From: Cong Wang >> >> It is scheduled to be removed in 3.6. > > This should go via Greg and linux-usb and you should put Pete Zaitcev on Cc. Will do. > I have a patc

Re: [PATCH v2] mtd: cmdlinepart: fix the wrong partitions number when truncating occurs

2012-08-25 Thread Shmulik Ladkani
Hi, On Sat, 25 Aug 2012 05:26:51 -0400 Huang Shijie wrote: > On Sat, Aug 25, 2012 at 5:02 AM, Shmulik Ladkani > wrote: > > Your analysis seems right, but let me offer an alternative approach. > > > > I would simply: > > > > - part->num_parts = i; > your code

[PATCH 2/3] mtd: cmdlinepart: sort the unsorted partitions

2012-08-25 Thread Huang Shijie
Assume we have a 1GB(8Gb) nand chip. It is legit if we set the partitions as the following: gpmi-nand:1g@200m(rootfs),100m@0(boot),100m@100m(kernel) But the current code can not parse out any partition with this cmdline. This patch sorts the unsorted partitions by the @offset. For there are m

[PATCH 3/3] mtd: cmdlinepart: fix the wrong partitions number when truncating occurs

2012-08-25 Thread Huang Shijie
This patch is based on the assumption that all the partitions are in the right offset order. Assume we have a 1GB(8Gb) nand chip, and we set the partitions in the command line like this: #gpmi-nand:100m(boot),100m(kernel),1g(rootfs) In this case, the partition truncating occurs. The current c

[PATCH 1/3] mtd: cmdlinepart: make the partitions rule more strict

2012-08-25 Thread Huang Shijie
There are typically two types to set the mtd partitions: <1> set with the `size`, such as gpmi-nand:100m(boot),100m(kernel),1g(rootfs) <2> set with the `offset`, such as gpmi-nand:100m@0(boot),100m@100m(kernel),1g@200m(rootfs) gpmi-nand:1g@200m(rootfs),100m@0(boot),100m@100m(kernel)

Re: [PATCH v2 21/31] arm64: 32-bit (compat) applications support

2012-08-25 Thread Arnd Bergmann
On Friday 24 August 2012, Catalin Marinas wrote: > > What good is the run-time BUG() here? Nothing should be calling these > > when CONFIG_COMPAT is disabled, so I think you should just remove > > the #ifdef around the declarations, and the entire #else case. > > They are called from handle_signal

Re: [GIT PULL] ARM: mach-pnx4008: Remove architecture

2012-08-25 Thread Arnd Bergmann
On Friday 24 August 2012, Roland Stigge wrote: > On 23/08/12 14:22, Roland Stigge wrote: > > This patch removes the ARM architecture mach-pnx4008. No direct support or > > user > > feedback since 2006. Acknowledgements from NXP/Philips and Linux arm-soc > > maintainers. > > > > Signed-off-by: Rol

Re: [PATCH 0/9] Get rid of FIQ_START/enable/disable_fiq() + some FIQ cleanups

2012-08-25 Thread Anton Vorontsov
Hello Russell, On Sun, Aug 05, 2012 at 04:02:38PM -0700, Anton Vorontsov wrote: > During KDB FIQ patches review you mentioned that I should not introduce > another FIQ_START. It seems that in v3.6-rc the FIQ_START issue was > somewhat band-aided, i.e. machines don't necessary need to define this >

Re: [PATCH] OMAP GPIO - don't wake from suspend unless requested.

2012-08-25 Thread Shilimkar, Santosh
+ Jon, On Sat, Aug 25, 2012 at 5:14 PM, NeilBrown wrote: > > > > Current kernel will wake from suspend on an event on any active > GPIO even if enable_irq_wake() wasn't called. > > There are two reasons that the hardware wake-enable bit should be set: > > 1/ while non-suspended the CPU might go i

Linux 3.5.3

2012-08-25 Thread Greg KH
I'm announcing the release of the 3.5.3 kernel. All users of the 3.5 kernel series must upgrade. The updated 3.5.y git tree can be found at: git://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable.git linux-3.5.y and can be browsed at the normal kernel.org git web browser:

Re: [patch 2/9] procfs: Convert /proc/pid/fdinfo/ handling routines to seq-file v2

2012-08-25 Thread Al Viro
On Thu, Aug 23, 2012 at 02:43:25PM +0400, Cyrill Gorcunov wrote: > This patch converts /proc/pid/fdinfo/ handling routines to seq-file which > is needed to extend seq operations and plug in auxiliary fdinfo provides > from subsystems like eventfd/eventpoll/fsnotify. > > Note the proc_fd_link no lo

Re: [ 40/46] KVM: PIC: call ack notifiers for irqs that are dropped form irr

2012-08-25 Thread Greg Kroah-Hartman
On Thu, Aug 23, 2012 at 01:02:55AM +0100, Ben Hutchings wrote: > On Sun, 2012-08-19 at 20:59 -0700, Greg Kroah-Hartman wrote: > > From: Greg KH > > > > 3.5-stable review patch. If anyone has any objections, please let me know. > > > > -- > > > > From: Gleb Natapov > > > > (ch

[PATCH] rtc_sysfs_show_hctosys 0 if resume failed

2012-08-25 Thread David Fries
The original rtc_sysfs_show_hctosys returnd 1 if this device was the CONFIG_RTC_HCTOSYS_DEVICE device, a later patch checked rtc_hctosys_ret from boot, this verifies boot do_settimeofday succeeded, and sets rtc_hctosys_ret in resume to indicate when adjusting the clock from resume fails. This uses

Re: Drop support for x86-32

2012-08-25 Thread Cruz Julian Bishop
On 26/08/12 12:18, Henrique de Moraes Holschuh wrote: > On Sat, 25 Aug 2012, wbrana wrote: >> On 8/25/12, Pekka Enberg wrote: >>> So despite my humble suggestion, you've filled up my inbox with >>> pointless rambling. Would it be at all possible you just got the f*ck >>> off LKML? I know it's diff

Re: Drop support for x86-32

2012-08-25 Thread Henrique de Moraes Holschuh
On Sat, 25 Aug 2012, wbrana wrote: > On 8/25/12, Pekka Enberg wrote: > > So despite my humble suggestion, you've filled up my inbox with > > pointless rambling. Would it be at all possible you just got the f*ck > > off LKML? I know it's difficult to hear this but nobody gives a shit > > about your

Re: [PATCH] ARM: mxs: remove duplicated include from mach-mxs.c

2012-08-25 Thread Shawn Guo
On 26 August 2012 09:49, Wei Yongjun wrote: > From: Wei Yongjun > > From: Wei Yongjun > > Remove duplicated include. > > Signed-off-by: Wei Yongjun Thanks for the patch. But the duplicated inclusion has been fixed on branch below, which is queued for 3.7. git://git.linaro.org/people/shawngu

Re: [PATCH v2] fork: fix oops after fork failure

2012-08-25 Thread Glauber Costa
On 08/24/2012 06:08 PM, Andrew Morton wrote: > On Thu, 23 Aug 2012 19:36:08 +0400 > Glauber Costa wrote: > >> When we want to duplicate a new process, dup_task_struct() will undergo >> a series of allocations. If alloc_thread_info_node() fails, we call >> free_task_struct() and return. >> >> This

Re: [PATCH] usb/xhci: unify parameter of xhci_msi_irq

2012-08-25 Thread Sarah Sharp
On Mon, Aug 13, 2012 at 01:59:00PM +0800, Alex Shi wrote: > On 06/12/2012 05:14 AM, gregkh wrote: > > > On Mon, Jun 11, 2012 at 01:47:09PM -0700, Sarah Sharp wrote: > >> Hi Alex, > >> > >> This got lost in the 3.5 push, but I think it's a good candidate for > >> 3.6. I think Greg is accepting 3.6

[PATCH] mfd: remove duplicated include from twl6040-core.c

2012-08-25 Thread Wei Yongjun
From: Wei Yongjun From: Wei Yongjun Remove duplicated include. Signed-off-by: Wei Yongjun --- drivers/mfd/twl6040-core.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/mfd/twl6040-core.c b/drivers/mfd/twl6040-core.c index b0fad0f..fbb82ba 100644 --- a/drivers/mfd/twl6040-core.c +

[PATCH] iommu/amd: remove duplicated include from amd_iommu_init.c

2012-08-25 Thread Wei Yongjun
From: Wei Yongjun From: Wei Yongjun Remove duplicated include. Signed-off-by: Wei Yongjun --- drivers/iommu/amd_iommu_init.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/iommu/amd_iommu_init.c b/drivers/iommu/amd_iommu_init.c index 0a2ea31..1292b5a 100644 --- a/drivers/iommu/am

[PATCH] ARM: mxs: remove duplicated include from mach-mxs.c

2012-08-25 Thread Wei Yongjun
From: Wei Yongjun From: Wei Yongjun Remove duplicated include. Signed-off-by: Wei Yongjun --- arch/arm/mach-mxs/mach-mxs.c | 1 - 1 file changed, 1 deletion(-) diff --git a/arch/arm/mach-mxs/mach-mxs.c b/arch/arm/mach-mxs/mach-mxs.c index 8dabfe8..5c91870 100644 --- a/arch/arm/mach-mxs/mach

[PATCH] ARM: LPC32xx: remove duplicated include from phy3250.c

2012-08-25 Thread Wei Yongjun
From: Wei Yongjun From: Wei Yongjun Remove duplicated include. Signed-off-by: Wei Yongjun --- arch/arm/mach-lpc32xx/phy3250.c | 1 - 1 file changed, 1 deletion(-) diff --git a/arch/arm/mach-lpc32xx/phy3250.c b/arch/arm/mach-lpc32xx/phy3250.c index b07dcc9..ed0a50d 100644 --- a/arch/arm/mach

[PATCH] ARM i.MX: remove duplicated include from clk-imx21.c

2012-08-25 Thread Wei Yongjun
From: Wei Yongjun From: Wei Yongjun Remove duplicated include. Signed-off-by: Wei Yongjun --- arch/arm/mach-imx/clk-imx21.c | 1 - 1 file changed, 1 deletion(-) diff --git a/arch/arm/mach-imx/clk-imx21.c b/arch/arm/mach-imx/clk-imx21.c index ea13e61..cf65148 100644 --- a/arch/arm/mach-imx/c

Re: [PATCH] staging: r8712u: use is_broadcast_ether_addr() to simplify the code

2012-08-25 Thread Larry Finger
On 08/25/2012 08:22 PM, Wei Yongjun wrote: From: Wei Yongjun Using is_broadcast_ether_addr() to simplify the code. spatch with a semantic match is used to found this problem. (http://coccinelle.lip6.fr/) Signed-off-by: Wei Yongjun --- drivers/staging/rtl8712/ethernet.h| 8

Re: [PATCH 02/14] aoe: kernel thread handles I/O completions for simple locking

2012-08-25 Thread ecashin
On Fri, 24 Aug 2012 14:22:57 -0700, Andrew Morton wrote: > On Fri, 17 Aug 2012 21:24:08 -0400 > Ed Cashin wrote: ... > > +#ifdef PF_NOFREEZE > > PF_NOFREEZE can never be undefined. > > > + current->flags |= PF_NOFREEZE; > > +#endif > > + set_user_nice(current, -10); > > + sigfillset(&bloc

[PATCH] staging: r8712u: use is_broadcast_ether_addr() to simplify the code

2012-08-25 Thread Wei Yongjun
From: Wei Yongjun Using is_broadcast_ether_addr() to simplify the code. spatch with a semantic match is used to found this problem. (http://coccinelle.lip6.fr/) Signed-off-by: Wei Yongjun --- drivers/staging/rtl8712/ethernet.h| 8 drivers/staging/rtl8712/rtl871x_ioctl_li

[PATCH] staging: rtl8192e: use is_broadcast_ether_addr() instead of memcmp()

2012-08-25 Thread Wei Yongjun
From: Wei Yongjun Using is_broadcast_ether_addr() instead of directly use memcmp() to determine if the ethernet address is broadcast address. spatch with a semantic match is used to found this problem. (http://coccinelle.lip6.fr/) Signed-off-by: Wei Yongjun --- drivers/staging/rtl8192e/rtlli

[PATCH] Staging: vt6656: using is_broadcast_ether_addr() to simplify the code

2012-08-25 Thread Wei Yongjun
From: Wei Yongjun Using is_broadcast_ether_addr() to simplify the code. Signed-off-by: Wei Yongjun --- drivers/staging/vt6656/hostap.c | 8 ++-- 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/drivers/staging/vt6656/hostap.c b/drivers/staging/vt6656/hostap.c index 682002a..4d

[PATCH] Staging: vt6655: using is_broadcast_ether_addr() to simplify the code

2012-08-25 Thread Wei Yongjun
From: Wei Yongjun Using is_broadcast_ether_addr() to simplify the code. Signed-off-by: Wei Yongjun --- drivers/staging/vt6655/hostap.c | 8 ++-- 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/drivers/staging/vt6655/hostap.c b/drivers/staging/vt6655/hostap.c index 6ac6f45..67

[PATCH] Staging: rtl8192u: use is_broadcast_ether_addr() instead of memcmp()

2012-08-25 Thread Wei Yongjun
From: Wei Yongjun Using is_broadcast_ether_addr() instead of directly use memcmp() to determine if the ethernet address is broadcast address. spatch with a semantic match is used to found this problem. (http://coccinelle.lip6.fr/) Signed-off-by: Wei Yongjun --- drivers/staging/rtl8192u/ieee8

[PATCH] staging: rtl8187se: use is_broadcast_ether_addr() instead of memcmp()

2012-08-25 Thread Wei Yongjun
From: Wei Yongjun Using is_broadcast_ether_addr() instead of directly use memcmp() to determine if the ethernet address is broadcast address. spatch with a semantic match is used to found this problem. (http://coccinelle.lip6.fr/) Signed-off-by: Wei Yongjun --- drivers/staging/rtl8187se/ieee

[PATCH] et131x: using is_zero_ether_addr() to simplify the code

2012-08-25 Thread Wei Yongjun
From: Wei Yongjun Using is_zero_ether_addr() to simplify the code Signed-off-by: Wei Yongjun --- drivers/staging/et131x/et131x.c | 7 +-- 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/drivers/staging/et131x/et131x.c b/drivers/staging/et131x/et131x.c index 029725c..c13499d 10

[PATCH] staging: wlags49_h2: use is_broadcast_ether_addr() instead of memcmp()

2012-08-25 Thread Wei Yongjun
From: Wei Yongjun Using is_broadcast_ether_addr() instead of directly use memcmp() to determine if the ethernet address is broadcast address. Signed-off-by: Wei Yongjun --- drivers/staging/wlags49_h2/wl_wext.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/stagin

Re: [PATCH v2 2/2] apple-gmux: Obtain version info from indexed gmux

2012-08-25 Thread Seth Forshee
On Sat, Aug 25, 2012 at 10:30:49AM +0200, Bernhard Froemel wrote: > From: Bernhard Froemel > > This patch extracts and displays version information from the indexed > gmux device as it is also done for the classic gmux device. > > Signed-off-by: Bernhard Froemel Acked-by: Seth Forshee -- To

Re: [PATCH v2 1/2] apple-gmux: Fix index read functions

2012-08-25 Thread Seth Forshee
On Sat, Aug 25, 2012 at 10:30:48AM +0200, Bernhard Froemel wrote: > From: Bernhard Froemel > > Study of Apple's binary driver revealed that the GMUX_READ_PORT should > be written between calls to gmux_index_wait_ready and > gmux_index_wait_complete (i.e., the new index protocol must be > followed

Re: [REVIEW][PATCH 11/15] userns: Teach trace to use from_kuid

2012-08-25 Thread Eric W. Biederman
Steven Rostedt writes: > On Sat, 2012-08-25 at 17:04 -0700, Eric W. Biederman wrote: >> - When tracing capture the kuid. >> - When displaying the data to user space convert the kuid into the >> user namespace of the process that opened the report file. >> > >> index 5c38c81..c9ace83 100644 >>

Re: [REVIEW][PATCH 11/15] userns: Teach trace to use from_kuid

2012-08-25 Thread Steven Rostedt
On Sat, 2012-08-25 at 17:04 -0700, Eric W. Biederman wrote: > - When tracing capture the kuid. > - When displaying the data to user space convert the kuid into the > user namespace of the process that opened the report file. > > index 5c38c81..c9ace83 100644 > --- a/kernel/trace/trace.c > +++ b

Re: [PATCH 00/11] rcu: Add missing RCU idle APIs on idle loop v2

2012-08-25 Thread Paul E. McKenney
On Sun, Aug 26, 2012 at 09:15:27AM +1200, Michael Cree wrote: > On 26/08/12 04:18, Paul E. McKenney wrote: > > On Sat, Aug 25, 2012 at 03:16:49PM +0200, Frederic Weisbecker wrote: > >> On Fri, Aug 24, 2012 at 08:50:47PM -0700, Paul E. McKenney wrote: > >>> On Sat, Aug 25, 2012 at 02:19:14AM +0100,

[REVIEW][PATCH 15/15] userns: Convert configfs to use kuid and kgid where appropriate

2012-08-25 Thread Eric W. Biederman
Cc: Joel Becker Acked-by: Serge Hallyn Signed-off-by: Eric W. Biederman --- fs/configfs/inode.c |4 ++-- init/Kconfig|1 - 2 files changed, 2 insertions(+), 3 deletions(-) diff --git a/fs/configfs/inode.c b/fs/configfs/inode.c index 0074362..a9d35b0 100644 --- a/fs/configfs/in

[REVIEW][PATCH 12/15] userns: Convert drm to use kuid and kgid and struct pid where appropriate

2012-08-25 Thread Eric W. Biederman
Blink Blink this had not been converted to use struct pid ages ago? - On drm open capture the openers kuid and struct pid. - On drm close release the kuid and struct pid - When reporting the uid and pid convert the kuid and struct pid into values in the appropriate namespace. Cc: David Airlie

[REVIEW][PATCH 11/15] userns: Teach trace to use from_kuid

2012-08-25 Thread Eric W. Biederman
- When tracing capture the kuid. - When displaying the data to user space convert the kuid into the user namespace of the process that opened the report file. Cc: Steven Rostedt Cc: Frederic Weisbecker Cc: Ingo Molnar Signed-off-by: Eric W. Biederman --- init/Kconfig |1 - kern

[REVIEW][PATCH 10/15] userns: Convert debugfs to use kuid/kgid where appropriate.

2012-08-25 Thread Eric W. Biederman
Cc: Greg Kroah-Hartman Acked-by: Serge Hallyn Signed-off-by: Eric W. Biederman --- fs/debugfs/inode.c | 26 ++ init/Kconfig |1 - 2 files changed, 18 insertions(+), 9 deletions(-) diff --git a/fs/debugfs/inode.c b/fs/debugfs/inode.c index 4733eab..36e2b66 1

[REVIEW][PATCH 09/15] userns: Convert process event connector to handle kuids and kgids

2012-08-25 Thread Eric W. Biederman
- Only allow asking for events from the initial user and pid namespace, where we generate the events in. - Convert kuids and kgids into the initial user namespace to report them via the process event connector. Cc: Evgeniy Polyakov Cc: David Miller Acked-by: Serge Hallyn Signed-off-by: Er

Re: INFO: suspicious RCU usage in rcu_torture_writer()

2012-08-25 Thread Paul E. McKenney
On Sat, Aug 25, 2012 at 11:36:23AM +0800, Fengguang Wu wrote: > Greetings, > > I got this warning on 3.6.0-rc2. Full dmesg/config attached. > > [3.051375] Initializing RT-Tester: OK > [3.052491] rcu-torture:--- Start of test: nreaders=2 nfakewriters=4 > stat_interval=0 verbose=0 test_no_

[REVIEW][PATCH 07/15] userns: Convert taskstats to handle the user and pid namespaces.

2012-08-25 Thread Eric W. Biederman
- Explicitly limit exit task stat broadcast to the initial user and pid namespaces, as it is already limited to the initial network namespace. - For broadcast task stats explicitly generate all of the idenitiers in terms of the initial user namespace and the initial pid namespace. - For

[REVIEW][PATCH 04/15] userns: net: Call key_alloc with GLOBAL_ROOT_UID, GLOBAL_ROOT_GID instead of 0, 0

2012-08-25 Thread Eric W. Biederman
>From 088e7f2778bd228a4ce05f8dfaa4eb999e2207d2 Mon Sep 17 00:00:00 2001 From: "Eric W. Biederman" Date: Fri, 25 May 2012 16:37:54 -0600 Subject: In net/dns_resolver/dns_key.c and net/rxrpc/ar-key.c make them work with user namespaces enabled where key_alloc takes kuids and kgids. Pass GLOBAL_ROO

[REVIEW][PATCH 05/15] userns: Convert ipc to use kuid and kgid where appropriate

2012-08-25 Thread Eric W. Biederman
- Store the ipc owner and creator with a kuid - Store the ipc group and the crators group with a kgid. - Add error handling to ipc_update_perms, allowing it to fail if the uids and gids can not be converted to kuids or kgids. - Modify the proc files to display the ipc creator and owner in th

[REVIEW][PATCH 03/15] userns: Convert security/keys to the new userns infrastructure

2012-08-25 Thread Eric W. Biederman
- Replace key_user ->user_ns equality checks with kuid_has_mapping checks. - Use from_kuid to generate key descriptions - Use kuid_t and kgid_t and the associated helpers instead of uid_t and gid_t - Avoid potential problems with file descriptor passing by displaying keys in the user namespace o

[REVIEW][PATCH 02/15] userns: Make credential debugging user namespace safe.

2012-08-25 Thread Eric W. Biederman
Cc: David Howells Acked-by: Serge Hallyn Signed-off-by: Eric W. Biederman --- init/Kconfig |1 - kernel/cred.c | 10 -- 2 files changed, 8 insertions(+), 3 deletions(-) diff --git a/init/Kconfig b/init/Kconfig index 448b701..fdabc51 100644 --- a/init/Kconfig +++ b/init/Kconfig

[REVIEW][PATCH 01/15] userns: Enable building of pf_key sockets when user namespace support is enabled.

2012-08-25 Thread Eric W. Biederman
Enable building of pf_key sockets and user namespace support at the same time. This combination builds successfully so there is no reason to forbid it. Signed-off-by: "Eric W. Biederman" --- init/Kconfig |1 - 1 files changed, 0 insertions(+), 1 deletions(-) diff --git a/init/Kconfig b/in

[REVIEW][PATCH 0/15] userns subsystem conversions

2012-08-25 Thread Eric W. Biederman
This patchset updates all of the major linux subsystems that use uids and gids to store them in kuid_t and kgid_t types. This update allows some of the subsystems to work in all user namespaces while other subsystems were updated to only work in the initial user namespace. kuid_t and kgid_t valu

Re: [PATCH RFC 0/3] Add guest cpu_entitlement reporting

2012-08-25 Thread Glauber Costa
On 08/24/2012 11:11 AM, Michael Wolf wrote: > On Fri, 2012-08-24 at 08:53 +0400, Glauber Costa wrote: >> On 08/24/2012 03:14 AM, Michael Wolf wrote: >>> This is an RFC regarding the reporting of stealtime. In the case of >>> where you have a system that is running with partial processors such as >

Re: [patch 1/9] procfs: Move /proc/pid/fd[info] handling code to fd.[ch]

2012-08-25 Thread Al Viro
On Sat, Aug 25, 2012 at 06:16:05PM +0100, Al Viro wrote: > On Thu, Aug 23, 2012 at 02:43:24PM +0400, Cyrill Gorcunov wrote: > > This patch prepares the ground for further extension of > > /proc/pid/fd[info] handling code by moving fdinfo handling > > code into fs/proc/fd.c. > > > > I think such mo

Re: [PATCH 2/5] vmscan: sleep only if backingdev is congested

2012-08-25 Thread Rik van Riel
On 08/22/2012 03:15 AM, Minchan Kim wrote: +++ b/mm/vmscan.c @@ -2705,8 +2705,16 @@ loop_again: if (total_scanned && (sc.priority < DEF_PRIORITY - 2)) { if (has_under_min_watermark_zone) count_vm_event(KSWAPD_SKIP_CONGESTION

Re: [patch 1/9] procfs: Move /proc/pid/fd[info] handling code to fd.[ch]

2012-08-25 Thread Al Viro
On Sat, Aug 25, 2012 at 11:43:25PM +0400, Cyrill Gorcunov wrote: > Would the patch below improve the code? Look, I've not dropped > find_inode_number call since it's a bit unclear for me what > would happen if !child case hit > > child = d_lookup(dir, &qname); > if (!child) { >

Re: [PATCH]:dir.c patch

2012-08-25 Thread Bernd Schubert
On 08/25/2012 10:37 PM, Christopher Sacchi wrote: > Here is a non-style issue dir.c-patch, and as far as I can see from > the lines of code, the compilation errors weren't about what I put in. > This patch fixes a "break" statement inside an "if" statement, as > obviously not correct. Why should t

[PATCH] Input: bcm5974 - only setup button urb for TYPE1 devices

2012-08-25 Thread Henrik Rydberg
The early generations with this trackpad used the separate mouse interface to produce button events. With the introduction of the button pads, this information was moved to the trackpad interface, leaving the mouse interface unused. The driver is still setting up both interfaces, which has not caus

Re: [PATCH 00/11] rcu: Add missing RCU idle APIs on idle loop v2

2012-08-25 Thread Michael Cree
On 26/08/12 04:18, Paul E. McKenney wrote: > On Sat, Aug 25, 2012 at 03:16:49PM +0200, Frederic Weisbecker wrote: >> On Fri, Aug 24, 2012 at 08:50:47PM -0700, Paul E. McKenney wrote: >>> On Sat, Aug 25, 2012 at 02:19:14AM +0100, Ben Hutchings wrote: On Fri, 2012-08-24 at 14:26 -0700, Paul E. M

[PATCH] scripts/coccinelle/tests/odd_ptr_err.cocci: semantic patch for IS_ERR/PTR_ERR inconsistency

2012-08-25 Thread Julia Lawall
From: Julia Lawall Signed-off-by: Julia Lawall --- scripts/coccinelle/tests/odd_ptr_err.cocci | 65 + 1 file changed, 65 insertions(+) diff --git a/scripts/coccinelle/tests/odd_ptr_err.cocci b/scripts/coccinelle/tests/odd_ptr_err.cocci new file mode 100644 index

[PATCH]:dir.c patch

2012-08-25 Thread Christopher Sacchi
Here is a non-style issue dir.c-patch, and as far as I can see from the lines of code, the compilation errors weren't about what I put in. This patch fixes a "break" statement inside an "if" statement, as obviously not correct. Here's the patch for the kernel version v3.6.0rc3: -- Signed-off-by: C

Re: [PATCH can-next v6] can: add tx/rx LED trigger support

2012-08-25 Thread Kurt Van Dijck
On Sat, Aug 25, 2012 at 12:01:42AM +0200, Fabio Baltieri wrote: > Hello Kurt, > > On Fri, Aug 24, 2012 at 02:42:48PM +0200, Kurt Van Dijck wrote: > > On Fri, Aug 24, 2012 at 01:28:16PM +0200, Marc Kleine-Budde wrote: > > > On 08/24/2012 07:10 AM, Kurt Van Dijck wrote: > > > > Hello, > > > > > > >

[PATCH 2/6] drivers/media/platform/mx2_emmaprp.c: adjust inconsistent IS_ERR and PTR_ERR

2012-08-25 Thread Julia Lawall
From: Julia Lawall Change the call to IS_ERR to test the value that was just initialized and is returned using PTR_ERR. The semantic match that finds this problem is as follows: (http://coccinelle.lip6.fr/) // @@ expression e,e1; @@ ( if (IS_ERR(e)) { ... PTR_ERR(e) ... } | if (IS_ERR(e=e1))

[PATCH 1/6] fs/nfsd/nfs4idmap.c: adjust inconsistent IS_ERR and PTR_ERR

2012-08-25 Thread Julia Lawall
From: Julia Lawall Change the call to PTR_ERR to access the value just tested by IS_ERR. The semantic match that finds this problem is as follows: (http://coccinelle.lip6.fr/) // @@ expression e,e1; @@ ( if (IS_ERR(e)) { ... PTR_ERR(e) ... } | if (IS_ERR(e=e1)) { ... PTR_ERR(e) ... } | *if (I

[PATCH 6/6] drivers/iio/adc/at91_adc.c: adjust inconsistent IS_ERR and PTR_ERR

2012-08-25 Thread Julia Lawall
From: Julia Lawall Change the call to PTR_ERR to access the value just tested by IS_ERR. The semantic match that finds this problem is as follows: (http://coccinelle.lip6.fr/) // @@ expression e,e1; @@ ( if (IS_ERR(e)) { ... PTR_ERR(e) ... } | if (IS_ERR(e=e1)) { ... PTR_ERR(e) ... } | *if (I

[PATCH 5/6] drivers/staging/crystalhd/crystalhd_lnx.c: adjust inconsistent IS_ERR and PTR_ERR

2012-08-25 Thread Julia Lawall
From: Julia Lawall Change the call to PTR_ERR to access the value just tested by IS_ERR. The semantic match that finds this problem is as follows: (http://coccinelle.lip6.fr/) // @@ expression e,e1; @@ ( if (IS_ERR(e)) { ... PTR_ERR(e) ... } | if (IS_ERR(e=e1)) { ... PTR_ERR(e) ... } | *if (I

[PATCH 3/6] drivers/usb/gadget/lpc32xx_udc.c: adjust inconsistent IS_ERR and PTR_ERR

2012-08-25 Thread Julia Lawall
From: Julia Lawall Change the call to PTR_ERR to access the value just tested by IS_ERR. The semantic match that finds this problem is as follows: (http://coccinelle.lip6.fr/) // @@ expression e,e1; @@ ( if (IS_ERR(e)) { ... PTR_ERR(e) ... } | if (IS_ERR(e=e1)) { ... PTR_ERR(e) ... } | *if (I

[PATCH 4/6] drivers/usb/host/ohci-nxp.c: adjust inconsistent IS_ERR and PTR_ERR

2012-08-25 Thread Julia Lawall
From: Julia Lawall Change the call to PTR_ERR to access the value just tested by IS_ERR. The semantic match that finds this problem is as follows: (http://coccinelle.lip6.fr/) // @@ expression e,e1; @@ ( if (IS_ERR(e)) { ... PTR_ERR(e) ... } | if (IS_ERR(e=e1)) { ... PTR_ERR(e) ... } | *if (I

[PATCH 0/6] adjust inconsistent IS_ERR and PTR_ERR

2012-08-25 Thread Julia Lawall
These patches fix cases where PTR_ERR does not access the value just tested by IS_ERR. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the

Re: [patch 1/9] procfs: Move /proc/pid/fd[info] handling code to fd.[ch]

2012-08-25 Thread Cyrill Gorcunov
On Sat, Aug 25, 2012 at 08:12:18PM +0100, Al Viro wrote: > On Sat, Aug 25, 2012 at 10:58:29PM +0400, Cyrill Gorcunov wrote: > > On Sat, Aug 25, 2012 at 06:55:04PM +0100, Al Viro wrote: > > > > Well, this could be simplified indeed, if I understand you correctly > > > > you propose just save f_mode

Re: [PATCH 07/19] Input: evdev - Add the events() callback

2012-08-25 Thread Henrik Rydberg
> Reading the time just once and applying it as the timestamp to an > entire frame is very nice. > However, is it ever possible for the SYN_REPORT to get delayed until > the next batch of input_values, therefore breaking the assumption that > the SYN_REPORT timestamp applies to the rest of the inpu

Re: [PATCH 06/19] Input: Send events one packet at a time

2012-08-25 Thread Henrik Rydberg
Hi Daniel, > On Mon, Aug 13, 2012 at 5:42 AM, Henrik Rydberg wrote: > > On heavy event loads, such as a multitouch driver, the irqsoff latency > > can be as high as 250 us. By accumulating a frame worth of data > > before passing it on, the latency can be dramatically reduced. As a > > side eff

Re: linux-next: manual merge of the rr tree with the mips tree

2012-08-25 Thread Geert Uytterhoeven
Hi Stephen, David, On Wed, Aug 22, 2012 at 5:11 AM, Stephen Rothwell wrote: > Today's linux-next merge of the rr tree got a conflict in > arch/mips/kernel/module.c between commit c54de490a2e4 ("MIPS: Module: > Deal with malformed HI16/LO16 relocation sequences") from the mips tree > and commit 9d

Re: [patch 1/9] procfs: Move /proc/pid/fd[info] handling code to fd.[ch]

2012-08-25 Thread Al Viro
On Sat, Aug 25, 2012 at 10:58:29PM +0400, Cyrill Gorcunov wrote: > On Sat, Aug 25, 2012 at 06:55:04PM +0100, Al Viro wrote: > > > Well, this could be simplified indeed, if I understand you correctly > > > you propose just save f_mode in flexible array and use it instead > > > of struct file, right?

Re: [PATCH] Bluetooth: Add support for Apple vendor-specific devices

2012-08-25 Thread Shea Levy
On 08/25/2012 01:28 PM, Henrik Rydberg wrote: As pointed out by Gustavo and Marcel, all Apple-specific Broadcom devices seen so far have the same interface class, subclass and protocol numbers. This patch adds an entry which matches all of them, using the new USB_VENDOR_AND_INTERFACE_INFO() macro

Re: udev 182: response timeout for request_firmware in module_probe path

2012-08-25 Thread Linus Torvalds
On Thu, Aug 23, 2012 at 8:31 AM, Kay Sievers wrote: > > Calling out from inside the kernel and blocking in a firmware loading > userspace transaction during module_init() is kind of weird. It's *very* common. I personally would prefer if drivers did their firmware loading not at probe time, but

Re: [patch 1/9] procfs: Move /proc/pid/fd[info] handling code to fd.[ch]

2012-08-25 Thread Cyrill Gorcunov
On Sat, Aug 25, 2012 at 06:55:04PM +0100, Al Viro wrote: > > Well, this could be simplified indeed, if I understand you correctly > > you propose just save f_mode in flexible array and use it instead > > of struct file, right? (which will require to rewrite code a bit) > > Yes. FWIW, proc_fill_ca

Re: [PATCH 0/6] x86, fpu: cleanups, introduce non-lazy FPU restore for xsave

2012-08-25 Thread Linus Torvalds
On Fri, Aug 24, 2012 at 2:33 PM, H. Peter Anvin wrote: > I have applied this to tip:x86/fpu, but I have also asked Suresh to > prepare a followon patch to decouple eager save from the existence of > the XSAVE instruction. It seems pretty clear that eager save is a net > benefit in the presence of

Re: [PATCH 1/8] mfd: Add Dialog DA906x core driver.

2012-08-25 Thread Mark Brown
On Fri, Aug 24, 2012 at 02:50:00PM +0100, Krystian Garbaciak wrote: > This is MFD module providing access to registers and interrupts of DA906x > series PMIC. It is used by other functional modules, registered as MFD cells. > Driver uses regmap with paging to access extended register list. Registe

Re: [PATCH] Bluetooth: Add support for Apple vendor-specific devices

2012-08-25 Thread Henrik Rydberg
> I'm getting a build error due to lack of > USB_VENDOR_AND_INTERFACE_INFO macro when patching 3.5.2 with that > patch. Which kernel tree should I patch with this patch to test? The macro is in the pipe to 3.5.x, probably only a couple of days away. The patch is for 3.6, as stated, meaning it appl

Re: [PATCH] swiotlb: Add support to disable swiotlb overflow buffer with deprecation warning

2012-08-25 Thread Konrad Rzeszutek Wilk
On Fri, Aug 24, 2012 at 03:14:05PM -0600, Shuah Khan wrote: > Add support for disabling swiotlb overflow buffer using zero size swiotlb > overflow buffer to help test disable overflow scenarios to find drivers that > don't check dma mapping errors. Add kernel error message to track overflow > buffe

Re: [PATCH] Bluetooth: Add support for Apple vendor-specific devices

2012-08-25 Thread Shea Levy
Hi On 08/25/2012 01:43 PM, Henrik Rydberg wrote: On Sat, Aug 25, 2012 at 01:28:52PM -0400, Shea Levy wrote: Hello, On 08/25/2012 01:28 PM, Henrik Rydberg wrote: + /* Apple-specific (Broadcom) devices */ + { USB_VENDOR_AND_INTERFACE_INFO(0x05ac, 0xff, 0x01, 0x01) }, + Would this

[PATCH] clkdev: Add default clkdev.h

2012-08-25 Thread Mark Brown
Ease the deployment of clkdev by providing a default asm/clkdev.h which will be used if the arch does not have an include/asm/clkdev.h. Signed-off-by: Mark Brown Reviewed-by: Stephen Rothwell --- include/asm-generic/Kbuild.asm |4 include/asm-generic/clkdev.h | 28 +

[PATCH v8 04/12] x86, hotplug: Support functions for CPU0 online/offline

2012-08-25 Thread Fenghua Yu
From: Fenghua Yu Add smp_store_boot_cpu_info() to store cpu info for BSP during boot time. Now smp_store_cpu_info() stores cpu info for bringing up BSP or AP after it's offline. Continue to online CPU0 in native_cpu_up(). Continue to offline CPU0 in native_cpu_disable(). Signed-off-by: Fenghu

[PATCH v8 03/12] x86, topology: Don't offline CPU0 if any PIC irq can not be migrated out of it

2012-08-25 Thread Fenghua Yu
From: Fenghua Yu If CONFIG_BOOTPARAM_HOTPLUG_CPU is turned on, CPU0 hotplug feature is enabled by default. If CONFIG_BOOTPARAM_HOTPLUG_CPU is not turned on, CPU0 hotplug feature is not enabled by default. The kernel parameter cpu0_hotplug can enable CPU0 hotplug feature at boot. Signed-off-by:

[PATCH v8 06/12] x86-64, hotplug: Add start_cpu0() entry point to head_64.S

2012-08-25 Thread Fenghua Yu
From: Fenghua Yu start_cpu0() is defined in head_64.S for 64-bit. The function sets up stack and jumps to start_secondary() for CPU0 wake up. Signed-off-by: Fenghua Yu --- arch/x86/kernel/head_64.S | 15 +++ 1 files changed, 15 insertions(+), 0 deletions(-) diff --git a/arch/x86

[PATCH v8 07/12] x86-32, hotplug: Add start_cpu0() entry point to head_32.S

2012-08-25 Thread Fenghua Yu
From: Fenghua Yu start_cpu0() is defined in head_32.S for 32-bit. The function sets up stack and jumps to start_secondary() for CPU0 wake up. Signed-off-by: Fenghua Yu --- arch/x86/kernel/head_32.S | 12 1 files changed, 12 insertions(+), 0 deletions(-) diff --git a/arch/x86/ke

[PATCH v8 09/12] x86, hotplug: During CPU0 online, enable x2apic, set_numa_node.

2012-08-25 Thread Fenghua Yu
From: Fenghua Yu Previously these functions were not run on the BSP (CPU 0, the boot processor) since the boot processor init would only be executed before this functionality was initialized. Signed-off-by: Fenghua Yu --- arch/x86/kernel/cpu/common.c |5 ++--- 1 files changed, 2 insertions

[PATCH v8 08/12] x86, hotplug: Wake up CPU0 via NMI instead of INIT, SIPI, SIPI

2012-08-25 Thread Fenghua Yu
From: Fenghua Yu Instead of waiting for STARTUP after INITs, BSP will execute the BIOS boot-strap code which is not a desired behavior for waking up BSP. To avoid the boot-strap code, wake up CPU0 by NMI instead. This works to wake up soft offlined CPU0 only. If CPU0 is hard offlined (i.e. physi

[PATCH v8 11/12] x86/i387.c: Initialize thread xstate only on CPU0 only once

2012-08-25 Thread Fenghua Yu
From: Fenghua Yu init_thread_xstate() is only called once to avoid overriding xstate_size during boot time or during CPU hotplug. Signed-off-by: Fenghua Yu --- arch/x86/kernel/i387.c |6 +- 1 files changed, 5 insertions(+), 1 deletions(-) diff --git a/arch/x86/kernel/i387.c b/arch/x86

[PATCH v8 10/12] x86, hotplug: The first online processor saves the MTRR state

2012-08-25 Thread Fenghua Yu
From: Fenghua Yu Ask the first online CPU to save mtrr instead of asking BSP. BSP could be offline when mtrr_save_state() is called. Signed-off-by: Fenghua Yu --- arch/x86/kernel/cpu/mtrr/main.c |9 +++-- 1 files changed, 7 insertions(+), 2 deletions(-) diff --git a/arch/x86/kernel/cp

[PATCH v8 12/12] x86, topology: Debug CPU00 hotplug

2012-08-25 Thread Fenghua Yu
From: Fenghua Yu CONFIG_DEBUG_HOTPLUG_CPU0 is for debugging the CPU0 hotplug feature. The switch offlines CPU0 as soon as possible and boots userspace up with CPU0 offlined. User can online CPU0 back after boot time. The default value of the switch is off. To debug CPU0 hotplug, you need to enab

[PATCH v8 02/12] x86, Kconfig: Add config switch for CPU0 hotplug

2012-08-25 Thread Fenghua Yu
From: Fenghua Yu New config switch CONFIG_BOOTPARAM_HOTPLUG_CPU0 sets default state of whether the CPU0 hotplug is on or off. If the switch is off, CPU0 is not hotpluggable by default. But the CPU0 hotplug feature can still be turned on by kernel parameter cpu0_hotplug at boot. If the switch is

[PATCH v8 05/12] x86, hotplug, suspend: Online CPU0 for suspend or hibernate

2012-08-25 Thread Fenghua Yu
From: Fenghua Yu Because x86 BIOS requires CPU0 to resume from sleep, suspend or hibernate can't be executed if CPU0 is detected offline. To make suspend or hibernate and further resume succeed, CPU0 must be online. Signed-off-by: Fenghua Yu --- arch/x86/power/cpu.c | 44

[PATCH v8 01/12] doc: Add x86 CPU0 online/offline feature

2012-08-25 Thread Fenghua Yu
From: Fenghua Yu If CONFIG_BOOTPARAM_HOTPLUG_CPU0 is turned on, CPU0 is hotpluggable. Otherwise, by default CPU0 is not hotpluggable and kernel parameter cpu0_hotplug enables CPU0 online/offline feature. The documentations point out two known CPU0 dependencies. First, resume from hibernate or su

[PATCH v8 0/12] x86: Arbitrary CPU hot(un)plug support

2012-08-25 Thread Fenghua Yu
From: Fenghua Yu CPU0 or BSP (Bootstrap Processor) has been the last processor that can not be hot removed on x86. This patchset implements CPU0 or BSP online and offline and removes this obstacle to CPU hotplug. RAS needs the feature. If socket0 needs to be hotplugged for any reason (any thread

Re: [patch 1/9] procfs: Move /proc/pid/fd[info] handling code to fd.[ch]

2012-08-25 Thread Al Viro
On Sat, Aug 25, 2012 at 09:39:58PM +0400, Cyrill Gorcunov wrote: > On Sat, Aug 25, 2012 at 06:16:05PM +0100, Al Viro wrote: > > On Thu, Aug 23, 2012 at 02:43:24PM +0400, Cyrill Gorcunov wrote: > > > This patch prepares the ground for further extension of > > > /proc/pid/fd[info] handling code by mo

Re: Drop support for x86-32

2012-08-25 Thread wbrana
On 8/25/12, Jochen Striepe wrote: > You wrote unrelated stuff. > Enough of this for me. *plonk* Which unrelated stuff I wrote? -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel

Re: [PATCH v2 1/2] apple-gmux: Fix index read functions

2012-08-25 Thread Henrik Rydberg
On Sat, Aug 25, 2012 at 10:30:48AM +0200, Bernhard Froemel wrote: > From: Bernhard Froemel > > Study of Apple's binary driver revealed that the GMUX_READ_PORT should > be written between calls to gmux_index_wait_ready and > gmux_index_wait_complete (i.e., the new index protocol must be > followed

Re: Drop support for x86-32

2012-08-25 Thread Jochen Striepe
Hi, On Sat, Aug 25, 2012 at 07:22:21PM +0200, wbrana wrote: > On 8/25/12, Jochen Striepe wrote: > > You demand stuff. You offer nothing. You don't listen to the arguments > > people very patiently explain to you. > I replied to (almost) all arguments. You wrote unrelated stuff. Enough of

  1   2   >