Re: [PATCH v05 61/72] include/uapi/rdma/rdma_user_rxe.h: include in.h and in6.h

2016-08-23 Thread Doug Ledford
On 8/22/2016 2:33 PM, Mikko Rapeli wrote: > Fixes these userspace compilation errors when rdma_user_rxe.h is compiled > alone: > > rdma/rdma_user_rxe.h:59:20: error: field ‘_sockaddr’ has incomplete type >struct sockaddr _sockaddr; > ^ >

Re: [PATCH 07/13] scpi: ignore init_versions failure if reported not supported

2016-08-23 Thread Neil Armstrong
On 08/23/2016 04:54 PM, Sudeep Holla wrote: > > > On 23/08/16 09:23, Neil Armstrong wrote: >> On 08/19/2016 06:46 PM, Sudeep Holla wrote: >>> >>> >>> On 18/08/16 11:11, Neil Armstrong wrote: In Amlogic GXBB Legacy SCPI, the LEGACY_SCPI_CMD_SCPI_CAPABILITIES report as SCPI_ERR_SUPPORT,

[PATCH v4] clocksource/drivers/pxa: fix include files for compilation

2016-08-23 Thread Baoyou Xie
We get 1 warning about global functions without a declaration in the clocksource/drivers/pxa driver when building with W=1: drivers/clocksource/pxa_timer.c:221:13: warning: no previous prototype for 'pxa_timer_nodt_init' [-Wmissing-prototypes] void __init pxa_timer_nodt_init(int irq, void

Re: musb: am3358: having problem with high-speed on usb1 at peripheral

2016-08-23 Thread Bin Liu
Hi, On Tue, Aug 23, 2016 at 09:57:17AM +0800, Ayaka wrote: > > > 從我的 iPad 傳送 > > > Bin Liu 於 2016年8月19日 上午12:08 寫道: > > > > Hi, > > > >> On Fri, Aug 12, 2016 at 10:23:15AM +0800, ayaka wrote: > >> Hello all: > >> I recently add a support for customize am3358 board using the

[PATCH net-next 5/5] rxrpc: Use a tracepoint for skb accounting debugging

2016-08-23 Thread David Howells
Use a tracepoint to log various skb accounting points to help in debugging refcounting errors. Signed-off-by: David Howells --- include/trace/events/rxrpc.h | 56 ++ net/rxrpc/af_rxrpc.c |1 + net/rxrpc/ar-internal.h |

[PATCH -next] ASoC: samsung: i2s: Add missing clk_disable_unprepare() on error in samsung_i2s_probe()

2016-08-23 Thread Wei Yongjun
Add the missing clk_disable_unprepare() before return from samsung_i2s_probe() in the error handling case. Signed-off-by: Wei Yongjun --- sound/soc/samsung/i2s.c | 8 ++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/sound/soc/samsung/i2s.c

Re: ptrace group stop signal number not reset before PTRACE_INTERRUPT is delivered?

2016-08-23 Thread Oleg Nesterov
On 08/18, Keno Fischer wrote: > > On Thu, Aug 18, 2016 at 12:23 PM, Oleg Nesterov wrote: > > > > And you if you get PTRACE_EVENT_STOP and WSTOPSIG() == SIGTTIN after > > PTRACE_INTERRUPT, you know that the tracee did not report the "new" > > SIGTTIN. > > It seems possible to

Re: UBSAN: Undefined behaviour in linux-4.7.2/drivers/usb/core/devio.c:1713:25

2016-08-23 Thread Vittorio Zecca
After applying the patch above the UBSAN issue in devio.c disappeared. However, I got the following messages in dmesg, probably due to a NetworkManager malfunctioning, and if I click on the NetworkManager icon I get "NetworkManager is not running" but maybe this is another issue and your patch

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

2016-08-23 Thread Davidlohr Bueso
What's the motivation here? Is it just to unify counter and owner for the starvation issue? If so, is this really the path we wanna take for a small debug corner case? I have not looked at the patches yet, but are there any performance minutia to be aware of? 46 files changed, 160

[PATCH 1/3] x86: branchless clear_page()

2016-08-23 Thread Alexey Dobriyan
Apply alternatives at the call site instead of function body. Save branch per clean page. Bonus: tell gcc to not flush whole shebang of registers, just RDI, RAX, RCX. Rename individual clear_page() functions closer to implementation details. Signed-off-by: Alexey Dobriyan

Re: [PATCH -next] ARM: pxa: remove duplicated include from spitz.c

2016-08-23 Thread Robert Jarzmik
Wei Yongjun writes: > Remove duplicated include. How so duplicated ? Can you elaborate please ? Moreover, how do you do think symbol_get() can stay in spitz.c without having this include ? Cheers. -- Robert

Re: [PATCH 1/1] printk: fix parsing of "brl=" option

2016-08-23 Thread Joe Perches
On Tue, 2016-08-23 at 17:47 +0200, Nicolas Iooss wrote: > Commit bbeddf52adc1 ("printk: move braille console support into separate > braille.[ch] files") moved the parsing of braille-related options into > _braille_console_setup(), changing the type of variable str from char* > to char**. In this

[PATCH 2/3] x86: support REP MOVSB copy_page()

2016-08-23 Thread Alexey Dobriyan
Microbenchmark shows that "REP MOVSB" copy_page() is faster than "REP MOVSQ" version on Intel i5-something Haswell REP_GOOD/ERMS capable CPU. N=1<<27 rep movsq: 6.758841901 ± 0.04% rep movsb: 6.253927309 ± 0.02% --- -7.5%

Re: [alsa-devel] [RFC PATCH] mfd: arizona: Add gating of external MCLKn clocks

2016-08-23 Thread Sylwester Nawrocki
On 08/22/2016 07:41 PM, Mark Brown wrote: > On Mon, Aug 22, 2016 at 07:22:47PM +0200, Sylwester Nawrocki wrote: > >> If we provided an interface like: > >> struct clk * arizona_get_mclk(struct arizona *arizona, int id); >> void arizona_put_mclk(struct clk *clk); > >> the machine driver would

[PATCH] crypto: rockchip - use devm_add_action_or_reset()

2016-08-23 Thread Sudip Mukherjee
If devm_add_action() fails we are explicitly calling the cleanup to free the resources allocated. Lets use the helper devm_add_action_or_reset() and return directly in case of error, as we know that the cleanup function has been already called by the helper if there was any error. Signed-off-by:

[PATCH net-next 2/5] rxrpc: Tidy up the rxrpc_call struct a bit

2016-08-23 Thread David Howells
Do a little tidying of the rxrpc_call struct: (1) in_clientflag is no longer compared against the value that's in the packet, so keeping it in this form isn't necessary. Use a flag in flags instead and provide a pair of wrapper functions. (2) We don't read the epoch value, so that

[PATCH net-next 4/5] rxrpc: Drop channel number field from rxrpc_call struct

2016-08-23 Thread David Howells
Drop the channel number (channel) field from the rxrpc_call struct to reduce the size of the call struct. The field is redundant: if the call is attached to a connection, the channel can be obtained from there by AND'ing with RXRPC_CHANNELMASK. Signed-off-by: David Howells

[PATCH net-next 3/5] rxrpc: When clearing a socket, clear the call sets in the right order

2016-08-23 Thread David Howells
When clearing a socket, we should clear the securing-in-progress list first, then the accept queue and last the main call tree because that's the order in which a call progresses. Not that a call should move from the accept queue to the main tree whilst we're shutting down a socket, but it a call

Re: [PATCH v4 54/57] x86/mm: convert arch_within_stack_frames() to use the new unwinder

2016-08-23 Thread Josh Poimboeuf
On Mon, Aug 22, 2016 at 03:11:32PM -0700, Linus Torvalds wrote: > On Thu, Aug 18, 2016 at 6:06 AM, Josh Poimboeuf wrote: > > Convert arch_within_stack_frames() to use the new unwinder. > > Please don't do this. > > There's no real reason to unwind the stack frame. If it's

Re: [v.1] clk:clk-rockchip:-of_iomap failed to remap then abort clock initialization.

2016-08-23 Thread Heiko Stübner
Hi Arvind, Am Samstag, 13. August 2016, 20:56:18 schrieb Arvind Yadav: > -check return of 'of_iomap'.if It's falied to remap then abort. > > -Unmap a region obtained by remap. iounmap frees the mapping when failed > to allocate the memory. > > Signed-off-by: Arvind Yadav

Re: [alsa-devel] [RFC PATCH] mfd: arizona: Add gating of external MCLKn clocks

2016-08-23 Thread Charles Keepax
On Mon, Aug 22, 2016 at 07:22:47PM +0200, Sylwester Nawrocki wrote: > On 08/22/2016 11:22 AM, Charles Keepax wrote: > > Yeah I am not sure this is quite the correct approach, there are > > quite a few corner cases that would not be covered well here. For > > example an internally divided down 32k

[PATCH 3/3] x86: branchless copy_page()

2016-08-23 Thread Alexey Dobriyan
Apply alternatives at the call site instead of function body. Save branch per copied page. Rename individual function, show immediately in profiles which method is being used. RBX and R12 are saved and restored by unrolled-mov implementation, don't clobber them explicitly. Signed-off-by: Alexey

Re: [PATCH v2] kernel/fork: fix CLONE_CHILD_CLEARTID regression in nscd

2016-08-23 Thread Oleg Nesterov
On 08/23, Michal Hocko wrote: > > --- a/kernel/fork.c > +++ b/kernel/fork.c > @@ -913,14 +913,11 @@ void mm_release(struct task_struct *tsk, struct > mm_struct *mm) > deactivate_mm(tsk, mm); > > /* > - * If we're exiting normally, clear a user-space tid field if > - *

Re: [PATCH v2 2/2] Documentation/sphinx: link dma-buf rsts

2016-08-23 Thread Mauro Carvalho Chehab
Em Tue, 23 Aug 2016 08:16:33 -0600 Jonathan Corbet escreveu: > On Tue, 23 Aug 2016 15:28:55 +0200 > Daniel Vetter wrote: > > > I think the more interesting story is, what's your plan with all the > > other driver related subsystem? Especially the ones which

Re: [PATCH] fs/seq_file: fix out-of-bounds read

2016-08-23 Thread Vegard Nossum
On 08/17/2016 05:21 PM, Vegard Nossum wrote: seq_read() is a nasty piece of work, not to mention buggy. I was getting these: BUG: KASAN: slab-out-of-bounds in seq_read+0xcd2/0x1480 at addr 880116889880 Read of size 2713 by task trinity-c2/1329 CPU: 2 PID: 1329 Comm: trinity-c2

Re: perf: WARNING: kernel/events/core.c:4893 perf_mmap_close

2016-08-23 Thread Alexander Shishkin
Will Deacon writes: > Hi Alexander, > > On Fri, Aug 12, 2016 at 08:54:49PM +0300, Alexander Shishkin wrote: >> Yes, I tracked to a race between unmapping and set_output, trying to >> come up with a good fix now. > > Did you get anywhere with this? I think I just hit the same

[PATCH] x86/CPU: Remove outdated comment about SYSCALL targets

2016-08-23 Thread Borislav Petkov
From: Borislav Petkov The comment probably meant some old AMD64 incarnation which most likely never saw the light of day. STAR and LSTAR are two different registers and STAR sets CS/SS(DS) selectors for *all* modes, not only 32-bit. So simply remove that comment. Signed-off-by:

Re: [PATCH v2 3/8] pinctrl: dt-bindings: Add documentation for Aspeed pin controllers

2016-08-23 Thread Rob Herring
On Fri, Aug 19, 2016 at 10:14:09PM +0930, Andrew Jeffery wrote: > Outline expectations on the pin controller's relationship with the > System Control Unit (SCU) IP through syscon, and document the compatible > strings for 4th and 5th generation Aspeed SoC pin controllers. > > Signed-off-by:

[PATCH 1/1] usb: usbip: vudc: fix left shift overflow

2016-08-23 Thread Nicolas Iooss
In v_recv_cmd_submit(), urb_p->urb->pipe has the type unsigned int (which is 32-bit long on x86_64) but 11<<30 results in a 34-bit integer. Therefore the 2 leading bits are truncated and urb_p->urb->pipe &= ~(11 << 30); has the same meaning as urb_p->urb->pipe &= ~(3 << 30); This

Re: [PATCH] DW: Read "is_memcpy" and "is_nollp" property from device tree.

2016-08-23 Thread Eugeniy Paltsev
On Fri, 2016-08-19 at 17:39 +0300, Andy Shevchenko wrote: > On Tue, 2016-08-16 at 14:31 +0300, Eugeniy Paltsev wrote: > > > > DW DMAC on ARC SDP became broken after df5c7386 ("dmaengine: dw: > > some > > Intel > > devices has no memcpy support") and 30cb2639 ("dmaengine: dw: don't > > override >

Re: [PATCH v2 2/2] Documentation/sphinx: link dma-buf rsts

2016-08-23 Thread Daniel Vetter
On Tue, Aug 23, 2016 at 08:16:33AM -0600, Jonathan Corbet wrote: > On Tue, 23 Aug 2016 15:28:55 +0200 > Daniel Vetter wrote: > > > I think the more interesting story is, what's your plan with all the > > other driver related subsystem? Especially the ones which already have > >

Re: [PATCH v2 2/4] remoteproc: Introduce Qualcomm ADSP PIL

2016-08-23 Thread Stanimir Varbanov
Hi Bjorn, On 08/23/2016 08:57 AM, Bjorn Andersson wrote: > From: Bjorn Andersson > > The Qualcomm ADSP Peripheral Image Loader is used on a variety of > different Qualcomm platforms for loading firmware into and controlling > the Hexagon based ADSP. > >

Re: mm: kswapd struggles reclaiming the pages on 64GB server

2016-08-23 Thread Andriy Tkachuk
Well, as appeared - the patch did not affect the problem at all since the memory cgroup was on (in which case zone's inactive_ratio is not used, but the ratio is calculated directly at mem_cgroup_inactive_anon_is_low()). So the patch will be retested with memory cgroup off. Andriy On Mon, Aug

[PATCH net-next 2/3] rxrpc: Calculate serial skew on packet reception

2016-08-23 Thread David Howells
Calculate the serial number skew in the data_ready handler when a packet has been received and a connection looked up. The skew is cached in the sk_buff's priority field. The connection highest received serial number is updated at this time also. This can be done without locks or atomic

Re: net: Zeroing the structure ethtool_wolinfo in ethtool_get_wol()

2016-08-23 Thread Joe Perches
On Tue, 2016-08-23 at 19:27 +0400, Loganaden Velvindron wrote: > Better be safe than sorry. Better still would be to create a tool via something like coccinelle that could be run on the entire kernel than submit a single patch for a construct that likely occurs dozens of times in the kernel

[PATCH net-next 3/3] rxrpc: Perform terminal call ACK/ABORT retransmission from conn processor

2016-08-23 Thread David Howells
Perform terminal call ACK/ABORT retransmission in the connection processor rather than in the call processor. With this change, once last_call is set, no more incoming packets will be routed to the corresponding call or any earlier calls on that channel (call IDs must only increase on a channel

Re: [PATCH] drm/bridge: dw-hdmi: Delete unnecessary assignment for the field "owner"

2016-08-23 Thread Sean Paul
On Tue, Aug 16, 2016 at 7:58 AM, SF Markus Elfring wrote: > From: Markus Elfring > Date: Tue, 16 Aug 2016 13:52:19 +0200 > > The field "owner" is set by the core. > Thus delete an unneeded initialisation. > > Generated by:

Re: [PATCH v1 00/10] Optimize sched avgs computation and implement flat util hierarchy

2016-08-23 Thread Dietmar Eggemann
On 23/08/16 15:45, Vincent Guittot wrote: > On 23 August 2016 at 16:13, Peter Zijlstra wrote: >> On Tue, Aug 23, 2016 at 03:28:19PM +0200, Vincent Guittot wrote: >>> I still wonder if using a flat util hierarchy is the right solution to >>> solve this problem with

[PATCH v2] kernel/fork: fix CLONE_CHILD_CLEARTID regression in nscd

2016-08-23 Thread Michal Hocko
From: Michal Hocko fec1d0115240 ("[PATCH] Disable CLONE_CHILD_CLEARTID for abnormal exit") has caused a subtle regression in nscd which uses CLONE_CHILD_CLEARTID to clear the nscd_certainly_running flag in the shared databases, so that the clients are notified when nscd is

Re: [PATCH] gpu: ipu-v3: fix a possible NULL dereference

2016-08-23 Thread Corentin LABBE
On 23/08/2016 17:24, Sean Paul wrote: > On Tue, Aug 16, 2016 at 9:33 AM, LABBE Corentin > wrote: >> of_match_device could return NULL, and so cause a NULL pointer >> dereference later. >> >> For fixing this problem, we use of_device_get_match_data(), this will >>

Re: [PATCH] IB/qib: Use memdup_user() rather than duplicating its implementation

2016-08-23 Thread Doug Ledford
On 8/19/2016 11:27 AM, Marciniszyn, Mike wrote: >> Subject: [PATCH] IB/qib: Use memdup_user() rather than duplicating its >> diff --git a/drivers/infiniband/hw/qib/qib_fs.c > > I would be even more aggressive at reducing lines of code. > > For example do direct returns when ok to do: >

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

2016-08-23 Thread Daniel Vetter
On Tue, Aug 23, 2016 at 11:56:14AM +1000, Stephen Rothwell wrote: > Hi all, > > After merging the drm-intel tree, today's linux-next build (arm > multi_v7_defconfig) failed like this: > > In file included from drivers/gpio/gpiolib-of.c:19:0: > include/linux/io-mapping.h:115:31: fatal error:

Re: [PATCH -next] ARM: pxa: remove duplicated include from spitz.c

2016-08-23 Thread Robert Jarzmik
Wei Yongjun writes: > Hi Robert, > On 08/24/2016 12:24 AM, Robert Jarzmik wrote: >> Wei Yongjun writes: >> >>> Remove duplicated include. >> How so duplicated ? Can you elaborate please ? >> >> Moreover, how do you do think symbol_get() can stay in

[PATCH 2/3] usb: gadget: f_hid: use free_ep_req()

2016-08-23 Thread Felipe F. Tonello
We should always use free_ep_req() when allocating requests with alloc_ep_req(). Signed-off-by: Felipe F. Tonello --- drivers/usb/gadget/function/f_hid.c | 10 +++--- 1 file changed, 3 insertions(+), 7 deletions(-) diff --git a/drivers/usb/gadget/function/f_hid.c

[PATCH 3/3] usb: gadget: f_hid: use alloc_ep_req()

2016-08-23 Thread Felipe F. Tonello
Use gadget's framework allocation function instead of directly calling usb_ep_alloc_request(). Signed-off-by: Felipe F. Tonello --- drivers/usb/gadget/function/f_hid.c | 6 +- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git

[PATCH 1/3] usb: gadget: remove useless parameter in alloc_ep_req()

2016-08-23 Thread Felipe F. Tonello
The default_length parameter of alloc_ep_req was not really necessary and gadget drivers would almost always create an inline function to pass the same value to len and default_len. This patch removes that parameter and updates all calls to alloc_ep_req() to use the new API. Signed-off-by:

Re: [RFC][PATCH 7/7] cpufreq: intel_pstate: Change P-state selection algorithm for Core

2016-08-23 Thread Rafael J. Wysocki
On Tuesday, August 23, 2016 12:50:14 PM Wanpeng Li wrote: > 2016-08-23 12:08 GMT+08:00 Srinivas Pandruvada > : > > On Tue, 2016-08-23 at 11:48 +0800, Wanpeng Li wrote: > >> Hi Rafael, > >> 2016-08-01 7:38 GMT+08:00 Rafael J. Wysocki : > >> >

Re: [RFC RESEND] serial: 8250: fix regression in 8250 uart driver

2016-08-23 Thread Andy Shevchenko
On Tue, 2016-08-23 at 08:53 -0500, Dinh Nguyen wrote: > Hi Andy, > > On 08/17/2016 06:14 AM, Andy Shevchenko wrote: > > > > > > I sent a v2 of the series for internal review, same you may found on > > [1]. If Heikki is okay to that I'll send it here. > > > > [1]

Re: [PATCHv2 1/2] mm: introduce get_task_exe_file

2016-08-23 Thread Mateusz Guzik
On Tue, Aug 23, 2016 at 04:48:13PM +0200, Oleg Nesterov wrote: > On 08/23, Mateusz Guzik wrote: > > > > +struct file *get_task_exe_file(struct task_struct *task) > > +{ > > + struct file *exe_file = NULL; > > + struct mm_struct *mm; > > + > > + task_lock(task); > > + mm = task->mm; > > +

Re: UBSAN: Undefined behaviour in linux-4.7.2/drivers/usb/core/devio.c:1713:25

2016-08-23 Thread Alan Stern
On Mon, 22 Aug 2016, Vittorio Zecca wrote: > I can reproduce the UBSAN report by inserting in the USB receptacle a > HUAWEI Mobile Broadband E353 HSPA+ USB Stick. > If there is any patch I'll be happy to try it. Thank you. Please let us know what happens with the patch below. Alan Stern

Re: [PATCH] proc, smaps: reduce printing overhead

2016-08-23 Thread Michal Hocko
On Thu 18-08-16 13:31:28, Michal Hocko wrote: > From: Michal Hocko > > seq_printf (used by show_smap) can be pretty expensive when dumping a > lot of numbers. Say we would like to get Rss and Pss from a particular > process. In order to measure a pathological case let's

Re: [PATCH 4/7] fs: make remaining filesystems use .rename2

2016-08-23 Thread Boaz Harrosh
On 08/23/2016 07:24 PM, Boaz Harrosh wrote: > On 08/23/2016 05:05 PM, Miklos Szeredi wrote: >> This is trivial to do: >> >> - add flags argument to foo_rename() >> - check if flags is zero >> - assign foo_rename() to .rename2 instead of .rename >> >> This doesn't mean it's impossible to support

[PATCH] ath10k: replace config_enabled() with IS_REACHABLE()

2016-08-23 Thread Masahiro Yamada
Commit 97f2645f358b ("tree-wide: replace config_enabled() with IS_ENABLED()") mostly did away with config_enabled(). This is one of the postponed TODO items as config_enabled() is used for a tristate option here. Theoretically, config_enabled() is equivalent to IS_BUILTIN(), but I guess

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

2016-08-23 Thread Jason Low
On Tue, 2016-08-23 at 09:17 -0700, Davidlohr Bueso wrote: > What's the motivation here? Is it just to unify counter and owner for > the starvation issue? If so, is this really the path we wanna take for > a small debug corner case? And we thought our other patch was a bit invasive :-) > I have

[PATCH] treewide: replace config_enabled() with IS_ENABLED() (2nd round)

2016-08-23 Thread Masahiro Yamada
Commit 97f2645f358b ("tree-wide: replace config_enabled() with IS_ENABLED()") mostly killed config_enabled(), but some new users have appeared for v4.8-rc1. They are all used for a boolean option, so can be replaced with IS_ENABLED() safely. Signed-off-by: Masahiro Yamada

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

2016-08-23 Thread Peter Zijlstra
On Tue, Aug 23, 2016 at 09:35:03AM -0700, Jason Low wrote: > On Tue, 2016-08-23 at 09:17 -0700, Davidlohr Bueso wrote: > > What's the motivation here? Is it just to unify counter and owner for > > the starvation issue? If so, is this really the path we wanna take for > > a small debug corner case?

[PATCH v2 1/1] printk: fix parsing of "brl=" option

2016-08-23 Thread Nicolas Iooss
Commit bbeddf52adc1 ("printk: move braille console support into separate braille.[ch] files") moved the parsing of braille-related options into _braille_console_setup(), changing the type of variable str from char* to char**. In this commit, memcmp(str, "brl,", 4) was correctly updated to

Re: [PATCH 5/6] f2fs: enable inline_dentry by default

2016-08-23 Thread Jaegeuk Kim
Hi Chao, On Mon, Aug 22, 2016 at 09:49:13AM +0800, Chao Yu wrote: > Hi Jaegeuk, > > On 2016/5/10 7:04, Jaegeuk Kim wrote: > > On Mon, May 09, 2016 at 07:56:34PM +0800, Chao Yu wrote: > >> Make inline_dentry as default mount option to improve space usage and > >> IO performance in scenario of

Re: [PATCH 07/13] scpi: ignore init_versions failure if reported not supported

2016-08-23 Thread Sudeep Holla
On 23/08/16 15:55, Neil Armstrong wrote: On 08/23/2016 04:54 PM, Sudeep Holla wrote: On 23/08/16 09:23, Neil Armstrong wrote: On 08/19/2016 06:46 PM, Sudeep Holla wrote: On 18/08/16 11:11, Neil Armstrong wrote: In Amlogic GXBB Legacy SCPI, the LEGACY_SCPI_CMD_SCPI_CAPABILITIES report

[PATCH -next] f2fs: fix non static symbol warning

2016-08-23 Thread Wei Yongjun
From: Wei Yongjun Fixes the following sparse warning: fs/f2fs/data.c:969:12: warning: symbol 'f2fs_grab_bio' was not declared. Should it be static? Signed-off-by: Wei Yongjun --- fs/f2fs/data.c | 4 ++-- 1 file changed, 2 insertions(+), 2

Re: [PATCH] gpu: ipu-v3: fix a possible NULL dereference

2016-08-23 Thread Sean Paul
On Tue, Aug 16, 2016 at 9:33 AM, LABBE Corentin wrote: > of_match_device could return NULL, and so cause a NULL pointer > dereference later. > > For fixing this problem, we use of_device_get_match_data(), this will > simplify the code a little by using a standard

Re: [PATCH v2 1/2] input: misc: Add generic input driver to read encoded GPIO lines

2016-08-23 Thread Andrew F. Davis
On 08/23/2016 01:16 AM, Vignesh R wrote: > Add a driver to read group of GPIO lines and provide its status as a > numerical value as input event to the system. This will help in > intefacing devices, that can be connected over GPIOs, that provide input > to the system by driving GPIO lines

Re: [PATCH] crypto: rockchip - use devm_add_action_or_reset()

2016-08-23 Thread Heiko Stübner
Am Dienstag, 23. August 2016, 20:28:54 schrieb Sudip Mukherjee: > If devm_add_action() fails we are explicitly calling the cleanup to free > the resources allocated. Lets use the helper devm_add_action_or_reset() > and return directly in case of error, as we know that the cleanup function > has

[PATCH net-next] net: dsa: rename switch operations structure

2016-08-23 Thread Vivien Didelot
Now that the dsa_switch_driver structure contains only function pointers as it is supposed to, rename it to the more appropriate dsa_switch_ops, uniformly to any other operations structure in the kernel. No functional changes here, basically just the result of something like: s/dsa_switch_driver

Re: [RFC PATCH v2 0/7] AC97 device/driver model revamp

2016-08-23 Thread Robert Jarzmik
Robert Jarzmik writes: > It all started in the pxa device-tree submission here : >https://lkml.org/lkml/2016/2/25/965 > > It will be maintained in : >git fetch https://github.com/rjarzmik/linux.git work/ac97 > > And now it transformed into this RFC, which

Re: CVE-2014-9900 fix is not upstream

2016-08-23 Thread David Miller
From: Luis Henriques Date: Tue, 23 Aug 2016 14:41:07 +0100 > Digging through some old CVEs I came across this one that doesn't seem be > in mainline. Was there a good reason for not being sent upstream? Maybe it > was > rejected for some reason and I failed to

3.0.101: "blk_rq_check_limits: over max size limit."

2016-08-23 Thread Ulrich Windl
Hello! While performance-testing a 3PARdata StorServ 8400 with SLES11SP4, I noticed that I/Os dropped, until everything stood still more or less. Looking into the syslog I found that multipath's TUR-checker considered the paths (FC, BTW) as dead. Amazingly I did not have this problem when I

Re: [RFC PATCH] kernel/fork: fix CLONE_CHILD_CLEARTID regression in nscd

2016-08-23 Thread Oleg Nesterov
On 08/19, Michal Hocko wrote: > > On Fri 12-08-16 11:41:13, Michal Hocko wrote: > > On Wed 03-08-16 23:08:04, Oleg Nesterov wrote: > > > > > > So if we want this change, why we can't simply do > > > > > > - if (!(tsk->flags & PF_SIGNALED) && > > > + if (!(tsk->signal->flags &

Re: net: Zeroing the structure ethtool_wolinfo in ethtool_get_wol()

2016-08-23 Thread Joe Perches
On Tue, 2016-08-23 at 07:21 -0700, Eric Dumazet wrote: > On Tue, 2016-08-23 at 14:41 +0100, Luis Henriques wrote: > > From: Avijit Kanti Das > > > > memset() the structure ethtool_wolinfo that has padded bytes > > but the padded bytes have not been zeroed out. [] > >

[PATCH net-next 1/3] rxrpc: Set connection expiry on idle, not put

2016-08-23 Thread David Howells
Set the connection expiry time when a connection becomes idle rather than doing this in rxrpc_put_connection(). This makes the put path more efficient (it is likely to be called occasionally whilst a connection has outstanding calls because active workqueue items needs to be given a ref). The

what is the purpose of SLAB and SLUB (was: Re: [PATCH v3] mm/slab: Improve performance of gathering slabinfo) stats

2016-08-23 Thread Michal Hocko
On Tue 23-08-16 11:13:03, Joonsoo Kim wrote: > On Thu, Aug 18, 2016 at 01:52:19PM +0200, Michal Hocko wrote: [...] > > I am not opposing the patch (to be honest it is quite neat) but this > > is buggering me for quite some time. Sorry for hijacking this email > > thread but I couldn't resist. Why

Re: [PATCH 1/2] drivers/clocksource/pistachio: fix memory corruption in init

2016-08-23 Thread Daniel Lezcano
On 08/17/2016 12:22 PM, Marcin Nowakowski wrote: > Driver init code incorrectly uses the block base address and as a result > clears clocksource structure's fields instead of the hardware registers. > > Commit 09a998201649 ("timekeeping: Lift clocksource cacheline > restriction") has changed the

Re: [PATCH RFC RESEND] Perf: lookup dwarf unwind stack info in debug file pointed by .gnu_debuglink

2016-08-23 Thread Matija Glavinic Pecotic
On 23/08/16 14:33, Matija Glavinic Pecotic wrote: > On 23/08/16 13:22, Jiri Olsa wrote: >> On Tue, Aug 23, 2016 at 07:09:18AM +0200, Matija Glavinic Pecotic wrote: >>> On 22/08/16 17:19, Jiri Olsa wrote: should you also set following? *offset = ofs dso->debug_frame_offset

Re: [PATCH v2 2/4] remoteproc: Introduce Qualcomm ADSP PIL

2016-08-23 Thread Bjorn Andersson
On Tue 23 Aug 08:14 PDT 2016, Stanimir Varbanov wrote: > Hi Bjorn, > > On 08/23/2016 08:57 AM, Bjorn Andersson wrote: > > From: Bjorn Andersson > > > > The Qualcomm ADSP Peripheral Image Loader is used on a variety of > > different Qualcomm platforms for loading

Re: net: Zeroing the structure ethtool_wolinfo in ethtool_get_wol()

2016-08-23 Thread Andrey Ryabinin
2016-08-23 18:36 GMT+03:00 Eric Dumazet : > On Tue, 2016-08-23 at 08:05 -0700, Joe Perches wrote: > >> A compiler does not have a standards based requirement to >> initialize arbitrary padding bytes. >> >> I believe gcc always does zero all padding anyway. > > I would not

Re: [PATCH] DW: Read "is_memcpy" and "is_nollp" property from device tree.

2016-08-23 Thread Vineet Gupta
On 08/23/2016 10:02 AM, Andy Shevchenko wrote: > On Tue, 2016-08-23 at 15:14 +, Eugeniy Paltsev wrote: > >> DW DMAC on ARC SDP became broken after df5c7386 ("dmaengine: dw: some Intel devices has no memcpy support") and 30cb2639 ("dmaengine: dw: don't override platform data with

Re: [PATCH v2 2/8] syscon: dt-bindings: Add documentation for Aspeed system control units

2016-08-23 Thread Rob Herring
On Fri, Aug 19, 2016 at 10:14:08PM +0930, Andrew Jeffery wrote: > Signed-off-by: Andrew Jeffery > --- > > Since v1: > > * Add SoC-specific compatible strings > > Documentation/devicetree/bindings/mfd/aspeed-scu.txt | 18 ++ > 1 file changed, 18 insertions(+) >

Re: [PATCH] drm/nouveau/tegra: Modify error handling

2016-08-23 Thread Alexandre Courbot
On Mon, Aug 15, 2016 at 6:54 AM, Amitoj Kaur Chawla wrote: > iommu_domain_alloc returns NULL on error so replace an incorrect > IS_ERR check with a NULL check. > > The Coccinelle semantic patch used to find this issue is as follows: > @@ > expression e; > statement S; > @@ >

[PATCH -next] ARM: pxa: remove duplicated include from corgi.c

2016-08-23 Thread Wei Yongjun
Remove duplicated include. Signed-off-by: Wei Yongjun --- arch/arm/mach-pxa/corgi.c | 1 - 1 file changed, 1 deletion(-) diff --git a/arch/arm/mach-pxa/corgi.c b/arch/arm/mach-pxa/corgi.c index 10bfdb1..183cd34 100644 --- a/arch/arm/mach-pxa/corgi.c +++

[PATCH -next] ARM: pxa: remove duplicated include from spitz.c

2016-08-23 Thread Wei Yongjun
Remove duplicated include. Signed-off-by: Wei Yongjun --- arch/arm/mach-pxa/spitz.c | 1 - 1 file changed, 1 deletion(-) diff --git a/arch/arm/mach-pxa/spitz.c b/arch/arm/mach-pxa/spitz.c index 2c150bf..67d66c7 100644 --- a/arch/arm/mach-pxa/spitz.c +++

Re: [PATCH 3/4] gpio: spear-spics: drop unused MODULE_ tags from non-modular code

2016-08-23 Thread Masahiro Yamada
2016-08-23 1:48 GMT+09:00 Paul Gortmaker : > The Kconfig currently controlling compilation of this code is: > > drivers/gpio/Kconfig:config GPIO_SPEAR_SPICS > drivers/gpio/Kconfig: bool "ST SPEAr13xx SPI Chip Select as GPIO support" > > ...meaning that it currently

[PATCH net-next 1/5] rxrpc: Remove RXRPC_CALL_PROC_BUSY

2016-08-23 Thread David Howells
Remove RXRPC_CALL_PROC_BUSY as work queue items are now 100% non-reentrant. Signed-off-by: David Howells --- net/rxrpc/ar-internal.h |1 - net/rxrpc/call_event.c |6 -- 2 files changed, 7 deletions(-) diff --git a/net/rxrpc/ar-internal.h

[PATCH net-next 0/5] rxrpc: Cleanups

2016-08-23 Thread David Howells
: git://git.kernel.org/pub/scm/linux/kernel/git/dhowells/linux-fs.git rxrpc-rewrite-20160823-1 David --- David Howells (5): rxrpc: Remove RXRPC_CALL_PROC_BUSY rxrpc: Tidy up the rxrpc_call struct a bit rxrpc: When clearing a socket, clear the call sets in the right order

Re: [PATCH 4/7] fs: make remaining filesystems use .rename2

2016-08-23 Thread David Howells
Miklos Szeredi wrote: > This is trivial to do: > > - add flags argument to foo_rename() > - check if flags is zero > - assign foo_rename() to .rename2 instead of .rename > > This doesn't mean it's impossible to support RENAME_NOREPLACE for these > filesystems, but it is

Re: [PATCH] drm/rockchip: Delete unnecessary assignment for the field "owner"

2016-08-23 Thread Sean Paul
On Tue, Aug 16, 2016 at 8:43 AM, SF Markus Elfring wrote: > From: Markus Elfring > Date: Tue, 16 Aug 2016 14:25:35 +0200 > > The field "owner" is set by the core. > Thus delete an unneeded initialisation. > > Generated by:

Re: [PATCH v7] usb: ohci-at91: Forcibly suspend ports while USB suspend

2016-08-23 Thread Alan Stern
On Tue, 23 Aug 2016, Wenyou Yang wrote: > The usb controller does not manage correctly the suspend mode for > the ehci. In echi mode, there is no way to suspend without any > device connected to it. This is why this specific control is added > to fix this issue. Since the suspend mode works in

Re: [PATCH v6 4/8] usb: core: add power sequence handling for USB devices

2016-08-23 Thread Alan Stern
On Tue, 23 Aug 2016, Peter Chen wrote: > I will add #ifdef CONFIG_OF for related code. And put the content at > hub_pwrseq_on at hub_probe directly, how about below? > > hub_probe() { > > ... > > if (hub_configure(hub, endpoint) >= 0) { > #ifdef CONFIG_OF >

[PATCH net-next 0/3] rxrpc: Miscellaneous improvements

2016-08-23 Thread David Howells
: http://git.kernel.org/cgit/linux/kernel/git/dhowells/linux-fs.git/log/?h=rxrpc-rewrite Tagged thusly: git://git.kernel.org/pub/scm/linux/kernel/git/dhowells/linux-fs.git rxrpc-rewrite-20160823-2 David --- David Howells (3): rxrpc: Set connection expiry on idle, not put

[PATCH 3/3] arm64: dts: xgene: Add DT node for APM X-Gene 2 CPU clocks

2016-08-23 Thread Hoan Tran
Add DT nodes to enable APM X-Gene 2 CPU clocks. Signed-off-by: Hoan Tran --- arch/arm64/boot/dts/apm/apm-shadowcat.dtsi | 64 ++ 1 file changed, 64 insertions(+) diff --git a/arch/arm64/boot/dts/apm/apm-shadowcat.dtsi

[PATCH 2/3] clk: xgene: Add PMD clock

2016-08-23 Thread Hoan Tran
Add X-Gene PMD clock support. PMD clock is implemented for a single register field. Output rate = parent_rate * (denominator - scale) / denominator with - denominator = bitmask of register field + 1 - scale = value of register field For example, for bitmask is 0x7, denominator will be 8

Re: ALSA: compress: Memset timestamp structure to zero.

2016-08-23 Thread Vinod Koul
On Tue, Aug 23, 2016 at 03:45:38PM +0200, Takashi Iwai wrote: > On Tue, 23 Aug 2016 15:40:37 +0200, > Luis Henriques wrote: > > > > From: Krishnankutty Kolathappilly > > > > snd_compr_tstamp is initialized using aggregate initialization > > that does not zero out the

[PATCH 1/3] Documentation: dtb: xgene: Add PMD clock binding

2016-08-23 Thread Hoan Tran
Add APM X-Gene clock binding documentation for PMD clock. Signed-off-by: Hoan Tran --- Documentation/devicetree/bindings/clock/xgene.txt | 22 ++ 1 file changed, 22 insertions(+) diff --git a/Documentation/devicetree/bindings/clock/xgene.txt

[PATCH 0/3] clk: xgene: Add PMD clock support

2016-08-23 Thread Hoan Tran
Add X-Gene PMD clock support. PMD clock is implemented for a single register field. Output rate = parent_rate * (denominator - scale) / denominator with - denominator = bitmask of register field + 1 - scale = value of register field For example, for bitmask is 0x7, denominator will be 8

Re: [PATCH v5] Force cppc_cpufreq to report values in KHz to fix user space reporting

2016-08-23 Thread Al Stone
On 08/22/2016 10:31 PM, Pandruvada, Srinivas wrote: > On Mon, 2016-08-22 at 12:12 -0600, Al Stone wrote: >> On 08/22/2016 11:45 AM, Ashwin Chaugule wrote: >>> >>> Hi Al, >>> >>> On Mon, Aug 22, 2016 at 10:16 AM, Al Stone wrote: Maybe a top-post will get attention

Re: [PATCH] x86: branchless clear_page()

2016-08-23 Thread Alexey Dobriyan
On Tue, Aug 23, 2016 at 09:43:25AM -0400, Brian Gerst wrote: > > -ENTRY(clear_page_c_e) > > +ENTRY(clear_page_rep_stosb) > > movl $4096,%ecx > > xorl %eax,%eax > > rep stosb > > ret > > -ENDPROC(clear_page_c_e) > > +ENDPROC(clear_page_rep_stosb) > > I like this

Re: [PATCH v4 54/57] x86/mm: convert arch_within_stack_frames() to use the new unwinder

2016-08-23 Thread Josh Poimboeuf
On Mon, Aug 22, 2016 at 06:27:28PM -0700, Kees Cook wrote: > On Mon, Aug 22, 2016 at 3:11 PM, Linus Torvalds > wrote: > > On Thu, Aug 18, 2016 at 6:06 AM, Josh Poimboeuf wrote: > >> Convert arch_within_stack_frames() to use the new unwinder. >

Re: [PATCH 10/16] cpuidle: pseries: Convert to hotplug state machine

2016-08-23 Thread Sebastian Andrzej Siewior
On 2016-08-23 16:16:12 [+0200], Daniel Lezcano wrote: > Yes. If we can limit the number of duplicating enum for the same purpose > right now, it would be nice. Okay. > > We could do that but you would have to ensure that only one CPUIDLE > > driver registers itself at a time and for those powerpc

Re: [PATCH] dmaengine: pl330: Acquire dmac's spinlock in pl330_tx_status

2016-08-23 Thread Guenter Roeck
On Tue, Aug 23, 2016 at 2:16 AM, Hsin-Yu Chao wrote: > There is a racing when accessing dmac thread in pl330_tx_status that > the pl330_update is handling active request at the same time and > changing the status of descriptors. This could cause an invalid > transferred count

Re: [PATCH 0/3] perf, bts: Fallout from the fuzzer for perf/urgent

2016-08-23 Thread Alexander Shishkin
Vince Weaver writes: > On Tue, 23 Aug 2016, Alexander Shishkin wrote: > >> Recently Vince has reported warnings and panics coming from the >> general direction of AUX tracing. I found two bugs which manifest >> similarly, one in intel_bts driver and one in AUX unmapping path.

Re: net: Zeroing the structure ethtool_wolinfo in ethtool_get_wol()

2016-08-23 Thread Vegard Nossum
On 23 August 2016 at 17:05, Joe Perches wrote: > On Tue, 2016-08-23 at 07:21 -0700, Eric Dumazet wrote: >> On Tue, 2016-08-23 at 14:41 +0100, Luis Henriques wrote: >> > From: Avijit Kanti Das >> > >> > memset() the structure ethtool_wolinfo that has

  1   2   3   4   5   6   7   8   9   10   >