Re: Question: Now that we have IPv6, do we need TCP/UDP port numbers?

2015-05-27 Thread Cong Wang
(Cc'ing netdev for discussion) On Mon, May 25, 2015 at 4:17 PM, Coroutines wrote: > I have a potentially dumb question to ask, and I'm posting it here > because out of everywhere on the web I thought I would find the most > experts relating to TCP/UDP's design/handling and its history here. > >

[RFC PATCH 5/5] livepatch: arm64: support relocation in a module

2015-05-27 Thread Li Bin
From: Xie XiuQi This patch implement klp_write_module_reloc on arm64 platform. Signed-off-by: Xie XiuQi Signed-off-by: Li Bin --- arch/arm64/kernel/livepatch.c |7 +- arch/arm64/kernel/module.c| 355 + 2 files changed, 186 insertions(+), 176

Re: [PATCH] usb: ulpi: don't register drivers if bus doesn't exist

2015-05-27 Thread Felipe Balbi
On Thu, May 28, 2015 at 11:09:38AM +0530, Sudip Mukherjee wrote: > On Wed, May 27, 2015 at 08:21:16AM -0700, Greg KH wrote: > > On Wed, May 27, 2015 at 11:16:34AM -0400, Alan Stern wrote: > > > On Wed, 27 May 2015, Heikki Krogerus wrote: > > > > Maybe we need to test for this in the driver core,

[RFC PATCH 4/5] livepatch: arm64: add support for livepatch on arm64

2015-05-27 Thread Li Bin
This patch add support for livepatch on arm64 based on the gcc -mfentry feature and the ftrace DYNAMIC_FTRACE_WITH_REGS feature. Signed-off-by: Li Bin --- arch/arm64/Kconfig |3 ++ arch/arm64/include/asm/livepatch.h | 45

[RFC PATCH 1/5] livepatch: ftrace: arm64: Add support for DYNAMIC_FTRACE_WITH_REGS

2015-05-27 Thread Li Bin
If ftrace_ops is registered with flag FTRACE_OPS_FL_SAVE_REGS, the arch that support DYNAMIC_FTRACE_WITH_REGS will pass a full set of pt_regs to the ftrace_ops callback function, which may read or change the value of the pt_regs and the pt_regs will be restored to support work flow redirection

[RFC PATCH 3/5] livepatch: ftrace: arm64: Add support for -mfentry on arm64

2015-05-27 Thread Li Bin
This patch depends on the compiler's mfentry feature for arm64 that proposed by this patchset. If the kernel is compiled with this feature, the entry of each function like: foo: mov x9, x30 bl __fentry__ mov x30, x9 When -mfentry is used, the call is to '__fentry__' and not

[RFC PATCH 2/5] livepatch: ftrace: add ftrace_function_stub_ip function

2015-05-27 Thread Li Bin
The function of ftrace_function_stub_ip is to convert the function address to the ftrace stub calling instruction address. This is needed for the platform that the complier does not support "profile before prologue" feature and the profile calling instruction is not at begin of the function.

Re: [PATCH v4 01/11] block: make generic_make_request handle arbitrarily sized bios

2015-05-27 Thread Ming Lin
On Wed, May 27, 2015 at 5:36 PM, Alasdair G Kergon wrote: > On Wed, May 27, 2015 at 04:42:44PM -0700, Ming Lin wrote: >> Here are fio results of XFS on a DM stripped target with 2 SSDs + 1 HDD. >> Does it make sense? > > To stripe across devices with different characteristics? I intended to test

[RFC PATCH 0/5] livepatch: add support on arm64

2015-05-27 Thread Li Bin
This patchset propose a method for gcc -mfentry feature(profile before prologue) implementation for arm64, and propose the livepatch implementation for arm64 based on this feature. The gcc implementation about this feature will be post to the gcc community soon. With this -mfentry feature, the

[PATCH] perf probe: Move kernel_get_module_dso() to be built only with DWARF support

2015-05-27 Thread Namhyung Kim
The kernel_get_module_dso() is called only from open_debugginfo() and the latter is defined only if HAVE_DWARF_SUPPORT is defined. So build without dwarf failed like below: CC util/probe-event.o util/probe-event.c:203:12: error: ‘kernel_get_module_dso’ defined but not used

Re: Lost network connectivity in 4.0.x

2015-05-27 Thread Cong Wang
(Please always Cc netdev for networking bugs.) On Sat, May 23, 2015 at 8:29 PM, Ken Moffat wrote: > On Sun, May 24, 2015 at 03:43:52AM +0100, Ken Moffat wrote: >> Anybody else suffering frm lost network connectivity in 4.0.x >> kernels ? A couple of times this week, vim on an nfs-3 mount hung

Re: [RFC PATCH v4 09/29] bpf tools: Collect version and license from ELF sections

2015-05-27 Thread Alexei Starovoitov
On Thu, May 28, 2015 at 11:34:37AM +0800, Wangnan (F) wrote: > >... > >>+static int > >>+bpf_object__init_kversion(struct bpf_object *obj, > >>+ void *data, size_t size) > >>+{ > >>+ u32 kver; > >>+ if (size < sizeof(kver)) { > >shouldn't it be '!=' ? > > Is it possible

Re: [PATCH] drm/panel: add lg4573 driver

2015-05-27 Thread Heiko Schocher
Hello, Am 06.05.2015 09:49, schrieb Heiko Schocher: The patch adds LG4573 parallel RGB panel driver with SPI control interface. The driver uses drm_panel framework. Signed-off-by: Heiko Schocher --- .../devicetree/bindings/panel/lg,lg4573.txt| 42 +++

Re: [PATCH RFC] coccinelle: flag constants being passed for jiffies

2015-05-27 Thread Julia Lawall
On Wed, 27 May 2015, Nicholas Mc Guire wrote: > schedule_timeout_* takes a jiffies value as timeout - passing in a constant > makes the timeout HZ dependent which is wrong. Checking for contstants only > yields many false positives so they are filtered for digits only. A numeric > value of 1 is

Re: [PATCH] staging: gdm72xx: remove unneeded test

2015-05-27 Thread Joe Perches
On Thu, 2015-05-28 at 07:43 +0200, Julia Lawall wrote: > On Wed, 27 May 2015, Joe Perches wrote: > > Perhaps all of the uses like: > > > > goto ; > > : > > > > could be modified. There are ~150 in the kernel. > > I wrote a semantic patch recently for that as well... Maybe I can take >

Re: [PATCHv2 2/2] mailbox: Adding driver for Xilinx LogiCORE IP mailbox.

2015-05-27 Thread Michal Simek
On 05/27/2015 08:35 PM, Moritz Fischer wrote: > The Xilinx LogiCORE IP mailbox is a FPGA core that allows for > interprocessor communication via AXI4 memory mapped / AXI4 stream > interfaces. > > It is single channel per core and allows for transmit and receive. > > Changes from v1: > - Added

Re: [PATCH] staging: gdm72xx: remove unneeded test

2015-05-27 Thread Julia Lawall
On Wed, 27 May 2015, Joe Perches wrote: > On Wed, 2015-05-27 at 22:25 +0200, Laurent Navet wrote: > > The same code is executed regardless ret value, so this test can be > > removed. > [] > > diff --git a/drivers/staging/gdm72xx/usb_boot.c > > b/drivers/staging/gdm72xx/usb_boot.c > [] > > @@

Re: [PATCH-v2 0/4] target: Eliminate se_port + t10_alua_tg_pt_gp_member

2015-05-27 Thread Nicholas A. Bellinger
On Wed, 2015-05-27 at 13:36 -0700, Paul E. McKenney wrote: > On Tue, May 26, 2015 at 10:13:02PM -0700, Nicholas A. Bellinger wrote: > > On Tue, 2015-05-26 at 14:44 +0200, Bart Van Assche wrote: > > > On 05/26/15 08:57, Nicholas A. Bellinger wrote: > > > >- Add various rcu_dereference and

Re: [RFC 0/4] arm64: add livepatch support

2015-05-27 Thread Li Bin
On 2015/4/24 14:05, Masami Hiramatsu wrote: > (2015/04/24 12:24), Li Bin wrote: >> On 2015/4/24 10:44, AKASHI Takahiro wrote: >>> This patchset enables livepatch support on arm64. >>> >>> Livepatch was merged in v4.0, and allows replacying a function dynamically >>> based on ftrace framework, but

[PATCH] rtl8723au: replaced asm/atomic.h by linux/atomic.h

2015-05-27 Thread Xavier Roche
Minor additional checkpatch.pl fix for include/rtw_io.h Signed-off-by: Xavier Roche --- drivers/staging/rtl8723au/include/rtw_io.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/rtl8723au/include/rtw_io.h b/drivers/staging/rtl8723au/include/rtw_io.h index

Re: [PATCH] usb: ulpi: don't register drivers if bus doesn't exist

2015-05-27 Thread Sudip Mukherjee
On Wed, May 27, 2015 at 08:21:16AM -0700, Greg KH wrote: > On Wed, May 27, 2015 at 11:16:34AM -0400, Alan Stern wrote: > > On Wed, 27 May 2015, Heikki Krogerus wrote: > > Maybe we need to test for this in the driver core, not allowing drivers > for busses that are not registered, that might solve

Re: [PATCH] x86: skip delays during SMP initialization similar to Xen

2015-05-27 Thread Len Brown
>> >> BTW. this time can be reduced by 7% (113 ms) by deleting >> >> announce_cpu(): >> >> >> >> [1.445815] x86: Booted up 4 nodes, 120 CPUs >> > >> > so that kind of info looks pretty useful, especially when there's >> > hangs/failures. >> >> I think the messages we print on failure are

Re: [PATCHv2 1/2] dts: Adding docs for Xilinx LogiCORE IP mailbox driver.

2015-05-27 Thread Michal Simek
On 05/27/2015 08:35 PM, Moritz Fischer wrote: > Changes from v1: > - Added common clock framework support > > Changes from v0: > - Fixed example bindings > > Signed-off-by: Moritz Fischer > --- > .../bindings/mailbox/xilinx-mailbox.txt | 44 > 1 file changed, 44

Re: [PATCH tip/perf/core] perf probe: Fix 'function unused' warning

2015-05-27 Thread Masami Hiramatsu
On 2015/05/28 11:25, Wang Nan wrote: > By 'make build-test' a warning is found in probe-event.c that, after > commit 419e873828 (perf probe: Show the error reason comes from > invalid DSO) the only user of kernel_get_module_dso() is > open_debuginfo(). Which is not compiled if HAVE_DWARF_SUPPORT

Re: [PATCH v8 6/7] clk: hi6220: Clock driver support for Hisilicon hi6220 SoC

2015-05-27 Thread Michael Turquette
Quoting Bintian Wang (2015-05-23 21:11:11) > Add clock drivers for hi6220 SoC, this driver controls the SoC > registers to supply different clocks to different IPs in the SoC. > > We add one divider clock for hi6220 because the divider in hi6220 > also has a mask bit but it doesnot obey the rule

Re: [PATCH v6 8/8] macvtap/tun: cross-endian support for little-endian hosts

2015-05-27 Thread David Gibson
On Fri, Apr 24, 2015 at 02:50:36PM +0200, Greg Kurz wrote: > The VNET_LE flag was introduced to fix accesses to virtio 1.0 headers > that are always little-endian. It can also be used to handle the special > case of a legacy little-endian device implemented by a big-endian host. > > Let's add a

Re: [PATCH v6 6/8] virtio: add explicit big-endian support to memory accessors

2015-05-27 Thread David Gibson
On Fri, Apr 24, 2015 at 02:26:24PM +0200, Greg Kurz wrote: > The current memory accessors logic is: > - little endian if little_endian > - native endian (i.e. no byteswap) if !little_endian > > If we want to fully support cross-endian vhost, we also need to be > able to convert to big endian. >

Re: [PATCH v6 7/8] vhost: cross-endian support for legacy devices

2015-05-27 Thread David Gibson
On Fri, Apr 24, 2015 at 02:27:24PM +0200, Greg Kurz wrote: > This patch brings cross-endian support to vhost when used to implement > legacy virtio devices. Since it is a relatively rare situation, the > feature availability is controlled by a kernel config option (not set > by default). > > The

Re: [PATCH v6 1/8] virtio: introduce virtio_is_little_endian() helper

2015-05-27 Thread David Gibson
On Fri, Apr 24, 2015 at 02:24:27PM +0200, Greg Kurz wrote: > Signed-off-by: Greg Kurz Reviewed-by: David Gibson -- David Gibson| I'll have my music baroque, and my code david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_

Re: [PATCH v6 4/8] vringh: introduce vringh_is_little_endian() helper

2015-05-27 Thread David Gibson
On Fri, Apr 24, 2015 at 02:24:58PM +0200, Greg Kurz wrote: > Signed-off-by: Greg Kurz Reviewed-by: David Gibson -- David Gibson| I'll have my music baroque, and my code david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_

Re: [PATCH v6 3/8] macvtap: introduce macvtap_is_little_endian() helper

2015-05-27 Thread David Gibson
On Fri, Apr 24, 2015 at 02:24:48PM +0200, Greg Kurz wrote: > Signed-off-by: Greg Kurz Reviewed-by: David Gibson -- David Gibson| I'll have my music baroque, and my code david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_

Re: [PATCH v6 5/8] vhost: introduce vhost_is_little_endian() helper

2015-05-27 Thread David Gibson
On Fri, Apr 24, 2015 at 02:25:12PM +0200, Greg Kurz wrote: > Signed-off-by: Greg Kurz Reviewed-by: David Gibson -- David Gibson| I'll have my music baroque, and my code david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_

Re: [PATCH v6 2/8] tun: add tun_is_little_endian() helper

2015-05-27 Thread David Gibson
On Fri, Apr 24, 2015 at 02:24:38PM +0200, Greg Kurz wrote: > Signed-off-by: Greg Kurz Reviewed-by: David Gibson -- David Gibson| I'll have my music baroque, and my code david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_

Re: [GIT PULL v4 00/21] libnd: non-volatile memory device support

2015-05-27 Thread Williams, Dan J
On Thu, 2015-05-28 at 03:01 +0200, Rafael J. Wysocki wrote: > >> And how much time is it going to take to rebase it, actually? > >> > >> If all is so clean as you're suggesting, a "git rebase" should be > >> sufficient for that really. Is it not the case? > > > > Of course the rebase is trivial,

Re: [PATCH 1/3] X.509: Fix certificate gathering again

2015-05-27 Thread Michal Marek
Dne 27.5.2015 v 00:15 David Howells napsal(a): > Hi Michal, > > Could you have a look at the patch at the end of my branch: > > > http://git.kernel.org/cgit/linux/kernel/git/dhowells/linux-fs.git/log/?h=modsign-pkcs7 > > It changes things from picking up arbitrary *.x509 files dropped in

Re: kmalloc panic

2015-05-27 Thread Cong Wang
(Cc'ing netdev and wireless... Looks like a bug in wireless ext.) On Wed, May 27, 2015 at 6:46 AM, pavani wrote: > Hi, > > I connected to AP with the help of wpa_supplicant in linux.After connecting > to AP I am facing an issue like "kmalloc panic".can you help me > how to solve this issue.Logs

Re: [PATCH 3/5] dt-bindings: ARM: Mediatek: Document devicetree bindings for clock controllers

2015-05-27 Thread Yong Wu
On Wed, 2015-05-27 at 09:27 +0200, Sascha Hauer wrote: > On Wed, May 27, 2015 at 02:12:49PM +0800, Yong Wu wrote: > > On Tue, 2015-05-26 at 13:08 +0200, Sascha Hauer wrote: > > > On Tue, May 26, 2015 at 04:55:36PM +0800, James Liao wrote: > > > > Hi Sascha, > > > > > > > > On Tue, 2015-05-26 at

Re: [PATCH] proc/schedstat: Expose /proc//schedstat if delay accounting is enabled

2015-05-27 Thread Naveen N. Rao
Ingo, Balbir, Any comments on this? Can you please take a look? Thanks, Naveen On 2015/05/25 12:42PM, Naveen N Rao wrote: > /proc//schedstat is currently only available if CONFIG_SCHEDSTATS is > enabled. But, all the fields that this exposes are available and valid > if CONFIG_TASK_DELAY_ACCT is

Re: [PATCH] Staging: vt6655: Remove unnecessary equality checks for a bool variable

2015-05-27 Thread Sudip Mukherjee
On Wed, May 27, 2015 at 12:07:06PM +, Harisangam, Sharvari (S.) wrote: > On Tue, May 26, 2015 at 10:35:57AM +0530, Sudip Mukherjee wrote: > > On Mon, May 25, 2015 at 12:22:25PM +, Harisangam, Sharvari (S.) wrote: > > > Modified the if-else statements to remove unnecessary comparisons. > >

[PATCH v2] clockevents: timer-stm32: Fix build warning spotted by kbuild test robot

2015-05-27 Thread Maxime Coquelin
This patch fixes below warning spotted by kbuild test robot when building with ARCH=powerpc: drivers/clocksource/timer-stm32.c: In function 'stm32_clockevent_init': >> drivers/clocksource/timer-stm32.c:140:9: warning: large integer implicitly >> truncated to unsigned type [-Woverflow]

Re: [PATCH] clockevents: timer-stm32: Fix build warning spotted by kbuild test robot

2015-05-27 Thread Maxime Coquelin
2015-05-28 0:52 GMT+02:00 Luc Van Oostenryck : > On Wed, May 27, 2015 at 06:41:23PM +0200, Maxime Coquelin wrote: >> >> /* Detect whether the timer is 16 or 32 bits */ >> - writel_relaxed(~0UL, data->base + TIM_ARR); >> + writel_relaxed((u32)~0UL, data->base + TIM_ARR); >>

Re: Edited draft of bpf(2) man page for review/enhancement

2015-05-27 Thread Alexei Starovoitov
On 5/27/15 1:43 AM, Michael Kerrisk (man-pages) wrote: Hello Alexei, I took the draft 3 of the bpf(2) man page that you sent back in March and did some substantial editing to clarify the language and add a few technical details. Could you please check the revised version below, to ensure I did

Re: [PATCH 0/3] perf tests: don't use obsolete syscalls

2015-05-27 Thread Riku Voipio
Hi, On 16 April 2015 at 19:17, Ingo Molnar wrote: >> From: Riku Voipio >> >> The following hopefully trivial patches remove the use of legacy syscalls >> from perf test subcommands. Patch 2/3 is optional, and would make the delta >> a lot smaller. >> Riku Voipio (3): >> perf tests: switch

Re: [PATCH] net: tcp: Fix a PTO timing granularity issue

2015-05-27 Thread Ido Yariv
Hi Eric, On Wed, May 27, 2015 at 03:15:26PM -0400, Ido Yariv wrote: > Hi Eric, > > On Wed, May 27, 2015 at 10:24:16AM -0700, Eric Dumazet wrote: > > On Wed, 2015-05-27 at 12:54 -0400, Ido Yariv wrote: > > > Hi Eric, > > > > > That's a nice optimization ;) > > > > > > However, I think that with

Re: [PATCH 00/21] On-demand device registration

2015-05-27 Thread Rob Herring
On Mon, May 25, 2015 at 9:53 AM, Tomeu Vizoso wrote: > Hello, > > I have a problem with the panel on my Tegra Chromebook taking longer than > expected to be ready during boot (Stéphane Marchesin reported what is > basically the same issue in [0]), and have looked into ordered probing as a >

[PATCH] staging: wilc1000: Disable for S390

2015-05-27 Thread Guenter Roeck
The wilc1000 driver uses definitions such as DEBUG_LEVEL, DEBUG, and PRINT_INFO. This causes compile errors on S390 which has similar definitions in its core code. Disable the driver for S390 instead of giving the non-standard messaging code credit by trying to fix it. Signed-off-by: Guenter

[PATCH] f2fs crypto: avoid f2fs_inherit_context for symlink

2015-05-27 Thread Jaegeuk Kim
This patch fixes to call f2fs_inherit_context twice for newly created symlink. The original one is called by f2fs_add_link(), which invokes f2fs_setxattr. If the second one is called again, f2fs_setxattr is triggered again with same encryption index. Signed-off-by: Jaegeuk Kim ---

Re: [PATCH 2/3 v2] f2fs crypto: use bounce pages from mempool first

2015-05-27 Thread Jaegeuk Kim
Change log from v1: o remain the existing emergecy mempool o add a new mempool for writeback pages >From ab8c49a79c4a6cd0ca1093d5e42cb93b55b35bfd Mon Sep 17 00:00:00 2001 From: Jaegeuk Kim Date: Wed, 20 May 2015 19:12:30 -0700 Subject: [PATCH] f2fs crypto: introduce a mempool for bounce pages

linux-next: manual merge of the net-next tree with Linus' tree

2015-05-27 Thread Stephen Rothwell
Hi all, Today's linux-next merge of the net-next tree got a conflict in drivers/net/phy/amd-xgbe-phy.c between commit 983942a5eaca ("amd-xgbe-phy: Fix initial mode when autoneg is disabled") from Linus' tree and commit 7c12aa08779c ("amd-xgbe: Move the PHY support into amd-xgbe") from the

[no subject]

2015-05-27 Thread Andi Kleen
[Repost; I forgot to copy linux-kernel earlier. Apologies if you see it twice.] Skylake moved to 32 Last Branch Records, from previously 16. The current call stack LBR implementation reads all LBRs and also saves/restores them on context switch. This patchkit adds some optimizations to avoid

[PATCH 1/5] x86, perf: Allow time stamp for free running PEBSv3

2015-05-27 Thread Andi Kleen
From: Andi Kleen With PEBSv3 the PEBS record contains a time stamp. That means we can allow free-running PEBS without a PMI even if the user program requested a time stamp. This avoids the need to use -T to get free running PEBS, and also avoids any problems with mis-identifying MMAPs later.

[PATCH 3/5] perf, tools: Disable branch flags/cycles for lbr call graph

2015-05-27 Thread Andi Kleen
From: Andi Kleen Automatically disable collecting branch flags and cycles with --call-graph lbr. This allows avoiding a bunch of extra MSR reads in the PMI on Skylake. When the kernel doesn't support the new flags they are automatically cleared in the fallback code. Signed-off-by: Andi Kleen

[PATCH 2/5] x86, perf: Add option to disable reading branch flags/cycles

2015-05-27 Thread Andi Kleen
From: Andi Kleen With LBRv5 reading the extra LBR flags like mispredict, TSX, cycles is not free anymore, as it has moved to a separate MSR. For callstack mode we don't need any of this information; so we can avoid the unnecessary MSR read. Add flags to the perf interface where perf record can

[PATCH 4/5] x86, perf: Use correct index to save/restore LBR_INFO with callstack

2015-05-27 Thread Andi Kleen
From: Andi Kleen Use the correct index to save/restore the LBR_INFO_x MSR in callstack mode. This is more a cleanup, as even with the wrong index the register was correctly saved/restored, and also LBR callgraph mode in perf tools do not really need anything in LBR_INFO. But still better to use

[PATCH 5/5] x86, perf: Limit LBR accesses to TOS in callstack mode

2015-05-27 Thread Andi Kleen
From: Andi Kleen In callstack mode the LBR is not a ring buffer, but a stack that grows up and down. This means in this case we don't need to access all LBRs, only the ones up to TOS. Do this optimization for the normal LBR read, and the context switch save/restore code. For save/restore it can

Re: [PATCH 06/16] ia64: fix up obsolete cpu function usage.

2015-05-27 Thread Rusty Russell
Tony Luck writes: > On Tue, May 26, 2015 at 6:18 PM, Rusty Russell wrote: > >> cpumask_test_cpu() doesn't take volatile, unlike the obsoleted >> cpu_isset. The only place ia64 really cares is the spin waiting for a >> bit; udelay() is probably a barrier but insert rmb() to be sure. > > Good to

Re: + zram-check-comp-algorithm-availability-earlier-v2.patch added to -mm tree

2015-05-27 Thread Sergey Senozhatsky
On (05/27/15 13:23), a...@linux-foundation.org wrote: > This patch should soon appear at > > http://ozlabs.org/~akpm/mmots/broken-out/zram-check-comp-algorithm-availability-earlier-v2.patch > and later at > >

Re: [PATCH v2 1/5] clk: remove duplicated code with __clk_set_parent_after

2015-05-27 Thread Michael Turquette
Quoting Dong Aisheng (2015-05-14 06:28:59) > __clk_set_parent_after() actually used the second argument then we > could put this duplicate logic in there and call it with a different > order of arguments in the success vs. error paths in this function. > > Cc: Mike Turquette > Cc: Stephen Boyd

Re: [PATCH RFC v2 1/2] crypto: add PKE API

2015-05-27 Thread Herbert Xu
On Sat, May 23, 2015 at 07:20:15AM -0700, Tadeusz Struk wrote: > > The length would be redundant. It can be obtained by sg_nents(reg->inparams) > I don't limit the number of parameters. You can pass as many as you want. For > instance to pass 3 in and 2 out you do: > > struct scatterlist

Re: [PATCHv2] pktgen: Convert return type of process_ipsec to bool

2015-05-27 Thread Joe Perches
On Thu, 2015-05-28 at 00:00 -0400, Nicholas Krause wrote: > This converts the function, process_ipsec to the > return type of bool due to only returning either > one or zero. > v2 > Change incorrect patch subject and make commit message > clearer > > Signed-off-by: Nicholas Krause > --- The V2

RE: Build errors in -next due to 'drivers:usb:fsl:Make fsl ehci drv an independent driver module'

2015-05-27 Thread Ramneek Mehresh
> -Original Message- > From: Mehresh Ramneek-B31383 > Sent: Thursday, May 28, 2015 9:25 AM > To: 'Guenter Roeck' > Cc: Alan Stern; Greg Kroah-Hartman; linux-kernel@vger.kernel.org; linux- > n...@vger.kernel.org > Subject: RE: Build errors in -next due to 'drivers:usb:fsl:Make fsl ehci

RE: Build errors in -next due to 'drivers:usb:fsl:Make fsl ehci drv an independent driver module'

2015-05-27 Thread Ramneek Mehresh
> -Original Message- > From: Guenter Roeck [mailto:li...@roeck-us.net] > Sent: Thursday, May 28, 2015 1:29 AM > To: Mehresh Ramneek-B31383 > Cc: Alan Stern; Greg Kroah-Hartman; linux-kernel@vger.kernel.org; linux- > n...@vger.kernel.org > Subject: Build errors in -next due to

Re: [PATCH RESEND] sched: prefer an idle cpu vs an idle sibling for BALANCE_WAKE

2015-05-27 Thread Mike Galbraith
On Wed, 2015-05-27 at 17:22 -0400, Josef Bacik wrote: > [ sorry if you get this twice, it seems like the first submission got lost ] > > At Facebook we have a pretty heavily multi-threaded application that is > sensitive to latency. We have been pulling forward the old SD_WAKE_IDLE code >

Re: LOCKDEP warning due to ext_devt_lock

2015-05-27 Thread NeilBrown
On Wed, 27 May 2015 21:32:21 -0600 Jens Axboe wrote: > On 05/27/2015 08:32 PM, NeilBrown wrote: > > > > Hi Keith, > > I'm getting lockdep warning around use of ext_devt_lock. > > > > [11428.266019] inconsistent {SOFTIRQ-ON-W} -> {IN-SOFTIRQ-W} usage. > > > > As you say in > >

Re: [Spice-devel] [PATCH] Do not loop on ERESTARTSYS using interruptible waits

2015-05-27 Thread Dave Airlie
On 19 May 2015 at 19:54, Frediano Ziglio wrote: > This problem happens using KMS surfaces and QXL driver. > To easy reproduce use KDE Plasma (which use surfaces a lot) and assure > you are using KMS surfaces (QXL driver on Fedora/RedHat has a patch to > stop using them). Open some complex

Re: [RFC PATCH v4 09/29] bpf tools: Collect version and license from ELF sections

2015-05-27 Thread Wangnan (F)
On 2015/5/28 9:48, Alexei Starovoitov wrote: On Wed, May 27, 2015 at 05:19:44AM +, Wang Nan wrote: Expand bpf_obj_elf_collect() to collect license and kernel version information in eBPF object file. eBPF object file should have a section named 'license', which contains a string. It should

Re: [Spice-devel] [PATCH 11/11] Propagate correctly errors from qxlhw_handle_to_bo

2015-05-27 Thread Dave Airlie
On 27 May 2015 at 20:04, Frediano Ziglio wrote: > This function could return a NULL pointer in case of handle not > present and in case of out of memory conditions however caller > function always returned EINVAL error hiding a possible ENOMEM. > This patch change the function to return the error

Re: LOCKDEP warning due to ext_devt_lock

2015-05-27 Thread Jens Axboe
On 05/27/2015 08:32 PM, NeilBrown wrote: Hi Keith, I'm getting lockdep warning around use of ext_devt_lock. [11428.266019] inconsistent {SOFTIRQ-ON-W} -> {IN-SOFTIRQ-W} usage. As you say in 2da78092dda1 ("block: Fix dev_t minor allocation lifetime") ext_devt_lock can be taken from

Re: [Spice-devel] [PATCH 10/11] Simplify cleaning qxl processing command

2015-05-27 Thread Dave Airlie
On 27 May 2015 at 20:04, Frediano Ziglio wrote: > In qxlhw_handle_to_bo we incremented counters twice, one time for release > object > and one for reloc_info. > In the main function however reloc_info references was drop much earlier than > release so keeping the pointer only on release is safe

Re: [Spice-devel] [PATCH 09/11] Move main reference counter to GEM object instead of TTM ones

2015-05-27 Thread Dave Airlie
On 27 May 2015 at 20:04, Frediano Ziglio wrote: > qxl_bo structure has two reference counters, one in the GEM object and > another in the TTM object. The GEM object keep a counter to the TTM object > so when GEM counter reached zero the TTM counter (using qxl_bo_unref) was > decremented. The qxl

Re: [Spice-devel] [PATCH 08/11] Remove format string errors

2015-05-27 Thread Dave Airlie
On 27 May 2015 at 20:04, Frediano Ziglio wrote: > Enable format string checks for qxl_io_log and remove resulting warnings > which could lead to memory errors on different platform or just printing > wrong information. > > Signed-off-by: Frediano Ziglio Reviewed-by: Dave Airlie > --- >

Re: [Spice-devel] [PATCH 07/11] Handle correctly failures in qxl_alloc_relase_reserved

2015-05-27 Thread Dave Airlie
On 27 May 2015 at 20:04, Frediano Ziglio wrote: > Free resources correctly if function fails > > Signed-off-by: Frediano Ziglio Reviewed-by: Dave Airlie -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More

Re: [Spice-devel] [PATCH 06/11] Fix return for qxl_release_alloc

2015-05-27 Thread Dave Airlie
On 27 May 2015 at 20:04, Frediano Ziglio wrote: > This function return handle to allocated release object which is an int. Reviewed-by: Dave Airlie > > Signed-off-by: Frediano Ziglio > --- > qxl/qxl_release.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git

Re: [Spice-devel] [PATCH 05/11] Handle all errors in qxl_surface_evict

2015-05-27 Thread Dave Airlie
On 27 May 2015 at 20:04, Frediano Ziglio wrote: > Only EBUSY error was handled. This could cause code to believe > reserve was successful while it failed. > > Signed-off-by: Frediano Ziglio This has been there since I wrote qxl, so I expect I had some reason for it, but I can't remember it

Re: [Spice-devel] [PATCH 04/11] Avoid double free on error

2015-05-27 Thread Dave Airlie
On 27 May 2015 at 20:03, Frediano Ziglio wrote: > Is we are not able to get source bo object from handle we free > destination bo object and call cleanup code however destination > object was already inserted in reloc_info array (num_relocs was > already incremented) so on cleanup we free

Re: [RFC PATCH v4 10/29] bpf tools: Collect map definitions from 'maps' section

2015-05-27 Thread Wangnan (F)
On 2015/5/28 10:28, Alexei Starovoitov wrote: On Thu, May 28, 2015 at 10:03:04AM +0800, Wangnan (F) wrote: On 2015/5/28 9:53, Alexei Starovoitov wrote: On Wed, May 27, 2015 at 05:19:45AM +, Wang Nan wrote: If maps are used by eBPF programs, corresponding object file(s) should contain a

Re: [Spice-devel] [PATCH 03/11] Fix print statement not using uninitialized variable

2015-05-27 Thread Dave Airlie
On 27 May 2015 at 20:03, Frediano Ziglio wrote: > reloc_info[i] is not still initialized in the print statement. > > Signed-off-by: Frediano Ziglio Reviewed-by: Dave Airlie -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to

Re: [Spice-devel] [PATCH 02/11] Do not leak memory if qxl_release_list_add fails

2015-05-27 Thread Dave Airlie
On 27 May 2015 at 20:03, Frediano Ziglio wrote: > If the function fails reference counter to the object is not decremented > causing leaks. > This is hard to spot as it happens only on very low memory situations. > > Signed-off-by: Frediano Ziglio Looks good, Reviewed-by: Dave Airlie Dave.

Re: [Spice-devel] [PATCH 01/11] Do not cause spice-server to clean our objects

2015-05-27 Thread Dave Airlie
On 27 May 2015 at 20:03, Frediano Ziglio wrote: > If objects are moved back from system memory to VRAM (and spice id > created again) memory is already initialized so we need to set flag > to not clear memory. > If you don't do it after a while using desktop many images turns to > black or

Re: [Spice-devel] [PATCH 00/11] Miscellaneous stability patches

2015-05-27 Thread Dave Airlie
> I was using a different repository with only QXL driver. I tested and all > patches apply and compile perfectly even with Linus master branch. Lets only post patches people can apply, it makes it harder to figure out stuff. I'll take a look at the patches, but it would be good to get them

[PATCH] gpio/crystalcove: set IRQCHIP_SKIP_SET_WAKE for the irqchip

2015-05-27 Thread Aaron Lu
The CrystalCove GPIO irqchip doesn't have irq_set_wake callback defined so we should set IRQCHIP_SKIP_SET_WAKE for it or it would cause an irq desc's wake_depth unbalanced warning during system resume phase from the gpio_keys driver, which is the driver for the power button of the ASUS T100

Re: [PATCH v4 3/3] dell-laptop: Use dell-rbtn instead i8042 filter when possible

2015-05-27 Thread Darren Hart
On Wed, May 27, 2015 at 11:28:25PM +0200, Pali Rohár wrote: > Until now module dell-laptop registered rfkill device which used i8042 > filter function for receiving HW switch rfkill events (handling special > keycode). > > But for some dell laptops there is native ACPI driver dell-rbtn which can

[PATCH 1/2] perf tools: Introduce machines__new/delete()

2015-05-27 Thread Namhyung Kim
Create and maintain struct machines in struct perf_tool and pass it to struct perf_session. This is a preparation of sharing machines among sessions in case of multi-thread report. Suggested-by: Arnaldo Carvalho de Melo Signed-off-by: Namhyung Kim --- tools/perf/builtin-annotate.c | 5

[PATCH 2/2] perf session: Separate struct machines from session

2015-05-27 Thread Namhyung Kim
With multi-thread report, separate sessions can be passed to each thread, in this case we should keep a single machine state for all struct sessions. Separate machines and have a pointer in sessions. Signed-off-by: Namhyung Kim --- tools/perf/builtin-annotate.c | 2 +-

[RFC 0/2] perf session: Separate struct machines from session

2015-05-27 Thread Namhyung Kim
Hi Arnaldo, This is what I have now for the earlier discussion [1]. I just used machines__new/delete functions directly whenever a session is created/deleted. As it touches many callsites, and it needs bit more caring about the error path, you may want to split the first patch for each command.

Re: [RFC/PATCH 3/9] of: dma: Move range size workaround to of_dma_get_range()

2015-05-27 Thread Rob Herring
On Thu, May 14, 2015 at 6:00 PM, Laurent Pinchart wrote: > Invalid dma-ranges values should be worked around when retrieving the > DMA range in of_dma_get_range(), not by all callers of the function. > This isn't much of a problem now that we have a single caller, but that > situation will change

LOCKDEP warning due to ext_devt_lock

2015-05-27 Thread NeilBrown
Hi Keith, I'm getting lockdep warning around use of ext_devt_lock. [11428.266019] inconsistent {SOFTIRQ-ON-W} -> {IN-SOFTIRQ-W} usage. As you say in 2da78092dda1 ("block: Fix dev_t minor allocation lifetime") ext_devt_lock can be taken from call_rcu's soft-irq. I think that means it

[GIT PULL] platform-drivers-x86 for 4.1-3

2015-05-27 Thread Darren Hart
Hi Linus, A well intentioned switch to the DEVICE_ATTR_* macros changed the sys attributes, causing some problems with userspace. Thanks, Darren Hart Intel Open Source Technology Center The following changes since commit df6dd1b35b0ec0ac6a5298378ceaf487091f448c: thinkpad_acpi: Fix warning

Re: [RFC PATCH v4 10/29] bpf tools: Collect map definitions from 'maps' section

2015-05-27 Thread Alexei Starovoitov
On Thu, May 28, 2015 at 10:03:04AM +0800, Wangnan (F) wrote: > > > On 2015/5/28 9:53, Alexei Starovoitov wrote: > >On Wed, May 27, 2015 at 05:19:45AM +, Wang Nan wrote: > >>If maps are used by eBPF programs, corresponding object file(s) should > >>contain a section named 'map'. Which

[PATCH tip/perf/core] perf probe: Fix 'function unused' warning

2015-05-27 Thread Wang Nan
By 'make build-test' a warning is found in probe-event.c that, after commit 419e873828 (perf probe: Show the error reason comes from invalid DSO) the only user of kernel_get_module_dso() is open_debuginfo(). Which is not compiled if HAVE_DWARF_SUPPORT not set. 'make build-test' found this problem

Re: [PATCH] PCI: Only enable IO window if supported

2015-05-27 Thread Guenter Roeck
On Wed, May 27, 2015 at 04:04:47PM -0500, Bjorn Helgaas wrote: > [+cc Lorenzo, Suravee, Will] > > I cc'd Lorenzo, Suravee, and Will because Lorenzo is working on calling > pci_read_bases() from the PCI core instead of from arch code, and there are > likely some dependencies between these two

Re: [PATCH v5 00/10] latched RB-trees and __module_address()

2015-05-27 Thread Rusty Russell
Peter Zijlstra writes: > On Tue, Apr 14, 2015 at 12:27:05PM +0930, Rusty Russell wrote: Hmm, this blows up: > +static void module_assert_mutex_or_preempt(void) > +{ > +#ifdef CONFIG_LOCKDEP > + if (!unlikely(debug_locks)) > + return; > + > +

Re: [RFC PATCH v4 18/29] bpf tools: Introduce bpf_load_program() to bpf.c

2015-05-27 Thread Alexei Starovoitov
On Wed, May 27, 2015 at 05:19:53AM +, Wang Nan wrote: > bpf_load_program() can be used to load bpf program into kernel. To make > loading faster, first try to load without logbuf. Try again with logbuf > if the first try failed. > > Signed-off-by: Wang Nan Acked-by: Alexei Starovoitov --

Re: [RFC PATCH v4 10/29] bpf tools: Collect map definitions from 'maps' section

2015-05-27 Thread Wangnan (F)
On 2015/5/28 9:53, Alexei Starovoitov wrote: On Wed, May 27, 2015 at 05:19:45AM +, Wang Nan wrote: If maps are used by eBPF programs, corresponding object file(s) should contain a section named 'map'. Which contains map definitions. This patch copies the data of the whole section. Map

Re: [PATCH v2] cpuidle: Do not use CPUIDLE_DRIVER_STATE_START in cpuidle.c

2015-05-27 Thread Preeti U Murthy
On 05/28/2015 07:39 AM, Rafael J. Wysocki wrote: > From: Rafael J. Wysocki > > The CPUIDLE_DRIVER_STATE_START symbol is defined as 1 only if > CONFIG_ARCH_HAS_CPU_RELAX is set, otherwise it is defined as 0. > However, if CONFIG_ARCH_HAS_CPU_RELAX is set, the first (index 0) > entry in the

Re: [RFC PATCH v4 16/29] bpf tools: Create eBPF maps defined in an object file

2015-05-27 Thread Alexei Starovoitov
On Wed, May 27, 2015 at 05:19:51AM +, Wang Nan wrote: > This patch creates maps based on 'map' section in object file using > bpf_create_map(), and store the fds into an array in > 'struct bpf_object'. Since the byte order of the object may differ > from the host, swap map definition before

Re: [RFC PATCH v4 15/29] bpf tools: Add bpf.c/h for common bpf operations

2015-05-27 Thread Alexei Starovoitov
On Wed, May 27, 2015 at 05:19:50AM +, Wang Nan wrote: > This patch introduces bpf.c and bpf.h, which hold common functions > issuing bpf syscall. The goal of these two files is to hide syscall > completly from user. Note that bpf.c and bpf.h only deal with kernel > interface. Things like

Re: [RFC PATCH v4 10/29] bpf tools: Collect map definitions from 'maps' section

2015-05-27 Thread Alexei Starovoitov
On Wed, May 27, 2015 at 05:19:45AM +, Wang Nan wrote: > If maps are used by eBPF programs, corresponding object file(s) should > contain a section named 'map'. Which contains map definitions. This > patch copies the data of the whole section. Map data parsing should be > acted just before map

linux-next: build failure after merge of the rr-fixes tree

2015-05-27 Thread Stephen Rothwell
'cpumask_set_cpu_local_first' [-Werror=implicit-function-declaration] cpumask_set_cpu_local_first(queue_index, ^ Caused by commit 8829fd02186d ("cpumask_set_cpu_local_first => cpumask_local_spread, lament"). I have used the version of the rr-fixes tree from next-20150527 for toda

Re: [RFC PATCH v4 09/29] bpf tools: Collect version and license from ELF sections

2015-05-27 Thread Alexei Starovoitov
On Wed, May 27, 2015 at 05:19:44AM +, Wang Nan wrote: > Expand bpf_obj_elf_collect() to collect license and kernel version > information in eBPF object file. eBPF object file should have a section > named 'license', which contains a string. It should also have a section > named 'version',

  1   2   3   4   5   6   7   8   9   10   >