Re: [PATCH 1/2] can: xilinx: use readl/writel instead of ioread/iowrite

2015-10-22 Thread Arnd Bergmann
On Thursday 22 October 2015 08:34:53 Appana Durga Kedareswara Rao wrote: > > On Thursday 22 October 2015 10:16:02 Kedareswara rao Appana wrote: > > > The driver only supports memory-mapped I/O [by ioremap()], so > > > readl/writel is actually the right thing to do, IMO. > > > During the validation

Re: [PATCH] regmap: rbtree: When adding a reg do a bsearch for target node

2015-10-22 Thread Charles Keepax
On Wed, Oct 21, 2015 at 02:16:14PM +0100, Nikesh Oswal wrote: > From: Nikesh Oswal If you are going to use your non-opensource email, might as well use the cirrus one here. > > A binary search is much more efficient rather than iterating > over the rbtree in ascending order which the current

Re: [PATCH block/for-4.4/core] blkcg: don't create "io.stat" on the root cgroup

2015-10-22 Thread Tejun Heo
On Wed, Oct 21, 2015 at 10:07:44PM -0600, Jens Axboe wrote: > On 10/21/2015 06:31 PM, Tejun Heo wrote: > >Hello, Jens. > > > >This is one of hopefully final adjustments to cgroup v2 behavior > >before lifting the devel mask. If the patch is okay, can I route the > >patch through cgroup/for-4.4 so

Re: kernel oops on mmotm-2015-10-15-15-20

2015-10-22 Thread Minchan Kim
On Thu, Oct 22, 2015 at 10:21:36AM +0900, Minchan Kim wrote: > Hello Hugh, > > On Wed, Oct 21, 2015 at 05:59:59PM -0700, Hugh Dickins wrote: > > On Thu, 22 Oct 2015, Minchan Kim wrote: > > > > > > I added the code to check it and queued it again but I had another oops > > > in this time but

Re: [PATCH v8 1/5] configfs: Allow dynamic group creation

2015-10-22 Thread Daniel Baluta
On Thu, Oct 22, 2015 at 11:40 AM, Christoph Hellwig wrote: > On Tue, Oct 20, 2015 at 01:53:42PM +0300, Daniel Baluta wrote: >> We don't want to hardcode default groups at subsystem >> creation time. We export: >> * configfs_register_group >> * configfs_unregister_group >> to allow

Re: [PATCH 1/2] can: xilinx: use readl/writel instead of ioread/iowrite

2015-10-22 Thread Arnd Bergmann
On Thursday 22 October 2015 10:21:58 Marc Kleine-Budde wrote: > On 10/22/2015 10:14 AM, Arnd Bergmann wrote: > > On Thursday 22 October 2015 10:16:02 Kedareswara rao Appana wrote: > >> The driver only supports memory-mapped I/O [by ioremap()], > >> so readl/writel is actually the right thing to

Re: [PATCH 5/6] ARM: dts: sunxi: Add Allwinner H3 DTSI

2015-10-22 Thread Jean-Francois Moine
On Thu, 22 Oct 2015 10:47:35 +0200 Maxime Ripard wrote: > Not really. The uart0 reset is the bit 16, in the reset register 4. > > 4 * 32 + 16 = 44. > > Not 112, but still not 208 either. The registers are numbered 1..5, then (4 - 1) * 32 + 16 = 112 -- Ken ar c'hentañ | **

Re: [PATCH] libata: add support for NCQ commands for SG interface

2015-10-22 Thread Tejun Heo
Hello, On Wed, Oct 21, 2015 at 10:39:32AM +0530, Vinayak Kale wrote: ... > > diff --git a/drivers/ata/libata-scsi.c b/drivers/ata/libata-scsi.c > > index 0d7f0da..5b0a5ab 100644 > > --- a/drivers/ata/libata-scsi.c > > +++ b/drivers/ata/libata-scsi.c > > @@ -2914,12 +2914,14 @@

Re: [linux-sunxi] Re: [PATCH 0/6] ARM: sunxi: Introduce Allwinner H3 support

2015-10-22 Thread Hans de Goede
Hi, On 22-10-15 09:49, Jens Kuske wrote: Hi, On 21/10/15 21:18, Hans de Goede wrote: Great to see that you've started working on this again. Last weekend I ended up working on this too together with Reinder E.N. de Haan (added to the Cc). We took a slightly different approach for the

Re: blk-mq: takes hours for scsi scanning finish when thousands of LUNs

2015-10-22 Thread Tejun Heo
Hello, On Mon, Oct 19, 2015 at 07:40:13AM -0700, Zhangqing Luo wrote: > So every time blk_mq_freeze_queue_start, it runs in this way > > blk_mq_freeze_queue_start > ->percpu_ref_kill->percpu_ref_kill_and_confirm > ->__percpu_ref_switch_to_atomic >

Re: [PATCH 5/6] ARM: dts: sunxi: Add Allwinner H3 DTSI

2015-10-22 Thread Maxime Ripard
On Thu, Oct 22, 2015 at 10:29:59AM +0200, Jean-Francois Moine wrote: > On Thu, 22 Oct 2015 10:05:08 +0200 > Maxime Ripard wrote: > > > > + uart0: serial@01c28000 { > > > + compatible = "snps,dw-apb-uart"; > > > + reg = <0x01c28000 0x400>; > > > +

Re: [PATCH 13/16] perf callchain: Switch default to 'graph,0.5,caller'

2015-10-22 Thread Ingo Molnar
* Arnaldo Carvalho de Melo wrote: > > 5) --no-children > > > > I agree that 'perf top -g --no-children' looks more intuitive than 'perf > > top -g'. > > So, what do you propose, to switch back the default to --no-children, for > both > tools, top and report? Now that I am getting used to

[PATCH 1/3] net: phy: Add nested variants of mdiobus read/write

2015-10-22 Thread Neil Armstrong
Since nested variants of mdiobus_read/write are used in multiple drivers, add nested variants in the mdiobus core. Suggested-by: Andrew Lunn Signed-off-by: Neil Armstrong --- drivers/net/phy/mdio_bus.c | 55 ++ include/linux/phy.h| 2 ++ 2

[PATCH 0/3] Refactor nested mdiobus read/write functions

2015-10-22 Thread Neil Armstrong
In order to avoid locked signal false positive for nested mdiobus read/write calls, nested code was introduced in mv88e6xxx and mdio-mux. But mv88e6060 also needs such nested mdiobus read/write calls. For sake of refactoring, introduce nested variants of mdiobus read/write and make them used by

RE: [PATCH 1/2] can: xilinx: use readl/writel instead of ioread/iowrite

2015-10-22 Thread Appana Durga Kedareswara Rao
Hi Marc, > -Original Message- > From: Marc Kleine-Budde [mailto:m...@pengutronix.de] > Sent: Thursday, October 22, 2015 1:52 PM > To: Arnd Bergmann; linux-arm-ker...@lists.infradead.org > Cc: Appana Durga Kedareswara Rao; Anirudha Sarangi; w...@grandegger.com; > Michal Simek; Soren

Re: [PATCH v8 1/5] configfs: Allow dynamic group creation

2015-10-22 Thread Christoph Hellwig
On Tue, Oct 20, 2015 at 01:53:42PM +0300, Daniel Baluta wrote: > We don't want to hardcode default groups at subsystem > creation time. We export: > * configfs_register_group > * configfs_unregister_group > to allow drivers to programatically create/destroy groups > later, after module

[PATCH 3/3] net: dsa: Make mv88e6060 use nested mdiobus read/write

2015-10-22 Thread Neil Armstrong
Like mv88e6xxx and mdio-mux, to avoid lockdep give false positives because of nested MDIO busses, switch to previously introduced nested mdiobus_read/write variants. Signed-off-by: Neil Armstrong --- drivers/net/dsa/mv88e6060.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff

[PATCH] serial: 8250_uniphier: add earlycon support

2015-10-22 Thread Masahiro Yamada
This reuses the code of drivers/tty/serial/8250/8250_early.c except - Overwrite device->port.iotype and device->port.regshift for UPIO_MEM32 because of_setup_earlycon() has set them for UPIO_MEM. - Set device->baud to zero to prevent early8250_setup() from initializing the divisor

[PATCH 2/3] net: dsa: Make mv88e6xxx use nested mdiobus read/write

2015-10-22 Thread Neil Armstrong
Make the mv88e6xxx driver use the previously introduced nested variants of mdiobus_read/write functions. Signed-off-by: Neil Armstrong --- drivers/net/dsa/mv88e6xxx.c | 46 + 1 file changed, 9 insertions(+), 37 deletions(-) diff --git

Re: [PATCH net-next RFC 1/2] vhost: introduce vhost_has_work()

2015-10-22 Thread Michael S. Tsirkin
On Thu, Oct 22, 2015 at 01:27:28AM -0400, Jason Wang wrote: > This path introduces a helper which can give a hint for whether or not > there's a work queued in the work list. > > Signed-off-by: Jason Wang > --- > drivers/vhost/vhost.c | 6 ++ > drivers/vhost/vhost.h | 1 + > 2 files

Re: [PATCH] clk: Remove clk_{register,unregister}_multiplier()

2015-10-22 Thread Maxime Ripard
Hi! On Wed, Oct 21, 2015 at 04:33:53PM -0700, Stephen Boyd wrote: > These APIs aren't used, so remove them. This can be reverted if > we get a user at some point. > > Cc: Maxime Ripard > Suggested-by: Michael Turquette > Signed-off-by: Stephen Boyd Reviewed-by: Maxime Ripard Thanks! Maxime

Re: [PATCH] mmc: pwrseq: Use highest priority for eMMC restart handler

2015-10-22 Thread Javier Martinez Canillas
Hello Anand, On 10/22/2015 07:03 AM, Anand Moon wrote: > Hi Javier, > > On 22 October 2015 at 08:22, Javier Martinez Canillas > wrote: >> Hello Krzysztof, >> >> On 10/22/2015 03:43 AM, Krzysztof Kozlowski wrote: >>> On 22.10.2015 10:20, Javier Martinez Canillas wrote:> Hello Krzysztof,

RE: [PATCH 1/2] can: xilinx: use readl/writel instead of ioread/iowrite

2015-10-22 Thread Appana Durga Kedareswara Rao
Hi Arnd, > -Original Message- > From: Arnd Bergmann [mailto:a...@arndb.de] > Sent: Thursday, October 22, 2015 1:45 PM > To: linux-arm-ker...@lists.infradead.org > Cc: Appana Durga Kedareswara Rao; Anirudha Sarangi; w...@grandegger.com; > m...@pengutronix.de; Michal Simek; Soren

[PATCH] mcb: Destroy IDA on module unload

2015-10-22 Thread Johannes Thumshirn
Destroy mcb_ida on module_unload Signed-off-by: Johannes Thumshirn --- drivers/mcb/mcb-core.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/mcb/mcb-core.c b/drivers/mcb/mcb-core.c index 9018ab8..a4be451 100644 --- a/drivers/mcb/mcb-core.c +++ b/drivers/mcb/mcb-core.c @@ -409,6

Re: [PATCH] efi: Fix warning of int-to-pointer-cast on x86 32-bit builds

2015-10-22 Thread Ard Biesheuvel
> On 22 okt. 2015, at 19:28, Taku Izumi wrote: > > commit-0f96a99 introduces the following warning message: > > drivers/firmware/efi/fake_mem.c:186:20: warning: cast to pointer > from integer of different size [-Wint-to-pointer-cast] > > new_memmap_phy was defined as a u64 value and casted

RE: [PATCH 1/2] dma: Add Freescale qDMA engine driver support

2015-10-22 Thread Yao Yuan
Hi Vinod, Thanks for your review, please see my comments inline. Best Regards, Yuan Yao > -Original Message- > From: Vinod Koul [mailto:vinod.k...@intel.com] > Sent: Monday, October 05, 2015 10:37 PM > To: Yuan Yao-B46683 > Cc: shawn@linaro.org; dan.j.willi...@intel.com; >

Re: [PATCH 3/6] pinctrl: sunxi: Add H3 PIO controller support

2015-10-22 Thread Maxime Ripard
Hi, On Thu, Oct 22, 2015 at 01:21:26AM -0700, josef.gajdu...@gmail.com wrote: > Hi everyone, > > I might be missing something, but why is the PL* GPIO bank not > declared here? Because it's in the PRCM. This one was adding support for the main port controller. Maxime -- Maxime Ripard, Free

Re: [PATCH 5/6] ARM: dts: sunxi: Add Allwinner H3 DTSI

2015-10-22 Thread Jean-Francois Moine
On Thu, 22 Oct 2015 10:05:08 +0200 Maxime Ripard wrote: > > + uart0: serial@01c28000 { > > + compatible = "snps,dw-apb-uart"; > > + reg = <0x01c28000 0x400>; > > + interrupts = ; > > + reg-shift = <2>; > > +

[PATCH] efi: Fix warning of int-to-pointer-cast on x86 32-bit builds

2015-10-22 Thread Taku Izumi
commit-0f96a99 introduces the following warning message: drivers/firmware/efi/fake_mem.c:186:20: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast] new_memmap_phy was defined as a u64 value and casted to void*. This causes a warning of int-to-pointer-cast on x86

Re: [PATCHv12 14/37] futex, thp: remove special case for THP in get_futex_key

2015-10-22 Thread Artem Savkov
d page futexes with the following panic: [ 33.465456] general protection fault: [#1] SMP [ 33.465991] CPU: 1 PID: 523 Comm: tst Not tainted 4.3.0-rc6-next-20151022 #139 [ 33.466585] Hardware name: QEMU Standard PC (i440FX + PIIX, 1996), BIOS 1.7.5-20140709_153950- 04/01/2014 [

Re: [PATCH 2/3] perf top: Support call-graph display options also

2015-10-22 Thread Ingo Molnar
* Namhyung Kim wrote: > Currently 'perf top --call-graph' option is same as 'perf record'. But > 'perf top' also need to receive display options in 'perf report'. To do > that, change parse_callchain_report_opt() to allow record options too. > > Now perf top can receive display options like

Re: [PATCH 1/2] can: xilinx: use readl/writel instead of ioread/iowrite

2015-10-22 Thread Marc Kleine-Budde
On 10/22/2015 10:14 AM, Arnd Bergmann wrote: > On Thursday 22 October 2015 10:16:02 Kedareswara rao Appana wrote: >> The driver only supports memory-mapped I/O [by ioremap()], >> so readl/writel is actually the right thing to do, IMO. >> During the validation of this driver or IP on ARM 64-bit

[PATCH] serial: 8250_early: do not save and restore IER in write callback

2015-10-22 Thread Masahiro Yamada
The IER has already been masked in early_serial8250_setup(), there is no reason to save and restore it every time early_serial8250_write() is called. Let wait_for_xmitr() melt into serial_putc(). Signed-off-by: Masahiro Yamada --- drivers/tty/serial/8250/8250_early.c | 20 ++--

Re: [PATCH 3/6] pinctrl: sunxi: Add H3 PIO controller support

2015-10-22 Thread Maxime Ripard
On Wed, Oct 21, 2015 at 06:30:46PM +0200, Jens Kuske wrote: > The H3 uses the same pin controller as previous SoC's from Allwinner. > Add support for the pins controlled by the main PIO controller. > > Signed-off-by: Jens Kuske Acked-by: Maxime Ripard Thanks! Maxime -- Maxime Ripard, Free

Re: [PATCH 1/2] can: xilinx: use readl/writel instead of ioread/iowrite

2015-10-22 Thread Arnd Bergmann
On Thursday 22 October 2015 10:16:02 Kedareswara rao Appana wrote: > The driver only supports memory-mapped I/O [by ioremap()], > so readl/writel is actually the right thing to do, IMO. > During the validation of this driver or IP on ARM 64-bit processor > while sending lot of packets observed

Re: [PATCH] net: dsa: mv88e6060: Fix false positive lockdep splat

2015-10-22 Thread Neil Armstrong
On 10/21/2015 06:14 PM, Andrew Lunn wrote: > On Wed, Oct 21, 2015 at 05:37:45PM +0200, Neil Armstrong wrote: >> Like the change made for mv88e6xxx, use mutex_lock_nested() to avoid >> lockdep to give false positives because of nested MDIO busses. > > Hi Neil > > We now have three instances of

Re: [PATCH] net: dsa: mv88e6060: Fix false positive lockdep splat

2015-10-22 Thread Neil Armstrong
On 10/21/2015 06:14 PM, Andrew Lunn wrote: > On Wed, Oct 21, 2015 at 05:37:45PM +0200, Neil Armstrong wrote: >> Like the change made for mv88e6xxx, use mutex_lock_nested() to avoid >> lockdep to give false positives because of nested MDIO busses. > > Hi Neil > > We now have three instances of

[PATCH] pstore: fix return type of pstore_is_mounted

2015-10-22 Thread Geliang Tang
This patch changes return type of pstore_is_mounted from int to bool. Signed-off-by: Geliang Tang --- fs/pstore/inode.c| 2 +- fs/pstore/internal.h | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/fs/pstore/inode.c b/fs/pstore/inode.c index 3586491..d8c439d 100644 ---

Re: [PATCH 26/31] perf tools: Support perf event alias name

2015-10-22 Thread Wangnan (F)
On 2015/10/22 15:53, Namhyung Kim wrote: On Thu, Oct 22, 2015 at 4:29 PM, Wangnan (F) wrote: [SNIP] I understand the need of using aliases but I think it's more natural to use event name for simple cases.. I will consider this. However, if we allow using event name directly like this:

Re: [PATCH 5/6] ARM: dts: sunxi: Add Allwinner H3 DTSI

2015-10-22 Thread Maxime Ripard
Hi, On Wed, Oct 21, 2015 at 06:20:27PM +0200, Jens Kuske wrote: > The Allwinner H3 is a home entertainment system oriented SoC with > four Cortex-A7 cores and a Mali-400MP2 GPU. > > Signed-off-by: Jens Kuske > --- > arch/arm/boot/dts/sun8i-h3.dtsi | 499 >

Re: [PATCH 19/57] perf tools: Add attr_update event

2015-10-22 Thread Jiri Olsa
On Mon, Oct 19, 2015 at 06:00:04PM -0300, Arnaldo Carvalho de Melo wrote: > Em Fri, Oct 16, 2015 at 12:40:54PM +0200, Jiri Olsa escreveu: > > It'll server as base event for additional event attributes > > details, that are not part of the attr event. > > > > At the moment this event is just dummy

[PATCH v2] wait: add comment before waitqueue_active noting memory barrier is required

2015-10-22 Thread Kosuke Tatsukawa
This patch adds a comment before waitqueue_active noting that memory barriers are required. In the following code, the wake_up thread might fail to wake up the waiting thread and leave it sleeping due to lack of memory barriers. wake_up thread waiting thread

Re: [PATCH 1/3] perf tools: Move callchain help messages to callchain.h

2015-10-22 Thread Ingo Molnar
* Namhyung Kim wrote: > +#define CALLCHAIN_HELP "setup and enables call-graph (stack chain/backtrace) > recording: " > + > +#ifdef HAVE_DWARF_UNWIND_SUPPORT > +#define CALLCHAIN_RECORD_HELP CALLCHAIN_HELP "fp dwarf lbr" > +#else > +#define CALLCHAIN_RECORD_HELP CALLCHAIN_HELP "fp lbr" >

Re: [PATCH 0/5] Fix regression introduced by set_irq_flags() removal

2015-10-22 Thread Thomas Gleixner
On Tue, 20 Oct 2015, Thomas Gleixner wrote: > On Tue, 20 Oct 2015, Russell King - ARM Linux wrote: > > I think Rob's patches need another review in light of this, to determine > > how much breakage there is here, and a decision how to proceed made on > > that basis. > > I'll go over them tomorrow

Re: [PATCH] wait: add comment before waitqueue_active noting memory barrier is required

2015-10-22 Thread Kosuke Tatsukawa
Peter Zijlstra wrote: > On Fri, Oct 09, 2015 at 12:35:59AM +, Kosuke Tatsukawa wrote: >> This patch adds a comment before waitqueue_active noting that a memory >> barrier is required. >> >> Besides the original problem in drivers/tty/n_tty.c which caused a >> program stall (described in

Re: [PATCH 0/6] ARM: sunxi: Introduce Allwinner H3 support

2015-10-22 Thread Jean-Francois Moine
On Wed, 21 Oct 2015 21:18:45 +0200 Hans de Goede wrote: > Great to see that you've started working on this again. Last weekend I > ended up working on this too together with Reinder E.N. de Haan > > (added to the Cc). > > We took a slightly different approach for the gates clocks, see: > >

Re: [PATCH v11 02/14] HMM: add special swap filetype for memory migrated to device v2.

2015-10-22 Thread Hillf Danton
> > When migrating anonymous memory from system memory to device memory > CPU pte are replaced with special HMM swap entry so that page fault, > get user page (gup), fork, ... are properly redirected to HMM helpers. > > This patch only add the new swap type entry and hooks HMM helpers >

Re: [PATCH 4/6] reset: sunxi: Add compatible for Allwinner H3 bus resets

2015-10-22 Thread Maxime Ripard
On Wed, Oct 21, 2015 at 06:20:26PM +0200, Jens Kuske wrote: > Adding a new compatible allows us to define SoC specific behaviour > if necessary, for example forcing a particular device out of reset > even if no driver is actually using it. > > Signed-off-by: Jens Kuske Acked-by: Maxime Ripard

Re: [PATCH 05/57] perf tools: Add cpu_map event

2015-10-22 Thread Jiri Olsa
On Mon, Oct 19, 2015 at 05:45:02PM -0300, Arnaldo Carvalho de Melo wrote: > Em Fri, Oct 16, 2015 at 12:40:40PM +0200, Jiri Olsa escreveu: > > Adding cpu_map event to pass/store cpu maps > > as data in pipe/perf.data. > > > > We store maps in 2 formats: > > - list of cpus > > - mask of cpus >

Re: [PATCH V5 1/1] bpf: control events stored in PERF_EVENT_ARRAY maps trace data output when perf sampling

2015-10-22 Thread Wangnan (F)
On 2015/10/22 15:39, Ingo Molnar wrote: * Wangnan (F) wrote: [SNIP] In summary, your either-or logic doesn't hold in BPF world. A BPF program can only access perf event in a highly restricted way. We don't allow it calling perf_event_read_local() across core, so it can't. Urgh, that's

Re: [PATCH 26/31] perf tools: Support perf event alias name

2015-10-22 Thread Namhyung Kim
On Thu, Oct 22, 2015 at 4:29 PM, Wangnan (F) wrote: > > > On 2015/10/22 15:16, Namhyung Kim wrote: >> >> Hi, >> >> On Wed, Oct 21, 2015 at 10:00 PM, Wangnan (F) wrote: >>> >>> >>> On 2015/10/21 16:53, Namhyung Kim wrote: Hi, On Wed, Oct 14, 2015 at 12:41:37PM +, Wang Nan

Re: [PATCH v7 6/8] scsi: ufs: make the UFS variant a platform device

2015-10-22 Thread Arnd Bergmann
On Thursday 22 October 2015 07:02:14 subha...@codeaurora.org wrote: > > > > Required properties: > > -- compatible: compatible list, contains "jedec,ufs-1.1" > > +- compatible: compatible list, contains "jedec,ufs-1.1" or > > + "qcom,msm8994-ufshc" or

Re: [linux-sunxi] [PATCH 2/6] clk: sunxi: Add H3 clocks support

2015-10-22 Thread Jens Kuske
On 22/10/15 02:15, Julian Calaby wrote: > > This hunk should be in patch 1: Indeed, Thanks. Jens > >> @@ -1000,9 +1005,8 @@ static void __init sunxi_divs_clk_setup(struct >> device_node *node, >> >> for (i = 0; i < SUNXI_DIVS_BASE_NAME_MAX_LEN - 1 && >>

Re: [PATCH 0/6] ARM: sunxi: Introduce Allwinner H3 support

2015-10-22 Thread Jens Kuske
Hi, On 21/10/15 21:18, Hans de Goede wrote: > > Great to see that you've started working on this again. Last weekend I > ended up working on this too together with Reinder E.N. de Haan > > (added to the Cc). > > We took a slightly different approach for the gates clocks, see: > >

Re: [PATCH v3 1/1] eventfd: implementation of EFD_MASK flag

2015-10-22 Thread Damian Hobson-Garcia
Hello, > diff --git a/include/linux/eventfd.h b/include/linux/eventfd.h > index ff0b981..87de343 100644 > --- a/include/linux/eventfd.h > +++ b/include/linux/eventfd.h > > -/* > - * CAREFUL: Check include/uapi/asm-generic/fcntl.h when defining > - * new flags, since they might collide with O_*

[RFC/PATCH RESEND 3/3] perf tools: Defaults to 'caller' callchain order only if --children is enabled

2015-10-22 Thread Namhyung Kim
The caller callchain order is useful with --children option since it can show 'overview' style output, but other commands which don't use --children feature like 'perf script' or even 'perf report/top' without --children are better to keep callee order. Cc: Adrian Hunter Cc: Borislav Petkov

Re: Problem about CPU stalling in hrtimer_intterrupts()

2015-10-22 Thread Thomas Gleixner
On Thu, 22 Oct 2015, Yang Yingliang wrote: > I use the kernel-4.1.6 running on arm64. > My testcase is that it calls clock_settime and clock_adjtime alternately with > random params on each core. My system has 32 cores. > > I found the cpu stalling in hrtimer_intterrupts(). So I added some debug

Re: [PATCH V5 1/1] bpf: control events stored in PERF_EVENT_ARRAY maps trace data output when perf sampling

2015-10-22 Thread Ingo Molnar
* Wangnan (F) wrote: > > > On 2015/10/22 0:57, Peter Zijlstra wrote: > >On Wed, Oct 21, 2015 at 11:06:47PM +0800, pi3orama wrote: > >>>So explain; how does this eBPF stuff work. > >>I think I get your point this time, and let me explain the eBPF stuff to > >>you. > >> > >>You are aware that

Re: [PATCH v2 net-next] bpf: fix bpf_perf_event_read() helper

2015-10-22 Thread Wangnan (F)
On 2015/10/22 14:21, Alexei Starovoitov wrote: On 10/21/15 10:31 PM, Wangnan (F) wrote: +if ((attr->type != PERF_TYPE_RAW && + !(attr->type == PERF_TYPE_SOFTWARE && + attr->config == PERF_COUNT_SW_BPF_OUTPUT) && + attr->type != PERF_TYPE_HARDWARE) || +

Re: [RFC/PATCH 3/3] perf tools: Defaults to 'caller' callchain order only if --children is enabled

2015-10-22 Thread Namhyung Kim
Hi Ingo, On Thu, Oct 22, 2015 at 4:32 PM, Ingo Molnar wrote: > > * Namhyung Kim wrote: > >> The caller callchain order is useful with --children option since it can >> show 'overview' style output, but other commands which don't use >> --children feature like 'perf script' or even 'perf

Re: [GIT PULL 0/5] perf/core improvements and fixes

2015-10-22 Thread Ingo Molnar
* Arnaldo Carvalho de Melo wrote: > Hi Ingo, > > Please consider pulling, > > - Arnaldo > > The following changes since commit 43e41adc9e8c36545888d78fed2ef8d102a938dc: > > perf record: Add ability to sample call branches (2015-10-20 10:30:55 +0200) > > are available in the git

Re: [PATCH] net: dsa: mv88e6060: Fix false positive lockdep splat

2015-10-22 Thread Neil Armstrong
Hi Andrew, On 10/21/2015 06:14 PM, Andrew Lunn wrote: > On Wed, Oct 21, 2015 at 05:37:45PM +0200, Neil Armstrong wrote: >> Like the change made for mv88e6xxx, use mutex_lock_nested() to avoid >> lockdep to give false positives because of nested MDIO busses. > > Hi Neil > > We now have three

Re: [RFC/PATCH 3/3] perf tools: Defaults to 'caller' callchain order only if --children is enabled

2015-10-22 Thread Ingo Molnar
* Namhyung Kim wrote: > The caller callchain order is useful with --children option since it can > show 'overview' style output, but other commands which don't use > --children feature like 'perf script' or even 'perf report/top' without > --children are better to keep caller order. > > Cc:

[PATCH] sched/deadline: fix earliest_dl.next is not the pushable candidate

2015-10-22 Thread Wanpeng Li
earliest_dl.next is used to cache the next earliest ready task which also is pushable in order to be a candidate of pushable tasks during pull algorithm. If the earliest_dl.next deadline of the sr_rq is earlier than the earliest_dl.curr deadline of current rq, the task from the sr_rq can be

Re: [PATCH 26/31] perf tools: Support perf event alias name

2015-10-22 Thread Wangnan (F)
On 2015/10/22 15:16, Namhyung Kim wrote: Hi, On Wed, Oct 21, 2015 at 10:00 PM, Wangnan (F) wrote: On 2015/10/21 16:53, Namhyung Kim wrote: Hi, On Wed, Oct 14, 2015 at 12:41:37PM +, Wang Nan wrote: From: He Kuang This patch adds new bison rules for specifying an alias name to a

Re: [PATCH] mcb: Do not return zero on error path in mcb_pci_probe()

2015-10-22 Thread Johannes Thumshirn
On Sat, Oct 17, 2015 at 11:33 PM, Alexey Khoroshilov wrote: > There is an error path in mcb_pci_probe() where > it returns zero instead of error code. > > Found by Linux Driver Verification project (linuxtesting.org). > > Signed-off-by: Alexey Khoroshilov > --- > drivers/mcb/mcb-pci.c | 1 + >

RE: [PATCH v11 1/6] PCI: designware: move calculation of bus addresses to DRA7xx

2015-10-22 Thread Gabriele Paoloni
> -Original Message- > From: Bjorn Helgaas [mailto:helg...@kernel.org] > Sent: 21 October 2015 23:15 > To: Wangzhou (B) > Cc: Bjorn Helgaas; jingooh...@gmail.com; pratyush.an...@gmail.com; Arnd > Bergmann; li...@arm.linux.org.uk; thomas.petazz...@free-electrons.com; > Gabriele Paoloni;

Re: [PATCH] scsi: wd719x: Use module_pci_driver

2015-10-22 Thread Johannes Thumshirn
On Thu, 2015-10-22 at 04:16 +0530, Muhammad Falak R Wani wrote: > Remove boilerplate code by using macro module_pci_driver. > For drivers whose __init and __exit paths only register and > unregister to the pci API, it is preferred to use this macro. > > Signed-off-by: Muhammad Falak R Wani > ---

Re: [PATCH] scsi: wd719x: Remove use of macro DEFINE_PCI_DEVICE_TABLE

2015-10-22 Thread Johannes Thumshirn
On Thu, 2015-10-22 at 03:53 +0530, Muhammad Falak R Wani wrote: > Use struct pci_device_id instead of DEFINE_PCI_DEVICE_TABLE with > the goal of getting rid of this macro completely, as this macro > is deprecated. > > Signed-off-by: Muhammad Falak R Wani > --- >  drivers/scsi/wd719x.c | 2 +- >  

Re: [PATCH v3 04/15] scsi: ufs: clear outstanding_request bit in case query timeout

2015-10-22 Thread subhashj
Looks good to me. Reviewed-by: Subhash Jadavani > When sending a query to the device returns with a timeout error, > we clear the corresponding bit in the DOORBELL register but > we don't clear the outstanding_request field as we should. > This patch fixes this bug. > > Signed-off-by: Yaniv

Re: [PATCH v3 03/15] scsi: ufs: verify command tag validity

2015-10-22 Thread subhashj
Looks good to me. Reviewed-by: Subhash Jadavani > A race condition appear to exist between request completion when > scsi_done() is called to end the request and set the tag back to > -1 (at blk_queue_end_tag() scsi_end_request), and scsi layer error > handling which aborts the command and

Re: [PATCH 26/31] perf tools: Support perf event alias name

2015-10-22 Thread Namhyung Kim
Hi, On Wed, Oct 21, 2015 at 10:00 PM, Wangnan (F) wrote: > > > On 2015/10/21 16:53, Namhyung Kim wrote: >> >> Hi, >> >> On Wed, Oct 14, 2015 at 12:41:37PM +, Wang Nan wrote: >>> >>> From: He Kuang >>> >>> This patch adds new bison rules for specifying an alias name to a perf >>> event,

Re: [PATCH] leds: leds-wrap.c: Use devm_led_classdev_register

2015-10-22 Thread Jacek Anaszewski
Hi Fida, On 10/21/2015 12:58 PM, Fida Mohammad wrote: Use of resource managed function devm_led_classdev_register to make initialization path simpler. Also removed redundant remove function. Signed-off-by: Fida Mohammad --- drivers/leds/leds-wrap.c | 26 -- 1 file

Re: [PATCH v3 02/15] scsi: ufs: clear fields UTRD, UPIU req and rsp before new transfers

2015-10-22 Thread subhashj
Looks good to me. Reviewed-by: Subhash Jadavani > Some of the data structures (like response UPIU) and/or its elements > (unused fields) should be cleared before sending out the respective > command to UFS device. > > This change clears the UPIU response data structure for query commands > and

[PATCH v2 1/6] sched/fair: Generalize the load/util averages resolution definition

2015-10-22 Thread Yuyang Du
Integer metric needs fixed point arithmetic. In sched/fair, a few metrics, e.g., weight, load, load_avg, util_avg, freq, and capacity, may have different fixed point ranges, which makes their update and usage error-prone. In order to avoid the errors relating to the fixed point range, we definie

[PATCH v2 2/6] sched/fair: Remove SCHED_LOAD_SHIFT and SCHED_LOAD_SCALE

2015-10-22 Thread Yuyang Du
After cleaning up the sched metrics, these two definitions that cause ambiguity are not needed any more. Use NICE_0_LOAD_SHIFT and NICE_0_LOAD instead (the names suggest clearly who they are). Suggested-by: Ben Segall Signed-off-by: Yuyang Du --- kernel/sched/fair.c | 4 ++--

[PATCH v2 4/6] sched/fair: Remove scale_load_down() for load_avg

2015-10-22 Thread Yuyang Du
Currently, load_avg = scale_load_down(load) * runnable%. This does not make much sense, because load_avg is primarily the load that takes runnable time ratio into account. We therefore remove scale_load_down() for load_avg. But we need to carefully consider the overflow risk if load has higher

[PATCH v2 0/6] sched/fair: Clean up sched metric definitions

2015-10-22 Thread Yuyang Du
Hi Peter and Ingo, As discussed recently, the sched metrics need a little bit cleanup. This series of patches attempt to do that: refactor, rename, remove... Thanks a lot to Ben, Morten, Dietmar, Vincent, and others who provided valuable comments. v2 changes: - Rename SCHED_RESOLUTION_SHIFT to

[PATCH v2 5/6] sched/fair: Rename scale_load() and scale_load_down()

2015-10-22 Thread Yuyang Du
Rename scale_load() and scale_load_down() to user_to_kernel_load() and kernel_to_user_load() respectively, to allow the names to bear what they are really about. Signed-off-by: Yuyang Du [update calculate_imbalance] Signed-off-by: Vincent Guittot --- kernel/sched/core.c | 8

[PATCH v2 3/6] sched/fair: Add introduction to the sched load avg metrics

2015-10-22 Thread Yuyang Du
These sched metrics have become complex enough. We introduce them at their definition. Signed-off-by: Yuyang Du --- include/linux/sched.h | 60 +-- 1 file changed, 49 insertions(+), 11 deletions(-) diff --git a/include/linux/sched.h

[PATCH v2 6/6] sched/fair: Remove unconditionally inactive code

2015-10-22 Thread Yuyang Du
The increased load resolution (fixed point arithmetic range) is unconditionally deactivated with #if 0. As the increased load range is still used somewhere (e.g., in Google), we want to keep this feature. We define CONFIG_CFS_INCREASE_LOAD_RANGE and it depends on FAIR_GROUP_SCHED and 64BIT and

Re: [PATCH] leds: aat1290: Fix property naming of flash-timeout-us

2015-10-22 Thread Jacek Anaszewski
Hi Ingi, Thanks for catching this, applied. On 10/22/2015 03:14 AM, Ingi Kim wrote: flash-timeout-us property name is changed to flash-max-timeout-us. This patch fixes that name Signed-off-by: Ingi Kim --- Documentation/devicetree/bindings/leds/leds-aat1290.txt | 8 1 file

Re: [PATCH v11 01/14] fork: pass the dst vma to copy_page_range() and its sub-functions.

2015-10-22 Thread Hillf Danton
> > -int copy_page_range(struct mm_struct *dst, struct mm_struct *src, > - struct vm_area_struct *vma); > +int copy_page_range(struct mm_struct *dst_mm, struct mm_struct *src_mm, > + struct vm_area_struct *dst_vma, > + struct vm_area_struct

Re: lockdep-related warning in kernel/sched/deadline.c::find_lock_later_rq()

2015-10-22 Thread Luca Abeni
Hi, On 10/22/2015 07:35 AM, Wanpeng Li wrote: [...] Now, if I understand correctly the issue is that dl_task_timer() does: rq = task_rq_lock(p, ); [...] if (has_pushable_dl_tasks(rq)) push_dl_task(rq); with task_rq_lock() that pins rq->lock and push_tl_task() that invokes

Re: [PATCH v7 8/8] scsi: ufs-qcom: add QUniPro hardware support and power optimizations

2015-10-22 Thread subhashj
Looks good to me. Reviewed-by: Subhash Jadavani > New revisions of UFS host controller supports the new UniPro > hardware controller (referred as QUniPro). This patch adds > the support to enable this new UniPro controller hardware. > > This change also adds power optimization for bus scaling

Re: [PATCH v7 7/8] scsi: ufs-qcom: add debug prints for test bus

2015-10-22 Thread subhashj
Looks good to me. Reviewed-by: Subhash Jadavani > Adds support for configuring and reading the test bus and debug > registers. This change also adds another vops in order to print the > debug registers. > > Signed-off-by: Yaniv Gardi > > --- > drivers/scsi/ufs/ufs-qcom.c | 165 >

Re: [PATCH v7 6/8] scsi: ufs: make the UFS variant a platform device

2015-10-22 Thread subhashj
Comments inline below: > This change turns the UFS variant (SCSI_UFS_QCOM) into a UFS > a platform device. > In order to do so a few additional changes are required: > 1. The ufshcd-pltfrm is no longer serves as a platform device. >Now it only serves as a group of platform APIs such as PM

Re: [PATCH v7 4/8] add ufshcd_get_variant ufshcd_set_variant

2015-10-22 Thread subhashj
Looks good to me. Reviewed-by: Subhash Jadavani > Signed-off-by: Yaniv Gardi > > --- > drivers/scsi/ufs/ufs-qcom.c | 34 +- > drivers/scsi/ufs/ufshcd.h | 21 + > 2 files changed, 38 insertions(+), 17 deletions(-) > > diff --git

Re: [PATCH v7 3/8] scsi: ufs-qcom: update configuration option of SCSI_UFS_QCOM component

2015-10-22 Thread subhashj
Looks good to me. Reviewed-by: Subhash Jadavani > This change is required in order to be able to build the component > as a module. > > Signed-off-by: Yaniv Gardi > > --- > drivers/scsi/ufs/Kconfig | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git

Re: [PATCH v7 5/8] scsi: ufs: creates wrapper functions for vops

2015-10-22 Thread subhashj
Looks good to me. Reviewed-by: Subhash Jadavani > In order to simplify the code a set of wrapper functions is created > to test and call each of the variant operations. > > Signed-off-by: Yaniv Gardi > > --- > drivers/scsi/ufs/ufs-qcom.c | 1 - > drivers/scsi/ufs/ufshcd.c | 104 >

Re: [PATCH v2 05/15] net: wireless: ti: Return flow can be simplified for wl1271_cmd_interrogate

2015-10-22 Thread Eliad Peller
On Wed, Oct 21, 2015 at 10:07 PM, Punit Vara wrote: > Remove int ret suggested by kbuild test robot > > This patch is to the wlcore/acx.c file that fixes up warning > reported by coccicheck: > > WARNING: end returns can be simplified if negative or 0 value > > Prefer direct return value instead

Re: [PATCH v7 2/8] scsi: ufs-qcom: fix compilation warning if compiled as a module

2015-10-22 Thread subhashj
Looks good to me. Reviewed-by: Subhash Jadavani > This change fixes a compilation warning that happens if SCSI_UFS_QCOM > is compiled as a module. > Also this patch fixes an error happens when insmod the module: > "ufs_qcom: module license 'unspecified' taints kernel." > > Signed-off-by: Yaniv

[PATCH 2/4] crypto: hifn_795x: remove the hifn_test function

2015-10-22 Thread LABBE Corentin
The hifn_test function is redundant with test done at register time by the crypto API, so remove it. Signed-off-by: LABBE Corentin --- drivers/crypto/hifn_795x.c | 58 -- 1 file changed, 58 deletions(-) diff --git a/drivers/crypto/hifn_795x.c

[PATCH 1/4] crypto: hifn_795x: replace simple_strtoul by kstrtouint

2015-10-22 Thread LABBE Corentin
The simple_strtoul function is marked as obsolete. This patch replace it by kstrtouint at the cost of changing some function return type from void to int. Signed-off-by: LABBE Corentin --- drivers/crypto/hifn_795x.c | 28 1 file changed, 20 insertions(+), 8

[PATCH 3/4] crypto: hifn_795x: use dev_xx/pr_xx instead of printk

2015-10-22 Thread LABBE Corentin
This patch replace all printk by their dev_xx/pr_xx counterpart. The patch remove also all custom dprintk by pr_debug/dev_debug Signed-off-by: LABBE Corentin --- drivers/crypto/hifn_795x.c | 149 - 1 file changed, 67 insertions(+), 82 deletions(-)

[PATCH 4/4] crypto: hifn_795x: fix coding style

2015-10-22 Thread LABBE Corentin
The hifn_795x driver is old and have lots of style issue. This patch try to solve easy ones. Signed-off-by: LABBE Corentin --- drivers/crypto/hifn_795x.c | 301 + 1 file changed, 138 insertions(+), 163 deletions(-) diff --git

[PATCHv2 4.3-rc6] proc: fix convert from oom_score_adj to oom_adj

2015-10-22 Thread 方洪杰
The oom_adj has been replaced by oom_score_adj in kernel, but the /proc/pid/oom_adj is provided for legacy purposes. When write/read a value into/from /proc/pid/oom_adj, there is a transformation between oom_adj and oom_score_adj. After writing a new value into /proc/pid/oom_adj, then read it.

Re: [PATCH 04/14] init: deps: order network interfaces by link order

2015-10-22 Thread Alexander Holler
Am 19.10.2015 um 13:31 schrieb Alexander Holler: Am 19.10.2015 um 12:57 schrieb Alexander Holler: Am 18.10.2015 um 12:11 schrieb Alexander Holler: Am 18.10.2015 um 07:59 schrieb Greg Kroah-Hartman: On Sun, Oct 18, 2015 at 07:20:34AM +0200, Alexander Holler wrote: Am 18.10.2015 um 07:14

[PATCH 4/5] block: introduce file_bd_inode()

2015-10-22 Thread Dan Williams
Similar to the file_inode() helper, provide a helper to lookup the inode for a raw block device itself. Cc: Al Viro Signed-off-by: Dan Williams --- fs/block_dev.c | 19 --- 1 file changed, 12 insertions(+), 7 deletions(-) diff --git a/fs/block_dev.c b/fs/block_dev.c index

[PATCH 5/5] block: enable dax for raw block devices

2015-10-22 Thread Dan Williams
If an application wants exclusive access to all of the persistent memory provided by an NVDIMM namespace it can use this raw-block-dax facility to forgo establishing a filesystem. This capability is targeted primarily to hypervisors wanting to provision persistent memory for guests. Cc: Jan Kara

<    4   5   6   7   8   9   10   11   12   13   >