Re: [PATCH] arch: all: include: asm: bitops: Use bool instead of int for all bit test functions

2016-08-27 Thread kbuild test robot
Hi Chen, [auto build test WARNING on linus/master] [also build test WARNING on v4.8-rc3 next-20160825] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] [Suggest to use git(>=2.9.0) format-patch --base= (or --base=auto for convenience) to record w

[PATCH] memstick: r592: add sanity check in r592.c

2016-08-27 Thread Baoyou Xie
in function memstick_debug_get_tpc_name, it miss sanity check and will cause a panic. so this patch checks if the tpc parameter is valid, and if not so, it returns a dummy value to caller. Signed-off-by: Baoyou Xie --- drivers/memstick/host/r592.c | 3 +++ 1 file changed, 3 insertions(+) diff

[PATCH v2 3/5] checkkconfigsymbols.py: fix pylint and pep8 warnings

2016-08-27 Thread Valentin Rothberg
Fix pylint and pep8 warnings to have a consistent syntax and style. Signed-off-by: Valentin Rothberg --- scripts/checkkconfigsymbols.py | 18 -- 1 file changed, 8 insertions(+), 10 deletions(-) diff --git a/scripts/checkkconfigsymbols.py b/scripts/checkkconfigsymbols.py index 32

[PATCH v2 4/5] checkkconfigsymblos: consistent symbol terminology

2016-08-27 Thread Valentin Rothberg
'symbol' and 'feature' are used synonymously to refer to Kconfig symbols (configs, menus, etc.). Use the term 'symbol' to have a consistent terminology and to make the code more comprehensible. Signed-off-by: Valentin Rothberg --- scripts/checkkconfigsymbols.py | 134 ---

[PATCH v2 1/5] checkkconfigsymbols.py: port to Python 3

2016-08-27 Thread Valentin Rothberg
Python 2 is slowly dying, so port the script to Python 3. Signed-off-by: Valentin Rothberg --- scripts/checkkconfigsymbols.py | 33 + 1 file changed, 17 insertions(+), 16 deletions(-) diff --git a/scripts/checkkconfigsymbols.py b/scripts/checkkconfigsymbols.py in

[PATCH v2 2/5] checkkconfigsymbols: use ArgumentParser

2016-08-27 Thread Valentin Rothberg
Replace the deprecated OptionParser with ArgumentParser, as recommended by pylint. Signed-off-by: Valentin Rothberg --- scripts/checkkconfigsymbols.py | 142 - 1 file changed, 69 insertions(+), 73 deletions(-) diff --git a/scripts/checkkconfigsymbols.py b

[PATCH v2 0/5] checkkconfigsymbols: several updates

2016-08-27 Thread Valentin Rothberg
Note that the patches base on [1], which has already been applied to Greg's char-misc tree [2] (testing branch). [1] https://lkml.org/lkml/2016/8/27/50 [2] git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/char-misc.git Changelog v2: - Break v1 into multiple changes (i.e., python3 port, Argume

[PATCH v2 5/5] checkkconfigsymbols: use arglist instead of cmd string

2016-08-27 Thread Valentin Rothberg
Splitting a command string could lead to unintended arguments. Use an argument list in the execute() function instead. Signed-off-by: Valentin Rothberg --- scripts/checkkconfigsymbols.py | 27 --- 1 file changed, 16 insertions(+), 11 deletions(-) diff --git a/scripts/ch

[PATCH v2] raid6: fix the input of raid6 algorithm

2016-08-27 Thread liuzhengyuan
From: ZhengYuan Liu To test and choose the best algorithm for raid6, disks number and disks data must be offered. These input depend on page size and gfmul table at current time. It would cause the disk number less than 4 when the page size is more than 64KB.This patch would support arbitrarily p

[PATCH v06] uapi dm-log-userspace.h: use __u32, __s32, __u64 and __s64 from linux/types.h

2016-08-27 Thread Mikko Rapeli
Fixes userspace compilation errors like: linux/dm-log-userspace.h:416:2: error: unknown type name ‘uint64_t’ Signed-off-by: Mikko Rapeli Cc: Bart Van Assche --- include/uapi/linux/dm-log-userspace.h | 53 ++- 1 file changed, 27 insertions(+), 26 deletions(-) di

[PATCH] tpm: fix a race condition in tpm2_unseal_trusted()

2016-08-27 Thread Jarkko Sakkinen
Unseal and load operations should be done as an atomic operation. This commit introduces unlocked tpm_transmit() so that tpm2_unseal_trusted() can do the locking by itself. v2: Introduced an unlocked unseal operation instead of changing locking strategy in order to make less intrusive bug fix

Re: [PATCH v05 69/72] uapi rose.h: glibc netrose/rose.h header file compatibility fixes

2016-08-27 Thread Mikko Rapeli
On Fri, Aug 26, 2016 at 05:38:00PM +0200, walter harms wrote: > perhaps this not tested snipped would make sure that > you have included linux/rose.h ? > > #ifndef ROSE_KERNEL_H > #include > #endif > > #include Sorry, I did not quite get this. has conflicting definitions with glibc . The pat

[PATCH] memstick: r592: add missing declaration in r592.h

2016-08-27 Thread Baoyou Xie
We get 1 warning when build kernel with W=1: drivers/memstick/host/r592.c:50:13: warning: no previous prototype for 'memstick_debug_get_tpc_name' [-Wmissing-prototypes] in fact, function memstick_debug_get_tpc_name is implemented in r592.c and exported but need to be declared in header file. so

Re: [PATCH] fscrypto: fix to null-terminate encrypted filename in fname_encrypt

2016-08-27 Thread Chao Yu
Hi Ted, On 2016/8/28 13:13, Theodore Ts'o wrote: > On Sun, Aug 28, 2016 at 09:13:28AM +0800, Chao Yu wrote: >> From: Chao Yu >> >> This patch fixes to add null character at the end of encrypted filename >> in fname_encrypt, in order to avoid incorrectly traversing random data >> located after tar

Re: [PATCH 1/2] IB/hfi1: Fix a parameter of find_first_bit.

2016-08-27 Thread Leon Romanovsky
On Fri, Aug 26, 2016 at 03:34:48PM -0400, Doug Ledford wrote: > On 8/26/2016 3:29 PM, Leon Romanovsky wrote: > > On Fri, Aug 26, 2016 at 02:01:55PM -0400, Doug Ledford wrote: > >> On 8/26/2016 9:35 AM, Doug Ledford wrote: > >>> On 8/26/2016 12:49 AM, Christophe JAILLET wrote: > The 2nd paramet

Re: OOM detection regressions since 4.7

2016-08-27 Thread Arkadiusz Miskiewicz
On Thursday 25 of August 2016, Michal Hocko wrote: > On Tue 23-08-16 09:43:39, Michal Hocko wrote: > > On Mon 22-08-16 15:05:17, Andrew Morton wrote: > > > On Mon, 22 Aug 2016 15:42:28 +0200 Michal Hocko wrote: > > > > Of course, if Linus/Andrew doesn't like to take those compaction > > > > impro

[PATCH] arch: all: include: asm: bitops: Use bool instead of int for all bit test functions

2016-08-27 Thread chengang
From: Chen Gang Also use the same changing to asm-generic, and also use bool variable instead of int variable for mips, mn10300, parisc and tile related functions, and also avoid checkpatch.pl to report ERROR. Originally, except powerpc and xtensa, all another architectures intend to return 0 or

Re: [PATCH] fscrypto: fix to null-terminate encrypted filename in fname_encrypt

2016-08-27 Thread Theodore Ts'o
On Sun, Aug 28, 2016 at 09:13:28AM +0800, Chao Yu wrote: > From: Chao Yu > > This patch fixes to add null character at the end of encrypted filename > in fname_encrypt, in order to avoid incorrectly traversing random data > located after target filename. The call stack is as below: > > - f2fs_ad

Re: [PATCH 1/3] checkkconfigsymbols: port to python3

2016-08-27 Thread Valentin Rothberg
On Sat, Aug 27, 2016 at 7:53 PM, Greg KH wrote: > On Sat, Aug 27, 2016 at 05:25:29PM +0200, Valentin Rothberg wrote: >> Port the script to python3 and fix some pylint warnings. > > "and" in a changelog description usually means this should be 2 > different patches. > > Can't you fix up the pylint

Re: [RFC 1/1] drivers: i2c: omap: Add slave support

2016-08-27 Thread Wolfram Sang
> Making a mess of I2C controllers seems to be a popular hobby among > chip designers :P Well, I2C is simple, what could go wrong? :/ > A lot of the details (including the completely bizarre behaviour of > its innocuous-looking irq registers) would be quite non-trivial to > figure out without pu

[PATCH] fix:memory:omap-gpmc:mark symbols static where possible

2016-08-27 Thread Baoyou Xie
We get 1 warning when build kernel with W=1: drivers/memory/omap-gpmc.c:354:14: warning: no previous prototype for 'gpmc_clk_ticks_to_ns' [-Wmissing-prototypes] In fact, this function is only used in the file in which it is declared and don't need a declaration, but can be made static. so this pa

[PATCH] fix:overlay: add missing header dependencies

2016-08-27 Thread Baoyou Xie
We get 1 warning when build kernel with W=1: drivers/gpu/drm/nouveau/dispnv04/overlay.c:496:1: warning: no previous prototype for 'nouveau_overlay_init' [-Wmissing-prototypes] In fact, this function is declared in disp.h, so this patch add missing header dependencies Signed-off-by: Baoyou Xie -

drivers/gpio/gpio-mcp23s08.c:568:11: error: 'struct gpio_chip' has no member named 'of_gpio_n_cells'

2016-08-27 Thread kbuild test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: af56ff27eba54fceee5f5643e79bf6531f2e1739 commit: 2527ecc9195e9c66252af24c4689e8a67cd4ccb9 gpio: Fix OF build problem on UM date: 9 days ago config: um-allyesconfig (attached as .config) compiler: gcc-6 (De

Re: [PATCH v3 0/5] kexec_file: Add buffer hand-over for the next kernel

2016-08-27 Thread Mimi Zohar
On Thu, 2016-08-25 at 19:17 -0300, Thiago Jung Bauermann wrote: > Am Donnerstag, 25 August 2016, 14:12:43 schrieb Andrew Morton: > > I grabbed these two patch series. I also merged the "IMA: > > Demonstration code for kexec buffer passing." demonstration patch just > > to get things a bit of testi

[PATCH v2] fix:infiniband:hw:cxgb4:qp:mark symbols static where possible

2016-08-27 Thread Baoyou Xie
We get 1 warning when build kernel with W=1: drivers/infiniband/hw/cxgb4/qp.c:686:6: warning: no previous prototype for '_free_qp' [-Wmissing-prototypes] In fact, this function is only used in the file in which it is declared and don't need a declaration, but can be made static. so this patch mar

Re: [PATCH v7 8/8] drm/rockchip: Add dmc notifier in vop driver

2016-08-27 Thread kbuild test robot
Hi Lin, [auto build test ERROR on rockchip/for-next] [cannot apply to v4.8-rc3 next-20160825] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] [Suggest to use git(>=2.9.0) format-patch --base= (or --base=auto for convenience) to record what (publ

Re: checkkpatch (in)sanity ?

2016-08-27 Thread Levin, Alexander
On Sat, Aug 27, 2016 at 09:42:59PM -0400, Joe Perches wrote: > On Sat, 2016-08-27 at 21:06 -0400, Levin, Alexander wrote: > > On Sat, Aug 27, 2016 at 04:40:52PM -0400, Joe Perches wrote: > > > On Fri, Aug 26, 2016 at 01:26:35PM +0200, Greg KH wrote: > > > > On Fri, Aug 26, 2016 at 12:46:51AM -0400,

Re: MAINTAINERS without commits in the last 3 years

2016-08-27 Thread Joe Perches
On Sat, 2016-08-27 at 22:31 -0400, Mark F. Brown wrote: > I've continued to author patches for the Linux kernel, but they have been > under my work email address. This email works but I do not monitor it very > often anymore. So should the MAINTAINERS address be changed ?

Re: [PATCH] rtlwifi/rtl8192de: Fix print format string

2016-08-27 Thread Larry Finger
On 08/26/2016 10:12 PM, Oleg Drokin wrote: %ul was likely meant as %lu to print an unsigned long, not an unsigned with a letter l at the end. But in fact the value printed is u32 anyway, so just drop the l completely. This patch looks OK, thus Acked-by: Larry Finger Thanks, Larry Signed-

Re: checkkpatch (in)sanity ?

2016-08-27 Thread Joe Perches
On Sat, 2016-08-27 at 18:42 -0700, Joe Perches wrote: >      46 WARNING:LONG_LINE_COMMENT >  45 WARNING:LEADING_SPACE >  37 WARNING:LONG_LINE >  16 ERROR:GIT_COMMIT_ID >  11 WARNING:COMMIT_LOG_LONG_LINE >   5 WARNING:BRACES >   2 WARNING:BAD_SIGN_OFF >   2 WARNING:AVOID

Re: [PATCH][v6] PM / hibernate: Print the possible panic reason when resuming with inconsistent e820 map

2016-08-27 Thread Chen Yu
Hi, On Fri, Aug 26, 2016 at 09:56:54PM +0200, Pavel Machek wrote: > Hi! > > > > > > What's the progress of this patch? Looks already have experts review > > > > > it. > > > > > Why this patch didn't accept? > > > > This patch is a little overkilled, and I have saved another simpler > > > > versio

Re: [PATCH][RFC v4] timekeeping: ignore the bogus sleep time if pm_trace is enabled

2016-08-27 Thread Chen Yu
On Sat, Aug 27, 2016 at 03:08:56PM +0800, Xunlei Pang wrote: > On 2016/08/18 at 18:43, Chen Yu wrote: > > Previously we encountered some memory overflow issues due to > > the bogus sleep time brought by inconsistent rtc, which is > > triggered when pm_trace is enabled, please refer to: > > https://

Re: checkkpatch (in)sanity ?

2016-08-27 Thread Joe Perches
On Sat, 2016-08-27 at 21:06 -0400, Levin, Alexander wrote: > On Sat, Aug 27, 2016 at 04:40:52PM -0400, Joe Perches wrote: > > On Fri, Aug 26, 2016 at 01:26:35PM +0200, Greg KH wrote: > > > On Fri, Aug 26, 2016 at 12:46:51AM -0400, Levin, Alexander wrote: > > > > > > > > - Making checkpatch che

[PATCH] fscrypto: fix to null-terminate encrypted filename in fname_encrypt

2016-08-27 Thread Chao Yu
From: Chao Yu This patch fixes to add null character at the end of encrypted filename in fname_encrypt, in order to avoid incorrectly traversing random data located after target filename. The call stack is as below: - f2fs_add_link - __f2fs_add_link - fscrypt_setup_filename - fscrypt_fname

Re: checkkpatch (in)sanity ?

2016-08-27 Thread Levin, Alexander
On Sat, Aug 27, 2016 at 04:40:52PM -0400, Joe Perches wrote: > On Fri, Aug 26, 2016 at 01:26:35PM +0200, Greg KH wrote: > > On Fri, Aug 26, 2016 at 12:46:51AM -0400, Levin, Alexander wrote: > >>- Making checkpatch check for (some) of the stable kernel rules > >>(and possibly recommend addin

[PATCH] staging: rts5208: Add two blank lines in comments.

2016-08-27 Thread MingChia Chung
This patch fixes a minor checkpatch warnings: "WARNING: Block comments use a trailing */ on a separate line" Signed-off-by: Ming-Chia Chung --- drivers/staging/rts5208/rtsx.c | 42 -- 1 file changed, 28 insertions(+), 14 deletions(-) diff --git a/drivers

LOANDS AFORDABLE 3% ANNUALLY

2016-08-27 Thread BOTHA
LOANS UP TO R150 000! - Blacklisted welcome to apply - Self-employed welcome to apply - Must be at least 18 years of age - Must have a valid ID document

Re: [PATCH 1/5] IA64-IRQ: Use kmalloc_array() in sn_irq_lh_init()

2016-08-27 Thread Joe Perches
On Sat, 2016-08-27 at 09:02 +0200, SF Markus Elfring wrote: > > If you _really wanted to clear up this code and make it more > > robust/better, it'd probably be nicer to convert the > > struct list_head **sn_irq_lh to a single struct list_head * > > That would be less data space overall given the a

Re: [RFC 1/1] drivers: i2c: omap: Add slave support

2016-08-27 Thread Matthijs van Duin
On 27 August 2016 at 19:22, Wolfram Sang wrote: > Uh, that sounds like bad HW... Making a mess of I2C controllers seems to be a popular hobby among chip designers :P ( I also really like how the RPi handles clock stretching... *cough* ) > While it surely is nice to have super detailed informati

Re: [PATCH RFC v3] x86,mm,sched: make lazy TLB mode even lazier

2016-08-27 Thread Linus Torvalds
Yeah, with those small fixes from Ingo, I definitely don't think this looks hacky at all. This all seems to be exactly what we should always have done. The only remaining comment is that I'd make that lazy_tlb_can_skip_flush() function just use a switch table for the tlbstate comparisons rather th

Re: Plan to support Rockchip VPU in DRM, is it a good idea

2016-08-27 Thread Theodore Kilgore
On Fri, 26 Aug 2016, Randy Li wrote: On 08/26/2016 05:34 PM, Hans Verkuil wrote: Hi Randi, On 08/26/2016 04:13 AM, Randy Li wrote: Hello, We always use some kind of hack work to make our Video Process Unit(Multi-format Video Encoder/Decoder) work in kernel. From a customize driver(vpu

Re: [PATCH] ravb: avoid unused function warnings

2016-08-27 Thread Sergei Shtylyov
Hello. On 08/27/2016 01:48 AM, Sergei Shtylyov wrote: When CONFIG_PM_SLEEP is disabled, we get a couple of harmless warnings: drivers/net/ethernet/renesas/ravb_main.c:2117:12: error: 'ravb_resume' defined but not used [-Werror=unused-function] drivers/net/ethernet/renesas/ravb_main.c:2104:12:

Re: [RFC v2 09/10] landlock: Handle cgroups (program types)

2016-08-27 Thread Mickaël Salaün
On 27/08/2016 22:56, Alexei Starovoitov wrote: > On Sat, Aug 27, 2016 at 09:55:01PM +0200, Mickaël Salaün wrote: >> >> On 27/08/2016 20:19, Alexei Starovoitov wrote: >>> On Sat, Aug 27, 2016 at 04:34:55PM +0200, Mickaël Salaün wrote: On 27/08/2016 01:05, Alexei Starovoitov wrote: > O

Re: [RFC v2 09/10] landlock: Handle cgroups (performance)

2016-08-27 Thread Mickaël Salaün
On 27/08/2016 22:43, Alexei Starovoitov wrote: > On Sat, Aug 27, 2016 at 09:35:14PM +0200, Mickaël Salaün wrote: >> On 27/08/2016 20:06, Alexei Starovoitov wrote: >>> On Sat, Aug 27, 2016 at 04:06:38PM +0200, Mickaël Salaün wrote: As said above, Landlock will not run an eBPF programs when not

Re: [PATCH 3.10 099/180] fix d_walk()/non-delayed __d_free() race

2016-08-27 Thread Willy Tarreau
On Sat, Aug 27, 2016 at 12:38:38PM +0100, Ben Hutchings wrote: > On Sat, 2016-08-27 at 11:31 +0200, Willy Tarreau wrote: > > Greg, Jiri, > > > > I checked Jari's explanation below and found that v3.14.77 and v3.12.62 > > are missing the same fix as 3.10. In fact Al's original commit 3d56c25 > > ("

Re: [RFC v2 09/10] landlock: Handle cgroups (program types)

2016-08-27 Thread Alexei Starovoitov
On Sat, Aug 27, 2016 at 09:55:01PM +0200, Mickaël Salaün wrote: > > On 27/08/2016 20:19, Alexei Starovoitov wrote: > > On Sat, Aug 27, 2016 at 04:34:55PM +0200, Mickaël Salaün wrote: > >> > >> On 27/08/2016 01:05, Alexei Starovoitov wrote: > >>> On Fri, Aug 26, 2016 at 05:10:40PM +0200, Mickaël Sa

Re: [RFC v2 09/10] landlock: Handle cgroups (performance)

2016-08-27 Thread Alexei Starovoitov
On Sat, Aug 27, 2016 at 09:35:14PM +0200, Mickaël Salaün wrote: > > On 27/08/2016 20:06, Alexei Starovoitov wrote: > > On Sat, Aug 27, 2016 at 04:06:38PM +0200, Mickaël Salaün wrote: > >> > >> On 27/08/2016 01:05, Alexei Starovoitov wrote: > >>> On Fri, Aug 26, 2016 at 05:10:40PM +0200, Mickaël Sa

checkkpatch (in)sanity ?

2016-08-27 Thread Joe Perches
On Fri, Aug 26, 2016 at 01:26:35PM +0200, Greg KH wrote: > On Fri, Aug 26, 2016 at 12:46:51AM -0400, Levin, Alexander wrote: >>- Making checkpatch check for (some) of the stable kernel rules >>(and possibly recommend adding the stable@ tag in certain cases?). >>  - Depends on: making ch

Re: [RFC v2 09/10] landlock: Handle cgroups (program types)

2016-08-27 Thread Mickaël Salaün
On 27/08/2016 20:19, Alexei Starovoitov wrote: > On Sat, Aug 27, 2016 at 04:34:55PM +0200, Mickaël Salaün wrote: >> >> On 27/08/2016 01:05, Alexei Starovoitov wrote: >>> On Fri, Aug 26, 2016 at 05:10:40PM +0200, Mickaël Salaün wrote: >>> > As far as safety and type checking that bpf programs h

Re: [RFC v2 09/10] landlock: Handle cgroups (performance)

2016-08-27 Thread Mickaël Salaün
On 27/08/2016 20:06, Alexei Starovoitov wrote: > On Sat, Aug 27, 2016 at 04:06:38PM +0200, Mickaël Salaün wrote: >> >> On 27/08/2016 01:05, Alexei Starovoitov wrote: >>> On Fri, Aug 26, 2016 at 05:10:40PM +0200, Mickaël Salaün wrote: > > - I don't think such 'for' loop can scale. The

constification and cocci / kernel build test robot ?

2016-08-27 Thread Joe Perches
On Sat, 2016-08-27 at 20:59 +0200, Julia Lawall wrote: >  Make sure (of/i2c/platform)_device_id tables are NULL terminated > Generated by: scripts/coccinelle/misc/of_table.cocci Along the same lines, I submitted a manually generated patch to add const to some of these structs. https://lkml.org/lk

Re: [PATCH 5/8] cris-cryptocop: Move an assignment for the variable "nooutpages" in cryptocop_ioctl_process()

2016-08-27 Thread Julia Lawall
On Fri, 26 Aug 2016, SF Markus Elfring wrote: > From: Markus Elfring > Date: Fri, 26 Aug 2016 13:38:30 +0200 > > Move the assignment for the local variable "nooutpages" behind > the source code for memory allocations by this function. > > Signed-off-by: Markus Elfring > --- > arch/cris/arch-v

Re: [PATCH 8/8] cris-cryptocop: Apply another recommendation from "checkpatch.pl"

2016-08-27 Thread Julia Lawall
On Fri, 26 Aug 2016, SF Markus Elfring wrote: > From: Markus Elfring > Date: Fri, 26 Aug 2016 14:23:06 +0200 > > The script "checkpatch.pl" can point out that assignments should usually > not be performed within condition checks. > Thus move the assignments for a local variable to separate stat

[PATCH] clk: sunxi-ng: Fix wrong reset register offsets

2016-08-27 Thread jorik
From: Jorik Jonker The reset register offsets for UART*, I2C* and SCR were off by a few bytes. Signed-off-by: Jorik Jonker --- drivers/clk/sunxi-ng/ccu-sun8i-h3.c | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/drivers/clk/sunxi-ng/ccu-sun8i-h3.c b/driver

[PATCH] thermal: fix of_table.cocci warnings

2016-08-27 Thread Julia Lawall
Make sure (of/i2c/platform)_device_id tables are NULL terminated Generated by: scripts/coccinelle/misc/of_table.cocci CC: Yegnesh S Iyer Signed-off-by: Julia Lawall Signed-off-by: Fengguang Wu --- Please take the patch only if it's a positive warning. Thanks! intel_bxt_pmic_thermal.c |1

Re: [RFC v2 09/10] landlock: Handle cgroups (netfilter match)

2016-08-27 Thread Alexei Starovoitov
On Sat, Aug 27, 2016 at 04:19:05PM +0200, Mickaël Salaün wrote: > > On 27/08/2016 01:05, Alexei Starovoitov wrote: > > On Fri, Aug 26, 2016 at 05:10:40PM +0200, Mickaël Salaün wrote: > >> To sum up, there is four related patchsets: > >> * "Landlock LSM: Unprivileged sandboxing" (this series) > >>

Re: [RFC][PATCH 0/3] locking/mutex: Rewrite basic mutex

2016-08-27 Thread Ingo Molnar
* Peter Zijlstra wrote: > Its because the mutex wasn't quite exclusive enough :-) If you let in > multiple > owner, like with that race you found, you get big gains in throughput ... Btw., do we know which mutex that was? That it didn't crash with a full AIM run suggests that whatever it is

Re: [PATCH] fix:infiniband:hw:cxgb4:qp:mark symbols static where possible

2016-08-27 Thread Yuval Shaia
One minor comment inline. Besides that: Reviewed-by: Yuval Shaia On Sun, Aug 28, 2016 at 12:00:18AM +0800, Baoyou Xie wrote: > We get 1 warning when biuld kernel with W=1: s/biuld/build > drivers/infiniband/hw/cxgb4/qp.c:686:6: warning: no previous prototype for > '_free_qp' [-Wmissing-prototy

Re: [RFC v2 09/10] landlock: Handle cgroups (program types)

2016-08-27 Thread Alexei Starovoitov
On Sat, Aug 27, 2016 at 04:34:55PM +0200, Mickaël Salaün wrote: > > On 27/08/2016 01:05, Alexei Starovoitov wrote: > > On Fri, Aug 26, 2016 at 05:10:40PM +0200, Mickaël Salaün wrote: > > > >>> As far as safety and type checking that bpf programs has to do, > >>> I like the approach of patch 06/10:

Re: [RFC v2 09/10] landlock: Handle cgroups

2016-08-27 Thread Alexei Starovoitov
On Sat, Aug 27, 2016 at 12:30:36AM -0700, Andy Lutomirski wrote: > > cgroup is the common way to group multiple tasks. > > Without cgroup only parent<->child relationship will be possible, > > which will limit usability of such lsm to a master task that controls > > its children. Such api restricti

Re: [RFC v2 09/10] landlock: Handle cgroups (performance)

2016-08-27 Thread Alexei Starovoitov
On Sat, Aug 27, 2016 at 04:06:38PM +0200, Mickaël Salaün wrote: > > On 27/08/2016 01:05, Alexei Starovoitov wrote: > > On Fri, Aug 26, 2016 at 05:10:40PM +0200, Mickaël Salaün wrote: > >> > >>> > >>> - I don't think such 'for' loop can scale. The solution needs to work > >>> with thousands of cont

Re: [PATCHv3 3/6] x86/arch_prctl/vdso: add ARCH_MAP_VDSO_*

2016-08-27 Thread kbuild test robot
ase= (or --base=auto for convenience) to record what (public, well-known) commit your patch series was built on] [Check https://git-scm.com/docs/git-format-patch for more information] url: https://github.com/0day-ci/linux/commits/Dmitry-Safonov/x86-32-bit-compatible-C-R-on-x86_64/20160827-011

Re: [PATCH] fix:memory:of_memory:add missing header dependencies

2016-08-27 Thread Greg KH
On Sun, Aug 28, 2016 at 01:31:28AM +0800, Baoyou Xie wrote: > We get 2 warnings when biuld kernel with W=1: Why would you do that? > drivers/memory/of_memory.c:30:30: warning: no previous prototype for > 'of_get_min_tck' [-Wmissing-prototypes] > drivers/memory/of_memory.c:106:30: warning: no pre

Re: [PATCH 1/3] checkkconfigsymbols: port to python3

2016-08-27 Thread Greg KH
On Sat, Aug 27, 2016 at 05:25:29PM +0200, Valentin Rothberg wrote: > Port the script to python3 and fix some pylint warnings. "and" in a changelog description usually means this should be 2 different patches. Can't you fix up the pylint warnings in a separate patch? thanks, greg k-h

[GIT PULL] KVM fixes for Linux 4.8-rc4

2016-08-27 Thread Paolo Bonzini
Linus, The following changes since commit 184ca823481c99dadd7d946e5afd4bb921eab30d: Merge git://git.kernel.org/pub/scm/linux/kernel/git/davem/net (2016-08-17 17:26:58 -0700) are available in the git repository at: git://git.kernel.org/pub/scm/virt/kvm/kvm.git tags/for-linus for you to fet

[PATCH] fix:memory:of_memory:add missing header dependencies

2016-08-27 Thread Baoyou Xie
We get 2 warnings when biuld kernel with W=1: drivers/memory/of_memory.c:30:30: warning: no previous prototype for 'of_get_min_tck' [-Wmissing-prototypes] drivers/memory/of_memory.c:106:30: warning: no previous prototype for 'of_get_ddr_timings' [-Wmissing-prototypes] In fact, these functions ar

Re: [RFC 1/1] drivers: i2c: omap: Add slave support

2016-08-27 Thread Wolfram Sang
> Greetings, unfortunate souls trying to use the omap-i2c peripheral in > slave mode! :-) Uh, that sounds like bad HW... > I recently posted some stuff about exactly that topic on TI's E2E > forum, you may want to read this warning: > http://e2e.ti.com/support/arm/sitara_arm/f/791/p/514961/19558

[PATCH] fix:mailbox:bcm-pdc-mailbox:mark symbols static where possible

2016-08-27 Thread Baoyou Xie
We get 2 warnings when biuld kernel with W=1: drivers/mailbox/bcm-pdc-mailbox.c:472:6: warning: no previous prototype for 'pdc_setup_debugfs' [-Wmissing-prototypes] drivers/mailbox/bcm-pdc-mailbox.c:488:6: warning: no previous prototype for 'pdc_free_debugfs' [-Wmissing-prototypes] In fact, thes

[PATCH] scsi: move function declarationes to scsi_priv.h

2016-08-27 Thread Baoyou Xie
We get 2 warnings about global functions without a declaration in the scsi driver when building with W=1: drivers/scsi/scsi_lib.c:467:6: warning: no previous prototype for 'scsi_requeue_run_queue' [-Wmissing-prototypes] drivers/scsi/scsi_lib.c:2609:6: warning: no previous prototype for 'scsi_evt_

Re: [PATCH] bridge: Fix format string for %ul

2016-08-27 Thread Oleg Drokin
On Aug 27, 2016, at 12:18 PM, Sergei Shtylyov wrote: > Hello. > > On 8/27/2016 6:58 PM, Oleg Drokin wrote: > %ul would print an unsigned value and a letter l, likely it was %lu that was meant to print the long int, but in reality the values printed there are just regular signed >

Re: [PATCH v2 3/3] ARM: dts: Set the clock rate for DREX block 800Mhz on exynos5422-odroidxu3

2016-08-27 Thread Krzysztof Kozlowski
On Thu, Aug 25, 2016 at 03:57:18PM +0900, Chanwoo Choi wrote: > This patch sets the clock rate for DREX (DRAM Express) block > on exynos5422-odroidxu3 board. In the exynos5422 TRM, > DRAM clocks use BPLL clock and CMU_CDREX generates > the 800MHz DRAM clock. > >From the commit message I don't get

[PATCH] fix:gpio: mark symbols static where possible

2016-08-27 Thread Baoyou Xie
We get 1 warning about global functions without a declaration in the ath9k gpio driver when building with W=1: drivers/net/wireless/ath/ath9k/gpio.c:25:6: warning: no previous prototype for 'ath_fill_led_pin' [-Wmissing-prototypes] In fact, this function is only used in the file in which it is de

[PATCH 2/2] drivers/perf: arm_pmu: Fix NULL pointer dereference during probe

2016-08-27 Thread Stefan Wahren
Patch 7f1d642fbb5c ("drivers/perf: arm-pmu: Fix handling of SPI lacking interrupt-affinity property") unintended also fixes perf_event support for bcm2835 which doesn't have PMU interrupts. Unfortunately this change introduce a NULL pointer dereference on bcm2835, because irq_is_percpu always expec

[PATCH 1/2] drivers/perf: arm_pmu: Fix leak in error path

2016-08-27 Thread Stefan Wahren
In case of a IRQ type mismatch in of_pmu_irq_cfg() the device node for interrupt affinity isn't freed. So fix this issue by calling of_node_put(). Signed-off-by: Stefan Wahren Fixes: fa8ad7889d83 ("arm: perf: factor arm_pmu core out to drivers") --- drivers/perf/arm_pmu.c |1 + 1 file change

Re: [PATCH] bridge: Fix format string for %ul

2016-08-27 Thread Sergei Shtylyov
Hello. On 8/27/2016 6:58 PM, Oleg Drokin wrote: %ul would print an unsigned value and a letter l, likely it was %lu that was meant to print the long int, but in reality the values printed there are just regular signed Signed? Then you need probably "%d" or "%i"… They are signed in the str

[PATCH] fix:rtl8xxxu_core: mark symbols static where possible

2016-08-27 Thread Baoyou Xie
We get 1 warning about global functions without a declaration in the rtl8xxxu rtl8xxxu_core.c when building with W=1: drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu_core.c:898:1: warning: no previous prototype for 'rtl8xxxu_gen1_h2c_cmd' [-Wmissing-prototypes] In fact, this function is only used

Re: KVM-S390: Less function calls in kvm_s390_import_bp_data() after error detection

2016-08-27 Thread SF Markus Elfring
>> @@ -273,10 +273,12 @@ int kvm_s390_import_bp_data(struct kvm_vcpu *vcpu, >> vcpu->arch.guestdbg.nr_hw_wp = nr_wp; >> vcpu->arch.guestdbg.hw_wp_info = wp_info; >> return 0; >> -error: >> -kfree(bp_data); >> -kfree(wp_info); >> +free_bp_info: >> kfree(bp_info); >> +free

[PATCH v3] console: Add persistent scrollback buffers for all VGA consoles

2016-08-27 Thread Manuel Schölling
Add a scrollback buffers for each VGA console. The benefit is that the scrollback history is not flushed when switching between consoles but is persistent. The buffers are allocated on demand when a new console is opened. It always annoys me when I switch back to a console and I can just read half

[PATCH] fix:infiniband:hw:cxgb4:qp:mark symbols static where possible

2016-08-27 Thread Baoyou Xie
We get 1 warning when biuld kernel with W=1: drivers/infiniband/hw/cxgb4/qp.c:686:6: warning: no previous prototype for '_free_qp' [-Wmissing-prototypes] In fact, this function is only used in the file in which it is declared and don't need a declaration, but can be made static. so this patch mar

Re: [PATCH] bridge: Fix format string for %ul

2016-08-27 Thread Oleg Drokin
On Aug 27, 2016, at 11:13 AM, Sergei Shtylyov wrote: > Hello. > > On 8/27/2016 6:10 AM, Oleg Drokin wrote: > >> %ul would print an unsigned value and a letter l, >> likely it was %lu that was meant to print the long int, >> but in reality the values printed there are just regular signed > >

[v4.7.2: Oops] When loading IT87 HARDWARE MONITORING DRIVER

2016-08-27 Thread Alexander Kapshuk
I get an Oops when loading it87.ko on a 4.7.2 kernel. See below for details: (1). ver_linux If some fields are empty or look unusual you may have an old version. Compare to the current minimal requirements in Documentation/Changes. Linux box1 4.7.2-vanilla #7 SMP Sat Aug 27 09:20:18 EEST 2016 i68

[PATCH] fix:pmcraid: mark symbols static where possible

2016-08-27 Thread Baoyou Xie
We get 4 warnings about global functions without a declaration in the scsi pmcraid driver when building with W=1: drivers/scsi/pmcraid.c:309:6: warning: no previous prototype for 'pmcraid_init_cmdblk' [-Wmissing-prototypes] drivers/scsi/pmcraid.c:404:6: warning: no previous prototype for 'pmcraid

re

2016-08-27 Thread Ali Saeed
Did you get my message?

Re: [PATCH] smc91x: remove ARM hack for unaligned 16-bit writes

2016-08-27 Thread Russell King - ARM Linux
On Sat, Aug 27, 2016 at 05:30:42PM +0200, Robert Jarzmik wrote: > Hi Russell, > > With your patch : > - lubbock, mainstone and zylonite boards are working correctly >=> ie. all my boards are working correctly > > - the message "ifconfig: SIOCSIFHWADDR: Device or resource busy" is still > t

Re: [PATCH] smc91x: remove ARM hack for unaligned 16-bit writes

2016-08-27 Thread Robert Jarzmik
Russell King - ARM Linux writes: > On Thu, Aug 25, 2016 at 08:02:35PM +0200, Robert Jarzmik wrote: >> Arnd Bergmann writes: >> >> > On Thursday, August 25, 2016 4:43:04 PM CEST Arnd Bergmann wrote: >> >> drivers/net/ethernet/smsc/smc91x.h | 50 >> >> +++--- >> >

[PATCH 2/2] mm/bootmem.c: substitute kzalloc_node() for kzalloc()

2016-08-27 Thread zijun_hu
From: zijun_hu in ___alloc_bootmem_node_nopanic(), substitute kzalloc_node() for kzalloc() in order to allocate memory within given node preferentially when slab is available free_all_bootmem_core() is optimized to make the first two parameters of __free_pages_bootmem() looks consistent with eac

Re: [PATCH v3 6/8] hwrng: amd: Replace global variable with private struct

2016-08-27 Thread Jason Cooper
On Sat, Aug 27, 2016 at 02:43:31PM +, Jason Cooper wrote: > Hi Corentin, > > On Fri, Aug 26, 2016 at 01:11:34PM +0200, LABBE Corentin wrote: > > Instead of having two global variable, it's better to use a > > private struct. This will permit to remove amd_pdev variable > > > > Signed-off-by:

[PATCH 1/2] mm/nobootmem.c: make CONFIG_NO_BOOTMEM depend on CONFIG_HAVE_MEMBLOCK

2016-08-27 Thread zijun_hu
From: zijun_hu this patch fixes the following bugs: - no bootmem is implemented by memblock currently, but config option CONFIG_NO_BOOTMEM doesn't depend on CONFIG_HAVE_MEMBLOCK - the same ARCH_LOW_ADDRESS_LIMIT statements are duplicated between header and relevant source - don't ensu

[PATCH 2/3] checkkconfigsymbols.py: consitent symbol terminology

2016-08-27 Thread Valentin Rothberg
Make the term 'symbol' consitent in the entire scripts. Signed-off-by: Valentin Rothberg --- scripts/checkkconfigsymbols.py | 134 - 1 file changed, 67 insertions(+), 67 deletions(-) diff --git a/scripts/checkkconfigsymbols.py b/scripts/checkkconfigsymbol

[PATCH 3/3] checkkconfigsymbols: use arglist instead of cmd string

2016-08-27 Thread Valentin Rothberg
Splitting a command string could lead to unintended arguments. Use an argument list in the execute() function instead. Signed-off-by: Valentin Rothberg --- scripts/checkkconfigsymbols.py | 29 ++--- 1 file changed, 18 insertions(+), 11 deletions(-) diff --git a/scripts/

[PATCH 1/3] checkkconfigsymbols: port to python3

2016-08-27 Thread Valentin Rothberg
Port the script to python3 and fix some pylint warnings. Signed-off-by: Valentin Rothberg --- scripts/checkkconfigsymbols.py | 280 +++-- 1 file changed, 156 insertions(+), 124 deletions(-) diff --git a/scripts/checkkconfigsymbols.py b/scripts/checkkconfigsym

Re: [RFC v2 00/10] Landlock LSM: Unprivileged sandboxing (cgroup delegation)

2016-08-27 Thread Mickaël Salaün
Cc Tejun and the cgroups ML. On 27/08/2016 17:10, Mickaël Salaün wrote: > On 27/08/2016 09:40, Andy Lutomirski wrote: >> On Thu, Aug 25, 2016 at 3:32 AM, Mickaël Salaün wrote: >>> >>> # Sandbox example with conditional access control depending on cgroup >>> >>> $ mkdir /sys/fs/cgroup/sandboxed

Re: [RFC v4 14/22] clockevents: clockevents_program_event(): turn clc into unsigned long

2016-08-27 Thread Nicolai Stange
Thomas Gleixner writes: > On Tue, 23 Aug 2016, Nicolai Stange wrote: > >> Right now, being of type unsigned long long, the clc local variable in >> clockevents_program_event() is unnecessarily wide as it gets cast to >> unsigned long anyway. > > Congrats! You just broke 32bit systems. When delta

Re: [RFC v4 13/22] clockevents: check a programmed delta's bounds in terms of cycles

2016-08-27 Thread Nicolai Stange
Nicolai Stange writes: > @@ -332,10 +337,10 @@ int clockevents_program_event(struct clock_event_device > *dev, ktime_t expires, > if (delta <= 0) > return force ? clockevents_program_min_delta(dev) : -ETIME; > > - delta = min(delta, (int64_t) dev->max_delta_ns); > -

Re: Random abnormal high CPU sys usage related to timer

2016-08-27 Thread Mac Lin
Hi Vegard, Thanks for the prompt response. The commit is introduced since 4.6, but the issue can be reproduced at 3.10 (earliest I have ever test). And testing on buildroot+4.7 with the commit reverted, the issue still happen. In fact, I did a test that ran a script that keep increase a counter fo

Re: [PATCH] bridge: Fix format string for %ul

2016-08-27 Thread Sergei Shtylyov
Hello. On 8/27/2016 6:10 AM, Oleg Drokin wrote: %ul would print an unsigned value and a letter l, likely it was %lu that was meant to print the long int, but in reality the values printed there are just regular signed Signed? Then you need probably "%d" or "%i"... ints, so just dropping

Re: [RFC v2 00/10] Landlock LSM: Unprivileged sandboxing

2016-08-27 Thread Mickaël Salaün
On 27/08/2016 09:40, Andy Lutomirski wrote: > On Thu, Aug 25, 2016 at 3:32 AM, Mickaël Salaün wrote: >> Hi, >> >> This series is a proof of concept to fill some missing part of seccomp as the >> ability to check syscall argument pointers or creating more dynamic security >> policies. The goal of

Re: [PATCH 2/2] arm64: dts: rockchip: add eMMC's power domain support for rk3399

2016-08-27 Thread Shawn Lin
On 2016/8/27 21:41, Ziyuan Xu wrote: Control power domain for eMMC via genpd to reduce power consumption. Signed-off-by: Elaine Zhang Signed-off-by: Ziyuan Xu It looks nice to me. But this should be merged after applying that[0] as your patch will break bind/unbind test for sdhci-of-arasan

Re: [f2fs-dev] [PATCH] f2fs: fix to set superblock dirty correctly

2016-08-27 Thread Chao Yu
Hi Yunlei, On 2016/8/27 9:54, heyunlei wrote: > Hi all, > > On 2016/8/27 9:01, Jaegeuk Kim wrote: >> On Fri, Aug 26, 2016 at 10:20:18PM +0800, Chao Yu wrote: >>> From: Chao Yu >>> >>> tests/generic/251 of fstest suit complains us with below message: >>> >>> [ cut here ] >

Re: [PATCH] f2fs: fix to set superblock dirty correctly

2016-08-27 Thread Chao Yu
On 2016/8/27 9:01, Jaegeuk Kim wrote: > On Fri, Aug 26, 2016 at 10:20:18PM +0800, Chao Yu wrote: >> From: Chao Yu >> >> tests/generic/251 of fstest suit complains us with below message: >> >> [ cut here ] >> invalid opcode: [#1] PREEMPT SMP >> CPU: 2 PID: 7698 Comm: fs

  1   2   >