[PATCH v2 00/21] lib, rtc: Print rtc_time via %ptR[dt][rv]

2018-02-20 Thread Andy Shevchenko
At some point I have noticed too many users of struct rtc_time that printing its content field by field. In this series I introduce %ptR[dt][rv] specifier to make life a bit easier. There are still users of detailed output of the struct rtc_time, but we can introduce an additional extension for t

[PATCH v2 13/21] rtc: rx6110: Switch to use %ptR

2018-02-20 Thread Andy Shevchenko
Use %ptR instead of open coded variant to print content of struct rtc_time in human readable format. Signed-off-by: Andy Shevchenko --- drivers/rtc/rtc-rx6110.c | 12 +++- 1 file changed, 3 insertions(+), 9 deletions(-) diff --git a/drivers/rtc/rtc-rx6110.c b/drivers/rtc/rtc-rx6110.c in

[PATCH v2 09/21] rtc: pic32: Switch to use %ptR

2018-02-20 Thread Andy Shevchenko
Use %ptR instead of open coded variant to print content of struct rtc_time in human readable format. Signed-off-by: Andy Shevchenko --- drivers/rtc/rtc-pic32.c | 18 -- 1 file changed, 4 insertions(+), 14 deletions(-) diff --git a/drivers/rtc/rtc-pic32.c b/drivers/rtc/rtc-pic32.

[PATCH v2 14/21] rtc: rx8025: Switch to use %ptR

2018-02-20 Thread Andy Shevchenko
Use %ptR instead of open coded variant to print content of struct rtc_time in human readable format. Signed-off-by: Andy Shevchenko --- drivers/rtc/rtc-rx8025.c | 19 --- 1 file changed, 4 insertions(+), 15 deletions(-) diff --git a/drivers/rtc/rtc-rx8025.c b/drivers/rtc/rtc-rx8

[PATCH v2 04/21] rtc: at91sam9: Switch to use %ptR

2018-02-20 Thread Andy Shevchenko
Use %ptR instead of open coded variant to print content of struct rtc_time in human readable format. Signed-off-by: Andy Shevchenko --- drivers/rtc/rtc-at91sam9.c | 16 1 file changed, 4 insertions(+), 12 deletions(-) diff --git a/drivers/rtc/rtc-at91sam9.c b/drivers/rtc/rtc-at

Re: [PATCH] RDMA/core: reduce IB_POLL_BATCH constant

2018-02-20 Thread Chuck Lever
> On Feb 20, 2018, at 4:14 PM, Bart Van Assche wrote: > > On Tue, 2018-02-20 at 21:59 +0100, Arnd Bergmann wrote: >> /* # of WCs to poll for with a single call to ib_poll_cq */ >> -#define IB_POLL_BATCH 16 >> +#define IB_POLL_BATCH 8 > > The purpose

[PATCH v2 07/21] rtc: mcp795: Switch to use %ptR

2018-02-20 Thread Andy Shevchenko
Use %ptR instead of open coded variant to print content of struct rtc_time in human readable format. Signed-off-by: Andy Shevchenko --- drivers/rtc/rtc-mcp795.c | 18 ++ 1 file changed, 6 insertions(+), 12 deletions(-) diff --git a/drivers/rtc/rtc-mcp795.c b/drivers/rtc/rtc-mcp7

[PATCH v2 12/21] rtc: rk808: Switch to use %ptR

2018-02-20 Thread Andy Shevchenko
Use %ptR instead of open coded variant to print content of struct rtc_time in human readable format. Signed-off-by: Andy Shevchenko --- drivers/rtc/rtc-rk808.c | 20 ++-- 1 file changed, 6 insertions(+), 14 deletions(-) diff --git a/drivers/rtc/rtc-rk808.c b/drivers/rtc/rtc-rk80

[PATCH v2 03/21] rtc: at91rm9200: Switch to use %ptR

2018-02-20 Thread Andy Shevchenko
Use %ptR instead of open coded variant to print content of struct rtc_time in human readable format. Signed-off-by: Andy Shevchenko --- drivers/rtc/rtc-at91rm9200.c | 15 --- 1 file changed, 4 insertions(+), 11 deletions(-) diff --git a/drivers/rtc/rtc-at91rm9200.c b/drivers/rtc/rtc

[PATCH v2 06/21] rtc: m48t59: Switch to use %ptR

2018-02-20 Thread Andy Shevchenko
Use %ptR instead of open coded variant to print content of struct rtc_time in human readable format. Signed-off-by: Andy Shevchenko --- drivers/rtc/rtc-m48t59.c | 8 ++-- 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/drivers/rtc/rtc-m48t59.c b/drivers/rtc/rtc-m48t59.c index 3

[PATCH v2 08/21] rtc: pcf50633: Switch to use %ptR

2018-02-20 Thread Andy Shevchenko
Use %ptR instead of open coded variant to print content of struct rtc_time in human readable format. Signed-off-by: Andy Shevchenko --- drivers/rtc/rtc-pcf50633.c | 8 ++-- 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/drivers/rtc/rtc-pcf50633.c b/drivers/rtc/rtc-pcf50633.c i

[PATCH v2 11/21] rtc: puv3: Switch to use %ptR

2018-02-20 Thread Andy Shevchenko
Use %ptR instead of open coded variant to print content of struct rtc_time in human readable format. Cc: Guan Xuetao Signed-off-by: Andy Shevchenko --- drivers/rtc/rtc-puv3.c | 18 -- 1 file changed, 4 insertions(+), 14 deletions(-) diff --git a/drivers/rtc/rtc-puv3.c b/drivers

[PATCH v2 02/21] rtc: Switch to use %ptR

2018-02-20 Thread Andy Shevchenko
Use %ptR instead of open coded variant to print content of struct rtc_time in human readable format. Cc: Arnd Bergmann Acked-by: Greg Kroah-Hartman Signed-off-by: Andy Shevchenko --- drivers/char/Kconfig| 1 + drivers/char/rtc.c | 7 +++ drivers/rtc/Kconfig | 1 + drivers/r

Re: [PATCH] RDMA/core: reduce IB_POLL_BATCH constant

2018-02-20 Thread Arnd Bergmann
On Tue, Feb 20, 2018 at 10:14 PM, Parav Pandit wrote: > Hi Arnd Bergmann, > >> -Original Message- >> From: linux-rdma-ow...@vger.kernel.org [mailto:linux-rdma- >> ow...@vger.kernel.org] On Behalf Of Arnd Bergmann >> Sent: Tuesday, February 20, 2018 2:59 PM >> To: Doug Ledford ; Jason Gunth

[PATCH 0/7] ARM: hacks for link-time optimization

2018-02-20 Thread Arnd Bergmann
Hi Nico, all, I was playing with ARM link-time optimization handling earlier this month, and eventually got it to build cleanly with randconfig kernels, but ended up with a lot of ugly hacks to actually pull it off. Here are the ones that I don't think we actually want to merge, but it may be hel

[PATCH 5/7] ARM: fix __inflate_kernel_data stack warning for LTO

2018-02-20 Thread Arnd Bergmann
Commit ca8b5d97d6bf ("ARM: XIP kernel: store .data compressed in ROM") moved the decompressor workspace to the stack and added a compiler flag to avoid the stack size warning. With LTO, that warning comes back. Moving the workspace into an initdata variable avoids that warning but presumably also

[PATCH 6/7] ARM: mark assembler-referenced symbols as __visible

2018-02-20 Thread Arnd Bergmann
I got link errors for references to local symbols from inline assembler, and also for referencing local symbols inside of inline assembler fragments from C. In both cases, making the symbols globally visible fixes the link process, but this seems a bit ugly, so I hope there is a better way to do t

[PATCH 3/7] [HACK] pass endianess flag to LTO linker

2018-02-20 Thread Arnd Bergmann
We need some way to pass -mbig-endian to the linker during the LTO link stage, otherwise we get a waning like arm-linux-gnueabi/bin/ld: arch/arm/lib/clearbit.o: compiled for a big endian system and target is little endian for each file we link in. There is probably a better method of passing th

[PATCH 4/7] ARM: io-acorn: fix LTO linking without CONFIG_PRINTK

2018-02-20 Thread Arnd Bergmann
When CONFIG_LTO is enabled, we get a link error for this file that contains a reference to printk(): arch/arm/lib/io-acorn.o: In function `outsl': (.text+0x38): undefined reference to `printk' Normally the file is simply dropped, but that doesn't happen with LTO. Making the reference conditional

[PATCH 1/7] ARM: disallow combining XIP and LTO

2018-02-20 Thread Arnd Bergmann
This fails during deflate_xip_data.sh /home/arnd/cross-gcc/bin/arm-linux-gnueabi-objcopy -O binary -R .comment -S vmlinux arch/arm/boot/xipImage && /bin/bash -c '/git/arm-soc/arch/arm/boot/deflate_xip_data.sh vmlinux arch/arm/boot/xipImage || { rm -f arch/arm/boot/xipImage; false; }' make -f

[PATCH 7/7] efi: disable LTO for EFI stub

2018-02-20 Thread Arnd Bergmann
Building the EFI stub on 32-bit ARM with LTO resulted in a build error: arm-linux-gnueabi-ld: drivers/firmware/efi/libstub/arm-stub.stub.o: plugin needed to handle lto object arch/arm/boot/compressed/head.o: In function `efi_stub_entry': Locally disabling LTO for this directory seems to do the t

[PATCH 2/7] ARM: LTO: avoid THUMB2_KERNEL+LTO

2018-02-20 Thread Arnd Bergmann
Trying to build an LTO-Enabled kernel with Thumb2 instructions failed horribly for me, with an endless output of things like ccVnNycO.s:2665: Error: thumb conditional instruction should be in IT block -- `bxne lr' ccVnNycO.s:7128: Error: thumb conditional instruction should be in IT block -- `st

Re: [PATCH 1/2] fs/efivarfs: restrict inode permissions

2018-02-20 Thread Linus Torvalds
On Tue, Feb 20, 2018 at 1:18 PM, Luck, Tony wrote: > ... >> > - inode = efivarfs_get_inode(sb, d_inode(root), S_IFREG | 0644, 0, >> > + inode = efivarfs_get_inode(sb, d_inode(root), S_IFREG | 0600, 0, >> >is_removable); > > Linus, > > Does this rate an exception

[PATCH] bpf: clean up unused-variable warning

2018-02-20 Thread Arnd Bergmann
The only user of this variable is inside of an #ifdef, causing a warning without CONFIG_INET: net/core/filter.c: In function 'bpf_sock_ops_cb_flags_set': net/core/filter.c:3382:6: error: unused variable 'val' [-Werror=unused-variable] int val = argval & BPF_SOCK_OPS_ALL_CB_FLAGS; This repla

Re: [PATCH] bpf: hide a possibly unused variable

2018-02-20 Thread Arnd Bergmann
On Tue, Feb 20, 2018 at 10:44 PM, Daniel Borkmann wrote: > Hi Arnd, > > On 02/20/2018 10:16 PM, Arnd Bergmann wrote: >> The only user of this variable is inside of an #ifdef, causing >> a warning without CONFIG_INET: >> >> net/core/filter.c: In function 'bpf_sock_ops_cb_flags_set': >> net/core

Re: [PATCH] bpf: hide a possibly unused variable

2018-02-20 Thread Daniel Borkmann
On 02/20/2018 11:08 PM, Arnd Bergmann wrote: > On Tue, Feb 20, 2018 at 10:44 PM, Daniel Borkmann > wrote: >> Hi Arnd, >> >> On 02/20/2018 10:16 PM, Arnd Bergmann wrote: >>> The only user of this variable is inside of an #ifdef, causing >>> a warning without CONFIG_INET: >>> >>> net/core/filter.c:

Re: [PATCH v7 0/5] Add Intel IOMMU debugfs support

2018-02-20 Thread Jacob Pan
On Sun, 18 Feb 2018 23:15:32 +0100 Yves-Alexis Perez wrote: > On Tue, 2018-02-13 at 13:40 -0800, Raj, Ashok wrote: > > This version has only hw dumps for now, but we plan to add some > > other things like walking 2nd level page-tables, or get some SVM > > specific data from the driver in the futu

Re: [RFC PATCH v3 1/6] x86/entry/64: move PUSH_AND_CLEAR_REGS from interrupt macro to helper function

2018-02-20 Thread Linus Torvalds
On Tue, Feb 20, 2018 at 1:01 PM, Dominik Brodowski wrote: > +ENTRY(interrupt_entry) > + UNWIND_HINT_FUNC > + > + PUSH_AND_CLEAR_REGS save_ret=1 > + ENCODE_FRAME_POINTER 8 > + > + ret > +END(interrupt_entry) There's nothing wrong with this patch, but it does expose what a n

Re: [PATCH v2 0/3] new driver for Valve Steam Controller

2018-02-20 Thread Pierre-Loup A. Griffais
Hi Rodrigo, Thanks for working on that! I have a few questions and remarks. For the reverse-engineering part, there's a lot of existing reference in existing (user-space) drivers for the controllers like sc-controller, but feel free to reach out if you have any questions. It's overall pretty

Re: [PATCH] blk: optimization for classic polling

2018-02-20 Thread Jens Axboe
On 2/20/18 3:21 AM, Peter Zijlstra wrote: > On Sun, May 30, 2083 at 09:51:06AM +0530, Nitesh Shetty wrote: >> This removes the dependency on interrupts to wake up task. Set task >> state as TASK_RUNNING, if need_resched() returns true, >> while polling for IO completion. >> Earlier, polling task us

Re: [PATCH] fs: proc: use down_read_killable in proc_pid_cmdline_read()

2018-02-20 Thread Alexey Dobriyan
On Wed, Feb 21, 2018 at 03:49:29AM +0800, Yang Shi wrote: > When running vm-scalability with large memory (> 300GB), the below hung > task issue happens occasionally. > > INFO: task ps:14018 blocked for more than 120 seconds. >Tainted: GE 4.9.79-009.ali3000.alios7.x86_64 #1 >

Re: [4.4-stable 08/22] ncr5380: shut up gcc indentation warning

2018-02-20 Thread Finn Thain
On Tue, 20 Feb 2018, Arnd Bergmann wrote: > gcc-6 and higher warn about the way some loops are written in > the ncr5380 driver: > > drivers/scsi/g_NCR5380.c: In function 'generic_NCR5380_pread': > drivers/scsi/g_NCR5380.c:541:3: error: this 'while' clause does not guard... > [-Werror=misleading-

[PATCH] arm64: cpufeature: Trim feature reporting and include PAN emulation

2018-02-20 Thread Kees Cook
The PAN emulation notification was only happening for non-boot CPUs if CPU capabilities had already been configured. This seems to be the wrong place, as it's system-wide and isn't attached to capabilities, so its reporting didn't normally happen. Instead, report it once from the boot CPU. Addition

[PATCH v3] MAINTAINERS: add Freescale pin controllers

2018-02-20 Thread Stefan Agner
Add Dong Aisheng, Fabio Estevam, Shawn Guo and myself as maintainer and the Pengutronix kernel team as reviewer. Signed-off-by: Stefan Agner Reviewed-by: Fabio Estevam Acked-by: Dong Aisheng --- MAINTAINERS | 11 +++ 1 file changed, 11 insertions(+) diff --git a/MAINTAINERS b/MAINTAIN

[PATCH V2] Allow configuring whether unsigned modules taint the kernel

2018-02-20 Thread Matthew Garrett
Distributions build kernels that are intended to satisfy multiple use-cases. One is that kernels must be usable in scenarios where module signing is required (such as many Secure Boot policies) and scenarios where it isn't (such as booting the same kernel on a legacy BIOS system). This requires tha

[PATCH v4] tpm: Trigger only missing TPM 2.0 self tests

2018-02-20 Thread Jarkko Sakkinen
From: Alexander Steffen My Nuvoton 6xx in a Dell XPS-13 has been intermittently failing to work (necessitating a reboot). The problem seems to be that the TPM gets into a state where the partial self-test doesn't return TPM_RC_SUCCESS (meaning all tests have run to completion), but instead return

[PATCH] floppy: Don't print kernel addresses to log in show_floppy

2018-02-20 Thread Brian Belleville
Outputting kernel addresses will reveal the locations of kernel code and data. Change the cases in show_floppy that print fd_timer.work.func and fd_timeout.work.func to use the %pf format specifier, which will print the symbol name, like what is done for the other function pointers printed by show_

Re: [PATCH v3] MAINTAINERS: add Freescale pin controllers

2018-02-20 Thread Stefan Agner
On 20.02.2018 23:51, Stefan Agner wrote: > Add Dong Aisheng, Fabio Estevam, Shawn Guo and myself as maintainer > and the Pengutronix kernel team as reviewer. > > Signed-off-by: Stefan Agner > Reviewed-by: Fabio Estevam > Acked-by: Dong Aisheng > --- > MAINTAINERS | 11 +++ > 1 file cha

[PATCH v4] MAINTAINERS: add Freescale pin controllers

2018-02-20 Thread Stefan Agner
Add Dong Aisheng, Fabio Estevam, Shawn Guo and myself as maintainer and the Pengutronix kernel team as reviewer. Signed-off-by: Stefan Agner Reviewed-by: Fabio Estevam Acked-by: Dong Aisheng --- MAINTAINERS | 11 +++ 1 file changed, 11 insertions(+) diff --git a/MAINTAINERS b/MAINTAIN

Re: [PATCH v4] tpm: Trigger only missing TPM 2.0 self tests

2018-02-20 Thread Jarkko Sakkinen
On Wed, Feb 21, 2018 at 12:53:47AM +0200, Jarkko Sakkinen wrote: > From: Alexander Steffen > > My Nuvoton 6xx in a Dell XPS-13 has been intermittently failing to work > (necessitating a reboot). The problem seems to be that the TPM gets into a > state where the partial self-test doesn't return TP

Re: [PATCHv2 6/8] MIPS: Loongson64: cleanup all mach files to use SPDX Identifier

2018-02-20 Thread James Hogan
On Sun, Dec 31, 2017 at 02:28:29AM +0800, Jiaxun Yang wrote: > To reduce unnecessary license text. > > Signed-off-by: Jiaxun Yang > --- > arch/mips/loongson64/Makefile | 1 + > arch/mips/loongson64/Platform | 1 + > 2 files changed, 2 insertions(+) > > diff --git a/arch/mips/loongson64/Makefile

Re: A "domain invalid" cgroup *can* sometimes have member tasks

2018-02-20 Thread Tejun Heo
On Tue, Feb 20, 2018 at 11:35:47AM -0800, Tejun Heo wrote: > Hmm... nr_populated_domain_children check should have caught that > condition and rejected it. Will look into what's going on. Ah, okay, I was special-casing the first level children case too early. If you nest the test case by one mor

Re: [PATCH 1/2 v3] tpm: cmd_ready command can be issued only after granting locality

2018-02-20 Thread Jarkko Sakkinen
On Tue, Feb 20, 2018 at 08:26:45PM +, Winkler, Tomas wrote: > > > > On Mon, 2018-02-19 at 11:43 +, Winkler, Tomas wrote: > > > > All local variable declarations must be in the beginning of the > > > > function. > > > > > > Who says? > > > > It is coherent how we have everything else. > I

Re: [PATCHv2 1/8] MIPS: Loongson64: cleanup all cs5536 files to use SPDX Identifier

2018-02-20 Thread James Hogan
On Sun, Dec 31, 2017 at 02:28:24AM +0800, Jiaxun Yang wrote: > diff --git a/arch/mips/loongson64/common/cs5536/cs5536_acc.c > b/arch/mips/loongson64/common/cs5536/cs5536_acc.c > index ab4d6cc57384..ba0474bb4a3d 100644 > --- a/arch/mips/loongson64/common/cs5536/cs5536_acc.c > +++ b/arch/mips/loongs

[PATCH v4 04/16] of: changesets: Introduce changeset helper methods

2018-02-20 Thread Laurent Pinchart
From: Pantelis Antoniou Changesets are very powerful, but the lack of a helper API makes using them cumbersome. Introduce a simple copy based API that makes things considerably easier. To wit, adding a property using the raw API. struct property *prop; prop = kzalloc(sizeof(*pro

[PATCH v4 07/16] i2c: demux: Use changeset helpers for clarity

2018-02-20 Thread Laurent Pinchart
From: Pantelis Antoniou The changeset helpers are easier to use, use them instead of using the static property. Signed-off-by: Pantelis Antoniou Acked-by: Wolfram Sang ["okay" -> "ok"] Signed-off-by: Laurent Pinchart --- drivers/i2c/muxes/i2c-demux-pinctrl.c | 12 +++- 1 file changed

[PATCH v4 06/16] of: unittest: changeset helpers

2018-02-20 Thread Laurent Pinchart
From: Pantelis Antoniou Add a unitest specific for the new changeset helpers. Signed-off-by: Pantelis Antoniou Signed-off-by: Laurent Pinchart --- drivers/of/unittest.c | 54 +++ 1 file changed, 54 insertions(+) diff --git a/drivers/of/unittest

[PATCH v4 05/16] of: changeset: Add of_changeset_node_move method

2018-02-20 Thread Laurent Pinchart
From: Pantelis Antoniou Adds a changeset helper for moving a subtree to a different place in the running tree. This is useful in advances cases of dynamic device tree construction. Signed-off-by: Pantelis Antoniou Signed-off-by: Laurent Pinchart --- drivers/of/dynamic.c | 66 +

[PATCH v4 00/16] R-Car DU: Convert LVDS code to bridge driver

2018-02-20 Thread Laurent Pinchart
Hello, This patch series addresses a design mistake that dates back from the initial DU support. Support for the LVDS encoders, which are IP cores separate from the DU, was bundled in the DU driver. Worse, both the DU and LVDS were described through a single DT node. To fix the, patches 01/16 and

[PATCH v4 03/16] of: dynamic: Add __of_node_dupv()

2018-02-20 Thread Laurent Pinchart
From: Pantelis Antoniou Add an __of_node_dupv() private method and make __of_node_dup() use it. This is required for the subsequent changeset accessors which will make use of it. Signed-off-by: Pantelis Antoniou Signed-off-by: Laurent Pinchart --- drivers/of/dynamic.c | 29 +++

Re: [PATCH 2/3] sysctl: Warn when a clamped sysctl parameter is set out of range

2018-02-20 Thread Andrew Morton
On Mon, 19 Feb 2018 11:53:50 -0500 Waiman Long wrote: > Even with clamped sysctl parameters, it is still not that straight > forward to figure out the exact range of those parameters. One may > try to write extreme parameter values to see if they get clamped. > To make it easier, a warning with t

Re: [PATCH 1/2] fs/efivarfs: restrict inode permissions

2018-02-20 Thread Andy Lutomirski
On Tue, Feb 20, 2018 at 7:18 PM, Andy Lutomirski wrote: > On 02/15/2018 10:22 AM, Joe Konno wrote: >> >> From: Joe Konno >> >> Efivarfs nodes are created with group and world readable permissions. >> Reading certain EFI variables trigger SMIs. So, this is a potential DoS >> surface. >> >> Make pe

Re: [PATCH v2 0/3] new driver for Valve Steam Controller

2018-02-20 Thread Rodrigo Rivas Costa
On Tue, Feb 20, 2018 at 02:29:48PM -0800, Pierre-Loup A. Griffais wrote: > Hi Rodrigo, > > Thanks for working on that! I have a few questions and remarks. > > For the reverse-engineering part, there's a lot of existing reference in > existing (user-space) drivers for the controllers like sc-contr

Re: [RFC PATCH V2 13/22] x86/intel_rdt: Support schemata write - pseudo-locking core

2018-02-20 Thread Thomas Gleixner
Reinette, On Tue, 20 Feb 2018, Reinette Chatre wrote: > On 2/20/2018 9:15 AM, Thomas Gleixner wrote: > > On Tue, 13 Feb 2018, Reinette Chatre wrote: > > > > Are you really sure that the life time rules of plr are correct vs. an > > application which still has the locked memory mapped? i.e. the fo

Re: [PATCH] mm: don't defer struct page initialization for Xen pv guests

2018-02-20 Thread Andrew Morton
On Mon, 19 Feb 2018 02:45:27 +0800 kbuild test robot wrote: > [auto build test ERROR on mmotm/master] > [also build test ERROR on v4.16-rc1 next-20180216] > [if your patch is applied to the wrong git tree, please drop us a note to > help improve the system] > > url: > https://github.com/0da

[PATCH RFC tools/lkmm 0/12] Miscellaneous fixes

2018-02-20 Thread Paul E. McKenney
Hello! This RFC series adds some miscellaneous updates to the Linux kernel memory model: 1. Clarify the origin and scope of the tool name to avoid confusion between "memory model" and "memory management", courtesy of Andrea Parri. 2. Move the maintainer list for LKMM to

[PATCH RFC tools/lkmm 11/12] tools/memory-model: Convert underscores to hyphens

2018-02-20 Thread Paul E. McKenney
Typical cat-language code uses hyphens for word separators in identifiers, but several LKMM identifiers use underscores instead. This commit therefore converts underscores to hyphens in the .bell- and .cat-file identifiers corresponding to smp_mb__before_atomic(), smp_mb__after_atomic(), and smp_mb

[PATCH RFC tools/lkmm 05/12] README: Fix a couple of punctuation errors

2018-02-20 Thread Paul E. McKenney
Signed-off-by: Paul E. McKenney --- tools/memory-model/litmus-tests/README | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tools/memory-model/litmus-tests/README b/tools/memory-model/litmus-tests/README index 9a3bb5949191..dca7d823ad57 100644 --- a/tools/memory-model/litm

[PATCH RFC tools/lkmm 01/12] tools/memory-model: Clarify the origin/scope of the tool name

2018-02-20 Thread Paul E. McKenney
From: Andrea Parri Ingo pointed out that: "The "memory model" name is overly generic, ambiguous and somewhat misleading, as we usually mean the virtual memory layout/model when we say "memory model". GCC too uses it in that sense [...]" Make it clear that tools/memory-model/ uses the te

[PATCH RFC tools/lkmm 04/12] EXP litmus_tests: Add comments explaining tests' purposes

2018-02-20 Thread Paul E. McKenney
This commit adds comments to the litmus tests summarizing what these tests are intended to demonstrate. Suggested-by: Ingo Molnar Signed-off-by: Paul E. McKenney [ paulmck: Apply Andrea's and Alan's feedback. ] --- .../memory-model/litmus-tests/CoRR+poonceonce+Once.litmus | 7 +++ .../mem

[PATCH RFC tools/lkmm 06/12] MAINTAINERS: Add Akira Yokosawa as an LKMM reviewer

2018-02-20 Thread Paul E. McKenney
Signed-off-by: Paul E. McKenney --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index 674b35c3f28e..cc4c1ac94593 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -8097,6 +8097,7 @@ M:David Howells M: Jade Alglave M: Luc Maranget M:

[PATCH RFC tools/lkmm 07/12] Documentation/memory-barriers.txt: Cross-reference "tools/memory-model/"

2018-02-20 Thread Paul E. McKenney
From: Andrea Parri A memory consistency model is now available for the Linux kernel [1], which "can (roughly speaking) be thought of as an automated version of memory-barriers.txt" and which is (in turn) "accompanied by extensive documentation on its use and its design". Inform the (occasional)

[PATCH RFC tools/lkmm 12/12] tools/memory-model: Remove rb-dep, smp_read_barrier_depends, and lockless_dereference

2018-02-20 Thread Paul E. McKenney
From: Alan Stern Since commit 76ebbe78f739 ("locking/barriers: Add implicit smp_read_barrier_depends() to READ_ONCE()") was merged for the 4.15 kernel, it has not been necessary to use smp_read_barrier_depends(). Similarly, commit 59ecbbe7b31c ("locking/barriers: Kill lockless_dereference()") rem

[PATCH RFC tools/lkmm 03/12] MAINTAINERS: List file memory-barriers.txt within the LKMM entry

2018-02-20 Thread Paul E. McKenney
From: Andrea Parri We now have a shiny new Linux-kernel memory model (LKMM) and the old tried-and-true Documentation/memory-barrier.txt. It would be good to keep these automatically synchronized, but in the meantime we need at least let people know that they are related. Will suggested adding t

[PATCH RFC tools/lkmm 10/12] tools/memory-model: Add a S lock-based external-view litmus test

2018-02-20 Thread Paul E. McKenney
From: Alan Stern This commit adds a litmus test in which P0() and P1() form a lock-based S litmus test, with the addition of P2(), which observes P0()'s and P1()'s accesses with a full memory barrier but without the lock. This litmus test asks whether writes carried out by two different processe

[PATCH RFC tools/lkmm 09/12] tools/memory-model: Add required herd7 version to README file

2018-02-20 Thread Paul E. McKenney
LKMM and the herd7 tool are co-evolving, and out-of-date herd7 tools produce inaccurate results, often with no obvious error messages. This commit therefore adds the required herd7 version to the LKMM README file. Longer term, it would be good if .cat files could specify the required version in a

[PATCH RFC tools/lkmm 02/12] MAINTAINERS: Add the Memory Consistency Model subsystem

2018-02-20 Thread Paul E. McKenney
From: Andrea Parri Move the contents of tools/memory-model/MAINTAINERS into the main MAINTAINERS file, removing tools/memory-model/MAINTAINERS. This allows get_maintainer.pl to correctly identify the maintainers of tools/memory-model/. Suggested-by: Ingo Molnar Signed-off-by: Andrea Parri Acke

[PATCH RFC tools/lkmm 08/12] memory-barriers: Fix description of data dependency barriers

2018-02-20 Thread Paul E. McKenney
From: Nikolay Borisov In the description of data dependency barriers the words 'before' is used erroneously. Since such barrier order dependent loads one after the other. So substitute 'before' with 'after'. Signed-off-by: Nikolay Borisov Signed-off-by: Paul E. McKenney --- Documentation/memo

Re: [PATCH 1/2] fs/efivarfs: restrict inode permissions

2018-02-20 Thread Luck, Tony
On Tue, Feb 20, 2018 at 02:01:51PM -0800, Linus Torvalds wrote: > And just on general principlies, I don't want to see weasel-wordy > commit messages like > > "Reading certain EFI variables trigger SMIs" > > I understand *writing* them causing SMI's due to some flash protection > scheme. What's

Re: [PATCH] fs: proc: use down_read_killable in proc_pid_cmdline_read()

2018-02-20 Thread Yang Shi
On 2/20/18 2:38 PM, Alexey Dobriyan wrote: On Wed, Feb 21, 2018 at 03:49:29AM +0800, Yang Shi wrote: When running vm-scalability with large memory (> 300GB), the below hung task issue happens occasionally. INFO: task ps:14018 blocked for more than 120 seconds. Tainted: GE

Re: [PATCH -mm -v5 RESEND] mm, swap: Fix race between swapoff and some swap operations

2018-02-20 Thread Andrew Morton
On Sun, 18 Feb 2018 09:06:47 +0800 huang ying wrote: > >> >> +struct swap_info_struct *get_swap_device(swp_entry_t entry) > >> >> +{ > >> >> + struct swap_info_struct *si; > >> >> + unsigned long type, offset; > >> >> + > >> >> + if (!entry.val) > >> >> + goto out; > >> >> + type =

Re: [PATCH 2/3] mm: memcg: plumbing memcg for kmalloc allocations

2018-02-20 Thread kbuild test robot
Hi Shakeel, Thank you for the patch! Yet something to improve: [auto build test ERROR on mmotm/master] [also build test ERROR on v4.16-rc2 next-20180220] [cannot apply to linus/master] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url

Re: [PATCH 1/2] fs/efivarfs: restrict inode permissions

2018-02-20 Thread Matthew Garrett
On Tue, Feb 20, 2018 at 3:30 PM Luck, Tony wrote: > [1] I didn't dig through the Linux code to check whether we manage to > get those four SMIs from a single EFI call, or whether we make multiple > EFI calls to open/read/close one file. It is possible that we stink a > bit too if we are doing more

Re: [PATCH v1 0/2] PCI: Sparc 64-bit resource fixups

2018-02-20 Thread Bjorn Helgaas
[+cc David Ahern, TJ] On Thu, Feb 15, 2018 at 09:17:58AM -0600, Bjorn Helgaas wrote: > I'm trying to make some progress on this old series from Yinghai [1]. > There's a lot more to do than just these first two patches, but maybe a > tiny bit of incremental progress is better than none. > > The fi

Re: [PATCH 1/3] sysctl: Add range clamping intvec helper functions

2018-02-20 Thread Luis R. Rodriguez
On Mon, Feb 19, 2018 at 11:53:49AM -0500, Waiman Long wrote: > The current intvec range helper functions will fail with error when > users try to assign an out-of-range value. As designed. > The following new non-failing range helper functions are now added: > - proc_dointvec_clamp_minmax() > -

metag build errors in mainline kernel

2018-02-20 Thread Guenter Roeck
The following errors are seen when building the kernel with the latest available metag toolchain (4.2.4 (IMG-1.4.0.700)). Kernel version tested is (v4.16-rc2-64-gaf3e79d29555). metag:allmodconfig arch/metag/kernel/devtree.c:54: error: implicit declaration of function 'pr_info' arch/metag/kernel

Re: [PATCH 5/6] powerpc: Implement DISCONTIGMEM and allow selection on PPC32

2018-02-20 Thread kbuild test robot
Hi Jonathan, Thank you for the patch! Yet something to improve: [auto build test ERROR on powerpc/next] [also build test ERROR on v4.16-rc2 next-20180220] [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

RE: [PATCH 1/2] fs/efivarfs: restrict inode permissions

2018-02-20 Thread Luck, Tony
> read() will make two calls - one to obtain the size of the variable, the > other to read it. It looks like cat will also trigger an fstat(), so we're > probably also making a call for that. There's presumably some optimisation > that could be made there if we trust the firmware not to change the

Re: [PATCH 2/3] sysctl: Warn when a clamped sysctl parameter is set out of range

2018-02-20 Thread Luis R. Rodriguez
On Tue, Feb 20, 2018 at 03:17:05PM -0800, Andrew Morton wrote: > On Mon, 19 Feb 2018 11:53:50 -0500 Waiman Long wrote: > > > Even with clamped sysctl parameters, it is still not that straight > > forward to figure out the exact range of those parameters. One may > > try to write extreme parameter

Re: [PATCH v2 01/21] lib/vsprintf: Print time and date in human readable format via %pt

2018-02-20 Thread Joe Perches
On Tue, 2018-02-20 at 23:43 +0200, Andy Shevchenko wrote: > There are users which print time and date represented by content of > struct rtc_time in human readable format. > > Instead of open coding that each time introduce %ptR[dt][rv] specifier. > > Note, users have to select PRINTK_PEXT_TIMEDA

Re: [PATCH 1/4] locking/ww_mutex: add ww_mutex_is_owned_by function v3

2018-02-20 Thread Daniel Vetter
On Tue, Feb 20, 2018 at 04:21:58PM +0100, Peter Zijlstra wrote: > On Tue, Feb 20, 2018 at 04:05:49PM +0100, Christian König wrote: > > Am 20.02.2018 um 15:54 schrieb Peter Zijlstra: > > > On Tue, Feb 20, 2018 at 03:34:07PM +0100, Christian König wrote: > > > > > OK, but neither case would in fact n

Re: [RFC PATCH v16 0/6] mm: security: ro protection for dynamic data

2018-02-20 Thread Matthew Wilcox
On Wed, Feb 21, 2018 at 08:36:04AM +1100, Dave Chinner wrote: > FWIW, I'm not wanting to use it to replace static variables. All the > structures are dynamically allocated right now, and get assigned to > other dynamically allocated pointers. I'd likely split the current > structures into a "ro aft

Re: [PATCH] fs: proc: use down_read_killable in proc_pid_cmdline_read()

2018-02-20 Thread Yang Shi
On 2/20/18 11:49 AM, Yang Shi wrote: When running vm-scalability with large memory (> 300GB), the below hung task issue happens occasionally. INFO: task ps:14018 blocked for more than 120 seconds. Tainted: GE 4.9.79-009.ali3000.alios7.x86_64 #1 "echo 0 > /proc/sys/kernel/

Re: [PATCH] proc/kpageflags: add KPF_WAITERS

2018-02-20 Thread Andrew Morton
On Sat, 17 Feb 2018 11:14:10 +0300 Konstantin Khlebnikov wrote: > On 17.02.2018 02:57, Andrew Morton wrote: > > On Sun, 11 Feb 2018 13:36:41 +0300 Konstantin Khlebnikov > > wrote: > > > >> KPF_WAITERS indicates tasks are waiting for a page lock or writeback. > >> This might be false-positive,

Re: [PATCH 2/2] proc: use set_puts() at /proc/*/wchan

2018-02-20 Thread Andrew Morton
On Sat, 17 Feb 2018 16:06:42 +0200 Andy Shevchenko wrote: > On Sat, Feb 17, 2018 at 9:20 AM, Alexey Dobriyan wrote: > > Signed-off-by: Alexey Dobriyan > > > > - seq_printf(m, "%s", symname); > > + seq_puts(m, symname); > > While this might have no security concer

[PATCH 3/3] ACPI / PMIC: Update to SPDX license identifier

2018-02-20 Thread Rajmohan Mani
Remove the GPL v2 license boilerplate and update with the SPDX license identifier. Signed-off-by: Rajmohan Mani --- drivers/acpi/pmic/tps68470_pmic.c | 10 +- 1 file changed, 1 insertion(+), 9 deletions(-) diff --git a/drivers/acpi/pmic/tps68470_pmic.c b/drivers/acpi/pmic/tps68470_pmic

[PATCH 2/3] gpio: Update to SPDX license identifier

2018-02-20 Thread Rajmohan Mani
Remove the GPL v2 license boilerplate and update with the SPDX license identifier. Signed-off-by: Rajmohan Mani --- drivers/gpio/gpio-tps68470.c | 10 +- 1 file changed, 1 insertion(+), 9 deletions(-) diff --git a/drivers/gpio/gpio-tps68470.c b/drivers/gpio/gpio-tps68470.c index fa2662f

[PATCH 0/3] Update TPS68470 PMIC drivers with SPDX tags

2018-02-20 Thread Rajmohan Mani
This patch series update the TPS68470 PMIC drivers with SPDX license identifiers, while removing the GPL v2 boilerplate license text. Rajmohan Mani (3): mfd: Update to SPDX license identifier gpio: Update to SPDX license identifier ACPI / PMIC: Update to SPDX license identifier drivers/acp

[PATCH 1/3] mfd: Update to SPDX license identifier

2018-02-20 Thread Rajmohan Mani
Remove the GPL v2 license boilerplate and update with the SPDX license identifier. Signed-off-by: Rajmohan Mani --- drivers/mfd/tps68470.c | 10 +- include/linux/mfd/tps68470.h | 17 +++-- 2 files changed, 4 insertions(+), 23 deletions(-) diff --git a/drivers/mfd/tps68

Re: [PATCH v2 0/3] new driver for Valve Steam Controller

2018-02-20 Thread Pierre-Loup A. Griffais
On 02/20/2018 03:20 PM, Rodrigo Rivas Costa wrote: On Tue, Feb 20, 2018 at 02:29:48PM -0800, Pierre-Loup A. Griffais wrote: Hi Rodrigo, Thanks for working on that! I have a few questions and remarks. For the reverse-engineering part, there's a lot of existing reference in existing (user-space)

Re: [PATCH 3/4] kernel/fork: switch vmapped stack callation to __vmalloc_area()

2018-02-20 Thread Andrew Morton
On Tue, 23 Jan 2018 16:57:21 +0300 Konstantin Khlebnikov wrote: > # stress-ng --clone 100 -t 10s --metrics-brief > at 32-core machine shows boost 35000 -> 36000 bogo ops > > Patch 4/4 is a kind of RFC. > Actually per-cpu cache of preallocated stacks works faster than buddy > allocator thus > p

Re: [PATCH] fork: Allow stack to be wiped on fork

2018-02-20 Thread Andrew Morton
On Tue, 16 Jan 2018 21:50:15 -0800 Kees Cook wrote: > One of the classes of kernel stack content leaks is exposing the contents > of prior heap or stack contents when a new process stack is allocated. > Normally, those stacks are not zeroed, and the old contents remain in > place. With some types

[PATCH v5] rtc: isl12026: Add driver.

2018-02-20 Thread David Daney
The ISL12026 is a combination RTC and EEPROM device with I2C interface. The standard RTC driver interface is provided. The EEPROM is accessed via the NVMEM interface via the "eeprom0" directory in the sysfs entry for the device. Reviewed-by: Rob Herring Reviewed-by: Andy Shevchenko Signed-off-

Re: [RFC patch] ioremap: don't set up huge I/O mappings when p4d/pud/pmd is zero

2018-02-20 Thread Kani, Toshi
On Tue, 2018-02-20 at 14:54 +0530, Chintan Pandya wrote: > > On 12/28/2017 4:54 PM, Hanjun Guo wrote: > > From: Hanjun Guo > > > > When we using iounmap() to free the 4K mapping, it just clear the PTEs > > but leave P4D/PUD/PMD unchanged, also will not free the memory of page > > tables. > > >

Re: [PATH 0/4] usbip: make vhci_hcd.* objects independent of vhci_hcd.0

2018-02-20 Thread Shuah Khan
Hi Salvador, On 01/30/2018 01:36 AM, Salvador Fandino wrote: > Let me start by explaining the problem that have motivated me to write > this patches: > > I work on the QVD, a virtual desktop platform for Linux. This software > runs Linux desktops (i.e. XFCE, KDE) and their applications inside LXC

Re: [PATCH v5 0/4] vm: add a syscall to map a process memory into a pipe

2018-02-20 Thread Andrew Morton
On Tue, 9 Jan 2018 08:30:49 +0200 Mike Rapoport wrote: > This patches introduces new process_vmsplice system call that combines > functionality of process_vm_read and vmsplice. All seems fairly strightforward. The big question is: do we know that people will actually use this, and get suffici

Re: [PATCH 1/2] fs/efivarfs: restrict inode permissions

2018-02-20 Thread Linus Torvalds
On Tue, Feb 20, 2018 at 3:30 PM, Luck, Tony wrote: > > Too much weasel there. Should say: > > EFI[1] stinks. Reading any file in /sys/firmware/efi/efivars/ generates > 4 (yes FOUR!) SMIs. Is that actualkly the normal implementation? Also, if these are just synchronous SMI's, then don't we just

Re: [PATCH 1/2] fs/efivarfs: restrict inode permissions

2018-02-20 Thread Peter Jones
On Tue, Feb 20, 2018 at 02:01:51PM -0800, Linus Torvalds wrote: > Which variables are actually used by user space tools? It sounds like > it may be exactly those boot order things that we already have flags > and special behavior for. Not that many are really part of a non-root workflow. The big

Re: [PATCH 2/3] mm: memcg: plumbing memcg for kmalloc allocations

2018-02-20 Thread kbuild test robot
Hi Shakeel, Thank you for the patch! Yet something to improve: [auto build test ERROR on mmotm/master] [also build test ERROR on v4.16-rc2 next-20180220] [cannot apply to linus/master] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url

[PATCH] media: dw9714: Update to SPDX license identifier

2018-02-20 Thread Rajmohan Mani
Remove the GPL v2 license boilerplate and update with the SPDX license identifier. Signed-off-by: Rajmohan Mani --- drivers/media/i2c/dw9714.c | 14 ++ 1 file changed, 2 insertions(+), 12 deletions(-) diff --git a/drivers/media/i2c/dw9714.c b/drivers/media/i2c/dw9714.c index 8dbbf0f

<    1   2   3   4   5   6   7   8   9   >