[PATCH v3 0/4] x86/mm: Straightforward TLB flush fixes/cleanups

2017-04-22 Thread Andy Lutomirski
I have some less straightforward cleanups coming, but these three are easy and might even be okay for 4.12 assuming that someone feels like reviewing them. Changes from v2: - Added patch 1 (I suck at grep) - Delete one more copy of flush_tlb() in patch 2 (ditto) Changes from v1: - Fixed the Cc

[PATCH v3 2/4] x86/mm: Remove flush_tlb() and flush_tlb_current_task()

2017-04-22 Thread Andy Lutomirski
I was trying to figure out what how flush_tlb_current_task() would possibly work correctly if current->mm != current->active_mm, but I realized I could spare myself the effort: it has no callers except the unused flush_tlb() macro. Cc: Rik van Riel Cc: Dave Hansen Cc: Nadav Amit Cc: Michal Hock

[PATCH v3 1/4] x86/vm86/32: Switch to flush_tlb_mm_range() in mark_screen_rdonly()

2017-04-22 Thread Andy Lutomirski
mark_screen_rdonly() is the last remaining caller of flush_tlb(). flush_tlb_mm_range() is potentially faster and isn't obsolete. Compile-tested only because I don't know whether software that uses this mechanism even exists. Cc: Rik van Riel Cc: Dave Hansen Cc: Nadav Amit Cc: Michal Hocko Cc:

[PATCH v3 3/4] x86/mm: Make flush_tlb_mm_range() more predictable

2017-04-22 Thread Andy Lutomirski
I'm about to rewrite the function almost completely, but first I want to get a functional change out of the way. Currently, if flush_tlb_mm_range() does not flush the local TLB at all, it will never do individual page flushes on remote CPUs. This seems to be an accident, and preserving it will be

Re: fsl_ifc_nand: are blank pages protected by ECC?

2017-04-22 Thread Pavel Machek
Hi! > > Maybe I figured it out. Unfortunately, it is only compile tested. Does > > it look approximately right? > > Yep that's definitely better. Just one thing missing (see below), > otherwise it looks good. Thanks for review. Unfortunately it is still untested, so... > > + if (res <

[PATCH v3 4/4] x86/mm: Fix flush_tlb_page() on Xen

2017-04-22 Thread Andy Lutomirski
flush_tlb_page() passes a bogus range to flush_tlb_others() and expects the latter to fix it up. native_flush_tlb_others() has the fixup but Xen's version doesn't. Move the fixup to flush_tlb_others(). AFAICS the only real effect is that, without this fix, Xen would flush everything instead of j

[PATCH] staging: wilc1000: Fixes Alignment should match open parenthesis

2017-04-22 Thread richard
From: Richard Miller This patch fixes the following checkpatch.pl check: CHECK: Alignment should match open parenthesis Signed-off-by: Richard Miller --- drivers/staging/wilc1000/host_interface.c | 2 +- drivers/staging/wilc1000/linux_wlan.c | 8 drivers/staging/

[patch 1/1] staging: speakup: add send_xchar and tiocmset methods

2017-04-22 Thread Okash Khawaja
This adds two methods to spk_synth struct: send_xchar and tiocmset, and creates serial implementation for each of them. It takes existing code in apollo, audptr and spkout which already fits the behaviour of send_xchar and tiocmset. In follow-up patches there will be TTY-based implementations of th

[patch 0/1] staging: speakup: refactor to allow smoother migration to TTY-based comms with external synths

2017-04-22 Thread Okash Khawaja
This is part of move away from raw serial i/o to TTY-based one. Details are in the patch header. Thanks, Okash

[RFC PATH] of/pci/dma: fix DMA configruation for PCI masters

2017-04-22 Thread Oza Pawandeep
current device frmework and of framework integration assumes dma-ranges in a way where memory-mapped devices define their dma-ranges. dma-ranges: (child-bus-address, parent-bus-address, length). but iproc based SOCs and other SOCs(suc as rcar) have PCI world dma-ranges. dma-ranges = <0x4300 0x

Re: [PATCH v3] selftests: ftrace: Allow some tests to be run in a tracing instance

2017-04-22 Thread Masami Hiramatsu
On Fri, 21 Apr 2017 23:38:50 -0400 Steven Rostedt wrote: > From 4464dc867ead3ea14654165ad3ab68263aff7b17 Mon Sep 17 00:00:00 2001 > From: "Steven Rostedt (VMware)" > Date: Thu, 20 Apr 2017 12:53:18 -0400 > Subject: [PATCH] selftests: ftrace: Allow some tests to be run in a tracing > instance >

Re: linux-next: build failure after merge of the arm tree

2017-04-22 Thread Mason
On 22/04/2017 01:43, Russell King - ARM Linux wrote: > Or I drop the offending patch (done) and we get the DMA subsystem fixed > first. Given how long it's been this way, I doubt there's any hurry to > get this change in for the next merge window. Your solution makes sense. Vinod, could you app

Re: [PATCH 1/5] lightnvm: pblk: fix race condition on line retry

2017-04-22 Thread Matias Bjørling
On 04/22/2017 01:32 AM, Javier González wrote: When a pblk line fails (or is recovered), make sure to take the line management lock. Fixes: a4bd217b4326 "lightnvm: physical block device (pblk) target" Signed-off-by: Javier González --- drivers/lightnvm/pblk-core.c | 6 +++--- 1 file changed,

Re: [PATCH 2/5] lightnvm: pblk: fix bad error check

2017-04-22 Thread Matias Bjørling
On 04/22/2017 01:32 AM, Javier González wrote: Fix bad error check Fixes: a4bd217b4326 "lightnvm: physical block device (pblk) target" Signed-off-by: Javier González --- drivers/lightnvm/pblk-core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/lightnvm/pblk-core

Re: [PATCH 3/5] lightnvm: pblk: fix memory leak on error path

2017-04-22 Thread Matias Bjørling
On 04/22/2017 01:32 AM, Javier González wrote: When write recovery fails, Free memory for the recovery structure. Fixes: a4bd217b4326 "lightnvm: physical block device (pblk) target" Signed-off-by: Javier González --- drivers/lightnvm/pblk-write.c | 3 +++ 1 file changed, 3 insertions(+) di

Re: [PATCH 4/5] lightnvm: pblk: free metadata on line alloc failure

2017-04-22 Thread Matias Bjørling
On 04/22/2017 01:32 AM, Javier González wrote: When a line allocation fails, for example, due to having too many bad blocks, free its metadata correctly. Fixes: a4bd217b4326 "lightnvm: physical block device (pblk) target" Signed-off-by: Javier González --- drivers/lightnvm/pblk-core.c | 16

Re: [PATCH 01/11] blk: remove bio_set arg from blk_queue_split()

2017-04-22 Thread Javier González
> On 20 Apr 2017, at 07.38, NeilBrown wrote: > > blk_queue_split() is always called with the last arg being q->bio_split, > where 'q' is the first arg. > > Also blk_queue_split() sometimes uses the passed-in 'bs' and sometimes uses > q->bio_split. > > This is inconsistent and unnecessary. Remo

Re: [PATCH 5/5] lightnvm: pblk: fix erase counters on error fail

2017-04-22 Thread Matias Bjørling
On 04/22/2017 01:32 AM, Javier González wrote: When block erases fail, these blocks are marked bad. The number of valid blocks in the line was not updated, which could cause an infinite loop on the erase path. Fix this atomic counter and, in order to avoid taking an irq lock on the interrupt con

Re: [PATCH v17 2/3] usb: USB Type-C connector class

2017-04-22 Thread Rajaram R
On Fri, Apr 21, 2017 at 10:13 PM, Guenter Roeck wrote: > On Fri, Apr 21, 2017 at 07:57:52PM +0530, Rajaram R wrote: >> On Fri, Apr 21, 2017 at 1:16 AM, Badhri Jagan Sridharan >> wrote: >> > Thanks for the responses :) >> > >> > So seems like we have a plan. >> > >> > In Type-C connector class the

Re: [PATCH 5/5] lightnvm: pblk: fix erase counters on error fail

2017-04-22 Thread Javier González
> On 22 Apr 2017, at 11.22, Matias Bjørling wrote: > > On 04/22/2017 01:32 AM, Javier González wrote: >> When block erases fail, these blocks are marked bad. The number of valid >> blocks in the line was not updated, which could cause an infinite loop >> on the erase path. >> >> Fix this atomic

[PATCH] serial: exar: Fix stuck MSIs

2017-04-22 Thread Jan Kiszka
From: Jan Kiszka After migrating 8250_exar to MSI in 172c33cb61da, we can get stuck without further interrupts because of the special wake-up event these chips send. They are only cleared by reading INT0. As we fail to do so during startup and shutdown, we can leave the interrupt line asserted, w

Re: [PATCH] drm: fourcc byteorder: brings header file comments in line with reality.

2017-04-22 Thread Ville Syrjälä
On Sat, Apr 22, 2017 at 01:07:57AM -0400, Ilia Mirkin wrote: > On Fri, Apr 21, 2017 at 12:59 PM, Ville Syrjälä > wrote: > > On Fri, Apr 21, 2017 at 10:49:49AM -0400, Ilia Mirkin wrote: > >> On Fri, Apr 21, 2017 at 3:58 AM, Gerd Hoffmann wrote: > >> > While working on graphics support for virtual

Re: [PATCH] drm: fourcc byteorder: brings header file comments in line with reality.

2017-04-22 Thread Ville Syrjälä
On Fri, Apr 21, 2017 at 06:14:31PM +0200, Gerd Hoffmann wrote: > Hi, > > > > My personal opinion is that formats in drm_fourcc.h should be > > > independent of the CPU byte order and the function > > > drm_mode_legacy_fb_format() and drivers depending on that incorrect > > > assumption be fix

[PATCH] tango_nand.c: fix ecc.stats_corrected in empty flash case

2017-04-22 Thread Pavel Machek
Fix ecc.stats_corrected in empty flash case. Signed-off-by: Pavel Machek --- This was suggested by Boris Brezillon in another context. Not tested; I don't have the hardware. diff --git a/drivers/mtd/nand/tango_nand.c b/drivers/mtd/nand/tango_nand.c index 4a5e948..db4bff4 100644 --- a/drivers/m

Re: [PATCH 2/5] mm: convert anon_vma.refcount from atomic_t to refcount_t

2017-04-22 Thread zhong jiang
Hi, Elean Do the issue had really occured, use-after-free. but why the patch is not received. or is is possible for the situation. Thanks zhongjiang On 2017/2/20 18:49, Elena Reshetova wrote: > refcount_t type and corresponding API should be > used instead of atomic_t when the variable is use

Re: [PATCH v3] selftests: ftrace: Allow some tests to be run in a tracing instance

2017-04-22 Thread Steven Rostedt
On Sat, 22 Apr 2017 17:41:49 +0900 Masami Hiramatsu wrote: > On Fri, 21 Apr 2017 23:38:50 -0400 > Steven Rostedt wrote: > > > From 4464dc867ead3ea14654165ad3ab68263aff7b17 Mon Sep 17 00:00:00 2001 > > From: "Steven Rostedt (VMware)" > > Date: Thu, 20 Apr 2017 12:53:18 -0400 > > Subject: [PATCH

[PATCH v2 0/3] mtd: sharpslpart partition parser

2017-04-22 Thread Andrea Adami
This patchset introduces a simple partition parser for the Sharp SL Series PXA handhelds. More details in the commit text. I have set in cc the ARM PXA maintainers because this is the MTD part of a planned wider patchset cleaning the Zaurus board files. Changelog: v1 initial import of 2.4 sources

[PATCH v2 3/3] mtd: nand: tmio_nand.c: prefer sharpslpart MTD partition parser

2017-04-22 Thread Andrea Adami
This is the specific parser for Sharp SL Series (Zaurus) Signed-off-by: Andrea Adami --- drivers/mtd/nand/tmio_nand.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/mtd/nand/tmio_nand.c b/drivers/mtd/nand/tmio_nand.c index fc5e773..f3612ac 100644 --- a/drivers/mtd

[PATCH v2 1/3] mtd: sharpsl: add sharpslpart MTD partition parser

2017-04-22 Thread Andrea Adami
The Sharp SL Series (Zaurus) PXA handhelds have 16/64/128M of NAND flash and share the same layout of the first 7M partition, managed by Sharp FTL. The purpose of this self-contained patch is to add a common parser and remove the hardcoded sizes in the board files (these devices are not yet conver

[PATCH v2 2/3] mtd: nand: sharpsl.c: prefer sharpslpart MTD partition parser

2017-04-22 Thread Andrea Adami
This is the specific parser for Sharp SL Series (Zaurus). Signed-off-by: Andrea Adami --- drivers/mtd/nand/sharpsl.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/mtd/nand/sharpsl.c b/drivers/mtd/nand/sharpsl.c index 064ca17..2d1ba0e 100644 --- a/drivers/mtd/nand

Re: [PATCH v2 0/5] mtd: nand: gpmi: add i.MX 7 support

2017-04-22 Thread Marek Vasut
On 04/22/2017 03:23 AM, Stefan Agner wrote: > This patchset adds support for i.MX 7 SoC for the GPMI NAND controller. > There have been similar patchsets already: > https://lkml.org/lkml/2016/2/23/912 > > However, this patchset does not make use of any of the new features. > The current feature se

Re: [PATCH 1/2] clk: ti: divider: try to fix ti_clk_register_divider

2017-04-22 Thread Arnd Bergmann
On Sat, Apr 22, 2017 at 3:58 AM, Stephen Boyd wrote: > On 04/20, Tero Kristo wrote: >> On 19/04/17 20:44, Arnd Bergmann wrote: >> The failing piece of code is only executed for legacy boot mode >> OMAP3 right now, which could be potentially stripped out of the >> kernel already (I think Tony remov

Re: A small window for a race condition in mm/rmap.c:page_lock_anon_vma_read

2017-04-22 Thread zhong jiang
Hi, Dashi The same issue I had occured every other week. Do you have solve it . I want to know how it is fixed. The patch exist in the mainline. Thanks zhongjiang On 2016/12/23 10:38, Dashi DS1 Cao wrote: > I'd expected that one or more tasks doing the free were the current task of > other cp

Re: [PATCH v3 2/2] modules:capabilities: add a per-task modules autoload restriction

2017-04-22 Thread Djalal Harouni
On Sat, Apr 22, 2017 at 1:28 AM, Andy Lutomirski wrote: > On Fri, Apr 21, 2017 at 4:19 PM, Kees Cook wrote: >> On Wed, Apr 19, 2017 at 7:41 PM, Andy Lutomirski wrote: >>> On Wed, Apr 19, 2017 at 4:43 PM, Kees Cook wrote: On Wed, Apr 19, 2017 at 4:15 PM, Andy Lutomirski wrote: > On Wed

[PATCH] net: netcp: fix spelling mistake: "memomry" -> "memory"

2017-04-22 Thread Colin King
From: Colin Ian King Trivial fix to spelling mistake in dev_err message and rejoin line. Signed-off-by: Colin Ian King --- drivers/net/ethernet/ti/netcp_ethss.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/net/ethernet/ti/netcp_ethss.c b/drivers/net/ethernet/t

[PATCH] staging/speakup: use __func__ instead of hardcoded name

2017-04-22 Thread Dhiru Kholia
This coding style issue was found by checkpatch script. Signed-off-by: Dhiru Kholia --- drivers/staging/speakup/main.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/speakup/main.c b/drivers/staging/speakup/main.c index d2ad596..3b4ba79 100644 --- a/drivers/s

[PATCH] ASoC: wm5100: fix spelling mistake: "micropone" -> "microphone"

2017-04-22 Thread Colin King
From: Colin Ian King Trivial fix to spelling mistake in dev_err message Signed-off-by: Colin Ian King --- sound/soc/codecs/wm5100.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sound/soc/codecs/wm5100.c b/sound/soc/codecs/wm5100.c index 560575000cc5..138a84efdd54 100644

Re: [linux-sunxi] [PATCH v2 1/6] clk: sunxi-ng: Add sun4i/sun7i CCU driver

2017-04-22 Thread Jonathan Liu
Hi Priit, On 27 March 2017 at 04:20, Priit Laes wrote: > Introduce a clock controller driver for sun4i A10 and sun7i A20 > series SoCs. > > Signed-off-by: Priit Laes > --- > drivers/clk/sunxi-ng/Kconfig | 13 +- > drivers/clk/sunxi-ng/Makefile |1 +- > dri

[PATCH] drivers/tty/hvc: fix spelling mistake: "missmanaged" -> "mismanaged"

2017-04-22 Thread Colin King
From: Colin Ian King Trivial fix to spelling mistake in printk message Signed-off-by: Colin Ian King --- drivers/tty/hvc/hvcs.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/tty/hvc/hvcs.c b/drivers/tty/hvc/hvcs.c index 7823d6d998cf..2113c64d565d 100644 --- a/dr

[PATCH] ore: fix spelling mistake: "Multples" -> "multiples"

2017-04-22 Thread Colin King
From: Colin Ian King trivial fix to spelling mistake in ORE_ERR message and make word all lower case. Signed-off-by: Colin Ian King --- fs/exofs/ore.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/fs/exofs/ore.c b/fs/exofs/ore.c index 8bb72807e70d..811522ae45e1 100644 ---

[tip:smp/hotplug 39/39] kernel/padata.c:990:2: error: implicit declaration of function 'cpuhp_state_add_instance_nocalls_cpuslocked'

2017-04-22 Thread kbuild test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git smp/hotplug head: b6a82f2d736a19456b4eff64d9b7c165f15e5698 commit: b6a82f2d736a19456b4eff64d9b7c165f15e5698 [39/39] padata: Fix fallout from get_online_cpus() rework config: ia64-allmodconfig (attached as .config) compiler: ia64

[PATCH] scsi: fusion: fix spelling mistake: "Persistancy" -> "Persistency"

2017-04-22 Thread Colin King
From: Colin Ian King trivial fix to spelling mistake Signed-off-by: Colin Ian King --- drivers/message/fusion/mptbase.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/message/fusion/mptbase.c b/drivers/message/fusion/mptbase.c index 1e73064b0fb2..62cff5afc6bd 10064

Re: [PATCH] backlight: pwm_bl: Fix condition to set enable gpio as output

2017-04-22 Thread Paul Kocialkowski
Hi, Le jeudi 20 avril 2017 à 10:55 +0100, Daniel Thompson a écrit : > On 18/04/17 09:46, Paul Kocialkowski wrote: > > Hi, > > > > Le lundi 17 avril 2017 à 21:36 +0200, Geert Uytterhoeven a écrit : > > > On Mon, Apr 17, 2017 at 5:38 PM, Paul Kocialkowski > > > wrote: > > > > Le dimanche 16 avril

[PATCH v2] net: natsemi: ns83820: add checks for dma mapping error

2017-04-22 Thread Alexey Khoroshilov
The driver does not check if mapping dma memory succeed. The patch adds the checks and failure handling. Found by Linux Driver Verification project (linuxtesting.org). Signed-off-by: Alexey Khoroshilov --- drivers/net/ethernet/natsemi/ns83820.c | 42 +++--- 1 file ch

[PATCH] serial: 8250: omap: Disable DMA for console UART

2017-04-22 Thread Vignesh R
Kernel always writes log messages to console via serial8250_console_write()->serial8250_console_putchar() which directly accesses UART_TX register _without_ using DMA. But, if other processes like systemd using same UART port, then these writes are handled by a different code flow using 8250_omap

[PATCH] ms_block: fix spelling mistake: "randomally" -> "randomly"

2017-04-22 Thread Colin King
From: Colin Ian King trivial fix to spelling mistake in dbg_verbose message Signed-off-by: Colin Ian King --- drivers/memstick/core/ms_block.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/memstick/core/ms_block.c b/drivers/memstick/core/ms_block.c index 99e651c27

[PATCH] orinoco: fix spelling mistake: "Registerred" -> "Registered"

2017-04-22 Thread Colin King
From: Colin Ian King trivial fix to spelling mistake in dbg_dbg message Signed-off-by: Colin Ian King --- drivers/net/wireless/intersil/orinoco/main.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/wireless/intersil/orinoco/main.c b/drivers/net/wireless/inters

Re: [PATCH] drm: fourcc byteorder: brings header file comments in line with reality.

2017-04-22 Thread Ilia Mirkin
On Sat, Apr 22, 2017 at 5:50 AM, Ville Syrjälä wrote: > On Sat, Apr 22, 2017 at 01:07:57AM -0400, Ilia Mirkin wrote: >> On Fri, Apr 21, 2017 at 12:59 PM, Ville Syrjälä >> wrote: >> > On Fri, Apr 21, 2017 at 10:49:49AM -0400, Ilia Mirkin wrote: >> >> On Fri, Apr 21, 2017 at 3:58 AM, Gerd Hoffmann

Re: [PATCH] drm: fourcc byteorder: brings header file comments in line with reality.

2017-04-22 Thread Ilia Mirkin
On Sat, Apr 22, 2017 at 9:40 AM, Ilia Mirkin wrote: > On Sat, Apr 22, 2017 at 5:50 AM, Ville Syrjälä > wrote: >> On Sat, Apr 22, 2017 at 01:07:57AM -0400, Ilia Mirkin wrote: >>> On Fri, Apr 21, 2017 at 12:59 PM, Ville Syrjälä >>> wrote: >>> > On Fri, Apr 21, 2017 at 10:49:49AM -0400, Ilia Mirkin

Memory order of `mprotect`

2017-04-22 Thread Yichao Yu
Hi, For the purpose of concurrent/parallel GC (in userspace obviously), I'm interested in what memory order guarantee is provided by the mprotect syscall. My questions are (assuming no compilier reordering or with sufficient compiler barrier) 1. If thread 1 triggers a segfault on an address due t

[PATCH v2 00/12] InfiniBand-OCRDMA: Fine-tuning for several function implementations

2017-04-22 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 22 Apr 2017 16:16:06 +0200 Several update suggestions were taken into account from static source code analysis. Markus Elfring (12): Use kcalloc() in ocrdma_mbx_alloc_pd_range() Use kcalloc() in five functions Improve size determinations in ten functions D

[PATCH v2 01/12] IB/ocrdma: Use kcalloc() in ocrdma_mbx_alloc_pd_range()

2017-04-22 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 22 Apr 2017 11:33:25 +0200 * Multiplications for the size determination of memory allocations indicated that array data structures should be processed. Thus reuse the corresponding function "kcalloc". This issue was detected by using the Coccinelle software.

[PATCH v2 02/12] IB/ocrdma: Use kcalloc() in five functions

2017-04-22 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 22 Apr 2017 11:41:55 +0200 * Multiplications for the size determination of memory allocations indicated that array data structures should be processed. Thus use the corresponding function "kcalloc". This issue was detected by using the Coccinelle software.

[PATCH v2 03/12] IB/ocrdma: Improve size determinations in ten functions

2017-04-22 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 22 Apr 2017 13:26:49 +0200 Replace the specification of data types by pointer dereferences as the parameter for the operator "sizeof" to make the corresponding size determinations a bit safer according to the Linux coding style convention. Signed-off-by: Markus El

[PATCH v2 04/12] IB/ocrdma: Delete unnecessary variable initialisations in ocrdma_mbx_get_dcbx_config()

2017-04-22 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 22 Apr 2017 13:38:33 +0200 The local variables "req" and "rsp" will be set to appropriate pointers a bit later. Thus omit the explicit initialisations at the beginning. Signed-off-by: Markus Elfring Reviewed-by: Yuval Shaia --- drivers/infiniband/hw/ocrdma/ocrd

[PATCH v2 05/12] IB/ocrdma: Delete unnecessary braces

2017-04-22 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 22 Apr 2017 13:54:28 +0200 Do not use curly brackets at some source code places where a single statement should be sufficient. Signed-off-by: Markus Elfring --- v2: Changes were rebased on source files from Linux next-20170421. These were recombined as requested

[PATCH v2 06/12] IB/ocrdma: Use kmalloc_array() in ocrdma_create_srq()

2017-04-22 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 22 Apr 2017 14:17:46 +0200 * A multiplication for the size determination of a memory allocation indicated that an array data structure should be processed. Thus use the corresponding function "kmalloc_array". This issue was detected by using the Coccinelle s

[PATCH v2 07/12] IB/ocrdma: Adjust 21 checks for null pointers

2017-04-22 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 22 Apr 2017 14:20:37 +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 affected source code places

[PATCH v2 08/12] IB/ocrdma: Delete an error message for a failed memory allocation in ocrdma_add()

2017-04-22 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 22 Apr 2017 14:36:30 +0200 Omit an extra message for a memory allocation failure in this function. Link: http://events.linuxfoundation.org/sites/events/files/slides/LCJ16-Refactor_Strings-WSang_0.pdf Signed-off-by: Markus Elfring Acked-by: Devesh Sharma Review

[PATCH v2 09/12] IB/ocrdma: Delete an unnecessary variable assignment in ocrdma_alloc_mr()

2017-04-22 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 22 Apr 2017 15:07:41 +0200 Delete an assignment for the local variable "status" in an if branch because the desired failure indication is already specified by a constant error code at the end. Signed-off-by: Markus Elfring Acked-by: Devesh Sharma Reviewed-by: Yu

Re: [linux-sunxi] [PATCH v2 1/6] clk: sunxi-ng: Add sun4i/sun7i CCU driver

2017-04-22 Thread Jonathan Liu
Hi Priit, On 27 March 2017 at 04:20, Priit Laes wrote: > +static struct ccu_nkmp pll_ve_clk = { > + .enable = BIT(31), > + .n = _SUNXI_CCU_MULT_OFFSET(8, 5, 0), > + .k = _SUNXI_CCU_MULT(4, 2), > + .m = _SUNXI_CCU_DIV(0, 2), >

[PATCH v2 10/12] IB/ocrdma: Returning only value constants in ocrdma_resize_cq()

2017-04-22 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 22 Apr 2017 15:15:09 +0200 Return constant integer values without storing them in the local variable "status". Signed-off-by: Markus Elfring Acked-by: Devesh Sharma Reviewed-by: Yuval Shaia --- drivers/infiniband/hw/ocrdma/ocrdma_verbs.c | 9 +++-- 1 file

[PATCH v2 11/12] IB/ocrdma: Delete an unnecessary variable in ocrdma_dealloc_pd()

2017-04-22 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 22 Apr 2017 15:32:05 +0200 1. Return zero in one case directly. 2. Return the value from a call of the function "_ocrdma_dealloc_pd" without using an extra assignment for the local variable. 3. Remove the variable "status" in this function then. Signed-off-by

[PATCH v2 12/12] IB/ocrdma: One jump label less in ocrdma_alloc_ucontext_pd()

2017-04-22 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 22 Apr 2017 15:45:21 +0200 This issue was detected by using the Coccinelle software. 1. Return directly if a call of the function "_ocrdma_alloc_pd" failed. 2. Reduce the scope for the local variable "status" to one case of an if statement. 3. Delete the jump

Re: [PATCH] staging: wilc1000: Fixes Alignment should match open parenthesis

2017-04-22 Thread Joe Perches
On Sat, 2017-04-22 at 07:28 +0100, rich...@netpro.work wrote: > This patch fixes the following checkpatch.pl > check: CHECK: Alignment should match open parenthesis Please strive for improved code for human readers and not just shutting up checkpatch. > diff --git a/drivers/staging/wilc1000/linux

Re: [PATCH v3 14/37] mtd: nand: denali: support "nand-ecc-strength" DT property

2017-04-22 Thread Masahiro Yamada
Hi Boris, 2017-04-14 17:19 GMT+09:00 Boris Brezillon : > On Fri, 14 Apr 2017 16:57:23 +0900 > Masahiro Yamada wrote: > >> Hi Boris, >> >> >> 2017-04-11 16:56 GMT+09:00 Boris Brezillon >> : >> > Hi Masahiro, >> > >> > On Tue, 11 Apr 2017 15:19:21 +0900 >> > Masahiro Yamada wrote: >> > >> >> Hi

Re: [PATCH] Introduce v3 namespaced file capabilities

2017-04-22 Thread Serge E. Hallyn
Quoting Eric W. Biederman (ebied...@xmission.com): > > "Serge E. Hallyn" writes: > > Overall this looks quite reasonable. > > My only big concern was the lack of verifying of magic_etc. As without Yes, I was relying too much on the size check. > that the code might not be future compatible w

[PATCH] staging: fbtft: fix character limit, trailing ; warning, etc.

2017-04-22 Thread Eric Curtin
checkpatch spits out a warning about the 80 character line limit. Split the parameters of these functions onto different lines. Put the ; with the macro caller instead. Lined up parameters as there was another CHECK warning about that. Signed-off-by: Eric Curtin --- drivers/staging/fbtft/fbtft-b

Re: [RFC] minimum gcc version for kernel: raise to gcc-4.3 or 4.6?

2017-04-22 Thread Arnd Bergmann
On Sat, Apr 22, 2017 at 5:10 AM, Maciej W. Rozycki wrote: > On Fri, 21 Apr 2017, Kees Cook wrote: > >> > The linux-4.2 x86 defconfig could still be built with gcc-4.0, but >> > later kernels have several minor problems with that, and >> > require at least gcc-4.3. >> > >> > If we are ok with this

Re: [resend PATCH v2 11/33] dm: add dax_device and dax_operations support

2017-04-22 Thread Mike Snitzer
On Thu, Apr 20 2017 at 12:30pm -0400, Dan Williams wrote: > On Mon, Apr 17, 2017 at 12:09 PM, Dan Williams > wrote: > > Allocate a dax_device to represent the capacity of a device-mapper > > instance. Provide a ->direct_access() method via the new dax_operations > > indirection that mirrors the

Re: [PATCH] rtc: sh: mark PM functions as unused

2017-04-22 Thread Alexandre Belloni
On 19/04/2017 at 19:52:43 +0200, Arnd Bergmann wrote: > The sh_rtc_set_irq_wake() function is only called from the suspend/resume > handlers > that may be hidden, causing a harmless warning: > > drivers/rtc/rtc-sh.c:724:13: error: 'sh_rtc_set_irq_wake' defined but not > used [-Werror=unused-func

Re: [PATCH] make TIOCSTI ioctl require CAP_SYS_ADMIN

2017-04-22 Thread Matt Brown
On 04/21/2017 01:24 AM, Serge E. Hallyn wrote: On Fri, Apr 21, 2017 at 01:09:59AM -0400, Matt Brown wrote: On 04/20/2017 01:41 PM, Serge E. Hallyn wrote: Quoting m...@nmatt.com (m...@nmatt.com): On 2017-04-20 11:19, Serge E. Hallyn wrote: Quoting Matt Brown (m...@nmatt.com): On 04/19/2017 07

[RFC 1/3] dt-binding: soc: qcom: Add binding for RFSA

2017-04-22 Thread Bjorn Andersson
This adds the binding for describing shared memory buffers for implementing the remote filesystem protocol. Signed-off-by: Bjorn Andersson --- My initial attempt was to mimic the ramoops of just adding the compatible to the reserved-memory node, but I have not been able to figure out a sane way

[RFC 3/3] soc: qcom: rfsa driver

2017-04-22 Thread Bjorn Andersson
The rfsa driver is used for allocating and exposing regions of shared memory with remote processors for the purpose of exchanging sector-data between the remote filesystem service and its clients. It provides accessors for the properties needed by the user space remote filesystem implementation th

[RFC 2/3] of: reserved_mem: Accessor for acquiring reserved_mem

2017-04-22 Thread Bjorn Andersson
In some cases drivers referencing a reserved-memory region might want to remap the entire region, but when defining the reserved-memory by "size" the client driver has no means to know the associated base address of the reserved memory region. This patch adds an accessor for such drivers to acquir

[PATCH 1/2] scripts: show_delta: add time stamp rounding option

2017-04-22 Thread Erik Stromdahl
An additional feature for show_delta: -r With this option the delta time stamps are rounded to the closest microsecond specified by the -r option. Rationale: Make it easier to diff kernel logs when looking for timing issues (less irrelevant diffs detected by a diff tool). Signed-off-by: Erik Str

[PATCH 2/2] scripts: show_delta: add --delta-only option

2017-04-22 Thread Erik Stromdahl
The --delta-only option makes show_delta print delta time stamps only (and not the combination of absolute and delta time stamps which is the default). Rationale: Easier to diff kernel logs (with graphical diff tool) when debugging timing issues. Having absolute time stamps when diffing the output

[PATCH 0/2] scripts: show_delta: additional features

2017-04-22 Thread Erik Stromdahl
This patch series adds two new features to the show_delta script: - rounding - delta time stamp only output The purpose of the patches is to make it easier to diff show_delta- processed kernel logs when debugging timing problems. Rounding of the delta time stamps removes small (perhaps random) di

Re: [HMM 03/15] mm/unaddressable-memory: new type of ZONE_DEVICE for unaddressable memory

2017-04-22 Thread Jerome Glisse
On Fri, Apr 21, 2017 at 10:30:01PM -0700, Dan Williams wrote: > On Fri, Apr 21, 2017 at 8:30 PM, Jérôme Glisse wrote: [...] > > +/* > > + * Specialize ZONE_DEVICE memory into multiple types each having differents > > + * usage. > > + * > > + * MEMORY_DEVICE_PERSISTENT: > > + * Persistent device

[PATCH] usb: core: Warn if an URB's transfer_buffer is on stack

2017-04-22 Thread Florian Fainelli
We see a large number of fixes to several drivers to remove the usage of on-stack buffers feeding into USB transfer functions. Make it easier to spot the offenders by adding a warning in usb_start_wait_urb() for urb->transfer_buffer to be located on the stack. Signed-off-by: Florian Fainelli ---

Re: [RFC][PATCH tip/sched/core] sched/rt: Simplify the IPI rt balancing logic

2017-04-22 Thread Steven Rostedt
[ Here's a back port of my IPI simplified patch for 4.9.20-rt16. ] When a CPU lowers its priority (schedules out a high priority task for a lower priority one), a check is made to see if any other CPU has overloaded RT tasks (more than one). It checks the rto_mask to determine this and if so it

Re: [PATCH v2] net: natsemi: ns83820: add checks for dma mapping error

2017-04-22 Thread Francois Romieu
Alexey Khoroshilov : [...] > diff --git a/drivers/net/ethernet/natsemi/ns83820.c > b/drivers/net/ethernet/natsemi/ns83820.c > index 729095db3e08..dfc64e1e31f9 100644 > --- a/drivers/net/ethernet/natsemi/ns83820.c > +++ b/drivers/net/ethernet/natsemi/ns83820.c [...] > @@ -1183,6 +1193,32 @@ static

Re: [PATCH] drm: fourcc byteorder: brings header file comments in line with reality.

2017-04-22 Thread Ilia Mirkin
On Sat, Apr 22, 2017 at 9:48 AM, Ilia Mirkin wrote: > On Sat, Apr 22, 2017 at 9:40 AM, Ilia Mirkin wrote: >> On Sat, Apr 22, 2017 at 5:50 AM, Ville Syrjälä >> wrote: >>> On Sat, Apr 22, 2017 at 01:07:57AM -0400, Ilia Mirkin wrote: On Fri, Apr 21, 2017 at 12:59 PM, Ville Syrjälä wrote:

[PATCH] uapi: remove false assertion that statx_timestamp.tv_nsec can be negative

2017-04-22 Thread Dmitry V. Levin
The comment asserting that the value of struct statx_timestamp.tv_nsec must be negative when statx_timestamp.tv_sec is negative, is wrong, as could be seen from the following example: #define _FILE_OFFSET_BITS 64 #include #include #include #include

Re: [PATCH v3 2/2] modules:capabilities: add a per-task modules autoload restriction

2017-04-22 Thread Kees Cook
On Fri, Apr 21, 2017 at 11:51 PM, Andy Lutomirski wrote: > On Fri, Apr 21, 2017 at 5:12 PM, Djalal Harouni wrote: >> On Sat, Apr 22, 2017 at 1:51 AM, Andy Lutomirski wrote: >> [...] > I personally like my implicit_rights idea, and it might be interesting > to prototype it. I do

Re: [PATCH V5 4/7] ARM: pxa: Use - instead of @ for DT OPP entries

2017-04-22 Thread Robert Jarzmik
Robert Jarzmik writes: > Viresh Kumar writes: > >> On 20-04-17, 22:14, Robert Jarzmik wrote: >>> Viresh Kumar writes: >>> >>> > Compiling the DT file with W=1, DTC warns like follows: >>> > >>> > Warning (unit_address_vs_reg): Node /opp_table0/opp@10 has a >>> > unit name, but no reg p

Re: [PATCH] make TIOCSTI ioctl require CAP_SYS_ADMIN

2017-04-22 Thread Serge E. Hallyn
Quoting Matt Brown (m...@nmatt.com): > On 04/21/2017 01:24 AM, Serge E. Hallyn wrote: > >On Fri, Apr 21, 2017 at 01:09:59AM -0400, Matt Brown wrote: > >>On 04/20/2017 01:41 PM, Serge E. Hallyn wrote: > >>>Quoting m...@nmatt.com (m...@nmatt.com): > On 2017-04-20 11:19, Serge E. Hallyn wrote: > >

Re: bug fix for registers debugfs file implementation [RFC]

2017-04-22 Thread Variksla
> On Apr 21, 2017, at 10:27 AM, Mark Brown wrote: Thanks for the response. > >> On Sat, Apr 01, 2017 at 02:13:41AM -0700, noman pouigt wrote: >> Current registers debugfs file implementation doesn't >> handle if the size exceeds 4k. It just dumps 4k of registers. >> Fix this by using the seq_f

[PATCH 0/2] dmaengine: zx: Fine-tuning for two function implementations

2017-04-22 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 22 Apr 2017 22:23:45 +0200 Two update suggestions were taken into account from static source code analysis. Markus Elfring (2): Use devm_kcalloc() in zx_dma_probe() Fix a typo in a comment line drivers/dma/zx_dma.c | 10 +- 1 file changed, 5 insertio

[PATCH 1/2] dmaengine: zx: Use devm_kcalloc() in zx_dma_probe()

2017-04-22 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 22 Apr 2017 22:00:10 +0200 * Multiplications for the size determination of memory allocations indicated that array data structures should be processed. Thus use the corresponding function "devm_kcalloc". This issue was detected by using the Coccinelle softwa

[PATCH 2/2] dmaengine: zx: Fix a typo in a comment line

2017-04-22 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 22 Apr 2017 22:17:11 +0200 Add a missing character in this description for a function call. Signed-off-by: Markus Elfring --- drivers/dma/zx_dma.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/dma/zx_dma.c b/drivers/dma/zx_dma.c ind

Re: Performance of low-cpu utilisation benchmark regressed severely since 4.6

2017-04-22 Thread Rafael J. Wysocki
On Friday, April 21, 2017 11:29:06 PM Doug Smythies wrote: > On 2017.04.20 18:18 Rafael wrote: > > On Thursday, April 20, 2017 07:55:57 AM Doug Smythies wrote: > >> On 2017.04.19 01:16 Mel Gorman wrote: > >>> On Fri, Apr 14, 2017 at 04:01:40PM -0700, Doug Smythies wrote: > Hi Mel, > > > > [cut

[PATCH 0/2] dmaengine: s3c24xx: Fine-tuning for s3c24xx_dma_probe()

2017-04-22 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 22 Apr 2017 23:14:32 +0200 Two update suggestions were taken into account from static source code analysis. Markus Elfring (2): Use devm_kcalloc() in s3c24xx_dma_probe() Fix a typo in a comment line drivers/dma/s3c24xx-dma.c | 10 +- 1 file changed,

[PATCH 1/2] dmaengine: s3c24xx: Use devm_kcalloc() in s3c24xx_dma_probe()

2017-04-22 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 22 Apr 2017 23:00:23 +0200 * A multiplication for the size determination of a memory allocation indicated that an array data structure should be processed. Thus use the corresponding function "devm_kcalloc". * Replace the specification of a data structure by a

[PATCH 2/2] dmaengine: s3c24xx: Fix a typo in a comment line

2017-04-22 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 22 Apr 2017 23:09:03 +0200 Add a missing character in this description for a data structure member. Signed-off-by: Markus Elfring --- drivers/dma/s3c24xx-dma.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/dma/s3c24xx-dma.c b/driver

Re: [PATCH] usb: core: Warn if an URB's transfer_buffer is on stack

2017-04-22 Thread Alan Stern
On Sat, 22 Apr 2017, Florian Fainelli wrote: > We see a large number of fixes to several drivers to remove the usage of > on-stack buffers feeding into USB transfer functions. Make it easier to spot > the offenders by adding a warning in usb_start_wait_urb() for > urb->transfer_buffer to be locate

Re: [PATCH] drm: fourcc byteorder: brings header file comments in line with reality.

2017-04-22 Thread Gerd Hoffmann
Hi, > > I guess that makes changing drm_mode_legacy_fb_format + drivers a > > reasonable option ... > > Yeah, I came to the same conclusion after chatting with some > folks on irc. > > So my current idea is that we change any driver that wants to follow the > CPU endianness to declare support

[PATCH] staging: rtl8192u: Improve readability and fix sparse warnings: cast from restricted __le16

2017-04-22 Thread Tuomo Rinne
This patch fixes following sparse warnings: drivers/staging/rtl8192u//r8192U_dm.c:2307:49: warning: cast from restricted __le16 drivers/staging/rtl8192u//r8192U_dm.c:2308:44: warning: cast from restricted __le16 drivers/staging/rtl8192u//r8192U_dm.c:2309:44: warning: cast from restricted __le16

Re: [PATCH] staging: fbtft: fix character limit, trailing ; warning, etc.

2017-04-22 Thread Joe Perches
On Sat, 2017-04-22 at 16:26 +0100, Eric Curtin wrote: > checkpatch spits out a warning about the 80 character line limit. Split > the parameters of these functions onto different lines. Put the ; with > the macro caller instead. Lined up parameters as there was another > CHECK warning about that. [

GEWONNEN

2017-04-22 Thread HEIDI MEIER
Euro Lotto s.l Calle Nicaragua 4a 28992 Leganes Madrid -Spain Attn: Gewinner Ihre E-Mail-Adresse hat die Summe von €1,200.000.00 Euro gewonnen. Im Email Benützer Online Programm Weihnachtslotterie Navida , in Madrid Spanien . Wir schreiben ihnen ,um sich offiziel über die Auszeichnung zu Ben

  1   2   >