[PATCH 5/6] mmc: rtsx_usb_sdmmc: Enable runtime PM autosuspend

2016-10-13 Thread Ulf Hansson
Enable runtime PM autosuspend for the rtsx_usb_sdmmc driver to avoid the device being runtime suspended and runtime resumed between each request. Let's use a default timeout of 50ms, to be consistent with other mmc hosts. Cc: Ritesh Raj Sarraf Cc: Alan Stern Signed-off-by: Ulf Hansson ---

[PATCH 3/6] memstick: rtsx_usb_ms: Runtime resume the device when polling for cards

2016-10-13 Thread Ulf Hansson
From: Alan Stern Accesses to the rtsx usb device, which is the parent of the rtsx memstick device, must not be done unless it's runtime resumed. Therefore when the rtsx_usb_ms driver polls for inserted memstick cards, let's add pm_runtime_get|put*() to make sure accesses is done when the rtsx

Re: [PATCH 0/5] i2c: i2c-piix4: set of coding style fixes

2016-10-13 Thread Jean Delvare
Hi Carlos, On Tue, 11 Oct 2016 18:24:36 +0100, Carlos Palminha wrote: > This patchset fixes several coding style issues. > > Carlos Palminha (5): > i2c: i2c-piix4: coding style fix - '*' adjacent to data name > i2c: i2c-piix4: coding style fix - assignment in if condition > i2c: i2c-piix4:

Re: [PATCH 2/2] perf, tools: Add documentation for perf user event parameters

2016-10-13 Thread Jiri Olsa
On Wed, Oct 12, 2016 at 02:02:07PM -0700, Andi Kleen wrote: > From: Andi Kleen > > There was no proper documention for the user parameters for > events that the perf parser implements. Document them all > in the perf list manpage. > > Signed-off-by: Andi Kleen > --- >

Re: [PATCH 1/2] perf, tools: Implement branch_type event parameter

2016-10-13 Thread Jiri Olsa
On Wed, Oct 12, 2016 at 02:02:06PM -0700, Andi Kleen wrote: > From: Andi Kleen > > It can be useful to specify branch type state per event, for example > if we want to collect both software trace points and last branch PMU > events in a single collection. Currently this doesn't work because >

Re: [PATCH v2 08/16] scsi: fc: implement kref backed reference counting

2016-10-13 Thread Hannes Reinecke
On 10/12/2016 03:06 PM, Johannes Thumshirn wrote: > Implement kref backed reference counting instead of rolling our own. This > elimnates the need of the following fields in 'struct fc_bsg_job': > * ref_cnt > * state_flags > * job_lock > bringing us close to unification of 'struct fc_bsg_job' and

Re: [PATCH v2 09/16] block: add reference counting for struct bsg_job

2016-10-13 Thread Hannes Reinecke
On 10/12/2016 03:06 PM, Johannes Thumshirn wrote: > Add reference counting to 'struct bsg_job' so we can implement a reuqest > timeout handler for bsg_jobs, which is needed for Fibre Channel. > > Signed-off-by: Johannes Thumshirn > --- > block/bsg-lib.c | 7 +-- >

Re: [PATCHv3 17/41] filemap: handle huge pages in filemap_fdatawait_range()

2016-10-13 Thread Jan Kara
On Thu 15-09-16 14:54:59, Kirill A. Shutemov wrote: > We writeback whole huge page a time. This is one of the things I don't understand. Firstly I didn't see where changes of writeback like this would happen (maybe they come later). Secondly I'm not sure why e.g. writeback should behave

Re: [PATCH 3/5] i2c: i2c-piix4: coding style fix - long comments blocks

2016-10-13 Thread Jean Delvare
Hi Carlos, On Tue, 11 Oct 2016 18:24:39 +0100, Carlos Palminha wrote: > Signed-off-by: Carlos Palminha > --- > drivers/i2c/busses/i2c-piix4.c | 94 > -- > 1 file changed, 54 insertions(+), 40 deletions(-) > > diff --git a/drivers/i2c/busses/i2c-piix4.c

Re: [PATCH v2 10/16] scsi: change FC drivers to use 'struct bsg_job'

2016-10-13 Thread Hannes Reinecke
On 10/12/2016 03:06 PM, Johannes Thumshirn wrote: > Change FC drivers to use 'struct bsg_job' from bsg-lib.h instead of 'struct > fc_bsg_job' from scsi_transport_fc.h and remove 'struct fc_bsg_job'. > > Signed-off-by: Johannes Thumshirn > --- > drivers/s390/scsi/zfcp_ext.h | 4 +-- >

Re: [PATCH v2 11/16] scsi: fc: Use bsg_destroy_job

2016-10-13 Thread Hannes Reinecke
On 10/12/2016 03:06 PM, Johannes Thumshirn wrote: > fc_destroy_bsgjob() and bsg_destroy_job() are now 1:1 copies, so use the > later. > > Signed-off-by: Johannes Thumshirn > --- > block/bsg-lib.c | 3 ++- > drivers/scsi/scsi_transport_fc.c | 20 ++-- >

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

2016-10-13 Thread Vlastimil Babka
On 10/13/2016 09:58 AM, Joonsoo Kim wrote: 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

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

2016-10-13 Thread Baolin Wang
Hi Felipe, On 13 October 2016 at 19:22, Felipe Balbi wrote: > > Hi, > > Janusz Dziedzic writes: 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

[PATCH v2] Input: elan_i2c - always output the device information

2016-10-13 Thread Benjamin Tissoires
it's always easier to retrieve these information in bug reports when it is always printed in the dmesg. Signed-off-by: Benjamin Tissoires --- changes in v2: - split the version information message and the touchpad characteristics - keep sending the versions as info (on one line) - send the

Re: [PATCH v2 14/16] block: add bsg_job_put() and bsg_job_get()

2016-10-13 Thread Hannes Reinecke
On 10/12/2016 03:06 PM, Johannes Thumshirn wrote: > Add bsg_job_put() and bsg_job_get() so don't need to export > bsg_destroy_job() any more. > > Signed-off-by: Johannes Thumshirn > --- > block/bsg-lib.c | 17 ++--- > drivers/scsi/scsi_transport_fc.c | 4 ++-- >

Re: [PATCH v2 13/16] scsi: fc: use bsg_job_done

2016-10-13 Thread Hannes Reinecke
On 10/12/2016 03:06 PM, Johannes Thumshirn wrote: > fc_bsg_jobdone() and bsg_job_done() are 1:1 copies now so use the bsg-lib one > instead of the FC private implementation. > > Signed-off-by: Johannes Thumshirn > --- > drivers/s390/scsi/zfcp_fc.c | 2 +- > drivers/scsi/bfa/bfad_bsg.c

Re: [PATCH RFC V2 2/2] KVM: x86: Support using the VMX preemption timer for APIC Timer periodic/oneshot mode

2016-10-13 Thread Wanpeng Li
2016-10-13 19:38 GMT+08:00 Wanpeng Li : > 2016-10-13 1:41 GMT+08:00 Radim Krčmář : >> 2016-10-12 14:52+0800, Wanpeng Li: [...] >>> @@ -1711,8 +1753,7 @@ u64 kvm_get_lapic_tscdeadline_msr(struct kvm_vcpu >>> *vcpu) >>> { >>> struct kvm_lapic *apic = vcpu->arch.apic; >>> >>> - if

Re: [PATCH] iwlwifi: pcie: fix SPLC structure parsing

2016-10-13 Thread Luca Coelho
On Thu, 2016-10-13 at 13:27 +0200, Paul Bolle wrote: > Luca, > > On Thu, 2016-10-13 at 13:21 +0300, Luca Coelho wrote: > > Could you please give this a spin? I have tested it with some handmade > > ACPI tables in QEMU and it seems to work fine now. > > > Tested-by: Paul Bolle > > Not that

[PATCH] staging:vt6656:baseband.h: fix function definition argument without identifier name issue

2016-10-13 Thread Mikhail Golubev
Function definitions arguments should also have an identifier name as reported by checkpatch.pl. Signed-off-by: Mikhail Golubev --- drivers/staging/vt6656/baseband.h | 20 ++-- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/drivers/staging/vt6656/baseband.h

[PATCH] virtio-gpu: fix vblank events

2016-10-13 Thread Gerd Hoffmann
virtio-gpu sends vblank events in virtio_gpu_crtc_atomic_flush, and because of that it must be called for disabled planes too. Ask drm_atomic_helper_commit_planes to do that. Signed-off-by: Gerd Hoffmann --- drivers/gpu/drm/virtio/virtgpu_display.c | 2 +- 1 file changed, 1 insertion(+), 1

Re: [PATCH V2 for-next 0/8] Bug Fixes and Code Improvement in HNS driver

2016-10-13 Thread Doug Ledford
On 10/13/2016 7:26 AM, Salil Mehta wrote: >> -Original Message- >> From: David Miller [mailto:da...@davemloft.net] >> Sent: Friday, September 30, 2016 6:34 AM >> To: Salil Mehta >> Cc: dledf...@redhat.com; Zhuangyuzeng (Yisen); Huwei (Xavier); oulijun; >> mehta.salil@gmail.com;

Re: [PATCHv3 17/41] filemap: handle huge pages in filemap_fdatawait_range()

2016-10-13 Thread Kirill A. Shutemov
On Thu, Oct 13, 2016 at 11:44:41AM +0200, Jan Kara wrote: > On Thu 15-09-16 14:54:59, Kirill A. Shutemov wrote: > > We writeback whole huge page a time. > > This is one of the things I don't understand. Firstly I didn't see where > changes of writeback like this would happen (maybe they come

[PATCH v5 3/6] i2c: i801: use BIT() macro for bits definition

2016-10-13 Thread Benjamin Tissoires
i801 mixes hexadecimal and decimal values for defining bits. However, we have a nice BIT() macro for this exact purpose. No functional changes, cleanup only. Reviewed-by: Jean Delvare Signed-off-by: Benjamin Tissoires --- no changes in v5 no changes in v4 no changes in v3 no changes in v2

[PATCH v5 5/6] i2c: i801: remove SMBNTFDDAT reads as they always seem to return 0

2016-10-13 Thread Benjamin Tissoires
On the platform tested, reading SMBNTFDDAT always returns 0 (using 1 read of a word or 2 of 2 bytes). Given that we are not sure why and that we don't need to rely on the data parameter in the current users of Host Notify, remove this part of the code. If someone wants to re-enable it, just

[PATCH v5 4/6] i2c: i801: use the BIT() macro for FEATURES_* also

2016-10-13 Thread Benjamin Tissoires
no functional changes Signed-off-by: Benjamin Tissoires --- no changes in v5 no changes in v4 no changes in v3 new in v2 --- drivers/i2c/busses/i2c-i801.c | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/drivers/i2c/busses/i2c-i801.c

[PATCH v5 6/6] i2c: use an IRQ to report Host Notify events, not alert

2016-10-13 Thread Benjamin Tissoires
The current SMBus Host Notify implementation relies on .alert() to relay its notifications. However, the use cases where SMBus Host Notify is needed currently is to signal data ready on touchpads. This is closer to an IRQ than a custom API through .alert(). Given that the 2 touchpad manufacturers

[PATCH v5 0/6] i2c: Host Notify / i801 fixes

2016-10-13 Thread Benjamin Tissoires
Hi Wolfram and Dmitry, 5th revision of the series, hopefully this time it will be OK. The changes were requested by Dmitry: now, SMBus Host Notify is transparent for clients drivers. The IRQ is attributed if the adapter has the capability and if nobody claimed an IRQ before. That means that

[PATCH v5 1/6] i2c: i801: store and restore the SLVCMD register at load and unload

2016-10-13 Thread Benjamin Tissoires
Also do not override any other configuration in this register. Signed-off-by: Benjamin Tissoires --- no changes in v5 changes in v4: - add the i801_disable_host_notify function here as this gets the first in the series no changes in v3 new in v2 --- drivers/i2c/busses/i2c-i801.c | 17

[PATCH v5 2/6] i2c: i801: minor formatting issues

2016-10-13 Thread Benjamin Tissoires
No functional changes, just typos and remove unused #define. Reviewed-by: Jean Delvare Signed-off-by: Benjamin Tissoires --- no changes in v5 no changes in v4 no changes in v3 no changes in v2 --- drivers/i2c/busses/i2c-i801.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-)

Re: [PATCH v2 2/9] mtd: spi-nor: add an alternative method to support memory >16MiB

2016-10-13 Thread Vignesh R
On Wednesday 05 October 2016 05:30 PM, Cyrille Pitchen wrote: > This patch provides an alternative mean to support memory above 16MiB > (128Mib) by replacing 3byte address op codes by their associated 4byte > address versions. > > Using the dedicated 4byte address op codes doesn't change the

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

2016-10-13 Thread Jani Nikula
On Thu, 13 Oct 2016, walter harms wrote: > 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. >> >>

Re: [PATCH v2] Input: elan_i2c - always output the device information

2016-10-13 Thread Benjamin Tissoires
On Oct 13 2016 or thereabouts, Michel Hermier wrote: > Le 13 oct. 2016 13:49, "Benjamin Tissoires" > a écrit : > > > > it's always easier to retrieve these information in bug reports when > > it is always printed in the dmesg. > > > > Signed-off-by: Benjamin Tissoires > > > > --- > > > > changes

Re: [PATCH] staging:vt6656:baseband.h: fix function definition argument without identifier name issue

2016-10-13 Thread Greg KH
On Thu, Oct 13, 2016 at 02:50:18PM +0300, Mikhail Golubev wrote: > Function definitions arguments should also have an identifier name as > reported by checkpatch.pl. Please wrap your changelog comments at 72 columns. > > Signed-off-by: Mikhail Golubev > --- >

Re: [PATCH RESEND] ARM: dts: keystone-k2*: Increase SPI Flash partition size for U-Boot

2016-10-13 Thread Vignesh R
On Wednesday 12 October 2016 05:27 PM, Russell King - ARM Linux wrote: > On Wed, Oct 12, 2016 at 04:30:28PM +0530, Vignesh R wrote: [...] >>> >>> - What is this "misc" partition? >>> >> >> This partition seems to exists from the very beginning. I believe, this >> is just a spare area of flash

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: @@ -241,6 +241,9 @@ int dwc3_send_gadget_ep_cmd(struct dwc3_ep *dep, unsigned cmd, int susphy = false; int ret = -EINVAL; + if

Re: [PATCH] x86/apic: Fix suspicious RCU usage in smp_trace_call_function_interrupt

2016-10-13 Thread Wanpeng Li
2016-10-12 21:17 GMT+08:00 Thomas Gleixner : > On Wed, 12 Oct 2016, Wanpeng Li wrote: >> irq_enter() which is called in scheduler_ipi() is too late to tell RCU >> susbstems to end the extended quiescent state before ack_APIC_irq(), >> any ideas? > > You can call irq_enter/exit() in

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

2016-10-13 Thread Mintz, Yuval
> 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 a good thing [from compilation coverage perspective]? > -

[bug/regression] libhugetlbfs testsuite failures and OOMs eventually kill my system

2016-10-13 Thread Jan Stancek
Hi, I'm running into ENOMEM failures with libhugetlbfs testsuite [1] on a power8 lpar system running 4.8 or latest git [2]. Repeated runs of this suite trigger multiple OOMs, that eventually kill entire system, it usually takes 3-5 runs: * Total System Memory..: 18024 MB * Shared Mem Max

Re: MPOL_BIND on memory only nodes

2016-10-13 Thread Anshuman Khandual
On 10/13/2016 03:37 PM, Michal Hocko wrote: > 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

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 20:17, Felipe Balbi wrote: > > Hi, > > Baolin Wang writes: > @@ -241,6 +241,9 @@ int dwc3_send_gadget_ep_cmd(struct dwc3_ep > *dep, unsigned cmd, > int susphy = false; > int

Re: [PATCH v2 12/16] scsi: fc: use bsg_softirq_done

2016-10-13 Thread Hannes Reinecke
On 10/12/2016 03:06 PM, Johannes Thumshirn wrote: > bsg_softirq_done() and fc_bsg_softirq_done() are copies of each other, so > ditch the fc specific one. > > Signed-off-by: Johannes Thumshirn > --- > block/bsg-lib.c | 3 ++- > drivers/scsi/scsi_transport_fc.c | 16

Re: [PATCH RFC V3 6/6] KVM: LAPIC: Add APIC Timer periodic/oneshot mode VMX preemption timer support

2016-10-13 Thread Paolo Bonzini
On 13/10/2016 13:34, Wanpeng Li wrote: > From: Wanpeng Li > > Most windows guests still utilize APIC Timer periodic/oneshot mode > instead of tsc-deadline mode, and the APIC Timer periodic/oneshot > mode are still emulated by high overhead hrtimer on host. This patch > converts the expected

Re: [GIT PULL REQUEST] watchdog - v4.9 merge window

2016-10-13 Thread Guenter Roeck
On 10/12/2016 10:54 PM, Wim Van Sebroeck wrote: Hi Linus, It's an old server with only 512Mb RAM. I'm in the progress off replacing it (in another datacenter). Meanwhile I'll restart git-daemon. Linus, if it still doesn't work, I pulled it, verified it, and pushed it into my repository at

Re: [PATCH v2] kvm: x86: remove the misleading comment in vmx_handle_external_intr

2016-10-13 Thread Radim Krčmář
2016-10-13 10:34+0800, Longpeng(Mike): > Since Paolo has removed irq-enable-operation in vmx_handle_external_intr > (KVM: x86: use guest_exit_irqoff), the original comment about the IF bit > in rflags is incorrect and stale now, so remove it. > > Signed-off-by: Longpeng(Mike) > --- > Hi Radim, >

Re: [PATCH] iwlwifi: pcie: fix SPLC structure parsing

2016-10-13 Thread Paul Bolle
On Thu, 2016-10-13 at 14:30 +0300, Luca Coelho wrote: > I forgot to say... could you load the iwlwifi module with debug=0x01 > (module parameter), so we can see the messages the driver is printing > when it doesn't find a proper structure? That makes a lot of noise! Here's the first 100 lines or

Re: MPOL_BIND on memory only nodes

2016-10-13 Thread Michal Hocko
On Thu 13-10-16 11:24:59, Mel Gorman wrote: > On Wed, Oct 12, 2016 at 03:16:27PM +0200, Michal Hocko wrote: > > On Wed 12-10-16 11:43:37, Michal Hocko wrote: > > > On Wed 12-10-16 14:55:24, Anshuman Khandual wrote: > > [...] > > > > Why we insist on __GFP_THISNODE ? > > > > > > AFAIU

Re: [PATCH 2/2] power/reset: at91-poweroff: timely shitdown LPDDR memories

2016-10-13 Thread Richard Genoud
2016-10-13 14:27 GMT+02:00 Jean-Jacques Hiblot : > 2016-10-13 13:03 GMT+02:00 Alexandre Belloni > : >> On 12/10/2016 at 14:48:27 +0200, Jean-Jacques Hiblot wrote : >>> > +static void at91_lpddr_poweroff(void) >>> > +{ >>> > + asm volatile( >>> > + /* Align to cache lines */ >>>

[PATCH 0/3] THREAD_INFO_IN_TASK_STRUCT vs generic preemption code

2016-10-13 Thread Heiko Carstens
Commit c65eacbe290b ("sched/core: Allow putting thread_info into task_struct") made struct thread_info a generic struct with only a single flags member if THREAD_INFO_IN_TASK_STRUCT is selected. This change however seems to be quite x86 centric, since at least the generic preemption code

[PATCH RFC V3 1/6] KVM: LAPIC: extract start_sw_period() to handle periodic/oneshot mode

2016-10-13 Thread Wanpeng Li
From: Wanpeng Li Extract start_sw_period() to handle periodic/oneshot mode, it will be used by later patch. Cc: Paolo Bonzini Cc: Radim Krčmář Cc: Yunhong Jiang Signed-off-by: Wanpeng Li --- arch/x86/kvm/lapic.c | 89 +++- 1 file changed, 47

Re: [PATCH] iwlwifi: pcie: fix SPLC structure parsing

2016-10-13 Thread Luca Coelho
On Thu, 2016-10-13 at 14:36 +0200, Paul Bolle wrote: > On Thu, 2016-10-13 at 14:30 +0300, Luca Coelho wrote: > > I forgot to say... could you load the iwlwifi module with debug=0x01 > > (module parameter), so we can see the messages the driver is printing > > when it doesn't find a proper

Re: MPOL_BIND on memory only nodes

2016-10-13 Thread Anshuman Khandual
On 10/12/2016 06:46 PM, Michal Hocko wrote: > On Wed 12-10-16 11:43:37, Michal Hocko wrote: >> On Wed 12-10-16 14:55:24, Anshuman Khandual wrote: > [...] >>> Why we insist on __GFP_THISNODE ? >> >> AFAIU __GFP_THISNODE just overrides the given node to the policy >> nodemask in case the current

Another gcc corruption bug (was Re: [PATCH] [RFC] x86: avoid -mtune=atom for objtool warnings)

2016-10-13 Thread Josh Poimboeuf
On Tue, Oct 11, 2016 at 10:38:42PM +0200, Arnd Bergmann wrote: > On Tuesday, October 11, 2016 10:51:46 AM CEST Josh Poimboeuf wrote: > > Notice how it just falls off the end of the function. We had a similar > > bug before: > > > >

Re: [PATCH 2/2] power/reset: at91-poweroff: timely shitdown LPDDR memories

2016-10-13 Thread Jean-Jacques Hiblot
2016-10-13 13:03 GMT+02:00 Alexandre Belloni : > On 12/10/2016 at 14:48:27 +0200, Jean-Jacques Hiblot wrote : >> > +static void at91_lpddr_poweroff(void) >> > +{ >> > + asm volatile( >> > + /* Align to cache lines */ >> > + ".balign 32\n\t" >> > + >> > +

Re: [PATCH 1/1 v8] ARM: imx: Added perf functionality to mmdc driver

2016-10-13 Thread Shawn Guo
On Mon, Oct 10, 2016 at 03:17:36PM -0500, Zhi Li wrote: > On Mon, Sep 26, 2016 at 11:40 AM, Zhi Li wrote: > > On Mon, Sep 19, 2016 at 12:57 PM, Frank Li wrote: > >> From: Zhengyu Shen > >> > >> MMDC is a multi-mode DDR controller that supports DDR3/DDR3L x16/x32/x64 > >> and LPDDR2 two channel

Re: [PATCH v4 0/3] Broadcom STB AVS CPUfreq driver

2016-10-13 Thread Rafael J. Wysocki
On Thu, Oct 13, 2016 at 7:45 AM, Viresh Kumar wrote: > On 12-10-16, 15:12, Markus Mayer wrote: >> This series contains the CPUfreq driver for Broadcom SoCs that use "AVS >> Firmware" for voltage and frequency scaling. All voltage and frequency >> transitions are performed by the firmware and are

Re: [PATCH] KVM: document lock orders

2016-10-13 Thread Radim Krčmář
2016-10-13 13:10+0200, Paolo Bonzini: > This is long overdue, and not really hard. > > Signed-off-by: Paolo Bonzini > --- Applied to kvm/queue, thanks. > Documentation/virtual/kvm/locking.txt | 12 +++- > 1 file changed, 11 insertions(+), 1 deletion(-) > > diff --git

Re: MPOL_BIND on memory only nodes

2016-10-13 Thread Michal Hocko
On Thu 13-10-16 16:28:27, Anshuman Khandual wrote: > On 10/13/2016 03:37 PM, Michal Hocko wrote: > > 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

Re: [PATCHv3 15/41] filemap: handle huge pages in do_generic_file_read()

2016-10-13 Thread Jan Kara
On Thu 15-09-16 14:54:57, Kirill A. Shutemov wrote: > Most of work happans on head page. Only when we need to do copy data to > userspace we find relevant subpage. > > We are still limited by PAGE_SIZE per iteration. Lifting this limitation > would require some more work. Hum, I'm kind of lost.

Re: [PATCH] iwlwifi: pcie: fix SPLC structure parsing

2016-10-13 Thread Paul Bolle
On Thu, 2016-10-13 at 15:44 +0300, Luca Coelho wrote: >  Even though there is apparently something wrong with this part of the > ACPI table on you laptop, since it doesn't match our specifications. >  In any case, it's mostly harmless. Would a correct implementation by Dell have any benefits for

[PATCH] mm, mempolicy: clean up __GFP_THISNODE confusion in policy_zonelist

2016-10-13 Thread Michal Hocko
From: Michal Hocko __GFP_THISNODE is documented to enforce the allocation to be satisified from the requested node with no fallbacks or placement policy enforcements. policy_zonelist seemingly breaks this semantic if the current policy is MPOL_MBIND and instead of taking the node it will

Re: [PATCH 1/3] binding: irqchip: mtk-cirq: Add binding document

2016-10-13 Thread Matthias Brugger
On 10/13/2016 07:06 AM, Youlin Pei wrote: This commit adds the device tree binding document for the mediatek cirq. Signed-off-by: Youlin Pei --- base on v4.8-rc1 --- .../interrupt-controller/mediatek,cirq.txt | 30 1 file changed, 30 insertions(+) create

Re: [PATCH V3 06/10] acpi: apei: panic OS with fatal error status block

2016-10-13 Thread Suzuki K Poulose
On 07/10/16 22:31, Tyler Baicar wrote: From: "Jonathan (Zhixiong) Zhang" Even if an error status block's severity is fatal, the kernel does not honor the severity level and panic. With the firmware first model, the platform could inform the OS about a fatal hardware error through the non-NMI

[PATCH 1/3] sched/core,x86: make struct thread_info arch specific again

2016-10-13 Thread Heiko Carstens
commit c65eacbe290b ("sched/core: Allow putting thread_info into task_struct") made struct thread_info a generic struct with only a single flags member if THREAD_INFO_IN_TASK_STRUCT is selected. This change however seems to be quite x86 centric, since at least the generic preemption code

Re: [PATCH v2 16/16] block: unexport bsg_softirq_done() again

2016-10-13 Thread Hannes Reinecke
On 10/12/2016 03:06 PM, Johannes Thumshirn wrote: > Unexport bsg_softirq_done() again, we don't need it outside of bsg-lib.c > anymore now that scsi_transport_fc is a pure bsg-lib client. > > Signed-off-by: Johannes Thumshirn > --- > block/bsg-lib.c | 3 +-- > include/linux/bsg-lib.h |

Re: [PATCH v2 15/16] scsi: fc: move FC transport's bsg code to bsg-lib

2016-10-13 Thread Hannes Reinecke
On 10/12/2016 03:06 PM, Johannes Thumshirn wrote: > Now that all conversions are done, move the FibreChannel bsg code over to the > bsg library. Note that CONFIG_SCSI_FC_ATTRS now needs to select BLK_DEV_BSGLIB > in order to build correctly. > > This patch is derived from work done by Mike

[PATCH 2/3] sched/preempt: include asm/current.h

2016-10-13 Thread Heiko Carstens
The generic preempt code needs to include . Otherwise compilation fails if THREAD_INFO_IN_TASK is selected and the generic preempt code is used: ./include/linux/thread_info.h:17:54: error: 'current' undeclared (first use in this function) #define current_thread_info() ((struct thread_info

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

2016-10-13 Thread Baolin Wang
Hi Felipe, On 13 October 2016 at 19:23, Felipe Balbi wrote: > > Hi, > > Baolin Wang writes: 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

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. > > > >

Re: [PATCH][v10] PM / hibernate: Verify the consistent of e820 memory map by md5 digest

2016-10-13 Thread Denys Vlasenko
On Fri, Sep 9, 2016 at 2:21 PM, Chen Yu wrote: > On some platforms, there is occasional panic triggered when trying to > resume from hibernation, a typical panic looks like: > > "BUG: unable to handle kernel paging request at 880085894000 > IP: [] load_image_lzo+0x8c2/0xe70" > > Investigation

Re: [PATCH v2] z3fold: add shrinker

2016-10-13 Thread Vitaly Wool
On Thu, 13 Oct 2016 11:20:06 +1100 Dave Chinner wrote: > > That's an incorrect assumption. Long spinlock holds prevent > scheduling on that CPU, and so we still get latency problems. Fair enough. The problem is, some of the z3fold code that need mutual exclusion runs with preemption disabled

Re: [PATCH V3 10/10] arm64: KVM: add guest SEA support

2016-10-13 Thread Punit Agrawal
Hi Tyler, I know I've had my last comment already ;), but I thought I'd rather raise the question than stay confused... Tyler Baicar writes: > Currently external aborts are unsupported by the guest abort > handling. Add handling for SEAs so that the host kernel reports > SEAs which occur in

[PATCH] leds: leds-pca963x: workaround group blink scaling issue

2016-10-13 Thread Matt Ranostay
PCA9632TK part seems to incorrectly blink at ~1.3x of the programmed rate. This patchset add a nxp,period-scale devicetree property to adjust for this misconfiguration. Cc: Tony Lindgren Cc: Jacek Anaszewski Signed-off-by: Matt Ranostay --- Documentation/devicetree/bindings/leds/pca963x.txt |

Re: [PATCH 1/4] remoteproc: Use fixed length field for firmware name

2016-10-13 Thread loic pallardy
On 10/11/2016 03:39 PM, Matt Redfearn wrote: Storage of the firmware name was inconsistent, either storing a pointer to a name stored with unknown ownership, or a variable length tacked onto the end of the struct proc allocated in rproc_alloc. In preparation for allowing the firmware of an

Re: [PATCH v2 05/16] scsi: fc: provide fc_bsg_to_shost() helper

2016-10-13 Thread Hannes Reinecke
On 10/12/2016 03:06 PM, Johannes Thumshirn wrote: > Provide fc_bsg_to_shost() helper that will become handy when we're moving from > struct fc_bsg_job to a plain struct bsg_job. Also use this little helper in > the LLDDs. > > Signed-off-by: Johannes Thumshirn > --- > drivers/s390/scsi/zfcp_fc.c

Re: [PATCH v2 04/16] scsi: Unify interfaces of fc_bsg_jobdone and bsg_job_done

2016-10-13 Thread Hannes Reinecke
On 10/12/2016 03:06 PM, Johannes Thumshirn wrote: > Unify the interfaces of fc_bsg_jobdone and bsg_job_done. This will reduce the > diff when moving from 'struct fc_bsg_job' to a plain 'struct bsg_job' later > on. > > Signed-off-by: Johannes Thumshirn > --- > drivers/s390/scsi/zfcp_fc.c |

[PATCH] [media] rc: Fix uninitialized variable in debug print

2016-10-13 Thread Geert Uytterhoeven
drivers/media/i2c/ir-kbd-i2c.c: In function ‘get_key_haup_common’: drivers/media/i2c/ir-kbd-i2c.c:120: warning: ‘toggle’ is used uninitialized in this function To fix this, use the intermediate "toggle" variable, like is done in other branches. Fixes: 00bb820755ed8ee9 ("[media] rc:

Re: [PATCHv3 17/41] filemap: handle huge pages in filemap_fdatawait_range()

2016-10-13 Thread Jan Kara
On Thu 13-10-16 15:08:44, Kirill A. Shutemov wrote: > On Thu, Oct 13, 2016 at 11:44:41AM +0200, Jan Kara wrote: > > On Thu 15-09-16 14:54:59, Kirill A. Shutemov wrote: > > > We writeback whole huge page a time. > > > > This is one of the things I don't understand. Firstly I didn't see where > >

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

2016-10-13 Thread Felipe Balbi
Hi, Baolin Wang writes: >> Baolin Wang writes: > 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. >

Re: [PATCH v3 net-next 0/2] net: centralize net_device MTU bounds checking

2016-10-13 Thread David Miller
From: Jarod Wilson Date: Fri, 7 Oct 2016 22:04:32 -0400 > While looking into an MTU issue with sfc, I started noticing that almost > every NIC driver with an ndo_change_mtu function implemented almost > exactly the same range checks, and in many cases, that was the only > practical thing their

Re: [PATCH] vmxnet3: avoid assumption about invalid dma_pa in vmxnet3_set_mc()

2016-10-13 Thread David Miller
From: Alexey Khoroshilov Date: Fri, 7 Oct 2016 22:53:43 -0400 > vmxnet3_set_mc() still assumes zero is invalid pa: > it assumes dma_mapping_error(...,0) returns true if vmxnet3_copy_mc() fails; > it calls dma_unmap_single() iff new_table_pa is not zero. > > The patch adds an explicit

Re: [PATCH v4 03/10] ARM: sun8i: dt: Add DT bindings documentation for Allwinner sun8i-emac

2016-10-13 Thread LABBE Corentin
On Mon, Oct 10, 2016 at 10:13:35AM -0500, Rob Herring wrote: > On Fri, Oct 07, 2016 at 10:25:50AM +0200, Corentin Labbe wrote: > > This patch adds documentation for Device-Tree bindings for the > > Allwinner sun8i-emac driver. > > > > Signed-off-by: Corentin Labbe > > --- > >

Re: [Intel-gfx] [PATCH v2 05/10] drm/i915/gen9: Get rid of redundant watermark values

2016-10-13 Thread Maarten Lankhorst
Op 08-10-16 om 02:11 schreef Lyude: > Now that we've make skl_wm_levels make a little more sense, we can > remove all of the redundant wm information. Up until now we'd been > storing two copies of all of the skl watermarks: one being the > skl_pipe_wm structs, the other being the global wm struct

Re: [mac80211] BUG_ON with current -git (4.8.0-11417-g24532f7)

2016-10-13 Thread Sergey Senozhatsky
On (10/13/16 08:02), Johannes Berg wrote: > On Wed, 2016-10-12 at 22:39 -0700, Andy Lutomirski wrote: > > > In a pinch, I have these patches sitting around: > > > > https://git.kernel.org/cgit/linux/kernel/git/luto/linux.git/commit/?h=x86/vmap_stack=0a39cfa6fbb5d5635c85253cc7d6b44b54822afd > >

Re: [PATCH 2/2] power/reset: at91-poweroff: timely shitdown LPDDR memories

2016-10-13 Thread Alexandre Belloni
On 13/10/2016 at 14:27:15 +0200, Jean-Jacques Hiblot wrote : > 2016-10-13 13:03 GMT+02:00 Alexandre Belloni > : > > On 12/10/2016 at 14:48:27 +0200, Jean-Jacques Hiblot wrote : > >> > +static void at91_lpddr_poweroff(void) > >> > +{ > >> > + asm volatile( > >> > + /* Align to

Re: [PATCH v2 1/3] net: smc91x: isolate u16 writes alignment workaround

2016-10-13 Thread David Miller
From: Robert Jarzmik Date: Sun, 9 Oct 2016 22:33:45 +0200 > Writes to u16 has a special handling on 3 PXA platforms, where the > hardware wiring forces these writes to be u32 aligned. > > This patch isolates this handling for PXA platforms as before, but > enables this "workaround" to be set

[PATCH] [media] dib0700: Fix uninitialized protocol for NEC repeat codes

2016-10-13 Thread Geert Uytterhoeven
drivers/media/usb/dvb-usb/dib0700_core.c: In function ‘dib0700_rc_urb_completion’: drivers/media/usb/dvb-usb/dib0700_core.c:679: warning: ‘protocol’ may be used uninitialized in this function When receiving an NEC repeat code, protocol is indeed not initialized. Set it to RC_TYPE_NECX

Re: [PATCH v3 2/2] Input: synaptics-rmi4 - add support for F34 device reflash

2016-10-13 Thread Benjamin Tissoires
On Oct 12 2016 or thereabouts, Nick Dyer wrote: > Hi Benjamin- > > Thanks for the review, I've answered in line. > > On Mon, Oct 10, 2016 at 03:48:07PM +0200, Benjamin Tissoires wrote: > > On Sep 20 2016 or thereabouts, Nick Dyer wrote: > > > Signed-off-by: Nick Dyer > > > Tested-by: Chris

Re: [PATCH 3/4] remoteproc: Add a sysfs interface for firmware and state

2016-10-13 Thread loic pallardy
On 10/11/2016 03:39 PM, Matt Redfearn wrote: This patch adds a sysfs interface to rproc allowing the firmware name and processor state to be changed dynamically. State was previously available in debugfs, and is replicated here. The firmware file allows retrieval of the running firmware name,

[cpuidle] 00da7dafbf: BUG: sleeping function called from invalid context at kernel/locking/mutex.c:620

2016-10-13 Thread kernel test robot
FYI, we noticed the following commit: http://git.linaro.org/people/daniel.lezcano/linux sched/idle commit 00da7dafbfd7d57e72ea4ca098570424e27d0949 ("cpuidle: irq: Add a new irq based governor") in testcase: boot on test machine: qemu-system-i386 -enable-kvm -cpu Haswell,+smep,+smap -m 360M

[PATCH RFC V3 0/6] KVM: LAPIC: Add APIC Timer periodic/oneshot mode VMX preemption timer support

2016-10-13 Thread Wanpeng Li
Most windows guests which I have on hand currently still utilize APIC Timer periodic/oneshot mode instead of APIC Timer tsc-deadline mode: - windows 2008 server r2 - windows 2012 server r2 - windows 7 - windows 10 This patchset adds the APIC Timer periodic/oneshot mode VMX preemption timer

Re: [PATCH V3 01/10] acpi: apei: read ack upon ghes record consumption

2016-10-13 Thread Baicar, Tyler
Hello Punit, Thank you for the feedback! Responses below On 10/12/2016 9:39 AM, Punit Agrawal wrote: Hi Tyler, A few comments below. Tyler Baicar writes: A RAS (Reliability, Availability, Serviceability) controller may be a separate processor running in parallel with OS execution, and may

Re: [PATCH] qed: fix old-style function definition

2016-10-13 Thread David Miller
From: Arnd Bergmann Date: Mon, 10 Oct 2016 13:59:16 +0200 > The definition of qed_get_rdma_ops() is not a prototype unless > we add 'void' here, as indicated by this W=1 warning: > > drivers/net/ethernet/qlogic/qed/qed_roce.c: In function ‘qed_get_rdma_ops’: >

Re: [PATCH V3 04/10] arm64: exception: handle Synchronous External Abort

2016-10-13 Thread Baicar, Tyler
Hello Punit, On 10/12/2016 11:46 AM, Punit Agrawal wrote: Hi Tyler, A couple of hopefully not bike shedding comments below. Tyler Baicar writes: SEA exceptions are often caused by an uncorrected hardware error, and are handled when data abort and instruction abort exception classes have

Re: [PATCH] iwlwifi: pcie: fix SPLC structure parsing

2016-10-13 Thread Chris Rorvick
Hi Luca, > On Thu, 2016-10-13 at 13:21 +0300, Luca Coelho wrote: > Could you please give this a spin? I have tested it with some handmade > ACPI tables in QEMU and it seems to work fine now. Tested-by: Chris Rorvick I think the debug output looks as expected, see below for the first 20 lines

Re: [PATCH] tlan: avoid unused label with PCI=n

2016-10-13 Thread David Miller
From: Arnd Bergmann Date: Mon, 10 Oct 2016 14:41:10 +0200 > While build testing with randconfig on x86, I ran into this warning > that appears to have been around forever > > drivers/net/ethernet/ti/tlan.c: In function ‘tlan_probe1’: > drivers/net/ethernet/ti/tlan.c:614:1: error: label

Re: [mac80211] BUG_ON with current -git (4.8.0-11417-g24532f7)

2016-10-13 Thread Sergey Senozhatsky
On (10/13/16 22:42), Sergey Senozhatsky wrote: > > On (10/13/16 08:02), Johannes Berg wrote: > > On Wed, 2016-10-12 at 22:39 -0700, Andy Lutomirski wrote: > > > > > In a pinch, I have these patches sitting around: > > > > > >

[GIT PULL] PCI changes for v4.9

2016-10-13 Thread Bjorn Helgaas
Hi Linus, Here are some more changes I'd like to have in v4.9. There's one small Tegra bug fix in the PHY poweroff path, which is only used in failure paths. The rest is all strictly cleanup that should make host bridge drivers more readable, but shouldn't actually change any behavior. Bjorn

Re: [PATCH V3 05/10] acpi: apei: handle SEA notification type for ARMv8

2016-10-13 Thread Baicar, Tyler
Hello Punit, On 10/12/2016 12:00 PM, Punit Agrawal wrote: Tyler Baicar writes: ARM APEI extension proposal added SEA (Synchrounous External Abort) notification type for ARMv8. Add a new GHES error source handling function for SEA. If an error source's notification type is SEA, then this

Re: [mac80211] BUG_ON with current -git (4.8.0-11417-g24532f7)

2016-10-13 Thread Johannes Berg
On Thu, 2016-10-13 at 22:42 +0900, Sergey Senozhatsky wrote: >  > > > https://git.kernel.org/cgit/linux/kernel/git/luto/linux.git/commi > > > t/?h=x86/vmap_stack=0a39cfa6fbb5d5635c85253cc7d6b44b54822afd > > > https://git.kernel.org/cgit/linux/kernel/git/luto/linux.git/commi > > >

[PATCH] [media] s5p_cec: Mark runtime suspend/resume as __maybe_unused

2016-10-13 Thread Geert Uytterhoeven
If CONFIG_PM_SLEEP=n: drivers/staging/media/s5p-cec/s5p_cec.c:235: warning: ‘s5p_cec_runtime_suspend’ defined but not used drivers/staging/media/s5p-cec/s5p_cec.c:243: warning: ‘s5p_cec_runtime_resume’ defined but not used Mark these functions as__maybe_unused to fix this without

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