Re: [PATCH 3/6] dma: acpi-dma: parse CSRT to extract additional resources

2013-03-30 Thread Mika Westerberg
On Sat, Mar 30, 2013 at 03:03:50AM +0530, Vinod Koul wrote: > On Wed, Mar 27, 2013 at 10:57:59AM +0200, Andy Shevchenko wrote: > > Since we have CSRT only to get additional DMA controller resources, let's > > get > > rid of drivers/acpi/csrt.c and move its logic inside ACPI DMA helpers code. > >

Re: [PATCH v3 5/7] USB: EHCI: make ehci-atmel a separate driver

2013-03-30 Thread Nicolas Ferre
On 03/29/2013 09:02 PM, Alan Stern : > On Thu, 28 Mar 2013, Arnd Bergmann wrote: > >> From: Manjunath Goudar >> >> Separate the Atmel host controller driver from ehci-hcd host code >> so that it can be built as a separate driver module. >> This work is part of enabling multi-platform kernels on A

Re: [PATCH 3/6] i2c: Ignore return value of i2c_del_adapter()

2013-03-30 Thread Jean Delvare
Hi Lars-Peter, On Sat, 9 Mar 2013 19:16:46 +0100, Lars-Peter Clausen wrote: > i2c_del_adapter() always returns 0. I beg you pardon? i2c_del_adapter() starts with: int i2c_del_adapter(struct i2c_adapter *adap) { int res = 0; struct i2c_adapter *found; struct i2c_client *c

[PATCH 1/1] TTY Fix NULL pointer error in tty_wakeup

2013-03-30 Thread Frankie Lin
When calling tty_wakeup and the parameter is NULL,the oops will happen. The virtual address &tty->flags can not be handled in the kernel space, while PC goto test_bit(TTY_DO_WRITE_WAKEUP, &tty->flags). Tested on ARM Machine msm8625. Signed-off-by: Frankie Lin --- drivers/tty/tty_io.c |5 +++

Re: [PATCH 3/6] i2c: Ignore return value of i2c_del_adapter()

2013-03-30 Thread Lars-Peter Clausen
Hi Jean, On 03/30/2013 08:55 AM, Jean Delvare wrote: > Hi Lars-Peter, > > On Sat, 9 Mar 2013 19:16:46 +0100, Lars-Peter Clausen wrote: >> i2c_del_adapter() always returns 0. > > I beg you pardon? i2c_del_adapter() starts with: > > int i2c_del_adapter(struct i2c_adapter *adap) > { > int r

Re: [PATCH] dma: Make the 'mask' parameter of __dma_request_channel const

2013-03-30 Thread Lars-Peter Clausen
On 03/29/2013 11:47 PM, Vinod Koul wrote: > On Mon, Mar 25, 2013 at 01:23:52PM +0100, Lars-Peter Clausen wrote: >> The 'mask' parameter is not modified in __dma_request_channel and really >> shouldn't be. Make this explicit by making the parameter const. >> >> Signed-off-by: Lars-Peter Clausen > A

Re: [PATCH 1/2] dma: of: Add generic xlate function for matching by channel id

2013-03-30 Thread Lars-Peter Clausen
On 03/29/2013 10:49 PM, Vinod Koul wrote: > On Mon, Mar 25, 2013 at 02:16:51PM +0100, Lars-Peter Clausen wrote: >> This patch adds a new generic of dma xlate callback function which will >> match a >> channel by its id. > how will the client know which "id" to request? The devicetree tells it, o

Re: [PATCH 3/6] i2c: Ignore return value of i2c_del_adapter()

2013-03-30 Thread Jean Delvare
On Sat, 30 Mar 2013 09:11:54 +0100, Lars-Peter Clausen wrote: > Hi Jean, > > On 03/30/2013 08:55 AM, Jean Delvare wrote: > > Hi Lars-Peter, > > > > On Sat, 9 Mar 2013 19:16:46 +0100, Lars-Peter Clausen wrote: > >> i2c_del_adapter() always returns 0. > > > > I beg you pardon? i2c_del_adapter() s

[PATCH] [RESEND] arch: remove CONFIG_GENERIC_FIND_NEXT_BIT again

2013-03-30 Thread Paul Bolle
CONFIG_GENERIC_FIND_NEXT_BIT was removed in v3.0, but reappeared in two architectures. Remove it again. Signed-off-by: Paul Bolle Acked-by: Jonas Bonn --- 0) Richard asked (off list) to resend. There appear to be problems locating the patch in the archives for LKML and/or linux-hexagon. 1) Adde

Re: [PATCH 4/4] nohz: New option to force all CPUs in full dynticks range

2013-03-30 Thread Ingo Molnar
* Frederic Weisbecker wrote: > 2013/3/28 Ingo Molnar : > > > > * Frederic Weisbecker wrote: > > > >> +config NO_HZ_EXTENDED_ALL > >> + bool "Full dynticks system on all CPUs" > >> + depends on NO_HZ_EXTENDED > >> + help > >> + Force all CPUs to be full dynticks. The range

Re: [BUG] Crash with NULL pointer dereference in bond_handle_frame in -rt (possibly mainline)

2013-03-30 Thread Jiri Pirko
Fri, Mar 29, 2013 at 07:36:24PM CET, rost...@goodmis.org wrote: >On Fri, 2013-03-29 at 10:48 +0100, Jiri Pirko wrote: > >> Because, if rcu_dereference(dev->rx_handler) is null, >> rcu_dereference(dev->rx_handler_data) is never done. Therefore I believe >> we are hitting following scenario: >> >>

Re: [PATCH 4/9] ia64: cpufreq: move cpufreq driver to drivers/cpufreq

2013-03-30 Thread Viresh Kumar
On 25 March 2013 22:24, Viresh Kumar wrote: > diff --git a/arch/ia64/Kconfig b/arch/ia64/Kconfig > index 9a02f71..dd59848 100644 > --- a/arch/ia64/Kconfig > +++ b/arch/ia64/Kconfig > @@ -591,9 +591,8 @@ source "kernel/power/Kconfig" > source "drivers/acpi/Kconfig" > > if PM > - > -source "arch/i

[PATCH] ARM: S5P64X0: Fix typo "CONFIG_S5P64X0_SETUP_SDHCI"

2013-03-30 Thread Paul Bolle
Commit ebc433c2890f8ecad2da39fe2dbf2b6e7b309afa ("ARM: S5P64X0: Add HSMMC setup for host Controller") added the Kconfig symbol S5P64X0_SETUP_SDHCI_GPIO. By accident it also used a macro CONFIG_S5P64X0_SETUP_SDHCI. Fix that typo. Signed-off-by: Paul Bolle --- Untested. Since this typo has been in

Re: [PATCH v2] pwm: spear: Fix enable pwm output

2013-03-30 Thread Viresh Kumar
On 30 March 2013 11:07, Axel Lin wrote: > The logic to check return value of clk_enable() is reversed, thus > when clk_enable() passes PWMCR_PWM_ENABLE bit is not set. Fix it. > > Signed-off-by: Axel Lin > --- > drivers/pwm/pwm-spear.c |4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(

Re: [PATCH] ARM: timer: Shutdown clock event device when stopping local timer

2013-03-30 Thread Ning Jiang
2013/3/30 Stephen Boyd : > On 03/29/13 02:24, ning.n.ji...@gmail.com wrote: >> From: Ning Jiang >> >> Currently there are two problems when we try to stop local timer. >> First, it calls set_mode function directly so mode state is not >> updated for the clock event device. Second, it makes the dev

Re: Patch "efi: Build EFI stub with EFI-appropriate options" has been added to the 3.6-stable tree

2013-03-30 Thread Daniel Vacek
On Sat, Mar 30, 2013 at 4:28 AM, Ben Hutchings wrote: > Daniel, this is an obvious fix but I noticed it still hasn't been > applied. Please can you re-send with the proper Signed-off-by line? Resending patch. It got lost somewhere. Thanks for pushing Ben. >From 53864fe932aa87be62f1c70944c386cff

Re: [PATCH] ARM: timer: Shutdown clock event device when stopping local timer

2013-03-30 Thread Russell King - ARM Linux
On Sat, Mar 30, 2013 at 05:57:38PM +0800, Ning Jiang wrote: > 2013/3/30 Stephen Boyd : > > On 03/29/13 02:24, ning.n.ji...@gmail.com wrote: > >> From: Ning Jiang > >> > >> Currently there are two problems when we try to stop local timer. > >> First, it calls set_mode function directly so mode stat

Re: 3.8.3 build error, compiler segfault

2013-03-30 Thread Tom H
On Fri, Mar 29, 2013 at 9:27 AM, Gene Heskett wrote: > On Friday 29 March 2013 09:08:27 Tom H did opine: > > CCing the lkml too. > >> On Thu, Mar 28, 2013 at 10:49 AM, Nils Kassube wrote: >>> I don't know of a particilar tool to extract that info. I can't tell >>> you for your -rtai kernel but th

Re: [PATCH] hfs: add error checking for hfs_find_init()

2013-03-30 Thread Vyacheslav Dubeyko
Hi Alexey, On Mar 30, 2013, at 12:44 AM, Alexey Khoroshilov wrote: > hfs_find_init() may fail with ENOMEM, but there are places, > where the returned value is not checked. The consequences can be > very unpleasant, e.g. kfree uninitialized pointer and > inappropriate mutex unlocking. > > The pat

[PATCH] sparc:cleanup unused code in smp_32.h

2013-03-30 Thread Kefeng Wang
After genirq and generic clockevent support at sparc32, smp4m_irq_rotate(), prof_multiplier() and prof_counter() are no longer used and should be removed. Find more info from commit 6baa9b20 & 62f08283. Signed-off-by: Kefeng Wang --- arch/sparc/include/asm/smp_32.h |4 1 files changed,

Re: [PATCH] checkpatch: do not exit in main process subroutine

2013-03-30 Thread zhangwei(Jovi)
On 2013/3/28 11:43, zhangwei(Jovi) wrote: > Currently checkpatch program exit when process empty file, > This will cause issue when @ARGV include many files ready to check. > > For example, there may have Module.symvers empty file in kernel directory. > > [root@jovi ~]# scripts/checkpatch.pl -f k

Re: [patch v5 14/15] sched: power aware load balance

2013-03-30 Thread Preeti U Murthy
On 03/29/2013 07:09 PM, Alex Shi wrote: > On 03/29/2013 08:42 PM, Preeti U Murthy wrote: did you try the simplest benchmark: while true; do :; done >> Yeah I tried out this while true; do :; done benchmark on a vm which ran > > Thanks a lot for trying! > > What's do you mean 'vm'? Virtual ma

[PATCH] cpufreq: Documentation: Fix cpufreq_frequency_table name

2013-03-30 Thread Viresh Kumar
At few places in documentation cpufreq_frequency_table is written as cpufreq_freq_table. Fix these. Signed-off-by: Viresh Kumar --- Documentation/cpu-freq/cpu-drivers.txt | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/Documentation/cpu-freq/cpu-drivers.txt b/Documentati

Re: [PATCH v3 1/7] USB: EHCI: make ehci-orion a separate driver

2013-03-30 Thread Arnd Bergmann
On Friday 29 March 2013, Alan Stern wrote: > I don't know about this last phrase. When someone is running "make > menuconfig", for example, what shows up is the symbol's description, > not the symbol's name. That person would see "EHCI support for Marvell > on-chip controller", not "USB_EHCI_M

[PATCH] cpufreq: Raname index as data in cpufreq_frequency_table

2013-03-30 Thread Viresh Kumar
"Index" field of struct cpufreq_frequency_table was never index and isn't used at all by cpufreq core. And is only useful for cpufreq drivers for their personal use. Many people now a days blindly set it in ascending order with the assumption that core is using it for some work. This patch rename

Re: [PATCH 3/5] dump_stack: consolidate dump_stack() implementations and unify their behaviors

2013-03-30 Thread Heiko Carstens
On Fri, Mar 29, 2013 at 07:27:06PM -0700, Tejun Heo wrote: > Both dump_stack() and show_stack() are currently implemented by each > architecture. show_stack(NULL, NULL) dumps the backtrace for the > current task as does dump_stack(). On some archs, dump_stack() prints > extra information - pid, u

Re: WARNING: at net/batman-adv/hard-interface.c:92 batadv_is_on_batman_iface()

2013-03-30 Thread Antonio Quartulli
Hello all, On Sat, Mar 30, 2013 at 05:36:35PM +0800, Fengguang Wu wrote: > Greetings, > > I got the below WARNING in net-next/master (head commit f498354793d5) > and the first bad commit is > > commit c54419321455631079c7d6e60bc732dd0c5914c5 > Author: Pravin B Shelar > Date: Mon Mar 25 14:49:

Re: [PATCH] ARM: timer: Shutdown clock event device when stopping local timer

2013-03-30 Thread Ning Jiang
2013/3/30 Russell King - ARM Linux : > On Sat, Mar 30, 2013 at 05:57:38PM +0800, Ning Jiang wrote: >> 2013/3/30 Stephen Boyd : >> > On 03/29/13 02:24, ning.n.ji...@gmail.com wrote: >> >> From: Ning Jiang >> >> >> >> Currently there are two problems when we try to stop local timer. >> >> First, it

Re: [PATCH v3 2/7] USB: EHCI: make ehci-spear a separate driver

2013-03-30 Thread Arnd Bergmann
On Friday 29 March 2013, Alan Stern wrote: > On Thu, 28 Mar 2013, Arnd Bergmann wrote: > > > From: Manjunath Goudar > > > > Separate the SPEAr host controller driver from ehci-hcd host code > > so that it can be built as a separate driver module. > > This work is part of enabling multi-platform

Re: [PATCH] sparc:cleanup unused code in smp_32.h

2013-03-30 Thread Sam Ravnborg
On Sat, Mar 30, 2013 at 06:28:26PM +0800, Kefeng Wang wrote: > After genirq and generic clockevent support at sparc32, > smp4m_irq_rotate(), prof_multiplier() and prof_counter() > are no longer used and should be removed. > > Find more info from commit 6baa9b20 & 62f08283. > > Signed-off-by: Kefe

Re: [PATCH v3 3/7] USB: EHCI: make ehci-s5p a separate driver

2013-03-30 Thread Arnd Bergmann
On Friday 29 March 2013, Alan Stern wrote: > On Thu, 28 Mar 2013, Arnd Bergmann wrote: > > Personally, I would have left these two functions the way they were and > relied on the compiler to inline them when appropriate. Eliminating > them just makes the code more complicated. Yes, makes sens

Re: [PATCH v3 5/7] USB: EHCI: make ehci-atmel a separate driver

2013-03-30 Thread Arnd Bergmann
On Friday 29 March 2013, Alan Stern wrote: > While not absolutely necessary, it would be nice to have the #include > files in alphabetical order. > > > + > > +#include "ehci.h" > > + > > +#define DRIVER_DESC "EHCI atmel driver" > > "atmel" should have a capital 'A'. > Ok, added these changes

n_tty_write() going into schedule but NOT coming out

2013-03-30 Thread Vineet Gupta
Hi, I've been stress testing ARC Linux 3.8 (same happens for 3.9-rc3 as well). The setup has 3 telnet sessions, each running find . -name "*" in a loop. The platform is a FPGA @ 80 MHz, running a single core ARC700 so kernel .config has !SMP and PREEMPT_NONE. After ~10 mins of run, I see that one

Re: r8169 auto speed down issue

2013-03-30 Thread Francois Romieu
hayeswang : [...] > Sorry for my unclear descriptor. I just think a case that the nic suspends or > shutdowns without cable plugging. Then, the cable is plugged again. If the nic > speed down to 10M and the link partner force 100M, the issue appears again. If > the nic doesn't speed down for norma

Re: [PATCH] r8169: fix auto speed down issue

2013-03-30 Thread Francois Romieu
Hayes Wang : [...] > diff --git a/drivers/net/ethernet/realtek/r8169.c > b/drivers/net/ethernet/realtek/r8169.c > index 28fb50a..a9eedf7 100644 > --- a/drivers/net/ethernet/realtek/r8169.c > +++ b/drivers/net/ethernet/realtek/r8169.c > @@ -3818,6 +3818,21 @@ static void rtl_init_mdio_ops(struct r

[PATCH] pwm: spear: Fix checking return value of clk_enable() and clk_prepare()

2013-03-30 Thread Axel Lin
The logic to check return value of clk_enable() and clk_prepare() is reversed, fix it. Signed-off-by: Axel Lin Cc: --- This is v3 of "pwm: spear: Fix enable pwm output". I change the subject line because it also fixes checking return value of clk_prepare(). Also add cc stable. Axel drivers/pw

Re: [PATCH] pwm: spear: Fix checking return value of clk_enable() and clk_prepare()

2013-03-30 Thread Viresh Kumar
On 30 March 2013 18:13, Axel Lin wrote: > The logic to check return value of clk_enable() and clk_prepare() is reversed, > fix it. > > Signed-off-by: Axel Lin > Cc: > --- > This is v3 of "pwm: spear: Fix enable pwm output". > I change the subject line because it also fixes checking return value

Re: [PATCH] pwm: spear: Fix checking return value of clk_enable() and clk_prepare()

2013-03-30 Thread Axel Lin
> > Sorry for the delays caused due to me. > > Acked-by: Viresh Kumar Actually, I'm really appreciate having your review. Thanks, Axel -- 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://v

Re: [PATCH v3 6/7] USB: EHCI: make ehci-msm a separate driver

2013-03-30 Thread Arnd Bergmann
On Friday 29 March 2013, Alan Stern wrote: > On Thu, 28 Mar 2013, Arnd Bergmann wrote: > This patch is good. However the ehci-msm driver itself is not. While > checking through the code, I was struck by the fact that it never calls > usb_add_hcd() or usb_remove_hcd(). Obviously the driver canno

Re: [PATCH v3 7/7] USB: OHCI: avoid conflicting platform drivers

2013-03-30 Thread Arnd Bergmann
On Friday 29 March 2013, Alan Stern wrote: > I have not checked the details of all the changes; however, the basic > idea is okay as a stop-gap measure. Ok, thanks. > I guess this means the onus is now on me to split up ohci-hcd into a > central library and separate bus drivers, like ehci-hcd..

[PATCH] cpufreq: cpufreq-cpu0: No need to check cpu number in init()

2013-03-30 Thread Viresh Kumar
It is not possible for init() to be called for any cpu other than cpu0. During bootup whatever cpu is used to boot system will be assigned as cpu0. And later on policy->cpu can only change if we hotunplug all cpus first and then hotplug them back in different order, which isn't possible (system req

[PATCH] acpi/video: lcd_get_level_current doen't return current level

2013-03-30 Thread Artem Savkov
acpi_video_device_lcd_get_level_current() called acpi_video_bqc_value_to_level() with "*level" as a second argument, resulting in level being returned based on initial input, not current brightness, breaking backlight controls. Signed-off-by: Artem Savkov --- drivers/acpi/video.c | 3 ++- 1 file

[PATCH v4 1/6] USB: EHCI: make ehci-orion a separate driver

2013-03-30 Thread Arnd Bergmann
From: Manjunath Goudar Separate the Orion host controller driver from ehci-hcd host code into its own driver module because of following reason. With the multiplatform changes in arm-soc tree, it becomes possible to enable the mvebu platform (which uses ehci-orion) at the same time as other plat

[PATCH v4 5/6] USB: EHCI: make ehci-msm a separate driver

2013-03-30 Thread Arnd Bergmann
From: Manjunath Goudar Separate the Qualcomm QSD/MSM on-chip host controller driver from ehci-hcd host code so that it can be built as a separate driver module. This work is part of enabling multi-platform kernels on ARM; however, note that other changes are still needed before Qualcomm QSD/MSM

[PATCH v4 3/6] USB: EHCI: make ehci-s5p a separate driver

2013-03-30 Thread Arnd Bergmann
From: Manjunath Goudar Separate the Samsung S5P/EXYNOS host controller driver from ehci-hcd host code so that it can be built as a separate driver module. This work is part of enabling multi-platform kernels on ARM; however, note that other changes are still needed before S5P/EXYNOS can be boote

[PATCH v4 2/6] USB: EHCI: make ehci-spear a separate driver

2013-03-30 Thread Arnd Bergmann
From: Manjunath Goudar Separate the SPEAr host controller driver from ehci-hcd host code so that it can be built as a separate driver module. This work is part of enabling multi-platform kernels on ARM; however, note that other changes are still needed before SPEAr can be booted with a multi-plat

[PATCH v4 4/6] USB: EHCI: make ehci-atmel a separate driver

2013-03-30 Thread Arnd Bergmann
From: Manjunath Goudar Separate the Atmel host controller driver from ehci-hcd host code so that it can be built as a separate driver module. This work is part of enabling multi-platform kernels on ARM; however, note that other changes are still needed before Atmel can be booted with a multi-plat

[PATCH v4 0/6] USB EHCI multiplatform conversion

2013-03-30 Thread Arnd Bergmann
Hi Alan, This is a quick update to v3, addressing all issues you pointed in yesterday's review. Thanks for taking a thorough look on short notice on a public holiday! Arnd Arnd Bergmann (1): USB: OHCI: avoid conflicting platform drivers Manjunath Goudar (5): USB: EHCI: make ehci-ori

[PATCH 0/3] posix_timers: A few expiry caching fixes

2013-03-30 Thread Frederic Weisbecker
Hi, This series is on top of latest -mmotm. [1/3] fixes some omitting sample capture after a build warning suddenly became visible [2/3] fixes a wrong timer_gettime() result on a timer whose target has been reaped. [3/3] needs some more thinking probably Thanks. Frederic Weisbecker (3): pos

[PATCH 2/3] posix_timers: Fix racy timer delta caching on task exit

2013-03-30 Thread Frederic Weisbecker
When a task exits, we perform a caching of the remaining cputime delta before expiring of its timers. This is done from the following places: * When the task is reaped. We iterate through its list of posix cpu timers and store the remaining timer delta to the timer struct instead of the absol

[PATCH 3/3] posix_timers: Remove dead task timer expiry caching

2013-03-30 Thread Frederic Weisbecker
When reading a timer sample, posix_cpu_timer_get() and posix_cpu_timer_schedule() both perform a caching of the timer expiry time by converting its value from absolute to relative if the task has exited. The reason for this caching is not clear though, it could be: 1) For performance reasons: no

[PATCH 1/3] posix-timers: Correctly get dying task time sample in posix_cpu_timer_schedule()

2013-03-30 Thread Frederic Weisbecker
In order to arm the next timer to schedule, we take a sample of the current process or thread cputime. If the task is dying though, we don't arm anything but we cache the remaining timer expiration delta for further reads. Something similar is performed in posix_cpu_timer_get() but here we forget

Re: [PATCH] Fix IXP4xx coherent allocations

2013-03-30 Thread Krzysztof Halasa
David Miller writes: >> ARM core code currently requires coherent DMA mask to be set. > This requirement is not reasonable. > > The DMA API documentation clearly states what the default must be, > and what drivers are guarenteed will be the default. ARM people - what do you think? -- Krzysztof

[PATCH v4 6/6] USB: OHCI: avoid conflicting platform drivers

2013-03-30 Thread Arnd Bergmann
Like the EHCI driver, OHCI supports a large number of different platform glue drivers by directly including them, which causes problems with conflicting macro definitions in some cases. As more ARM architecture specific back-ends are required to coexist in a single build, we should split those out

Re: [PATCH] Fix IXP4xx coherent allocations

2013-03-30 Thread Russell King - ARM Linux
On Sun, Mar 24, 2013 at 10:15:36PM +0100, Krzysztof Halasa wrote: > The problem on ARM (and probably on powerpc, and on something called > "metag" - grep -r 'coherent DMA mask is unset' arch) is that the default > coherent DMA mask is zero. IOW, coherent DMA allocations are, by > default, disabled.

Re: [PATCH -mm -next] ipc,sem: untangle RCU locking with find_alloc_undo

2013-03-30 Thread Sasha Levin
On 03/28/2013 11:32 AM, Rik van Riel wrote: > On Tue, 26 Mar 2013 13:33:07 -0400 > Sasha Levin wrote: > >> > [ 96.347341] >> > [ 96.348085] [ BUG: lock held when returning to user space! ] >> > [ 96.348834] 3.9.0-rc4-next-20130326-sasha-00011

Re: [PATCH] regulator: max1586: Constify v6_voltages_uv table

2013-03-30 Thread Mark Brown
On Sat, Mar 30, 2013 at 01:57:29PM +0800, Axel Lin wrote: > Make v6_voltages_uv to be const unsigned int. Applied, thanks. signature.asc Description: Digital signature

Re: [PATCH] hfs: add error checking for hfs_find_init()

2013-03-30 Thread Hin-Tak Leung
--- On Sat, 30/3/13, Vyacheslav Dubeyko wrote: > From: Vyacheslav Dubeyko > Subject: Re: [PATCH] hfs: add error checking for hfs_find_init() > To: "Alexey Khoroshilov" > Cc: "Al Viro" , "Artem Bityutskiy" > , "Christoph Hellwig" , > linux-fsde...@vger.kernel.org, linux-kernel@vger.kernel.org,

Re: [patch v5 14/15] sched: power aware load balance

2013-03-30 Thread Alex Shi
On 03/30/2013 07:25 PM, Preeti U Murthy wrote: >> > I still give the rq->util weight even the nr_running is 0, because some >> > transitory tasks may actived on the cpu, but just missed on balancing >> > point. >> > >> > I just wondering that forgetting rq->util when nr_running = 0 is the >> > re

[PATCH] cpufreq: cpufreq-cpu0: Call CPUFREQ_POSTCHANGE notifier for failure cases too

2013-03-30 Thread Viresh Kumar
Currently we are simply returning from target() if we encounter some error after broadcasting CPUFREQ_PRECHANGE notifier. Which looks to be wrong as others might depend on POSTCHANGE notifier for their functioning. So, better broadcast CPUFREQ_POSTCHANGE notifier for these failure cases too, but w

Re: [PATCH] Fix IXP4xx coherent allocations

2013-03-30 Thread Krzysztof Halasa
Russell King - ARM Linux writes: > I'm having a hard time understanding what is an ARM issue here, what is > an ARM bug, and what the DMA API requires. The DMA API documentation > is extremely sparse in describing what's required of the DMA masks, > what these functions are supposed to do, and w

[patch v6 0/21] sched: power aware scheduling

2013-03-30 Thread Alex Shi
This patch set implement/consummate the rough power aware scheduling proposal: https://lkml.org/lkml/2012/8/13/139. The code also on this git tree: https://github.com/alexshi/power-scheduling.git power-scheduling The patch defines a new policy 'powersaving', that try to pack tasks on each sched g

[patch v6 01/21] Revert "sched: Introduce temporary FAIR_GROUP_SCHED dependency for load-tracking"

2013-03-30 Thread Alex Shi
Remove CONFIG_FAIR_GROUP_SCHED that covers the runnable info, then we can use runnable load variables. Signed-off-by: Alex Shi --- include/linux/sched.h | 7 +-- kernel/sched/core.c | 7 +-- kernel/sched/fair.c | 13 ++--- kernel/sched/sched.h | 9 + 4 files chang

[patch v6 03/21] sched: only count runnable avg on cfs_rq's nr_running

2013-03-30 Thread Alex Shi
Old function count the runnable avg on rq's nr_running even there is only rt task in rq. That is incorrect, so correct it to cfs_rq's nr_running. Signed-off-by: Alex Shi --- kernel/sched/fair.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/kernel/sched/fair.c b/kernel/sched

[patch v6 06/21] sched: log the cpu utilization at rq

2013-03-30 Thread Alex Shi
The cpu's utilization is to measure how busy is the cpu. util = cpu_rq(cpu)->avg.runnable_avg_sum * SCHED_POEWR_SCALE / cpu_rq(cpu)->avg.runnable_avg_period; Since the util is no more than 1, we scale its value with 1024, same as SCHED_POWER_SCALE and set the FULL_UTIL as 1

[patch v6 13/21] sched: using avg_idle to detect bursty wakeup

2013-03-30 Thread Alex Shi
Sleeping task has no utiliation, when they were bursty waked up, the zero utilization make scheduler out of balance, like aim7 benchmark. rq->avg_idle is 'to used to accommodate bursty loads in a dirt simple dirt cheap manner' -- Mike Galbraith. With this cheap and smart bursty indicator, we can

[patch v6 14/21] sched: packing transitory tasks in wakeup power balancing

2013-03-30 Thread Alex Shi
If the waked task is transitory enough, it will has a chance to be packed into a cpu which is busy but still has time to care it. For powersaving policy, only the history util < 25% task has chance to be packed. If there is no cpu eligible to handle it, will use a idlest cpu in leader group. Mort

[patch v6 21/21] sched: don't do power balance on share cpu power domain

2013-03-30 Thread Alex Shi
Packing tasks among such domain can't save power, just performance losing. So no power balance on them. Signed-off-by: Alex Shi --- kernel/sched/fair.c | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/kernel/sched/fair.c b/kernel/sched/fair.c index 430904b..88c8bd6 1006

[patch v6 18/21] sched: add new members of sd_lb_stats

2013-03-30 Thread Alex Shi
Added 4 new members in sb_lb_stats, that will be used in incomming power aware balance. group_min; //least utliszation group in domain min_load_per_task; //load_per_task in group_min leader_util;// sum utilizations of group_leader min_util; // sum utilizations of group_min Signed-o

[patch v6 17/21] sched: no balance for prefer_sibling in power scheduling

2013-03-30 Thread Alex Shi
In power aware scheduling, we don't want to balance 'prefer_sibling' groups just because local group has capacity. If the local group has no tasks at the time, that is the power balance hope so. Signed-off-by: Alex Shi --- kernel/sched/fair.c | 6 +- 1 file changed, 5 insertions(+), 1 deleti

[patch v6 19/21] sched: power aware load balance

2013-03-30 Thread Alex Shi
This patch enabled the power aware consideration in load balance. As mentioned in the power aware scheduler proposal, Power aware scheduling has 2 assumptions: 1, race to idle is helpful for power saving 2, less active sched_groups will reduce power consumption The first assumption make performan

[patch v6 08/21] sched: move sg/sd_lb_stats struct ahead

2013-03-30 Thread Alex Shi
Power aware fork/exec/wake balancing needs both of structs in incoming patches. So move ahead before it. Signed-off-by: Alex Shi --- kernel/sched/fair.c | 99 +++-- 1 file changed, 50 insertions(+), 49 deletions(-) diff --git a/kernel/sched/fair.c

[patch v6 15/21] sched: add power/performance balance allow flag

2013-03-30 Thread Alex Shi
If a sched domain is idle enough for regular power balance, LBF_POWER_BAL will be set, LBF_PERF_BAL will be clean. If a sched domain is busy, their value will be set oppositely. If the domain is suitable for power balance, but balance should not be down by this cpu(this cpu is already idle or full

[patch v6 16/21] sched: pull all tasks from source group

2013-03-30 Thread Alex Shi
In power balance, we hope some sched groups are fully empty to save CPU power of them. So, we want to move any tasks from them. Signed-off-by: Alex Shi --- kernel/sched/fair.c | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/kernel/sched/fair.c b/kernel/sched/fair.c ind

[patch v6 07/21] sched: add new sg/sd_lb_stats fields for incoming fork/exec/wake balancing

2013-03-30 Thread Alex Shi
For power aware balancing, we care the sched domain/group's utilization. So add: sd_lb_stats.sd_util and sg_lb_stats.group_util. And want to know which group is busiest but still has capability to handle more tasks, so add: sd_lb_stats.group_leader Signed-off-by: Alex Shi --- kernel/sched/fair.

[patch v6 12/21] sched: add power aware scheduling in fork/exec/wake

2013-03-30 Thread Alex Shi
This patch add power aware scheduling in fork/exec/wake. It try to select cpu from the busiest while still has utilization group. That's will save power since it leaves more groups idle in system. The trade off is adding a power aware statistics collection in group seeking. But since the collectio

Re: [PATCH 1/3] kthread: implement probe_kthread_data()

2013-03-30 Thread Oleg Nesterov
On 03/29, Tejun Heo wrote: > > Implement probe_kthread_data() which returns kthread_data if > accessible. The function is equivalent to kthread_data() except that > the specified @task may not be a kthread ^ This doesn't necessarily mean its ->vfork_done !

[patch v6 11/21] sched: detect wakeup burst with rq->avg_idle

2013-03-30 Thread Alex Shi
rq->avg_idle is 'to used to accommodate bursty loads in a dirt simple dirt cheap manner' -- Mike Galbraith. With this cheap and smart bursty indicator, we can find the wake up burst, and just use nr_running as instant utilization only. The 'sysctl_sched_burst_threshold' used for wakeup burst, set

[patch v6 04/21] sched: add sched balance policies in kernel

2013-03-30 Thread Alex Shi
Current scheduler behavior is just consider for larger performance of system. So it try to spread tasks on more cpu sockets and cpu cores To adding the consideration of power awareness, the patchset adds a powersaving scheduler policy. It will use runnable load util in scheduler balancing. The cur

[patch v6 10/21] sched: get rq potential maximum utilization

2013-03-30 Thread Alex Shi
Since the rt task priority is higher than fair tasks, cfs_rq utilization is just the left of rt utilization. When there are some cfs tasks in queue, the potential utilization may be yielded, so mulitiplying cfs task number to get max potential utilization of cfs. Then the rq utilization is sum of

[patch v6 02/21] sched: set initial value of runnable avg for new forked task

2013-03-30 Thread Alex Shi
We need initialize the se.avg.{decay_count, load_avg_contrib} for a new forked task. Otherwise random values of above variables cause mess when do new task enqueue: enqueue_task_fair enqueue_entity enqueue_entity_load_avg and make forking balancing imbalance since incorrect

[patch v6 09/21] sched: scale_rt_power rename and meaning change

2013-03-30 Thread Alex Shi
The scale_rt_power() used to represent the left CPU utilization after reduce rt utilization. so named it as scale_rt_power has a bit inappropriate. Since we need to use the rt utilization in some incoming patches, we just change return value of this function to rt utilization, and rename it as sca

[patch v6 05/21] sched: add sysfs interface for sched_balance_policy selection

2013-03-30 Thread Alex Shi
This patch add the power aware scheduler knob into sysfs: $cat /sys/devices/system/cpu/sched_balance_policy/available_sched_balance_policy performance powersaving $cat /sys/devices/system/cpu/sched_balance_policy/current_sched_balance_policy powersaving This means the using sched balance policy i

[RFC] gameport: limit the PDPI L4 driver to X86.

2013-03-30 Thread Diego Elio Pettenò
This driver, at least on ARM, can cause the kernel to fail with a pointer dereference to address 0x210 (which it tries to outb to). Signed-off-by: Diego Elio Pettenò --- drivers/input/gameport/Kconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/input/gameport/Kconfig b/drivers/in

Re: [PATCH 1/6] i2c: Remove detach_adapter

2013-03-30 Thread Jean Delvare
Hi Lars, On Sat, 9 Mar 2013 19:16:44 +0100, Lars-Peter Clausen wrote: > The detach_adapter callback has been deprecated for quite some time and has no > user left. Keeping it alive blocks other cleanups, so remove it. I'm all for it. Originally I intended to remove both attach_adapter and detach

Re: gpio-ucb1400

2013-03-30 Thread Guenter Roeck
On Fri, Mar 29, 2013 at 08:46:39PM +0100, Jean Delvare wrote: > Hi all, > > In September 2009, a driver for the GPIO function of the UCB1400 chip > was added to the kernel tree. The probe function of this driver requires > ucbdata to be set. The only place where this happens is in function > ucb14

Re: [PATCH 1/1] TTY Fix NULL pointer error in tty_wakeup

2013-03-30 Thread Greg Kroah-Hartman
On Sat, Mar 30, 2013 at 03:58:43PM +0800, Frankie Lin wrote: > When calling tty_wakeup and the parameter is NULL,the oops will happen. Who is calling that function with NULL? Shouldn't that be fixed instead? thanks, greg k-h -- To unsubscribe from this list: send the line "unsubscribe linux-ker

Re: [PATCH] Fix IXP4xx coherent allocations

2013-03-30 Thread Russell King - ARM Linux
On Sat, Mar 30, 2013 at 03:22:46PM +0100, Krzysztof Halasa wrote: > Russell King - ARM Linux writes: > > > I'm having a hard time understanding what is an ARM issue here, what is > > an ARM bug, and what the DMA API requires. The DMA API documentation > > is extremely sparse in describing what's

Re: [patch v5 14/15] sched: power aware load balance

2013-03-30 Thread Preeti U Murthy
Hi, On 03/30/2013 07:34 PM, Alex Shi wrote: > On 03/30/2013 07:25 PM, Preeti U Murthy wrote: I still give the rq->util weight even the nr_running is 0, because some transitory tasks may actived on the cpu, but just missed on balancing point. I just wondering that forgetti

[RFT][PATCH 1/2] pwm: lpc32xx: Properly set PWM_ENABLE bit in lpc32xx_pwm_[enable|disable]

2013-03-30 Thread Axel Lin
According to the LPC32x0 User Manual [1]: PWMx Control Registers: BIT 31 (PWMx_EN): 0 = PWM disabled. (Default) 1 = PWM enabled In lpc32xx_pwm_enable(), we should set PWM_ENABLE bit. In lpc32xx_pwm_disable(), we should just clear PWM_ENABLE bit rather than write 0 to the register which will also

[RFT][PATCH 2/2] pwm: lpc32xx: Don't change PWM_ENABLE bit in lpc32xx_pwm_config

2013-03-30 Thread Axel Lin
lpc32xx_pwm_config() is supposed to set duty_ns and period_ns, it should not change PWM_ENABLE bit. Signed-off-by: Axel Lin --- drivers/pwm/pwm-lpc32xx.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/pwm/pwm-lpc32xx.c b/drivers/pwm/pwm-lpc32xx.c index 1a5075e..ac

Re: [PATCH 1/3] kthread: implement probe_kthread_data()

2013-03-30 Thread Tejun Heo
Hello, Oleg. On Sat, Mar 30, 2013 at 7:36 AM, Oleg Nesterov wrote: > > +void *probe_kthread_data(struct task_struct *task) > > +{ > > + struct kthread *kthread = to_kthread(task); > > + void *data = NULL; > > + > > + probe_kernel_read(&data, &kthread->data, sizeof(data)); > > + re

[PATCH] [trivial]treewide: Fix typos in printk

2013-03-30 Thread Masanari Iida
Correct spelling typos in various part of printk. Signed-off-by: Masanari Iida --- drivers/gpu/drm/i915/intel_dp.c | 2 +- drivers/i2c/busses/i2c-puv3.c | 2 +- drivers/media/platform/s5p-mfc/s5p_mfc_opr_v6.c | 2 +- drivers/net/ethernet/atheros/atl1c/atl1c_hw.c

[RFT][PATCH v2 2/2] pwm: lpc32xx: Don't change PWM_ENABLE bit in lpc32xx_pwm_config

2013-03-30 Thread Axel Lin
lpc32xx_pwm_config() is supposed to set duty_ns and period_ns, it should not change PWM_ENABLE and other bits. Signed-off-by: Axel Lin --- drivers/pwm/pwm-lpc32xx.c |7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/drivers/pwm/pwm-lpc32xx.c b/drivers/pwm/pwm-lpc32xx.c

Re: PNP: List Rafael Wysocki as a maintainer

2013-03-30 Thread Bjorn Helgaas
On Fri, Mar 29, 2013 at 3:59 PM, Rafael J. Wysocki wrote: > From: Rafael J. Wysocki > > The Adam Belay's e-mail address in MAINTAINERS under PNP SUPPORT > is not valid any more and I started to maintain that code in the > meantime as a matter of fact, so list myself as a maintainer of it > along

Re: [PATCH 1/3] kthread: implement probe_kthread_data()

2013-03-30 Thread Oleg Nesterov
Hi Tejun, On 03/30, Tejun Heo wrote: > > On Sat, Mar 30, 2013 at 7:36 AM, Oleg Nesterov wrote: > > > +void *probe_kthread_data(struct task_struct *task) > > > +{ > > > + struct kthread *kthread = to_kthread(task); > > > + void *data = NULL; > > > + > > > + probe_kernel_read(&data, &kt

[PATCH 2/2 tj/for-3.10] workqueue: modify wq->freezing only when freezable

2013-03-30 Thread Lai Jiangshan
simplify pwq_adjust_max_active(). make freeze_workqueues_begin() and thaw_workqueues() fast skip non-freezable wq. Signed-off-by: Lai Jiangshan --- kernel/workqueue.c | 16 +--- 1 files changed, 9 insertions(+), 7 deletions(-) diff --git a/kernel/workqueue.c b/kernel/workqueue.c i

[PATCH 1/2 tj/for-3.10] workqueue: add wq->freezing and remove POOL_FREEZING

2013-03-30 Thread Lai Jiangshan
freezing is nothing related to pools, but POOL_FREEZING adds a connection, and causes freeze_workqueues_begin() and thaw_workqueues() complicated. Since freezing is workqueue instance attribute, so we introduce wq->freezing instead and remove POOL_FREEZING. Signed-off-by: Lai Jiangshan --- kern

[PATCH] net/wireless/sme: sched_scan_mtx lock in cfg80211_conn_work()

2013-03-30 Thread Artem Savkov
Introduced in f9f475292dbb0e7035fb6661d1524761ea0888d9 cfg80211_conn_scan() which requires sched_scan_mtx to be held can be called from cfg80211_conn_work(). Without this we are hitting multiple warnings like the following: [ 51.996440] [ cut here ] [ 51.996484] WARNIN

Re: [PATCH v4 13/14] workqueue: implement NUMA affinity for unbound workqueues

2013-03-30 Thread Lai Jiangshan
On 31/03/13 00:32, Tejun Heo wrote: > Hello, Lai. > > > On Sat, Mar 30, 2013 at 9:13 AM, Lai Jiangshan > wrote: > > > + /* all pwqs have been created successfully, let's install'em > */ > mutex_lock(&wq->mutex); > > copy

  1   2   >