Re: [PATCH v2] sdhci-esdhc-imx: Correct two register accesses

2016-10-13 Thread Ulf Hansson
On 10 October 2016 at 20:39, Aaron Brice wrote: > > - The DMA error interrupt bit is in a different position as >compared to the sdhci standard. This is accounted for in >many cases, but not handled in the case of clearing the >INT_STATUS register by writing a 1 to that location. >

Re: [PATCH 4/4] staging/vchi: Remove dependency on CONFIG_BROKEN.

2016-10-13 Thread Greg Kroah-Hartman
On Mon, Oct 03, 2016 at 11:52:09AM -0700, Eric Anholt wrote: > The driver builds now. > > Signed-off-by: Eric Anholt > --- > drivers/staging/vc04_services/Kconfig | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/staging/vc04_services/Kconfig > b/drivers/staging/

Re: [PATCH v7 2/8] power: add power sequence library

2016-10-13 Thread Heiko Stuebner
Am Donnerstag, 13. Oktober 2016, 09:22:16 CEST schrieb Peter Chen: > On Wed, Oct 12, 2016 at 12:30:29PM +0200, Heiko Stuebner wrote: > > Hi, > > > > Am Dienstag, 20. September 2016, 11:36:41 CEST schrieb Peter Chen: > > > We have an well-known problem that the device needs to do some power > > > s

Re: [RESEND PATCH v3 2/2] usb: dwc3: Wait for control tranfer completed when stopping gadget

2016-10-13 Thread Felipe Balbi
Hi, Baolin Wang writes: > @@ -1487,10 +1496,22 @@ static int dwc3_gadget_pullup(struct usb_gadget *g, > int is_on) > > is_on = !!is_on; > > +try_again: > spin_lock_irqsave(&dwc->lock, flags); > ret = dwc3_gadget_run_stop(dwc, is_on, false); > spin_unlock_irqrestore(&

Re: [RESEND PATCH v3 1/2] usb: dwc3: gadget: Add disconnect checking when changing function dynamically

2016-10-13 Thread Felipe Balbi
Hi, Baolin Wang writes: > When system has stpped the gadget, we should avoid queuing any requests queueing is *not* a problem. Starting is. In fact, that's what your patch is doing. > which will cause tranfer failed. Thus adding some disconnect checking to ^^^

Re: [PATCH] gpio / ACPI: fix returned error from acpi_dev_gpio_irq_get()

2016-10-13 Thread Mika Westerberg
On Wed, Oct 12, 2016 at 06:40:30PM +0200, Benjamin Tissoires wrote: > From: David Arcari > > acpi_dev_gpio_irq_get() currently ignores the error returned > by acpi_get_gpiod_by_index() and overwrites it with -ENOENT. > > Problem is this error can be -EPROBE_DEFER, which just blows > up some driv

Re: [PATCH] Staging:greybus:arche-apb-ctrl: fixed some coding style issue

2016-10-13 Thread Greg KH
On Thu, Oct 13, 2016 at 06:00:45AM +0200, Alexander Alemayhu wrote: > On Wed, Oct 12, 2016 at 04:45:54PM +0200, Greg KH wrote: > > > > You can subscribe to the driverdev mailing list to see what othes are > > working on, and you can follow my staging-testing kernel branch in the > > staging.git tr

Re: [PATCH v6] net: ip, diag -- Add diag interface for raw sockets

2016-10-13 Thread Cyrill Gorcunov
On Wed, Oct 12, 2016 at 07:55:04PM -0400, David Miller wrote: > From: Cyrill Gorcunov > Date: Wed, 12 Oct 2016 09:53:29 +0300 > > > I can't rename the field, neither a can use union. > > Remind me again what is wrong with using an anonymous union? Anon union would be a preferred but Eric pointe

Re: EXT: pre 4.9-rc dma regression, imx6 sound etc

2016-10-13 Thread Han, Nandor (GE Healthcare)
On 12/10/2016 16:25, Mika Penttilä wrote: Hi! Bisected that commit 5881826 - "dmaengine: imx-sdma - update the residue calculation for cyclic channels" is first bad commit to cause audio regression on imx6q, sgtl5000 codec. It causes audible disturbing background noise when playing wavs.

Re: [PATCH 2/2] MIPS: DTS: img: add device tree for Marduk board

2016-10-13 Thread Rahul Bedarkar
Hi, On Monday 10 October 2016 07:51 PM, Rob Herring wrote: + + memory { Is 0 the actual base, or that gets filled in by bootloader? If the formet, add unit address. Bootloader (uboot) can override or fixup memory node. But with version of bootloader I tested with, base address is har

Re: [PATCH v2] usb: dwc3: gadget: Wait for end transfer complete before free irq

2016-10-13 Thread Felipe Balbi
Hi, Baolin Wang writes: > @@ -1742,6 +1791,17 @@ static int dwc3_gadget_stop(struct usb_gadget *g) > dwc->gadget_driver = NULL; > spin_unlock_irqrestore(&dwc->lock, flags); > > + /* > + * Since the xHCI will share the same interrupt with gadget, thus when > + * f

Re: [PATCH] mfd: intel-lpss: Do not put device in reset state on suspend

2016-10-13 Thread Mika Westerberg
On Wed, Oct 12, 2016 at 10:12:20AM -0700, Azhar Shaikh wrote: > Commit 41a3da2b8e163 ("mfd: intel-lpss: Save register context on > suspend") saved the register context while going to suspend and > also put the device in reset state. > > Due to the resetting of device, system cannot enter S3/S0ix >

Re: [PATCH] irqchip/jcore: fix lost per-cpu interrupts

2016-10-13 Thread Paul E. McKenney
On Wed, Oct 12, 2016 at 06:19:27PM -0400, Rich Felker wrote: > On Wed, Oct 12, 2016 at 01:34:17PM -0700, Paul E. McKenney wrote: > > On Wed, Oct 12, 2016 at 12:35:43PM -0400, Rich Felker wrote: > > > On Wed, Oct 12, 2016 at 10:18:02AM +0200, Thomas Gleixner wrote: > > > > On Tue, 11 Oct 2016, Rich

Re: Regression in mobility grouping?

2016-10-13 Thread Joonsoo Kim
Sorry for late response. On Thu, Sep 29, 2016 at 12:14:02PM -0400, Johannes Weiner wrote: > On Thu, Sep 29, 2016 at 03:14:33PM +0900, Joonsoo Kim wrote: > > On Wed, Sep 28, 2016 at 10:25:40PM -0400, Johannes Weiner wrote: > > > On Wed, Sep 28, 2016 at 11:39:25AM -0400, Johannes Weiner wrote: > > >

Re: [PATCH v2] usb: dwc3: gadget: Wait for end transfer complete before free irq

2016-10-13 Thread Baolin Wang
Hi, On 13 October 2016 at 15:02, Felipe Balbi wrote: > > Hi, > > Baolin Wang writes: >> @@ -1742,6 +1791,17 @@ static int dwc3_gadget_stop(struct usb_gadget *g) >> dwc->gadget_driver = NULL; >> spin_unlock_irqrestore(&dwc->lock, flags); >> >> + /* >> + * Since the xHCI

Re: [PATCH] perf jit: Fix build issue on Ubuntu

2016-10-13 Thread Stephane Eranian
On Wed, Oct 12, 2016 at 7:20 PM, Anton Blanchard wrote: > > From: Anton Blanchard > > When building on Ubuntu 16.04, I get the following error: > > Makefile:49: *** the openjdk development package appears to me missing, > install and try again. Stop. > > The problem is that update-java-alternat

Re: [RESEND PATCH v3 1/2] usb: dwc3: gadget: Add disconnect checking when changing function dynamically

2016-10-13 Thread Baolin Wang
Hi, On 13 October 2016 at 15:06, Felipe Balbi wrote: > > Hi, > > Baolin Wang writes: >> When system has stpped the gadget, we should avoid queuing any requests > > queueing is *not* a problem. Starting is. In fact, that's what your > patch is doing. OK. > >> which will cause tranfer failed. Th

Re: [PATCH 00/10] mm: adjust get_user_pages* functions to explicitly pass FOLL_* flags

2016-10-13 Thread Christian König
Am 13.10.2016 um 02:20 schrieb Lorenzo Stoakes: This patch series adjusts functions in the get_user_pages* family such that desired FOLL_* flags are passed as an argument rather than implied by flags. The purpose of this change is to make the use of FOLL_FORCE explicit so it is easier to grep fo

Re: [RESEND PATCH v3 2/2] usb: dwc3: Wait for control tranfer completed when stopping gadget

2016-10-13 Thread Baolin Wang
Hi, On 13 October 2016 at 15:08, Felipe Balbi wrote: > > Hi, > > Baolin Wang writes: >> @@ -1487,10 +1496,22 @@ static int dwc3_gadget_pullup(struct usb_gadget *g, >> int is_on) >> >> is_on = !!is_on; >> >> +try_again: >> spin_lock_irqsave(&dwc->lock, flags); >> ret = dwc3_gad

Re: [PATCH v2] usb: dwc3: gadget: Wait for end transfer complete before free irq

2016-10-13 Thread Felipe Balbi
Hi, Baolin Wang writes: > Hi, > > On 13 October 2016 at 15:02, Felipe Balbi wrote: >> >> Hi, >> >> Baolin Wang writes: >>> @@ -1742,6 +1791,17 @@ static int dwc3_gadget_stop(struct usb_gadget *g) >>> dwc->gadget_driver = NULL; >>> spin_unlock_irqrestore(&dwc->lock, flags); >>>

Re: [PATCH] mm/slab: fix kmemcg cache creation delayed issue

2016-10-13 Thread Joonsoo Kim
On Fri, Oct 07, 2016 at 10:20:55AM +0200, Michal Hocko wrote: > On Fri 07-10-16 14:14:01, Joonsoo Kim wrote: > > On Thu, Oct 06, 2016 at 09:02:00AM -0700, Doug Smythies wrote: > > > It was my (limited) understanding that the subsequent 2 patch set > > > superseded this patch. Indeed, the 2 patch se

Re: [RFC 4/4] mm, page_alloc: disallow migratetype fallback in fastpath

2016-10-13 Thread Joonsoo Kim
On Thu, Sep 29, 2016 at 11:05:48PM +0200, Vlastimil Babka wrote: > The previous patch has adjusted async compaction so that it helps against > longterm fragmentation when compacting for a non-MOVABLE high-order > allocation. > The goal of this patch is to force such allocations go through compacti

Re: [RESEND PATCH v3 2/2] usb: dwc3: Wait for control tranfer completed when stopping gadget

2016-10-13 Thread Felipe Balbi
Hi, Baolin Wang writes: > Hi, > > On 13 October 2016 at 15:08, Felipe Balbi wrote: >> >> Hi, >> >> Baolin Wang writes: >>> @@ -1487,10 +1496,22 @@ static int dwc3_gadget_pullup(struct usb_gadget *g, >>> int is_on) >>> >>> is_on = !!is_on; >>> >>> +try_again: >>> spin_lock_irqsave(

Re: [PATCH v2] usb: dwc3: gadget: Wait for end transfer complete before free irq

2016-10-13 Thread Baolin Wang
Hi, On 13 October 2016 at 15:51, Felipe Balbi wrote: > > Hi, > > Baolin Wang writes: >> Hi, >> >> On 13 October 2016 at 15:02, Felipe Balbi wrote: >>> >>> Hi, >>> >>> Baolin Wang writes: @@ -1742,6 +1791,17 @@ static int dwc3_gadget_stop(struct usb_gadget *g) dwc->gadget_driver

Re: [RESEND PATCH v3 2/2] usb: dwc3: Wait for control tranfer completed when stopping gadget

2016-10-13 Thread Baolin Wang
On 13 October 2016 at 15:54, Felipe Balbi wrote: > > Hi, > > Baolin Wang writes: >> Hi, >> >> On 13 October 2016 at 15:08, Felipe Balbi wrote: >>> >>> Hi, >>> >>> Baolin Wang writes: @@ -1487,10 +1496,22 @@ static int dwc3_gadget_pullup(struct usb_gadget *g, int is_on)

[patch] tipc: info leak in __tipc_nl_add_udp_addr()

2016-10-13 Thread Dan Carpenter
We should clear out the padding and unused struct members so that we don't expose stack information to userspace. Fixes: fdb3accc2c15 ('tipc: add the ability to get UDP options via netlink') Signed-off-by: Dan Carpenter --- This goes into the net tree. diff --git a/net/tipc/udp_media.c b/net/tip

[RFC PATCH 2/5] mm/page_alloc: use smallest fallback page first in movable allocation

2016-10-13 Thread js1304
From: Joonsoo Kim When we try to find freepage in fallback buddy list, we always serach the largest one. This would help for fragmentation if we process unmovable/reclaimable allocation request because it could cause permanent fragmentation on movable pageblock and spread out such allocations wou

[RFC PATCH 1/5] mm/page_alloc: always add freeing page at the tail of the buddy list

2016-10-13 Thread js1304
From: Joonsoo Kim Currently, freeing page can stay longer in the buddy list if next higher order page is in the buddy list in order to help coalescence. However, it doesn't work for the simplest sequential free case. For example, think about the situation that 8 consecutive pages are freed in seq

[RFC PATCH 0/5] Reduce fragmentation

2016-10-13 Thread js1304
From: Joonsoo Kim Hello, This is a patchset to reduce fragmentation. Patch 1 ~ 3 changes allocation/free logic to reduce fragmentation. Patch 4 ~ 5 is to manually control number of unmovable/reclaimable pageblock by user. Usually user has more knowledge about their system and if the number of un

[RFC PATCH 4/5] mm/page_alloc: add fixed migratetype pageblock infrastructure

2016-10-13 Thread js1304
From: Joonsoo Kim Following patch will support permanent migratetype pageblock by kernel boot parameter. For preparation, this patch adds infrastructure for it. Once fixed, migratetype cannot be changed anymore until power off. Signed-off-by: Joonsoo Kim --- include/linux/pageblock-flags.h |

[RFC PATCH 3/5] mm/page_alloc: stop instantly reusing freed page

2016-10-13 Thread js1304
From: Joonsoo Kim Allocation/free pattern is usually sequantial. If they are freed to the buddy list, they can be coalesced. However, we first keep these freed pages at the pcp list and try to reuse them until threshold is reached so we don't have enough chance to get a high order freepage. This

[RFC PATCH 5/5] mm/page_alloc: support fixed migratetype pageblock

2016-10-13 Thread js1304
From: Joonsoo Kim We have migratetype facility to minimise fragmentation. It dynamically changes migratetype of pageblock based on some criterias but it never be perfect. Some migratetype pages are often placed in the other migratetype pageblock. We call this pageblock as mixed pageblock. There

Re: [PATCH RFC v2 net-next 0/8] net: qualcomm: add QCA7000 UART driver

2016-10-13 Thread Stefan Wahren
> Stefan Wahren hat am 26. September 2016 um 13:07 > geschrieben: > > > The Qualcomm QCA7000 HomePlug GreenPHY supports two interfaces: > UART and SPI. This patch series adds the missing support for UART. > > This driver based on the Qualcomm code [1], but contains some changes: > * use random

Re: [PATCH 3.4 088/125] ser_gigaset: fix deallocation of platform device structure

2016-10-13 Thread Paul Bolle
On Thu, 2016-10-13 at 10:52 +0800, Zefan Li wrote: > On 2016/10/12 20:52, Paul Bolle wrote: > > There has been a follow up for this fix. I'll have to dive into my > > archive to see why that was needed. > > > > It was complicated, because there has been a short period in which this > > fix was cor

Re: EXT: pre 4.9-rc dma regression, imx6 sound etc

2016-10-13 Thread Mika Penttilä
On 10/13/2016 08:25 AM, Han, Nandor (GE Healthcare) wrote: > > > On 12/10/2016 16:25, Mika Penttilä wrote: >> Hi! >> >> Bisected that commit 5881826 - "dmaengine: imx-sdma - update the residue >> calculation for cyclic channels" is first bad commit to cause audio >> regression on imx6q, sgt

Re: [PATCH RFC v2 net-next 0/8] net: qualcomm: add QCA7000 UART driver

2016-10-13 Thread Greg Kroah-Hartman
On Thu, Oct 13, 2016 at 10:09:09AM +0200, Stefan Wahren wrote: > > > Stefan Wahren hat am 26. September 2016 um 13:07 > > geschrieben: > > > > > > The Qualcomm QCA7000 HomePlug GreenPHY supports two interfaces: > > UART and SPI. This patch series adds the missing support for UART. > > > > This

Re: [RESEND PATCH v3 1/2] usb: dwc3: gadget: Add disconnect checking when changing function dynamically

2016-10-13 Thread Janusz Dziedzic
On 13 October 2016 at 09:37, Baolin Wang wrote: > Hi, > > On 13 October 2016 at 15:06, Felipe Balbi wrote: >> >> Hi, >> >> Baolin Wang writes: >>> When system has stpped the gadget, we should avoid queuing any requests >> >> queueing is *not* a problem. Starting is. In fact, that's what your >>

Re: [PATCH v2 00/16] Convert FibreChannel bsg code to use bsg-lib

2016-10-13 Thread Johannes Thumshirn
On Wed, Oct 12, 2016 at 05:54:45PM +0200, Steffen Maier wrote: > Hi Johannes, > > On 10/12/2016 03:06 PM, Johannes Thumshirn wrote: > > This series converts the current bsg usage in the FibreChannel drivers over > > to use bsg-lib. SAS will follow once FC is in a good enough shape. > > > > I did

Re: [PATCH v2] mm: exclude isolated non-lru pages from NR_ISOLATED_ANON or NR_ISOLATED_FILE.

2016-10-13 Thread Michal Hocko
On Thu 13-10-16 14:39:09, ming.ling wrote: > From: Ming Ling > > Non-lru pages don't belong to any lru, so counting them to > NR_ISOLATED_ANON or NR_ISOLATED_FILE doesn't make any sense. > It may misguide functions such as pgdat_reclaimable_pages and > too_many_isolated. That doesn't make much s

Re: [RESEND PATCH v3 1/2] usb: dwc3: gadget: Add disconnect checking when changing function dynamically

2016-10-13 Thread Baolin Wang
Hi, On 13 October 2016 at 16:16, Janusz Dziedzic wrote: > On 13 October 2016 at 09:37, Baolin Wang wrote: >> Hi, >> >> On 13 October 2016 at 15:06, Felipe Balbi wrote: >>> >>> Hi, >>> >>> Baolin Wang writes: When system has stpped the gadget, we should avoid queuing any requests >>> >>> q

Re: [PATCH 2/3] irqchip: mtk-cirq: Add mediatek mtk-cirq implement

2016-10-13 Thread Marc Zyngier
On Thu, 13 Oct 2016 13:06:33 +0800 Youlin Pei wrote: > This commit add the mtk-cirq implement for mt2701. Can you please expand this a bit? > > Signed-off-by: Youlin Pei > --- > drivers/irqchip/Makefile |2 +- > drivers/irqchip/irq-mtk-cirq.c | 257 > +

Re: [Xen-devel] [RFC KERNEL PATCH 0/2] Add Dom0 NVDIMM support for Xen

2016-10-13 Thread Jan Beulich
>>> On 12.10.16 at 18:19, wrote: > On Wed, Oct 12, 2016 at 9:01 AM, Jan Beulich wrote: > On 12.10.16 at 17:42, wrote: >>> On Wed, Oct 12, 2016 at 8:39 AM, Jan Beulich wrote: >>> On 12.10.16 at 16:58, wrote: > On 10/12/16 05:32 -0600, Jan Beulich wrote: > On 12.10.16 at 12:3

Re: [PATCH] Don't touch single threaded PTEs which are on the right node

2016-10-13 Thread Mel Gorman
On Wed, Oct 12, 2016 at 09:15:49AM -0700, Andi Kleen wrote: > From: Andi Kleen > > We had some problems with pages getting unmapped in single threaded > affinitized processes. It was tracked down to NUMA scanning. > > In this case it doesn't make any sense to unmap pages if the > process is sing

Rewording language in mbind(2) to "threads" not "processes"

2016-10-13 Thread Michael Kerrisk (man-pages)
Christoph, Piotr, and Brice Since you (Christoph and Piotr) helped with documenting MPOL_LOCAL just recently, I wonder if I might ask you to review a patch that I propose for the mbind(2) manual page. As far as I understand, memory policy, as set by set_mempolicy(2) is a per-thread attribute. T

Re: [RESEND PATCH v3 1/2] usb: dwc3: gadget: Add disconnect checking when changing function dynamically

2016-10-13 Thread Baolin Wang
On 13 October 2016 at 16:28, Janusz Dziedzic wrote: > On 13 October 2016 at 10:21, Baolin Wang wrote: >> Hi, >> >> On 13 October 2016 at 16:16, Janusz Dziedzic >> wrote: >>> On 13 October 2016 at 09:37, Baolin Wang wrote: Hi, On 13 October 2016 at 15:06, Felipe Balbi wrote: >>>

[RFC PATCH] perf: honouring the cpuid for number of fixed counters in hypervisors

2016-10-13 Thread Imre Palik
From: Imre Palik perf doesn't seem to honour the number of fixed counters specified by cpuid leaf 0xa. It always assume that Intel CPUs have at least 3 fixed counters. So if some of the fixed counters are masked out by the hypervisor, it still tries to check/set them. This patch makes perf beh

Re: [PATCH V3 02/10] ras: acpi/apei: cper: generic error data entry v3 per ACPI 6.1

2016-10-13 Thread Suzuki K Poulose
On 12/10/16 23:10, Baicar, Tyler wrote: Hello Suzuki, Thank you for the feedback! Responses below. On 10/11/2016 11:28 AM, Suzuki K Poulose wrote: On 07/10/16 22:31, Tyler Baicar wrote: Currently when a RAS error is reported it is not timestamped. The ACPI 6.1 spec adds the timestamp field t

RE: [PATCH 2/2] MIPS: DTS: img: add device tree for Marduk board

2016-10-13 Thread James Hartley
Hi Rahul, > -Original Message- > From: Rahul Bedarkar > Sent: 13 October 2016 08:02 > To: Rob Herring > Cc: Ralf Baechle; Mark Rutland; James Hartley; linux-m...@linux-mips.org; > devicet...@vger.kernel.org; linux-kernel@vger.kernel.org > Subject: Re: [PATCH 2/2] MIPS: DTS: img: add device

[patch] ASoC: rt5663: fix a debug statement

2016-10-13 Thread Dan Carpenter
We increment "i" before printing the debug statement. That makes it the wrong sleep_time[] information and Smatch complains that the last increment could be beyond the end of the array. Signed-off-by: Dan Carpenter diff --git a/sound/soc/codecs/rt5663.c b/sound/soc/codecs/rt5663.c index 01a18d8

Re: [PATCH 3.4 088/125] ser_gigaset: fix deallocation of platform device structure

2016-10-13 Thread Zefan Li
On 2016/10/13 16:11, Paul Bolle wrote: > On Thu, 2016-10-13 at 10:52 +0800, Zefan Li wrote: >> On 2016/10/12 20:52, Paul Bolle wrote: >>> There has been a follow up for this fix. I'll have to dive into my >>> archive to see why that was needed. >>> >>> It was complicated, because there has been a s

Re: [PATCH v3 1/2] phy: Add a driver for simple phy

2016-10-13 Thread Vivek Gautam
On 10/13/2016 03:47 AM, Alban wrote: On Fri, 7 Oct 2016 13:57:08 +0530 Vivek Gautam wrote: Hi Alban, On Wed, Feb 17, 2016 at 7:51 PM, Alban wrote: On Mon, 1 Feb 2016 17:12:42 +0530 Kishon Vijay Abraham I wrote: Hi, On Friday 29 January 2016 01:22 AM, Alban Bedel wrote: This driver i

[patch] relay: check array offset before using it

2016-10-13 Thread Dan Carpenter
Smatch complains that we started using the array offset before we checked that it was valid. Fixes: 017c59c042d0 ('relay: Use per CPU constructs for the relay channel buffer pointers') Signed-off-by: Dan Carpenter diff --git a/kernel/relay.c b/kernel/relay.c index da79a10..8f18d31 100644 --- a/

Re: [PATCH] kexec: Export memory sections virtual addresses to vmcoreinfo

2016-10-13 Thread Baoquan He
Hi Pratyush, On 10/12/16 at 02:39pm, Pratyush Anand wrote: > > > On Wednesday 12 October 2016 05:56 AM, Baoquan He wrote: > > > PAGE_OFFSET can be get via vaddr - paddr from elf pt_loads so only > > > > VMALLOC_BASE and VMEMMAP_BASE is necessary.. > > Well, yes, I was wrong. I wrongly thought of

[patch] drm/i915: fix a read size argument

2016-10-13 Thread Dan Carpenter
We want to read 3 bytes here, but because the parenthesis are in the wrong place we instead read: sizeof(intel_dp->edp_dpcd) == sizeof(intel_dp->edp_dpcd) which is one byte. Fixes: fe5a66f91c88 ("drm/i915: Read PSR caps/intermediate freqs/etc. only once on eDP") Signed-off-by: Dan Carpe

Re: [Xen-devel] [RFC KERNEL PATCH 0/2] Add Dom0 NVDIMM support for Xen

2016-10-13 Thread Haozhong Zhang
On 10/13/16 02:34 -0600, Jan Beulich wrote: On 12.10.16 at 18:19, wrote: On Wed, Oct 12, 2016 at 9:01 AM, Jan Beulich wrote: On 12.10.16 at 17:42, wrote: On Wed, Oct 12, 2016 at 8:39 AM, Jan Beulich wrote: On 12.10.16 at 16:58, wrote: On 10/12/16 05:32 -0600, Jan Beulich wrote: On 12.1

[patch] zfcp: spin_lock_irqsave() is not nestable

2016-10-13 Thread Dan Carpenter
We accidentally overwrite the original saved value of "flags" so that we can't re-enable IRQs at the end of the function. Presumably this function is mostly called with IRQs disabled or it would be obvious in testing. Fixes: aceeffbb59bb ("zfcp: trace full payload of all SAN records (req,resp,ie

[PATCH 1/2] kernel/smp: Make the SMP boot message common on all arches

2016-10-13 Thread Michael Ellerman
Currently after bringing up secondary CPUs all arches print "Brought up %d CPUs". On x86 they also print the number of nodes that were brought online. It would be nice to also print the number of nodes on other arches. Although we could override smp_announce() on the other ~10 NUMA aware arches, i

[PATCH 2/2] kernel/smp: Tell the user we're bringing up secondary CPUs

2016-10-13 Thread Michael Ellerman
Currently we don't print anything before starting to bring up secondary CPUs. This can be confusing if it takes a long time to bring up the secondaries, or if the kernel crashes while doing so and produces no further output. On x86 they work around this by detecting when the first secondary CPU co

Re: [PATCH v7 2/8] power: add power sequence library

2016-10-13 Thread Peter Chen
On Thu, Oct 13, 2016 at 09:04:42AM +0200, Heiko Stuebner wrote: > > > > +static int __init pwrseq_generic_register(void) > > > > +{ > > > > + struct pwrseq_generic *pwrseq_gen; > > > > + int i; > > > > + > > > > + for (i = 0; i < CONFIG_PWRSEQ_GENERIC_INSTANCE_NUMBER; i++) { > > >

Re: [Intel-gfx] [patch] drm/i915: fix a read size argument

2016-10-13 Thread Chris Wilson
On Thu, Oct 13, 2016 at 11:55:08AM +0300, Dan Carpenter wrote: > We want to read 3 bytes here, but because the parenthesis are in the > wrong place we instead read: > > sizeof(intel_dp->edp_dpcd) == sizeof(intel_dp->edp_dpcd) > > which is one byte. > > Fixes: fe5a66f91c88 ("drm/i915: Read

Re: [Xen-devel] [RFC KERNEL PATCH 0/2] Add Dom0 NVDIMM support for Xen

2016-10-13 Thread Jan Beulich
>>> On 13.10.16 at 10:53, wrote: > On 10/13/16 02:34 -0600, Jan Beulich wrote: > On 12.10.16 at 18:19, wrote: >>> On Wed, Oct 12, 2016 at 9:01 AM, Jan Beulich wrote: >>> On 12.10.16 at 17:42, wrote: > On Wed, Oct 12, 2016 at 8:39 AM, Jan Beulich wrote: > On 12.10.16 at 16:5

Re: [patch] drm/i915: fix a read size argument

2016-10-13 Thread Eric Engestrom
On Thu, Oct 13, 2016 at 11:55:08AM +0300, Dan Carpenter wrote: > We want to read 3 bytes here, but because the parenthesis are in the > wrong place we instead read: > > sizeof(intel_dp->edp_dpcd) == sizeof(intel_dp->edp_dpcd) > > which is one byte. > > Fixes: fe5a66f91c88 ("drm/i915: Read

Re: [Xen-devel] [RFC KERNEL PATCH 0/2] Add Dom0 NVDIMM support for Xen

2016-10-13 Thread Haozhong Zhang
+Dan Williams I accidentally dropped him in my last reply. Add him back. On 10/13/16 16:53 +0800, Haozhong Zhang wrote: On 10/13/16 02:34 -0600, Jan Beulich wrote: On 12.10.16 at 18:19, wrote: On Wed, Oct 12, 2016 at 9:01 AM, Jan Beulich wrote: On 12.10.16 at 17:42, wrote: On Wed, Oct 12

Re: [PATCH] qede: fix CONFIG_INFINIBAND_QEDR=m build error

2016-10-13 Thread Arnd Bergmann
On Thursday, October 13, 2016 8:50:21 AM CEST Mintz, Yuval wrote: > > config INFINIBAND_QEDR > > - tristate "QLogic qede RoCE sources [debug]" > > + bool "QLogic qede RoCE sources [debug]" > > Given that the qedr submission is going to turn this back into a tristate, > are you certain this is

Re: [RFC PATCH 2/5] mm/page_alloc: use smallest fallback page first in movable allocation

2016-10-13 Thread Vlastimil Babka
On 10/13/2016 10:08 AM, js1...@gmail.com wrote: From: Joonsoo Kim When we try to find freepage in fallback buddy list, we always serach the largest one. This would help for fragmentation if we process unmovable/reclaimable allocation request because it could cause permanent fragmentation on mov

Re: [PATCH v2 01/16] scsi: Get rid of struct fc_bsg_buffer

2016-10-13 Thread Hannes Reinecke
On 10/12/2016 03:06 PM, Johannes Thumshirn wrote: > struct fc_bsg_buffer is just a clone of struct bsg_buffer from bsg-lib, > so use this one instead. > > Signed-off-by: Johannes Thumshirn > --- > drivers/scsi/lpfc/lpfc_bsg.c | 3 ++- > drivers/scsi/scsi_transport_fc.c | 2 +- > include/sc

RE: [patch] ASoC: rt5663: fix a debug statement

2016-10-13 Thread Bard Liao
> -Original Message- > From: Dan Carpenter [mailto:dan.carpen...@oracle.com] > Sent: Thursday, October 13, 2016 4:47 PM > To: Bard Liao > Cc: Oder Chiou; Liam Girdwood; Mark Brown; Jaroslav Kysela; Takashi Iwai; > alsa-de...@alsa-project.org; linux-kernel@vger.kernel.org; > kernel-janit...@

Re: [RESEND PATCH v3 1/2] usb: dwc3: gadget: Add disconnect checking when changing function dynamically

2016-10-13 Thread Janusz Dziedzic
On 13 October 2016 at 10:21, Baolin Wang wrote: > Hi, > > On 13 October 2016 at 16:16, Janusz Dziedzic > wrote: >> On 13 October 2016 at 09:37, Baolin Wang wrote: >>> Hi, >>> >>> On 13 October 2016 at 15:06, Felipe Balbi wrote: Hi, Baolin Wang writes: > When system has

Re: [PATCH] iwlwifi: pcie: reduce "unsupported splx" to a warning

2016-10-13 Thread Luca Coelho
On Wed, 2016-10-12 at 12:50 -0500, Chris Rorvick wrote: > Hi Luca, > > FYI, It seems that Google does not like your email as I'm not > receiving any of your messages in gmail. Some responses below: That's odd. It works for me. Replied privately to try to sort this out. > On Wed, 2016-10-12 a

Re: [RFC PATCH 1/5] mm/page_alloc: always add freeing page at the tail of the buddy list

2016-10-13 Thread Vlastimil Babka
On 10/13/2016 10:08 AM, js1...@gmail.com wrote: From: Joonsoo Kim Currently, freeing page can stay longer in the buddy list if next higher order page is in the buddy list in order to help coalescence. However, it doesn't work for the simplest sequential free case. For example, think about the s

Re: linux.git: printk() problem

2016-10-13 Thread Joe Perches
On Thu, 2016-10-13 at 08:26 +0200, Michal Hocko wrote: > On Wed 12-10-16 09:08:58, Joe Perches wrote: > > On Wed, 2016-10-12 at 16:35 +0200, Michal Hocko wrote: > > > On Wed 12-10-16 22:30:03, Tetsuo Handa wrote: > > > > Hello. > > > > > > > > I noticed that current linux.git generates hardly read

[PATCH 0/2] Tiny modification of i2c-hid

2016-10-13 Thread Benjamin Tissoires
Hi Jiri, David and I are facing an issue in RHEL with the HP Zbook 15 Studio mWS. This laptops uses the pinctrl-sunrisepoint controller for the GPIOs and it failed on RHEL. We found out what the issue was, but in the meantime realized that part of the code we have in i2c-hid is not required anymor

[PATCH 1/2] Revert "HID: i2c-hid: Add support for ACPI GPIO interrupts"

2016-10-13 Thread Benjamin Tissoires
From: David Arcari This reverts commit a485923efbb8 ("HID: i2c-hid: Add support for ACPI GPIO interrupts") and commit a7d2bf25a483 ("HID: i2c-hid: Do not fail probing if gpiolib is not enabled") at the same time. Since commit c884fbd45214 ("gpio / ACPI: Add support for retrieving GpioInt resourc

[PATCH 2/2] HID: i2c-hid: exit if the IRQ is not valid

2016-10-13 Thread Benjamin Tissoires
From: David Arcari When i2c-core doesn't find the IRQ associated to the GPIO because the gpiochip is not available, it assigns -EPROBE_DEFER to the irq. We need to bail out there and on any other error in an IRQ. Signed-off-by: David Arcari Signed-off-by: Benjamin Tissoires --- drivers/hid/i2

RE: [PATCH] qede: fix CONFIG_INFINIBAND_QEDR=m build error

2016-10-13 Thread Mintz, Yuval
> > > - if (cond) > > > + if (IS_ENABLED(CONFIG_INFINIBAND_QEDR) && cond) > > > qed_rdma_dpm_bar(p_hwfn, p_ptt); > > > > Why not simply fix the qed_roce.h empty implementation? > > Mainly for consistency: we have a couple of interfaces that are called from > the

Re: [PATCH v3 03/11] tracing/syscalls: add compat syscall metadata

2016-10-13 Thread Michael Ellerman
Marcin Nowakowski writes: > On 12.10.2016 10:50, Michael Ellerman wrote: >> <...> >> It's annoying that we have to duplicate all that just to do a + 1. >> >> How about this as a precursor? > > <...> > > Thanks for the suggestion - unless anyone sees a reason to keep the > current solution I'll

Re: [patch] drm/i915: fix a read size argument

2016-10-13 Thread walter harms
Am 13.10.2016 10:55, schrieb Dan Carpenter: > We want to read 3 bytes here, but because the parenthesis are in the > wrong place we instead read: > > sizeof(intel_dp->edp_dpcd) == sizeof(intel_dp->edp_dpcd) > > which is one byte. > > Fixes: fe5a66f91c88 ("drm/i915: Read PSR caps/intermed

[PATCH] drm: atomic: Clarify documentation around drm_atomic_crtc_needs_modeset

2016-10-13 Thread Brian Starkey
Add some additional comments to more explicitly describe the meaning and usage of the three CRTC modeset detection booleans: mode_changed, connectors_changed and active_changed. Suggested-by: Daniel Vetter Signed-off-by: Brian Starkey --- Hi Daniel, I guess I asked for this one :-), please jus

Re: [RESEND PATCH v3 1/2] usb: dwc3: gadget: Add disconnect checking when changing function dynamically

2016-10-13 Thread Felipe Balbi
Hi, Baolin Wang writes: >>> diff --git a/drivers/usb/dwc3/gadget.c b/drivers/usb/dwc3/gadget.c >>> index 1783406..ca2ae5b 100644 >>> --- a/drivers/usb/dwc3/gadget.c >>> +++ b/drivers/usb/dwc3/gadget.c >>> @@ -241,6 +241,9 @@ int dwc3_send_gadget_ep_cmd(struct dwc3_ep *dep, >

Re: [Intel-gfx] [patch] drm/i915: fix a read size argument

2016-10-13 Thread Dan Carpenter
On Thu, Oct 13, 2016 at 10:01:03AM +0100, Chris Wilson wrote: > On Thu, Oct 13, 2016 at 11:55:08AM +0300, Dan Carpenter wrote: > > We want to read 3 bytes here, but because the parenthesis are in the > > wrong place we instead read: > > > > sizeof(intel_dp->edp_dpcd) == sizeof(intel_dp->edp_dp

Re: [PATCH v2] usb: dwc3: gadget: Wait for end transfer complete before free irq

2016-10-13 Thread Felipe Balbi
Hi, Baolin Wang writes: I'm thinking this is a bug in configfs interface of Gadget API, not dwc3. The only reason for this to happen would be if we get to ->udc_stop() with endpoints still enabled. Can you check if that's the case? i.e. can you check if any endpoints >>>

Re: [PATCH 1/2] debugfs: improve DEFINE_DEBUGFS_ATTRIBUTE for !CONFIG_DEBUGFS_FS

2016-10-13 Thread Nicolai Stange
Hi Arnd, thanks for this (and sorry for the late reply)! Arnd Bergmann writes: > The slp_s0_residency_usec debugfs file currently uses > DEFINE_DEBUGFS_ATTRIBUTE(), but that macro cannot really be used to > define files outside of the debugfs code, as it has no reference to > the get/set functi

Re: linux.git: printk() problem

2016-10-13 Thread Michal Hocko
On Thu 13-10-16 02:29:46, Joe Perches wrote: > On Thu, 2016-10-13 at 08:26 +0200, Michal Hocko wrote: [...] > > I think they are not critical and can be fix once somebody notices. > > As do I, but Linus objected to applying a patch when Colin Ian King > noticed one. > > I think the 250,000 or so

Re: [RFC PATCH] mm, compaction: allow compaction for GFP_NOFS requests

2016-10-13 Thread Michal Hocko
On Thu 13-10-16 11:29:24, Dave Chinner wrote: > On Fri, Oct 07, 2016 at 03:18:14PM +0200, Michal Hocko wrote: [...] > > Unpatched kernel: > > # Version 3.3, 16 thread(s) starting at Fri Oct 7 09:55:05 2016 > > # Sync method: NO SYNC: Test does not issue sync() or fsync() calls. > > #

[GIT PULL] Please pull powerpc/linux.git powerpc-4.9-2 tag

2016-10-13 Thread Michael Ellerman
Hi Linus, Please pull some more powerpc updates for 4.9: The following changes since commit b7b7013cac55d794940bd9cb7b7c55c9dececac4: powerpc/bpf: Add support for bpf constant blinding (2016-10-04 20:33:20 +1100) are available in the git repository at: git://git.kernel.org/pub/scm/linux/ke

Re: MPOL_BIND on memory only nodes

2016-10-13 Thread Michal Hocko
On Thu 13-10-16 15:24:54, Anshuman Khandual wrote: [...] > Which makes the function look like this. Even with these changes, MPOL_BIND is > still going to pick up the local node's zonelist instead of the first node in > policy->v.nodes nodemask. It completely ignores policy->v.nodes which it > sho

[PATCH] regmap: include from include/linux/regmap.h

2016-10-13 Thread Masahiro Yamada
The readx_poll_timeout() macro calls usleep_range(), which is declared in . Make include/linux/regmap.h include , like include/linux/iopoll.h does. Otherwise, users of the macro will see "implicit declaration of function 'usleep_range'" error. Signed-off-by: Masahiro Yamada --- include/linux/

Re: linux.git: printk() problem

2016-10-13 Thread Joe Perches
On Thu, 2016-10-13 at 12:04 +0200, Michal Hocko wrote: > On Thu 13-10-16 02:29:46, Joe Perches wrote: > > On Thu, 2016-10-13 at 08:26 +0200, Michal Hocko wrote: > > [...] > > > I think they are not critical and can be fix once somebody notices. > > > > > > As do I, but Linus objected to applyin

Re: [PATCH] qede: fix CONFIG_INFINIBAND_QEDR=m build error

2016-10-13 Thread Arnd Bergmann
On Thursday, October 13, 2016 9:34:41 AM CEST Mintz, Yuval wrote: > > > > > - if (cond) > > > > + if (IS_ENABLED(CONFIG_INFINIBAND_QEDR) && cond) > > > > qed_rdma_dpm_bar(p_hwfn, p_ptt); > > > > > > Why not simply fix the qed_roce.h empty implementation?

Re: [PATCH v2 1/2] clocksource: arm_arch_timer: Don't assume clock runs in suspend

2016-10-13 Thread Marc Zyngier
On Tue, 4 Oct 2016 11:12:09 -0700 Brian Norris wrote: > The ARM ARM specifies that the system counter "must be implemented in an > always-on power domain," and so we try to use the counter as a source of > timekeeping across suspend/resume. Unfortunately, some SoCs (e.g., > Rockchip's RK3399) do

Re: [PATCH 3/4] arm64: Allow hw watchpoint of length 3,5,6 and 7

2016-10-13 Thread Pratyush Anand
On Wednesday 12 October 2016 04:46 PM, Yao Qi wrote: On Wed, Oct 12, 2016 at 6:58 AM, Pratyush Anand wrote: Since, arm64 can support all offset within a double word limit. Therefore, now support other lengths within that range as well. How does ptracer (like GDB) detect kernel has already s

Re: [PATCH RESEND v3] dma: omap-dma: add support for pause of non-cyclic transfers

2016-10-13 Thread Peter Ujfalusi
On 10/10/16 17:12, Peter Ujfalusi wrote: > On 10/10/16 15:07, Vignesh R wrote: >> From: Sebastian Andrzej Siewior >> >> This DMA driver is used by 8250-omap on DRA7-evm. There is one >> requirement that is to pause a transfer. This is currently used on the RX >> side. It is possible that the UART

[PATCH] iwlwifi: pcie: fix SPLC structure parsing

2016-10-13 Thread Luca Coelho
From: Luca Coelho The SPLC data parsing is too restrictive and was not trying find the correct element for WiFi. This causes problems with some BIOSes where the SPLC method exists, but doesn't have a WiFi entry on the first element of the list. The domain type values are also incorrect accordin

Re: [PATCH 2/2] HID: i2c-hid: exit if the IRQ is not valid

2016-10-13 Thread Mika Westerberg
On Thu, Oct 13, 2016 at 11:30:45AM +0200, Benjamin Tissoires wrote: > From: David Arcari > > When i2c-core doesn't find the IRQ associated to the GPIO because > the gpiochip is not available, it assigns -EPROBE_DEFER to the irq. > We need to bail out there and on any other error in an IRQ. > > S

Re: [PATCH 1/2] Revert "HID: i2c-hid: Add support for ACPI GPIO interrupts"

2016-10-13 Thread Mika Westerberg
On Thu, Oct 13, 2016 at 11:30:44AM +0200, Benjamin Tissoires wrote: > From: David Arcari > > This reverts commit a485923efbb8 ("HID: i2c-hid: Add support for ACPI > GPIO interrupts") and commit a7d2bf25a483 ("HID: i2c-hid: Do not fail > probing if gpiolib is not enabled") at the same time. > > S

Re: [PATCH 3/7] f2fs: rename free nid cache operation

2016-10-13 Thread Chao Yu
On 2016/10/13 1:15, Jaegeuk Kim wrote: > Then, how about this one? > >>From 4f4b48de34ffaf94baa2065bcd5d44566f8d25b9 Mon Sep 17 00:00:00 2001 > From: Jaegeuk Kim > Date: Wed, 12 Oct 2016 10:09:59 -0700 > Subject: [PATCH] f2fs: clean up free nid list operations > > This patch cleans up to use con

[PATCH] Coccinelle: misc: Improve the script for more accurate results

2016-10-13 Thread Vaishali Thakkar
Currently because of the left associativity of the operators, pattern IRQF_ONESHOT | flags does not match with the pattern when we have more than one flag after the disjunction. This eventually results in giving false positives by the script. The patch eliminates these FPs by improving the rule. A

Re: [PATCH 1/8] f2fs: clear nlink if fail to add_link

2016-10-13 Thread Chao Yu
On 2016/10/13 1:22, Jaegeuk Kim wrote: > No, it happened xfstests/027 under encrypted mode only. Got it. :) Thanks,

Re: [PATCH] f2fs: fix wrong sum_page pointer in f2fs_gc

2016-10-13 Thread Chao Yu
Hi Jaegeuk, On 2016/10/13 7:23, Jaegeuk Kim wrote: > This patch fixes using a wrong pointer for sum_page in f2fs_gc. > > Signed-off-by: Jaegeuk Kim > --- > fs/f2fs/gc.c | 10 +- > 1 file changed, 5 insertions(+), 5 deletions(-) > > diff --git a/fs/f2fs/gc.c b/fs/f2fs/gc.c > index e4814

Re: [PATCH 1/2] debugfs: improve DEFINE_DEBUGFS_ATTRIBUTE for !CONFIG_DEBUGFS_FS

2016-10-13 Thread Arnd Bergmann
On Thursday, October 13, 2016 11:59:54 AM CEST Nicolai Stange wrote: > > > > +ssize_t debugfs_attr_read(struct file *file, char __user *buf, > > + size_t len, loff_t *ppos); > > +ssize_t debugfs_attr_write(struct file *file, const char __user *buf, > > + si

  1   2   3   4   5   6   7   8   >