Re: [PATCH] mm/memory-failure: fix race with compound page split/merge

2016-04-18 Thread Naoya Horiguchi
# CCed Andrew, On Mon, Apr 18, 2016 at 02:43:45PM +0300, Konstantin Khlebnikov wrote: > Get_hwpoison_page() must recheck relation between head and tail pages. > > Signed-off-by: Konstantin Khlebnikov Looks good to me. Without this recheck, the race causes kernel to

Re: [PATCH] mm/memory-failure: fix race with compound page split/merge

2016-04-18 Thread Naoya Horiguchi
# CCed Andrew, On Mon, Apr 18, 2016 at 02:43:45PM +0300, Konstantin Khlebnikov wrote: > Get_hwpoison_page() must recheck relation between head and tail pages. > > Signed-off-by: Konstantin Khlebnikov Looks good to me. Without this recheck, the race causes kernel to pin an irrelevant page, and

[PATCH] perf jit: memset variable st using the correct size

2016-04-18 Thread Colin King
From: Colin Ian King The current code is memsetting the struct stat variable st with the size of stat (which turns out to be 1 byte) rather than the size of variable sz. Signed-off-by: Colin Ian King --- tools/perf/util/jitdump.c | 4 ++-- 1

[PATCH] perf jit: memset variable st using the correct size

2016-04-18 Thread Colin King
From: Colin Ian King The current code is memsetting the struct stat variable st with the size of stat (which turns out to be 1 byte) rather than the size of variable sz. Signed-off-by: Colin Ian King --- tools/perf/util/jitdump.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff

[PATCH] tpm_crb: fix mapping of the buffers

2016-04-18 Thread Jarkko Sakkinen
On my Lenovo x250 the following situation occurs: [18697.813871] tpm_crb MSFT0101:00: can't request region for resource [mem 0xacdff080-0xacdf] The mapping of the control area interleaves the mapping of the command buffer. The control area is mapped over page, which is not right. It should

[PATCH] tpm_crb: fix mapping of the buffers

2016-04-18 Thread Jarkko Sakkinen
On my Lenovo x250 the following situation occurs: [18697.813871] tpm_crb MSFT0101:00: can't request region for resource [mem 0xacdff080-0xacdf] The mapping of the control area interleaves the mapping of the command buffer. The control area is mapped over page, which is not right. It should

Re: [PATCH] fs: add the FIGETFROZEN ioctl call

2016-04-18 Thread Dave Chinner
On Mon, Apr 18, 2016 at 11:20:22AM -0400, Eric Sandeen wrote: > > > On 4/14/16 10:17 PM, Dave Chinner wrote: > > On Thu, Apr 14, 2016 at 09:57:07AM +0200, Florian Margaine wrote: > >> This lets userland get the filesystem freezing status, aka whether the > >> filesystem is frozen or not. This is

Re: [PATCH] fs: add the FIGETFROZEN ioctl call

2016-04-18 Thread Dave Chinner
On Mon, Apr 18, 2016 at 11:20:22AM -0400, Eric Sandeen wrote: > > > On 4/14/16 10:17 PM, Dave Chinner wrote: > > On Thu, Apr 14, 2016 at 09:57:07AM +0200, Florian Margaine wrote: > >> This lets userland get the filesystem freezing status, aka whether the > >> filesystem is frozen or not. This is

Re: [PATCH] IMA: add INTEGRITY_ASYMMETRIC_KEYS dependency

2016-04-18 Thread Mimi Zohar
Hi Arnd, On Sat, 2016-04-16 at 22:33 +0200, Arnd Bergmann wrote: > The newly added CONFIG_IMA_KEYRINGS_PERMIT_SIGNED_BY_BUILTIN_OR_SECONDARY > option only makes sense in combination with INTEGRITY_ASYMMETRIC_KEYS, > otherwise we get a build error: > > warning:

Re: [PATCH] IMA: add INTEGRITY_ASYMMETRIC_KEYS dependency

2016-04-18 Thread Mimi Zohar
Hi Arnd, On Sat, 2016-04-16 at 22:33 +0200, Arnd Bergmann wrote: > The newly added CONFIG_IMA_KEYRINGS_PERMIT_SIGNED_BY_BUILTIN_OR_SECONDARY > option only makes sense in combination with INTEGRITY_ASYMMETRIC_KEYS, > otherwise we get a build error: > > warning:

Re: [PATCHv7 00/29] THP-enabled tmpfs/shmem using compound pages

2016-04-18 Thread Shi, Yang
Hi Kirill, Finally, I got some time to look into and try yours and Hugh's patches, got two problems. 1. A quick boot up test on my ARM64 machine with your v7 tree shows some unexpected error: systemd-journald[285]: Failed to save stream data /run/systemd/journal/streams/8:16863: No space

Re: [PATCHv7 00/29] THP-enabled tmpfs/shmem using compound pages

2016-04-18 Thread Shi, Yang
Hi Kirill, Finally, I got some time to look into and try yours and Hugh's patches, got two problems. 1. A quick boot up test on my ARM64 machine with your v7 tree shows some unexpected error: systemd-journald[285]: Failed to save stream data /run/systemd/journal/streams/8:16863: No space

Re: [PATCH] net: phy: Ensure the state machine is called when phy is UP

2016-04-18 Thread Alexandre Belloni
On 18/04/2016 at 15:17:58 -0700, Florian Fainelli wrote : > Yes, seems like it, how about adding this: > > diff --git a/drivers/net/ethernet/cadence/macb.c > b/drivers/net/ethernet/cadence/macb.c > index 98b99149ce0b..21096dfb0e83 100644 > --- a/drivers/net/ethernet/cadence/macb.c > +++

Re: [PATCH V4 4/6] slim: qcom: Add Qualcomm Slimbus controller driver

2016-04-18 Thread Sagar Dharia
On 4/18/2016 3:19 AM, Mark Brown wrote: On Fri, Apr 15, 2016 at 10:17:36AM -0600, Sagar Dharia wrote: Please leave blank lines between paragraphs, it makes things much easier to read than a wall of uninterrupted text. + ret = devm_request_irq(>dev, dev->irq, msm_slim_interrupt, +

Re: [PATCH] net: phy: Ensure the state machine is called when phy is UP

2016-04-18 Thread Alexandre Belloni
On 18/04/2016 at 15:17:58 -0700, Florian Fainelli wrote : > Yes, seems like it, how about adding this: > > diff --git a/drivers/net/ethernet/cadence/macb.c > b/drivers/net/ethernet/cadence/macb.c > index 98b99149ce0b..21096dfb0e83 100644 > --- a/drivers/net/ethernet/cadence/macb.c > +++

Re: [PATCH V4 4/6] slim: qcom: Add Qualcomm Slimbus controller driver

2016-04-18 Thread Sagar Dharia
On 4/18/2016 3:19 AM, Mark Brown wrote: On Fri, Apr 15, 2016 at 10:17:36AM -0600, Sagar Dharia wrote: Please leave blank lines between paragraphs, it makes things much easier to read than a wall of uninterrupted text. + ret = devm_request_irq(>dev, dev->irq, msm_slim_interrupt, +

Chevrolet Auto Mobile Award 2016‏

2016-04-18 Thread Chevrolet Company Uk
Good news to you from the Chevrolet Company Uk.Your email-I.D was selected in the Chevrolet Auto mobile award 2016.The sum of $570,000,00USD has been awarded to you as the beneficiary. Send your details for claim: chevroletcompna...@outlook.com Name: Add: Mobile Number: Country: Reply on:

Chevrolet Auto Mobile Award 2016‏

2016-04-18 Thread Chevrolet Company Uk
Good news to you from the Chevrolet Company Uk.Your email-I.D was selected in the Chevrolet Auto mobile award 2016.The sum of $570,000,00USD has been awarded to you as the beneficiary. Send your details for claim: chevroletcompna...@outlook.com Name: Add: Mobile Number: Country: Reply on:

Re: [PATCH V1] rtc: da9053: fix access ordering error during RTC interrupt at system power on

2016-04-18 Thread Alexandre Belloni
On 14/04/2016 at 12:04:54 +0100, Steve Twiss wrote : > From: Steve Twiss > > This fix alters the ordering of the IRQ and device registrations in the RTC > driver probe function. This change will apply to the RTC driver that supports > both DA9052 and DA9053 PMICs.

Re: [PATCH V1] rtc: da9053: fix access ordering error during RTC interrupt at system power on

2016-04-18 Thread Alexandre Belloni
On 14/04/2016 at 12:04:54 +0100, Steve Twiss wrote : > From: Steve Twiss > > This fix alters the ordering of the IRQ and device registrations in the RTC > driver probe function. This change will apply to the RTC driver that supports > both DA9052 and DA9053 PMICs. > > A problem could occur with

Re: [PATCH] rtc: ds1685: actually spin forever in poweroff path, part 2

2016-04-18 Thread Alexandre Belloni
On 15/04/2016 at 09:21:10 -0500, Josh Poimboeuf wrote : > objtool reports the following warning: > > drivers/rtc/rtc-ds1685.o: warning: objtool: ds1685_rtc_poweroff() falls > through to next function ds1685_rtc_work_queue() > > Similar to commit 361c6ed6b153 ("rtc: ds1685: actually spin

Re: [PATCH] rtc: ds1685: actually spin forever in poweroff path, part 2

2016-04-18 Thread Alexandre Belloni
On 15/04/2016 at 09:21:10 -0500, Josh Poimboeuf wrote : > objtool reports the following warning: > > drivers/rtc/rtc-ds1685.o: warning: objtool: ds1685_rtc_poweroff() falls > through to next function ds1685_rtc_work_queue() > > Similar to commit 361c6ed6b153 ("rtc: ds1685: actually spin

[PATCH net-next] net: dsa: remove tag_protocol from dsa_switch

2016-04-18 Thread Vivien Didelot
Having the tag protocol in dsa_switch_driver for setup time and in dsa_switch_tree for runtime is enough. Remove dsa_switch's one. Signed-off-by: Vivien Didelot --- include/net/dsa.h | 5 - net/dsa/dsa.c | 5 ++--- 2 files changed, 2 insertions(+), 8

[PATCH net-next] net: dsa: remove tag_protocol from dsa_switch

2016-04-18 Thread Vivien Didelot
Having the tag protocol in dsa_switch_driver for setup time and in dsa_switch_tree for runtime is enough. Remove dsa_switch's one. Signed-off-by: Vivien Didelot --- include/net/dsa.h | 5 - net/dsa/dsa.c | 5 ++--- 2 files changed, 2 insertions(+), 8 deletions(-) diff --git

Re: [PATCH] net: phy: Ensure the state machine is called when phy is UP

2016-04-18 Thread Florian Fainelli
On 18/04/16 15:14, Alexandre Belloni wrote: > On 15/04/2016 at 15:23:39 -0700, Florian Fainelli wrote : >> On 15/04/16 15:17, Alexandre Belloni wrote: >>> On 16/04/2016 at 00:05:08 +0200, Andrew Lunn wrote : > Trace without my patch: > libphy: MACB_mii_bus: probed > macb

Re: [PATCH] net: phy: Ensure the state machine is called when phy is UP

2016-04-18 Thread Florian Fainelli
On 18/04/16 15:14, Alexandre Belloni wrote: > On 15/04/2016 at 15:23:39 -0700, Florian Fainelli wrote : >> On 15/04/16 15:17, Alexandre Belloni wrote: >>> On 16/04/2016 at 00:05:08 +0200, Andrew Lunn wrote : > Trace without my patch: > libphy: MACB_mii_bus: probed > macb

Re: [PATCH] drivers: led: is31fl319x: 6/9-channel light effect led driver

2016-04-18 Thread kbuild test robot
Hi Nikolaus, [auto build test ERROR on j.anaszewski-leds/for-next] [also build test ERROR on v4.6-rc4 next-20160418] [if your patch is applied to the wrong git tree, please drop us a note to help improving the system] url: https://github.com/0day-ci/linux/commits/H-Nikolaus-Schaller/drivers

Re: [PATCH] drivers: led: is31fl319x: 6/9-channel light effect led driver

2016-04-18 Thread kbuild test robot
Hi Nikolaus, [auto build test ERROR on j.anaszewski-leds/for-next] [also build test ERROR on v4.6-rc4 next-20160418] [if your patch is applied to the wrong git tree, please drop us a note to help improving the system] url: https://github.com/0day-ci/linux/commits/H-Nikolaus-Schaller/drivers

[PATCH] tools/acpi: fix inverted logic when closing fd

2016-04-18 Thread Colin King
From: Colin Ian King The current close on fd only occurs when fd is -ve, it should actually be closing the file if it a valid fd. Fix the inverted logic. Signed-off-by: Colin Ian King --- tools/power/acpi/tools/acpidbg/acpidbg.c | 2 +- 1

[PATCH] tools/acpi: fix inverted logic when closing fd

2016-04-18 Thread Colin King
From: Colin Ian King The current close on fd only occurs when fd is -ve, it should actually be closing the file if it a valid fd. Fix the inverted logic. Signed-off-by: Colin Ian King --- tools/power/acpi/tools/acpidbg/acpidbg.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff

Re: [PATCH net-next 2/8] perf, bpf: allow bpf programs attach to tracepoints

2016-04-18 Thread Steven Rostedt
On Mon, 18 Apr 2016 14:43:07 -0700 Alexei Starovoitov wrote: > I was worried about this too, but single 'if' and two calls > (as in commit 98b5c2c65c295) is a better way, since it's faster, cleaner > and doesn't need to refactor the whole perf_trace_buf_submit() to pass > extra

Re: [PATCH net-next 2/8] perf, bpf: allow bpf programs attach to tracepoints

2016-04-18 Thread Steven Rostedt
On Mon, 18 Apr 2016 14:43:07 -0700 Alexei Starovoitov wrote: > I was worried about this too, but single 'if' and two calls > (as in commit 98b5c2c65c295) is a better way, since it's faster, cleaner > and doesn't need to refactor the whole perf_trace_buf_submit() to pass > extra event_call

Re: [PATCH] net: phy: Ensure the state machine is called when phy is UP

2016-04-18 Thread Alexandre Belloni
On 15/04/2016 at 15:23:39 -0700, Florian Fainelli wrote : > On 15/04/16 15:17, Alexandre Belloni wrote: > > On 16/04/2016 at 00:05:08 +0200, Andrew Lunn wrote : > >>> Trace without my patch: > >>> libphy: MACB_mii_bus: probed > >>> macb f802.ethernet eth0: Cadence GEM rev 0x00020120 at

Re: [PATCH] net: phy: Ensure the state machine is called when phy is UP

2016-04-18 Thread Alexandre Belloni
On 15/04/2016 at 15:23:39 -0700, Florian Fainelli wrote : > On 15/04/16 15:17, Alexandre Belloni wrote: > > On 16/04/2016 at 00:05:08 +0200, Andrew Lunn wrote : > >>> Trace without my patch: > >>> libphy: MACB_mii_bus: probed > >>> macb f802.ethernet eth0: Cadence GEM rev 0x00020120 at

alloc failure in qla1280 probe -- need to decrease can_queue?

2016-04-18 Thread Laura Abbott
Hi, We received a bug report https://bugzilla.redhat.com/show_bug.cgi?id=1321033 of qla1280 scsi host failure on 4.4 based kernels that looks to be caused by page alloc failure: [4.804166] scsi host0: QLogic QLA1040 PCI to SCSI Host Adapter Firmware version: 7.65.06,

alloc failure in qla1280 probe -- need to decrease can_queue?

2016-04-18 Thread Laura Abbott
Hi, We received a bug report https://bugzilla.redhat.com/show_bug.cgi?id=1321033 of qla1280 scsi host failure on 4.4 based kernels that looks to be caused by page alloc failure: [4.804166] scsi host0: QLogic QLA1040 PCI to SCSI Host Adapter Firmware version: 7.65.06,

Re: [PART1 RFC v4 02/11] KVM: x86: Introducing kvm_x86_ops VM init/uninit hooks

2016-04-18 Thread Suravee Suthikulpanit
Paolo, On 04/12/2016 04:55 PM, Paolo Bonzini wrote: On 11/04/2016 22:49, Radim Krčmář wrote: @@ -7781,6 +7784,9 @@ static void kvm_free_vcpus(struct kvm *kvm) kvm_for_each_vcpu(i, vcpu, kvm) kvm_arch_vcpu_free(vcpu); + if (kvm_x86_ops->vm_uninit) +

Re: [PART1 RFC v4 02/11] KVM: x86: Introducing kvm_x86_ops VM init/uninit hooks

2016-04-18 Thread Suravee Suthikulpanit
Paolo, On 04/12/2016 04:55 PM, Paolo Bonzini wrote: On 11/04/2016 22:49, Radim Krčmář wrote: @@ -7781,6 +7784,9 @@ static void kvm_free_vcpus(struct kvm *kvm) kvm_for_each_vcpu(i, vcpu, kvm) kvm_arch_vcpu_free(vcpu); + if (kvm_x86_ops->vm_uninit) +

[PATCH] rtl8xxxu: hide unused tables

2016-04-18 Thread Arnd Bergmann
The references to some arrays in the rtl8xxxu driver were moved inside of an #ifdef, but the symbols remain outside, resulting in build warnings: rtl8xxxu/rtl8xxxu.c:1506:33: error: 'rtl8188ru_radioa_1t_highpa_table' defined but not used rtl8xxxu/rtl8xxxu.c:1431:33: error:

[PATCH] rtl8xxxu: hide unused tables

2016-04-18 Thread Arnd Bergmann
The references to some arrays in the rtl8xxxu driver were moved inside of an #ifdef, but the symbols remain outside, resulting in build warnings: rtl8xxxu/rtl8xxxu.c:1506:33: error: 'rtl8188ru_radioa_1t_highpa_table' defined but not used rtl8xxxu/rtl8xxxu.c:1431:33: error:

[PATCH] net: w5100: don't build spi driver without w5100

2016-04-18 Thread Arnd Bergmann
The w5100-spi driver front-end only makes sense when the w5100 core driver is enabled, not for a configuration that only has w5300: drivers/net/built-in.o: In function `w5100_spi_remove': drivers/net/ethernet/wiznet/w5100-spi.c:277: undefined reference to `w5100_remove' drivers/net/built-in.o:

[PATCH] net: w5100: don't build spi driver without w5100

2016-04-18 Thread Arnd Bergmann
The w5100-spi driver front-end only makes sense when the w5100 core driver is enabled, not for a configuration that only has w5300: drivers/net/built-in.o: In function `w5100_spi_remove': drivers/net/ethernet/wiznet/w5100-spi.c:277: undefined reference to `w5100_remove' drivers/net/built-in.o:

Re: [PATCH 0/2] Persistent scrollback buffers for all VGA consoles

2016-04-18 Thread Jakub Wilk
* Manuel Schölling , 2016-04-18, 21:56: To offer a way for userland tools to flush the buffer my second patch adds a ioctl call for that. In f8df13e0a901fe55631fed66562369b4dba40f8b, the escape sequence \e[3J was added, which erases the whole display including the

Re: [PATCH 0/2] Persistent scrollback buffers for all VGA consoles

2016-04-18 Thread Jakub Wilk
* Manuel Schölling , 2016-04-18, 21:56: To offer a way for userland tools to flush the buffer my second patch adds a ioctl call for that. In f8df13e0a901fe55631fed66562369b4dba40f8b, the escape sequence \e[3J was added, which erases the whole display including the scroll-back buffer. So the

Re: [PATCH 4/5] input: twl6040-vibra: ignore return value of schedule_work

2016-04-18 Thread Dmitry Torokhov
On Mon, Apr 18, 2016 at 09:55:40PM +0200, H. Nikolaus Schaller wrote: > returning ret is wrong. And checking for an error as well. User space > may call multiple times until the work is really scheduled. > > twl4030-vibra.c also ignores the return value. > > Signed-off-by: H. Nikolaus Schaller

Re: [PATCH 4/5] input: twl6040-vibra: ignore return value of schedule_work

2016-04-18 Thread Dmitry Torokhov
On Mon, Apr 18, 2016 at 09:55:40PM +0200, H. Nikolaus Schaller wrote: > returning ret is wrong. And checking for an error as well. User space > may call multiple times until the work is really scheduled. > > twl4030-vibra.c also ignores the return value. > > Signed-off-by: H. Nikolaus Schaller

Re: [PATCH 3/5] input: twl6040-vibra: fix NULL pointer dereference by removing workqueue

2016-04-18 Thread Dmitry Torokhov
On Mon, Apr 18, 2016 at 09:55:39PM +0200, H. Nikolaus Schaller wrote: > commit 21fb9f0d5e91 ("Input: twl6040-vibra - use system workqueue") > > says that it switches to use the system workqueue but it did neither > > - remove the workqueue struct variable > - replace code to really use the

Re: [PATCH 3/5] input: twl6040-vibra: fix NULL pointer dereference by removing workqueue

2016-04-18 Thread Dmitry Torokhov
On Mon, Apr 18, 2016 at 09:55:39PM +0200, H. Nikolaus Schaller wrote: > commit 21fb9f0d5e91 ("Input: twl6040-vibra - use system workqueue") > > says that it switches to use the system workqueue but it did neither > > - remove the workqueue struct variable > - replace code to really use the

Re: [PATCH] macsec: fix crypto Kconfig dependency

2016-04-18 Thread Stephen Rothwell
Hi Dave, On Mon, 18 Apr 2016 12:43:16 -0400 (EDT) David Miller wrote: > > From: Herbert Xu > Date: Mon, 18 Apr 2016 18:43:36 +0800 > > > Right, the problem is that nothing within crypto ever selects > > CRYPTO since it's also used as a way of

Re: [PATCH] macsec: fix crypto Kconfig dependency

2016-04-18 Thread Stephen Rothwell
Hi Dave, On Mon, 18 Apr 2016 12:43:16 -0400 (EDT) David Miller wrote: > > From: Herbert Xu > Date: Mon, 18 Apr 2016 18:43:36 +0800 > > > Right, the problem is that nothing within crypto ever selects > > CRYPTO since it's also used as a way of hiding the crypto menu > > options. > > As far

Re: [PATCH net-next 2/8] perf, bpf: allow bpf programs attach to tracepoints

2016-04-18 Thread Alexei Starovoitov
On 4/18/16 1:29 PM, Steven Rostedt wrote: On Mon, 4 Apr 2016 21:52:48 -0700 Alexei Starovoitov wrote: introduce BPF_PROG_TYPE_TRACEPOINT program type and allow it to be attached to tracepoints. The tracepoint will copy the arguments in the per-cpu buffer and pass it to the bpf

Re: [PATCH net-next 2/8] perf, bpf: allow bpf programs attach to tracepoints

2016-04-18 Thread Alexei Starovoitov
On 4/18/16 1:29 PM, Steven Rostedt wrote: On Mon, 4 Apr 2016 21:52:48 -0700 Alexei Starovoitov wrote: introduce BPF_PROG_TYPE_TRACEPOINT program type and allow it to be attached to tracepoints. The tracepoint will copy the arguments in the per-cpu buffer and pass it to the bpf program as its

Re: [PATCH 1/2] localmodconfig: Fix parsing of Kconfig "source" statements

2016-04-18 Thread Steven Rostedt
On Mon, 18 Apr 2016 13:29:12 -0700 Benjamin Poirier wrote: > On 2016/04/10 16:52, Benjamin Poirier wrote: > > On 2016/04/08 14:29, Steven Rostedt wrote: > > > On Sat, 2 Apr 2016 10:55:21 -0700 > > > Benjamin Poirier wrote: > > > > > > > The parameter

Re: [PATCH 1/2] localmodconfig: Fix parsing of Kconfig "source" statements

2016-04-18 Thread Steven Rostedt
On Mon, 18 Apr 2016 13:29:12 -0700 Benjamin Poirier wrote: > On 2016/04/10 16:52, Benjamin Poirier wrote: > > On 2016/04/08 14:29, Steven Rostedt wrote: > > > On Sat, 2 Apr 2016 10:55:21 -0700 > > > Benjamin Poirier wrote: > > > > > > > The parameter of Kconfig "source" statements does

Re: [PATCH 0/2] memory_hotplug: introduce config and command line options to set the default onlining policy

2016-04-18 Thread David Rientjes
On Thu, 7 Apr 2016, Vitaly Kuznetsov wrote: > >> > This patchset continues the work I started with: > >> > > >> > commit 31bc3858ea3ebcc3157b3f5f0e624c5962f5a7a6 > >> > Author: Vitaly Kuznetsov > >> > Date: Tue Mar 15 14:56:48 2016 -0700 > >> > > >> > memory-hotplug:

Re: [PATCH 0/2] memory_hotplug: introduce config and command line options to set the default onlining policy

2016-04-18 Thread David Rientjes
On Thu, 7 Apr 2016, Vitaly Kuznetsov wrote: > >> > This patchset continues the work I started with: > >> > > >> > commit 31bc3858ea3ebcc3157b3f5f0e624c5962f5a7a6 > >> > Author: Vitaly Kuznetsov > >> > Date: Tue Mar 15 14:56:48 2016 -0700 > >> > > >> > memory-hotplug: add automatic

Re: [PATCH net-next 0/8] allow bpf attach to tracepoints

2016-04-18 Thread Alexei Starovoitov
On 4/18/16 1:47 PM, Steven Rostedt wrote: On Mon, 18 Apr 2016 12:51:43 -0700 Alexei Starovoitov wrote: yeah, it could be added to ftrace as well, but it won't be as effective as perf_trace, since the cost of trace_event_buffer_reserve() in trace_event_raw_event_() handler is

Re: [PATCH net-next 0/8] allow bpf attach to tracepoints

2016-04-18 Thread Alexei Starovoitov
On 4/18/16 1:47 PM, Steven Rostedt wrote: On Mon, 18 Apr 2016 12:51:43 -0700 Alexei Starovoitov wrote: yeah, it could be added to ftrace as well, but it won't be as effective as perf_trace, since the cost of trace_event_buffer_reserve() in trace_event_raw_event_() handler is significantly

Re: [patch v2] mm, hugetlb_cgroup: round limit_in_bytes down to hugepage size

2016-04-18 Thread David Rientjes
On Fri, 15 Apr 2016, Michal Hocko wrote: > > > > +static void hugetlb_cgroup_init(struct hugetlb_cgroup *h_cgroup, > > > > + struct hugetlb_cgroup *parent_h_cgroup) > > > > +{ > > > > + int idx; > > > > + > > > > + for (idx = 0; idx < HUGE_MAX_HSTATE;

Re: [patch v2] mm, hugetlb_cgroup: round limit_in_bytes down to hugepage size

2016-04-18 Thread David Rientjes
On Fri, 15 Apr 2016, Michal Hocko wrote: > > > > +static void hugetlb_cgroup_init(struct hugetlb_cgroup *h_cgroup, > > > > + struct hugetlb_cgroup *parent_h_cgroup) > > > > +{ > > > > + int idx; > > > > + > > > > + for (idx = 0; idx < HUGE_MAX_HSTATE;

Re: [PATCH v2] bus: brcmstb_gisb: Rework dependencies

2016-04-18 Thread Florian Fainelli
On 16/04/16 13:46, Florian Fainelli wrote: > Do not have the machine Kconfig entry point need to select > BRCMSTB_GISB_ARB, instead, just let it be default ARCH_BRCMSTB which is > a better way to deal with this. While at it, also make it default > BMIPS_GENERIC so the legacy MIPS-based STB

Re: [PATCH v2] bus: brcmstb_gisb: Rework dependencies

2016-04-18 Thread Florian Fainelli
On 16/04/16 13:46, Florian Fainelli wrote: > Do not have the machine Kconfig entry point need to select > BRCMSTB_GISB_ARB, instead, just let it be default ARCH_BRCMSTB which is > a better way to deal with this. While at it, also make it default > BMIPS_GENERIC so the legacy MIPS-based STB

Re: [PATCH v2 0/4] Introduce bulk mode for crypto engine framework

2016-04-18 Thread Milan Broz
On 04/18/2016 03:36 PM, Mike Snitzer wrote: > On Mon, Apr 18 2016 at 1:31am -0400, > Baolin Wang wrote: > >> Hi Herbert, >> >> On 15 April 2016 at 21:48, Herbert Xu wrote: >>> On Tue, Mar 15, 2016 at 03:47:58PM +0800, Baolin Wang wrote:

Re: [PATCH v2 0/4] Introduce bulk mode for crypto engine framework

2016-04-18 Thread Milan Broz
On 04/18/2016 03:36 PM, Mike Snitzer wrote: > On Mon, Apr 18 2016 at 1:31am -0400, > Baolin Wang wrote: > >> Hi Herbert, >> >> On 15 April 2016 at 21:48, Herbert Xu wrote: >>> On Tue, Mar 15, 2016 at 03:47:58PM +0800, Baolin Wang wrote: Now some cipher hardware engines prefer to handle

Re: [PATCH 1/5] input: twl6040-vibra: fix DT node memory management

2016-04-18 Thread Dmitry Torokhov
On Mon, Apr 18, 2016 at 09:55:37PM +0200, H. Nikolaus Schaller wrote: > commit e7ec014a47e4 ("Input: twl6040-vibra - update for device tree support") > > made the separate vibra DT node to a subnode of the twl6040. > > It now calls of_find_node_by_name() to locate the "vibra" subnode. > This

Re: [PATCH 1/5] input: twl6040-vibra: fix DT node memory management

2016-04-18 Thread Dmitry Torokhov
On Mon, Apr 18, 2016 at 09:55:37PM +0200, H. Nikolaus Schaller wrote: > commit e7ec014a47e4 ("Input: twl6040-vibra - update for device tree support") > > made the separate vibra DT node to a subnode of the twl6040. > > It now calls of_find_node_by_name() to locate the "vibra" subnode. > This

Re: [PATCH 5/5] input: twl6040-vibra: remove mutex

2016-04-18 Thread Dmitry Torokhov
On Mon, Apr 18, 2016 at 09:55:41PM +0200, H. Nikolaus Schaller wrote: > The mutex does not seem to be needed. twl6040_vibra_suspend() and vibra_play_work() may run concurrently, no? > twl4030-vibra doesn't > use one either. It probably should. Thanks. > > Signed-off-by: H. Nikolaus Schaller

Re: [PATCH 5/5] input: twl6040-vibra: remove mutex

2016-04-18 Thread Dmitry Torokhov
On Mon, Apr 18, 2016 at 09:55:41PM +0200, H. Nikolaus Schaller wrote: > The mutex does not seem to be needed. twl6040_vibra_suspend() and vibra_play_work() may run concurrently, no? > twl4030-vibra doesn't > use one either. It probably should. Thanks. > > Signed-off-by: H. Nikolaus Schaller

[PATCH v4 02/14] acpi, numa: Replace ACPI_DEBUG_PRINT() with pr_debug()

2016-04-18 Thread David Daney
From: Hanjun Guo ACPI_DEBUG_PRINT is a bit fragile in acpi/numa.c, the first thing is that component ACPI_NUMA(0x8000) is not described in the Documentation/acpi/debug.txt, and even not defined in the struct acpi_dlayer acpi_debug_layers which we can not dynamically

[PATCH v4 02/14] acpi, numa: Replace ACPI_DEBUG_PRINT() with pr_debug()

2016-04-18 Thread David Daney
From: Hanjun Guo ACPI_DEBUG_PRINT is a bit fragile in acpi/numa.c, the first thing is that component ACPI_NUMA(0x8000) is not described in the Documentation/acpi/debug.txt, and even not defined in the struct acpi_dlayer acpi_debug_layers which we can not dynamically enable/disable it with

[PATCH v4 06/14] arm64, numa: rework numa_add_memblk()

2016-04-18 Thread David Daney
From: Hanjun Guo Rework numa_add_memblk() to update the parameter "u64 size" to "u64 end", this will make it consistent with x86 and simplifies the arm64 ACPI NUMA code to be added later. Signed-off-by: Hanjun Guo Signed-off-by: Robert Richter

[PATCH v4 06/14] arm64, numa: rework numa_add_memblk()

2016-04-18 Thread David Daney
From: Hanjun Guo Rework numa_add_memblk() to update the parameter "u64 size" to "u64 end", this will make it consistent with x86 and simplifies the arm64 ACPI NUMA code to be added later. Signed-off-by: Hanjun Guo Signed-off-by: Robert Richter Signed-off-by: David Daney ---

[PATCH v4 01/14] acpi, numa: Use pr_fmt() instead of printk

2016-04-18 Thread David Daney
From: Hanjun Guo Just do some cleanups to replace printk with pr_fmt(). Signed-off-by: Hanjun Guo Signed-off-by: Robert Richter Signed-off-by: David Daney --- drivers/acpi/numa.c | 17

[PATCH v4 01/14] acpi, numa: Use pr_fmt() instead of printk

2016-04-18 Thread David Daney
From: Hanjun Guo Just do some cleanups to replace printk with pr_fmt(). Signed-off-by: Hanjun Guo Signed-off-by: Robert Richter Signed-off-by: David Daney --- drivers/acpi/numa.c | 17 +++-- 1 file changed, 7 insertions(+), 10 deletions(-) diff --git a/drivers/acpi/numa.c

[PATCH v4 07/14] x86, acpi, numa: cleanup acpi_numa_processor_affinity_init()

2016-04-18 Thread David Daney
From: Hanjun Guo Cleanup acpi_numa_processor_affinity_init() in preparation for its move to drivers/acpi/numa.c. It will be reused by arm64, this has no functional change. Signed-off-by: Hanjun Guo Signed-off-by: Robert Richter

[PATCH v4 09/14] acpi, numa: remove unneeded acpi_numa=1

2016-04-18 Thread David Daney
From: Hanjun Guo acpi_numa is default to 0, it's set to -1 when disable acpi numa or when a bad SRAT is parsed, and it's only consumed in srat_disabled() (compare it with 0) to continue parse the SRAT or not, so we don't need to set acpi_numa to 1 when we get a valid SRAT

[PATCH v4 04/14] acpi, numa: Move acpi_numa_arch_fixup() to ia64 only

2016-04-18 Thread David Daney
From: Robert Richter Since acpi_numa_arch_fixup() is only used in arch ia64, move it there to make a generic interface easier. This avoids empty function stubs or some complex kconfig options for x86 and arm64. Signed-off-by: Robert Richter

[PATCH v4 07/14] x86, acpi, numa: cleanup acpi_numa_processor_affinity_init()

2016-04-18 Thread David Daney
From: Hanjun Guo Cleanup acpi_numa_processor_affinity_init() in preparation for its move to drivers/acpi/numa.c. It will be reused by arm64, this has no functional change. Signed-off-by: Hanjun Guo Signed-off-by: Robert Richter Signed-off-by: David Daney --- arch/x86/mm/srat.c | 21

[PATCH v4 09/14] acpi, numa: remove unneeded acpi_numa=1

2016-04-18 Thread David Daney
From: Hanjun Guo acpi_numa is default to 0, it's set to -1 when disable acpi numa or when a bad SRAT is parsed, and it's only consumed in srat_disabled() (compare it with 0) to continue parse the SRAT or not, so we don't need to set acpi_numa to 1 when we get a valid SRAT entry. Signed-off-by:

[PATCH v4 04/14] acpi, numa: Move acpi_numa_arch_fixup() to ia64 only

2016-04-18 Thread David Daney
From: Robert Richter Since acpi_numa_arch_fixup() is only used in arch ia64, move it there to make a generic interface easier. This avoids empty function stubs or some complex kconfig options for x86 and arm64. Signed-off-by: Robert Richter Signed-off-by: David Daney ---

[PATCH v4 08/14] acpi, numa: move bad_srat() and srat_disabled() to drivers/acpi/numa.c

2016-04-18 Thread David Daney
From: Hanjun Guo bad_srat() and srat_disabled() are shared by x86 and follow-on arm64 patches. Move them to drivers/acpi/numa.c in preparation for arm64 support. Signed-off-by: Hanjun Guo Signed-off-by: Robert Richter

[PATCH v4 08/14] acpi, numa: move bad_srat() and srat_disabled() to drivers/acpi/numa.c

2016-04-18 Thread David Daney
From: Hanjun Guo bad_srat() and srat_disabled() are shared by x86 and follow-on arm64 patches. Move them to drivers/acpi/numa.c in preparation for arm64 support. Signed-off-by: Hanjun Guo Signed-off-by: Robert Richter Signed-off-by: David Daney --- arch/x86/include/asm/acpi.h | 1 -

[PATCH v4 11/14] acpi, numa, srat: Improve SRAT error detection and add messages.

2016-04-18 Thread David Daney
From: David Daney Loosely based on code from Robert Richter and Hanjun Guo. Improve out of range node detection as well as allow for Larger SRAT entities. Add printing of nice messages. Signed-off-by: David Daney --- drivers/acpi/numa.c | 15

[PATCH v4 14/14] arm64, acpi, numa: Default enable ACPI_NUMA with NUMA

2016-04-18 Thread David Daney
From: Robert Richter Signed-off-by: Robert Richter Signed-off-by: David Daney --- drivers/acpi/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/acpi/Kconfig b/drivers/acpi/Kconfig index

[PATCH v4 11/14] acpi, numa, srat: Improve SRAT error detection and add messages.

2016-04-18 Thread David Daney
From: David Daney Loosely based on code from Robert Richter and Hanjun Guo. Improve out of range node detection as well as allow for Larger SRAT entities. Add printing of nice messages. Signed-off-by: David Daney --- drivers/acpi/numa.c | 15 +++ 1 file changed, 11

[PATCH v4 14/14] arm64, acpi, numa: Default enable ACPI_NUMA with NUMA

2016-04-18 Thread David Daney
From: Robert Richter Signed-off-by: Robert Richter Signed-off-by: David Daney --- drivers/acpi/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/acpi/Kconfig b/drivers/acpi/Kconfig index 5ce9f66..bacf245 100644 --- a/drivers/acpi/Kconfig +++

[PATCH v4 03/14] acpi, numa: remove duplicate NULL check

2016-04-18 Thread David Daney
From: Hanjun Guo The argument "header" for acpi_table_print_srat_entry() is always checked before the function is called, it's duplicate to check it again, remove it. Signed-off-by: Hanjun Guo Signed-off-by: Robert Richter

[PATCH v4 03/14] acpi, numa: remove duplicate NULL check

2016-04-18 Thread David Daney
From: Hanjun Guo The argument "header" for acpi_table_print_srat_entry() is always checked before the function is called, it's duplicate to check it again, remove it. Signed-off-by: Hanjun Guo Signed-off-by: Robert Richter Signed-off-by: David Daney --- drivers/acpi/numa.c | 3 --- 1 file

[PATCH v4 10/14] acpi, numa: Move acpi_numa_memory_affinity_init() to drivers/acpi/numa.c

2016-04-18 Thread David Daney
From: Hanjun Guo acpi_numa_memory_affinity_init() will be reused by arm64. Move it to drivers/acpi/numa.c to facilitate reuse. No code change. Signed-off-by: Hanjun Guo Signed-off-by: Robert Richter Signed-off-by: David

[PATCH v4 10/14] acpi, numa: Move acpi_numa_memory_affinity_init() to drivers/acpi/numa.c

2016-04-18 Thread David Daney
From: Hanjun Guo acpi_numa_memory_affinity_init() will be reused by arm64. Move it to drivers/acpi/numa.c to facilitate reuse. No code change. Signed-off-by: Hanjun Guo Signed-off-by: Robert Richter Signed-off-by: David Daney --- arch/x86/mm/srat.c | 56

[PATCH v4 12/14] arm64, acpi, numa: NUMA support based on SRAT and SLIT

2016-04-18 Thread David Daney
From: Hanjun Guo Introduce a new file to hold ACPI based NUMA information parsing from SRAT and SLIT. SRAT includes the CPU ACPI ID to Proximity Domain mappings and memory ranges to Proximity Domain mapping. SLIT has the information of inter node distances(relative

[PATCH v4 12/14] arm64, acpi, numa: NUMA support based on SRAT and SLIT

2016-04-18 Thread David Daney
From: Hanjun Guo Introduce a new file to hold ACPI based NUMA information parsing from SRAT and SLIT. SRAT includes the CPU ACPI ID to Proximity Domain mappings and memory ranges to Proximity Domain mapping. SLIT has the information of inter node distances(relative number for access latency).

[PATCH v4 05/14] acpi, numa: move acpi_numa_slit_init() to drivers/acpi/numa.c

2016-04-18 Thread David Daney
From: Hanjun Guo Identical implementations of acpi_numa_slit_init() are used by both x86 and follow-on arm64 support. Move it to drivers/acpi/numa.c, and mark it as __weak because ia64 has its own architecture specific implementation. No code change. Signed-off-by:

[PATCH v4 05/14] acpi, numa: move acpi_numa_slit_init() to drivers/acpi/numa.c

2016-04-18 Thread David Daney
From: Hanjun Guo Identical implementations of acpi_numa_slit_init() are used by both x86 and follow-on arm64 support. Move it to drivers/acpi/numa.c, and mark it as __weak because ia64 has its own architecture specific implementation. No code change. Signed-off-by: Hanjun Guo Signed-off-by:

[PATCH v4 13/14] acpi, numa: Enable ACPI based NUMA on ARM64

2016-04-18 Thread David Daney
From: Hanjun Guo Add function needed for cpu to node mapping, and enable ACPI based NUMA for ARM64 in Kconfig Signed-off-by: Hanjun Guo Signed-off-by: Robert Richter Signed-off-by: David Daney ---

[PATCH v4 00/14] ACPI NUMA support for ARM64

2016-04-18 Thread David Daney
From: David Daney Based on v16 of device-tree NUMA patch set for arm64 [1],this patch set introduce the ACPI based configuration to provide NUMA information. ACPI 5.1 already introduced NUMA support for ARM64, which can get the NUMA domain information from SRAT and SLIT

[PATCH v4 13/14] acpi, numa: Enable ACPI based NUMA on ARM64

2016-04-18 Thread David Daney
From: Hanjun Guo Add function needed for cpu to node mapping, and enable ACPI based NUMA for ARM64 in Kconfig Signed-off-by: Hanjun Guo Signed-off-by: Robert Richter Signed-off-by: David Daney --- drivers/acpi/Kconfig | 2 +- drivers/acpi/numa.c | 35 +++

[PATCH v4 00/14] ACPI NUMA support for ARM64

2016-04-18 Thread David Daney
From: David Daney Based on v16 of device-tree NUMA patch set for arm64 [1],this patch set introduce the ACPI based configuration to provide NUMA information. ACPI 5.1 already introduced NUMA support for ARM64, which can get the NUMA domain information from SRAT and SLIT table, so parse those

Re: [PATCH 2/5] input: twl6040-vibra: add handler to unregister input if module is removed

2016-04-18 Thread Dmitry Torokhov
On Mon, Apr 18, 2016 at 09:55:38PM +0200, H. Nikolaus Schaller wrote: > commit 1f9e1470ab34 ("Input: twl6040-vibra - use devm functions") > > converted everything to devm but we still need to call > input_unregister_device(info->input_dev) No, this is not needed, because devm-managed input

Re: [PATCH 2/5] input: twl6040-vibra: add handler to unregister input if module is removed

2016-04-18 Thread Dmitry Torokhov
On Mon, Apr 18, 2016 at 09:55:38PM +0200, H. Nikolaus Schaller wrote: > commit 1f9e1470ab34 ("Input: twl6040-vibra - use devm functions") > > converted everything to devm but we still need to call > input_unregister_device(info->input_dev) No, this is not needed, because devm-managed input

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