Re: 4.9.0 regression in pipe-backed iov_iter with systemd-nspawn

2017-01-12 Thread Alan J. Wylie
at 14:26 on Thu 12-Jan-2017 Linus Torvalds (torva...@linux-foundation.org) wrote: > > Strace shows that the processes are hanging in write() and read() calls. > > If this is splice-related, I'm assuming that they aren't actually the > two ends of the same pipe, and there is somebody doing splice

Re: 4.9.0 regression in pipe-backed iov_iter with systemd-nspawn

2017-01-12 Thread Al Viro
On Thu, Jan 12, 2017 at 10:37:18PM +, Al Viro wrote: > On Thu, Jan 12, 2017 at 02:26:42PM -0800, Linus Torvalds wrote: > > On Thu, Jan 12, 2017 at 12:26 PM, Alan J. Wylie wrote: > > > > > > Strace shows that the processes are hanging in write() and read() calls. > > > > If this is splice-rela

[patch v2] mm, memcg: do not retry precharge charges

2017-01-12 Thread David Rientjes
When memory.move_charge_at_immigrate is enabled and precharges are depleted during move, mem_cgroup_move_charge_pte_range() will attempt to increase the size of the precharge. This can be allowed to do reclaim, but should not call the oom killer to oom kill a process. It's better to fail the atta

Re: [PATCH v2] tcp: fix tcp_fastopen unaligned access complaints on sparc

2017-01-12 Thread Eric Dumazet
On Thu, 2017-01-12 at 14:24 -0800, Shannon Nelson wrote: > Fix up a data alignment issue on sparc by swapping the order > of the cookie byte array field with the length field in > struct tcp_fastopen_cookie, and making it a proper union > to clean up the typecasting. > > This addresses log complai

Re: [PATCH net-next v2 05/10] drivers: base: Add device_find_class()

2017-01-12 Thread Florian Fainelli
On 01/12/2017 01:21 PM, David Miller wrote: > From: Florian Fainelli > Date: Wed, 11 Jan 2017 19:41:16 -0800 > >> Add a helper function to lookup a device reference given a class name. >> This is a preliminary patch to remove adhoc code from net/dsa/dsa.c and >> make it more generic. >> >> Signed

Re: [PATCH v4 1/2] eeprom: Add IDT 89HPESx EEPROM/CSR driver

2017-01-12 Thread Serge Semin
On Wed, Jan 11, 2017 at 09:21:19AM +0100, Greg KH wrote: > On Tue, Dec 13, 2016 at 05:22:50PM +0300, Serge Semin wrote: > > +struct idt_89hpesx_dev { > > + u32 eesize; > > + bool eero; > > + u8 eeaddr; > > + > > + u8 inieecmd; > > + u8 inicsrcmd; > > + u8 iniccode; > > + > > + atomi

Re: [PATCH] clk: stm32f4: avoid uninitialized variable access

2017-01-12 Thread Stephen Boyd
On 01/12/2017 02:42 PM, Arnd Bergmann wrote: > On Thu, Jan 12, 2017 at 11:06 PM, Stephen Boyd wrote: >> Applied to clk-next. Seems I need to update my compiler to find >> these warnings. > I'm currently playing with gcc-7, which adds a lot of new warnings > (including many false positives). gcc-6

[�PATCH net-next] ARM: dts: vf610-zii-dev: add EEPROM entry to Rev B

2017-01-12 Thread Vivien Didelot
The ZII Dev Rev B board has EEPROMs hanging the 88E6352 Ethernet switch chips. Add an "eeprom-length" property to allow access from ethtool. Signed-off-by: Vivien Didelot --- arch/arm/boot/dts/vf610-zii-dev-rev-b.dts | 2 ++ 1 file changed, 2 insertions(+) diff --git a/arch/arm/boot/dts/vf610-z

Re: 4.9.0 regression in pipe-backed iov_iter with systemd-nspawn

2017-01-12 Thread Al Viro
On Thu, Jan 12, 2017 at 10:46:06PM +, Alan J. Wylie wrote: > at 14:26 on Thu 12-Jan-2017 Linus Torvalds (torva...@linux-foundation.org) > wrote: > 13761 copy_file_range(5, NULL, 1, NULL, 9223372036854775807, 0) = -1 EXDEV > (Invalid cross-device link) > 13761 sendfile(1, 5, NULL, 92233720368

Re: 4.9.0 regression in pipe-backed iov_iter with systemd-nspawn

2017-01-12 Thread Linus Torvalds
On Thu, Jan 12, 2017 at 2:46 PM, Al Viro wrote: > > PS: what about the /proc/mounts contents? If it's something 9p-backed kvm, > your bisect might have been caught on the bug I'd mentioned - if the breakage > you are seeing in 4.9.3 has started after that commit and before the > backport of the f

Re: [PATCH] Input: egalax_ts - do not release gpio if probe successful

2017-01-12 Thread Gary Bisson
Hi Dmitri, On Thu, Jan 12, 2017 at 10:30:19AM -0800, Dmitry Torokhov wrote: > Hi Gary, > > On Wed, Jan 11, 2017 at 06:28:41PM +0100, Gary Bisson wrote: > > Thus preventing anyone to later modify the interrupt GPIO direction > > and/or state without the driver knowing. > > I am afraid not releasi

[PATCH] ARM: dts: vf610-zii-dev: add EEPROM entry to Rev C

2017-01-12 Thread Vivien Didelot
The ZII Dev Rev C board has EEPROMs hanging the 88E6390 Ethernet switch chips. Add an "eeprom-length" property to allow access from ethtool. Signed-off-by: Vivien Didelot --- arch/arm/boot/dts/vf610-zii-dev-rev-c.dts | 2 ++ 1 file changed, 2 insertions(+) diff --git a/arch/arm/boot/dts/vf610-z

[PATCH net-next] net: dsa: mv88e6xxx: add EEPROM support to 6390

2017-01-12 Thread Vivien Didelot
The Marvell 6352 chip has a 8-bit address/16-bit data EEPROM access. The Marvell 6390 chip has a 16-bit address/8-bit data EEPROM access. This patch implements the 8-bit data EEPROM access in the mv88e6xxx driver and adds its support to chips of the 6390 family. Signed-off-by: Vivien Didelot ---

Re: [PATCH] xen-netfront: Fix Rx stall during network stress and OOM

2017-01-12 Thread Vineeth Remanan Pillai
On 01/12/2017 12:17 PM, David Miller wrote: From: Vineeth Remanan Pillai Date: Wed, 11 Jan 2017 23:17:17 + @@ -1054,7 +1059,11 @@ static int xennet_poll(struct napi_struct *napi, int budget) napi_complete(napi); RING_FINAL_CHECK_FOR_RESPONSES(&queue->rx, more_to_d

Re: [PATCH] can: Fix kernel panic at security_sock_rcv_skb

2017-01-12 Thread Eric Dumazet
On Thu, 2017-01-12 at 14:40 -0800, william.c.robe...@intel.com wrote: > From: Zhang Yanmin > > The patch is for fix the below kernel panic: > BUG: unable to handle kernel NULL pointer dereference at (null) > IP: [] selinux_socket_sock_rcv_skb+0x65/0x2a0 Same patch was sent earlier, and we gave a

Re: [PATCH 2/4] efi/x86: move efi bgrt init code to early init code

2017-01-12 Thread Nicolai Stange
On Fri, Jan 13 2017, Dave Young wrote: > On 01/12/17 at 12:54pm, Nicolai Stange wrote: >> On Thu, Jan 12 2017, Dave Young wrote: >> >> > -void __init efi_bgrt_init(void) >> > +void __init efi_bgrt_init(struct acpi_table_header *table) >> > { >> > - acpi_status status; >> >void *image; >> >

Re: 4.9.0 regression in pipe-backed iov_iter with systemd-nspawn

2017-01-12 Thread Al Viro
On Thu, Jan 12, 2017 at 03:02:13PM -0800, Linus Torvalds wrote: > splice(5, NULL, 1, NULL, 9223372036854775807, 0) = -1 EAGAIN > (Resource temporarily unavailable) > > and note that the commit in question introduces that -EAGAIN error code. > > The old code never returned EAGAIN at all (well

Re: 4.9.0 regression in pipe-backed iov_iter with systemd-nspawn

2017-01-12 Thread Linus Torvalds
On Thu, Jan 12, 2017 at 3:02 PM, Linus Torvalds wrote: > > Looking at the callers of "do_splice_to()", we already have the > wait_for_space() in do_splice(), but we do *not* have it in the > do_splice_from() case when both the input and output file descriptors > are pipes. Bah. That case doesn't

Re: 4.9.0 regression in pipe-backed iov_iter with systemd-nspawn

2017-01-12 Thread Al Viro
On Thu, Jan 12, 2017 at 03:14:41PM -0800, Linus Torvalds wrote: > On Thu, Jan 12, 2017 at 3:02 PM, Linus Torvalds > wrote: > > > > Looking at the callers of "do_splice_to()", we already have the > > wait_for_space() in do_splice(), but we do *not* have it in the > > do_splice_from() case when both

Re: [PATCH v4 0/4] Introduce the initify gcc plugin

2017-01-12 Thread Kees Cook
On Thu, Jan 12, 2017 at 1:41 PM, Emese Revfy wrote: > On Tue, 10 Jan 2017 17:09:31 -0800 > Kees Cook wrote: > >> WARNING: vmlinux.o(.text+0x1087e7): Section mismatch in reference from >> the function rebind_subsystems() to the variable >> .init.rodata.str:__func__.4400 >> The function rebind_subs

Re: 4.9.0 regression in pipe-backed iov_iter with systemd-nspawn

2017-01-12 Thread Linus Torvalds
On Thu, Jan 12, 2017 at 2:46 PM, Alan J. Wylie wrote: > > I'm off to bed now with a stinking head cold I'm afraid. So assuming Al hasn't figured it out by the time you get back, can you try to send us the same strace for the working case? That fd5 (ptmx) does have O_NONBLOCK, so maybe the EAGAIN

Re: [PATCH] MIPS: zboot: fix build failure

2017-01-12 Thread Sudip Mukherjee
On Wednesday 04 January 2017 06:04 PM, Sudip Mukherjee wrote: The build of mips ar7_defconfig was failing with the error: arch/mips/boot/compressed/Makefile:21: *** insufficient number of arguments (1) to function `filter-out'. Stop. A ',' was missing while adding filter-out. Fixes:

Re: linux-next: build failure after merge of the akpm-current tree

2017-01-12 Thread Andrew Morton
On Thu, 12 Jan 2017 13:06:01 +0800 Eric Ren wrote: > On 01/12/2017 11:49 AM, Stephen Rothwell wrote: > > Hi Andrew, > > > > After merging the akpm tree, today's linux-next build (powerpc > > allyesconfig) failed like this: > > > > In file included from fs/ocfs2/file.c:49:0: > > fs/ocfs2/file.c: I

Re: [PATCH v3 10/24] ARM: dts: imx6-sabreauto: add pinctrl for gpt input capture

2017-01-12 Thread Steve Longerbeam
On 01/12/2017 11:37 AM, Tim Harvey wrote: On Fri, Jan 6, 2017 at 6:11 PM, Steve Longerbeam wrote: Add pinctrl groups for both GPT input capture channels. Signed-off-by: Steve Longerbeam --- arch/arm/boot/dts/imx6qdl-sabreauto.dtsi | 12 1 file changed, 12 insertions(+) diff

Re: [PATCH] Input: egalax_ts - do not release gpio if probe successful

2017-01-12 Thread Dmitry Torokhov
On Fri, Jan 13, 2017 at 12:03:43AM +0100, Gary Bisson wrote: > Hi Dmitri, > > On Thu, Jan 12, 2017 at 10:30:19AM -0800, Dmitry Torokhov wrote: > > Hi Gary, > > > > On Wed, Jan 11, 2017 at 06:28:41PM +0100, Gary Bisson wrote: > > > Thus preventing anyone to later modify the interrupt GPIO directio

Re: [PATCH v4 0/4] Introduce the initify gcc plugin

2017-01-12 Thread Kees Cook
On Thu, Jan 12, 2017 at 3:27 PM, Kees Cook wrote: > On Thu, Jan 12, 2017 at 1:41 PM, Emese Revfy wrote: >> On Tue, 10 Jan 2017 17:09:31 -0800 >> Kees Cook wrote: >> >>> WARNING: vmlinux.o(.text+0x1087e7): Section mismatch in reference from >>> the function rebind_subsystems() to the variable >>>

[PATCH] radix-tree: Fix private list warnings

2017-01-12 Thread Matthew Wilcox
From: Matthew Wilcox The newly introduced warning in radix_tree_free_nodes() was testing the wrong variable; it should have been 'old' instead of 'node'. Rather than replace that one instance, I noticed that we can simply put the WARN_ON_ONCE in radix_tree_node_free() and it will be just as effe

Re: [PATCH] usb: dwc2: gadget: Fix GUSBCFG.USBTRDTIM value

2017-01-12 Thread John Youn
On 1/12/2017 7:41 AM, Amelie DELAUNAY wrote: > Hi all, > Sorry, I did not see Pengcheng Li patch which is exactly the same: > https://urldefense.proofpoint.com/v2/url?u=https-3A__patchwork.kernel.org_patch_9347979_&d=DgIC-g&c=DPL6_X_6JkXFx7AXWqB0tg&r=U3o8uKoKhWme5_V9D-eeCkB11BFwt4KvWztBgdE9ZpA&m=6V

Re: [PATCH] ARM: dts: NSP: Fix DT ranges error

2017-01-12 Thread Florian Fainelli
On 01/12/2017 07:50 AM, Jon Mason wrote: > The range size for axi is 0x2 bytes too small, as the QSPI needs > 0x11c408 + 0x004 (which is 0x0011c40c, not 0x0011c40a). No errors have > been observed with this shortcoming, but fixing it for correctness. > > Signed-off-by: Jon Mason Applied to devi

[PATCH] PCI: iproc: fix resource allocation for BCMA PCIe

2017-01-12 Thread Abylay Ospan
Resource allocated on stack was saved by 'devm_request_resource' to global 'iomem_resource' but become invalid after 'iproc_pcie_bcma_probe' exit. So the global 'iomem_resource' was poisoned. This may cause kernel crash or second PCIe bridge registration failure. Tested on Broadcom NorthStar machi

Re: [PATCH v4 06/15] lockdep: Make save_trace can skip stack tracing of the current

2017-01-12 Thread Byungchul Park
On Thu, Jan 12, 2017 at 05:37:57PM +0100, Peter Zijlstra wrote: > On Fri, Dec 09, 2016 at 02:12:02PM +0900, Byungchul Park wrote: > > Currently, save_trace() always performs save_stack_trace() for the > > current. However, crossrelease needs to use stack trace data of another > > context instead of

linux-next: build warning after merge of the wireless-drivers-next tree

2017-01-12 Thread Stephen Rothwell
Hi all, After merging the wireless-drivers-next tree, today's linux-next build (x86_64 allmodconfig) produced this warning: drivers/net/wireless/marvell/mwifiex/pcie.c: In function 'mwifiex_pcie_remove': drivers/net/wireless/marvell/mwifiex/pcie.c:303:5: warning: 'fw_status' may be used uninitia

[PATCH] PCI: iproc: fix kernel crash if dev->of_node not defined

2017-01-12 Thread Abylay Ospan
pcie->dev->of_node not always defined (NULL) and can cause crash: [ 19.053195] Unable to handle kernel NULL pointer dereference at virtual address 0020 [] (of_n_addr_cells) from [] (iproc_pcie_setup+0x30c/0xce0) this patch adds sanity check to prevent crash. Signed-off-by: Abylay Ospan --

Re: [PATCH] PCI: iproc: fix kernel crash if dev->of_node not defined

2017-01-12 Thread Florian Fainelli
On 01/12/2017 04:20 PM, Abylay Ospan wrote: > pcie->dev->of_node not always defined (NULL) and can cause crash: > > [ 19.053195] Unable to handle kernel NULL pointer dereference at > virtual address 0020 > [] (of_n_addr_cells) from [] > (iproc_pcie_setup+0x30c/0xce0) > > this patch adds san

Re: [PATCH v4 0/4] Application Data Integrity feature introduced by SPARC M7

2017-01-12 Thread Khalid Aziz
On 01/12/2017 10:53 AM, Dave Hansen wrote: On 01/12/2017 08:50 AM, Khalid Aziz wrote: 2. Any shared page that has ADI protection enabled on it, must stay ADI protected across all processes sharing it. Is that true? What happens if a page with ADI tags set is accessed via a PTE without the ADI

Re: [PATCH v2 RESEND] video: backlight: pwm_bl: Initialize fb_bl_on[x] and use_count during pwm_backlight_probe()

2017-01-12 Thread Lukasz Majewski
Dear All, > Thierry, Jingoo, > > Please respond to Lukasz. Yes, your response is more than welcome... :-) Thanks in advance, Łukasz Majewski > > On Mon, 26 Dec 2016, Lukasz Majewski wrote: > > > The commit a55944ca82d2 > > ("backlight: update bd state & fb_blank properties when necessa

Re: [PATCH V2] usb: dwc2: host: fix Wmaybe-uninitialized warning

2017-01-12 Thread John Youn
mt, \ >^ > drivers/usb/dwc2/hcd.c:4492:8: note: 'pipetype' was declared here > char *pipetype; > ^ > Patch was compile tested with: x86_64_defconfig + CONFIG_USB_DWC2=m + > CONFIG_USB_DWC2_VERBOSE=y > > Patch is against 4.10-rc3 (localversion-next is

Re: [PATCH RFC] usb: dwc2: host: use msleep() for long delay

2017-01-12 Thread John Youn
2: Add functions to set and clear force mode") > so not sure if I am not overlooking some reason for the highrestimer here ? > > This needs to be clarified by someone that knows the details and history > of this device/driver. > > Patch was compile tested with: x86_64_defcon

Re: [PATCHv5 3/8] rtc: add STM32 RTC driver

2017-01-12 Thread Alexandre Belloni
On 11/01/2017 at 14:46:43 +0100, Amelie Delaunay wrote : > This patch adds support for the STM32 RTC. > > Signed-off-by: Amelie Delaunay > --- > drivers/rtc/Kconfig | 11 + > drivers/rtc/Makefile| 1 + > drivers/rtc/rtc-stm32.c | 727 >

Re: [PATCHv3 2/8] dt-bindings: document the STM32 RTC bindings

2017-01-12 Thread Alexandre Belloni
On 05/01/2017 at 14:43:23 +0100, Amelie Delaunay wrote : > This patch adds documentation of device tree bindings for the STM32 RTC. > > Signed-off-by: Amelie Delaunay > Acked-by: Rob Herring > --- > .../devicetree/bindings/rtc/st,stm32-rtc.txt | 27 > ++ > 1 file chan

Re: [PATCH] usb: dwc2: host: use msleep() for long delays

2017-01-12 Thread John Youn
; Signed-off-by: Nicholas Mc Guire > --- > > Patch was compile tested with: x86_64_defconfig + CONFIG_USB_DWC2 > (1 sparse and 6 coccinelle warning - preparing separate patches for that) > > Patch is against 4.10-rc3 (localversion-next is next-20170112) > > drivers/usb/

Re: [PATCH] PCI: iproc: fix resource allocation for BCMA PCIe

2017-01-12 Thread Ray Jui
Hi Abylay, On 1/12/2017 3:58 PM, Abylay Ospan wrote: > Resource allocated on stack was saved by 'devm_request_resource' to > global 'iomem_resource' but become invalid after 'iproc_pcie_bcma_probe' exit. > So the global 'iomem_resource' was poisoned. This may cause kernel crash > or second PCIe br

Re: [PATCH] Input: synaptics-rmi4 - make F03 a tristate symbol

2017-01-12 Thread Andrew Duggan
On 01/11/2017 11:27 AM, Christopher Heiny wrote: On Wed, 2017-01-11 at 18:48 +0100, Benjamin Tissoires wrote: On Jan 11 2017 or thereabouts, Arnd Bergmann wrote: On Wednesday, January 11, 2017 5:28:28 PM CET Benjamin Tissoires wrote: Yep, it was initially written that way, and IIRC there was s

Re: [PATCH] PCI: iproc: fix kernel crash if dev->of_node not defined

2017-01-12 Thread Ray Jui
On 1/12/2017 4:20 PM, Abylay Ospan wrote: > pcie->dev->of_node not always defined (NULL) and can cause crash: Ah I guess this can happen with the BCMA based platforms that do not use device tree for PCIe? > > [ 19.053195] Unable to handle kernel NULL pointer dereference at > virtual address

Re: [PATCH] PCI: iproc: fix kernel crash if dev->of_node not defined

2017-01-12 Thread Ray Jui
Hi Florian, On 1/12/2017 4:22 PM, Florian Fainelli wrote: > On 01/12/2017 04:20 PM, Abylay Ospan wrote: >> pcie->dev->of_node not always defined (NULL) and can cause crash: >> >> [ 19.053195] Unable to handle kernel NULL pointer dereference at >> virtual address 0020 >> [] (of_n_addr_cells)

Re: [PATCH 2/3] perf/x86/amd/uncore: Dynamically allocate uncore counters

2017-01-12 Thread Natarajan, Janakarajan
On 1/12/2017 3:20 AM, Peter Zijlstra wrote: On Wed, Jan 11, 2017 at 10:02:17AM -0600, Janakarajan Natarajan wrote: This patch updates the AMD uncore driver to support AMD Family17h processors. In Family17h, there are two extra last level cache counters. The counters are, therefore, allocated dy

Re: [PATCH] ARM64: dts: meson-gxbb-odroidc2: fix GbE tx link breakage

2017-01-12 Thread Kevin Hilman
Jerome Brunet writes: > OdroidC2 GbE link breaks under heavy tx transfer. This happens even if the > MAC does not enable Energy Efficient Ethernet (No Low Power state Idle on > the Tx path). The problem seems to come from the phy Rx path, entering the > LPI state. > > Disabling EEE advertisement

Re: [PATCH 5/6] treewide: use kv[mz]alloc* rather than opencoded variants

2017-01-12 Thread Dilger, Andreas
> On Jan 12, 2017, at 08:37, Michal Hocko wrote: > > From: Michal Hocko > > There are many code paths opencoding kvmalloc. Let's use the helper > instead. The main difference to kvmalloc is that those users are usually > not considering all the aspects of the memory allocator. E.g. allocation

Re: [tpmdd-devel] [PATCH RFC v2 3/5] tpm: infrastructure for TPM spaces

2017-01-12 Thread James Bottomley
On Thu, 2017-01-12 at 19:46 +0200, Jarkko Sakkinen wrote: > @@ -189,6 +190,12 @@ struct tpm_chip *tpm_chip_alloc(struct device > *pdev, > chip->cdev.owner = THIS_MODULE; > chip->cdev.kobj.parent = &chip->dev.kobj; > > + chip->work_space.context_buf = kzalloc(PAGE_SIZE, > GFP_KERNEL

Re: [lustre-devel] [PATCH 1/2] Fixed signedness check

2017-01-12 Thread Dilger, Andreas
On Dec 26, 2016, at 08:43, Guillermo O. Freschi wrote: > > Was `unsigned int`, but `enum`s are signed. Interesting that GCC didn't complain that the itree_overlap_cb() function signature didn't match the argument prototype for interval_search(): typedef enum interval_iter (*interval_callback_

Re: 4.10-rc1: thinkpad x60: who ate my cpu?

2017-01-12 Thread Woody Suwalski
Pavel Machek wrote: Hi! I used to have two cpus, and Thinkpad X60 should have two cores, but I only see one on 4.10-rc1. This machine went through many suspend/resume cycles. When backups finish, I'll try -rc2. Whoever did it, he seems to have returned the cpu in -rc3. All seems to be good now

Re: [lustre-devel] [PATCH 2/2] Style fixes

2017-01-12 Thread Dilger, Andreas
> On Dec 26, 2016, at 08:43, Guillermo O. Freschi wrote: > > Missing braces on `if` statement; misaligned parameter. > > Signed-off-by: Guillermo O. Freschi > --- > drivers/staging/lustre/lustre/ldlm/ldlm_lock.c | 12 +++- > 1 file changed, 7 insertions(+), 5 deletions(-) > > diff --gi

Re: [PATCH] PCI: iproc: fix kernel crash if dev->of_node not defined

2017-01-12 Thread Florian Fainelli
On 01/12/2017 04:48 PM, Ray Jui wrote: > Hi Florian, > > On 1/12/2017 4:22 PM, Florian Fainelli wrote: >> On 01/12/2017 04:20 PM, Abylay Ospan wrote: >>> pcie->dev->of_node not always defined (NULL) and can cause crash: >>> >>> [ 19.053195] Unable to handle kernel NULL pointer dereference at >>>

Re: [lustre-devel] [PATCH] staging: lustre: selftest: Make brw_inject_one_error() static

2017-01-12 Thread Dilger, Andreas
On Dec 23, 2016, at 08:42, Karthik Nayak wrote: > > Since the function brw_inject_one_error() is used only within > brw_test.c, make it static. This was reported as a warning by sparse. > > Signed-off-by: Karthik Nayak Reviewed-by: Andreas Dilger > --- > drivers/staging/lustre/lnet/selftest/

Re: [PATCH] usb: dwc2: host: use true/false for boolean

2017-01-12 Thread John Youn
ING: Assignment of bool to 0/1 > ./drivers/usb/dwc2/hcd.c:3299:1-21: WARNING: Assignment of bool to 0/1 > > Patch was compile tested with: x86_64_defconfig + CONFIG_USB_DWC2 > > Patch is against 4.10-rc3 (localversion-next is next-20170112) > > drivers/usb/dwc2/hcd.c | 12

Re: [PATCH v20 0/4] Mediatek MT8173 CMDQ support

2017-01-12 Thread Horng-Shyang Liao
On Wed, 2017-01-04 at 11:06 +0800, HS Liao wrote: > Hi, > > This is Mediatek MT8173 Command Queue(CMDQ) driver. The CMDQ is used > to help write registers with critical time limitation, such as > updating display configuration during the vblank. It controls Global > Command Engine (GCE) hardware t

Re: [PATCH v2 7/7] uapi: export all headers under uapi directories

2017-01-12 Thread Jeff Epler
On Thu, Jan 12, 2017 at 05:32:09PM +0100, Nicolas Dichtel wrote: > What I was trying to say is that I export those directories like other are. > Removing those files is not related to that series. Perhaps the correct solution is to only copy files matching "*.h" to reduce the risk of copying files

Re: [PATCH v4 0/4] Application Data Integrity feature introduced by SPARC M7

2017-01-12 Thread Rob Gardner
On 01/12/2017 05:22 PM, Khalid Aziz wrote: On 01/12/2017 10:53 AM, Dave Hansen wrote: On 01/12/2017 08:50 AM, Khalid Aziz wrote: 2. Any shared page that has ADI protection enabled on it, must stay ADI protected across all processes sharing it. Is that true? What happens if a page with ADI ta

Re: [PATCH v2 1/1] serial: 8250_dw: Allow hardware flow control to be used

2017-01-12 Thread Kefeng Wang
On 2017/1/12 3:48, Jason Uy wrote: > In the most common use case, the Synopsys DW UART driver does not > set the set_termios callback function. This prevents UPSTAT_AUTOCTS > from being set when the UART flag CRTSCTS is set. As a result, the > driver will use software flow control as opposed to

Re: + mm-vmscan-add-mm_vmscan_inactive_list_is_low-tracepoint.patch added to -mm tree

2017-01-12 Thread Minchan Kim
Hello, On Thu, Jan 12, 2017 at 10:10:17AM +0100, Michal Hocko wrote: > On Thu 12-01-17 17:48:13, Minchan Kim wrote: > > On Thu, Jan 12, 2017 at 09:15:54AM +0100, Michal Hocko wrote: > > > On Thu 12-01-17 14:12:47, Minchan Kim wrote: > > > > Hello, > > > > > > > > On Wed, Jan 11, 2017 at 04:52:39P

Re: Duplicate inode number when mount --bind some directories to same mountpoint. (from Fedora18 to 4.10-rc3)

2017-01-12 Thread Nakajima Akira
On 2017/01/12 19:24, Al Viro wrote: On Thu, Jan 12, 2017 at 06:16:35PM +0900, Nakajima Akira wrote: Bug: Duplicate inode number when mount --bind some directories to same mountpoint. (from Fedora18 to 4.10-rc3) Fedora17 and earlier works correctly. Explain, please. "Duplicate inode number" be

Re: [PATCH] partially revert "xen: Remove event channel notification through Xen PCI platform device"

2017-01-12 Thread Stefano Stabellini
On Thu, 12 Jan 2017, Boris Ostrovsky wrote: > On 01/12/2017 04:33 PM, Stefano Stabellini wrote: > > On Thu, 12 Jan 2017, Boris Ostrovsky wrote: > >> On 01/11/2017 06:36 PM, Stefano Stabellini wrote: > >>> The following commit: > >>> > >>> commit 72a9b186292d98494f26cfd24a1621796209 > >>> Author

Re: x86-64: Maintain 16-byte stack alignment

2017-01-12 Thread Andy Lutomirski
On Thu, Jan 12, 2017 at 12:15 PM, Josh Poimboeuf wrote: > On Thu, Jan 12, 2017 at 12:08:07PM -0800, Andy Lutomirski wrote: >> On Thu, Jan 12, 2017 at 11:51 AM, Linus Torvalds >> wrote: >> > On Thu, Jan 12, 2017 at 6:02 AM, Josh Poimboeuf >> > wrote: >> >> >> >> Just to clarify, I think you're a

[PATCH] libnvdimm, namespace: fix pmem namespace leak, delete when size set to zero

2017-01-12 Thread Dan Williams
Commit 98a29c39dc68 ("libnvdimm, namespace: allow creation of multiple pmem-namespaces per region") added support for establishing additional pmem namespace beyond the seed device, similar to blk namespaces. However, it neglected to delete the namespace when the size is set to zero. Fixes: 98a29c3

RE: [PATCH v8 1/1] crypto: add virtio-crypto driver

2017-01-12 Thread Gonglei (Arei)
> > On Thu, Jan 12, 2017 at 03:10:25PM +0100, Christian Borntraeger wrote: > > On 01/10/2017 01:56 PM, Christian Borntraeger wrote: > > > On 01/10/2017 01:36 PM, Gonglei (Arei) wrote: > > >> Hi, > > >> > > >>> > > >>> On 12/15/2016 03:03 AM, Gonglei wrote: > > >>> [...] > > + > > +static

Re: [PATCH] coresight: STM: Balance enable/disable

2017-01-12 Thread Chunyan Zhang
On 11 January 2017 at 21:59, Suzuki K Poulose wrote: > On 11/01/17 11:41, Chunyan Zhang wrote: >> >> On 11 January 2017 at 01:36, Mathieu Poirier >> wrote: >>> >>> On Tue, Jan 10, 2017 at 11:21:55AM +, Suzuki K Poulose wrote: The stm is automatically enabled when an application sets

Re: eMMC boot problem: switch to bus width 8 ddr failed

2017-01-12 Thread Shawn Lin
On 2017/1/13 0:51, Ulf Hansson wrote: + Haibo, Gary, Fabio, Shawn Gou On 6 January 2017 at 16:56, Clemens Gruber wrote: On Fri, Jan 06, 2017 at 10:33:49AM +0800, Shawn Lin wrote: On 2017/1/6 8:41, Clemens Gruber wrote: Hi, with the current mainline 4.10-rc2 kernel, I can no longer boot from

Re: [PATCH 2/4] efi/x86: move efi bgrt init code to early init code

2017-01-12 Thread Dave Young
On 01/13/17 at 12:11am, Nicolai Stange wrote: > On Fri, Jan 13 2017, Dave Young wrote: > > > On 01/12/17 at 12:54pm, Nicolai Stange wrote: > >> On Thu, Jan 12 2017, Dave Young wrote: > >> > >> > -void __init efi_bgrt_init(void) > >> > +void __init efi_bgrt_init(struct acpi_table_header *table) >

Re: [PATCH] mm/page_alloc: Wait for oom_lock before retrying.

2017-01-12 Thread Sergey Senozhatsky
On (01/12/17 15:18), Petr Mladek wrote: > On Mon 2016-12-26 20:34:07, Sergey Senozhatsky wrote: > > console_trylock() used to always forbid rescheduling; but it got changed > > like a yaer ago. > > > > the other thing is... do we really need to console_conditional_schedule() > > from fbcon_*()? co

[PATCH] arm64: dts: mt8173: Fix cpu_thermal cooling-maps contributions

2017-01-12 Thread Daniel Kurtz
According to [0], the contribution field for each cooling-device express their relative power efficiency. Higher weights express higher power efficiency. Weighting is relative such that if each cooling device has a weight of 1 they are considered equal. This is particularly useful in heterogeneous

Re: [PATCH 0/3] ata: add m68k/Atari Falcon PATA support

2017-01-12 Thread Finn Thain
On Wed, 11 Jan 2017, Michael Schmitz wrote: > What is still correct is that the IDE driver does use the interrupt > only, not the ST-DMA chip. And a single IDE interrupt can be correctly > assigned to IDE by looking at the status register. > > With the SCSI (and IIRC also floppy) interrupts, w

Re: [PATCH net-next v2 00/10] net: dsa: Support for pdata in dsa2

2017-01-12 Thread Vivien Didelot
Hi Florian, Florian Fainelli writes: > Hi all, > > This is not exactly new, and was sent before, although back then, I did not > have an user of the pre-declared MDIO board information, but now we do. Note > that I have additional changes queued up to have b53 register platform data > for > MIP

Re: [PATCH] usb: hcd: initialize hcd->flags to 0 when rm hcd

2017-01-12 Thread wlf
Hi Roger, 在 2017年01月12日 23:38, Roger Quadros 写道: On 12/01/17 17:33, Alan Stern wrote: On Thu, 12 Jan 2017, Roger Quadros wrote: William, On 12/01/17 14:03, William Wu wrote: From: William wu On some platforms(e.g. rk3399 board), we can call hcd_add/remove consecutively without calling usb

[PATCH] rcu: Narrow early boot window of illegal synchronous grace periods

2017-01-12 Thread Paul E. McKenney
The current preemptible RCU implementation goes through three phases during bootup. In the first phase, there is only one CPU that is running with preemption disabled, so that a no-op is a synchronous grace period. In the second mid-boot phase, the scheduler is running, but RCU has not yet gotten

Re: [PATCH v4 05/15] lockdep: Make check_prev_add can use a separate stack_trace

2017-01-12 Thread Byungchul Park
On Thu, Jan 12, 2017 at 05:16:43PM +0100, Peter Zijlstra wrote: > On Fri, Dec 09, 2016 at 02:12:01PM +0900, Byungchul Park wrote: > > check_prev_add() saves a stack trace of the current. But crossrelease > > feature needs to use a separate stack trace of another context in > > check_prev_add(). So

Re: [PATCH] mm/page_alloc: Wait for oom_lock before retrying.

2017-01-12 Thread Sergey Senozhatsky
On (01/12/17 14:10), Petr Mladek wrote: [..] > > /** > > * console_lock - lock the console system for exclusive use. > > * > > @@ -2316,7 +2321,7 @@ EXPORT_SYMBOL(console_unlock); > > */ > > void __sched console_conditional_schedule(void) > > { > > - if (console_may_schedule) > > + if

RE: [PATCH v2 1/4] ASoC: rt5660: Add ACPI support

2017-01-12 Thread Bard Liao
> -Original Message- > From: Shrirang Bagul [mailto:shrirang.ba...@canonical.com] > Sent: Thursday, January 12, 2017 8:01 PM > To: alsa-de...@alsa-project.org > Cc: linux-kernel@vger.kernel.org; Bard Liao; Oder Chiou; Liam Girdwood; Mark > Brown; Jaroslav Kysela; Takashi Iwai > Subject: [P

[PATCH v3] ocfs2/journal: fix umount hang after flushing journal failure

2017-01-12 Thread Changwei Ge
Hi Joseph, Do you think my last version of patch to fix umount hang after journal flushing failure is OK? If so, I 'd like to ask Andrew's help to merge this patch into his test tree. Thanks, Br. Changwei From 686b52ee2f06395c53e36e2c7515c276dc7541fb Mon Sep 17 00:00:00 2001 From: Changwei

[PATCH] lockdep: Fix wrong condition to print bug msgs for MAX_LOCKDEP_CHAIN_HLOCKS

2017-01-12 Thread Byungchul Park
Bug messages and stack dump for MAX_LOCKDEP_CHAIN_HLOCKS should be printed only at the first time. Signed-off-by: Byungchul Park --- kernel/locking/lockdep.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/kernel/locking/lockdep.c b/kernel/locking/lockdep.c index f37156f..a14

Re: [PATCH 2/4] efi/x86: move efi bgrt init code to early init code

2017-01-12 Thread Dave Young
On 01/13/17 at 10:21am, Dave Young wrote: > On 01/13/17 at 12:11am, Nicolai Stange wrote: > > On Fri, Jan 13 2017, Dave Young wrote: > > > > > On 01/12/17 at 12:54pm, Nicolai Stange wrote: > > >> On Thu, Jan 12 2017, Dave Young wrote: > > >> > > >> > -void __init efi_bgrt_init(void) > > >> > +voi

Re: [PATCH] virtio_mmio: expose header to userspace

2017-01-12 Thread Jason Wang
On 2017年01月13日 05:37, Michael S. Tsirkin wrote: It's handy for userspace emulators like QEMU. Signed-off-by: Michael S. Tsirkin --- drivers/virtio/virtio_mmio.c | 2 +- include/{ => uapi}/linux/virtio_mmio.h | 0 2 files changed, 1 insertion(+), 1 deletion(-) rename include/{

Re: x86-64: Maintain 16-byte stack alignment

2017-01-12 Thread Josh Poimboeuf
On Thu, Jan 12, 2017 at 05:46:55PM -0800, Andy Lutomirski wrote: > On Thu, Jan 12, 2017 at 12:15 PM, Josh Poimboeuf wrote: > > On Thu, Jan 12, 2017 at 12:08:07PM -0800, Andy Lutomirski wrote: > >> On Thu, Jan 12, 2017 at 11:51 AM, Linus Torvalds > >> wrote: > >> > On Thu, Jan 12, 2017 at 6:02 AM,

[RESEND PATCH] usb: hcd: initialize hcd->flags to 0 when rm hcd

2017-01-12 Thread William Wu
From: William wu On some platforms(e.g. rk3399 board), we can call hcd_add/remove consecutively without calling usb_put_hcd/usb_create_hcd in between, so hcd->flags can be stale. If the HC dies due to whatever reason then without this patch we get the below error on next hcd_add. [173.296154] x

RE: eMMC boot problem: switch to bus width 8 ddr failed

2017-01-12 Thread Bough Chen
> -Original Message- > From: Shawn Lin [mailto:shawn@rock-chips.com] > Sent: Friday, January 13, 2017 10:11 AM > To: Ulf Hansson ; Clemens Gruber > > Cc: shawn@rock-chips.com; linux-...@vger.kernel.org; Linus Walleij > ; Adrian Hunter ; A.S. > Dong ; linux-kernel@vger.kernel.org; B

linux-next: build warning after merge of the rtc tree

2017-01-12 Thread Stephen Rothwell
Hi Alexandre, After merging the rtc tree, today's linux-next build (x86_64 allmodconfig) produced this warning: In file included from drivers/rtc/rtc-stm32.c:14:0: drivers/rtc/rtc-stm32.c: In function 'stm32_rtc_probe': drivers/rtc/rtc-stm32.c:653:51: warning: large integer implicitly truncated t

[PATCH] usb: host: xhci: plat: check hcc_params after add hcd

2017-01-12 Thread William Wu
From: William wu The commit 4ac53087d6d4 ("usb: xhci: plat: Create both HCDs before adding them") move add hcd to the end of probe, this cause hcc_params uninitiated, because xHCI driver sets hcc_params in xhci_gen_setup() called from usb_add_hcd(). This patch checks the Maximum Primary Stream A

Re: x86-64: Maintain 16-byte stack alignment

2017-01-12 Thread Andy Lutomirski
On Thu, Jan 12, 2017 at 7:11 PM, Josh Poimboeuf wrote: > On Thu, Jan 12, 2017 at 05:46:55PM -0800, Andy Lutomirski wrote: >> On Thu, Jan 12, 2017 at 12:15 PM, Josh Poimboeuf wrote: >> > On Thu, Jan 12, 2017 at 12:08:07PM -0800, Andy Lutomirski wrote: >> >> On Thu, Jan 12, 2017 at 11:51 AM, Linus

Re: Problem on SCTP

2017-01-12 Thread Sun Paul
Hi All below is the packet trace in text format. Packet received (From client to router) [ Source IP: 192.168.206.83, Destination IP: 192.168.206.65. the IPv4 address parameter is 192.168.206.83 and 192.168.1.83 ] == No. Time Source

Re: Duplicate inode number when mount --bind some directories to same mountpoint. (from Fedora18 to 4.10-rc3)

2017-01-12 Thread Al Viro
On Fri, Jan 13, 2017 at 10:40:08AM +0900, Nakajima Akira wrote: > On 2017/01/12 19:24, Al Viro wrote: > > On Thu, Jan 12, 2017 at 06:16:35PM +0900, Nakajima Akira wrote: > > > Bug: > > > Duplicate inode number when mount --bind some directories to same > > > mountpoint. (from Fedora18 to 4.10-rc3)

Re: lkp: make.cross: old aarch64-gcc has been removed

2017-01-12 Thread Masami Hiramatsu
On Thu, 12 Jan 2017 11:31:57 +0800 Fengguang Wu wrote: > Hi Masami, > > On Wed, Jan 11, 2017 at 07:08:40PM +0900, Masami Hiramatsu wrote: > >Ping? > > Oops, sorry for overlooking your previous email! > > >BTW, I found that the old toolchain is archived in release.linaro.org. > >So I made a pat

Re: Problem on SCTP

2017-01-12 Thread Sun Paul
Packet to SERVER (from router to SERVER)[ Source IP: 192.168.206.83, Destination IP: 192.168.206.66. the IPv4 address parameter is 192.168.206.83 and 192.168.1.83 ] No. Time SourceSPort Destination Protocol DPort Length Info

Re: Problem on SCTP

2017-01-12 Thread Sun Paul
Packet to SERVER (From router to SERVER) [ Source IP: 192.168.206.83, Destination IP: 192.168.206.66. the IPv4 address parameter is 192.168.206.83 and 192.168.1.83 ] No. Time SourceSPort Destination Protocol DPort Length In

Re: kvm: use-after-free in process_srcu

2017-01-12 Thread Steve Rutherford
I'm not that familiar with the kernel's workqueues, but this seems like the classic "callback outlives the memory it references" use-after-free, where the process_srcu callback is outliving struct kvm (which contains the srcu_struct). If that's right, then calling srcu_barrier (which should wait fo

Re: [PATCH v6 23/25] usb: chipidea: Pullup D+ in device mode via phy APIs

2017-01-12 Thread Peter Chen
On Thu, Jan 12, 2017 at 02:49:51PM -0800, Stephen Boyd wrote: > Quoting Peter Chen (2017-01-12 01:50:40) > > On Wed, Jan 11, 2017 at 04:19:53PM -0800, Stephen Boyd wrote: > > > Quoting Peter Chen (2017-01-02 22:53:19) > > > > On Wed, Dec 28, 2016 at 02:57:09PM -0800, Stephen Boyd wrote: > > > > > I

Re: linux-next: build failure after merge of the akpm-current tree

2017-01-12 Thread Stephen Rothwell
Hi Andrew, On Thu, 12 Jan 2017 15:33:53 -0800 Andrew Morton wrote: > > I'll drop > > ocfs2-dlmglue-prepare-tracking-logic-to-avoid-recursive-cluster-lock.patch > and > ocfs2-fix-deadlocks-when-taking-inode-lock-at-vfs-entry-points.patch I removed them from linux-next today. -- Cheers, Stephe

[PATCH] libfc: Fix variable name in fc_set_wwpn

2017-01-12 Thread Fam Zheng
The parameter name should be wwpn instead of wwnn. Signed-off-by: Fam Zheng --- include/scsi/libfc.h | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/include/scsi/libfc.h b/include/scsi/libfc.h index 96dd0b3..da5033d 100644 --- a/include/scsi/libfc.h +++ b/include/scsi/l

Re: linux-next: build failure after merge of the akpm-current tree

2017-01-12 Thread Stephen Rothwell
Hi Eric, On Thu, 12 Jan 2017 13:06:01 +0800 Eric Ren wrote: > > Thanks for your report and the fix for it. The 0-day project has reported > several days ago, > but this patch set is still in discussion, so I am waiting for more days to > see if other > developers > have any other questions. >

Re: [RFC PATCH] ext4: increase the protection of drop nlink and ext4 inode destroy

2017-01-12 Thread Al Viro
On Thu, Jan 12, 2017 at 12:03:28PM -0500, Theodore Ts'o wrote: > On Thu, Jan 12, 2017 at 04:00:16PM +0800, zhangyi (F) wrote: > > > > At the same time, I think other file systems may have the same problem, do > > you think we should put these detections on the VFS layer? Thus other file > > system

Re: [PATCH 2/3] gpio: davinci: Redesign driver to accommodate ngpios in one gpio chip

2017-01-12 Thread Keerthy
On Friday 13 January 2017 12:36 AM, Grygorii Strashko wrote: On 01/11/2017 08:00 PM, Keerthy wrote: On Wednesday 11 January 2017 11:23 PM, Grygorii Strashko wrote: On 01/10/2017 11:00 PM, Keerthy wrote: The Davinci GPIO driver is implemented to work with one monolithic Davinci GPIO pla

Re: [PATCH v5 2/5] powernv:stop: Uniformly rename power9 to arch300

2017-01-12 Thread Gautham R Shenoy
On Thu, Jan 12, 2017 at 03:17:33PM +0530, Balbir Singh wrote: > On Tue, Jan 10, 2017 at 02:37:01PM +0530, Gautham R. Shenoy wrote: > > From: "Gautham R. Shenoy" > > > > Balbir pointed out that in idle_book3s.S and powernv/idle.c some > > functions and variables had power9 in their names while som

  1   2   3   4   5   6   7   8   9   10   >