Re: [PATCH 2/3] staging: lustre: Delete an unnecessary variable initialisation in class_register_type()

2015-11-05 Thread Dan Carpenter
Looks good. regards, dan carpenter -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/

Re: [PATCH 4/4] mm: prepare page_referenced() and page_idle to new THP refcounting

2015-11-05 Thread Kirill A. Shutemov
On Thu, Nov 05, 2015 at 03:53:54PM +0300, Vladimir Davydov wrote: > On Thu, Nov 05, 2015 at 02:36:06PM +0200, Kirill A. Shutemov wrote: > > On Thu, Nov 05, 2015 at 03:07:26PM +0300, Vladimir Davydov wrote: > > > @@ -849,30 +836,23 @@ static int page_referenced_one(struct page *page, > > > struct v

Re: [RFC PATCH 0/3]perf/core: extend perf_reg and perf_sample_regs_intr

2015-11-05 Thread Peter Zijlstra
On Thu, Nov 05, 2015 at 02:16:15AM +0530, Madhavan Srinivasan wrote: > Second patch updates struct arch_misc_reg for arch/powerpc with pmu registers > and adds offsetof macro for the same. It extends perf_reg_value() > to use reg idx to decide on struct to return value from. Why; what's in those r

[PATCH v4 1/5] net: stmmac: replace all pr_xxx by their netdev_xxx counterpart

2015-11-05 Thread LABBE Corentin
The stmmac driver use lots of pr_xxx functions to print information. This is bad since we cannot know which device logs the information. (moreover if two stmmac device are present) Furthermore, it seems that it assumes wrongly that all logs will always be subsequent by using a dev_xxx then some in

[PATCH v4] stmmac: improve logging

2015-11-05 Thread LABBE Corentin
Hello This patch series try to improve logging of the stmmac driver. Changes since v3 - Fix a missing comma for letting patch to be used atomaticly. Changes since v2 - Rollback to dev_ for some early init printing - rebased on 4.4-rc1 Changes since v1 - Use netdev_xxx instead of dev_xxx - Use n

[PATCH v4 5/5] net: stmmac: replace if (netif_msg_type) by their netif_xxx counterpart

2015-11-05 Thread LABBE Corentin
As sugested by Joe Perches, we could replace all if (netif_msg_type(priv)) dev_xxx(priv->devices, ...) by the simplier macro netif_xxx(priv, hw, priv->dev, ...) Signed-off-by: LABBE Corentin --- drivers/net/ethernet/stmicro/stmmac/stmmac_main.c | 58 ++- 1 file changed, 25 in

[PATCH v4 4/5] net: stmmac: Fix simple style problem

2015-11-05 Thread LABBE Corentin
This patch fix the following warnings: - braces {} should be used on all arms of this statement - Prefer seq_puts to seq_printf - No space is necessary after a cast - Missing a blank line after declarations - Please don't use multiple blank lines - Comparison to NULL could be written - networking b

[PATCH v4 3/5] net: stmmac: remove some __func__ printing

2015-11-05 Thread LABBE Corentin
Now that stmmac use netdev_xxx, some __func__ are not necessary since their use was to clearly identify which driver was logging. Moreover, as pointed by Joe Perches, using __func__ in xxx_dbg functions is relatively low value and dynamic debug can add it. This patch remove __func__ where such pri

[PATCH v4 2/5] net: stmmac: replace hardcoded function name by __func__

2015-11-05 Thread LABBE Corentin
Some printing have the function name hardcoded. It is better to use __func__ instead. Signed-off-by: LABBE Corentin --- drivers/net/ethernet/stmicro/stmmac/stmmac_main.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/net/ethernet/stmicro/stmmac/stmmac_main.c b

[PATCH 0/2] perf tools: Two bugfixs related to perf probe

2015-11-05 Thread Wang Nan
I found two bugs in perf probe related code. This two patches fix them. Wang Nan (2): perf probe: Only call probe_file__get_events() when fd is valid perf tools: Fix find_perf_probe_point_from_map() which incorrectly returns success tools/perf/builtin-probe.c| 12 ++-- tools/

[PATCH 2/2] perf tools: Fix find_perf_probe_point_from_map() which incorrectly returns success

2015-11-05 Thread Wang Nan
It is possible that find_perf_probe_point_from_map() fails to find symbol but still returns 0 because of an small error when coding: find_perf_probe_point_from_map() set 'ret' to error code at first, but also use it to hold return value of kernel_get_symbol_address_by_name(). This patch resets 're

[PATCH 1/2] perf probe: Only call probe_file__get_events() when fd is valid

2015-11-05 Thread Wang Nan
In system with kprobe enabled but uprobe turned off, 'perf probe -d' causes segfault because it calls probe_file__get_events() with a negative fd (when deleting uprobe events). This patch validates fds before calling probe_file__get_events(). Signed-off-by: Wang Nan Cc: Arnaldo Carvalho de Melo

Re: [PATCH] perf tools: Change FEATURE-DUMP to FEATURE-DUMP.libbpf for lib bpf

2015-11-05 Thread Arnaldo Carvalho de Melo
Em Thu, Nov 05, 2015 at 02:56:34PM +0800, Yunlong Song escreveu: > Commit ed63f34c026e9a60d17fa750ecdfe3f600d49393 ("perf tools: Make perf > depend on libbpf") dynamically creates FEATURE-DUMP.libbpf during the > perf building, but the .gitignore under tools/lib/bpf/ mistakes > FEATURE-DUMP.libbpf

[PATCH] mfd: syscon: check 'property' before of_node_put

2015-11-05 Thread Peng Fan
When 'property' is not NULL, of_parse_phandle will return the node pointer with refcount incremented. So when of_node_put, also need to check property. Signed-off-by: Peng Fan Cc: Lee Jones Cc: Arnd Bergmann --- drivers/mfd/syscon.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) dif

Re: [Bugfix v4] PCI, ACPI: Fix regressions caused by resource_size_t overflow with 32-bit kernel

2015-11-05 Thread Jiang Liu
On 2015/11/5 20:53, Tomasz Nowicki wrote: > On 02.11.2015 16:27, Tomasz Nowicki wrote: >> On 08.07.2015 09:26, Jiang Liu wrote: >>> Zoltan Boszormenyi reported this regression: >>>"There's a Realtek RTL8111/8168/8411 (PCI ID 10ec:8168, Subsystem ID >>> 1565:230e) network chip on the mainboa

How to mark suspicious code?

2015-11-05 Thread Ivan Safonov
Hi all! How can I mark suspicious code, if I can not fix it? -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.o

Re: ptrace and pseudoterminals

2015-11-05 Thread Peter Hurley
On 11/04/2015 02:43 PM, Oleg Nesterov wrote: > On 11/04, Peter Hurley wrote: >> >> Hi Pavel, >> >> On 11/03/2015 06:16 PM, Pavel Labath wrote: >>> Hello Oleg, everyone, >>> >>> I have noticed something, which may be considered a race in the >>> interaction of ptrace and pseudoterminal interfaces. B

Re: [PATCH 1/1] staging: rdma: hfi1 : Prefer using the BIT macro

2015-11-05 Thread Leon Romanovsky
On Thu, Nov 05, 2015 at 05:28:03PM +0530, Sunny Kumar wrote: > This patch replaces bit shifting on 1 with the BIT(x) macro > > Signed-off-by: Sunny Kumar > --- > drivers/staging/rdma/hfi1/user_sdma.c | 10 +- > 1 file changed, 5 insertions(+), 5 deletions(-) > > /* Last packet in the r

Re: [PATCH] mm: slab: Only move management objects off-slab for sizes larger than KMALLOC_MIN_SIZE

2015-11-05 Thread Andrew Morton
On Thu, 5 Nov 2015 11:50:35 + Catalin Marinas wrote: > Commit 8fc9cf420b36 ("slab: make more slab management structure off the > slab") enables off-slab management objects for sizes starting with > PAGE_SIZE >> 5. This means 128 bytes for a 4KB page configuration. > However, on systems with

[for-next][PATCH] tracefs: Fix refcount imbalance in start_creating()

2015-11-05 Thread Steven Rostedt
git://git.kernel.org/pub/scm/linux/kernel/git/rostedt/linux-trace.git for-next Head SHA1: d227c3ae4e94e5eb11dd780a811f59e1a7b74ccd Daniel Borkmann (1): tracefs: Fix refcount imbalance in start_creating() fs/tracefs/inode.c | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-)

Re: [for-next][PATCH] tracefs: Fix refcount imbalance in start_creating()

2015-11-05 Thread Steven Rostedt
Forgot to Cc Daniel. -- Steve On Thu, 5 Nov 2015 08:29:33 -0500 Steven Rostedt wrote: > git://git.kernel.org/pub/scm/linux/kernel/git/rostedt/linux-trace.git > for-next > > Head SHA1: d227c3ae4e94e5eb11dd780a811f59e1a7b74ccd > > > Daniel Borkmann (1): > tracefs: Fix refcount imbala

Re: [Outreachy kernel] [PATCH v2] firewire: Replace timeval with timespec64

2015-11-05 Thread Stefan Richter
On Oct 22 Arnd Bergmann wrote: > On Thursday 22 October 2015 04:05:00 Amitoj Kaur Chawla wrote: > > 32 bit systems using 'struct timeval' will break in the year 2038, so > > we replace the code appropriately. However, this driver is not broken > > in 2038 since we are using only the microseconds po

Re: [PATCH 01/29] pci: Decouple quirks.c from i915_reg.h

2015-11-05 Thread Chris Wilson
On Wed, Nov 04, 2015 at 11:19:49PM +0200, ville.syrj...@linux.intel.com wrote: > From: Ville Syrjälä > > i915 register defines are going to become type safe, so going forward > the register defines can't be used as straight numbers. Since quirks.c > needs just a few extra register defines from i9

[PATCH 7/7] arm64: dts: Add mtcmos and pmic node for hi6220 HiKey board

2015-11-05 Thread Chen Feng
Add dts node for hi665x pmic and hi6220 mtcmos driver Signed-off-by: Chen Feng Signed-off-by: Fei Wang --- arch/arm64/boot/dts/hisilicon/hi6220.dtsi | 210 ++ 1 file changed, 210 insertions(+) diff --git a/arch/arm64/boot/dts/hisilicon/hi6220.dtsi b/arch/arm64/boot

Re: [RFC PATCH v6 00/17] ILP32 for ARM64

2015-11-05 Thread Andrew Pinski
On Thu, Nov 5, 2015 at 7:36 PM, Andreas Schwab wrote: > Yury Norov writes: > >> v6: >> - time_t, __kenel_off_t and other types turned to be 32-bit >>for compatibility reasons (after v5 discussion); > > Are the updated glibc patches available somewhere? Not in an useful form right now but i

Re: [RFC PATCH 4/7] pinctrl: sunxi: remove redundant "depends on RESET_CONTROLLER"

2015-11-05 Thread Linus Walleij
On Thu, Nov 5, 2015 at 12:15 PM, Masahiro Yamada wrote: > ARCH_SUNXI selects RESET_CONTROLLER. > The dependency "depends on RESET_CONTROLLER" is already met. > > Signed-off-by: Masahiro Yamada NACK, this pin controller needs that as dependency, I don't worry that it's overspecified. They both

[PATCH 4/7] mfd: hi655x: Add hi665x pmic driver

2015-11-05 Thread Chen Feng
Add pmic driver to support hisilicon hi665x pmic. Signed-off-by: Chen Feng Signed-off-by: Fei Wang --- drivers/mfd/Kconfig | 9 ++ drivers/mfd/Makefile| 1 + drivers/mfd/hi655x-pmic.c | 315 include/linux/mfd/hi655x-pmic

[PATCH 1/7] doc:bindings:Add document for mfd hi665x PMIC

2015-11-05 Thread Chen Feng
Add document for mfd driver hi655x pmic driver Signed-off-by: Chen Feng Signed-off-by: Fei Wang --- .../devicetree/bindings/mfd/hisilicon,hi655x.txt | 18 ++ 1 file changed, 18 insertions(+) create mode 100644 Documentation/devicetree/bindings/mfd/hisilicon,hi655x.txt di

[PATCH] staging: most: Delete an unnecessary check before the function call "module_put"

2015-11-05 Thread SF Markus Elfring
From: Markus Elfring Date: Thu, 5 Nov 2015 14:34:43 +0100 The module_put() function tests whether its argument is NULL and then returns immediately. Thus the test around the call is not needed. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring --- drivers

Re: [PATCH] w1: w1_process() is not freezable kthread

2015-11-05 Thread Evgeniy Polyakov
Hi 28.10.2015, 08:26, "Jiri Kosina" : > On Tue, 27 Oct 2015, Evgeniy Polyakov wrote: > >>  > w1_process() calls try_to_freeze(), but the thread doesn't mark itself >>  > freezable through set_freezable(), so the try_to_freeze() call is useless. >> >>  I believe it is better to mark it freezable, w

[PATCH 0/7] Add Support for Hi6220 PMIC Hi6553 MFD Core

2015-11-05 Thread Chen Feng
The patch sets add support for Hi6220 PMIC Hi655x MFD core and its regulator driver. Current testing and support board is Hikey which is one of 96boards. It is an arm64 open source board. For more information about this board, please access https://www.96boards.org. This is hardware layout for

[PATCH 3/7] doc:bindings:Document for hi655x pmic driver

2015-11-05 Thread Chen Feng
Add Document for hi655x pmic driver Signed-off-by: Chen Feng Signed-off-by: Fei Wang --- .../regulator/hisilicon,hi655x-regulator.txt | 50 ++ 1 file changed, 50 insertions(+) create mode 100644 Documentation/devicetree/bindings/regulator/hisilicon,hi655x-regulator.t

Re: [Linaro-acpi] [PATCH v8 5/5] Watchdog: introduce ARM SBSA watchdog driver

2015-11-05 Thread Timur Tabi
Fu Wei wrote: (1)It is not new. pre-timeout concept has been used by two drivers before this driver. and this concept has been in kernel documentation. It's "new" in that it's a new infrastructure. The private API of two other drivers doesn't count. (1) if we don't, for this two stages t

Re: [Linaro-acpi] [PATCH v8 5/5] Watchdog: introduce ARM SBSA watchdog driver

2015-11-05 Thread Timur Tabi
Guenter Roeck wrote: I would feel much more comfortable if the driver would just use the standard watchdog timeout and live with (worst case) 20 seconds timeout for now. Actually, I'm wondering where the 20 seconds comes from. When I load my driver on our hardware, it calculates a maximum tim

Re: [PATCH] mm, hugetlb: use memory policy when available

2015-11-05 Thread Vlastimil Babka
On 10/20/2015 09:53 PM, Dave Hansen wrote: > From: Dave Hansen > > I have a hugetlbfs user which is never explicitly allocating huge pages > with 'nr_hugepages'. They only set 'nr_overcommit_hugepages' and then let > the pages be allocated from the buddy allocator at fault time. > > This works,

[PATCH 5/7] regulator: add driver for mtcmos voltage regulator on hi6220 SoC

2015-11-05 Thread Chen Feng
Add driver to support mtcmos on hi6220 Signed-off-by: Chen Feng Signed-off-by: Fei Wang --- drivers/regulator/hi6220-mtcmos.c | 245 ++ 1 file changed, 245 insertions(+) create mode 100644 drivers/regulator/hi6220-mtcmos.c diff --git a/drivers/regulator/hi6

Re: [PATCH] PCI: rcar (usb): DT can override default window settings

2015-11-05 Thread Rob Herring
On Tue, Nov 03, 2015 at 04:19:26PM +, Phil Edworthy wrote: > If the dtb specifies dma-ranges, we use those values. Otherwise, we > default to the values that were previously hardcoded into the driver. > > Signed-off-by: Phil Edworthy > --- > .../devicetree/bindings/pci/pci-rcar-gen2.txt

[PATCH 6/7] regulator: hisilicon: Add hi655x pmic voltage regulator driver

2015-11-05 Thread Chen Feng
Add driver support for HiSilicon Hi655x voltage regulators. Signed-off-by: Chen Feng Signed-off-by: Fei Wang --- drivers/regulator/Kconfig | 13 ++ drivers/regulator/Makefile | 2 + drivers/regulator/hi655x-regulator.c | 246

[PATCH 2/7] doc:bindings:Document for mtcmos regulator on hi6220 SoC

2015-11-05 Thread Chen Feng
Add Document for mtcmos driver on hi6220 SoC Signed-off-by: Chen Feng Signed-off-by: Fei Wang --- .../bindings/regulator/hisilicon,hi6220-mtcmos.txt | 32 ++ 1 file changed, 32 insertions(+) create mode 100644 Documentation/devicetree/bindings/regulator/hisilicon,hi6220-mt

Re: [PATCH v6 13/17] arm64:ilp32: add sys_ilp32.c and a separate table (in entry.S) to use it

2015-11-05 Thread Arnd Bergmann
On Tuesday 03 November 2015 02:30:42 Yury Norov wrote: > From: Andrew Pinski > > Add a separate syscall-table for ILP32, which dispatches either to native > LP64 system call implementation or to compat-syscalls, as appropriate. The uapi/asm-generic/unistd.h already contains a list of compat sysc

Re: [PATCH] staging: most: Delete an unnecessary check before the function call "module_put"

2015-11-05 Thread Dan Carpenter
Relying on hidden sanity checks makes the code harder to read. A human being cannot remember which functions have sanity checks and which do not. These sorts of patches are easy to generate automatically but they make the code worse. There are so many *better* things to do instead of focusing on

Re: [Xen-devel] Linux 4.4 MW: Boot under Xen fails with CONFIG_DEBUG_WX enabled: RIP: ptdump_walk_pgd_level_core

2015-11-05 Thread Boris Ostrovsky
On 11/05/2015 04:13 AM, Sander Eikelenboom wrote: It makes "cat /sys/kernel/debug/kernel_page_tables" work and prevents a kernel with CONFIG_DEBUG_WX=y from crashing at boot. Great. Our nightly runs also failed spectacularly due to this bug. It now does give a warning about an insecure W+X

Re: [Bugfix v4] PCI, ACPI: Fix regressions caused by resource_size_t overflow with 32-bit kernel

2015-11-05 Thread Tomasz Nowicki
On 05.11.2015 14:24, Jiang Liu wrote: On 2015/11/5 20:53, Tomasz Nowicki wrote: On 02.11.2015 16:27, Tomasz Nowicki wrote: On 08.07.2015 09:26, Jiang Liu wrote: Zoltan Boszormenyi reported this regression: "There's a Realtek RTL8111/8168/8411 (PCI ID 10ec:8168, Subsystem ID 1565:230e)

Re: [PATCH v6 11/17] ptrace: Allow compat to use the native siginfo

2015-11-05 Thread Arnd Bergmann
On Tuesday 03 November 2015 02:30:40 Yury Norov wrote: > From: Andrew Pinski > > Set COMPAT_USE_NATIVE_SIGINFO to be true for non AARCH32 tasks. > > With ARM64 ILP32 ABI, we want to use the non-compat > siginfo as we want to simplify signal handling for this new ABI. > This patch just adds a new

Re: [PATCH 1/4] arm64: dts: Reserve memory regions for hi6220

2015-11-05 Thread Leo Yan
On Thu, Oct 29, 2015 at 04:33:01PM +0800, Leo Yan wrote: > On Wed, Oct 28, 2015 at 11:32:29PM -0500, Rob Herring wrote: > > On Fri, Oct 9, 2015 at 9:20 AM, Leo Yan wrote: > > > On Fri, Oct 09, 2015 at 08:50:13AM -0500, Rob Herring wrote: > > >> On Fri, Oct 9, 2015 at 8:30 AM, Mark Rutland > > >>

Re: [PATCH 1/1] I2C: designware: fix IO timeout issue for AMD controller

2015-11-05 Thread Mika Westerberg
On Thu, Nov 05, 2015 at 08:34:44PM +0800, Xiangliang Yu wrote: > Because of some hardware limitation, AMD I2C controller can't > trigger next interrupt if interrupt status has been changed > after clearing interrupt status bits. Then, I2C will lost > interrupt and IO timeout. > > According to hard

Re: staging: most: Delete an unnecessary check before the function call "module_put"

2015-11-05 Thread SF Markus Elfring
> Please stop sending these patches for drivers/staging. Will further contributors take another look at similar update suggestions? > You are welcome to send them for other subsystems Thanks for this suggestion. > which I don't care about. I am curious if other software developers will give

Re: [Linaro-acpi] [PATCH v8 5/5] Watchdog: introduce ARM SBSA watchdog driver

2015-11-05 Thread Fu Wei
Hi Timur On 5 November 2015 at 21:47, Timur Tabi wrote: > Guenter Roeck wrote: >> >> I would feel much more comfortable if the driver would just use the >> standard >> watchdog timeout and live with (worst case) 20 seconds timeout for now. > > > Actually, I'm wondering where the 20 seconds comes

Re: [Xen-devel] Linux 4.4 MW: Boot under Xen fails with CONFIG_DEBUG_WX enabled: RIP: ptdump_walk_pgd_level_core

2015-11-05 Thread Sander Eikelenboom
Thursday, November 5, 2015, 2:53:40 PM, you wrote: > On 11/05/2015 04:13 AM, Sander Eikelenboom wrote: >> >> It makes "cat /sys/kernel/debug/kernel_page_tables" work and >> prevents a kernel with CONFIG_DEBUG_WX=y from crashing at boot. > Great. Our nightly runs also failed spectacularly due to

Re: [PATCH 1/7] doc:bindings:Add document for mfd hi665x PMIC

2015-11-05 Thread Mark Brown
On Thu, Nov 05, 2015 at 09:34:42PM +0800, Chen Feng wrote: Please use subject lines matching the style for the subsystem. This makes it easier for people to identify relevant patches. > +- #interrupt-cells: Should be 2, two cells are needed for irq. > +- interrupt-controller: hi655x has internal

Re: perf related lockdep bug

2015-11-05 Thread Paul E. McKenney
On Thu, Nov 05, 2015 at 03:46:07AM +0100, Andi Kleen wrote: > On Wed, Nov 04, 2015 at 04:55:28PM -0800, Paul E. McKenney wrote: > > On Wed, Nov 04, 2015 at 09:58:36PM +0100, Andi Kleen wrote: > > > > > > I tested my perf stress workload with the patch applied on 4.3, > > > unfortunately got a hang

Re: [RFC PATCH 4/7] pinctrl: sunxi: remove redundant "depends on RESET_CONTROLLER"

2015-11-05 Thread Linus Walleij
On Thu, Nov 5, 2015 at 2:40 PM, Linus Walleij wrote: > On Thu, Nov 5, 2015 at 12:15 PM, Masahiro Yamada > wrote: > >> ARCH_SUNXI selects RESET_CONTROLLER. >> The dependency "depends on RESET_CONTROLLER" is already met. >> >> Signed-off-by: Masahiro Yamada > > NACK, this pin controller needs that

Re: [RFC PATCH 3/3] sched: introduce synchronized idle injection

2015-11-05 Thread Peter Zijlstra
On Tue, Nov 03, 2015 at 02:31:20PM +0100, Peter Zijlstra wrote: > > @@ -5136,6 +5148,16 @@ pick_next_task_fair(struct rq *rq, struct > > task_struct *prev) > > struct task_struct *p; > > int new_tasks; > > > > +#ifdef CONFIG_CFS_IDLE_INJECT > > + if (cfs_rq->force_throttled && > > +

Re: [Linaro-acpi] [PATCH v8 5/5] Watchdog: introduce ARM SBSA watchdog driver

2015-11-05 Thread Timur Tabi
Fu Wei wrote: SBSA 2.3 Page 23 : Note: the watchdog offset register is 32 bits wide. This gives a maximum watch period of around 10s at a system counter frequency of 400MHz. If a larger watch period is required then the compare value can be programmed directly into the compare value register. 21

RE: [PATCH 2/2] perf tools: Fix find_perf_probe_point_from_map() which incorrectly returns success

2015-11-05 Thread 平松雅巳 / HIRAMATU,MASAMI
From: Wang Nan [mailto:wangn...@huawei.com] > >It is possible that find_perf_probe_point_from_map() fails to find >symbol but still returns 0 because of an small error when coding: >find_perf_probe_point_from_map() set 'ret' to error code at first, >but also use it to hold return value of >kernel_g

Re: [PATCH v6 14/17] aarch64: ilp32: use generic stat64 structure

2015-11-05 Thread Arnd Bergmann
On Tuesday 03 November 2015 02:30:43 Yury Norov wrote: > Generic 32-bit and AARCH32 stat64 structures has same names. > ILP32 needs generic stat64. So we can either make ILP32 mutual > exclusive with AARCH32, or introduce __stat64, and where needed, > cast stat64 to it with #define. > > Second opt

Re: [PATCH] perf trace: fix documentation for -i

2015-11-05 Thread Arnaldo Carvalho de Melo
Em Wed, Nov 04, 2015 at 09:21:46AM -0800, Peter Feiner escreveu: > The -i flag was incorrectly listed as a short flag for --no-inherit. > It should have only been listed as a short flag for --input. > > This documentation error has existed since the --input flag was > introduced in 6810fc915f7a89d

Re: [PATCH 2/7] doc:bindings:Document for mtcmos regulator on hi6220 SoC

2015-11-05 Thread Mark Brown
On Thu, Nov 05, 2015 at 09:34:43PM +0800, Chen Feng wrote: > +- hisilicon,mtcmos-steady-us: The time to wait for power steady > +- hisilicon,mtcmos-sc-on-base: address of mtcmos on hi6220 SoC > + > +Required child device properties: > +- regulator-name: The name of mtcmos > +- hisilicon,ctrl-regs:

Re: [PATCH] kconfig: fix missing symbol error

2015-11-05 Thread kbuild test robot
Hi yalin, [auto build test WARNING on: v4.3-rc7] [also build test WARNING on: next-20151105] url: https://github.com/0day-ci/linux/commits/yalin-wang/kconfig-fix-missing-symbol-error/20151105-204811 config: x86_64-allmodconfig (attached as .config) reproduce: # save the attached

Re: [RFC PATCH 3/3] sched: introduce synchronized idle injection

2015-11-05 Thread Arjan van de Ven
On 11/5/2015 2:09 AM, Peter Zijlstra wrote: I can see such a scheme having a fairly big impact on latency, esp. with forced idleness such as this. That's not going to be popular for many workloads. idle injection is a last ditch effort in thermal management, before this gets used the hardware

RE: [PATCH 1/2] perf probe: Only call probe_file__get_events() when fd is valid

2015-11-05 Thread 平松雅巳 / HIRAMATU,MASAMI
From: Wang Nan [mailto:wangn...@huawei.com] > >In system with kprobe enabled but uprobe turned off, 'perf probe -d' >causes segfault because it calls probe_file__get_events() with a >negative fd (when deleting uprobe events). Hmm, OK. This may happen if user runs perf probe on the kernel which onl

RE: [PATCH 1/1] staging: rdma: hfi1 : Prefer using the BIT macro

2015-11-05 Thread Marciniszyn, Mike
> Subject: [PATCH 1/1] staging: rdma: hfi1 : Prefer using the BIT macro > > This patch replaces bit shifting on 1 with the BIT(x) macro > > Signed-off-by: Sunny Kumar > --- Nak. The patch leaves the shift in. Mike -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in t

Re: [Patch v7 4/7] PCI/ACPI: Add interface acpi_pci_root_create()

2015-11-05 Thread Tomasz Nowicki
On 14.10.2015 08:29, Jiang Liu wrote: Introduce common interface acpi_pci_root_create() and related data structures to create PCI root bus for ACPI PCI host bridges. It will be used to kill duplicated arch specific code for IA64 and x86. It may also help ARM64 in future. Reviewed-by: Lorenzo Pie

Re: [PATCH 3/7] doc:bindings:Document for hi655x pmic driver

2015-11-05 Thread Mark Brown
On Thu, Nov 05, 2015 at 09:34:44PM +0800, Chen Feng wrote: > +Required properties: > +- compatible: Must be "hisilicon,hi655x-regulator-pmic"; If this is a subfunction of a MFD it shouldn't have a compatible string. If it is instead a standalone device it should just have a name in the form "vend

Re: GPF in shm_lock ipc

2015-11-05 Thread Kirill A. Shutemov
Dmitry Vyukov wrote: > On Tue, Oct 13, 2015 at 8:30 PM, Kirill A. Shutemov > wrote: > > On Mon, Oct 12, 2015 at 08:18:21PM -0700, Davidlohr Bueso wrote: > >> On Mon, 12 Oct 2015, Bueso wrote: > >> > >> >On Mon, 12 Oct 2015, Kirill A. Shutemov wrote: > >> > > >> >>On Mon, Oct 12, 2015 at 10:49:45AM

Re: [PATCH v7] PCI: Xilinx-NWL-PCIe: Added support for Xilinx NWL PCIe Host Controller

2015-11-05 Thread Rob Herring
On Tue, Nov 03, 2015 at 08:53:47PM +0530, Bharat Kumar Gogada wrote: > Adding PCIe Root Port driver for Xilinx PCIe NWL bridge IP. > > Signed-off-by: Bharat Kumar Gogada > Signed-off-by: Ravi Kiran Gummaluri > --- > Removed msi_controller and added irq_domian for MSI domain hierarchy. > Modified

Re: [PATCH 4/7] mfd: hi655x: Add hi665x pmic driver

2015-11-05 Thread Mark Brown
On Thu, Nov 05, 2015 at 09:34:45PM +0800, Chen Feng wrote: > +config MFD_HI655X_PMIC > +bool "HiSilicon Hi655X series PMU/Codec IC" Why is this bool and not tristate? > +depends on ARCH_HISI Can we have an || COMPILE_TEST here? > +static irqreturn_t hi655x_pmic_irq_handler(int

Re: [PATCH V2 1/3] dma: add Qualcomm Technologies HIDMA management driver

2015-11-05 Thread Rob Herring
On Mon, Nov 2, 2015 at 12:30 PM, Timur Tabi wrote: > On 11/02/2015 12:25 PM, Rob Herring wrote: >> >> Then document it with "" and fill that in later. Just don't make >> up version numbers. > > > I don't think you understand. We literally have no name for our chip. The > closest is what I used on

Re: [PATCH v6 01/17] arm64:ilp32: add documentation on the ILP32 ABI for ARM64

2015-11-05 Thread Arnd Bergmann
On Tuesday 03 November 2015 02:30:30 Yury Norov wrote: > From: Philipp Tomsich > > Based on Andrew Pinski's original patch-series and adapted with changes > to reduce the duplication of code-paths and resolve issue found during > LTP testing. > > Signed-off-by: Philipp Tomsich > Signed-off-by:

Re: [RFC PATCH 3/3] sched: introduce synchronized idle injection

2015-11-05 Thread Peter Zijlstra
On Thu, Nov 05, 2015 at 06:22:58AM -0800, Arjan van de Ven wrote: > On 11/5/2015 2:09 AM, Peter Zijlstra wrote: > > >I can see such a scheme having a fairly big impact on latency, esp. with > >forced idleness such as this. That's not going to be popular for many > >workloads. > > idle injection i

Re: [Linaro-acpi] [PATCH v8 5/5] Watchdog: introduce ARM SBSA watchdog driver

2015-11-05 Thread Fu Wei
Hi Timur, On 5 November 2015 at 22:08, Timur Tabi wrote: > Fu Wei wrote: >> >> SBSA 2.3 Page 23 : >> Note: the watchdog offset register is 32 bits wide. This gives a >> maximum watch period of around 10s at a system >> counter frequency of 400MHz. If a larger watch period is required then >> the

Re: [PATCH v3 2/5] mfd: arizona: Update DT bindings to add CS47L24 and WM1831

2015-11-05 Thread Rob Herring
On Tue, Nov 03, 2015 at 03:08:33PM +, Richard Fitzgerald wrote: > This updates the Arizona MFD device tree bindings to add the > Cirrus Logic CS47L24 and WM1831 codecs. Note that unlike all the > other codecs the DCVDD-supply and MICVDD-supply are mandatory. > > Signed-off-by: Richard Fitzgera

Re: [PATCH V4] usb: remove unnecessary CONFIG_PM dependency from USB_OTG

2015-11-05 Thread Felipe Balbi
Hi, Peter Chen writes: > On Tue, Nov 03, 2015 at 09:51:11PM -0600, Felipe Balbi wrote: >> >> Hi, >> >> Peter Chen writes: >> > On Tue, Nov 03, 2015 at 07:56:55AM -0600, Felipe Balbi wrote: >> >> >> >> Hi, >> >> >> >> Nathan Sullivan writes: >> >> > The USB OTG support currently depends on

Re: [Linaro-acpi] [PATCH v8 5/5] Watchdog: introduce ARM SBSA watchdog driver

2015-11-05 Thread Timur Tabi
Fu Wei wrote: Did you really read the "Note" above OK, let me paste it again and again: SBSA 2.3 Page 23 : If a larger watch period is required then the compare value can be programmed directly into the compare value register. Well, okay. Sorry, I should have read what you pasted more

Re: [PATCH 5/8] mm: memcontrol: account socket memory on unified hierarchy

2015-11-05 Thread Michal Hocko
On Wed 04-11-15 14:50:37, Johannes Weiner wrote: [...] > Because it goes without saying that once the cgroupv2 interface is > released, and people use it in production, there is no way we can then > *add* dentry cache, inode cache, and others to memory.current. That > would be an unacceptable chang

[PATCH 02/25] perf stat record: Add record command

2015-11-05 Thread Jiri Olsa
Add 'perf stat record' command support. It creates simple (header only) perf.data file ATM. The record command could be specified anywhere among stat options. All stat command options are valid for stat record command with '-o' option exception. If specified for record command it denotes the perf

Re: [PATCH V2 1/3] dma: add Qualcomm Technologies HIDMA management driver

2015-11-05 Thread Timur Tabi
Rob Herring wrote: I'm saying document it as "qcom,-hidma-mgmt" and when you have the part number update the binding. Meanwhile push on the powers that be to decide on a part number. Got it. But we should we do about this: static const struct of_device_id qcom_hidma_mgmt_match[] = { {

[tip:x86/apic] x86/apic: Implement single target IPI function for x2apic_cluster

2015-11-05 Thread tip-bot for Linus Torvalds
Commit-ID: 7b6ce46cb3d096831dea3accacee4717c66abac8 Gitweb: http://git.kernel.org/tip/7b6ce46cb3d096831dea3accacee4717c66abac8 Author: Linus Torvalds AuthorDate: Wed, 4 Nov 2015 22:57:00 + Committer: Thomas Gleixner CommitDate: Thu, 5 Nov 2015 13:07:52 +0100 x86/apic: Implement sin

[tip:x86/apic] x86/apic: Remove pointless indirections from apic_physflat

2015-11-05 Thread tip-bot for Thomas Gleixner
Commit-ID: 449112f4f35074f1dc70d4f0e769cb14150c159c Gitweb: http://git.kernel.org/tip/449112f4f35074f1dc70d4f0e769cb14150c159c Author: Thomas Gleixner AuthorDate: Wed, 4 Nov 2015 22:57:02 + Committer: Thomas Gleixner CommitDate: Thu, 5 Nov 2015 13:07:52 +0100 x86/apic: Remove point

Re: [PATCH 5/7] regulator: add driver for mtcmos voltage regulator on hi6220 SoC

2015-11-05 Thread Mark Brown
On Thu, Nov 05, 2015 at 09:34:46PM +0800, Chen Feng wrote: > Add driver to support mtcmos on hi6220 I just noticed that these patches are all being posted to the IOMMU list - that seems a bit odd? > +static int hi6220_mtcmos_is_on(struct hi6220_mtcmos *mtcmos, > +unsig

[tip:x86/apic] x86/apic: Add a single-target IPI function to the apic

2015-11-05 Thread tip-bot for Linus Torvalds
Commit-ID: 539da7877275edb21a76aa02fb2c147eff02c559 Gitweb: http://git.kernel.org/tip/539da7877275edb21a76aa02fb2c147eff02c559 Author: Linus Torvalds AuthorDate: Wed, 4 Nov 2015 22:57:00 + Committer: Thomas Gleixner CommitDate: Thu, 5 Nov 2015 13:07:51 +0100 x86/apic: Add a single-

Re: [PATCH 00/28] tracing: 'hist' triggers

2015-11-05 Thread Namhyung Kim
Hi Tom, I implemented 'log2' modifier on top of your v11 patchset. Could you please take a look at this? >From 5fc74778b9af5e4e121a0563fb8e54e8f495ca29 Mon Sep 17 00:00:00 2001 From: Namhyung Kim Date: Thu, 5 Nov 2015 23:34:05 +0900 Subject: [PATCH] tracing: Add hist trigger 'log2' modifier

Re: [RFC PATCH 0/3]perf/core: extend perf_reg and perf_sample_regs_intr

2015-11-05 Thread Stephane Eranian
Hi, On Wed, Nov 4, 2015 at 9:46 PM, Madhavan Srinivasan wrote: > > This patchset extend the perf sample regs infrastructure > to include architecture specific regs. Current perf_sample_regs_intr > exports only registers in the pt_regs to perf.data using > PERF_SAMPLE_REGS_INTR sample type. But so

[tip:x86/apic] x86/apic: Wire up single IPI for bigsmp_apic

2015-11-05 Thread tip-bot for Thomas Gleixner
Commit-ID: 5789a12e28f7bf6a37564a5fc9ebc60dc86659b5 Gitweb: http://git.kernel.org/tip/5789a12e28f7bf6a37564a5fc9ebc60dc86659b5 Author: Thomas Gleixner AuthorDate: Wed, 4 Nov 2015 22:57:04 + Committer: Thomas Gleixner CommitDate: Thu, 5 Nov 2015 13:07:52 +0100 x86/apic: Wire up sing

[tip:x86/apic] x86/apic: Implement single IPI for x2apic_phys

2015-11-05 Thread tip-bot for Thomas Gleixner
Commit-ID: f2bffe8a3eef42a1cd3393d56acd9fe598d2119c Gitweb: http://git.kernel.org/tip/f2bffe8a3eef42a1cd3393d56acd9fe598d2119c Author: Thomas Gleixner AuthorDate: Wed, 4 Nov 2015 22:57:04 + Committer: Thomas Gleixner CommitDate: Thu, 5 Nov 2015 13:07:53 +0100 x86/apic: Implement si

[PATCH 24/25] perf script: Add python support for stat events

2015-11-05 Thread Jiri Olsa
Add support to get stat events data in perf python scripts. The python script shall implement following new interface to process stat data: def stat___[](cpu, thread, time, val, ena, run): - is called for every stat event for given counter, if user monitors 'cycles,instructions:u" fo

[tip:x86/apic] x86/apic: Implement default single target IPI function

2015-11-05 Thread tip-bot for Thomas Gleixner
Commit-ID: 53be0fac8bdaeec87e0df7d0334345421d2be187 Gitweb: http://git.kernel.org/tip/53be0fac8bdaeec87e0df7d0334345421d2be187 Author: Thomas Gleixner AuthorDate: Wed, 4 Nov 2015 22:57:01 + Committer: Thomas Gleixner CommitDate: Thu, 5 Nov 2015 13:07:52 +0100 x86/apic: Implement de

Re: [Outreachy kernel] [PATCH v2] firewire: Replace timeval with timespec64

2015-11-05 Thread Arnd Bergmann
On Thursday 05 November 2015 14:34:48 Stefan Richter wrote: > On Oct 22 Arnd Bergmann wrote: > > On Thursday 22 October 2015 04:05:00 Amitoj Kaur Chawla wrote: > > > 32 bit systems using 'struct timeval' will break in the year 2038, so > > > we replace the code appropriately. However, this driver i

[tip:x86/apic] x86/apic: Wire up single IPI for apic_physflat

2015-11-05 Thread tip-bot for Thomas Gleixner
Commit-ID: 68cd88ff8df97846eb07080f17264a4de50cb012 Gitweb: http://git.kernel.org/tip/68cd88ff8df97846eb07080f17264a4de50cb012 Author: Thomas Gleixner AuthorDate: Wed, 4 Nov 2015 22:57:02 + Committer: Thomas Gleixner CommitDate: Thu, 5 Nov 2015 13:07:52 +0100 x86/apic: Wire up sing

[tip:x86/apic] x86/apic: Remove pointless indirections from bigsmp_apic

2015-11-05 Thread tip-bot for Thomas Gleixner
Commit-ID: 500bd02fb17e5d9296c77ccc07db61fd5d4922a4 Gitweb: http://git.kernel.org/tip/500bd02fb17e5d9296c77ccc07db61fd5d4922a4 Author: Thomas Gleixner AuthorDate: Wed, 4 Nov 2015 22:57:03 + Committer: Thomas Gleixner CommitDate: Thu, 5 Nov 2015 13:07:52 +0100 x86/apic: Remove point

[PATCH 22/25] perf script: Add stat default handlers

2015-11-05 Thread Jiri Olsa
Implement struct scripting_ops::(process_stat|process_stat_interval) handlers - calling scripting handlers from stat events handlers. Tested-by: Kan Liang Link: http://lkml.kernel.org/n/tip-3iu6vmvuur1bntbnh43v3...@git.kernel.org Signed-off-by: Jiri Olsa --- tools/perf/builtin-script.c | 31 +++

[PATCH 25/25] perf script: Add stat-cpi.py script

2015-11-05 Thread Jiri Olsa
Adding stat-cpi.py as an example of how to do stat scripting. It computes the CPI metrics from cycles and instructions events. Following stat record/report/script combinations could be used: - get CPI for given workload $ perf stat -e cycles,instructions record ls SNIP Performance

[PATCH 17/25] perf stat report: Move csv_sep initialization before report command

2015-11-05 Thread Jiri Olsa
So we have csv_sep properly initialized before report command leg. Tested-by: Kan Liang Link: http://lkml.kernel.org/n/tip-i5mmbomj7lt4eg04awzoc...@git.kernel.org Signed-off-by: Jiri Olsa --- tools/perf/builtin-stat.c | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff --

[PATCH 07/25] perf stat record: Write stat events on record

2015-11-05 Thread Jiri Olsa
Writing stat events on 'perf stat record' at the time we read counter values from kernel. Tested-by: Kan Liang Link: http://lkml.kernel.org/n/tip-ojc826gy0x9jj74elbjfc...@git.kernel.org Signed-off-by: Jiri Olsa --- tools/perf/builtin-stat.c | 19 +++ 1 file changed, 19 insertion

[PATCH 05/25] perf stat record: Store events IDs in perf data file

2015-11-05 Thread Jiri Olsa
Store event IDs in evlist object so it get stored into perf.data file. Also making perf_evlist__id_add_fd global. Tested-by: Kan Liang Link: http://lkml.kernel.org/n/tip-v2xwpeots3tz08e44lej5...@git.kernel.org Signed-off-by: Jiri Olsa --- tools/perf/builtin-stat.c | 35

[PATCH 06/25] perf stat record: Add pipe support for record command

2015-11-05 Thread Jiri Olsa
Allowing storing stat record data into pipe, so report tools (report/script) could read data directly from record. Tested-by: Kan Liang Link: http://lkml.kernel.org/n/tip-m8fj758gty57hgvss5efy...@git.kernel.org Signed-off-by: Jiri Olsa --- tools/perf/builtin-stat.c | 39

[PATCH 21/25] perf script: Add process_stat/process_stat_interval scripting interface

2015-11-05 Thread Jiri Olsa
Python and perl scripting code will define those callbacks and get stat data. Tested-by: Kan Liang Link: http://lkml.kernel.org/n/tip-6802z3siu5f59wdsmhaym...@git.kernel.org Signed-off-by: Jiri Olsa --- tools/perf/util/trace-event.h | 4 1 file changed, 4 insertions(+) diff --git a/tools/

[PATCH 23/25] perf script: Display stat events by default

2015-11-05 Thread Jiri Olsa
If no script is specified for stat data, display stat events in raw form. $ perf stat record ls SNIP Performance counter stats for 'ls': 0.851585 task-clock (msec) #0.717 CPUs utilized 0 context-switches #0.000 K/sec

[PATCH 19/25] perf script: Process cpu/threads maps

2015-11-05 Thread Jiri Olsa
Adding processing of cpu/threads maps. Configuring session's evlist with these maps. Tested-by: Kan Liang Link: http://lkml.kernel.org/n/tip-s3txa1u2qv3bi8uspp4hi...@git.kernel.org Signed-off-by: Jiri Olsa --- tools/perf/builtin-script.c | 67 + 1 fil

[PATCH 09/25] perf stat record: Do not allow record with multiple runs mode

2015-11-05 Thread Jiri Olsa
We currently don't support storing multiple session in perf.data, so we can't allow -r option in stat record. $ perf stat -e cycles -r 2 record ls Cannot use -r option with perf stat record. Tested-by: Kan Liang Link: http://lkml.kernel.org/n/tip-wulio303py16c5hbbo8zr...@git.kernel.org Sig

<    1   2   3   4   5   6   7   8   9   >