Re: [PATCH] jffs2: Fix use-after-free bug in jffs2_iget()'s error handling path

2017-09-21 Thread Richard Weinberger
On Thu, Sep 21, 2017 at 6:00 PM, Jake Daryll Obina wrote: > If jffs2_iget() fails for a newly-allocated inode, jffs2_do_clear_inode() > can get called twice in the error handling path, the first call in > jffs2_iget() itself and the second through iget_failed(). This can result > to a use-after-fr

Re: Traceback due to 'powerpc/mm: Fix kernel RAM protection...' when running ppc image in qemu

2017-09-21 Thread Christophe LEROY
Le 20/09/2017 à 05:45, Guenter Roeck a écrit : On 09/19/2017 08:05 PM, Michael Ellerman wrote: Guenter Roeck writes: Hi, I see a the following traceback when running an SMP image based on 85xx/mpc85xx_cds_defconfig in qemu. [ cut here ] WARNING: CPU: 0 PID: 1 at ke

Re: [mainline][DLPAR][Oops] OF: ERROR: Bad of_node_put() on /cpus

2017-09-21 Thread Tyrel Datwyler
On 09/21/2017 02:57 AM, Michael Ellerman wrote: > Tyrel Datwyler writes: >> On 09/20/2017 04:39 AM, Michael Ellerman wrote: >>> Rob Herring writes: >>> >>> Testing a fix, will report back. >> >> So, that patch slipped past me. Not only is the parent reference not ours to >> drop, but >> when

Re: [PATCH v2 1/6] ACPI/PPTT: Add Processor Properties Topology Table parsing

2017-09-21 Thread Jeremy Linton
Hi, On 09/20/2017 02:15 AM, Xiongfeng Wang wrote: Hi Jeremy, On 2017/9/20 2:47, Jeremy Linton wrote: ACPI 6.2 adds a new table, which describes how processing units are related to each other in tree like fashion. Caches are also sprinkled throughout the tree and describe the properties of the

[PATCH RT v2] locking/rtmutex: don't drop the wait_lock twice

2017-09-21 Thread Sebastian Andrzej Siewior
Since the futex rework, __rt_mutex_start_proxy_lock() does no longer acquire the wait_lock so it must not drop it. Otherwise the lock is not only unlocked twice but also the preemption counter is underflown. It is okay to remove that line because this function does not disable interrupts nor does

[PATCH v3 2/3] x86/fpu: tighten validation of user-supplied xstate_header

2017-09-21 Thread Eric Biggers
From: Eric Biggers Move validation of user-supplied xstate_headers into a helper function and call it from both the ptrace and sigreturn syscall paths. The new function also considers it to be an error if *any* reserved bits are set, whereas before we were just clearing most of them. This shoul

[PATCH v3 1/3] x86/fpu: don't let userspace set bogus xcomp_bv

2017-09-21 Thread Eric Biggers
From: Eric Biggers On x86, userspace can use the ptrace() or rt_sigreturn() system calls to set a task's extended state (xstate) or "FPU" registers. ptrace() can set them for another task using the PTRACE_SETREGSET request with NT_X86_XSTATE, while rt_sigreturn() can set them for the current tas

[PATCH v3 0/3] x86/fpu: prevent leaking FPU registers via invalid FPU state

2017-09-21 Thread Eric Biggers
From: Eric Biggers This series fixes the bug found by syzkaller where the ptrace syscall can be used to set invalid bits in a task's FPU state. I also found that an equivalent bug was reachable using the sigreturn syscall, so the first patch fixes the bug in both cases. The other two patches st

[PATCH v3 3/3] x86/fpu: reinitialize FPU registers if restoring FPU state fails

2017-09-21 Thread Eric Biggers
From: Eric Biggers Userspace can change the FPU state of a task using the ptrace() or rt_sigreturn() system calls. Because reserved bits in the FPU state can cause the XRSTOR instruction to fail, the kernel has to carefully validate that no reserved bits or other invalid values are being set. U

Re: [PATCH v4 0/4] Add cross-compilation support to eBPF samples

2017-09-21 Thread David Miller
From: Joel Fernandes Date: Wed, 20 Sep 2017 09:04:32 -0700 > These patches fix issues seen when cross-compiling eBPF samples on arm64. > Compared to [1], I dropped the controversial inline-asm patch and exploring > other options to fix it. However these patches are a step in the right > direction

Re: [Xen-devel] [PATCH] xen: support 52 bit physical addresses in pv guests

2017-09-21 Thread Boris Ostrovsky
On 09/21/2017 12:16 PM, Andrew Cooper wrote: On 21/09/17 17:00, Boris Ostrovsky wrote: Signed-off-by: Juergen Gross ---    arch/x86/include/asm/xen/page.h | 11 ++-    arch/x86/xen/mmu_pv.c   |  4 ++--    2 files changed, 12 insertions(+), 3 deletions(-) diff --git a/ar

Re: usb/storage/uas: slab-out-of-bounds in uas_probe

2017-09-21 Thread Alan Stern
On Thu, 21 Sep 2017, Andrey Konovalov wrote: > On Thu, Sep 21, 2017 at 6:10 PM, Greg Kroah-Hartman > wrote: > > On Thu, Sep 21, 2017 at 05:39:05PM +0200, Andrey Konovalov wrote: > >> Hi! > >> > >> I've got the following report while fuzzing the kernel with syzkaller. > >> > >> On commit ebb2c2437

[PATCH] vfio/pci: Virtualize Maximum Read Request Size

2017-09-21 Thread Alex Williamson
MRRS defines the maximum read request size a device is allowed to make. Drivers will often increase this to allow more data transfer with a single request. Completions to this request are bound by the MPS setting for the bus. Aside from device quirks (none known), it doesn't seem to make sense t

Re: [PATCH v2 03/22] fpga: bridge: remove OF dependency for fpga-bridge

2017-09-21 Thread Moritz Fischer
Hi, On Mon, Jun 26, 2017 at 09:51:59AM +0800, Wu Hao wrote: > This patch removes OF dependency of fpga-bridge, it allows drivers > to use fpga-bridge class without device tree support. > > Signed-off-by: Wu Hao > --- > drivers/fpga/Kconfig | 1 - > 1 file changed, 1 deletion(-) > > diff --git

[PATCH] staging: vc04_services: Remove typedef struct

2017-09-21 Thread Harsha Sharma
Remove typedef from struct as linux-kernel coding style tends to avoid using typedefs Done using following coccinelle semantic patch @r1@ type T; @@ typedef struct { ... } T; @script:python c1@ T2; T << r1.T; @@ if T[-2:] =="_t" or T[-2:] == "_T": coccinelle.T2 = T[:-2]; else: c

[PATCH RT] rt/locking: allow recursive local_trylock()

2017-09-21 Thread Sebastian Andrzej Siewior
required for networking patch which does recursive try-lock. While at it, add the !RT version of it because it did not yet exist. Cc: rt-sta...@vger.kernel.org Signed-off-by: Sebastian Andrzej Siewior --- include/linux/locallock.h | 9 + 1 file changed, 9 insertions(+) diff --git a/incl

Re: [PATCH v3 1/6] serial: imx: remove CTSC and CTS handling from imx_disable_dma

2017-09-21 Thread Martyn Welch
On Thu, Sep 21, 2017 at 08:20:17PM +0200, Uwe Kleine-König wrote: > On Thu, Sep 21, 2017 at 05:18:12PM +0100, Martyn Welch wrote: > > From: Nandor Han > > > > The CTSC and CTS bits affect operation of the CTS/RTS hardware flow > > control signal (depending on whether the device is in DCE or DTE m

Re: Read-only `slaves` with shared subtrees?

2017-09-21 Thread Ram Pai
On Wed, Sep 20, 2017 at 08:00:57PM -0700, Dawid Ciezarkiewicz wrote: > On Wed, Sep 20, 2017 at 5:39 PM, Ram Pai wrote: > > Anyway; so something like this should be possible without breaking > > existing semantics. > > > > mount -o bind,remount,ro /mnt > > mount --make-pass-on-access /mnt > > > >

Re: [PATCH V5 1/4] soc: brcmstb: Add Product ID and Family ID helper functions

2017-09-21 Thread Al Cooper
Kishon, I ran checkpatch on all my patches before submitting them and didn't get any warnings. It looks like you may be running checkpatch with the "--subjective" option. Is this a requirement and if so are there any other checkpatch options I should be using? Thanks Al On Thu, Sep 21, 2017 at 4

[PATCH] pinctrl: intel: Mask interrupts on driver probe

2017-09-21 Thread Kyle Roeschley
Powering off the system on Apollo Lake does not clear the interrupt enable registers for the GPIOs. To avoid an interrupt storm on driver probe, clear all interrupt enables before enabling our interrupt line. Signed-off-by: Kyle Roeschley --- drivers/pinctrl/intel/pinctrl-intel.c | 43 ++

RE: [PATCH 09/12] platform/x86: wmi: create character devices when requested by drivers

2017-09-21 Thread Mario.Limonciello
> -Original Message- > From: Andy Shevchenko [mailto:andy.shevche...@gmail.com] > Sent: Thursday, September 21, 2017 11:47 AM > To: Limonciello, Mario > Cc: dvh...@infradead.org; LKML ; Platform Driver > ; quasi...@google.com; Pali Rohár > > Subject: Re: [PATCH 09/12] platform/x86: wmi: c

[PATCH 0/3] [media] uvcvideo: Fine-tuning for some function implementations

2017-09-21 Thread SF Markus Elfring
From: Markus Elfring Date: Thu, 21 Sep 2017 21:20:12 +0200 Three update suggestions were taken into account from static source code analysis. Markus Elfring (3): Use common error handling code in uvc_ioctl_g_ext_ctrls() Adjust 14 checks for null pointers Add some spaces for better code rea

[PATCH 1/3] [media] uvcvideo: Use common error handling code in uvc_ioctl_g_ext_ctrls()

2017-09-21 Thread SF Markus Elfring
From: Markus Elfring Date: Thu, 21 Sep 2017 20:47:02 +0200 Add a jump target so that a bit of exception handling can be better reused at the end of this function. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring --- drivers/media/usb/uvc/uvc_v4l2.c | 13

[PATCH 2/3] [media] uvcvideo: Adjust 14 checks for null pointers

2017-09-21 Thread SF Markus Elfring
From: Markus Elfring Date: Thu, 21 Sep 2017 21:00:21 +0200 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The script “checkpatch.pl” pointed information out like the following. Comparison to NULL could be written … Thus fix the affected source code pla

[PATCH 3/3] [media] uvcvideo: Add some spaces for better code readability

2017-09-21 Thread SF Markus Elfring
From: Markus Elfring Date: Thu, 21 Sep 2017 21:12:29 +0200 Use space characters at some source code places according to the Linux coding style convention. Signed-off-by: Markus Elfring --- drivers/media/usb/uvc/uvc_v4l2.c | 13 +++-- 1 file changed, 7 insertions(+), 6 deletions(-) dif

[PATCH v2] Documentation: rewrite confusing statement about memory barriers

2017-09-21 Thread Guilherme G. Piccoli
In this specific portion of the write memory barriers description, the documentation mentions sequential order of stores, which is confusing since sequential ordering is not guaranteed. This patch tries to improve the doc in order to avoid any mis-understanding. Cc: Paul E. McKenney Signed-off-b

Re: [linux-sunxi] Re: [PATCH v4 1/6] dt-bindings: update the Allwinner GPADC device tree binding for H3

2017-09-21 Thread Maxime Ripard
On Wed, Sep 20, 2017 at 08:04:02AM +, Icenowy Zheng wrote: > 于 2017年9月20日 GMT+08:00 下午3:52:23, Maxime Ripard > 写到: > >On Mon, Sep 18, 2017 at 03:47:25PM +, icen...@aosc.io wrote: > >> 在 2017-09-18 16:30,Maxime Ripard 写道: > >> > On Mon, Sep 18, 2017 at 03:36:43PM +0800, Icenowy Zheng wrote

Re: [v2,1/3] can: m_can: Make hclk optional

2017-09-21 Thread Franklin S Cooper Jr
On 09/21/2017 09:08 AM, Sekhar Nori wrote: > On Thursday 21 September 2017 06:01 AM, Franklin S Cooper Jr wrote: >> >> >> On 08/24/2017 03:00 AM, Sekhar Nori wrote: >>> + some OMAP folks and Linux OMAP list >>> >>> On Tuesday 25 July 2017 04:21 AM, Franklin Cooper wrote: Hclk is the MCAN's i

Re: [tip:timers/core] kselftests: timers: set-timer-lat: Tweak reporting when timer fires early

2017-09-21 Thread Greg Hackmann
On 09/20/2017 04:38 PM, Shuah Khan wrote: On 09/20/2017 06:31 AM, Prarit Bhargava wrote: On 09/19/2017 11:36 PM, John Stultz wrote: On Tue, Sep 19, 2017 at 6:02 PM, Shuah Khan wrote: Hi Greg, On 08/20/2017 03:56 AM, tip-bot for Greg Hackmann wrote: Commit-ID: a524b1184b8e86141d689fa78ad150

Re: [PATCH 1/3] dt-bindings: Add document for rockchip RGB output interface

2017-09-21 Thread Rob Herring
On Tue, Sep 19, 2017 at 9:57 PM, Sandy Huang wrote: > > > 在 2017/9/20 9:51, Sandy Huang 写道: >> >> Hi rob, >> thanks for you review. >> >> 在 2017/9/19 22:46, Rob Herring 写道: >>> >>> On Thu, Sep 14, 2017 at 11:43:18AM +0800, Sandy Huang wrote: This path add support rv1108 rgb output i

Re: [PATCH v4 06/13] xen/pvcalls: implement bind command

2017-09-21 Thread Boris Ostrovsky
On 09/15/2017 07:00 PM, Stefano Stabellini wrote: Send PVCALLS_BIND to the backend. Introduce a new structure, part of struct sock_mapping, to store information specific to passive sockets. Introduce a status field to keep track of the status of the passive socket. Signed-off-by: Stefano Stab

Re: [tip:timers/core] kselftests: timers: set-timer-lat: Tweak reporting when timer fires early

2017-09-21 Thread Shuah Khan
On 09/21/2017 01:40 PM, Greg Hackmann wrote: > On 09/20/2017 04:38 PM, Shuah Khan wrote: >> On 09/20/2017 06:31 AM, Prarit Bhargava wrote: >>> On 09/19/2017 11:36 PM, John Stultz wrote: On Tue, Sep 19, 2017 at 6:02 PM, Shuah Khan wrote: > Hi Greg, > > On 08/20/2017 03:56 AM, tip-b

[PATCH] selftests: timers: set-timer-lat: Fix hang when testing unsupported alarms

2017-09-21 Thread Shuah Khan
When timer_create() fails on a bootime or realtime clock, setup_timer() returns 0 as if timer has been set. Callers wait forever for the timer to expire. This hang is seen on a system that doesn't have support for: CLOCK_REALTIME_ALARM ABSTIME missing CAP_WAKE_ALARM? : [UNSUPPORTED] Test hangs

Re: [PATCH v2 03/22] fpga: bridge: remove OF dependency for fpga-bridge

2017-09-21 Thread Alan Tull
On Thu, Sep 21, 2017 at 2:11 PM, Moritz Fischer wrote: Hi Moritz, > Hi, > > On Mon, Jun 26, 2017 at 09:51:59AM +0800, Wu Hao wrote: >> This patch removes OF dependency of fpga-bridge, it allows drivers >> to use fpga-bridge class without device tree support. >> >> Signed-off-by: Wu Hao >> --- >

Re: [PATCH v2] Documentation: rewrite confusing statement about memory barriers

2017-09-21 Thread Paul E. McKenney
On Thu, Sep 21, 2017 at 04:29:01PM -0300, Guilherme G. Piccoli wrote: > In this specific portion of the write memory barriers description, > the documentation mentions sequential order of stores, which is > confusing since sequential ordering is not guaranteed. > > This patch tries to improve the

Re: [PATCH] seccomp: fix the usage of get/put_seccomp_filter() in seccomp_get_filter()

2017-09-21 Thread Kees Cook
On Thu, Sep 21, 2017 at 3:57 AM, Oleg Nesterov wrote: > On 09/20, Kees Cook wrote: >> >> On Wed, Sep 20, 2017 at 5:56 AM, Oleg Nesterov wrote: >> > @@ -908,13 +912,13 @@ long seccomp_get_filter(struct task_struct *task, >> > unsigned long filter_off, >> > if (!data) >> >

Re: [PATCH v2] Documentation: rewrite confusing statement about memory barriers

2017-09-21 Thread Guilherme G. Piccoli
On 09/21/2017 04:50 PM, Paul E. McKenney wrote: > On Thu, Sep 21, 2017 at 04:29:01PM -0300, Guilherme G. Piccoli wrote: >> In this specific portion of the write memory barriers description, >> the documentation mentions sequential order of stores, which is >> confusing since sequential ordering is

Re: [kernel-hardening] [PATCH v3 1/3] x86/fpu: don't let userspace set bogus xcomp_bv

2017-09-21 Thread Rik van Riel
On Thu, 2017-09-21 at 11:52 -0700, Eric Biggers wrote: > From: Eric Biggers > > Fix the bug by checking that the user-supplied value of xcomp_bv is 0 > in > the uncompacted case, and returning an error otherwise. > Reviewed-by: Rik van Riel -- All rights reversed signature.asc Description: T

Re: [PATCH v2 07/22] fpga: intel: pcie: parse feature list and create platform device for features.

2017-09-21 Thread Alan Tull
)On Wed, Sep 20, 2017 at 4:24 PM, Alan Tull wrote: Hi Hao, A few more minor things below. > a (wh., *()On Sun, Jun 25, 2017 at 8:52 PM, Wu Hao wrote: > > Hi Hao, > > I'm done with some board bringup so I have time to look at your patchset > again. > > Something I can't help but notice is tha

Re: [PATCH 1/1] scsi: bnx: use setup_timer

2017-09-21 Thread Chad Dupuis
On Thu, 21 Sep 2017, 7:47am, Allen Pais wrote: > Use setup_timer function instead of initializing timer with the > function and data fields. > > Signed-off-by: Allen Pais > --- > drivers/scsi/bnx2fc/bnx2fc_fcoe.c | 5 ++--- > 1 file changed, 2 insertions(+), 3 deletions(-) > > diff --

[PATCH] selftests: timers: set-timer-lat: fix hang when std out/err are redirected

2017-09-21 Thread Shuah Khan
do_timer_oneshot() uses select() as a timer with FD_SETSIZE and readfs is cleared with FD_ZERO without FD_SET. When stdout and stderr are redirected, the test hangs in select forever. Fix the problem calling select() with readfds empty and nfds zero. This is sufficient for using select() for timer

Re: [PATCH 02/14] um: Use %pS printk format for symbols from direct addresses

2017-09-21 Thread Richard Weinberger
On Tue, Sep 12, 2017 at 2:10 PM, Petr Mladek wrote: > On Wed 2017-09-06 22:27:49, Helge Deller wrote: >> Use the %pS printk format for printing symbols from direct addresses. >> In usermode-linux there is actually no difference between %pS and %pF, but >> for >> consistency throughout the kernel

Re: [PATCH V5 1/4] soc: brcmstb: Add Product ID and Family ID helper functions

2017-09-21 Thread Joe Perches
On Thu, 2017-09-21 at 15:16 -0400, Al Cooper wrote: > Kishon, > > I ran checkpatch on all my patches before submitting them and didn't > get any warnings. It looks like you may be running checkpatch with the > "--subjective" option. Is this a requirement and if so are there any > other checkpatch

Re: [PATCH v2 00/40] tracing: Inter-event (e.g. latency) support

2017-09-21 Thread Steven Rostedt
On Tue, 19 Sep 2017 13:44:28 -0500 Tom Zanussi wrote: > Yeah, it's almost ready. At this point, I've addressed all the comments > except for: > > - PATCH v2 25/40] tracing: Add support for dynamic tracepoints > > which I need to do a little bit of research on to figure out what > exactly I n

Re: [kernel-hardening] [PATCH v3 2/3] x86/fpu: tighten validation of user-supplied xstate_header

2017-09-21 Thread Rik van Riel
On Thu, 2017-09-21 at 11:52 -0700, Eric Biggers wrote: > From: Eric Biggers > > Move validation of user-supplied xstate_headers into a helper > function > and call it from both the ptrace and sigreturn syscall paths.  The > new > function also considers it to be an error if *any* reserved bits ar

[PATCH 1/3] x86/kernel: Add option that TSC on Socket 0 being non-null is valid

2017-09-21 Thread mike.travis
Add a flag to indicate that a TSC ADJUST value of non-zero is valid on Socket 0. This is required on multiple chassis systems for which the Time Stamp Counter on all the chassis are started asynchronously. The UV architecture is an example of this. In this scenario the UV system BIOS will adjust

[PATCH 2/3] x86/kernel: Skip TSC test and error messages if already unstable

2017-09-21 Thread mike.travis
If the TSC has already been determined to be unstable, then checking TSC ADJUST values is a waste of time and generates unnecessary error messages (840 for a 16 socket Skylake 28/2 core/ht system). Signed-off-by: Mike Travis Reviewed-by: Dimitri Sivanich Reviewed-by: Russ Anderson --- arch/x86

[PATCH 0/3] x86/platform/UV: Update TSC support

2017-09-21 Thread mike.travis
The UV BIOS goes to considerable effort to get the TSC synchronization accurate across the entire system. Included in that are multiple chassis that can have 32+ sockets. The architecture also supports an external high resolution clock to help all the chassis and CPU's in maintaining this synchr

[PATCH] tracing: Reverse the order of trace_types_lock and event_mutex

2017-09-21 Thread Steven Rostedt
From: Steven Rostedt (VMware) In order to make future changes where we need to call tracing_set_clock() from within an event command, the order of trace_types_lock and event_mutex must be reversed, as the event command will hold event_mutex and the trace_types_lock is taken from within tracing_s

[PATCH 3/3] x86/platform/UV: Add check of TSC state set by UV BIOS

2017-09-21 Thread mike.travis
Insert a check early in UV system startup that checks whether BIOS was able to obtain satisfactory TSC Sync stability. If not, it usually is caused by an error in the external TSC clock generation source. In this case the best fallback is to use the builtin hardware RTC as the kernel would also no

Re: [PATCH] KEYS: fix cred refcount leak in request_key_auth_new()

2017-09-21 Thread Eric Biggers
On Tue, Sep 19, 2017 at 04:46:08PM +0100, David Howells wrote: > Eric Biggers wrote: > > > In request_key_auth_new(), if alloc_key() or key_instantiate_and_link() > > were to fail, we would leak a reference to the 'struct cred'. Currently > > this can only happen if alloc_key() fails to to alloc

Re: [PATCH 4/4] ALSA: emu10k1: Cocci spatch "alloc_cast"

2017-09-21 Thread Takashi Iwai
On Thu, 21 Sep 2017 19:39:15 +0200, Al Viro wrote: > > On Thu, Sep 21, 2017 at 08:33:46AM +0200, Thomas Meyer wrote: > > Remove casting the values returned by memory allocation functions like > > kmalloc, kzalloc, kmem_cache_alloc, kmem_cache_zalloc etc." > > Found by coccinelle spatch "api/alloc/

Re: [kernel-hardening] [PATCH v3 3/3] x86/fpu: reinitialize FPU registers if restoring FPU state fails

2017-09-21 Thread Rik van Riel
On Thu, 2017-09-21 at 11:52 -0700, Eric Biggers wrote: > From: Eric Biggers > > Userspace can change the FPU state of a task using the ptrace() or > rt_sigreturn() system calls.  Because reserved bits in the FPU state > can > cause the XRSTOR instruction to fail, the kernel has to carefully > val

[PATCH 1/2] tools: slabinfo: add "-U" option to show unreclaimable slabs only

2017-09-21 Thread Yang Shi
Add "-U" option to show unreclaimable slabs only. "-U" and "-S" together can tell us what unreclaimable slabs use the most memory to help debug huge unreclaimable slabs issue. Signed-off-by: Yang Shi Acked-by: Christoph Lameter Acked-by: David Rientjes --- tools/vm/slabinfo.c | 11 ++-

[PATCH 0/2 v5] oom: capture unreclaimable slab info in oom message when kernel panic

2017-09-21 Thread Yang Shi
Recently we ran into a oom issue, kernel panic due to no killable process. The dmesg shows huge unreclaimable slabs used almost 100% memory, but kdump doesn't capture vmcore due to some reason. So, it may sound better to capture unreclaimable slab info in oom message when kernel panic to aid tr

[PATCH 2/2] mm: oom: show unreclaimable slab info when kernel panic

2017-09-21 Thread Yang Shi
Kernel may panic when oom happens without killable process sometimes it is caused by huge unreclaimable slabs used by kernel. Although kdump could help debug such problem, however, kdump is not available on all architectures and it might be malfunction sometime. And, since kernel already panic it

Re: [Outreachy kernel] [PATCH] staging: vc04_services: Remove typedef struct

2017-09-21 Thread Julia Lawall
On Fri, 22 Sep 2017, Harsha Sharma wrote: > Remove typedef from struct as linux-kernel coding style tends to > avoid using typedefs > > Done using following coccinelle semantic patch > > @r1@ > type T; > @@ > > typedef struct { ... } T; > > @script:python c1@ > T2; > T << r1.T; > @@ > if T[-2:]

RE: [PATCH 07/12] platform/x86: dell-wmi-smbios: Use Dell WMI descriptor check

2017-09-21 Thread Mario.Limonciello
> -Original Message- > From: Andy Shevchenko [mailto:andy.shevche...@gmail.com] > Sent: Thursday, September 21, 2017 11:44 AM > To: Limonciello, Mario > Cc: dvh...@infradead.org; LKML ; Platform Driver > ; quasi...@google.com; Pali Rohár > > Subject: Re: [PATCH 07/12] platform/x86: dell-w

Re: [PATCH] Input: add support for the Samsung S6SY761 touchscreen

2017-09-21 Thread Dmitry Torokhov
Hi Andi, On Thu, Sep 21, 2017 at 10:29:50PM +0900, Andi Shyti wrote: > The S6SY761 touchscreen is a capicitive multi-touch controller > for mobile use. It's connected with i2c at the address 0x48. > > This commit provides a basic version of the driver which can > handle only initialization, touch

[PATCH v2 1/3] KEYS: fix cred refcount leak in request_key_auth_new()

2017-09-21 Thread Eric Biggers
From: Eric Biggers In request_key_auth_new(), if key_alloc() or key_instantiate_and_link() were to fail, we would leak a reference to the 'struct cred'. Currently this can only happen if key_alloc() fails to allocate memory. But it still should be fixed, as it is a more severe bug waiting to ha

[PATCH v2 3/3] KEYS: use kmemdup() in request_key_auth_new()

2017-09-21 Thread Eric Biggers
From: Eric Biggers kmemdup() is preferred to kmalloc() followed by memcpy(). Signed-off-by: Eric Biggers --- security/keys/request_key_auth.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/security/keys/request_key_auth.c b/security/keys/request_key_auth.c index 2df92

[PATCH v2 0/3] KEYS: fix error handling in request_key_auth_new()

2017-09-21 Thread Eric Biggers
From: Eric Biggers Fix a reference leak and a NULL pointer dereference in the error handling paths of request_key_auth_new(). Eric Biggers (3): KEYS: fix cred refcount leak in request_key_auth_new() KEYS: don't revoke uninstantiated key in request_key_auth_new() KEYS: use kmemdup() in requ

[PATCH v2 2/3] KEYS: don't revoke uninstantiated key in request_key_auth_new()

2017-09-21 Thread Eric Biggers
From: Eric Biggers If key_instantiate_and_link() were to fail (which fortunately isn't possible currently), the call to key_revoke(authkey) would crash with a NULL pointer dereference in request_key_auth_revoke() because the key has not yet been instantiated. Fix this by removing the call to key

[GIT PULL] libnvdimm fixes for 4.14-rc2

2017-09-21 Thread Williams, Dan J
Hi Linus, please pull from: git://git.kernel.org/pub/scm/linux/kernel/git/nvdimm/nvdimm libnvdimm-fixes ...to receive a crash fix and corresponding regression test enabling for the crash scenario. The unit test for this crash is available in ndctl-v58.2. This branch has received a build success

Re: [PATCH v2 00/40] tracing: Inter-event (e.g. latency) support

2017-09-21 Thread Tom Zanussi
Hi Steve, On Thu, 2017-09-21 at 16:20 -0400, Steven Rostedt wrote: > On Tue, 19 Sep 2017 13:44:28 -0500 > Tom Zanussi wrote: > > > Yeah, it's almost ready. At this point, I've addressed all the comments > > except for: > > > > - PATCH v2 25/40] tracing: Add support for dynamic tracepoints > >

Contribution to Linux Kernel.

2017-09-21 Thread Javier Romero
Hello, My name is Javier, live in Argentina, and work as a Linux Server admin in the Network Operations Center of an ISP. Would like to know, how can I start contributing to the Linux kernel. Sorry for the inconvenience. Regards, Javier Romero

Re: [v8 0/4] cgroup-aware OOM killer

2017-09-21 Thread David Rientjes
On Thu, 21 Sep 2017, Johannes Weiner wrote: > That's a ridiculous nak. > > The fact that this patch series doesn't solve your particular problem > is not a technical argument to *reject* somebody else's work to solve > a different problem. It's not a regression when behavior is completely > uncha

[PATCH] generic: Add nocheck shutdown stress test

2017-09-21 Thread Khazhismel Kumykov
Most shutdown tests only run on filesystems with metadata journaling, so we lose coverage. Add a shutdown stress test that doesn't check for consistency, so does not require journaling. Signed-off-by: Khazhismel Kumykov --- tests/generic/999 | 84 +

Re: [v8 0/4] cgroup-aware OOM killer

2017-09-21 Thread Johannes Weiner
On Thu, Sep 21, 2017 at 02:17:25PM -0700, David Rientjes wrote: > On Thu, 21 Sep 2017, Johannes Weiner wrote: > > > That's a ridiculous nak. > > > > The fact that this patch series doesn't solve your particular problem > > is not a technical argument to *reject* somebody else's work to solve > >

Re: [PATCH] gpio: thunderx: remove unused .map() hook from irq_domain_ops

2017-09-21 Thread David Daney
On 09/13/2017 07:37 PM, Masahiro Yamada wrote: This driver implements .alloc() hook, so .map() is not used. Although this comment is true for this driver, it is unclear to me if the statement is true in the general case. Signed-off-by: Masahiro Yamada Tested-by: David Daney --- d

[RFC PATCH 0/2] capability controlled user-namespaces

2017-09-21 Thread Mahesh Bandewar
From: Mahesh Bandewar TL;DR version - Creating a sandbox environment with namespaces is challenging considering what these sandboxed processes can engage into. e.g. CVE-2017-6074, CVE-2017-7184, CVE-2017-7308 etc. just to name few. Current form of user-namespaces, however, if changed

[RFC PATCH 1/2] capability: introduce sysctl for controlled user-ns capability whitelist

2017-09-21 Thread Mahesh Bandewar
From: Mahesh Bandewar Add a sysctl variable kernel.controlled_userns_caps_whitelist. This takes input as capability mask expressed as two comma separated hex u32 words. The mask, however, is stored in kernel as kernel_cap_t type. Any capabilities that are not part of this mask will be controlled

[RFC PATCH 2/2] userns: control capabilities of some user namespaces

2017-09-21 Thread Mahesh Bandewar
From: Mahesh Bandewar With this new notion of "controlled" user-namespaces, the controlled user-namespaces are marked at the time of their creation while the capabilities of processes that belong to them are controlled using the global mask. Init-user-ns is always uncontrolled and a process that

Re: [PATCH] KEYS: prevent creating a different user's keyrings

2017-09-21 Thread Eric Biggers
On Tue, Sep 19, 2017 at 05:05:20PM +0100, David Howells wrote: > Eric Biggers wrote: > > > Fix it by marking user and user session keyrings with a flag > > KEY_FLAG_UID_KEYRING. Then, when searching for a user or user session > > keyring by name, skip all keyrings that don't have the flag set. >

Re: [PATCH -tip v3 0/7] kprobes/x86: Preempt related enhancements

2017-09-21 Thread Alexei Starovoitov
On 9/19/17 2:58 AM, Masami Hiramatsu wrote: Hi, Here is the 3rd version of the series to improve preempt related behavior in kprobes/x86. This actually includes many enhancements/fixes from the 2nd version, which is https://lkml.org/lkml/2017/9/11/482 With the previous patch, lkp-bot reported

[PATCH] e1000: avoid null pointer dereference on invalid stat type

2017-09-21 Thread Colin King
From: Colin Ian King Currently if the stat type is invalid then data[i] is being set either by dereferencing a null pointer p, or it is reading from an incorrect previous location if we had a valid stat type previously. Fix this by nullify pointer p if a stat type is invalid and only setting dat

Re: [PATCH net-next] net: dsa: use dedicated CPU port

2017-09-21 Thread David Miller
From: Vivien Didelot Date: Wed, 20 Sep 2017 12:28:05 -0400 > Each port in DSA has its own dedicated CPU port currently available in > its parent switch's ds->ports[port].cpu_dp. Use it instead of getting > the unique tree CPU port, which will be deprecated soon. > > Signed-off-by: Vivien Didelot

Re: [PATCH 1/3] kcov: remove #ifdef CONFIG_RANDOMIZE_BASE

2017-09-21 Thread kbuild test robot
Hi Andrey, [auto build test ERROR on linus/master] [also build test ERROR on v4.14-rc1 next-20170921] [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/Andrey-Ryabinin/kcov-remove-ifdef

[PATCH] lib/lz4: make arrays static const, reduces object code size

2017-09-21 Thread Colin King
From: Colin Ian King Don't populate the read-only arrays dec32table and dec64table on the stack, instead make them both static const. Makes the object code smaller by over 10K bytes: Before: textdata bss dec hex filename 31500 0 0 315007b0c lib/lz4/lz4_dec

Re: [PATCH net-next] net: dsa: better scoping of slave functions

2017-09-21 Thread David Miller
From: Vivien Didelot Date: Wed, 20 Sep 2017 19:31:57 -0400 > A few DSA slave functions take a dsa_slave_priv pointer as first > argument, whereas the scope of the slave.c functions is the slave > net_device structure. Fix this and rename dsa_netpoll_send_skb to > dsa_slave_netpoll_send_skb. > >

Re: [PATCH net-next] net: dsa: add port fdb dump

2017-09-21 Thread David Miller
From: Vivien Didelot Date: Wed, 20 Sep 2017 19:32:14 -0400 > Dumping a DSA port's FDB entries is not specific to a DSA slave, so add > a dsa_port_fdb_dump function, similarly to dsa_port_fdb_add and > dsa_port_fdb_del. > > Signed-off-by: Vivien Didelot Applied.

[PATCH] i2c: designware: make const array supported_speeds static to shink object code size

2017-09-21 Thread Colin King
From: Colin Ian King Don't populate const array supported_speeds on the stack, instead make it static. Makes the object code smaller by 150 bytes: Before: textdata bss dec hex filename 84741440 0991426ba i2c-designware-platdrv.o After: textdata

[PATCH] staging: xgifb: make const array static to shink object code size

2017-09-21 Thread Colin King
From: Colin Ian King Don't populate const array LCDARefreshIndex on the stack, instead make it static. Makes the object code smaller by 340 bytes: Before: textdata bss dec hex filename 84949 12336 0 97285 17c05 drivers/staging/xgifb/vb_setmode.o After: text

Re: [PATCH v5 1/4] dt-bindings: Document the STM32 DMAMUX bindings

2017-09-21 Thread Rob Herring
On Thu, Sep 21, 2017 at 05:29:18PM +0200, Pierre-Yves MORDRET wrote: > This patch adds the documentation of device tree bindings for the STM32 > DMAMUX. > > Signed-off-by: M'boumba Cedric Madianga > Signed-off-by: Pierre-Yves MORDRET > --- > Version history: > v5: > v4: > * Add

Re: [PATCH 1/1] net:nfc: use setup_timer

2017-09-21 Thread David Miller
From: Allen Pais Date: Thu, 21 Sep 2017 16:29:33 +0530 > Use setup_timer function instead of initializing timer with the > function and data fields. > > Signed-off-by: Allen Pais Applied.

Re: [PATCH 1/1] net: wan : hdlc: use setup_timer() helper

2017-09-21 Thread David Miller
From: Allen Pais Date: Thu, 21 Sep 2017 18:17:55 +0530 > Use setup_timer function instead of initializing timer with the > function and data fields. > > Signed-off-by: Allen Pais Applied.

Re: [PATCH 1/1] net: usb: catc: use setup_timer() helper

2017-09-21 Thread David Miller
From: Allen Pais Date: Thu, 21 Sep 2017 18:24:15 +0530 > Use setup_timer function instead of initializing timer with the > function and data fields. > > Signed-off-by: Allen Pais Applied.

Re: [PATCH 1/1] net: ti: netcp: use setup_timer

2017-09-21 Thread David Miller
From: Allen Pais Date: Thu, 21 Sep 2017 18:32:58 +0530 > Use setup_timer function instead of initializing timer with the > function and data fields. > > Signed-off-by: Allen Pais Applied.

Re: [PATCH v9 4/7] clk: qcom: Add A53 PLL support

2017-09-21 Thread Rob Herring
On Thu, Sep 21, 2017 at 07:49:37PM +0300, Georgi Djakov wrote: > The CPUs on Qualcomm MSM8916-based platforms are clocked by two PLLs, > a primary (A53) CPU PLL and a secondary fixed-rate GPLL0. These sources > are connected to a mux and half-integer divider, which is feeding the > CPU cores. > >

Re: [PATCH] dt-bindings: nand: denali: reduce the register space in the example

2017-09-21 Thread Rob Herring
On Thu, Sep 14, 2017 at 04:17:36PM +0900, Masahiro Yamada wrote: > This example allocates too much for register regions. Especially, > there are only two registers in the "nand_data" interface of this > hardware (ADDR: 0x00, DATA: 0x10). > > Signed-off-by: Masahiro Yamada > --- > > Documentati

[PATCH] wireless: iwlegacy: make const array static to shink object code size Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 8bit

2017-09-21 Thread Colin King
From: Colin Ian King Don't populate const array ac_to_fifo on the stack in an inlined function, instead make it static. Makes the object code smaller by over 800 bytes: textdata bss dec hex filename 159029 331541216 193399 2f377 4965-mac.o textdata bss

Re: [PATCH] dt-bindings: fix vendor prefix for Abracon

2017-09-21 Thread Rob Herring
On Thu, Sep 14, 2017 at 11:39:38PM +0200, Alexandre Belloni wrote: > Commit 446810f2dd41 ("of: add vendor prefix for Abracon Corporation") > claimed that "abcn" was used as the vendor prefix while in fact "abracon" > was used in the subsequent commits. It is also the only prefix used in the > tree.

[PATCH] cnic: Fix an error handling path in 'cnic_alloc_bnx2x_resc()'

2017-09-21 Thread Christophe JAILLET
All the error handling paths 'goto error', except this one. We should also go to error in this case, or some resources will be leaking. Signed-off-by: Christophe JAILLET --- drivers/net/ethernet/broadcom/cnic.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/ether

Re: [PATCH v4 1/2] dt-bindings: interrupt-controller: add DT binding for meson GPIO interrupt controller

2017-09-21 Thread Rob Herring
On Mon, Sep 18, 2017 at 03:46:09PM +0200, Jerome Brunet wrote: > This commit adds the device tree bindings description for Amlogic's GPIO > interrupt controller available on the meson8b, gxbb and gxl SoC families > > Cc: Heiner Kallweit > Signed-off-by: Jerome Brunet > --- > .../amlogic,meson-g

Re: [PATCH] lib/lz4: make arrays static const, reduces object code size

2017-09-21 Thread Christophe JAILLET
Le 22/09/2017 à 00:19, Colin King a écrit : From: Colin Ian King Don't populate the read-only arrays dec32table and dec64table on the stack, instead make them both static const. Makes the object code smaller by over 10K bytes: 10k? Wouaouh! This is way much more than what you usually win with

Re: [PATCH v3 1/7] dt-bindings: timer: renesas,cmt: Fix SoC-specific compatible values

2017-09-21 Thread Rob Herring
On Mon, Sep 18, 2017 at 03:46:41PM +0200, Geert Uytterhoeven wrote: > While the new family-specific compatible values introduced by commit > 6f54cc1adcc8957f ("devicetree: bindings: R-Car Gen2 CMT0 and CMT1 > bindings") use the recommended order ",-", the > new SoC-specific compatible values still

Re: [PATCH] lib/lz4: make arrays static const, reduces object code size

2017-09-21 Thread Colin Ian King
On 22/09/17 00:09, Christophe JAILLET wrote: > Le 22/09/2017 à 00:19, Colin King a écrit : >> From: Colin Ian King >> >> Don't populate the read-only arrays dec32table and dec64table on the >> stack, instead make them both static const. Makes the object code >> smaller by over 10K bytes: > 10k? W

[RFC LINUX PATCH] Dcoumentation: dt: mailbox: Add Xilinx IPI Mailbox

2017-09-21 Thread Wendy Liang
Xilinx ZynqMP IPI(Inter Processor Interrupt) is a hardware block in ZynqMP SoC used for the communication between various processor systems. Signed-off-by: Wendy Liang --- .../bindings/mailbox/xlnx,zynqmp-ipi-mailbox.txt | 88 ++ 1 file changed, 88 insertions(+) create mod

Re: [PATCH] nvmem: sunxi-sid: add support for A64/H5's SID controller

2017-09-21 Thread Rob Herring
On Mon, Sep 18, 2017 at 11:42:04PM +0800, Icenowy Zheng wrote: > Allwinner A64/H5 SoCs come with a SID controller like the one in H3, but > without the silicon bug that makes the initial value at 0x200 wrong, so > the value at 0x200 can be directly read. > > Add support for this kind of SID contro

Re: [v5 04/12] dt-bindings: soc/fsl: Update reserved memory binding for QBMan

2017-09-21 Thread Rob Herring
On Mon, Sep 18, 2017 at 04:39:39PM -0400, Roy Pledge wrote: > Updates the QMan and BMan device tree bindings for reserved memory > nodes. This makes the reserved memory allocation compatible with > the shared-dma-pool usage. > > Signed-off-by: Roy Pledge > --- > Documentation/devicetree/bindings

<    2   3   4   5   6   7   8   9   >