Re: [PATCH 2/2] Documentation: DMA-ISA-LPC.txt

2017-02-12 Thread Jonathan Corbet
On Thu, 9 Feb 2017 18:36:36 -0500 Nathan Howard wrote: > Fixed spelling issue. > > Signed-off-by: Nathan Howard > --- > Documentation/DMA-ISA-LPC.txt | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/Documentation/DMA-ISA-LPC.txt b/Documentation/DMA-ISA-LPC.txt > index

Re: [PATCH] Documentation: input: fix path to input code definitions

2017-02-12 Thread Jonathan Corbet
On Thu, 9 Feb 2017 07:53:29 +0100 Martin Kepplinger wrote: > Signed-off-by: Martin Kepplinger > --- > Documentation/input/input.txt | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/Documentation/input/input.txt b/Documentation/input/input.txt > index 0acfddb..7ebce

Re: [PATCH 3/20] staging: rtl8192u: Prefer using the BIT macro

2017-02-12 Thread Greg KH
On Mon, Feb 13, 2017 at 03:26:37AM +0530, simran singhal wrote: > Fix the checkpatch.pl issue: > CHECK: Prefer using the BIT macro replacing > bit shifting on 1 with the BIT(x) macro. > > Signed-off-by: simran singhal > --- > drivers/staging/rtl8192u/ieee80211/ieee80211.h | 140 > --

drivers/base/power/opp/core.c:1389:19: error: redefinition of 'dev_pm_opp_set_regulators'

2017-02-12 Thread kbuild test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: 7089db84e356562f8ba737c29e472cc42d530dbc commit: dfbe4678d709e25e0f36e6b6333e2a7a67aefb7e PM / OPP: Add infrastructure to manage multiple regulators date: 10 weeks ago config: x86_64-randconfig-x008-201707

Re: [PATCH] smiapp: add CCP2 support

2017-02-12 Thread Sakari
Hi Pavel, On Sun, Feb 12, 2017 at 12:22:58AM +0100, Pavel Machek wrote: > Hi! > > > Besides this patch, what else is needed? The CSI-2 / CCP2 support is > > missing in V4L2 OF at least. It'd be better to have this all in the same > > set. > > Quite a lot of is needed. > > > I pushed the two DT

Re: [PATCH v2 2/2] x86/purgatory: Fix sparse warning, symbol not declared

2017-02-12 Thread kbuild test robot
Hi Tobin, [auto build test ERROR on tip/x86/core] [also build test ERROR on v4.10-rc8 next-20170210] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com/0day-ci/linux/commits/Tobin-C-Harding/x86-purgatory-Fix-sparse-warni

[BUG] 4.9.9 shrink_slab prune_icache_sb crash

2017-02-12 Thread Sami Farin
4.4 kernel had working [non-crashing] shrink_slab and friends... x86_64 arch , Core i5 2500K, 16 GiB. kernel: BUG: unable to handle kernel paging request at 00020157 kernel: IP: [] __lock_acquire.isra.15+0x39a/0x930 kernel: PGD 0 kernel: kernel: Oops: [#1] PREEMPT SMP kernel: Module

Re: [PATCH] spi-nor: use ERR_CAST in return

2017-02-12 Thread Boris Brezillon
+Mika On Sun, 12 Feb 2017 22:45:43 +0100 Boris Brezillon wrote: > On Sun, 12 Feb 2017 17:43:43 +0100 > Nicholas Mc Guire wrote: > > > This fixes a sparse warning about > > Your commit message seems to be incomplete. And here too, please change the prefix to "mtd: spi-nor: intel: ". Thank

Re: [PATCH] spi-nor: use true/false for bool

2017-02-12 Thread Boris Brezillon
+Mika On Sun, 12 Feb 2017 22:50:44 +0100 Boris Brezillon wrote: > On Sun, 12 Feb 2017 17:43:31 +0100 > Nicholas Mc Guire wrote: > > > writeable in struct intel_spi is a boolean and assignment should be to > > true/false not 1/0 as recommended by boolinit.cocci. > > > > Signed-off-by: Nichola

Re: [PATCH RFC] spi-nor: provide a range for poll_timout

2017-02-12 Thread Boris Brezillon
+Mika On Sun, 12 Feb 2017 17:42:57 +0100 Nicholas Mc Guire wrote: > The overall poll time here is INTEL_SPI_TIMEOUT * 1000 which is > 5000 * 1000 - so 5seconds and it is coded as a tight loop here delay_us > to readl_poll_timeout() is set to 0. As this is never called in an atomic > context sle

[PATCH 3/20] staging: rtl8192u: Prefer using the BIT macro

2017-02-12 Thread simran singhal
Fix the checkpatch.pl issue: CHECK: Prefer using the BIT macro replacing bit shifting on 1 with the BIT(x) macro. Signed-off-by: simran singhal --- drivers/staging/rtl8192u/ieee80211/ieee80211.h | 140 - 1 file changed, 70 insertions(+), 70 deletions(-) diff --git a/driv

Re: [PATCH] spi-nor: use true/false for bool

2017-02-12 Thread Boris Brezillon
On Sun, 12 Feb 2017 17:43:31 +0100 Nicholas Mc Guire wrote: > writeable in struct intel_spi is a boolean and assignment should be to > true/false not 1/0 as recommended by boolinit.cocci. > > Signed-off-by: Nicholas Mc Guire Acked-by: Boris Brezillon > --- > > make coccicheck complained wi

[PATCH] Staging: media: bcm2048: Fixed coding style issue.

2017-02-12 Thread Ran Algawi
Fixed a coding style issue, where an octal value was needed insted of decimal. Signed-off-by: Ran Algawi --- drivers/staging/media/bcm2048/radio-bcm2048.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/media/bcm2048/radio-bcm2048.c b/drivers/staging/media/bc

Re: [PATCH] spi-nor: use ERR_CAST in return

2017-02-12 Thread Boris Brezillon
On Sun, 12 Feb 2017 17:43:43 +0100 Nicholas Mc Guire wrote: > This fixes a sparse warning about Your commit message seems to be incomplete. Once fixed, Acked-by: Boris Brezillon > > Signed-off-by: Nicholas Mc Guire > --- > > sparse complained about: > drivers/mtd/spi-nor/intel-spi.c:731:

drivers/base/power/opp/of.c:181:6: error: redefinition of 'dev_pm_opp_of_remove_table'

2017-02-12 Thread kbuild test robot
Hi Viresh, FYI, the error/warning still remains. tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: 7089db84e356562f8ba737c29e472cc42d530dbc commit: f47b72a15a9679dd4dc1af681d4d2f1ca2815552 PM / OPP: Move CONFIG_OF dependent code in a separate file date:

[PATCH] staging:xgifb:vb_setmode.h: Add missing identifier names

2017-02-12 Thread Jacob Zachariah
Fix multiple checkpatch.pl warnings WARNING: function definition argument '..' should also have an identifier name Signed-off-by: Jacob Zachariah --- drivers/staging/xgifb/vb_setmode.h | 18 +- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/drivers/staging/xgifb/vb

Re: [PATCH] lz4: fix performance regressions

2017-02-12 Thread Willy Tarreau
On Sun, Feb 12, 2017 at 04:20:00PM +0100, Sven Schmidt wrote: > On Sun, Feb 12, 2017 at 02:05:08PM +0100, Willy Tarreau wrote: > > Hi Sven, > > > > On Sun, Feb 12, 2017 at 12:16:18PM +0100, Sven Schmidt wrote: > > > Fix performance regressions compared to current kernel LZ4 > > > > Your patch con

Linux 4.10-rc8

2017-02-12 Thread Linus Torvalds
Hey, it's another week, and I could have released the final 4.10. It's not been all that busy, although we did have a number of small last-minute regression fixes (some just reverting stuff that caused problems and needed more thought, others fixing things). But nothing out of the ordinary, and I

Re: [PATCH] Staging: media: bcm2048: Fixed an error

2017-02-12 Thread Greg Kroah-Hartman
On Sun, Feb 12, 2017 at 11:12:42PM +0200, Ran Algawi wrote: > Hello Greg, > First, I appreciate you taking the time to educate me. I used the checkpatch > script on the file I fixed and he reported the line as an error. Do you > consider all checkpatch warnings/error/checks as coding style fixes?

Re: [PATCH] Staging: media: bcm2048: Fixed an error

2017-02-12 Thread Ran Algawi
Hello Greg, I apperaciate you taking the time to educate me. Do you consider all checkpatch errors/warnnings/checks as code style fixes? With regards, Ran A.

[PATCH v1] samples/bpf: Add a .gitignore for binaries

2017-02-12 Thread Mickaël Salaün
Signed-off-by: Mickaël Salaün Cc: Alexei Starovoitov Cc: Arnaldo Carvalho de Melo Cc: Daniel Borkmann Cc: Wang Nan --- samples/bpf/.gitignore | 32 1 file changed, 32 insertions(+) create mode 100644 samples/bpf/.gitignore diff --git a/samples/bpf/.gitignore

[PATCH] perf: Fix CONFIG_KPROBE_EVENTS and CONFIG_UPROBE_EVENTS typos

2017-02-12 Thread Anton Blanchard
From: Anton Blanchard Fix some incorrect Kconfig options, they should be CONFIG_KPROBE_EVENT and CONFIG_UPROBE_EVENT. Signed-off-by: Anton Blanchard --- arch/sparc/configs/sparc64_defconfig | 2 +- tools/perf/util/probe-file.c | 8 2 files changed, 5 insertions(+), 5 deletions

[PATCH 2/6] x86/cpu: drop wp_works_ok member of struct cpuinfo_x86

2017-02-12 Thread Mathias Krause
Remove the wp_works_ok member of struct cpuinfo_x86. It's an optimization back from Linux v0.99 times where we had no fixup support yet and did the CR0.WP test via special code in the page fault handler. The < 0 test was an optimization to not do the special casing for each NULL ptr access violatio

Re: Linux 4.9.6 ( Restore IO-APIC irq_chip retrigger callback , breaks my box )

2017-02-12 Thread Borislav Petkov
On Sun, Feb 12, 2017 at 09:21:53PM +0100, Gabriel C wrote: > There is what I get : > > http://ftp.frugalware.org/pub/other/people/crazy/kernel/t/crash2.mp4 Ok, I'm watching it frame-by-frame. I can see the microcode getting updated to revision 0x19 as in your working dmesg. The machine hangs her

[PATCH 4/6] sparc: remove unused wp_works_ok macro

2017-02-12 Thread Mathias Krause
It's unused for ages, used to be required for ksyms.c back in the v1.1 times. Cc: David S. Miller Signed-off-by: Mathias Krause --- arch/sparc/include/asm/processor_32.h |6 -- arch/sparc/include/asm/processor_64.h |4 2 files changed, 10 deletions(-) diff --git a/arch/sparc/i

[PATCH 3/6] x86/cpu: proc - remove "wp" status line in cpuinfo

2017-02-12 Thread Mathias Krause
As of commit a5c2a893dbd4 ("x86, 386 removal: Remove CONFIG_X86_WP_WORKS_OK") the kernel won't boot if CR0.WP isn't working correctly. This makes a process reading this file always see "wp : yes" here -- otherwise there would be no process to begin with ;) As this status line in /proc/cpuinfo serv

[PATCH 6/6] m68k: paging_init - remove dead code

2017-02-12 Thread Mathias Krause
The macro TEST_VERIFY_AREA can never be defined as there's no wp_works_ok variable. So just remove the dead code. Cc: Geert Uytterhoeven Signed-off-by: Mathias Krause --- arch/m68k/mm/sun3mmu.c |3 --- 1 file changed, 3 deletions(-) diff --git a/arch/m68k/mm/sun3mmu.c b/arch/m68k/mm/sun3mm

[PATCH 5/6] cris: remove unused wp_works_ok macro

2017-02-12 Thread Mathias Krause
It had no use since it's introduction in v2.4.1.2. Get rid of it. Cc: Jesper Nilsson Cc: Mikael Starvik Signed-off-by: Mathias Krause --- arch/cris/include/arch-v10/arch/processor.h |3 --- 1 file changed, 3 deletions(-) diff --git a/arch/cris/include/arch-v10/arch/processor.h b/arch/cri

[PATCH 0/6] struct cpuinfo_x86 related cleanups

2017-02-12 Thread Mathias Krause
This small series slims down the x86 specific struct cpuinfo_x86 (patches 1 and 2). It's kind of a continuation of Boris' work from 2013. Beside the x86 specific part it also cleans up other arches that had to have a wp_works_ok variable / define back in the old v1.1 times. But those times are lon

[PATCH 1/6] x86: drop unneded members of struct cpuinfo_x86

2017-02-12 Thread Mathias Krause
Those member serve no purpose -- not even fill padding for alignment or such. So just get rid of them. Cc: Borislav Petkov Cc: H. Peter Anvin Signed-off-by: Mathias Krause --- arch/x86/include/asm/processor.h |7 +-- 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/arch/x86

drivers/base/power/opp/core.c:1901:6: error: redefinition of 'dev_pm_opp_remove_table'

2017-02-12 Thread kbuild test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: 1ce42845f987e92eabfc6e026d44d826c25c74a5 commit: 411466c5081d2f649b3583cae0f6c9ad5edec636 PM / OPP: add non-OF versions of dev_pm_opp_{cpumask_, }remove_table date: 9 months ago config: x86_64-randconfig-x

[PATCH v2 2/2] x86/purgatory: Fix sparse warning, symbol not declared

2017-02-12 Thread Tobin C. Harding
Sparse emits warning, 'symbol not declared' for a function that has neither file scope nor a forward declaration. The functions only call site is an ASM file. Add a header file with the function declaration. Include the header file in the C source file defining the function in order to fix the spa

[PATCH v2 0/2] x86/purgatory: Fix sparse warnings

2017-02-12 Thread Tobin C. Harding
Sparse emits multiple warnings of type 'symbol was not declared'. Separate fixes into two patches. In first patch add static keyword to functions and variables which have file scope only. In the second patch add a header file with the function declaration. Include the header file in the C source

[PATCH v2 1/2] x86/purgatory: Fix sparse warning, symbol not declared

2017-02-12 Thread Tobin C. Harding
Sparse emits several 'symbol not declared' warnings for various functions and variables. Add static keyword to functions and variables which have file scope only. Signed-off-by: Tobin C. Harding --- arch/x86/purgatory/purgatory.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-)

Re: [PATCH v3 04/14] GPIO: Add gpio-ingenic driver

2017-02-12 Thread Linus Walleij
On Thu, Feb 9, 2017 at 6:14 PM, Paul Cercueil wrote: >> If you're not just replacing these with GPIO_GENERIC, please also >> include a .get_direction() callback. > > My .direction_input() and .direction_output() callbacks just call into > the pinctrl driver, using pinctrl_gpio_direction_[in,out]p

drivers/base/power/opp/core.c:595:5: error: redefinition of 'dev_pm_opp_set_rate'

2017-02-12 Thread kbuild test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: 1ce42845f987e92eabfc6e026d44d826c25c74a5 commit: 6a0712f6f199e737aa5913d28ec4bd3a25de9660 PM / OPP: Add dev_pm_opp_set_rate() date: 1 year ago config: x86_64-randconfig-x008-201707 (attached as .config) co

[PATCH] net: nuvoton: w90p910: use new api ethtool_{get|set}_link_ksettings

2017-02-12 Thread Philippe Reynes
The ethtool api {get|set}_settings is deprecated. We move this driver to new api {get|set}_link_ksettings. As I don't have the hardware, I'd be very pleased if someone may test this patch. Signed-off-by: Philippe Reynes --- drivers/net/ethernet/nuvoton/w90p910_ether.c | 14 -- 1 f

Re: [PATCH 3/3] DT: add Faraday Tec. as vendor

2017-02-12 Thread Linus Walleij
On Wed, Feb 8, 2017 at 9:00 PM, Hans Ulli Kroll wrote: > add Faraday Technology Corporation as vendor faraday for DT > > Signed-off-by: Hans Ulli Kroll I realized a lot of Gemini IP blocks are actually from Faraday, so now I'm editing the drivers a bit... Rob could you pick this patch directly

Re: [tpmdd-devel] [RFC] tpm2-space: add handling for global session exhaustion

2017-02-12 Thread Ken Goldman
On 2/10/2017 11:46 AM, James Bottomley wrote: On Fri, 2017-02-10 at 04:03 -0600, Dr. Greg Wettstein wrote: On Feb 9, 11:24am, James Bottomley wrote: quote: 810 milliseconds verify signature: 635 milliseconds ... Part of the way of reducing the latency is not to use the TPM for things that d

Re: Linux 4.9.6 ( Restore IO-APIC irq_chip retrigger callback , breaks my box )

2017-02-12 Thread Gabriel C
On 11.02.2017 22:32, Borislav Petkov wrote: On Sat, Feb 11, 2017 at 09:58:26PM +0100, Gabriel C wrote: Yes , it will hang before tsc message .. Also sometimes I have same trace sometimes it just hangs forever. It doesn't sound like dis_ucode_ldr changes anything. Or maybe it does, maybe the

drivers/base/power/opp/core.c:302:15: error: redefinition of 'dev_pm_opp_get_max_transition_latency'

2017-02-12 Thread kbuild test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: 1ce42845f987e92eabfc6e026d44d826c25c74a5 commit: 2174344765f472895c076d703c9cdc58215e1393 PM / OPP: Introduce dev_pm_opp_get_max_transition_latency() date: 1 year ago config: x86_64-randconfig-x008-201707

Re: [PATCH] clocksource: add __ro_after_init to cyclecounter

2017-02-12 Thread Daniel Lezcano
On Sun, Feb 12, 2017 at 07:56:41PM +0100, Thomas Gleixner wrote: [ ... ] > I've applied it already and fixed up the subject/changelog. You should have > mail from tip-bot ... For the record, I did not receive the mails from the tip-bot when the patches from the PR were applied. -- Daniel

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

2017-02-12 Thread Pavel Machek
Hi! > >The kernel is 4.8.17 EOL, installed almost a month ago. > >Platform ThinkPad x60, Intel(R) Core(TM) Duo CPU T2400 @ 1.83GHz > > > >In dmesg I see that it used to be when 2 CPUs were OK: > >[690409.476107] PM: noirq suspend of devices complete after 79.914 msecs > >[690409.476547] ACP

drivers/staging/rtl8192u

2017-02-12 Thread Colin Ian King
Hi, Static analysis via CoverityScan picked up an issue in cmpk_handle_query_config_rx where the following operation on an u8 is clearly not correct: rx_query_cfg.cfg_action = (pmsg[4] & 0x8000) >> 31; The result of this operation is always zero. I suspect this should be:

Re: [RFC][PATCH 4/7] x86, mpx: context-switch new MPX address size MSR

2017-02-12 Thread Thomas Gleixner
On Wed, 1 Feb 2017, Dave Hansen wrote: > +/* > + * The MPX tables change sizes based on the size of the virtual > + * (aka. linear) address space. There is an MSR to tell the CPU > + * whether we want the legacy-style ones or the larger ones when > + * we are running with an eXtended virtual addre

drivers/base/power/opp/core.c:241:15: error: redefinition of 'dev_pm_opp_get_max_volt_latency'

2017-02-12 Thread kbuild test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: 1ce42845f987e92eabfc6e026d44d826c25c74a5 commit: 655c9df961751ce21466f6e97e8033932c27a675 PM / OPP: Introduce dev_pm_opp_get_max_volt_latency() date: 1 year ago config: x86_64-randconfig-x008-201707 (attac

[tip:timers/core] clocksource/drivers/arm_arch_timer:: Mark cyclecounter __ro_after_init

2017-02-12 Thread tip-bot for Bhumika Goyal
Commit-ID: 3d837bc01c2153565333b2ce71d613b6e7cc761c Gitweb: http://git.kernel.org/tip/3d837bc01c2153565333b2ce71d613b6e7cc761c Author: Bhumika Goyal AuthorDate: Sun, 12 Feb 2017 00:50:18 +0530 Committer: Thomas Gleixner CommitDate: Sat, 11 Feb 2017 21:39:04 +0100 clocksource/drivers/ar

Re: [RFC][PATCH 3/7] x86, mpx: extend MPX prctl() to pass in size of bounds directory

2017-02-12 Thread Thomas Gleixner
On Wed, 1 Feb 2017, Dave Hansen wrote: > FIXME: we also need to ensure that we check the current state of the > larger address space opt-in. If we've opted in to larger address spaces > we can not allow a small bounds directory to be used. Also, if we've > not opted in, we can not allow the large

Re: [RFC][PATCH 2/7] x86, mpx: update MPX to grok larger bounds tables

2017-02-12 Thread Thomas Gleixner
On Wed, 1 Feb 2017, Dave Hansen wrote: > /* > - * The upper 28 bits [47:20] of the virtual address in 64-bit > - * are used to index into bounds directory (BD). > + * The uppermost bits [56:20] of the virtual address in 64-bit > + * are used to index into bounds directory (BD). On processors > +

Re: [GIT PULL] PCI fixes for v4.10

2017-02-12 Thread Lukas Wunner
On Fri, Feb 10, 2017 at 06:39:16PM -0800, Yinghai Lu wrote: > On Thu, Feb 9, 2017 at 12:11 PM, Bjorn Helgaas wrote: > > On Thu, Feb 09, 2017 at 09:09:50AM -0600, Bjorn Helgaas wrote: > > > On Thu, Feb 09, 2017 at 05:06:48AM +0100, Lukas Wunner wrote: > > > > https://patchwork.kernel.org/patch/9557

Re: [PATCH] clocksource: add __ro_after_init to cyclecounter

2017-02-12 Thread Bhumika Goyal
On Mon, Feb 13, 2017 at 12:26 AM, Thomas Gleixner wrote: > On Sun, 12 Feb 2017, Bhumika Goyal wrote: > >> On Sun, Feb 12, 2017 at 2:01 AM, Ard Biesheuvel >> wrote: >> > On 11 February 2017 at 19:20, Bhumika Goyal wrote: >> >> The object cyclecounter of type cyclecounter is not getting modified >

Re: [PATCH] clocksource: add __ro_after_init to cyclecounter

2017-02-12 Thread Thomas Gleixner
On Sun, 12 Feb 2017, Bhumika Goyal wrote: > On Sun, Feb 12, 2017 at 2:01 AM, Ard Biesheuvel > wrote: > > On 11 February 2017 at 19:20, Bhumika Goyal wrote: > >> The object cyclecounter of type cyclecounter is not getting modified > >> after getting initialized by arch_counter_register. Apart fro

[tip:irq/core] genirq/devres: Use dev_name(dev) as default for devname

2017-02-12 Thread tip-bot for Heiner Kallweit
Commit-ID: 899b5fbf9d3fcb721690b4d58cf58cc018517003 Gitweb: http://git.kernel.org/tip/899b5fbf9d3fcb721690b4d58cf58cc018517003 Author: Heiner Kallweit AuthorDate: Sun, 12 Feb 2017 16:31:44 +0100 Committer: Thomas Gleixner CommitDate: Sun, 12 Feb 2017 19:49:25 +0100 genirq/devres: Use d

[PATCH] usb: misc: usbtest: remove redundant check on retval < 0

2017-02-12 Thread Colin King
From: Colin Ian King The check for retval being less than zero is always true since retval equal to -EPIPE at that point. Replace the existing conditional with just return retval. Detected with CoverityScan, CID#114349 ("Logically dead code") Signed-off-by: Colin Ian King --- drivers/usb/mis

Re: [ibm-acpi-devel] [PATCH 1/1] thinkpad_acpi: Add support for status (external cover) LED

2017-02-12 Thread Adam Goode
On Sun, Feb 12, 2017 at 1:22 PM, Sebastian Reichel wrote: > Hi, > > On Tue, Feb 07, 2017 at 11:01:32PM -0200, Henrique de Moraes Holschuh wrote: >> Hello Adam, >> >> I apologise for the delay on answering you. >> >> On Tue, 31 Jan 2017, Adam Goode wrote: >> > On Thu, Jan 19, 2017 at 12:21 PM, Adam

Re: [ibm-acpi-devel] [PATCH 1/1] thinkpad_acpi: Add support for status (external cover) LED

2017-02-12 Thread Sebastian Reichel
Hi, On Tue, Feb 07, 2017 at 11:01:32PM -0200, Henrique de Moraes Holschuh wrote: > Hello Adam, > > I apologise for the delay on answering you. > > On Tue, 31 Jan 2017, Adam Goode wrote: > > On Thu, Jan 19, 2017 at 12:21 PM, Adam Goode wrote: > > > This allows the control of the red status LED,

Re: [PATCH] clocksource: add __ro_after_init to cyclecounter

2017-02-12 Thread Bhumika Goyal
On Sun, Feb 12, 2017 at 2:05 AM, Thomas Gleixner wrote: > On Sun, 12 Feb 2017, Bhumika Goyal wrote: > > Please be more careful with your subject line. The prefix for this is > definitely not 'clocksource'. git log would have told you the proper one: > > clocksource/drivers/arm_arch_timer > > 'cloc

Re: [PATCH] clocksource: add __ro_after_init to cyclecounter

2017-02-12 Thread Bhumika Goyal
On Sun, Feb 12, 2017 at 2:01 AM, Ard Biesheuvel wrote: > On 11 February 2017 at 19:20, Bhumika Goyal wrote: >> The object cyclecounter of type cyclecounter is not getting modified >> after getting initialized by arch_counter_register. Apart from >> initialization in arch_counter_register it is al

Клиентские базы +79139230330 Skype: prodawez390 Email: prodawez...@gmail.com Узнайте подробнее!

2017-02-12 Thread linux-kernel
Клиентские базы +79139230330 Skype: prodawez390 Email: prodawez...@gmail.com Узнайте подробнее!

Re: [PATCH v5 3/8] PCI: Don't block runtime PM for Thunderbolt host hotplug ports

2017-02-12 Thread Lukas Wunner
On Fri, Feb 10, 2017 at 12:39:12PM -0600, Bjorn Helgaas wrote: > On Sun, Jan 15, 2017 at 09:03:45PM +0100, Lukas Wunner wrote: > > Hotplug ports generally block their parents from suspending to D3hot as > > otherwise their interrupts couldn't be delivered. > > This sounds related to PCIe r3.1, sec

Re: net/llc: BUG in llc_sap_state_process/skb_set_owner_r

2017-02-12 Thread Eric Dumazet
On Sun, Feb 12, 2017 at 8:44 AM, Andrey Konovalov wrote: > Hi, > > I've got the following error report while fuzzing the kernel with syzkaller. > > On commit 926af6273fc683cd98cd0ce7bf0d04a02eed6742. > > A reproducer and .config are attached Thanks for the report. llc sets skb->sk without corr

[PATCH 2/20] staging: rtl8192u: Fixed 'tabstop' coding style warnings

2017-02-12 Thread simran singhal
Fixed checkpatch warning "Statements should start on a tabstop" in rtl8192u module. Signed-off-by: simran singhal --- drivers/staging/rtl8192u/ieee80211/ieee80211_wx.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/rtl8192u/ieee80211/ieee80211_wx.c b/drivers

[PATCH 1/20] staging: rtl8192u: Replace symbolic permissions with octal permissions

2017-02-12 Thread simran singhal
WARNING: Symbolic permissions 'S_IRUGO | S_IWUSR' are not preferred. Consider using octal permissions '0644'. This warning is detected by checkpatch.pl Signed-off-by: simran singhal --- drivers/staging/rtl8192u/ieee80211/ieee80211_module.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) d

Re: net/ipv6: use-after-free in sock_wfree

2017-02-12 Thread Andrey Konovalov
On Mon, Jan 9, 2017 at 6:21 PM, Eric Dumazet wrote: > On Mon, Jan 9, 2017 at 9:11 AM, Andrey Konovalov > wrote: >> On Mon, Jan 9, 2017 at 6:08 PM, Andrey Konovalov >> wrote: >>> Hi! >>> >>> I've got the following error report while running the syzkaller fuzzer. >>> >>> On commit a121103c922847

Re: [PATCH v5 1/8] PCI: Recognize Thunderbolt devices

2017-02-12 Thread Lukas Wunner
On Fri, Feb 10, 2017 at 11:42:50AM -0600, Bjorn Helgaas wrote: > On Sun, Jan 29, 2017 at 01:26:16AM +0100, Lukas Wunner wrote: > > On Sat, Jan 28, 2017 at 03:52:08PM -0600, Bjorn Helgaas wrote: > > > On Sun, Jan 15, 2017 at 09:03:45PM +0100, Lukas Wunner wrote: > > > > > +static void set_pcie_vend

Re: net/llc: bug in llc_pdu_init_as_xid_cmd/skb_over_panic

2017-02-12 Thread Andrey Konovalov
On Fri, Feb 10, 2017 at 4:12 PM, Andrey Konovalov wrote: > Hi, > > I've got the following error report while fuzzing the kernel with syzkaller. > > On commit 926af6273fc683cd98cd0ce7bf0d04a02eed6742. > > A reproducer and .config are attached > > kernel BUG at net/core/skbuff.c:105! > invalid opcod

net/llc: BUG in llc_sap_state_process/skb_set_owner_r

2017-02-12 Thread Andrey Konovalov
Hi, I've got the following error report while fuzzing the kernel with syzkaller. On commit 926af6273fc683cd98cd0ce7bf0d04a02eed6742. A reproducer and .config are attached kernel BUG at ./include/linux/skbuff.h:2389! invalid opcode: [#1] SMP KASAN Dumping ftrace buffer: (ftrace buffer em

[PATCH] spi-nor: use ERR_CAST in return

2017-02-12 Thread Nicholas Mc Guire
This fixes a sparse warning about Signed-off-by: Nicholas Mc Guire --- sparse complained about: drivers/mtd/spi-nor/intel-spi.c:731:28: warning: incorrect type in return expression (different address spaces) drivers/mtd/spi-nor/intel-spi.c:731:28:expected struct intel_spi * drivers/mtd/spi

[PATCH RFC] spi-nor: provide a range for poll_timout

2017-02-12 Thread Nicholas Mc Guire
The overall poll time here is INTEL_SPI_TIMEOUT * 1000 which is 5000 * 1000 - so 5seconds and it is coded as a tight loop here delay_us to readl_poll_timeout() is set to 0. As this is never called in an atomic context sleeping should be no issue and there is no reasons for the tight-loop here. Si

[PATCH] spi-nor: use true/false for bool

2017-02-12 Thread Nicholas Mc Guire
writeable in struct intel_spi is a boolean and assignment should be to true/false not 1/0 as recommended by boolinit.cocci. Signed-off-by: Nicholas Mc Guire --- make coccicheck complained with: ./drivers/mtd/spi-nor/intel-spi.c:707:3-18: WARNING: Assignment of bool to 0/1 Patch was compile te

[PATCH] net: neterion: vxge: use new api ethtool_{get|set}_link_ksettings

2017-02-12 Thread Philippe Reynes
The ethtool api {get|set}_settings is deprecated. We move this driver to new api {get|set}_link_ksettings. As I don't have the hardware, I'd be very pleased if someone may test this patch. Signed-off-by: Philippe Reynes --- drivers/net/ethernet/neterion/vxge/vxge-ethtool.c | 47 --

Re: [PATCH v5 7/8] thunderbolt: Power down controller when idle

2017-02-12 Thread Lukas Wunner
On Sat, Jan 28, 2017 at 05:32:37PM -0600, Bjorn Helgaas wrote: > On Sun, Jan 15, 2017 at 09:03:45PM +0100, Lukas Wunner wrote: > > A Thunderbolt controller appears to the OS as a set of virtual devices: > > One upstream bridge, multiple downstream bridges and one NHI (Native > > Host Interface). T

Re: [PATCH v3] softirq: Prevent looping on disabled tasklets

2017-02-12 Thread Chris Wilson
On Sun, Feb 12, 2017 at 03:46:09PM +, Chris Wilson wrote: > +void tasklet_enable(struct tasklet_struct *t) > +{ > + if (!atomic_dec_and_test(&t->count)) > + return; > + > + if (test_bit(TASKLET_STATE_SCHED, &t->state)) > + raise_softirq(HI_SOFTIRQ | TASKLET_SOFTI

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

2017-02-12 Thread Arushi Singhal
Function definitions arguments should also have an identifier name as reported by checkpatch.pl. Signed-off-by: Arushi Singhal --- drivers/staging/vt6656/channel.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/vt6656/channel.h b/drivers/staging/vt6656/channe

Re: [REGRESSION] Two issues that prevent process accounting (taskstats) from working correctly

2017-02-12 Thread Dmitry Romanov
On Mon, Dec 19, 2016 at 01:06:00PM +0100, Martin Steigerwald wrote: > > 1) Sometimes process accounting does not work at all. > > The acct() system call (to activate process accounting) return value 0, > which means that process accounting is activated successfully. > However, no process accountin

S2ram + btrfs with recent kernel => soft lockup warning and hard hang

2017-02-12 Thread Bastien ROUCARIES
Hi, Since 3.8 s2ram on btrfs system lead to hang. Could not save a trace will try to get a few picture but trace are similar to : https://github.com/docker/docker/issues/15654 and https://patchwork.kernel.org/patch/8352811/ It is 100% reproducible. mount table /dev/mapper/portable2015--bastien-

[PATCH v3] softirq: Prevent looping on disabled tasklets

2017-02-12 Thread Chris Wilson
Disabling a tasklet causes it not to run during tasklet_action, but is put back onto the runnable tasklet list, and a new softirq raised. As the softirq is raised from within __do_softirq() this causing __do_softirq() to loop constantly until its timeslice expires and is transferred to the ksoftirq

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

2017-02-12 Thread Woody Suwalski
Woody Suwalski wrote: Pavel Machek wrote: On Sat 2017-01-14 12:30:54, Pavel Machek wrote: Hi! On Thu 2017-01-12 20:19:31, Woody Suwalski wrote: Pavel Machek wrote: Hi! I used to have two cpus, and Thinkpad X60 should have two cores, but I only see one on 4.10-rc1. This machine went throug

Re: [PATCH v4 2/7] drm/tinydrm: Add helper functions

2017-02-12 Thread Noralf Trønnes
Den 12.02.2017 12.50, skrev Andy Shevchenko: On Sat, Feb 11, 2017 at 8:48 PM, Noralf Trønnes wrote: Add common functionality needed by many tinydrm drivers. +int tinydrm_enable_backlight(struct backlight_device *backlight) +{ + unsigned int old_state; + int ret; + + if (!back

[PATCH] genirq: devres: use dev_name(dev) as default for devname

2017-02-12 Thread Heiner Kallweit
Allow the devname parameter to be NULL and use dev_name(dev) in this case. This should be an appropriate default for most use cases. Signed-off-by: Heiner Kallweit --- kernel/irq/devres.c | 10 -- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/kernel/irq/devres.c b/kernel/

[PATCH 2/3] [media] si2168: add support for Si2168-D60

2017-02-12 Thread Stefan Brüns
Add handling for new revision, requiring new firmware. Signed-off-by: Stefan Brüns --- drivers/media/dvb-frontends/si2168.c | 4 drivers/media/dvb-frontends/si2168_priv.h | 2 ++ 2 files changed, 6 insertions(+) diff --git a/drivers/media/dvb-frontends/si2168.c b/drivers/media/dvb-fr

[PATCH 3/3] [media] cxusb: MyGica T230C support

2017-02-12 Thread Stefan Brüns
Mygica T230 DVB-T/T2/C USB stick support. It uses the same FX2/Si2168 bridge/demodulator combo as the T230, but uses the Si2141 tuner. Factor out the common code and pass the tuner type and if port as parameter, to avoid duplicating the initialization code. Signed-off-by: Stefan Brüns --- driver

[PATCH 0/3] Add support for MyGica T230C DVB-T2 stick

2017-02-12 Thread Stefan Brüns
The required command sequence for the new tuner (Si2141) was traced from the current Windows driver and verified with a small python script/libusb. The changes to the Si2168 and cxusb driver are mostly addition of the required IDs and some glue code. Stefan Brüns (3): [media] si2157: Add support

[PATCH 1/3] [media] si2157: Add support for Si2141-A10

2017-02-12 Thread Stefan Brüns
The Si2141 needs two distinct commands for powerup/reset, otherwise it will not respond to chip revision requests. It also needs a firmware to run properly. Signed-off-by: Stefan Brüns --- drivers/media/tuners/si2157.c | 23 +-- drivers/media/tuners/si2157_priv.h | 2 ++

Re: [PATCH v6] fork: free vmapped stacks in cache when cpus are offline

2017-02-12 Thread Thomas Gleixner
On Sun, 12 Feb 2017, Hoeun Ryu wrote: > Using virtually mapped stack, kernel stacks are allocated via vmalloc. > In the current implementation, two stacks per cpu can be cached when > tasks are freed and the cached stacks are used again in task duplications. > but the cached stacks may remain unf

Re: [PATCH] lz4: fix performance regressions

2017-02-12 Thread Sven Schmidt
On Sun, Feb 12, 2017 at 02:05:08PM +0100, Willy Tarreau wrote: > Hi Sven, > > On Sun, Feb 12, 2017 at 12:16:18PM +0100, Sven Schmidt wrote: > > Fix performance regressions compared to current kernel LZ4 > > Your patch contains mostly style cleanups which certainly are welcome > but make the whole

Re: [PATCH v9 0/8] drivers: touchscreen: tsc2007 and ads7846/tsc2046 improvements (use common touchscreen bindings, pre-calibration, spi fix and provide iio raw values)

2017-02-12 Thread H. Nikolaus Schaller
Hi Dmitry, > Am 28.01.2017 um 19:16 schrieb H. Nikolaus Schaller : > > Hi Dmitry, > there have been no further comments/complaints about this patch series in the > last month. > And it appears as if only your action is needed. > > What is missing for this patch series to be finally accepted? D

Re: [Intel-gfx] [PATCH] softirq: Prevent looping on disabled tasklets

2017-02-12 Thread kbuild test robot
-disabled-tasklets/20170212-220435 config: x86_64-rhel (attached as .config) compiler: gcc-6 (Debian 6.2.0-3) 6.2.0 20160901 reproduce: # save the attached .config to linux build tree make ARCH=x86_64 All errors (new ones prefixed by >>): >> ERROR: "raise_softirq"

[PATCH RFC] crypto: testmgr drop wrong init_completion

2017-02-12 Thread Nicholas Mc Guire
init_completion() is called here to reinitialize a completion object that was already re-initialized in wait_async_op() by reinit_completion() if complete (via tcrypt_complete()) had been called and wait_for_completion() returned, so no need to reinit it here. Fixes: commit 946cc46372dc ("crypto

Re: [Intel-gfx] [PATCH] softirq: Prevent looping on disabled tasklets

2017-02-12 Thread kbuild test robot
-disabled-tasklets/20170212-220435 config: x86_64-lkp (attached as .config) compiler: gcc-6 (Debian 6.2.0-3) 6.2.0 20160901 reproduce: # save the attached .config to linux build tree make ARCH=x86_64 All errors (new ones prefixed by >>): >> ERROR: "raise_softirq"

[PATCH v2] [media] dvb-usb-firmware: don't do DMA on stack

2017-02-12 Thread Stefan Brüns
The buffer allocation for the firmware data was changed in commit 43fab9793c1f ("dvb-usb: don't use stack for firmware load"), but the same applies for the reset value. Signed-off-by: Stefan Brüns --- This patch replaces the earlier submission conflicting with commit 43fab9793c1f, which was a sub

[PATCH] video/viafbdev: Make PROC_FS dependency explicit

2017-02-12 Thread Borislav Petkov
From: Borislav Petkov Fix: drivers/video/fbdev/via/viafbdev.c: In function ‘viafb_remove_proc’: drivers/video/fbdev/via/viafbdev.c:1635:4: warning: unused variable ‘iga2_entry’ [-Wunused-variable] *iga2_entry = shared->iga2_proc_entry; ^~ drivers/video/fbdev/via/viafbde

Re: [PATCH v2] PCI: pciehp: Don't enable PME on runtime suspend

2017-02-12 Thread Lukas Wunner
On Thu, Feb 09, 2017 at 05:32:54AM +0100, Lukas Wunner wrote: > On Wed, Feb 08, 2017 at 11:57:36AM -0600, Bjorn Helgaas wrote: > > On Tue, Feb 07, 2017 at 07:21:01AM +0100, Lukas Wunner wrote: > > > On Mon, Feb 06, 2017 at 04:15:02PM -0600, Bjorn Helgaas wrote: > > > > On Mon, Feb 06, 2017 at 10:20

Re: [PATCH] smiapp: add CCP2 support

2017-02-12 Thread kbuild test robot
Hi Pavel, [auto build test ERROR on linuxtv-media/master] [also build test ERROR on v4.10-rc7 next-20170210] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com/0day-ci/linux/commits/Pavel-Machek/smiapp-add-CCP2-support/2

[PATCH v2] softirq: Prevent looping on disabled tasklets

2017-02-12 Thread Chris Wilson
Disabling a tasklet causes it not to run during tasklet_action, but is put back onto the runnable tasklet list, and a new softirq raised. As the softirq is raised from within __do_softirq() this causing __do_softirq() to loop constantly until its timeslice expires and is transferred to the ksoftirq

Re: [PATCH] softirq: Prevent looping on disabled tasklets

2017-02-12 Thread Chris Wilson
On Sun, Feb 12, 2017 at 02:00:19PM +, Chris Wilson wrote: > Disabling a tasklet causes it not to run during tasklet_action, but is > put back onto the runnable tasklet list, and a new softirq raised. As > the softirq is raised from within __do_softirq() this causing > __do_softirq() to loop con

Trying to understand OOM killer

2017-02-12 Thread Paul Menzel
Dear Linux folks, since some time, at Linux 4.8, 4.9, and 4.10-rc6, the OOM kicks in on a 8 GB machine. ``` Feb 12 08:21:50 asrocke350m1 kernel: updatedb.mlocat invoked oom-killer: gfp_mask=0x16040d0(GFP_TEMPORARY|__GFP_COMP|__GFP_NOTRACK), nodemask= Feb 12 08:21:50 asrocke350m1 kernel: updated

Re: [PATCH] x86/vm86: fix compilation warning on a unused variable

2017-02-12 Thread Borislav Petkov
On Fri, Dec 16, 2016 at 11:19:16PM -0500, Jérémy Lefaure wrote: > I don't know why gcc raises a warning on that even if it is not used. > Anyway, I'm sure that the warning is reproducible. Both of your > solutions fix the issue. I still see the warning here. You wanna choose one solution, write a

[PATCH] softirq: Prevent looping on disabled tasklets

2017-02-12 Thread Chris Wilson
Disabling a tasklet causes it not to run during tasklet_action, but is put back onto the runnable tasklet list, and a new softirq raised. As the softirq is raised from within __do_softirq() this causing __do_softirq() to loop constantly until its timeslice expires and is transferred to the ksoftirq

Re: [PATCH v4 1/7] drm: Add DRM support for tiny LCD displays

2017-02-12 Thread Noralf Trønnes
Den 12.02.2017 12.05, skrev Andy Shevchenko: On Sat, Feb 11, 2017 at 8:48 PM, Noralf Trønnes wrote: tinydrm provides helpers for very simple displays that can use CMA backed framebuffers and need flushing on changes. +/** + * tinydrm_gem_cma_free_object - Free resources associated with a CMA G

<    1   2   3   4   >