Re: [PATCH v3 2/4] KVM: X86: Fix loss of exception which has not yet injected

2017-08-24 Thread Paolo Bonzini
On 24/08/2017 11:34, Wanpeng Li wrote: > 2017-08-24 17:13 GMT+08:00 Wanpeng Li : >> 2017-08-24 16:57 GMT+08:00 Paolo Bonzini : >>> On 24/08/2017 08:52, Wanpeng Li wrote: > @@ -6862,6 +6876,7 @@ static int vcpu_enter_guest(struct kvm_vcpu *vcpu) > kvm_x86_ops-

[PATCH net-next 3/4] net: mvpp2: fix use of the random mac address for PPv2.2

2017-08-24 Thread Antoine Tenart
The MAC retrieval logic is using a variable to store an h/w stored mac address and checks this mac against invalid ones before using it. But the mac address is only read from h/w when using PPv2.1. So when using PPv2.2 it defaults to its init state. This patches fixes the logic to only check if th

Re: [PATCH v3 2/4] KVM: X86: Fix loss of exception which has not yet injected

2017-08-24 Thread Wanpeng Li
2017-08-24 17:35 GMT+08:00 Paolo Bonzini : > On 24/08/2017 11:13, Wanpeng Li wrote: >> 2017-08-24 16:57 GMT+08:00 Paolo Bonzini : >>> On 24/08/2017 08:52, Wanpeng Li wrote: > @@ -6862,6 +6876,7 @@ static int vcpu_enter_guest(struct kvm_vcpu *vcpu) > kvm_x86_o

Re: [PATCH 0/3] cpuidle: Rework the handling of the poll state

2017-08-24 Thread Sudeep Holla
On 23/08/17 22:18, Rafael J. Wysocki wrote: > Hi, > > On x86 the fist idle state is a polling one, but the way it is set up is far > from straightforward and then it is avoided by governors in rather somewhat > convoluted fashion. > > Make this more clear by explicitly flagging that state as "p

[PATCH net-next 2/4] net: mvpp2: move the mac retrieval/copy logic into its own function

2017-08-24 Thread Antoine Tenart
The MAC retrieval has a quite complicated logic (which is broken). Moves it to its own function to prepare for patches fixing its logic, so that reviews are easier. Signed-off-by: Antoine Tenart --- drivers/net/ethernet/marvell/mvpp2.c | 45 +--- 1 file changed, 2

Re: [PATCH v2 1/1] futex: remove duplicated code and fix UB

2017-08-24 Thread Will Deacon
Hi Jiri, On Thu, Aug 24, 2017 at 09:31:05AM +0200, Jiri Slaby wrote: > There is code duplicated over all architecture's headers for > futex_atomic_op_inuser. Namely op decoding, access_ok check for uaddr, > and comparison of the result. > > Remove this duplication and leave up to the arches only

[PATCH net-next 1/4] net: mvpp2: fix the mac address used when using PPv2.2

2017-08-24 Thread Antoine Tenart
The mac address is only retrieved from h/w when using PPv2.1. Otherwise the variable holding it is still checked and used if it contains a valid value. As the variable isn't initialized to an invalid mac address value, we end up with random mac addresses which can be the same for all the ports hand

[PATCH net-next 4/4] net: mvpp2: fallback using h/w and random mac if the dt one isn't valid

2017-08-24 Thread Antoine Tenart
When using a mac address described in the device tree, a check is made to see if it is valid. When it's not, no fallback is defined. This patches tries to get the mac address from h/w (or use a random one if the h/w one isn't valid) when the dt mac address isn't valid. Signed-off-by: Antoine Tenar

[PATCH net-next 0/4] net: mvpp2: fix the mac address retrieval logic

2017-08-24 Thread Antoine Tenart
Hi all, The MAC address retrieval logic was broken and when using the PPv2 driver on PPv2.2 engines I ended up using the same mac address on all ports. This series of patches fixes this, and also tackle a possible bug when defining the mac address in the device tree. To fix this in a nice way I e

Re: [PATCH v2] x86: enable RCU based table free

2017-08-24 Thread Kirill A. Shutemov
On Thu, Aug 24, 2017 at 11:22:58AM +0200, Vitaly Kuznetsov wrote: > On x86 software page-table walkers depend on the fact that remote TLB flush > does an IPI: walk is performed lockless but with interrupts disabled and in > case the pagetable is freed the freeing CPU will get blocked as remote TLB

[PATCH 2/3] ata: pdc2027x: constify ata_port_info

2017-08-24 Thread Arvind Yadav
ata_port_info are not supposed to change at runtime. All functions working with ata_port_info provided by work with const ata_port_info. So mark the non-const structs as const. Signed-off-by: Arvind Yadav --- drivers/ata/pata_pdc2027x.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) dif

[PATCH 3/3] ata: constify ata_port_info

2017-08-24 Thread Arvind Yadav
ata_port_info are not supposed to change at runtime. All functions working with ata_port_info provided by work with const ata_port_info. So mark the non-const structs as const. Signed-off-by: Arvind Yadav --- drivers/ata/pdc_adma.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --g

[PATCH 1/3] ata: ftide010: constify ata_port_info

2017-08-24 Thread Arvind Yadav
ata_port_info are not supposed to change at runtime. All functions working with ata_port_info provided by work with const ata_port_info. So mark the non-const structs as const. Signed-off-by: Arvind Yadav --- drivers/ata/pata_ftide010.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) dif

Re: [v9] wlcore: add missing nvs file name info for wilink8

2017-08-24 Thread Kalle Valo
"Reizer, Eyal" wrote: > The following commits: > commit c815fdebef44 ("wlcore: spi: Populate config firmware data") > commit d776fc86b82f ("wlcore: sdio: Populate config firmware data") > > Populated the nvs entry for wilink6 and wilink7 only while it is > still needed for wilink8 as well. > Thi

[PATCH 0/3] constify ata_port_info

2017-08-24 Thread Arvind Yadav
ata_port_info are not supposed to change at runtime. All functions working with ata_port_info provided by work with const ata_port_info. So mark the non-const structs as const. Arvind Yadav (3): [PATCH 1/3] ata: ftide010: constify ata_port_info [PATCH 2/3] ata: pdc2027x: constify ata_port_inf

Re: [PATCH V6] perf: Add PERF_SAMPLE_PHYS_ADDR

2017-08-24 Thread Michael Ellerman
Peter Zijlstra writes: > On Thu, Aug 24, 2017 at 11:26:17AM +1000, Michael Ellerman wrote: >> Peter Zijlstra writes: >> >> diff --git a/kernel/events/core.c b/kernel/events/core.c >> >> index d704e23..b991af3 100644 >> >> --- a/kernel/events/core.c >> >> +++ b/kernel/events/core.c >> >> @@ -6027

Re: rtlwifi: btcoex: 23b 1ant: fix duplicated code for different branches

2017-08-24 Thread Kalle Valo
"Gustavo A. R. Silva" wrote: > Refactor code in order to avoid identical code for different branches. > > This issue was detected with the help of Coccinelle. > > Addresses-Coverity-ID: 1415177 > Signed-off-by: Gustavo A. R. Silva Patch applied to wireless-drivers-next.git, thanks. 14cc696c0

RE: [PATCH] RDMA/qedr: fix spelling mistake: "invlaid" -> "invalid"

2017-08-24 Thread Amrani, Ram
> From: Colin Ian King > > Trivial fix to spelling mistake in DP_ERR error message > > Signed-off-by: Colin Ian King > --- > drivers/infiniband/hw/qedr/verbs.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/infiniband/hw/qedr/verbs.c > b/drivers/infiniband/h

Re: [PATCH] clk: versatile: make clk_ops const

2017-08-24 Thread Liviu Dudau
On Tue, Aug 22, 2017 at 07:36:26PM +0530, Bhumika Goyal wrote: > Make this const as it is only stored in the const field of a > clk_init_data structure. > > Signed-off-by: Bhumika Goyal Acked-by: Liviu Dudau Thanks for the patch! Sudeep, can you going to pick this up for 4.14? Best regards,

Re: [v2] mt7601u: check memory allocation failure

2017-08-24 Thread Kalle Valo
Christophe Jaillet wrote: > Check memory allocation failure and return -ENOMEM in such a case, as > already done a few lines below. > > As 'dev->tx_q' can be NULL, we also need to check for that in > 'mt7601u_free_tx()', and return early. > > Signed-off-by: Christophe JAILLET > Acked-by: Jakub

Re: rtlwifi: rtl8821ae: fix spelling mistake: "faill" -> "failed"

2017-08-24 Thread Kalle Valo
Colin Ian King wrote: > From: Colin Ian King > > Trivial fix to spelling mistake in RT_TRACE message > > Signed-off-by: Colin Ian King Patch applied to wireless-drivers-next.git, thanks. 2cc8918eb987 rtlwifi: rtl8821ae: fix spelling mistake: "faill" -> "failed" -- https://patchwork.kernel

Re: [PATCH v5 2/4] perf tools arm64: Add support for get_cpuid_str function.

2017-08-24 Thread Ganapatrao Kulkarni
On Wed, Aug 23, 2017 at 3:47 PM, Will Deacon wrote: > On Wed, Aug 16, 2017 at 12:40:46PM +0530, Ganapatrao Kulkarni wrote: >> function get_cpuid_str returns MIDR string of the first online >> cpu from the range of cpus associated with the pmu core device. >> >> Signed-off-by: Ganapatrao Kulkarni

Re: [RESEND,v12,7/8] wireless: ipw2200: Replace PCI pool old API

2017-08-24 Thread Kalle Valo
Romain Perier wrote: > The PCI pool API is deprecated. This commit replaces the PCI pool old > API by the appropriate function with the DMA pool API. > > Signed-off-by: Romain Perier > Reviewed-by: Peter Senna Tschudin > Acked-by: Stanislav Yakovlev Patch applied to wireless-drivers-next.git

Re: [1/3] net: rtlwifi: constify rate_control_ops

2017-08-24 Thread Kalle Valo
Arvind Yadav wrote: > rate_control_ops are not supposed to change at runtime. All functions > working with rate_control_ops provided by work with > const rate_control_ops. So mark the non-const structs as const. > > Signed-off-by: Arvind Yadav 2 patches applied to wireless-drivers-next.git, t

Re: wireless: ipw2x00: make iw_handler_def const

2017-08-24 Thread Kalle Valo
Bhumika Goyal wrote: > Make these const as they are only stored in the const field of a > net_device structure. > > Signed-off-by: Bhumika Goyal Patch applied to wireless-drivers-next.git, thanks. 2c1dca3c6b22 wireless: ipw2x00: make iw_handler_def const -- https://patchwork.kernel.org/patc

[PATCH v2 3/3] mm: Consider the number in local CPUs when *reads* NUMA stats

2017-08-24 Thread Kemi Wang
To avoid deviation, the per cpu number of NUMA stats in vm_numa_stat_diff[] is included when a user *reads* the NUMA stats. Since NUMA stats does not be read by users frequently, and kernel does not need it to make a decision, it will not be a problem to make the readers more expensive. Changelog

[PATCH v2 1/3] mm: Change the call sites of numa statistics items

2017-08-24 Thread Kemi Wang
In this patch, NUMA statistics is separated from zone statistics framework, all the call sites of NUMA stats are changed to use numa-stats-specific functions, it does not have any functionality change except that the number of NUMA stats is shown behind zone page stats when users *read* the zone i

[PATCH v2 2/3] mm: Update NUMA counter threshold size

2017-08-24 Thread Kemi Wang
There is significant overhead in cache bouncing caused by zone counters (NUMA associated counters) update in parallel in multi-threaded page allocation (suggested by Dave Hansen). This patch updates NUMA counter threshold to a fixed size of MAX_U16 - 2, as a small threshold greatly increases the u

[PATCH v2 0/3] Separate NUMA statistics from zone statistics

2017-08-24 Thread Kemi Wang
Each page allocation updates a set of per-zone statistics with a call to zone_statistics(). As discussed in 2017 MM summit, these are a substantial source of overhead in the page allocator and are very rarely consumed. This significant overhead in cache bouncing caused by zone counters (NUMA associ

Re: [PATCH v2 3/3] ACPI / PMIC: Add opregion driver for Intel Dollar Cove TI PMIC

2017-08-24 Thread Takashi Iwai
On Thu, 24 Aug 2017 11:40:43 +0200, Takashi Iwai wrote: > > On Thu, 24 Aug 2017 11:14:53 +0200, > Mika Westerberg wrote: > > > > On Thu, Aug 24, 2017 at 10:11:41AM +0200, Takashi Iwai wrote: > > > This patch adds the opregion driver for Dollar Cove TI PMIC on Intel > > > Cherry Trail devices. Th

Re: Kernels v4.9+ cause short reads of block devices

2017-08-24 Thread Doug Nazar
On 8/23/17 4:13 PM, Linus Torvalds wrote: Oh, right you are - I'm much too used to 64-bit, where MAX_LFS_FILESIZE is basically infinite, and was jusr assuming that it was something like the UFS bug we had not that long ago that was due to the 32-bit limit. But yes, on 32-bit, we are limited by

Re: [PATCH v6] mtd: sharpslpart: Add sharpslpart partition parser

2017-08-24 Thread Boris Brezillon
On Thu, 24 Aug 2017 11:19:56 +0200 Andrea Adami wrote: > >> +/** > >> + * struct sharpsl_ftl - Sharp FTL Logical Table > >> + * @logmax: number of logical blocks > >> + * @log2phy: the logical-to-physical table > >> + * > >> + * Stripped down from 2.4 sources for read-only purpos

Re: [PATCH 1/2] KVM: x86: simplify handling of PKRU

2017-08-24 Thread Yang Zhang
On 2017/8/24 17:19, Paolo Bonzini wrote: On 24/08/2017 11:09, Yang Zhang wrote: +if (static_cpu_has(X86_FEATURE_OSPKE) && We expose protection key to VM without check whether OSPKE is enabled or not. Why not check guest's cpuid here which also can avoid unnecessary access to pkru? Checki

[PATCH 2/3] KVM: x86: simplify handling of PKRU

2017-08-24 Thread Paolo Bonzini
Move it to struct kvm_arch_vcpu, replacing guest_pkru_valid with a simple comparison against the host value of the register. The write of PKRU in addition can be skipped if the guest has not enabled the feature. Once we do this, we need not test OSPKE in the host anymore, because guest_CR4.PKE=1 i

[PATCH 3/3] KVM, pkeys: do not use PKRU value in vcpu->arch.guest_fpu.state

2017-08-24 Thread Paolo Bonzini
The host pkru is restored right after vcpu exit (commit 1be0e61), so KVM_GET_XSAVE will return the host PKRU value instead. Fix this by using the guest PKRU explicitly in fill_xsave and load_xsave. This part is based on a patch by Junkang Fu. The host PKRU data may also not match the value in vc

[PATCH 0/3] KVM, pkeys: fix handling of PKRU across migration

2017-08-24 Thread Paolo Bonzini
The host pkru is restored right after vcpu exit (commit 1be0e61), so KVM_GET_XSAVE will return the host PKRU value instead. In general, the PKRU value in vcpu->arch.guest_fpu.state cannot be trusted. Series as follows: 1) fix independent bug which would cause an oops 2) remove an unnecessary ab

[PATCH 1/3] KVM: x86: block guest protection keys unless the host has them enabled

2017-08-24 Thread Paolo Bonzini
If the host has protection keys disabled, we cannot read and write the guest PKRU---RDPKRU and WRPKRU fail with #GP(0) if CR4.PKE=0. Block the PKU cpuid bit in that case. This ensures that guest_CR4.PKE=1 implies host_CR4.PKE=1. Fixes: 1be0e61c1f255faaeab04a390e00c8b9b9042870 Signed-off-by: Paol

Re: gcc-8 objtool warnings

2017-08-24 Thread Arnd Bergmann
On Wed, Aug 23, 2017 at 6:01 PM, Josh Poimboeuf wrote: > On Wed, Aug 23, 2017 at 03:38:02PM +0200, Arnd Bergmann wrote: >> On Wed, Aug 23, 2017 at 2:48 PM, Josh Poimboeuf wrote: >> > On Wed, Aug 23, 2017 at 02:22:34PM +0200, Arnd Bergmann wrote: >> >> ... >> >> >> >> : >> >>0

Re: [PATCH 3/5] mmc: sdhci-msm: Add support to wait for power irq

2017-08-24 Thread Adrian Hunter
On 18/08/17 08:19, Vijay Viswanath wrote: > From: Sahitya Tummala > > Add support API which will check if power irq is expected to be > generated and wait for the power irq to come and complete if the irq is > expected. > > Signed-off-by: Sahitya Tummala > Signed-off-by: Vijay Viswanath > ---

Re: [PATCH 1/2] KVM: x86: simplify handling of PKRU

2017-08-24 Thread Paolo Bonzini
On 24/08/2017 12:05, Yang Zhang wrote: > On 2017/8/24 17:19, Paolo Bonzini wrote: >> On 24/08/2017 11:09, Yang Zhang wrote: +if (static_cpu_has(X86_FEATURE_OSPKE) && >>> >>> We expose protection key to VM without check whether OSPKE is enabled or >>> not. Why not check guest's cpuid here w

[PATCH v8 0/2] perf/core: addressing 4x slowdown during per-process profiling of STREAM benchmark on Intel Xeon Phi

2017-08-24 Thread Alexey Budankov
Hi, This patch set v8 addresses feedback captured on review of v7. For more background details please refer to v7 and earlier. Specifically in this version mux switch is renamed to rotation switch and the value of rotation switch may be the one of defined by ROTATION_DISABLED or ROTATION_ENABL

Re: [PATCH 4/5] mmc: sdhci-msm: Add ops to do sdhc register write

2017-08-24 Thread Adrian Hunter
On 18/08/17 08:19, Vijay Viswanath wrote: > Register writes which change voltage of IO lines or turn the IO bus > on/off require controller to be ready before progressing further. When > the controller is ready, it will generate a power irq which needs to be > handled. The thread which initiated th

Re: [PATCH] ASoC: Add a sanity check before using dai driver name

2017-08-24 Thread Mark Brown
On Thu, Aug 24, 2017 at 11:29:42AM +0800, jeffy wrote: > and i was trying to do something like: > if (dlc->dai_name && strcmp(dai->name, dlc->dai_name) > && (!dai->driver->name || strcmp(dai->driver->name, dlc->dai_name))) >continue; > which is add an accept case for: dai driver name is v

[PATCH v8 1/2] perf/core: use rb trees for pinned/flexible groups

2017-08-24 Thread Alexey Budankov
This patch moves event groups into rb tree sorted by CPU, so that multiplexing hrtimer interrupt handler would be able skipping to the current CPU's list and ignore groups allocated for the other CPUs. New API for manipulating event groups in the trees is implemented as well as adoption on the

[PATCH v8 2/2] perf/core: add rotation switch to skip to the current CPU's events list on mux interrupt

2017-08-24 Thread Alexey Budankov
This patch implements rotation switch that triggers skipping to the current CPU's events list at mulitplexing hrtimer interrupt handler as well as adoption of the switch in the existing implementation. The value of rotation switch may be the one of defined by ROTATION_DISABLED or ROTATION_ENAB

Re: [PATCH] clk: versatile: make clk_ops const

2017-08-24 Thread Sudeep Holla
On 24/08/17 10:54, Liviu Dudau wrote: > On Tue, Aug 22, 2017 at 07:36:26PM +0530, Bhumika Goyal wrote: >> Make this const as it is only stored in the const field of a >> clk_init_data structure. >> >> Signed-off-by: Bhumika Goyal > > Acked-by: Liviu Dudau > > Thanks for the patch! > > Sudeep

Re: Kernels v4.9+ cause short reads of block devices

2017-08-24 Thread Doug Nazar
On 8/23/17 5:01 PM, Andreas Dilger wrote: Doug, I noticed while checking for other implications of changing MAX_LFS_FILESIZE that fs/jfs/super.c is also working around this limit. If you are going to submit a patch for this, it also makes sense to fix jfs_fill_super() to use MAX_LFS_FILESIZE ins

Re: [PATCH v7 12/13] ACPI / init: Invoke early ACPI initialization earlier

2017-08-24 Thread Baoquan He
On 08/24/17 at 05:28pm, Dou Liyang wrote: > Hi Baoquan, > > Thanks for your reply. > > At 08/24/2017 04:05 PM, Baoquan He wrote: > > Hi Liyang, > > > > On 08/24/17 at 11:54am, Dou Liyang wrote: > > > > > Test in my own PC(Lenovo M4340). > > > > > Ask help for doing regression testing for the bug

[PATCH 3/5] arm: dts: rockchip: add tsadc node for RV1108 SoC

2017-08-24 Thread Rocky Hao
Add tsadc needed main information for RV1108 SoC. 75Hz is the max clock rate supported by tsadc module. Signed-off-by: Rocky Hao --- arch/arm/boot/dts/rv1108.dtsi | 29 + 1 file changed, 29 insertions(+) diff --git a/arch/arm/boot/dts/rv1108.dtsi b/arch/arm/boot/

[PATCH 1/5] dt-bindings: rockchip-thermal: Support the RV1108 SoC compatible

2017-08-24 Thread Rocky Hao
Add a new compatible for thermal founding on RV1108 SoCs. Signed-off-by: Rocky Hao --- Documentation/devicetree/bindings/thermal/rockchip-thermal.txt | 1 + 1 file changed, 1 insertion(+) diff --git a/Documentation/devicetree/bindings/thermal/rockchip-thermal.txt b/Documentation/devicetree/bin

[PATCH 0/5] thermal: rockchip: add tsadc support in thermal driver and IPA thermal control for RV1108 in dts

2017-08-24 Thread Rocky Hao
This series patches add the tsadc support in thermal driver and in devicetree for RV1108. Also add thermal control with Intelligent Power Allocation (IPA) policy by default. Please refer to https://developer.arm.com/open-source/intelligent-power-allocation for more information about IPA. Rocky

[PATCH] irqchip: mmp: make mmp_intc_conf const

2017-08-24 Thread Bhumika Goyal
Make these const as they are only used during a copy operation. Done using Coccinelle. @match disable optional_qualifier@ identifier s; @@ static struct mmp_intc_conf s = {...}; @ref@ position p; identifier match.s; @@ s@p @good1@ position ref.p; identifier match.s,f,c; expression e; @@ ( e = s@

[PATCH 5/5] arm: dts: rockchip: enable tsadc module on RV1108 evaluation board

2017-08-24 Thread Rocky Hao
Enable tsadc module on RV1108 evaluation board Signed-off-by: Rocky Hao --- arch/arm/boot/dts/rv1108-evb.dts | 4 1 file changed, 4 insertions(+) diff --git a/arch/arm/boot/dts/rv1108-evb.dts b/arch/arm/boot/dts/rv1108-evb.dts index 39197c3e9e0e..885f760d8ca1 100644 --- a/arch/arm/boot/dts

[PATCH 4/5] arm: dts: rockchip: add thermal nodes for RV1108 SoC

2017-08-24 Thread Rocky Hao
Add thermal zone and dynamic CPU power coefficients for RV1108 Signed-off-by: Rocky Hao --- arch/arm/boot/dts/rv1108.dtsi | 40 1 file changed, 40 insertions(+) diff --git a/arch/arm/boot/dts/rv1108.dtsi b/arch/arm/boot/dts/rv1108.dtsi index dbdd8c2180e7

Re: [PATCH v6] mtd: sharpslpart: Add sharpslpart partition parser

2017-08-24 Thread Andrea Adami
On Thu, Aug 24, 2017 at 12:04 PM, Boris Brezillon wrote: > On Thu, 24 Aug 2017 11:19:56 +0200 > Andrea Adami wrote: > >> >> +/** >> >> + * struct sharpsl_ftl - Sharp FTL Logical Table >> >> + * @logmax: number of logical blocks >> >> + * @log2phy: the logical-to-physical table >>

[PATCH 2/5] thermal: rockchip: Support the RV1108 SoC in thermal driver

2017-08-24 Thread Rocky Hao
RV1108 SOC has one Temperature Sensor for CPU. Signed-off-by: Rocky Hao --- drivers/thermal/rockchip_thermal.c | 67 ++ 1 file changed, 67 insertions(+) diff --git a/drivers/thermal/rockchip_thermal.c b/drivers/thermal/rockchip_thermal.c index 9da3e1819210..

[PATCH] rtc: ds1307: call the platform's logic for handle IRQs.

2017-08-24 Thread Enric Balletbo i Serra
On some systems the nIRQ pin is often connect to a GPIO, then, if a given interrupt line is supposed to wake up the system, the corresponding input of that interrupt controller need to be enabled to receive signal from the line in question. Before this commit such systems would not wake up because

[PATCH v4] x86/boot/KASLR: exclude EFI_BOOT_SERVICES_{CODE|DATA} from KASLR's choice

2017-08-24 Thread Naoya Horiguchi
KASLR chooses kernel location from E820_TYPE_RAM regions by walking over e820 entries now. E820_TYPE_RAM includes EFI_BOOT_SERVICES_CODE and EFI_BOOT_SERVICES_DATA, so those regions can be the target. According to UEFI spec, all memory regions marked as EfiBootServicesCode and EfiBootServicesData a

[PATCH v4 2/4] KVM: X86: Fix loss of exception which has not yet injected

2017-08-24 Thread Wanpeng Li
From: Wanpeng Li vmx_complete_interrupts() assumes that the exception is always injected, so it would be dropped by kvm_clear_exception_queue(). This patch separates exception.pending from exception.injected, exception.inject represents the exception is injected or the exception should be rein

[PATCH v4 1/4] KVM: VMX: use kvm_event_needs_reinjection

2017-08-24 Thread Wanpeng Li
From: Wanpeng Li Use kvm_event_needs_reinjection() encapsulation. Cc: Paolo Bonzini Cc: Radim Krčmář Signed-off-by: Wanpeng Li --- arch/x86/kvm/vmx.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/arch/x86/kvm/vmx.c b/arch/x86/kvm/vmx.c index dd710d3..c5f43ab 100644 -

[PATCH v4 4/4] KVM: nVMX: Fix trying to cancel vmlauch/vmresume

2017-08-24 Thread Wanpeng Li
From: Wanpeng Li [ cut here ] WARNING: CPU: 7 PID: 3861 at /home/kernel/ssd/kvm/arch/x86/kvm//vmx.c:11299 nested_vmx_vmexit+0x176e/0x1980 [kvm_intel] CPU: 7 PID: 3861 Comm: qemu-system-x86 Tainted: GW OE 4.13.0-rc4+ #11 RIP: 0010:nested_vmx_vmexit+0x176e/0x1980

[PATCH v4 3/4] KVM: VMX: Move the nested_vmx_inject_exception_vmexit call from nested_vmx_check_exception to vmx_queue_exception

2017-08-24 Thread Wanpeng Li
From: Wanpeng Li Move the nested_vmx_inject_exception_vmexit call from nested_vmx_check_exception to vmx_queue_exception. Cc: Paolo Bonzini Cc: Radim Krčmář Signed-off-by: Wanpeng Li --- arch/x86/kvm/vmx.c | 21 +++-- 1 file changed, 11 insertions(+), 10 deletions(-) diff --

Re: [PATCH v2 01/10] ASoC: s3c24xx_uda134x: Remove unnecessary function call

2017-08-24 Thread Mark Brown
On Wed, Aug 23, 2017 at 05:17:41AM -0700, Peng Donglin wrote: > The work of the function platform_set_drvdata is done by > devm_snd_soc_register_card. This doens't apply against current code, please check and resend. signature.asc Description: PGP signature

Re: [PATCH] irqchip: mmp: make mmp_intc_conf const

2017-08-24 Thread Marc Zyngier
On Thu, Aug 24 2017 at 3:56:21 pm BST, Bhumika Goyal wrote: > Make these const as they are only used during a copy operation. Done > using Coccinelle. > > @match disable optional_qualifier@ > identifier s; > @@ > static struct mmp_intc_conf s = {...}; > > @ref@ > position p; > identifier match.s

Re: [PATCH v2 02/10] ASoC: mediatek: Remove unnecessary function call

2017-08-24 Thread Mark Brown
On Wed, Aug 23, 2017 at 05:18:38AM -0700, Peng Donglin wrote: > The work of the function platform_set_drvdata is done by > devm_snd_soc_register_card. This also doesn't apply against current code, it looks like versions of this series have already been sent - please check the entire series and res

[PATCH] xfs: Drop setting redundant PF_KSWAPD in kswapd context

2017-08-24 Thread Anshuman Khandual
xfs_btree_split() calls xfs_btree_split_worker() with args.kswapd set if current->flags alrady has PF_KSWAPD. Hence we should not again add PF_KSWAPD into the current flags inside kswapd context. So drop this redundant flag addition. Signed-off-by: Anshuman Khandual --- fs/xfs/libxfs/xfs_btree.c

Re: [PATCH] ASoC: Add a sanity check before using dai driver name

2017-08-24 Thread jeffy
hi Mark, On 08/24/2017 06:18 PM, Mark Brown wrote: On Thu, Aug 24, 2017 at 11:29:42AM +0800, jeffy wrote: and i was trying to do something like: if (dlc->dai_name && strcmp(dai->name, dlc->dai_name) && (!dai->driver->name || strcmp(dai->driver->name, dlc->dai_name))) continue; whic

[RFC][PATCH]: sched/fair: search a task from the tail of the queue

2017-08-24 Thread Uladzislau Rezki (Sony)
Hi, while doing a load balancing (plus active) if we change a direction of task search we may benefit when it comes to performance. See below results of running hackbench with following parameters: 1000 samples 40 groups [1] ftp://vps418301.ovh.net/incoming/hackbench_40_groups_1000_samples_defau

Re: [PATCH v7 12/13] ACPI / init: Invoke early ACPI initialization earlier

2017-08-24 Thread Dou Liyang
At 08/24/2017 06:21 PM, Baoquan He wrote: On 08/24/17 at 05:28pm, Dou Liyang wrote: Hi Baoquan, Thanks for your reply. At 08/24/2017 04:05 PM, Baoquan He wrote: Hi Liyang, On 08/24/17 at 11:54am, Dou Liyang wrote: Test in my own PC(Lenovo M4340). Ask help for doing regression testing for

[RFC][PATCH]: sched/fair: search a task from the tail of the queue

2017-08-24 Thread Uladzislau Rezki (Sony)
From: Uladzislau Rezki When a task is enqueued back from a physical CPU to the running list it is placed in the beginning of the queue. Thus, the cfs_tasks list is more or less sorted (except woken tasks) starting from recently given CPU time tasks toward tasks with max wait time in a run-queue.

[PATCH nf-next] netfilter: xt_CHECKSUM: avoid bad offload warnings on GSO packets

2017-08-24 Thread Michal Kubecek
When --checksum_fill action is applied to a GSO packet, checksum_tg() calls skb_checksum_help() which is only meant to be applied to non-GSO packets so that it issues a warning. This can be easily triggered by using e.g. iptables -t mangle -A OUTPUT -j CHECKSUM --checksum-fill and sending TCP

Applied "spi: pl022: constify amba_id" to the spi tree

2017-08-24 Thread Mark Brown
The patch spi: pl022: constify amba_id has been applied to the spi tree at git://git.kernel.org/pub/scm/linux/kernel/git/broonie/spi.git All being well this means that it will be integrated into the linux-next tree (usually sometime in the next 24 hours) and sent to Linus during the next

Re: [PATCH 4/4] powerpc/32: remove a NOP from memset()

2017-08-24 Thread Michael Ellerman
Christophe Leroy writes: > memset() is patched after initialisation to activate the > optimised part which uses cache instructions. > > Today we have a 'b 2f' to skip the optimised patch, which then gets > replaced by a NOP, implying a useless cycle consumption. > As we have a 'bne 2f' just befor

Re: [PATCH 2/2] mmc: sdhci-msm: Enable delay circuit calibration clocks

2017-08-24 Thread Ulf Hansson
On 23 August 2017 at 19:28, Bjorn Andersson wrote: > On Tue 22 Aug 03:45 PDT 2017, Ulf Hansson wrote: > >> [...] >> >> > diff --git a/drivers/mmc/host/sdhci-msm.c b/drivers/mmc/host/sdhci-msm.c >> > index 71e01cbc38b6..7b47906ba447 100644 >> > --- a/drivers/mmc/host/sdhci-msm.c >> > +++ b/drivers/

Re: [PATCH 07/14] asm-generic: add kexec_file_load system call to unistd.h

2017-08-24 Thread Arnd Bergmann
On Thu, Aug 24, 2017 at 10:18 AM, AKASHI Takahiro wrote: > The initial user of this system call number is arm64. > > Signed-off-by: AKASHI Takahiro > Cc: Arnd Bergmann Acked-by:Arnd Bergmann

Re: [PATCH nf-next] netfilter: xt_CHECKSUM: avoid bad offload warnings on GSO packets

2017-08-24 Thread Florian Westphal
Michal Kubecek wrote: > When --checksum_fill action is applied to a GSO packet, checksum_tg() calls > skb_checksum_help() which is only meant to be applied to non-GSO packets so > that it issues a warning. > > This can be easily triggered by using e.g. > > iptables -t mangle -A OUTPUT -j CHECK

Re: [PATCH V9 1/2] powerpc/numa: Update CPU topology when VPHN enabled

2017-08-24 Thread Michael Ellerman
Nathan Fontenot writes: > On 08/23/2017 06:41 AM, Michael Ellerman wrote: >> Michael Bringmann writes: >> >>> powerpc/numa: Correct the currently broken capability to set the >>> topology for shared CPUs in LPARs. At boot time for shared CPU >>> lpars, the topology for each shared CPU is set t

Re: [PATCH] xfs: Drop setting redundant PF_KSWAPD in kswapd context

2017-08-24 Thread Michal Hocko
On Thu 24-08-17 16:12:47, Anshuman Khandual wrote: > xfs_btree_split() calls xfs_btree_split_worker() with args.kswapd set > if current->flags alrady has PF_KSWAPD. Hence we should not again add > PF_KSWAPD into the current flags inside kswapd context. So drop this > redundant flag addition. I am

Re: [PATCH v2] scsi: lpfc: avoid false positive gcc-8 warning

2017-08-24 Thread Arnd Bergmann
On Thu, Aug 24, 2017 at 2:34 AM, James Smart wrote: > Arnd Bergmann, testing gcc-8, encountered the following: > >> This is an interesting regression with gcc-8, showing a harmless >> warning for correct code: >> >>In file included from include/linux/kernel.h:13:0, >>... >>from drivers/scs

Re: [PATCH v7 3/4] lib/cmdline.c Remove quotes symmetrically.

2017-08-24 Thread Michael Ellerman
Michal Suchanek writes: > Remove quotes from argument value only if there is qoute on both sides. > > Signed-off-by: Michal Suchanek > --- > arch/powerpc/kernel/fadump.c | 6 ++ > lib/cmdline.c| 7 ++- Can you split that into two patches? cheers > 2 files changed, 4 i

Re: Do we really need d_weak_revalidate???

2017-08-24 Thread Michael Kerrisk (man-pages)
Hi Neil, On 24 August 2017 at 06:07, NeilBrown wrote: > On Wed, Aug 23 2017, Ian Kent wrote: > >> >> That inconsistency has bothered me for quite a while now. >> >> It was carried over from the autofs module behavior when automounting >> support was added to the VFS. What's worse is it prevents t

Re: [PATCH v7 4/4] boot/param: add pointer to next argument to unknown parameter callback

2017-08-24 Thread Michael Ellerman
Michal Suchanek writes: > The fadump parameter processing re-does the logic of next_arg quote > stripping to determine where the argument ends. Pass pointer to the > next argument instead to make this more robust. > > Signed-off-by: Michal Suchanek > --- > arch/powerpc/kernel/fadump.c | 13 +++

Re: [PATCH] mm/mlock: use page_zone() instead of page_zone_id()

2017-08-24 Thread Vlastimil Babka
+CC Mel On 08/24/2017 09:20 AM, js1...@gmail.com wrote: > From: Joonsoo Kim > > page_zone_id() is a specialized function to compare the zone for the pages > that are within the section range. If the section of the pages are > different, page_zone_id() can be different even if their zone is the s

Re: [PATCH nf-next] netfilter: xt_CHECKSUM: avoid bad offload warnings on GSO packets

2017-08-24 Thread Michal Kubecek
On Thu, Aug 24, 2017 at 12:51:18PM +0200, Florian Westphal wrote: > Michal Kubecek wrote: > > When --checksum_fill action is applied to a GSO packet, checksum_tg() calls > > skb_checksum_help() which is only meant to be applied to non-GSO packets so > > that it issues a warning. > > > > This can

Re: [PATCH v5 0/4] Add support for ThunderX2 pmu events using json files

2017-08-24 Thread John Garry
On 24/08/2017 08:23, Ganapatrao Kulkarni wrote: Hi John, On Wed, Aug 23, 2017 at 2:52 PM, John Garry wrote: On 22/08/2017 05:21, Ganapatrao Kulkarni wrote: Hi Arnaldo, Will, are there any comments on this series? Hi Ganapatrao, Is it possible to have vendor sub-folder in tools/perf/pmu-

Re: [PATCH] ASoC: rt5677: Reintroduce I2C device IDs

2017-08-24 Thread Tom Rini
On Thu, Aug 24, 2017 at 10:39:09AM +0300, Andy Shevchenko wrote: > On Wed, 2017-08-23 at 20:05 -0400, Tom Rini wrote: > > On Wed, Aug 23, 2017 at 01:39:12PM -0400, Tom Rini wrote: > > > On Wed, Aug 23, 2017 at 05:29:33PM +0300, Andy Shevchenko wrote: > > > > > > > On Tue, 2017-08-22 at 21:51 -0400

Re: [v6 1/4] mm, oom: refactor the oom_kill_process() function

2017-08-24 Thread Michal Hocko
This patch fails to apply on top of the mmotm tree. It seems the only reason is the missing http://lkml.kernel.org/r/20170810075019.28998-2-mho...@kernel.org On Wed 23-08-17 17:51:57, Roman Gushchin wrote: > The oom_kill_process() function consists of two logical parts: > the first one is responsi

[PATCH] USB: core: constify vm_operations_struct

2017-08-24 Thread Arvind Yadav
vm_operations_struct are not supposed to change at runtime. All functions working with const vm_operations_struct. So mark the non-const structs as const. Signed-off-by: Arvind Yadav --- drivers/usb/core/devio.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/usb/core

Re: [PATCH v6] mtd: sharpslpart: Add sharpslpart partition parser

2017-08-24 Thread Boris Brezillon
On Thu, 24 Aug 2017 12:30:02 +0200 Andrea Adami wrote: > On Thu, Aug 24, 2017 at 12:04 PM, Boris Brezillon > wrote: > > On Thu, 24 Aug 2017 11:19:56 +0200 > > Andrea Adami wrote: > > > >> >> +/** > >> >> + * struct sharpsl_ftl - Sharp FTL Logical Table > >> >> + * @logmax: number of

Re: [PATCH] rwsem: fix missed wakeup due to reordering of load

2017-08-24 Thread Peter Zijlstra
WTH did you not Cc the people that commented on your patch last time? On Wed, Aug 23, 2017 at 04:58:55PM +0530, Prateek Sood wrote: > If a spinner is present, there is a chance that the load of > rwsem_has_spinner() in rwsem_wake() can be reordered with > respect to decrement of rwsem count in __

Re: [PATCH v3 3/5] dt-bindings: sdhci-omap: Add bindings for the sdhci-omap controller

2017-08-24 Thread Ulf Hansson
On 23 August 2017 at 15:56, Kishon Vijay Abraham I wrote: > Hi Uffe, > > On Wednesday 23 August 2017 06:37 PM, Ulf Hansson wrote: >> On 23 August 2017 at 07:42, Kishon Vijay Abraham I wrote: >>> Add binding for the TI's sdhci-omap controller. This now includes only >>> a subset of properties docu

[PATCH] rapidio: remove global irq spinlocks from the subsystem

2017-08-24 Thread Ioan Nicu
Locking of config and doorbell operations should be done only if the underlying hardware requires it. This patch removes the global spinlocks from the rapidio subsystem and moves them to the mport drivers (fsl_rio and tsi721), only to the necessary places. For example, local config space read and

Re: [PATCH RFC] media: open.rst: document devnode-centric and mc-centric types

2017-08-24 Thread Mauro Carvalho Chehab
Em Thu, 24 Aug 2017 12:18:56 +0300 Sakari Ailus escreveu: > Hi Mauro, > > On Wed, Aug 23, 2017 at 10:42:28AM -0300, Mauro Carvalho Chehab wrote: > > Em Wed, 23 Aug 2017 12:37:30 +0300 > > Sakari Ailus escreveu: > > > > > Hi Mauro, > > > > > > Thanks for the patch! Something like this was lo

[PATCH][gpio-next] gpio: mockup: remove unused variable gc

2017-08-24 Thread Colin King
From: Colin Ian King The variable gc is assigned but never read and is redundant. Remove it. Cleans up clang warning: drivers/gpio/gpio-mockup.c:169:2: warning: Value stored to 'gc' is never read Signed-off-by: Colin Ian King --- drivers/gpio/gpio-mockup.c | 2 -- 1 file changed, 2 deletions(

Re: [PATCH v4 2/4] KVM: X86: Fix loss of exception which has not yet injected

2017-08-24 Thread Paolo Bonzini
On 24/08/2017 12:35, Wanpeng Li wrote: > From: Wanpeng Li > > vmx_complete_interrupts() assumes that the exception is always injected, > so it would be dropped by kvm_clear_exception_queue(). This patch separates > exception.pending from exception.injected, exception.inject represents the > ex

Intermittent memory corruption with v4.13-rc6+ and earlier

2017-08-24 Thread Mark Rutland
ues that I suspect are the result of this memory corruption. I've included an example splat below, complete with KASAN's alloc/free traces at the end of this mail. I've dumped more info in the bugs/20170824-file-uaf directory in my kernel.org web space [1], including a number of lo

[PATCH] ARM: at91: Replace uses of virt_to_phys with __pa_symbol

2017-08-24 Thread Alexandre Belloni
The PM code wrongly uses virt_to_phys() instead of __pa_symbol() and was not updated by commit 64fc2a947a98 ("ARM: 8641/1: treewide: Replace uses of virt_to_phys with __pa_symbol") because it was not yet in tree. Signed-off-by: Alexandre Belloni --- arch/arm/mach-at91/pm.c | 4 ++-- 1 file chang

Re: [PATCH v2 2/3] platform/x86: Add support for Dollar Cove TI power button

2017-08-24 Thread Andy Shevchenko
On Thu, Aug 24, 2017 at 12:45 PM, Takashi Iwai wrote: > On Thu, 24 Aug 2017 11:20:04 +0200, > Andy Shevchenko wrote: >> >> On Thu, Aug 24, 2017 at 11:11 AM, Takashi Iwai wrote: >> > This provides a new input driver for supporting the power button on >> > Dollar Cove TI PMIC, found on Cherrytrail-

Re: [v6 2/4] mm, oom: cgroup-aware OOM killer

2017-08-24 Thread Michal Hocko
This doesn't apply on top of mmotm cleanly. You are missing http://lkml.kernel.org/r/20170807113839.16695-3-mho...@kernel.org On Wed 23-08-17 17:51:59, Roman Gushchin wrote: > Traditionally, the OOM killer is operating on a process level. > Under oom conditions, it finds a process with the highest

Re: Intermittent memory corruption with v4.13-rc6+ and earlier

2017-08-24 Thread Michal Hocko
On Thu 24-08-17 12:37:43, Mark Rutland wrote: > Hi, > > I'm chasing intermittent memory corruption bugs seen at least on rc5, > rc6, and yesterday's HEAD (98b9f8a4549909c6), on arm64. > > It looks like we make use of dangling references to a freed struct file, > which is caught by KASAN. Without

<    1   2   3   4   5   6   7   8   9   10   >