Re: [PATCH 2/2] Export initial ramdisk compression config

2013-09-24 Thread P J P
Hello Andrew, Thank you so much for reviewing these patches. +-- On Mon, 23 Sep 2013, Andrew Morton wrote --+ | It's a bit confusing whether all this appiles to initrd, to initramfs | or to both. Can you please clarify all this and be sure that it's all | consistent? IIUC, we no longer use

Re: [PATCH v4] media: st-rc: Add ST remote control driver

2013-09-24 Thread Stephen Warren
On 09/24/2013 02:08 AM, Srinivas KANDAGATLA wrote: > Thanks Stephen, > On 23/09/13 21:40, Stephen Warren wrote: >> On 09/19/2013 02:59 AM, Srinivas KANDAGATLA wrote: >>> This patch adds support to ST RC driver, which is basically a IR/UHF >>> receiver and transmitter. This IP (IRB) is common across

[PATCH v2] clockevents: Sanitize ticks to nsec conversion

2013-09-24 Thread Uwe Kleine-König
From: Thomas Gleixner Marc Kleine-Budde pointed out, that commit 77cc982 "clocksource: use clockevents_config_and_register() where possible" caused a regression for some of the converted subarchs. The reason is, that the clockevents core code converts the minimal hardware tick delta to a nanosec

Re: [PATCH] xen: Do not enable spinlocks before jump_label_init() has executed

2013-09-24 Thread Boris Ostrovsky
On 09/12/2013 10:47 PM, Steven Rostedt wrote: On Thu, 12 Sep 2013 22:29:44 -0400 Boris Ostrovsky wrote: From: Konrad Rzeszutek Wilk xen_init_spinlocks() currently calls static_key_slow_inc() before jump_label_init() is invoked. When CONFIG_JUMP_LABEL is set (which usually is the case) the ef

Re: [RFCv2 3/3] ARM: dts: N900: Add SSI information

2013-09-24 Thread Stephen Warren
On 09/23/2013 05:46 PM, Sebastian Reichel wrote: > Hi, > > On Mon, Sep 23, 2013 at 02:35:35PM -0600, Stephen Warren wrote: >> On 09/15/2013 02:44 PM, Sebastian Reichel wrote: >>> Add SSI device tree data for OMAP34xx and Nokia N900. ... >>> +- ti,hwmods: Name of the hwmod associated

Re: checkpatch guide for newbies

2013-09-24 Thread Dan Carpenter
On Tue, Sep 24, 2013 at 09:29:49PM +0200, Peter Senna Tschudin wrote: > I was about to disagree because I've never seen variables named a, b > or c, but I found that there are at least 2238 variables named a, b or > c in linux-next. This is not good. > In XGIfb_mode_rate_to_ddata() we have:

Re: [RFCv2 3/3] ARM: dts: N900: Add SSI information

2013-09-24 Thread Tony Lindgren
* Stephen Warren [130924 13:03]: > On 09/23/2013 05:46 PM, Sebastian Reichel wrote: > > Hi, > > > > On Mon, Sep 23, 2013 at 02:35:35PM -0600, Stephen Warren wrote: > >> On 09/15/2013 02:44 PM, Sebastian Reichel wrote: > >>> Add SSI device tree data for OMAP34xx and Nokia N900. > > ... > >>> +- t

Re: [PATCH] xen: Do not enable spinlocks before jump_label_init() has executed

2013-09-24 Thread Konrad Rzeszutek Wilk
On Tue, Sep 24, 2013 at 12:50:45PM -0700, Boris Ostrovsky wrote: > On 09/12/2013 10:47 PM, Steven Rostedt wrote: > >On Thu, 12 Sep 2013 22:29:44 -0400 > >Boris Ostrovsky wrote: > > > >>From: Konrad Rzeszutek Wilk > >> > >>xen_init_spinlocks() currently calls static_key_slow_inc() before > >>jump_

Re: checkpatch guide for newbies

2013-09-24 Thread Bjorn Helgaas
On Tue, Sep 24, 2013 at 1:29 PM, Peter Senna Tschudin wrote: > On Tue, Sep 24, 2013 at 7:26 PM, Alexander Holler > wrote: >>> On Mon, Sep 23, 2013 at 3:01 AM, Dan Carpenter >>> wrote: Long Lines Historically screens were 80 characters wide and it was annoyin

[PATCH] ARM: bcm281xx: more descriptive machine string

2013-09-24 Thread Christian Daudt
Add SoC model to DT_MACHINE_START string. Signed-off-by: Christian Daudt Reviewed-by: Markus Mayer Reviewed-by: Mark Hambleton Reviewed-by: James King diff --git a/arch/arm/mach-bcm/board_bcm281xx.c b/arch/arm/mach-bcm/board_bcm281xx.c index 8d9f931..dcfaf4a 100644 --- a/arch/arm/mach-bcm/bo

[PATCH] ARM: bcm281xx: Add ARCH Timers to config

2013-09-24 Thread Christian Daudt
Add HAVE_ARM_ARCH_TIMER to Broadcom Kconfig as it is required for some Mobile SoCs. Signed-off-by: Christian Daudt Reviewed-by: Markus Mayer Reviewed-by: Mark Hambleton Reviewed-by: James King diff --git a/arch/arm/mach-bcm/Kconfig b/arch/arm/mach-bcm/Kconfig index 69d67f7..635537b 100644 ---

Re: [PATCH] hotplug: Optimize {get,put}_online_cpus()

2013-09-24 Thread Peter Zijlstra
On Mon, Sep 23, 2013 at 07:32:03PM +0200, Oleg Nesterov wrote: > > static void cpuhp_wait_refcount(void) > > { > > for (;;) { > > unsigned int rc1, rc2; > > > > rc1 = cpuhp_refcount(); > > set_current_state(TASK_UNINTERRUPTIBLE); /* MB */ > > rc2

Kernel Summit 2013 keysigning

2013-09-24 Thread Konstantin Ryabitsev
Hi, all: We're about a month away from the Kernel Summit in Edinburgh, so I'm organizing another keysigning event. Since this is the second time I'm doing it, hopefully I'll be bitter^W better prepared and have cuter stickers. We'll be using the "hashed list" approach again, to make this quick an

Re: [PATCH] hotplug: Optimize {get,put}_online_cpus()

2013-09-24 Thread Peter Zijlstra
On Tue, Sep 24, 2013 at 08:00:05PM +0200, Oleg Nesterov wrote: > On 09/24, Paul E. McKenney wrote: > > > > On Tue, Sep 24, 2013 at 07:06:31PM +0200, Oleg Nesterov wrote: > > > > > > If gcc can actually do something wrong, then I suspect this barrier() > > > should be unconditional. > > > > If you a

Re: [PATCH 4/6] IB/qib: Use pcie_set_mps() and pcie_get_mps() to simplify code

2013-09-24 Thread Bjorn Helgaas
On Mon, Sep 09, 2013 at 02:55:46PM +, Marciniszyn, Mike wrote: > > Subject: [PATCH 4/6] IB/qib: Use pcie_set_mps() and pcie_get_mps() to > > simplify > > code > > > > Refactor qib_tune_pcie_caps() function, use pcie_set_mps() and > > pcie_get_mps() to simply code. Because pci core caches the

Re: [PATCH 4/6] IB/qib: Use pcie_set_mps() and pcie_get_mps() to simplify code

2013-09-24 Thread Bjorn Helgaas
On Mon, Sep 09, 2013 at 09:13:06PM +0800, Yijing Wang wrote: > Refactor qib_tune_pcie_caps() function, use pcie_set_mps() > and pcie_get_mps() to simply code. Because pci core caches > the "PCI-E Max Payload Size Supported" in pci_dev->pcie_mpss, > so use that instead of pcie_capability_read_word()

Re: [PATCH] Allow userspace code to use flag IFA_F_SECONDARY to specify an ip address to be primary or secondary ip on an interface

2013-09-24 Thread Vincent Li
Ok, I will resend the patch with your suggestions. Vincent On Tue, Sep 24, 2013 at 12:28 PM, David Miller wrote: > From: Vincent Li > Date: Tue, 24 Sep 2013 11:11:21 -0700 > >> the current behavior is when an IP is added to an interface, the primary >> or secondary attributes is depending on th

[RFC 1/1] f2fs: don't GC or take an fs_lock from f2fs_initxattrs()

2013-09-24 Thread Russ Knize
From: Russ Knize f2fs_initxattrs() is called internally from within F2FS and should not call functions that are used by VFS handlers. This avoids certain deadlocks: - vfs_create() - f2fs_create() <-- takes an fs_lock - f2fs_add_link() - __f2fs_add_link() - init_inode_metadata() -

Re: [PATCH] PCI: add quirk for 3ware 9650SE controller

2013-09-24 Thread Bjorn Helgaas
On Fri, Sep 6, 2013 at 4:47 PM, Bjorn Helgaas wrote: > On Fri, Sep 6, 2013 at 3:51 AM, Jiri Kosina wrote: >>> >> > Commit d5dea7d95 ("PCI: msi: Disable msi interrupts when we initialize >>> >> > a >>> >> > pci device") makes MSIs be forcibly disabled at boot time. >>> >> > >>> >> > It turns out

Re: [PATCH 4/4] RX-51: Add platform function and data for bq24150a charger

2013-09-24 Thread Sebastian Reichel
Hi, On Tue, Sep 24, 2013 at 07:05:47PM +0200, Pali Rohár wrote: > No, isp1704 driver is doing fastcharger detection (and then > export charger type via sysfs power supply) based on musb usb > events. > > Real charging (enabling/disabling and setting properties) is done > by bq24150a chip which

Re: [RFC 1/1] f2fs: don't GC or take an fs_lock from f2fs_initxattrs()

2013-09-24 Thread Russ Knize
This is an alternate solution to the deadlock problem I mentioned in my previous patch. On Tue, Sep 24, 2013 at 3:49 PM, Russ Knize wrote: > From: Russ Knize > > f2fs_initxattrs() is called internally from within F2FS and should > not call functions that are used by VFS handlers. This avoids >

Re: [PATCH resend] typo: replace kernelcore with Movable

2013-09-24 Thread Randy Dunlap
On 09/24/13 01:48, Weiping Pan wrote: > Han Pingtian found a typo in Documentation/kernel-parameters.txt > about "kernelcore=", that "kernelcore" should be replaced with "Movable" here. > > I sent this patch a 8 months ago and got ack from Mel Gorman, > http://marc.info/?l=linux-mm&m=1357567206026

Re: [RFC GIT PULL] softirq: Consolidation and stack overrun fix

2013-09-24 Thread Benjamin Herrenschmidt
On Tue, 2013-09-24 at 15:56 +0200, Frederic Weisbecker wrote: > On Tue, Sep 24, 2013 at 02:42:57PM +1000, Benjamin Herrenschmidt wrote: > > On Tue, 2013-09-24 at 04:44 +0200, Frederic Weisbecker wrote: > > > So the safest way to fix this is to unconditionally call do_softirq() > > > from irq_exit()

Re: [RFC GIT PULL] softirq: Consolidation and stack overrun fix

2013-09-24 Thread Benjamin Herrenschmidt
On Tue, 2013-09-24 at 15:27 -0400, Chris Metcalf wrote: > > It's hard to tell with gcc ... the best I've had so far as an option was > > something that would mark my per-cpu register (r13) *itself* as clobbered... > > Well, as I said above, that would be better, but it requires providing an > alte

Re: [PATCH] hotplug: Optimize {get,put}_online_cpus()

2013-09-24 Thread Peter Zijlstra
On Tue, Sep 24, 2013 at 10:24:23PM +0200, Peter Zijlstra wrote: > +void __get_online_cpus(void) > +{ > + if (__cpuhp_writer == 1) { take_ref: > + /* See __srcu_read_lock() */ > + __this_cpu_inc(__cpuhp_refcount); > + smp_mb(); > + __this_cpu_inc(c

[PATCH] ARC: Use clockevents_config_and_register over clockevents_register_device

2013-09-24 Thread Uwe Kleine-König
clockevents_config_and_register is more clever and correct than doing it by hand; so use it. Signed-off-by: Uwe Kleine-König --- arch/arc/kernel/time.c | 7 ++- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/arch/arc/kernel/time.c b/arch/arc/kernel/time.c index 0e51e69..273df8

Re: [PATCH] Allow userspace code to use flag IFA_F_SECONDARY to specify an ip address to be primary or secondary ip on an interface

2013-09-24 Thread Julian Anastasov
Hello, On Tue, 24 Sep 2013, Vincent Li wrote: > the current behavior is when an IP is added to an interface, the primary > or secondary attributes is depending on the order of ip added to the interface > the first IP will be primary and second, third,... or alias IP will be > secondary

Re: [PATCH] ipc/sem.c: fix update sem_otime when calling sem_op in semaphore initialization

2013-09-24 Thread Manfred Spraul
On 09/22/2013 05:14 PM, Jia He wrote: Hi Manfred On Sun, 22 Sep 2013 12:42:05 +0200 from manf...@colorfullife.com wrote: Hi all, On 09/22/2013 10:26 AM, Mike Galbraith wrote: On Sun, 2013-09-22 at 10:17 +0200, Mike Galbraith wrote: On Sun, 2013-09-22 at 10:11 +0800, Jia He wrote: In commi

Re: [PATCH] hotplug: Optimize {get,put}_online_cpus()

2013-09-24 Thread Paul E. McKenney
On Tue, Sep 24, 2013 at 06:09:59PM +0200, Peter Zijlstra wrote: > On Tue, Sep 24, 2013 at 07:42:36AM -0700, Paul E. McKenney wrote: > > > +#define cpuhp_writer_wake() > > > \ > > > + wake_up_process(cpuhp_writer_task) > > > + > > > +#define cpuhp_writer

[PATCH 00/10 - GIT PULL] drivers: net: Next block of Remove extern from function prototypes

2013-09-24 Thread Joe Perches
The following changes since commit b0983d3c9b132c33b6fb2e28d157a1edc18a173c: tcp: fix dynamic right sizing (2013-09-24 11:07:32 -0400) are available in the git repository at: git://repo.or.cz/linux-2.6/trivial-mods.git 20130924_dn_externs_2 for you to fetch changes up to 294da3abaa73a0b69fd

[PATCH 04/10] qlogic: Remove extern from function prototypes

2013-09-24 Thread Joe Perches
There are a mix of function prototypes with and without extern in the kernel sources. Standardize on not using extern for function prototypes. Function prototypes don't need to be written with extern. extern is assumed by the compiler. Its use is as unnecessary as using auto to declare automatic

[PATCH 02/10] intel: Remove extern from function prototypes

2013-09-24 Thread Joe Perches
There are a mix of function prototypes with and without extern in the kernel sources. Standardize on not using extern for function prototypes. Function prototypes don't need to be written with extern. extern is assumed by the compiler. Its use is as unnecessary as using auto to declare automatic

[ipv4:PATCH] Allow userspace to specify primary or secondary ip on interface

2013-09-24 Thread Vincent Li
Allow userspace code to use flag IFA_F_SECONDARY to specify an ip address to be primary or secondary ip on an interface. the current behavior is when an IP is added to an interface, the primary or secondary attributes is depending on the order of ip added to the interface the first IP will be prim

[PATCH 09/10] fddi/skfp: Remove extern from function prototypes

2013-09-24 Thread Joe Perches
There are a mix of function prototypes with and without extern in the kernel sources. Standardize on not using extern for function prototypes. Function prototypes don't need to be written with extern. extern is assumed by the compiler. Its use is as unnecessary as using auto to declare automatic

[PATCH 10/10] irda: Remove extern from function prototypes

2013-09-24 Thread Joe Perches
There are a mix of function prototypes with and without extern in the kernel sources. Standardize on not using extern for function prototypes. Function prototypes don't need to be written with extern. extern is assumed by the compiler. Its use is as unnecessary as using auto to declare automatic

[PATCH 08/10] toshiba: Remove extern from function prototypes

2013-09-24 Thread Joe Perches
There are a mix of function prototypes with and without extern in the kernel sources. Standardize on not using extern for function prototypes. Function prototypes don't need to be written with extern. extern is assumed by the compiler. Its use is as unnecessary as using auto to declare automatic

[PATCH 06/10] stmicro: Remove extern from function prototypes

2013-09-24 Thread Joe Perches
There are a mix of function prototypes with and without extern in the kernel sources. Standardize on not using extern for function prototypes. Function prototypes don't need to be written with extern. extern is assumed by the compiler. Its use is as unnecessary as using auto to declare automatic

Timekeeping on at91rm9200 [Was: [PATCH v2] clockevents: Sanitize ticks to nsec conversion]

2013-09-24 Thread Uwe Kleine-König
Hello Nicolas, hello Jean-Christophe, I expect you should be able to revert b7a8ca5 (ARM: at91: rm9200 fix time support) which reverted 838a2ae (ARM: use clockevents_config_and_register() where possible) for at91rm9200 on top of the patch this mail is a reply to. Can you ple

Re: [PATCH] Remove VLAIS usage from gadget code - alternate patch

2013-09-24 Thread Behan Webster
On 09/24/13 13:56, charl...@gmail.com wrote: From: Mark Charlebois The use of variable length arrays in structs (VLAIS) in the Linux Kernel code precludes the use of compilers which don't implement VLAIS (for instance the Clang compiler). This alternate patch calculates offsets into the kmalloc

[PATCH 03/10] oki-semi: Remove extern from function prototypes

2013-09-24 Thread Joe Perches
There are a mix of function prototypes with and without extern in the kernel sources. Standardize on not using extern for function prototypes. Function prototypes don't need to be written with extern. extern is assumed by the compiler. Its use is as unnecessary as using auto to declare automatic

[PATCH 05/10] sfc: Remove extern from function prototypes

2013-09-24 Thread Joe Perches
There are a mix of function prototypes with and without extern in the kernel sources. Standardize on not using extern for function prototypes. Function prototypes don't need to be written with extern. extern is assumed by the compiler. Its use is as unnecessary as using auto to declare automatic

[PATCH 07/10] ti: Remove extern from function prototypes

2013-09-24 Thread Joe Perches
There are a mix of function prototypes with and without extern in the kernel sources. Standardize on not using extern for function prototypes. Function prototypes don't need to be written with extern. extern is assumed by the compiler. Its use is as unnecessary as using auto to declare automatic

[PATCH 01/10] ibm/emac: Remove extern from function prototypes

2013-09-24 Thread Joe Perches
There are a mix of function prototypes with and without extern in the kernel sources. Standardize on not using extern for function prototypes. Function prototypes don't need to be written with extern. extern is assumed by the compiler. Its use is as unnecessary as using auto to declare automatic

Re: [PATCH V3 1/6] clk: exynos-audss: convert to platform device

2013-09-24 Thread Sylwester Nawrocki
On 09/24/2013 08:06 PM, Andrew Bresticker wrote: +static int exynos_audss_clk_probe(struct platform_device *pdev) { [...] clk_table[EXYNOS_MOUT_AUDSS] = clk_register_mux(NULL, "mout_audss", mout_audss_p, ARRAY_SIZE(mout_audss_p), @@ -123,13 +124,83 @@ s

Re: [PATCH v2] clockevents: Sanitize ticks to nsec conversion

2013-09-24 Thread Uwe Kleine-König
Hello Thomas, just found another little issue ... On Tue, Sep 24, 2013 at 09:50:23PM +0200, Uwe Kleine-König wrote: > +/** > + * clockevents_delta2ns - Convert a latch value (device ticks) to nanoseconds > + * @latch: value to convert > + * @evt: pointer to clock event device descriptor > +

[RFC patch] Update Jens Osterkamp email address

2013-09-24 Thread Joe Perches
The old address isn't working. Signed-off-by: Joe Perches --- Maybe if you're still interested in receiving emails on this old driver... On Tue, 2013-09-24 at 14:10 -0700, postmas...@de.ibm.com wrote: > Your message > Subject: [PATCH 08/10] toshiba: Remove extern from function prototypes > wa

Re: new binutils needed for arm in 3.12-rc1

2013-09-24 Thread Rob Landley
On 09/24/2013 07:11:38 AM, Måns Rullgård wrote: Rob Landley writes: > On 09/23/2013 06:59:17 PM, Pavel Machek wrote: >> During 3.12-rc, Will Deacon introduced code into arch/arm that >> requires binutils 2.22. > > Um, my toolchain is using the last gplv2 snapshot of binutils out of > git, which

Re: [PATCH V3] pci: exynos: split into two parts such as Synopsys part and Exynos part

2013-09-24 Thread Arnd Bergmann
On Monday 23 September 2013, Kishon Vijay Abraham I wrote: > Btw if we hadn't programmed inbound translation table, the address will go > untranslated (according to the data book). I guess that's how it was working > for Jingoo Han. > > ** > 3.10.4 > Inbound iATU Operation > > When there is no ma

Re: [PATCH 08/10] toshiba: Remove extern from function prototypes

2013-09-24 Thread Geoff Levand
On Tue, 2013-09-24 at 14:10 -0700, Joe Perches wrote: > There are a mix of function prototypes with and without extern > in the kernel sources. Standardize on not using extern for > function prototypes. > > Function prototypes don't need to be written with extern. > extern is assumed by the compi

[PATCH v4 0/2] iommu: Add iommu_error class event to iommu trace

2013-09-24 Thread Shuah Khan
This patch set adds iommu_error class event to iommu trace. iommu_error class event can be enabled to trigger when an iommu error occurs. This trace event is intended to be called to report the error information. Trace information includes driver name, device name, iova, and flags. iommu_error:io_

[PATCH v4 2/2] iommu: Change iommu driver to call io_page_fault trace event

2013-09-24 Thread Shuah Khan
Change iommu driver call io_page_fault trace event. This iommu_error class event can be enabled to trigger when an iommu error occurs. Trace information includes driver name, device name, iova, and flags. Testing: Added trace calls to iommu_prepare_identity_map() for testing some of the conditions

[PATCH v4 1/2] iommu: Add iommu_error class event to iommu trace

2013-09-24 Thread Shuah Khan
iommu_error class event can be enabled to trigger when an iommu error occurs. This trace event is intended to be called to report the error information. Trace information includes driver name, device name, iova, and flags. iommu_error:io_page_fault Signed-off-by: Shuah Khan --- drivers/iommu/io

Re: [PATCH] Allow userspace code to use flag IFA_F_SECONDARY to specify an ip address to be primary or secondary ip on an interface

2013-09-24 Thread Vincent Li
Thanks Julian for the comments, I imagined it would not be so simple as it changed old behavior with ip binary and some actions in __inet_del_ifa() that I am not fully aware of. my intention is to preserve the old behavior and extend the flexibility, I am unable to come up with a good patch to achi

Re: [PATCHv6 02/16] drivers: thermal: introduce device tree parser

2013-09-24 Thread Eduardo Valentin
On 23-09-2013 06:40, Mark Rutland wrote: > > It would be nice to have a name for the cells after a phandle which > describe the cooling device's configuration. You've called them > parameters here, but it probably makes sense to call them a > cooling-specifier (following clock-specifier and interr

[PATCH] net: ipheth Add support for iPad 3.

2013-09-24 Thread Marcus Sundberg
>From ad23adfbe455f8ae44476d43e4b3033306fe536f Mon Sep 17 00:00:00 2001 From: Marcus Sundberg Date: Tue, 24 Sep 2013 22:37:07 +0200 Subject: [PATCH 1/2] net: ipheth: Sort device IDs and use consistent whitespace. Signed-off-by: Marcus Sundberg --- drivers/net/usb/ipheth.c | 20 ++

Re: [PATCH v6 0/5] clk: clock deregistration support

2013-09-24 Thread Sylwester Nawrocki
On 08/30/2013 04:53 PM, Sylwester Nawrocki wrote: This patch series implements clock deregistration in the common clock framework. Comparing to v5 it only includes further corrections of NULL clock handling. [...] clk: Provide not locked variant of of_clk_get_from_provider() clkdev: Fix r

[PATCH] driver core : Fix use after free of dev->parent in device_shutdown

2013-09-24 Thread Benson Leung
The put_device(dev) at the bottom of the loop of device_shutdown may result in the dev being cleaned up. In device_create_release, the dev is kfreed. However, device_shutdown attempts to use the dev pointer again after put_device by referring to dev->parent. Copy the parent pointer instead to avo

[Patch V3] drivers: power: Add support for bq24735 charger

2013-09-24 Thread Rhyland Klein
From: Darbha Sriharsha Adds support for the bq24735 charger chipset. The bq24735 is a high-efficiency, synchronous battery charger. It allows control of the charging current, input current, and the charger voltage DAC's through SMBus. Signed-off-by: Darbha Sriharsha Signed-off-by: Rhyland Klei

Re: [RFC patch] Update Jens Osterkamp email address

2013-09-24 Thread Joe Perches
On Tue, 2013-09-24 at 14:21 -0700, Joe Perches wrote: > The old address isn't working. > > Signed-off-by: Joe Perches > --- > > Maybe if you're still interested in receiving emails > on this old driver... > > On Tue, 2013-09-24 at 14:10 -0700, postmas...@de.ibm.com wrote: > > Your message > >

Re: new binutils needed for arm in 3.12-rc1

2013-09-24 Thread Russell King - ARM Linux
On Tue, Sep 24, 2013 at 04:23:48PM -0500, Rob Landley wrote: > What value is there in requiring the new toolchain? From what I could > see of the commits it was micro-optimizations around memory barriers. > > *shrug* I can revert the patch locally, or patch the extra instruction > into my toolc

RE: Drivers: scsi: FLUSH timeout

2013-09-24 Thread KY Srinivasan
> -Original Message- > From: Mike Christie [mailto:micha...@cs.wisc.edu] > Sent: Tuesday, September 24, 2013 10:21 AM > To: KY Srinivasan > Cc: Jack Wang; Greg KH; linux-kernel@vger.kernel.org; > de...@linuxdriverproject.org; oher...@suse.com; jbottom...@parallels.com; > h...@infradead.or

Re: [PATCH] Allow userspace code to use flag IFA_F_SECONDARY to specify an ip address to be primary or secondary ip on an interface

2013-09-24 Thread Vincent Li
sorry Julian to miss your point after reading the __inet_del_ifa and see the rtmsg_ifa, fib_del_ifaddr/fib_add_ifaddr, I can try another patch and actually test if the patches changes works as it is intended, not just checking from ip binary output. Vincent On Tue, Sep 24, 2013 at 2:34 PM, Vincen

Re: [PATCH 02/10] intel: Remove extern from function prototypes

2013-09-24 Thread Jeff Kirsher
On Tue, 2013-09-24 at 14:09 -0700, Joe Perches wrote: > There are a mix of function prototypes with and without extern > in the kernel sources. Standardize on not using extern for > function prototypes. > > Function prototypes don't need to be written with extern. > extern is assumed by the compi

Re: [PATCH] driver core : Fix use after free of dev->parent in device_shutdown

2013-09-24 Thread Greg KH
On Tue, Sep 24, 2013 at 02:39:08PM -0700, Benson Leung wrote: > The put_device(dev) at the bottom of the loop of device_shutdown > may result in the dev being cleaned up. In device_create_release, > the dev is kfreed. > > However, device_shutdown attempts to use the dev pointer again after > put_d

Re: checkpatch guide for newbies

2013-09-24 Thread Alexander Holler
Am 24.09.2013 22:13, schrieb Bjorn Helgaas: On Tue, Sep 24, 2013 at 1:29 PM, Peter Senna Tschudin wrote: On Tue, Sep 24, 2013 at 7:26 PM, Alexander Holler wrote: On Mon, Sep 23, 2013 at 3:01 AM, Dan Carpenter wrote: Long Lines Historically screens were 80 characters wide

Re: [PATCH V3 1/6] clk: exynos-audss: convert to platform device

2013-09-24 Thread Andrew Bresticker
>> +static int exynos_audss_clk_remove(struct platform_device *pdev) >> +{ >> + int i; >> + >> + of_clk_del_provider(pdev->dev.of_node); >> + >> + for (i = 0; i< EXYNOS_AUDSS_MAX_CLKS; i++) { >> + if (!IS_ERR_OR_NULL(clk_table[i])) >> + clk_unr

Re: [PATCH V3 1/6] clk: exynos-audss: convert to platform device

2013-09-24 Thread Stephen Boyd
On 09/24/13 11:06, Andrew Bresticker wrote: > +static int __init exynos_audss_clk_init(void) > +{ > + return platform_driver_register(&exynos_audss_clk_driver); > +} > +core_initcall(exynos_audss_clk_init); > + > +static void __init exynos_audss_clk_exit(void) __exit? > +{ > + platform_dr

Re: [PATCH 02/10] intel: Remove extern from function prototypes

2013-09-24 Thread Ben Hutchings
On Tue, 2013-09-24 at 15:07 -0700, Jeff Kirsher wrote: > On Tue, 2013-09-24 at 14:09 -0700, Joe Perches wrote: > > There are a mix of function prototypes with and without extern > > in the kernel sources. Standardize on not using extern for > > function prototypes. > > > > Function prototypes don

Re: [RFC PATCH] fpga: Introduce new fpga subsystem

2013-09-24 Thread Greg Kroah-Hartman
On Tue, Sep 24, 2013 at 11:22:54AM -0500, Alan Tull wrote: > Yes, I could see /sys/class/fpga/fpga0/image.rbf/loading and 'data' and > a few others (If I was requesting to load 'image.rbf'). It's a nice > interface. > > I just used the linux/Documentation/firmware_class/hotplug-script > without m

[PATCH v5 0/6] rwsem: performance optimizations

2013-09-24 Thread Tim Chen
We have incorporated various suggestions from Ingo for version 5 of this patchset and will like to have it merged if there are no objections. In this patchset, we introduce two categories of optimizations to read write semaphore. The first four patches from Alex Shi reduce cache bouncing of the

[PATCH v5 3/6] rwsem: remove try_reader_grant label do_wake

2013-09-24 Thread Tim Chen
That make code simple and more readable Signed-off-by: Alex Shi --- lib/rwsem.c | 12 +++- 1 files changed, 7 insertions(+), 5 deletions(-) diff --git a/lib/rwsem.c b/lib/rwsem.c index 42f1b1a..a8055cf 100644 --- a/lib/rwsem.c +++ b/lib/rwsem.c @@ -85,15 +85,17 @@ __rwsem_do_wake(stru

[PATCH v5 4/6] rwsem/wake: check lock before do atomic update

2013-09-24 Thread Tim Chen
Atomic update lock and roll back will cause cache bouncing in large machine. A lock status pre-read can relieve this problem Suggested-by: Davidlohr bueso Suggested-by: Tim Chen Signed-off-by: Alex Shi --- lib/rwsem.c |8 +++- 1 files changed, 7 insertions(+), 1 deletions(-) diff --gi

[PATCH v5 6/6] rwsem: do optimistic spinning for writer lock acquisition

2013-09-24 Thread Tim Chen
We want to add optimistic spinning to rwsems because the writer rwsem does not perform as well as mutexes. Tim noticed that for exim (mail server) workloads, when reverting commit 4fc3f1d6 and Davidlohr noticed it when converting the i_mmap_mutex to a rwsem in some aim7 workloads. We've noticed tha

[PATCH v5 2/6] rwsem: remove 'out' label in do_wake

2013-09-24 Thread Tim Chen
That make code simple and more readable. Signed-off-by: Alex Shi --- lib/rwsem.c |5 ++--- 1 files changed, 2 insertions(+), 3 deletions(-) diff --git a/lib/rwsem.c b/lib/rwsem.c index 19c5fa9..42f1b1a 100644 --- a/lib/rwsem.c +++ b/lib/rwsem.c @@ -75,7 +75,7 @@ __rwsem_do_wake(struct rw_se

[PATCH v5 1/6] rwsem: check the lock before cpmxchg in down_write_trylock

2013-09-24 Thread Tim Chen
Cmpxchg will cause the cacheline bouning when do the value checking, that cause scalability issue in a large machine (like a 80 core box). So a lock pre-read can relief this contention. Signed-off-by: Alex Shi --- include/asm-generic/rwsem.h |8 1 files changed, 4 insertions(+), 4

[PATCH v5 5/6] MCS Lock: Restructure the MCS lock defines and locking code into its own file

2013-09-24 Thread Tim Chen
We will need the MCS lock code for doing optimistic spinning for rwsem. Extracting the MCS code from mutex.c and put into its own file allow us to reuse this code easily for rwsem. Signed-off-by: Tim Chen Signed-off-by: Davidlohr Bueso --- kernel/mutex.c | 58 ++---

Re: [PATCH 02/10] intel: Remove extern from function prototypes

2013-09-24 Thread Joe Perches
On Tue, 2013-09-24 at 15:07 -0700, Jeff Kirsher wrote: > On Tue, 2013-09-24 at 14:09 -0700, Joe Perches wrote: > > There are a mix of function prototypes with and without extern > > in the kernel sources. Standardize on not using extern for > > function prototypes. > > > > Function prototypes do

Re: [PATCH 8/8][v4] powerpc/perf: Export Power7 memory hierarchy info to user space.

2013-09-24 Thread Sukadev Bhattiprolu
Anshuman Khandual [khand...@linux.vnet.ibm.com] wrote: | On 09/14/2013 06:19 AM, Sukadev Bhattiprolu wrote: | > +static void power7_get_mem_data_src(union perf_mem_data_src *dsrc, | > + struct pt_regs *regs) | > +{ | > + u64 idx; | > + u64 mmcra = regs->dsisr; | > + u64 addr

Re: [RFC PATCH] fpga: Introduce new fpga subsystem

2013-09-24 Thread H. Peter Anvin
On 09/19/2013 03:08 AM, Pavel Machek wrote: > >> The firmware approach is interesting. It might be less flexible >> compared with my original code (see link to git below) that this is > > On the other hand... that's the interface world wants, right? To most > users, fpga bitstream is just a firm

Re: [tip:x86/urgent] x86/reboot: Add quirk to make Dell C6100 use reboot=pci automatically

2013-09-24 Thread H. Peter Anvin
On 09/23/2013 06:37 AM, tip-bot for Masoud Sharbiani wrote: > > diff --git a/arch/x86/kernel/reboot.c b/arch/x86/kernel/reboot.c > index 563ed91..5f4ad27 100644 > --- a/arch/x86/kernel/reboot.c > +++ b/arch/x86/kernel/reboot.c > @@ -358,6 +358,22 @@ static struct dmi_system_id __initdata > reboot

Re: [PATCH 1/5] mfd: arizona: Add device tree helper functions

2013-09-24 Thread Stephen Warren
On 09/24/2013 03:49 AM, Charles Keepax wrote: > On Mon, Sep 23, 2013 at 03:36:36PM -0600, Stephen Warren wrote: >> On 09/23/2013 12:30 PM, Charles Keepax wrote: >>> - - AVDD1-supply, DBVDD1-supply, DBVDD2-supply, DBVDD3-supply, >>> CPVDD-supply, >>> + - AVDD-supply, DBVDD1-supply, DBVDD2-supply,

Re: [Patch V2] drivers: power: Add support for bq24735 charger

2013-09-24 Thread Stephen Warren
On 09/24/2013 10:39 AM, Rhyland Klein wrote: > On 9/23/2013 6:05 PM, Stephen Warren wrote: >> On 09/23/2013 04:01 PM, Rhyland Klein wrote: >>> On 9/23/2013 5:53 PM, Stephen Warren wrote: On 09/19/2013 10:18 AM, Rhyland Klein wrote: > Adding driver support for bq24735 charger chipset.

Re: [PATCH v5 1/6] rwsem: check the lock before cpmxchg in down_write_trylock

2013-09-24 Thread Jason Low
Should we do something similar with __down_read_trylock, such as the following? Signed-off-by: Jason Low --- include/asm-generic/rwsem.h |3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) diff --git a/include/asm-generic/rwsem.h b/include/asm-generic/rwsem.h index bb1e2cd..47990dc 10

Re: [PATCH v5 1/6] rwsem: check the lock before cpmxchg in down_write_trylock

2013-09-24 Thread Tim Chen
On Tue, 2013-09-24 at 16:22 -0700, Jason Low wrote: > Should we do something similar with __down_read_trylock, such as > the following? > > > Signed-off-by: Jason Low > --- > include/asm-generic/rwsem.h |3 +++ > 1 files changed, 3 insertions(+), 0 deletions(-) > > diff --git a/include/asm

Re: Issues with AMD microcode updates

2013-09-24 Thread Sherry Hurwitz
On 09/19/2013 11:44 AM, Borislav Petkov wrote: On Thu, Sep 19, 2013 at 11:58:34AM -0300, Henrique de Moraes Holschuh wrote: Jacob, Andreas, I take care of the amd64 microcode update support for Debian, and I'm receiving user reports of lockup issues with the AMD microcode driver in several kern

Re: [PATCHv6 00/22] Transparent huge page cache: phase 1, everything but mmap()

2013-09-24 Thread Andrew Morton
On Mon, 23 Sep 2013 15:05:28 +0300 "Kirill A. Shutemov" wrote: > It brings thp support for ramfs, but without mmap() -- it will be posted > separately. We were never going to do this :( Has anyone reviewed these patches much yet? > Please review and consider applying. It appears rather too i

Re: [PATCH] i2o: add missing destroy_workqueue() on error in i2o_driver_register()

2013-09-24 Thread Andrew Morton
On Mon, 23 Sep 2013 17:56:56 +0800 Wei Yongjun wrote: > From: Wei Yongjun > > Add the missing destroy_workqueue() before return from > i2o_driver_register() in the error handling case. yup. > --- a/drivers/message/i2o/driver.c > +++ b/drivers/message/i2o/driver.c > @@ -105,7 +105,8 @@ int i2o

Re: [PATCHv6 00/22] Transparent huge page cache: phase 1, everything but mmap()

2013-09-24 Thread Andi Kleen
On Tue, Sep 24, 2013 at 04:37:40PM -0700, Andrew Morton wrote: > On Mon, 23 Sep 2013 15:05:28 +0300 "Kirill A. Shutemov" > wrote: > > > It brings thp support for ramfs, but without mmap() -- it will be posted > > separately. > > We were never going to do this :( > > Has anyone reviewed these p

[PATCH 1/1] i2c-ismt: add Initialize DMA buffer code ismt_access()

2013-09-24 Thread James Ralston
This patch adds code to Initialize the DMA buffer to compensate for possible hardware data corruption. Signed-off-by: James Ralston --- drivers/i2c/busses/i2c-ismt.c |3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) diff --git a/drivers/i2c/busses/i2c-ismt.c b/drivers/i2c/busses/i2c

Re: [PATCH 1/7] powerpc: Add interface to get msi region information

2013-09-24 Thread Bjorn Helgaas
On Thu, Sep 19, 2013 at 12:59:17PM +0530, Bharat Bhushan wrote: > This patch adds interface to get following information > - Number of MSI regions (which is number of MSI banks for powerpc). > - Get the region address range: Physical page which have the > address/addresses used for generat

Re: [PATCHv6 00/22] Transparent huge page cache: phase 1, everything but mmap()

2013-09-24 Thread Andrew Morton
On Tue, 24 Sep 2013 16:49:50 -0700 Andi Kleen wrote: > > At the very least we should get this done for a real filesystem to see > > how intrusive the changes are and to evaluate the performance changes. > > That would give even larger patches, and people already complain > the patchkit is too la

[GIT PULL] drivers/clocksource: sched_clock_register() conversion for 3.13 (via tip/timers/core)

2013-09-24 Thread John Stultz
Hey Daniel, Here's the sched_clock_register conversion that Stephen did earlier this summer, as part of extending the generic sched_clock code to support 64bit counters. Unfortunately the prereqs for this series missed getting into tip/timers/core for 3.12, so I never sent out this pull request

Re: linux-next: manual merge of the ipsec-next tree with the net-next tree

2013-09-24 Thread Stephen Rothwell
Hi Steffen, On Tue, 24 Sep 2013 12:25:05 +0200 Steffen Klassert wrote: > > On Tue, Sep 24, 2013 at 12:16:29PM +1000, Stephen Rothwell wrote: > > > > Today's linux-next merge of the ipsec-next tree got a conflict in > > include/net/xfrm.h between commit d511337a1eda ("xfrm.h: Remove extern > > f

[ 18/28] HID: pantherlord: validate output report details

2013-09-24 Thread Greg Kroah-Hartman
3.0-stable review patch. If anyone has any objections, please let me know. -- From: Kees Cook commit 412f30105ec6735224535791eed5cdc02888ecb4 upstream. A HID device could send a malicious output report that would cause the pantherlord HID driver to write beyond the output repo

[ 26/28] mmc: tmio_mmc_dma: fix PIO fallback on SDHI

2013-09-24 Thread Greg Kroah-Hartman
3.0-stable review patch. If anyone has any objections, please let me know. -- From: Sergei Shtylyov commit f936f9b67b7f8c2eae01dd303a0e90bd777c4679 upstream. I'm testing SH-Mobile SDHI driver in DMA mode with a new DMA controller using 'bonnie++' and getting DMA error after

[ 01/28] SCSI: sd: Fix potential out-of-bounds access

2013-09-24 Thread Greg Kroah-Hartman
3.0-stable review patch. If anyone has any objections, please let me know. -- From: Alan Stern commit 984f1733fcee3fbc78d47e26c5096921c5d9946a upstream. This patch fixes an out-of-bounds error in sd_read_cache_type(), found by Google's AddressSanitizer tool. When the loop end

[ 25/28] isofs: Refuse RW mount of the filesystem instead of making it RO

2013-09-24 Thread Greg Kroah-Hartman
3.0-stable review patch. If anyone has any objections, please let me know. -- From: Jan Kara commit 17b7f7cf58926844e1dd40f5eb5348d481deca6a upstream. Refuse RW mount of isofs filesystem. So far we just silently changed it to RO mount but when the media is writeable, block lay

[ 21/28] HID: check for NULL field when setting values

2013-09-24 Thread Greg Kroah-Hartman
3.0-stable review patch. If anyone has any objections, please let me know. -- From: Kees Cook commit be67b68d52fa28b9b721c47bb42068f0c1214855 upstream. Defensively check that the field to be worked on is not NULL. Signed-off-by: Kees Cook Signed-off-by: Jiri Kosina Signed-o

[ 14/28] ASoC: wm8960: Fix PLL register writes

2013-09-24 Thread Greg Kroah-Hartman
3.0-stable review patch. If anyone has any objections, please let me know. -- From: Mike Dyer commit 85fa532b6ef920b32598df86b194571a7059a77c upstream. Bit 9 of PLL2,3 and 4 is reserved as '0'. The 24bit fractional part should be split across each register in 8bit chunks. Sig

[ 23/28] memcg: fix multiple large threshold notifications

2013-09-24 Thread Greg Kroah-Hartman
3.0-stable review patch. If anyone has any objections, please let me know. -- From: Greg Thelen commit 2bff24a3707093c435ab3241c47dcdb5f16e432b upstream. A memory cgroup with (1) multiple threshold notifications and (2) at least one threshold >=2G was not reliable. Specifical

<    1   2   3   4   5   6   7   8   9   10   >