[PATCH] mac80211: document IEEE80211_HW_SUPPORTS_HT_CCK_RATES

2013-10-19 Thread Michael Opdenacker
This patch documents the IEEE80211_HW_SUPPORTS_HT_CCK_RATES flag in ieee80211_hw_flags. Without this, you get countless warnings in "make htmldocs". Signed-off-by: Michael Opdenacker --- include/net/mac80211.h | 3 +++ 1 file changed, 3 insertions(+) diff --git a/include/net/mac80211.h b/inclu

RE: [PATCH] iwlwifi: remove duplicate includes

2013-10-19 Thread Grumbach, Emmanuel
> Subject: [PATCH] iwlwifi: remove duplicate includes > > Reported by "make includecheck" > > Tested that the corresponding sources still compile well on x86 > > Signed-off-by: Michael Opdenacker electrons.com> > --- Picked up. Thanks. -- To unsubscribe from this list: send the line "unsubscri

[PATCH] ethernet: moxa: remove duplicate includes

2013-10-19 Thread Michael Opdenacker
Reported by "make includecheck" Tested that drivers/net/ethernet/moxa/moxart_ether.c still compiles well on ARM Signed-off-by: Michael Opdenacker --- drivers/net/ethernet/moxa/moxart_ether.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/net/ethernet/moxa/moxart_ether.c b/drivers/n

[PATCH] cgxb4: remove duplicate include in cxgb4.h

2013-10-19 Thread Michael Opdenacker
Reported by "make includecheck" Tested that C sources including this file still compile well on x86 Signed-off-by: Michael Opdenacker --- drivers/net/ethernet/chelsio/cxgb4/cxgb4.h | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/net/ethernet/chelsio/cxgb4/cxgb4.h b/drivers/net/ether

nf_tables*.h: Remove extern from function prototypes

2013-10-19 Thread Joe Perches
There are a mix of function prototypes with and without extern in the kernel sources. Standardize on not using extern for function prototypes. Function prototypes don't need to be written with extern. extern is assumed by the compiler. Its use is as unnecessary as using auto to declare automatic

[PATCH] iwlwifi: remove duplicate includes

2013-10-19 Thread Michael Opdenacker
Reported by "make includecheck" Tested that the corresponding sources still compile well on x86 Signed-off-by: Michael Opdenacker --- drivers/net/wireless/iwlwifi/iwl-io.c | 1 - drivers/net/wireless/iwlwifi/mvm/mvm.h | 1 - 2 files changed, 2 deletions(-) diff --git a/drivers/net/wireless/iw

[PATCH net-next] net: fix build warnings because of net_get_random_once merge

2013-10-19 Thread Hannes Frederic Sowa
This patch fixes the following warning: In file included from include/linux/skbuff.h:27:0, from include/linux/netfilter.h:5, from include/net/netns/netfilter.h:5, from include/net/net_namespace.h:20, from include/li

Re: [userns-always-map-user-v136] BUG: unable to handle kernel NULL pointer dereference at (null)

2013-10-19 Thread Eric W. Biederman
Fengguang Wu writes: > Hi Eric, > > It's beyond me why this trivial patch will lead to kernel panic. But > the NULL pointer dereference bug is 100% reproducible since this > commit. My mistake apparently I failed to boot test that patch. The problem crash is because kthreadd_task == NULL. kth

[PATCH] cpufreq: intel_pstate: fix possible integer overflow

2013-10-19 Thread Geyslan G. Bem
The expression 'pstate << 8' is evaluated using 32-bit arithmetic while 'val' expects an expression of type u64. Signed-off-by: Geyslan G. Bem --- drivers/cpufreq/intel_pstate.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/cpufreq/intel_pstate.c b/drivers/cpufreq/i

Re: [PATCH net-next v4 0/9] Introduce support to lazy initialize mostly static keys

2013-10-19 Thread Hannes Frederic Sowa
On Sat, Oct 19, 2013 at 07:46:35PM -0400, David Miller wrote: > From: Hannes Frederic Sowa > Date: Sat, 19 Oct 2013 21:48:50 +0200 > > > This series implements support for delaying the initialization of secret > > keys, e.g. used for hashing, for as long as possible. This functionality > > is imp

[PATCH v2] net: remove function sk_reset_txq()

2013-10-19 Thread ZHAO Gang
What sk_reset_txq() does is just calls function sk_tx_queue_reset(), and sk_reset_txq() is used only in sock.h, by dst_negative_advice(). Let dst_negative_advice() calls sk_tx_queue_reset() directly so we can remove unneeded sk_reset_txq(). Signed-off-by: ZHAO Gang change a typo in patch descript

[PATCH] net: remove function sk_reset_txq()

2013-10-19 Thread ZHAO Gang
What sk_reset_txq() does is just calls function sk_tx_queue_reset(), and sk_reset_txq() is used only in sock.c, by dst_negative_advice(). Let dst_negative_advice() calls sk_tx_queue_reset() directly so we can remove unneeded sk_reset_txq(). Signed-off-by: ZHAO Gang --- include/net/sock.h | 4 +--

[PATCHv2] x86: add kconfig options for newer 64-bit processors

2013-10-19 Thread Austin S Hemmelgarn
From: Austin S. Hemmelgarn This patch adds options to specifically optimize for a number of newer 64-bit microarchitectures; specifically, Intel's Nehalem, Westmere, Ivy Bridge, and Sandy Bridge, and AMD's Family 10h, Bobcat, Jaguar, Bulldozer, Piledriver, and Steamroller. This serves primari

Re: [PATCH v2 2/4] perf: streamline append_chain() function

2013-10-19 Thread Andi Kleen
Waiman Long writes: > as well as > using ?: statement which can be more efficient than the regular if > statement in some architectures. I don't think that's true, the compiler does if conversion anyways for both. But change seems reasonable. -Andi -- a...@linux.intel.com -- Speaking for my

[PATCH] cpufreq, highbank: enable ECME thermal notifications

2013-10-19 Thread Mark Langsdorf
The ECME sends thermal messages with a maximum and minimum allowed frequency when the SoC status reaches certain trip points known to the ECME. Use a notifier function to capture those messages and pass them to a work-queued function that can trigger a policy re-evaluation by cpufreq, capping the a

Re: linux-next: manual merge of the userns tree

2013-10-19 Thread Eric W. Biederman
Mark Brown writes: > Today's linux-next merge of the userns tree got a conflict in > fs/fuse/dir.c between 3c70b8eed (fuse: don't check_submounts_and_drop() > in RCU walk) in the fuse tree and 40216baa0 (vfs: Lazily remove mounts > on unlinked files and directories. v2) in the userns tree. > > I

Re: [PATCH net-next v4 0/9] Introduce support to lazy initialize mostly static keys

2013-10-19 Thread David Miller
From: Hannes Frederic Sowa Date: Sat, 19 Oct 2013 21:48:50 +0200 > This series implements support for delaying the initialization of secret > keys, e.g. used for hashing, for as long as possible. This functionality > is implemented by a new macro, net_get_random_bytes. > > I already used it to p

Re: [PATCH] crypto: n2_core: insert '!err' condition in else scope

2013-10-19 Thread Geyslan Gregório Bem
2013/10/19 David Miller : > From: Geyslan Gregório Bem > Date: Sat, 19 Oct 2013 19:42:35 -0300 > >> 2013/10/19 David Miller : >>> From: "Geyslan G. Bem" >>> Date: Sat, 19 Oct 2013 10:09:31 -0300 >>> This patch moves the '!err' condition into the above else scope, what is more obvious an

Re: [PATCH 0/4] net: Remove extern from function prototypes

2013-10-19 Thread David Miller
From: Joe Perches Date: Fri, 18 Oct 2013 13:48:21 -0700 > Remove the remainder of extern function prototypes from net/.../*.h files. Series applied, thanks a lot Joe. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org

[PATCH] Revert "bridge: only expire the mdb entry when query is received"

2013-10-19 Thread Linus Lüssing
While this commit was a good attempt to fix issues occuring when no multicast querier is present, this commit still has two more issues: 1) There are cases where mdb entries do not expire even if there is a querier present. The bridge will unnecessarily continue flooding multicast packets on the a

Re: [PATCH v3] net: sctp: fix a cacc_saw_newack missetting issue

2013-10-19 Thread David Miller
From: Chang Xiangzhong Date: Wed, 16 Oct 2013 09:22:11 +0200 > For for each TSN t being newly acked (Not only cumulatively, > but also SELECTIVELY) cacc_saw_newack should be set to 1. > > Signed-off-by: Xiangzhong Chang SCTP folks, can you please review this patch? Thanks. -- To unsubscribe f

Re: [PATCH] crypto: n2_core: insert '!err' condition in else scope

2013-10-19 Thread David Miller
From: Geyslan Gregório Bem Date: Sat, 19 Oct 2013 19:42:35 -0300 > 2013/10/19 David Miller : >> From: "Geyslan G. Bem" >> Date: Sat, 19 Oct 2013 10:09:31 -0300 >> >>> This patch moves the '!err' condition into the above else scope, >>> what is more obvious and has the secondary goal of avoid fal

Re: [PATCH] vsprintf: add Bluetooth UUID %pU[rR] format specifier

2013-10-19 Thread Joe Perches
On Sun, 2013-10-20 at 00:30 +0200, Marcel Holtmann wrote: > Hi Joe, Hi Marcel > > Could there ever be a little endian reversed UUID? > I honestly do not know. I looked at the little endian one and it > looked a bit heavy misplaced if we would use a reversed stream of > bytes. > > Might it be usef

Re: [PATCH] crypto: n2_core: insert '!err' condition in else scope

2013-10-19 Thread Geyslan Gregório Bem
2013/10/19 David Miller : > From: "Geyslan G. Bem" > Date: Sat, 19 Oct 2013 10:09:31 -0300 > >> This patch moves the '!err' condition into the above else scope, >> what is more obvious and has the secondary goal of avoid false-positives >> in statical analyze tools. >> >> Signed-off-by: Geyslan G.

Re: [PATCH v2 00/15] slab: overload struct slab over struct page to reduce memory usage

2013-10-19 Thread Andi Kleen
Joonsoo Kim writes: > There is two main topics in this patchset. One is to reduce memory usage > and the other is to change a management method of free objects of a slab. I did a quick read over the whole patchset and it looks good to me. I especially like how much code you remove. And of course

Re: [PATCH v2 00/15] slab: overload struct slab over struct page to reduce memory usage

2013-10-19 Thread Andi Kleen
Andrew Morton writes: > > One example is mm/memory-failure.c:memory_failure(). It starts with a > raw pfn, uses that to get at the `struct page', then starts playing > around with it. Will that code still work correctly when some of the > page's fields have been overlayed with slab-specific cont

Re: [PATCH] crypto: n2_core: insert '!err' condition in else scope

2013-10-19 Thread David Miller
From: "Geyslan G. Bem" Date: Sat, 19 Oct 2013 10:09:31 -0300 > This patch moves the '!err' condition into the above else scope, > what is more obvious and has the secondary goal of avoid false-positives > in statical analyze tools. > > Signed-off-by: Geyslan G. Bem I do not think patches shoul

Re: [PATCH] vsprintf: add Bluetooth UUID %pU[rR] format specifier

2013-10-19 Thread Marcel Holtmann
Hi Joe, >> The Bluetooth UUID is used in big endian reversed order. Add new >> modifier to print a UUID in big endian, but where the input byte >> stream is actually in reversed order. >> This is similar to %pMR that allows to print a MAC address in >> reversed order since that is how the Bluetoot

[PATCH] using kfree_skb() instead of kfree() to free sk_buff

2013-10-19 Thread Salil Kapur
--- drivers/nfc/mei_phy.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/nfc/mei_phy.c b/drivers/nfc/mei_phy.c index 606bf55..bc30081 100644 --- a/drivers/nfc/mei_phy.c +++ b/drivers/nfc/mei_phy.c @@ -127,7 +127,7 @@ void nfc_mei_event_cb(struct mei_cl_device *device,

[kconfig] update: results of some syntactical checks

2013-10-19 Thread Martin Walch
This is an update to the syntactic results that I sent, back in July. With kernel 3.12 nearing completion, I would like to point to new sections in Kconfig files with potential problems: drivers/media/common/siano/Kconfig:21-26 > config SMS_SIANO_DEBUGFS > bool "Enable debugfs for smsdvb" >

Re: [PATCH 2/6] x86: allow to call text_poke_bp during boot

2013-10-19 Thread Steven Rostedt
On Sat, 19 Oct 2013 14:33:50 -0700 "Paul E. McKenney" wrote: > > > It's used to convert the calls to mcount to nops. But maybe a better > > > thing to do is to check if we only have a single CPU: > > > > > > static void run_sync(void) > > > { > > > if (num_online_cpus() != 1) > > > > Hmm, t

Re: [PATCH 2/6] x86: allow to call text_poke_bp during boot

2013-10-19 Thread Steven Rostedt
On Sat, 19 Oct 2013 14:33:50 -0700 "Paul E. McKenney" wrote: > > /* Ideally we would like to run on zero CPUS! */ > > if (num_online_cpus() < 2) > > To be really safe, shouldn't you use complex numbers? Just in case > you end up running on a system with 5i-3 CPUs or something. ;-) >

[PATCH] vfs: fix new kernel-doc warnings

2013-10-19 Thread Randy Dunlap
From: Randy Dunlap Move kernel-doc notation to immediately before its function to eliminate kernel-doc warnings. (from commit db14fc3ab: vfs: add d_walk()) Warning(fs/dcache.c:1343): No description found for parameter 'data' Warning(fs/dcache.c:1343): No description found for parameter 'dentry'

[PATCH] fs/namei.c: fix new kernel-doc warning

2013-10-19 Thread Randy Dunlap
From: Randy Dunlap Add @path parameter to fix kernel-doc warning. Also fix a spello/typo. Warning(fs/namei.c:2304): No description found for parameter 'path' Signed-off-by: Randy Dunlap --- fs/namei.c |3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) --- lnx-312-rc6.orig/fs/namei.c

[PATCH] sched/fair.c: remove kernel-doc warning

2013-10-19 Thread Randy Dunlap
From: Randy Dunlap Remove @balance parameter that is no longer used in the function to remove a kernel-doc warning. Warning(kernel/sched/fair.c:4704): Excess function parameter 'balance' description in 'update_sd_lb_stats' Signed-off-by: Randy Dunlap --- kernel/sched/fair.c |1 - 1 file

Re: [PATCH v2 5/5] gpiolib / ACPI: document the GPIO descriptor based interface

2013-10-19 Thread Linus Walleij
On Thu, Oct 10, 2013 at 10:01 AM, Mika Westerberg wrote: > In addition to the existing ACPI specific GPIO interface, document the new > descriptor based GPIO interface in Documentation/acpi/enumeration.txt, so > it is clear that this new interface is preferred over the ACPI specific > version. >

Re: [PATCH 2/6] x86: allow to call text_poke_bp during boot

2013-10-19 Thread Paul E. McKenney
On Sat, Oct 19, 2013 at 03:19:19PM -0400, Steven Rostedt wrote: > > [ Added Paul because he'll understand this ] > > On Sat, 19 Oct 2013 15:16:58 -0400 > Steven Rostedt wrote: > > > On Sun, 20 Oct 2013 00:02:32 +0900 > > Masami Hiramatsu wrote: > > > > > (2013/10/18 23:27), Petr Mladek wrote:

Re: [PATCH v3 0/3] New descriptor-based GPIO interface

2013-10-19 Thread Linus Walleij
On Fri, Oct 18, 2013 at 7:37 PM, Alexandre Courbot wrote: > On Fri, Oct 18, 2013 at 4:23 AM, Linus Walleij > wrote: >> We might have to take this as the first thing after the merge >> window and targeted for v3.14. > > No particular pressure from my side to get this in 3.13. Do for the > best,

Re: [PATCH] vsprintf: add Bluetooth UUID %pU[rR] format specifier

2013-10-19 Thread Joe Perches
On Sat, 2013-10-19 at 09:25 -0700, Marcel Holtmann wrote: > The Bluetooth UUID is used in big endian reversed order. Add new > modifier to print a UUID in big endian, but where the input byte > stream is actually in reversed order. > This is similar to %pMR that allows to print a MAC address in > r

Re: [PATCH 2/2 v4] usb: ohci: remove ep93xx bus glue platform driver

2013-10-19 Thread Greg KH
On Thu, Oct 17, 2013 at 01:29:11PM -0700, H Hartley Sweeten wrote: > Convert ep93xx to use the OHCI platform driver and remove the > ohci-ep93xx bus glue driver. > > Enable CONFIG_OHCI_HCD_PLATFORM in the ep93xx_defconfig so that USB > is still enabled by default on the EP93xx platform. > > Signe

Re: [PATCH 2/2 v4] usb: ohci: remove ep93xx bus glue platform driver

2013-10-19 Thread Greg KH
On Thu, Oct 17, 2013 at 05:11:54PM -0400, Alan Stern wrote: > On Thu, 17 Oct 2013, H Hartley Sweeten wrote: > > > Convert ep93xx to use the OHCI platform driver and remove the > > ohci-ep93xx bus glue driver. > > > > Enable CONFIG_OHCI_HCD_PLATFORM in the ep93xx_defconfig so that USB > > is still

adjust /proc/PID/oom_score_adj with CAP_SYS_RESOURCE

2013-10-19 Thread Azat Khuzhin
Recently I needed in adjusting /proc/PID/oom_score_adj to disable oom killer, but I didn't want to add suid/or run from root that binary. I decided to use CAP_SYS_RESOURCE. However it didn't work. I gdb/strace/printk a lot, and finally found the reason, the process can't open this file for writin

[GIT PULL] parisc fixes for v3.12

2013-10-19 Thread Helge Deller
Hi Linus, Please pull the latest parisc fixes from git://git.kernel.org/pub/scm/linux/kernel/git/deller/parisc-linux.git parisc-3.12 There are just two small fixes in here: - Revert a commit which exported the flush_cache_page function. This was noticed by Christoph Hellwig. - Enable the D

[PATCH] x86: remove duplicate includes

2013-10-19 Thread Michael Opdenacker
Found by running "make includecheck" Signed-off-by: Michael Opdenacker --- arch/x86/kernel/cpu/intel_cacheinfo.c | 1 - arch/x86/kernel/smpboot.c | 2 -- arch/x86/platform/ce4100/ce4100.c | 1 - 3 files changed, 4 deletions(-) diff --git a/arch/x86/kernel/cpu/intel_cacheinfo.c

Re: [PATCH v2] videobuf2: Add missing lock held on vb2_fop_relase

2013-10-19 Thread Ricardo Ribalda Delgado
Hello Sylwester On Sat, Oct 19, 2013 at 8:27 PM, Sylwester Nawrocki wrote: > On 10/19/2013 06:07 PM, Ricardo Ribalda wrote: > [...] >> >> --- >> drivers/media/platform/exynos4-is/fimc-capture.c | 2 +- >> drivers/media/platform/exynos4-is/fimc-lite.c| 2 +- >> drivers/media/usb/em28xx/

[PATCH net-next v4 4/9] x86/jump_label: expect default_nop if static_key gets enabled on boot-up

2013-10-19 Thread Hannes Frederic Sowa
net_get_random_once(intrduced in the next patch) uses static_keys in a way that they get enabled on boot-up instead of replaced with an ideal_nop. So check for default_nop on initial enabling. Other architectures don't check for this. Cc: Thomas Gleixner Cc: Ingo Molnar Cc: "H. Peter Anvin" Cc

[PATCH net-next v4 3/9] static_key: WARN on usage before jump_label_init was called

2013-10-19 Thread Hannes Frederic Sowa
Usage of the static key primitives to toggle a branch must not be used before jump_label_init() is called from init/main.c. jump_label_init reorganizes and wires up the jump_entries so usage before that could have unforeseen consequences. Following primitives are now checked for correct use: * sta

[PATCH net-next v4 6/9] inet: split syncookie keys for ipv4 and ipv6 and initialize with net_get_random_once

2013-10-19 Thread Hannes Frederic Sowa
This patch splits the secret key for syncookies for ipv4 and ipv6 and initializes them with net_get_random_once. This change was the reason I did this series. I think the initialization of the syncookie_secret is way to early. Cc: Florian Westphal Cc: Eric Dumazet Cc: "David S. Miller" Signed-o

Linux 3.12-rc6

2013-10-19 Thread Linus Torvalds
I'm at PDX, about to fly out to the kernel summit, and it has almost become a tradition to do an rc release using the airport wifi. So here it is.. The patch is still busily being pushed out, but the git trees are up-to-date, and the tar-file should already be out. Nothing major happened last week

[PATCH net-next v4 7/9] inet: convert inet_ehash_secret and ipv6_hash_secret to net_get_random_once

2013-10-19 Thread Hannes Frederic Sowa
Initialize the ehash and ipv6_hash_secrets with net_get_random_once. Each compilation unit gets its own secret now: ipv4/inet_hashtables.o ipv4/udp.o ipv6/inet6_hashtables.o ipv6/udp.o rds/connection.o The functions still get inlined into the hashing functions. In the fast path we have

[PATCH net-next v4 5/9] net: introduce new macro net_get_random_once

2013-10-19 Thread Hannes Frederic Sowa
net_get_random_once is a new macro which handles the initialization of secret keys. It is possible to call it in the fast path. Only the initialization depends on the spinlock and is rather slow. Otherwise it should get used just before the key is used to delay the entropy extration as late as poss

[PATCH net-next v4 0/9] Introduce support to lazy initialize mostly static keys

2013-10-19 Thread Hannes Frederic Sowa
Hi! This series implements support for delaying the initialization of secret keys, e.g. used for hashing, for as long as possible. This functionality is implemented by a new macro, net_get_random_bytes. I already used it to protect the socket hashes, the syncookie secret (most important) and the

[PATCH net-next v4 8/9] tcp: switch tcp_fastopen key generation to net_get_random_once

2013-10-19 Thread Hannes Frederic Sowa
Changed key initialization of tcp_fastopen cookies to net_get_random_once. If the user sets a custom key net_get_random_once must be called at least once to ensure we don't overwrite the user provided key when the first cookie is generated later on. Cc: Yuchung Cheng Cc: Eric Dumazet Cc: "David

[PATCH net-next v4 1/9] ipv4: split inet_ehashfn to hash functions per compilation unit

2013-10-19 Thread Hannes Frederic Sowa
This duplicates a bit of code but let's us easily introduce separate secret keys later. The separate compilation units are ipv4/inet_hashtabbles.o, ipv4/udp.o and rds/connection.o. Cc: Eric Dumazet Cc: "David S. Miller" Signed-off-by: Hannes Frederic Sowa --- include/net/inet_sock.h| 22 ++

[PATCH net-next v4 2/9] ipv6: split inet6_ehashfn to hash functions per compilation unit

2013-10-19 Thread Hannes Frederic Sowa
This patch splits the inet6_ehashfn into separate ones in ipv6/inet6_hashtables.o and ipv6/udp.o to ease the introduction of seperate secrets keys later. Cc: Eric Dumazet Cc: "David S. Miller" Signed-off-by: Hannes Frederic Sowa --- include/net/inet6_hashtables.h | 28 +++--

[PATCH net-next v4 9/9] net: switch net_secret key generation to net_get_random_once

2013-10-19 Thread Hannes Frederic Sowa
Cc: Eric Dumazet Cc: "David S. Miller" Signed-off-by: Hannes Frederic Sowa --- net/core/secure_seq.c | 14 ++ 1 file changed, 2 insertions(+), 12 deletions(-) diff --git a/net/core/secure_seq.c b/net/core/secure_seq.c index 3f1ec15..b02fd16 100644 --- a/net/core/secure_seq.c +++ b/

Re: [PATCH] tracing: fix referencing after memory freeing and refactors code

2013-10-19 Thread Geyslan Gregório Bem
2013/10/19 Steven Rostedt : > On Sat, 19 Oct 2013 11:41:10 -0300 > Geyslan Gregório Bem wrote: > >> Let's wait Steve's reply about further use of tracing_is_disabled(). > > Might want to ping me later. This weekend I'm taking my daughter to > colleges, and early Monday morning I'm leaving to Edinb

Re: [PATCH 2/6] x86: allow to call text_poke_bp during boot

2013-10-19 Thread Steven Rostedt
[ Added Paul because he'll understand this ] On Sat, 19 Oct 2013 15:16:58 -0400 Steven Rostedt wrote: > On Sun, 20 Oct 2013 00:02:32 +0900 > Masami Hiramatsu wrote: > > > (2013/10/18 23:27), Petr Mladek wrote: > > > We would like to use text_poke_bp in ftrace. It might be called also > > > d

Re: [PATCH 2/6] x86: allow to call text_poke_bp during boot

2013-10-19 Thread Steven Rostedt
On Sun, 20 Oct 2013 00:02:32 +0900 Masami Hiramatsu wrote: > (2013/10/18 23:27), Petr Mladek wrote: > > We would like to use text_poke_bp in ftrace. It might be called also during > > boot when the interupts are disabled. We need to enable them for syncing > > the cores on each CPU. Otherwise, th

Re: [PATCH] tracing: fix referencing after memory freeing and refactors code

2013-10-19 Thread Steven Rostedt
On Sat, 19 Oct 2013 11:41:10 -0300 Geyslan Gregório Bem wrote: > Let's wait Steve's reply about further use of tracing_is_disabled(). Might want to ping me later. This weekend I'm taking my daughter to colleges, and early Monday morning I'm leaving to Edinburgh (arriving on Tuesday). -- Steve

Re: 3.12-rcX - NFS regression - kswapd0 / kswapd1 stays using 100% CPU?

2013-10-19 Thread Helge Deller
On 10/18/2013 10:12 PM, Myklebust, Trond wrote: > On Fri, 2013-10-18 at 22:03 퍭, Helge Deller wrote: >> On 10/18/2013 09:36 PM, Myklebust, Trond wrote: >>> Also, could you please try a sysRQ-t the next time it happens, so that >>> we can get a trace of where the mount program is hanging. Knowing th

Re: [PATCH v2] videobuf2: Add missing lock held on vb2_fop_relase

2013-10-19 Thread Sylwester Nawrocki
On 10/19/2013 06:07 PM, Ricardo Ribalda wrote: [...] --- drivers/media/platform/exynos4-is/fimc-capture.c | 2 +- drivers/media/platform/exynos4-is/fimc-lite.c| 2 +- drivers/media/usb/em28xx/em28xx-video.c | 2 +- drivers/media/v4l2-core/videobuf2-core.c | 18 +++

[PATCH 2/2] Added support for the monitor hotkey present on laptops like the Dell N4110

2013-10-19 Thread Souvik Banerjee
Adds support for the monitor switching hotkey on laptops such as the N4110. Signed-off-by: Souvik Banerjee --- drivers/platform/x86/dell-wmi.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/platform/x86/dell-wmi.c b/drivers/platform/x86/dell-wmi.c index 5adb60a..a880183 100644 ---

[PATCH 1/2] Fixed style issue. Changed tabs to 8 spaces with expand(1).

2013-10-19 Thread Souvik Banerjee
Fixed a coding style issue. Tabs have to be 8 spaces. Signed-off-by: Souvik Banerjee --- drivers/platform/x86/dell-wmi.c | 396 1 file changed, 198 insertions(+), 198 deletions(-) diff --git a/drivers/platform/x86/dell-wmi.c b/drivers/platform/x86/dell-

Re: int3 doing rcu_read_lock()

2013-10-19 Thread Paul E. McKenney
On Fri, Oct 18, 2013 at 11:13:51PM -0400, Steven Rostedt wrote: > Hey Paul, > > I hit this in my tests: > > [ 1597.688015] === > [ 1597.688015] [ INFO: suspicious RCU usage. ] > [ 1597.688015] 3.12.0-rc4-test+ #48 Not tainted > [ 1597.688015] --

Re: net: sctp: possible dereference after freeing

2013-10-19 Thread Vlad Yasevich
On Oct 19, 2013, at 7:49 AM, Geyslan Gregório Bem wrote: > 2013/10/19 Geyslan Gregório Bem : >> Hi maintainers, >> >> I would like to know if these are catches: >> >> /net/sctp/endpointola.c (281) >> static void sctp_endpoint_destroy(struct sctp_endpoint *ep) >> { >>struct sock *sk; >> ..

Re: [PATCH v2 02/13] uprobes: allow ignoring of probe hits

2013-10-19 Thread Oleg Nesterov
On 10/15, David Long wrote: > > @@ -1732,9 +1732,6 @@ static void handle_swbp(struct pt_regs *regs) > return; > } > > - /* change it in advance for ->handler() and restart */ > - instruction_pointer_set(regs, bp_vaddr); > - Well, this looks obviously wrong. This SET_I

Re: linux-next: Tree for Oct 15 (asymmetric keys)

2013-10-19 Thread Randy Dunlap
On 10/15/13 15:43, Randy Dunlap wrote: > On 10/15/13 07:02, Thierry Reding wrote: >> Hi all, >> >> I've uploaded today's linux-next tree to the master branch of the >> repository below: >> >> git://gitorious.org/thierryreding/linux-next.git >> >> A next-20131015 tag is also provided for con

Re: [PATCH v2 05/13] uprobes: add arch write opcode hook

2013-10-19 Thread Oleg Nesterov
On 10/15, David Long wrote: > > Allow arches to write the opcode with a custom function. ARM needs to > customize the swbp instruction depending on the condition code of the > instruction it replaces. Well, we already have "__weak set_swbp(auprobe, ...)", can't arm use it? If not, > +void __wea

Re: [PATCH v2 04/13] uprobes: allow arch-specific initialization

2013-10-19 Thread Oleg Nesterov
On 10/15, David Long wrote: > > Add a weak function for any architecture-specific initialization. ARM > will use this to register the handlers for the undefined instructions it > uses to implement uprobes. Could you explain why ARM can't simply do the necessary initialization in arch/arm/kernel/u

Re: [PATCH v2 03/13] uprobes: allow arch access to xol slot

2013-10-19 Thread Oleg Nesterov
On 10/15, David Long wrote: > > Allow arches to customize how the instruction is filled into the xol > slot. ARM will use this to insert an undefined instruction after the > real instruction in order to simulate a single step of the instruction > without hardware support. OK, but > +void __weak

Re: [PATCH] typo fixes (coordiante -> coordinate) in am335x

2013-10-19 Thread Jan Lübbe
On Wed, 2013-08-21 at 00:58 -0700, Tony Lindgren wrote: > * Zubair Lutfullah [130715 08:33]: > > Did a grep for coordiante and replaced them all > > with coordinate. > > > > This applies to the mfd-next tree. > > This should be safe to apply via the MFD tree as a non-critical > fix assuming the

[PATCH] vsprintf: add Bluetooth UUID %pU[rR] format specifier

2013-10-19 Thread Marcel Holtmann
The Bluetooth UUID is used in big endian reversed order. Add new modifier to print a UUID in big endian, but where the input byte stream is actually in reversed order. This is similar to %pMR that allows to print a MAC address in reversed order since that is how the Bluetooth BD_ADDR is actually r

[PATCH 1/1] hung_task debugging: Add tracepoint to report the hang

2013-10-19 Thread Oleg Nesterov
Currently check_hung_task() prints a warning if it detects the problem, but it is not convenient to watch the system logs if user-space wants to be notified about the hang. Add the new trace_sched_process_hang() into check_hung_task(), this way a user-space monitor can easily wait for the hang and

[PATCH 0/1] hung_task debugging: Add tracepoint to report the hang

2013-10-19 Thread Oleg Nesterov
Hi, We have a feature request, the customer needs something more hookable than just printk's from check_hung_task() to implement the user-space watchdog which can potentially resolve the problems which caused the hang. The patch simply adds a tracepoint into check_hung_task(), do you think we can

Re: [PATCH 4/5] sched: Refactor iowait accounting

2013-10-19 Thread Frederic Weisbecker
On Sat, Oct 19, 2013 at 06:05:17PM +0200, Peter Zijlstra wrote: > On Sat, Oct 19, 2013 at 06:02:37PM +0200, Frederic Weisbecker wrote: > > > I just had a look at delayacct; wth wrote that crap; that too uses > > > gtod. > > > > I can't find where it does that. kernel/delayacct.c doesn't seem to at

Re: [PATCH] videobuf2: Add missing lock held on vb2_fop_relase

2013-10-19 Thread Ricardo Ribalda Delgado
Hello Sylwester I have just posted a new version. Please take a look to it, it should fix your issue. I havent tried it in hw because I am out of the office. Regards! On Sat, Oct 19, 2013 at 12:51 PM, Sylwester Nawrocki wrote: > On 10/19/2013 12:22 PM, Ricardo Ribalda Delgado wrote: >> >> On S

[PATCH v2] videobuf2: Add missing lock held on vb2_fop_relase

2013-10-19 Thread Ricardo Ribalda
vb2_fop_relase does not held the lock although it is modifying the queue->owner field. This could lead to race conditions on the vb2_perform_io function when multiple applications are accessing the video device via read/write API: [ 308.297741] BUG: unable to handle kernel NULL pointer dereferenc

Re: [PATCH 4/5] sched: Refactor iowait accounting

2013-10-19 Thread Peter Zijlstra
On Sat, Oct 19, 2013 at 06:02:37PM +0200, Frederic Weisbecker wrote: > > I just had a look at delayacct; wth wrote that crap; that too uses > > gtod. > > I can't find where it does that. kernel/delayacct.c doesn't seem to at least. Look for do_posix_clock_monotonic_gettime() ;-) -- To unsubscribe

Re: [PATCH v5 0/3] ARM: S5PV210: move to common clk framework

2013-10-19 Thread Tomasz Figa
Hi Mike, Kukjin, Rafael, On Tuesday 24 of September 2013 14:50:06 Mateusz Krawczuk wrote: > This patch series is the new s5pv210 clock implementation > (using common clk framework). > > This implementation is compatible with device tree definition and board > files. > > This patch series is base

Re: [PATCH 4/5] sched: Refactor iowait accounting

2013-10-19 Thread Frederic Weisbecker
On Sat, Oct 19, 2013 at 05:35:35PM +0200, Peter Zijlstra wrote: > On Sat, Oct 19, 2013 at 05:17:20PM +0200, Frederic Weisbecker wrote: > > +u64 get_cpu_iowait_time_us(int cpu, u64 *last_update_time) > > +{ > > + ktime_t iowait, delta = { .tv64 = 0 }; > > + struct rq *rq = cpu_rq(cpu); > > + k

Re: kprobe pre_handler change return IP

2013-10-19 Thread Masami Hiramatsu
(2013/10/17 21:57), Liuyongan wrote: > I use kprobe to probe a function suppose: > int is_winter(int num) { ... } > int replace_is_winter(int num) { ...} > I want to replace is_winter() with replace_is_winter(), so when we call > is_winter, replace_is_winter will be called. >

Re: [PATCH v3 2/3] Support for perf to probe into SDT markers:

2013-10-19 Thread Masami Hiramatsu
(2013/10/18 23:44), Hemant Kumar wrote: > This allows perf to probe into the sdt markers/notes present in > the libraries and executables. We try to find the associated location > and handle prelinking (since, stapsdt notes section is not allocated > during runtime). Prelinking is handled with the

Re: [PATCH 4/5] sched: Refactor iowait accounting

2013-10-19 Thread Peter Zijlstra
On Sat, Oct 19, 2013 at 05:17:20PM +0200, Frederic Weisbecker wrote: > +u64 get_cpu_iowait_time_us(int cpu, u64 *last_update_time) > +{ > + ktime_t iowait, delta = { .tv64 = 0 }; > + struct rq *rq = cpu_rq(cpu); > + ktime_t now = ktime_get(); > + unsigned int seq; > + > + do { >

Re: [PATCH v3 1/3] SDT markers listing by perf:

2013-10-19 Thread Masami Hiramatsu
(2013/10/18 23:44), Hemant Kumar wrote: > This patch will enable perf to list all the sdt markers present > in an elf file. The markers are present in the .note.stapsdt section > of the elf. We can traverse through this section and collect the > required info about the markers. > We can use '-M/--m

[PATCH 4/5] sched: Refactor iowait accounting

2013-10-19 Thread Frederic Weisbecker
The iowait time accounting has awkward semantics. It is performed per CPU. The iowait time of a CPU starts when a task sleeps on IO on this CPU and stops when that task later wakes up on any CPU. This assumes that a sleeping task is still assigned to the CPU it was running on last until it wakes u

[PATCH 2/5] nohz: Only update sleeptime stats locally

2013-10-19 Thread Frederic Weisbecker
The idle and io sleeptime stats can be updated concurrently from callers of get_cpu_idle_time_us(), get_cpu_iowait_time_us() and tick_nohz_stop_idle(). Updaters can easily race and mess up with internal datas coherency, for example when a governor calls a get_cpu_*_time_us() API and the target CPU

[RFC PATCH 0/5] nohz: Fix racy sleeptime stats v2

2013-10-19 Thread Frederic Weisbecker
Hi, So this is a new iteration with some more changes on top of the latest discussions we had. Unfortunately it's not yet a real viable solution. Check out patch 4/5 for details, I think that's too much overhead. Also it changes enough the semantics to break the idle sleeptime accounting along

[PATCH 5/5] nohz: Synchronize sleep time stats with seqlock

2013-10-19 Thread Frederic Weisbecker
When some call site uses get_cpu_*_time_us() to read a sleeptime stat, it deduces the total sleeptime by adding the pending time to the last sleeptime snapshot if the CPU target is idle. Namely this sums up to: sleeptime = ts($CPU)->idle_sleeptime; if (ts($CPU)->idle_active)

[PATCH 1/5] nohz: Convert a few places to use local per cpu accesses

2013-10-19 Thread Frederic Weisbecker
A few functions use remote per CPU access APIs when they deal with local values. Just to the right conversion to improve performance, code readability and debug checks. Signed-off-by: Frederic Weisbecker Cc: Fernando Luis Vazquez Cao Cc: Tetsuo Handa Cc: Thomas Gleixner Cc: Ingo Molnar Cc: P

[PATCH 3/5] timer: Change idle/iowait accounting semantics

2013-10-19 Thread Frederic Weisbecker
To prepare for fixing a race between iowait and idle time stats, this patch changes the following semantics: * iowait time is going to be accounted from the scheduler rather than the dynticks idle code, lets remove it from the /proc/timer_list dump. * idle sleeptime now also includes the iowait t

Re: [PATCH 2/6] x86: allow to call text_poke_bp during boot

2013-10-19 Thread Masami Hiramatsu
(2013/10/18 23:27), Petr Mladek wrote: > We would like to use text_poke_bp in ftrace. It might be called also during > boot when the interupts are disabled. We need to enable them for syncing > the cores on each CPU. Otherwise, there might be a deadlock, see the > warning in "smp_call_function_many

Re: [PATCH 1/6] x86: speed up int3-based patching using less paranoid write

2013-10-19 Thread Masami Hiramatsu
(2013/10/18 23:27), Petr Mladek wrote: > This change is inspired by the int3-based patching code used in > ftrace. See the commit fd4363fff3d9 (x86: Introduce int3 > (breakpoint)-based instruction patching). > > When trying to use text_poke_bp in ftrace, the result was slower > than the original i

[PATCHv4 05/16] staging: usbip: Add support for ACLs in usbipd

2013-10-19 Thread Dominik Paulus
Interpret the ACLs stored in sysfs in usbipd and reject clients not matching one of the ACLs. Signed-off-by: Kurt Kanzenbach Signed-off-by: Dominik Paulus Signed-off-by: Tobias Polzer --- drivers/staging/usbip/userspace/src/Makefile.am | 2 +- drivers/staging/usbip/userspace/src/usbipd.c|

[PATCHv4 08/16] staging: usbip: Improve debug output

2013-10-19 Thread Dominik Paulus
For IPv6, IP:Port is unreadable. Signed-off-by: Dominik Paulus Signed-off-by: Tobias Polzer --- drivers/staging/usbip/userspace/src/usbipd.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/usbip/userspace/src/usbipd.c b/drivers/staging/usbip/userspace/src/us

[PATCHv4 06/16] staging: usbip: Add proper error reporting

2013-10-19 Thread Dominik Paulus
This patch adds new error codes and features extended error reporting in op_common packets. Signed-off-by: Dominik Paulus Signed-off-by: Tobias Polzer --- drivers/staging/usbip/userspace/src/usbip_attach.c | 4 +- drivers/staging/usbip/userspace/src/usbip_list.c | 3 +- .../staging/usbip/us

[PATCHv4 01/16] staging: usbip: Add support for client authentication

2013-10-19 Thread Dominik Paulus
This patch adds support for authenticating both client and server using a pre-shared passphrase using SRP (Secure Remote Password) over TLS (see RFC 5054) using GnuTLS. Both usbip and usbipd now accept a shared secret as a command line argument. Currently, the established TLS connection is only use

[PATCHv4 07/16] staging: usbip: Handle usbip being started as user

2013-10-19 Thread Dominik Paulus
usbip now prints an error message when started as user and requiring root access. Also, some debug messages are changed to error messages so the command line utilities now print less confusing (and more verbose) error messages when not used correctly. Signed-off-by: Dominik Paulus Signed-off-by:

[PATCHv4 00/16] staging: usbip: Add kernel support for client ACLs

2013-10-19 Thread Dominik Paulus
Hi, sorry for the messed-up error checking. We fixed some of the style nitpicks and hopefully all of the mistakes you pointed out (thank you for that!). Regards, Dominik and Tobias -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majo

[PATCHv4 09/16] staging: usbip: Separate protocol/program version

2013-10-19 Thread Dominik Paulus
Not all new program versions necessarily introduce non-backwards-compatible protocol changes. We thus move the definition of the protocol version from configure.ac to usbip_network.h, where it logically belongs to. Signed-off-by: Dominik Paulus Signed-off-by: Tobias Polzer --- drivers/staging/u

  1   2   >