Re: [GIT PULL v2] irq fix for 3.12-rc

2013-10-01 Thread Ingo Molnar
* Frederic Weisbecker wrote: > > > Yeah, I did such things sporadically before. Well, it summed up to > > > simply remove the timestamps from backtraces but yeah, then I've > > > become less patient about that and now I simply paste the raw thing. > > > > > > I'll take care of that and prune

Re: [PATCH] mm: pagevec: cleanup: drop pvec->cold argument in all places

2013-10-01 Thread David Rientjes
On Mon, 30 Sep 2013, Andrew Morton wrote: > > Nobody uses the pvec->cold argument of pagevec and it's also unreasonable > > for > > pages in pagevec released as cold page, so drop the cold argument from > > pagevec. > > Is it unreasonable? I'd say it's unreasonable to assume that all pages >

Re: [PATCHv3 4/4] gpu: host1x: Add runtime pm support for host1x

2013-10-01 Thread Arto Merilainen
On 10/01/2013 09:17 PM, Stephen Warren wrote: On 09/24/2013 06:05 AM, Arto Merilainen wrote: From: Mayuresh Kulkarni This patch adds runtime pm support for host1x hardware unit. This allows host1x clock to be turned off when it is idle. If pm runtime is not configured, we enable host1x clock

Re: [PATCHv3 2/4] drm/tegra: Add runtime pm support for gr2d

2013-10-01 Thread Arto Merilainen
On 10/01/2013 09:14 PM, Stephen Warren wrote: On 09/24/2013 06:05 AM, Arto Merilainen wrote: diff --git a/drivers/gpu/host1x/drm/gr2d.c b/drivers/gpu/host1x/drm/gr2d.c @@ -327,11 +336,48 @@ static int __exit gr2d_remove(struct platform_device *pdev)

Re: [PATCH 1/4] kconfig: add short explanation to SYMBOL_WRITE

2013-10-01 Thread David Rientjes
On Wed, 2 Oct 2013, Martin Walch wrote: > diff --git a/scripts/kconfig/expr.h b/scripts/kconfig/expr.h > index df198a5..ba663e1 100644 > --- a/scripts/kconfig/expr.h > +++ b/scripts/kconfig/expr.h > @@ -93,7 +93,7 @@ struct symbol { > #define SYMBOL_CHOICEVAL 0x0020 /* used as a value in a

Re: [PATCH v6 0/7] Kernel base address randomization

2013-10-01 Thread Kees Cook
On Tue, Oct 1, 2013 at 10:30 PM, H. Peter Anvin wrote: > On 10/01/2013 10:25 PM, Ingo Molnar wrote: >> >> I mean, for example in an oops message we print data in words: the RIP, >> other registers and stack contents. If any of these values lies within the >> randomization range then we could

Re: [PATCH 3/4] kconfig: adjust warning message for conflicting types

2013-10-01 Thread David Rientjes
On Wed, 2 Oct 2013, Martin Walch wrote: > diff --git a/scripts/kconfig/menu.c b/scripts/kconfig/menu.c > index b970101..e511df5 100644 > --- a/scripts/kconfig/menu.c > +++ b/scripts/kconfig/menu.c > @@ -119,7 +119,7 @@ void menu_set_type(int type) > sym->type = type; >

Re: [PATCH v6 0/7] Kernel base address randomization

2013-10-01 Thread H. Peter Anvin
On 10/01/2013 10:25 PM, Ingo Molnar wrote: > > I mean, for example in an oops message we print data in words: the RIP, > other registers and stack contents. If any of these values lies within the > randomization range then we could de-randomize it. > > So instead of exposing randomized values,

Re: [PATCH v6 0/7] Kernel base address randomization

2013-10-01 Thread Ingo Molnar
* H. Peter Anvin wrote: > I think that the randomization offset would be necessary in order to > identify pointers. I mean, for example in an oops message we print data in words: the RIP, other registers and stack contents. If any of these values lies within the randomization range then we

[PATCH 4/4] kconfig: remove unused definition from scanner

2013-10-01 Thread Martin Walch
From: Martin Walch Date: Wed, 2 Oct 2013 06:58:46 +0200 Subject: [PATCH 4/4] kconfig: remove unused definition from scanner The definition ws [ \n\t] is not used anywhere. Drop it to avoid confusion. Signed-off-by: Martin Walch --- scripts/kconfig/zconf.l | 1 - 1 file changed, 1 deletion(-)

[PATCH 3/4] kconfig: adjust warning message for conflicting types

2013-10-01 Thread Martin Walch
From: Martin Walch Date: Wed, 2 Oct 2013 06:58:01 +0200 Subject: [PATCH 3/4] kconfig: adjust warning message for conflicting types Adjust warning: a second symbol type does *not* replace the first one. Signed-off-by: Martin Walch --- scripts/kconfig/menu.c | 2 +- 1 file changed, 1

[PATCH 1/4] kconfig: add short explanation to SYMBOL_WRITE

2013-10-01 Thread Martin Walch
From: Martin Walch Date: Wed, 2 Oct 2013 06:54:27 +0200 Subject: [PATCH 1/4] kconfig: add short explanation to SYMBOL_WRITE replace the question mark in the comment after SYMBOL_WRITE with an explanation Signed-off-by: Martin Walch --- scripts/kconfig/expr.h | 2 +- 1 file changed, 1

[PATCH 0/4]kconfig: minor cleanups and corrections

2013-10-01 Thread Martin Walch
Hello, these are some simple fixes to correct minor things I came across while reading the code. Martin Walch (4): kconfig: add short explanation to SYMBOL_WRITE kconfig: trivial: fix typos in comments kconfig: adjust warning message for conflicting types kconfig: remove unused

[PATCH 2/4] kconfig: trivial: fix typos

2013-10-01 Thread Martin Walch
From: Martin Walch Date: Wed, 2 Oct 2013 06:56:45 +0200 Subject: [PATCH 2/4] kconfig: trivial: fix typos inser() -> insert(), tell -> tells, peoperty -> property Signed-off-by: Martin Walch --- scripts/kconfig/mconf.c | 2 +- scripts/kconfig/menu.c | 2 +- scripts/kconfig/symbol.c | 2 +-

Re: [PATCH] Staging : android: binder.c: Prefer seq_puts to seq_printf

2013-10-01 Thread Ryan Mallon
On 02/10/13 11:57, Mathieu Rhéaume wrote: > This patch changes seq_printf for seq_puts in binder.c. > It fixes the warnings emitted by checkpatch.pl. > > Signed-off-by: Mathieu Rhéaume > --- > drivers/staging/android/binder.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > >

[PATCH] sparc: fix ldom_reboot buffer overflow harder

2013-10-01 Thread Kees Cook
The length argument to strlcpy was still wrong. It could overflow the end of full_boot_str by 5 bytes. Instead of strcat and strlcpy, just use snprint. Reported-by: Brad Spengler Signed-off-by: Kees Cook --- arch/sparc/kernel/ds.c |5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-)

Re: spinlock contention of files->file_lock

2013-10-01 Thread Ingo Molnar
* Al Viro wrote: > On Tue, Oct 01, 2013 at 02:41:58PM -0700, Eric Dumazet wrote: > > Maybe I am missing something obvious ? > > Yes. do_execve_common() starts with unshare_files(); there can be > no other thread capable of modifying that descriptor table. Btw., might the Android Binder:

Re: [PATCH v6 0/7] Kernel base address randomization

2013-10-01 Thread H. Peter Anvin
I think that the randomization offset would be necessary in order to identify pointers. Ingo Molnar wrote: > >* Kees Cook wrote: > >> Here is the latest version of the kASLR series. It has much improved >> e820 walking code, and expands the window available on 64-bit. >> >> This is rolled

Re: [PATCH 3/3] KVM: PPC: Book3S: Add support for hwrng found on some powernv systems

2013-10-01 Thread Paul Mackerras
On Tue, Oct 01, 2013 at 01:19:06PM +0200, Paolo Bonzini wrote: > Anyhow, I would like to know more about this hwrng and hypercall. > > Does the hwrng return random numbers (like rdrand) or real entropy (like > rdseed that Intel will add in Broadwell)? What about the hypercall? Well, firstly,

Re: [PATCH v6 0/7] Kernel base address randomization

2013-10-01 Thread Ingo Molnar
* Kees Cook wrote: > Here is the latest version of the kASLR series. It has much improved > e820 walking code, and expands the window available on 64-bit. > > This is rolled out on Chrome OS devices, and working well. There's one kernel debuggability detail that should be discussed I think:

[PATCH v3] Staging : android: binder.c: Prefer seq_puts to seq_printf

2013-10-01 Thread Mathieu Rhéaume
Changes since v2: * Fixed the indent in the method calls that had too many tab. >8--8< This patch changes seq_printf for seq_puts in binder.c. It fixes the warnings emitted by checkpatch.pl. Signed-off-by: Mathieu Rhéaume ---

Re: [PATCH] Staging: android: binder.c: Fixed 80 characters warnings from checkpatch.pl

2013-10-01 Thread Greg KH
On Tue, Oct 01, 2013 at 06:46:44PM -0700, Joe Perches wrote: > On Tue, 2013-10-01 at 21:39 -0400, Mathieu Rhéaume wrote: > > Fixed a lot of warnings from checkpatch.pl about the 80 characters per line > > limitation. > > I'd prefer > s/binder_user_error/binder_user_err/ > or >

Re: [PATCH v2] Staging : android: binder.c: Prefer seq_puts to seq_printf

2013-10-01 Thread Joe Perches
On Tue, 2013-10-01 at 23:57 -0400, Mathieu Rhéaume wrote: > Changes since v2: > * Fixed the indent problems (2 tab and 1 space) by Joe Perches. > * Found more calls to convert from seq_printf to seq_puts [] > diff --git a/drivers/staging/android/binder.c > b/drivers/staging/android/binder.c []

Re: applesmc oops in 3.10/3.11

2013-10-01 Thread Guenter Roeck
On 10/01/2013 08:55 PM, Chris Murphy wrote: On Oct 1, 2013, at 9:51 PM, Guenter Roeck wrote: On Tue, Oct 01, 2013 at 07:09:26PM -0600, Chris Murphy wrote: On Oct 1, 2013, at 10:24 AM, Guenter Roeck wrote: On Tue, Oct 01, 2013 at 09:33:13AM -0600, Chris Murphy wrote: On Oct 1, 2013, at

[PATCH v2] Staging : android: binder.c: Prefer seq_puts to seq_printf

2013-10-01 Thread Mathieu Rhéaume
Changes since v2: * Fixed the indent problems (2 tab and 1 space) by Joe Perches. * Found more calls to convert from seq_printf to seq_puts >8--8< This patch changes seq_printf for seq_puts in binder.c. It fixes the warnings emitted by

Re: applesmc oops in 3.10/3.11

2013-10-01 Thread Chris Murphy
On Oct 1, 2013, at 9:51 PM, Guenter Roeck wrote: > On Tue, Oct 01, 2013 at 07:09:26PM -0600, Chris Murphy wrote: >> >> On Oct 1, 2013, at 10:24 AM, Guenter Roeck wrote: >> >>> On Tue, Oct 01, 2013 at 09:33:13AM -0600, Chris Murphy wrote: On Oct 1, 2013, at 9:19 AM, Guenter Roeck

Re: applesmc oops in 3.10/3.11

2013-10-01 Thread Guenter Roeck
On Tue, Oct 01, 2013 at 07:09:26PM -0600, Chris Murphy wrote: > > On Oct 1, 2013, at 10:24 AM, Guenter Roeck wrote: > > > On Tue, Oct 01, 2013 at 09:33:13AM -0600, Chris Murphy wrote: > >> > >> On Oct 1, 2013, at 9:19 AM, Guenter Roeck wrote: > >> > >>> On Tue, Oct 01, 2013 at 12:55:26PM

Re: [PATCH V1] oom: avoid selecting threads sharing mm with init

2013-10-01 Thread David Rientjes
On Fri, 27 Sep 2013, Ming Liu wrote: > I might mislead you, when I talked about init, I meant the pid 1 process but > not the idle, and isn't the idle a kthread and has not this risk getting > killed by oom? You can disqualify for p->mm == _mm, but the oom killer has been rewritten since 2.6.27

Re: [PATCH v2 2/6] PCI/MSI: Factor out pci_get_msi_cap() interface

2013-10-01 Thread Tejun Heo
On Wed, Oct 02, 2013 at 12:33:38PM +1000, Michael Ellerman wrote: > > It is an interface which forces the driver writers to write > > complicated fallback code which won't usually be excercised. > > It does not force anyone to do anything. That's just bull. Yeah, sure, we don't have shitty

[Suggestion] about calling debug_hotplug_cpu() which enabled by 'allmodconfig' for a x86_64 dual core laptop.

2013-10-01 Thread Chen Gang
Hello Maintainers: Under my x86_64 dual core laptop, I build kernel next-20130927 with 'allmodconfig', and install it, the machine can not start. Related information is: after call debug_hotplug_cpu(), output "cpu 0 is offline" and then "Failed to execute /init". After remove

Re: [patch for-3.12] mm, memcg: protect mem_cgroup_read_events for cpu hotplug

2013-10-01 Thread David Rientjes
On Tue, 1 Oct 2013, Johannes Weiner wrote: > On Tue, Oct 01, 2013 at 04:31:23PM -0700, David Rientjes wrote: > > for_each_online_cpu() needs the protection of {get,put}_online_cpus() so > > cpu_online_mask doesn't change during the iteration. > > There is no problem report here. > > Is there a

Re: [PATCH] perf stat: Set child_pid after perf_evlist__prepare_workload()

2013-10-01 Thread Namhyung Kim
On Tue, 1 Oct 2013 13:51:33 -0300, Arnaldo Carvalho de Melo wrote: > Em Mon, Sep 30, 2013 at 07:47:25AM -0600, David Ahern escreveu: >> On 9/30/13 3:01 AM, Namhyung Kim wrote: >> >From: Namhyung Kim >> > >> >The commit acf2892270dc ("perf stat: Use perf_evlist__prepare/ >> >start_workload()")

Re: [PATCH v2 2/6] PCI/MSI: Factor out pci_get_msi_cap() interface

2013-10-01 Thread Mark Lord
On 13-09-26 09:03 AM, Alexander Gordeev wrote: > On Thu, Sep 26, 2013 at 08:32:53AM -0400, Mark Lord wrote: >> On 13-09-18 05:48 AM, Alexander Gordeev wrote: >>> The last pattern makes most of sense to me and could be updated with a more >>> clear sequence - a call to (bit modified)

Re: [PATCH v2 2/6] PCI/MSI: Factor out pci_get_msi_cap() interface

2013-10-01 Thread Michael Ellerman
On Tue, Oct 01, 2013 at 12:35:27PM +0200, Alexander Gordeev wrote: > On Tue, Oct 01, 2013 at 05:51:33PM +1000, Michael Ellerman wrote: > > The disadvantage is that any restriction imposed on us above the quota > > can only be reported as an error from pci_enable_msix(). > > > > The quota code,

Re: [ 70/71] cw1200: Dont perform SPI transfers in interrupt context

2013-10-01 Thread Solomon Peachy
On Sun, Sep 29, 2013 at 12:28:22PM -0700, Greg Kroah-Hartman wrote: > 3.11-stable review patch. If anyone has any objections, please let me know. If it's not too late, please drop this patch. Sorry aobut the delay in responding, I was travelling internationally and had very sporadic

Re: RFC: (re-)binding the VFIO platform driver to a platform device

2013-10-01 Thread Alex Williamson
On Wed, 2013-10-02 at 02:53 +0100, Christoffer Dall wrote: > On Tue, Oct 01, 2013 at 05:02:44PM -0500, Kim Phillips wrote: > > On Tue, 1 Oct 2013 13:00:54 -0700 > > Greg Kroah-Hartman wrote: > > > > > On Tue, Oct 01, 2013 at 01:38:31PM -0500, Kim Phillips wrote: > > > > Hi, > > > > > > > >

Re: [PATCH v2 2/6] PCI/MSI: Factor out pci_get_msi_cap() interface

2013-10-01 Thread Michael Ellerman
On Tue, Oct 01, 2013 at 07:55:03AM -0400, Tejun Heo wrote: > Hello, > > On Tue, Oct 01, 2013 at 05:35:48PM +1000, Michael Ellerman wrote: > > > > Roughly third of the drivers just do not care and bail out once > > > > pci_enable_msix() has not succeeded. Not sure how many of these are > > > >

Re: [patch for-3.12] mm, memcg: protect mem_cgroup_read_events for cpu hotplug

2013-10-01 Thread Johannes Weiner
On Tue, Oct 01, 2013 at 04:31:23PM -0700, David Rientjes wrote: > for_each_online_cpu() needs the protection of {get,put}_online_cpus() so > cpu_online_mask doesn't change during the iteration. There is no problem report here. Is there a crash? If it's just accuracy of the read, why would we

Re: [PATCH] Staging : android: binder.c: Prefer seq_puts to seq_printf

2013-10-01 Thread Joe Perches
On Tue, 2013-10-01 at 21:57 -0400, Mathieu Rhéaume wrote: > This patch changes seq_printf for seq_puts in binder.c. > It fixes the warnings emitted by checkpatch.pl. Please realign function argument to the open parenthesis. > diff --git a/drivers/staging/android/binder.c >

[PATCH net-next] net:drivers/net: Miscellaneous conversions to ETH_ALEN

2013-10-01 Thread Joe Perches
Convert the memset/memcpy uses of 6 to ETH_ALEN where appropriate. Also convert some struct definitions and u8 array declarations of [6] to ETH_ALEN. Signed-off-by: Joe Perches --- drivers/net/ethernet/8390/ax88796.c| 2 +- drivers/net/ethernet/amd/atarilance.c |

Re: [PATCH] [workqueue] check values of pwq and wq in print_worker_info() before use

2013-10-01 Thread John David Anglin
On 1-Oct-13, at 6:50 PM, James Bottomley wrote: On Wed, 2013-10-02 at 00:07 +0200, Helge Deller wrote: On 10/01/2013 11:40 PM, James Bottomley wrote: On Tue, 2013-10-01 at 16:43 -0400, Tejun Heo wrote: Hello, On Tue, Oct 01, 2013 at 10:35:20PM +0200, Helge Deller wrote: print_worker_info()

[PATCH] Staging : android: binder.c: Prefer seq_puts to seq_printf

2013-10-01 Thread Mathieu Rhéaume
This patch changes seq_printf for seq_puts in binder.c. It fixes the warnings emitted by checkpatch.pl. Signed-off-by: Mathieu Rhéaume --- drivers/staging/android/binder.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/android/binder.c

Re: RFC: (re-)binding the VFIO platform driver to a platform device

2013-10-01 Thread Christoffer Dall
On Tue, Oct 01, 2013 at 05:02:44PM -0500, Kim Phillips wrote: > On Tue, 1 Oct 2013 13:00:54 -0700 > Greg Kroah-Hartman wrote: > > > On Tue, Oct 01, 2013 at 01:38:31PM -0500, Kim Phillips wrote: > > > Hi, > > > > > > Santosh and I are having a problem figuring out how to enable binding > > >

Re: [PATCH 0/7] DRA7: DT thermal support

2013-10-01 Thread Eduardo Valentin
On 01-10-2013 18:58, Nishanth Menon wrote: > On 14:32-20131001, Eduardo Valentin wrote: >> Hello all, >> >> This is a complementary patch series with themal DT support for DRA7. >> >> Although this work depends on the thermal dt parser work [1], I decided &g

Re: [PATCH] Staging: android: binder.c: Fixed 80 characters warnings from checkpatch.pl

2013-10-01 Thread Joe Perches
On Tue, 2013-10-01 at 21:39 -0400, Mathieu Rhéaume wrote: > Fixed a lot of warnings from checkpatch.pl about the 80 characters per line > limitation. I'd prefer s/binder_user_error/binder_user_err/ or s/binder_user_error/binder_err/ without doing the strange wrapping checkpatch

[PATCH] Staging: android: binder.c: Fixed 80 characters warnings from checkpatch.pl

2013-10-01 Thread Mathieu Rhéaume
Fixed a lot of warnings from checkpatch.pl about the 80 characters per line limitation. Signed-off-by: Mathieu Rhéaume --- drivers/staging/android/binder.c | 520 ++- 1 file changed, 353 insertions(+), 167 deletions(-) diff --git

Re: [PATCH v2 0/9] procfs: protect /proc//* files with file->f_cred

2013-10-01 Thread Andy Lutomirski
On 10/01/2013 01:26 PM, Djalal Harouni wrote: > /proc//* entries varies at runtime, appropriate permission checks > need to happen during each system call. > > Currently some of these sensitive entries are protected by performing > the ptrace_may_access() check. However even with that the /proc

Re: [PATCH 1/3] clk: Add error handling to clk_fetch_parent_index()

2013-10-01 Thread Mike Turquette
Quoting Tomasz Figa (2013-09-28 17:37:14) > There are at least two different error cases that can happen in > clk_fetch_parent_index() function: > - allocation failure, > - parent clock lookup failure, > however it returns only an u8, which is supposed to contain parent clock > index. > > This

Re: [PATCH v2 6/9] procfs: add permission checks on the file's opener of /proc/*/stat

2013-10-01 Thread Andy Lutomirski
On 10/01/2013 01:26 PM, Djalal Harouni wrote: > Some fields of the /proc/*/stat are sensitive fields that need > appropriate protection. > > However, /proc file descriptors can be passed to a more privileged > process (e.g. a suid-exec) which will pass the classic > ptrace_may_access() permission

Re: [PATCH v2 2/9] procfs: add proc_allow_access() to check if file's opener may access task

2013-10-01 Thread Andy Lutomirski
On 10/01/2013 01:26 PM, Djalal Harouni wrote: > Since /proc entries varies at runtime, permission checks need to happen > during each system call. > > However even with that /proc file descriptors can be passed to a more > privileged process (e.g. a suid-exec) which will pass the classic >

Re: [PATCH 6/7] arm: dts: add tmp102 i2c sensor node on dra7-evm

2013-10-01 Thread Eduardo Valentin
On 01-10-2013 18:51, Nishanth Menon wrote: > On 14:32-20131001, Eduardo Valentin wrote: >> On dra7-evm there is an tmp102 temperature sensor on i2c bus 1. >> This patch adds its device tree node. >> >> Signed-off-by: Eduardo Valentin >> --- >> arch/arm/b

Re: [rfc][possible solution] RCU vfsmounts

2013-10-01 Thread Al Viro
On Mon, Sep 30, 2013 at 08:49:21PM +0100, Al Viro wrote: > OK... AFAICS, we are not too far from being able to handle RCU pathwalk > straying into fs in the middle of being shut down. > * There are 5 methods that can be called: > ->d_hash(...) > ->d_compare(...) > ->d_revalidate(...,

Re: [PATCH 1/7] arm: dts: add dra7 DSPEVE thermal data

2013-10-01 Thread Eduardo Valentin
On 01-10-2013 18:37, Nishanth Menon wrote: > On 14:32-20131001, Eduardo Valentin wrote: >> This patch changes a dtsi file to contain the thermal data >^^ introduces? >> for DSPEVE domain on DRA7 and later SoCs. This data will >> enable a thermal shutdown at

Re: [PATCH 4/6] ARM: shmobile: emev2: Define SMU clock DT bindings

2013-10-01 Thread Simon Horman
On Tue, Oct 01, 2013 at 01:36:33PM +0200, Laurent Pinchart wrote: > Hi Yoshii-san and Simon, > > On Tuesday 24 September 2013 13:52:15 Simon Horman wrote: > > [ Cc Laurent ] > > > > On Tue, Sep 24, 2013 at 01:13:31PM +0900, takas...@ops.dti.ne.jp wrote: > > > Device tree clock binding document

Re: [PATCH 3/7] arm: dts: dra7: add bandgap entry

2013-10-01 Thread Eduardo Valentin
On 01-10-2013 18:46, Nishanth Menon wrote: > On 14:32-20131001, Eduardo Valentin wrote: >> This patch adds bandgap IP entry on DRA7 dtsi device tree file. >> >> Signed-off-by: Eduardo Valentin >> --- >> arch/arm/boot/dts/dra7.dtsi | 12 >> 1 fi

Re: [PATCH 2/7] arm: dts: add dra7 IVA thermal data

2013-10-01 Thread Eduardo Valentin
On 01-10-2013 18:33, Nishanth Menon wrote: > On 14:32-20131001, Eduardo Valentin wrote: > minor comments follow >> This patch changes a dtsi file to contain the thermal data > s/changes/introduces? >> for IVA domain on DRA7 and later SoCs. This data will >> enable

Re: [PATCH v6 5/6] MCS Lock: Restructure the MCS lock defines and locking code into its own file

2013-10-01 Thread Waiman Long
On 10/01/2013 05:16 PM, Tim Chen wrote: On Tue, 2013-10-01 at 16:01 -0400, Waiman Long wrote: The cpu could still be executing out of order load instruction from the critical section before checking node->locked? Probably smp_mb() is still needed. Tim But this is the lock function, a

Re: [PATCH 1/3] perf tools: Separate lbfd check out of NO_DEMANGLE condition

2013-10-01 Thread Namhyung Kim
Hi Jiri, On Sun, 29 Sep 2013 15:52:06 +0900, Namhyung Kim wrote: > Hi Jiri, > > 2013-09-27 (금), 16:32 +0200, Jiri Olsa: >> We fail build with NO_DEMANGLE with missing -lbfd externals error. >> The reason is that we now use bfd code in srcline object: >> perf tools: Implement addr2line directly

Re: applesmc oops in 3.10/3.11

2013-10-01 Thread Chris Murphy
On Oct 1, 2013, at 10:24 AM, Guenter Roeck wrote: > On Tue, Oct 01, 2013 at 09:33:13AM -0600, Chris Murphy wrote: >> >> On Oct 1, 2013, at 9:19 AM, Guenter Roeck wrote: >> >>> On Tue, Oct 01, 2013 at 12:55:26PM +0200, Henrik Rydberg wrote: >> Warning message triggered with

Re: [PATCH 6/7] x86, kaslr: report kernel offset on panic

2013-10-01 Thread HATAYAMA Daisuke
(2013/10/02 9:38), HATAYAMA Daisuke wrote: > (2013/10/02 4:37), Kees Cook wrote: >> @@ -1242,3 +1256,15 @@ void __init i386_reserve_resources(void) >>} >> >>#endif /* CONFIG_X86_32 */ >> + >> +static struct notifier_block kernel_offset_notifier = { >> +.notifier_call =

Re: [PATCH] perf: Support for Openembedded/Yocto -dbg packages

2013-10-01 Thread Namhyung Kim
Hi Ingo and Arnaldo, On Wed, 18 Sep 2013 15:29:03 +0200, Ingo Molnar wrote: > * Arnaldo Carvalho de Melo wrote: > >> Em Wed, Sep 18, 2013 at 12:18:01PM +0200, Peter Zijlstra escreveu: >> > On Wed, Sep 18, 2013 at 12:02:12PM +0200, Ricardo Ribalda Delgado wrote: >> > > Perhaps this is even more

Re: [PATCH] [workqueue] check values of pwq and wq in print_worker_info() before use

2013-10-01 Thread John David Anglin
On 1-Oct-13, at 6:50 PM, James Bottomley wrote: On Wed, 2013-10-02 at 00:07 +0200, Helge Deller wrote: On 10/01/2013 11:40 PM, James Bottomley wrote: On Tue, 2013-10-01 at 16:43 -0400, Tejun Heo wrote: Hello, On Tue, Oct 01, 2013 at 10:35:20PM +0200, Helge Deller wrote: print_worker_info()

Re: [patch for-3.12] mm, memcg: protect mem_cgroup_read_events for cpu hotplug

2013-10-01 Thread KOSAKI Motohiro
(10/1/13 7:31 PM), David Rientjes wrote: for_each_online_cpu() needs the protection of {get,put}_online_cpus() so cpu_online_mask doesn't change during the iteration. Signed-off-by: David Rientjes Acked-by: KOSAKI Motohiro -- To unsubscribe from this list: send the line "unsubscribe

Re: [PATCH 6/7] x86, kaslr: report kernel offset on panic

2013-10-01 Thread HATAYAMA Daisuke
(2013/10/02 4:37), Kees Cook wrote: > When the system panics, include the kernel offset in the report to assist > in debugging. > > Signed-off-by: Kees Cook > --- > arch/x86/kernel/setup.c | 26 ++ > 1 file changed, 26 insertions(+) > > diff --git

Re: [PATCHv2 2/2] mm: add a field to store names for private anonymous memory

2013-10-01 Thread KOSAKI Motohiro
> +static void seq_print_vma_name(struct seq_file *m, struct vm_area_struct > *vma) > +{ > + const char __user *name = vma_get_anon_name(vma); > + struct mm_struct *mm = vma->vm_mm; > + > + unsigned long page_start_vaddr; > + unsigned long page_offset; > + unsigned long

[PATCH 5/9][v5] powerpc: implement is_instr_load_store().

2013-10-01 Thread Sukadev Bhattiprolu
Implement is_instr_load_store() to detect whether a given instruction is one of the fixed-point or floating-point load/store instructions. This function will be used in a follow-on patch to save memory hierarchy information of the load/store. Signed-off-by: Sukadev Bhattiprolu Reviewed-by:

[PATCH 8/9][v5] powerpc/perf: Export Power7 memory hierarchy info to user space.

2013-10-01 Thread Sukadev Bhattiprolu
On Power7, the DCACHE_SRC field in MMCRA register identifies the memory hierarchy level (eg: L2, L3 etc) from which a data-cache miss for a marked instruction was satisfied. Use the 'perf_mem_data_src' object to export this hierarchy level to user space. Some memory hierarchy levels in Power7

[PATCH 6/9][v5] powerpc/perf: Define big-endian version of perf_mem_data_src

2013-10-01 Thread Sukadev Bhattiprolu
perf_mem_data_src is an union that is initialized via the ->val field and accessed via the bitmap fields. For this to work on big endian platforms, we also need a big-endian represenation of perf_mem_data_src. Cc: Stephane Eranian Cc: Michael Ellerman Signed-off-by: Sukadev Bhattiprolu

[PATCH 9/9][v5] powerpc/perf: Update perf-mem man page for Power

2013-10-01 Thread Sukadev Bhattiprolu
Add a few lines to the perf-mem man page to indicate: - its dependence on the mem-loads and mem-stores events - how to use the feature on Power architecture. Signed-off-by: Sukadev Bhattiprolu --- tools/perf/Documentation/perf-mem.txt | 11 +++ 1 file changed, 11

[PATCH 2/9][v5] powerpc/perf: Export Power8 generic events in sysfs

2013-10-01 Thread Sukadev Bhattiprolu
Export generic perf events for Power8 in sysfs. Signed-off-by: Sukadev Bhattiprolu Reviewed-by: Anshuman Khandual --- arch/powerpc/perf/power8-pmu.c | 23 +++ 1 file changed, 23 insertions(+) diff --git a/arch/powerpc/perf/power8-pmu.c b/arch/powerpc/perf/power8-pmu.c

[PATCH 1/9][v5] powerpc/perf: Rename Power8 macros to start with PME

2013-10-01 Thread Sukadev Bhattiprolu
We use helpers like GENERIC_EVENT_ATTR() to list the generic events in sysfs. To avoid name collisions, GENERIC_EVENT_ATTR() requires the perf event macros to start with PME. Signed-off-by: Sukadev Bhattiprolu Reviewed-by: Anshuman Khandual --- arch/powerpc/perf/power8-pmu.c | 24

[PATCH 3/9][v5] powerpc/perf: Add Power8 event PM_MRK_GRP_CMPL to sysfs.

2013-10-01 Thread Sukadev Bhattiprolu
The perf event PM_MRK_GRP_CMPL is useful in analyzing memory hierarchy of applications. Signed-off-by: Sukadev Bhattiprolu Reviewed-by: Anshuman Khandual --- arch/powerpc/perf/power8-pmu.c |5 + 1 file changed, 5 insertions(+) diff --git a/arch/powerpc/perf/power8-pmu.c

[PATCH 7/9][v5] powerpc/perf: Export Power8 memory hierarchy info to user space.

2013-10-01 Thread Sukadev Bhattiprolu
On Power8, the LDST field in SIER identifies the memory hierarchy level (eg: L1, L2 etc), from which a data-cache miss for a marked instruction was satisfied. Use the 'perf_mem_data_src' object to export this hierarchy level to user space. Fortunately, the memory hierarchy levels in Power8 map

[no subject]

2013-10-01 Thread Sukadev Bhattiprolu
Subject: [PATCH 0/9][v5] powerpc/perf: Export memory hierarchy level in Power7/8. Power7 and Power8 processors save the memory hierarchy level (eg: L2, L3) from which a load or store instruction was satisfied. Export this hierarchy information to the user via the perf_mem_data_src object.

[PATCH 4/9][v5] powerpc: Rename branch_opcode() to instr_opcode()

2013-10-01 Thread Sukadev Bhattiprolu
The logic used in branch_opcode() to extract the opcode for an instruction applies to non branch instructions also. So rename to instr_opcode(). Signed-off-by: Sukadev Bhattiprolu Reviewed-by: Anshuman Khandual --- arch/powerpc/lib/code-patching.c |6 +++--- 1 file changed, 3

[PATCH 3/5] drm/bridge: Add PTN3460 bridge driver

2013-10-01 Thread Sean Paul
This patch adds a drm_bridge driver for the PTN3460 DisplayPort to LVDS bridge chip. Signed-off-by: Sean Paul --- .../devicetree/bindings/drm/bridge/ptn3460.txt | 27 ++ drivers/gpu/drm/Kconfig| 2 + drivers/gpu/drm/Makefile | 1 +

[PATCH 2/5] ARM: dts: Add dp-controller node to exynos5250-snow

2013-10-01 Thread Sean Paul
This patch adds the dp-controller node to the exynos5250-snow board dts file. Signed-off-by: Sean Paul --- arch/arm/boot/dts/exynos5250-snow.dts | 12 1 file changed, 12 insertions(+) diff --git a/arch/arm/boot/dts/exynos5250-snow.dts b/arch/arm/boot/dts/exynos5250-snow.dts index

[PATCH 1/5] ARM: dts: Add fimd display-timings for exynos5250-snow

2013-10-01 Thread Sean Paul
This patch adds the internal panel timings to the exynos5250-snow board dts file. Signed-off-by: Sean Paul --- arch/arm/boot/dts/exynos5250-snow.dts | 17 + 1 file changed, 17 insertions(+) diff --git a/arch/arm/boot/dts/exynos5250-snow.dts

[PATCH 0/5] Add some missing bits for exynos5250-snow

2013-10-01 Thread Sean Paul
This set adds some missing devicetree nodes to the exynos5250-snow file as well as adds a drm_bridge driver for the ptn3460 DP-LVDS chip. This chip is used in the exynos5250-snow board. Sean Sean Paul (5): ARM: dts: Add fimd display-timings for exynos5250-snow ARM: dts: Add dp-controller node

[PATCH 4/5] drm/exynos: Initialize ptn3460 if present

2013-10-01 Thread Sean Paul
This patch adds code to look for the ptn3460 in the device tree file on exynos initialization. If ptn node is found, the driver will initialize the ptn3460 driver and skip creating a DP connector (since the bridge driver will register its own connector). Signed-off-by: Sean Paul ---

[PATCH 5/5] ARM: dts: Add ptn3460 to exynos5250-snow

2013-10-01 Thread Sean Paul
This patch adds a node for the ptn3460 DP-LVDS chip in the exynos5250-snow board dts file. Signed-off-by: Sean Paul --- arch/arm/boot/dts/exynos5250-snow.dts | 19 +++ 1 file changed, 19 insertions(+) diff --git a/arch/arm/boot/dts/exynos5250-snow.dts

[patch for-3.12] mm, memcg: protect mem_cgroup_read_events for cpu hotplug

2013-10-01 Thread David Rientjes
for_each_online_cpu() needs the protection of {get,put}_online_cpus() so cpu_online_mask doesn't change during the iteration. Signed-off-by: David Rientjes --- mm/memcontrol.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/mm/memcontrol.c b/mm/memcontrol.c --- a/mm/memcontrol.c +++

UEFA League Promotion

2013-10-01 Thread UEFA LEAGUE PROMOTION
Congratulation!!Your Email Address has won £1,960,000.00 In the ongoing 2013 UEFA League Promotion with Ref No: UK/AFSA 001,send Reference Number: williams.freema...@gmail.com for claims immediately. Thanks Management . -- To unsubscribe from this list: send the line "unsubscribe

Re: [PATCH 08/13] tpm: Pull everything related to /dev/tpmX into tpm-dev.c

2013-10-01 Thread Jason Gunthorpe
On Wed, Oct 02, 2013 at 01:14:18AM +0200, Peter H?we wrote: > The makefile patch did fix it. Great, feel free squash into the broken commit, or I can respin things. > (along with a small fix for the tpm_i2c_atmel driver ;) Hum, I wonder why my compilers didn't whine, x86-64 should have

Re: [PATCH 08/13] tpm: Pull everything related to /dev/tpmX into tpm-dev.c

2013-10-01 Thread Peter Hüwe
Am Mittwoch, 2. Oktober 2013, 00:57:33 schrieb Jason Gunthorpe: > > When compiling the tpm drivers as modules I get > > ERROR: "tpm_sysfs_del_device" [drivers/char/tpm/tpm.ko] undefined! > > ERROR: "tpm_dev_add_device" [drivers/char/tpm/tpm.ko] undefined! > > ERROR: "tpm_dev_del_device"

Re: [PATCH 07/11] igb: Avoid uninitialized advertised variable in eee_set_cur

2013-10-01 Thread Jeff Kirsher
On Mon, 2013-09-30 at 13:29 -0700, Andi Kleen wrote: > From: Andi Kleen > > eee_get_cur assumes that the output data is already zeroed. It can > read-modify-write the advertised field: > > if (ipcnfg & E1000_IPCNFG_EEE_100M_AN) > 2594edata->advertised |=

Re: [PATCH 0/7] DRA7: DT thermal support

2013-10-01 Thread Nishanth Menon
On 14:32-20131001, Eduardo Valentin wrote: > Hello all, > > This is a complementary patch series with themal DT support for DRA7. > > Although this work depends on the thermal dt parser work [1], I decided > to share it before hand. It also depends on DRA7 DT base port support,

Re: [PATCH 08/13] tpm: Pull everything related to /dev/tpmX into tpm-dev.c

2013-10-01 Thread Jason Gunthorpe
On Wed, Oct 02, 2013 at 12:52:40AM +0200, Peter H?we wrote: > Am Montag, 23. September 2013, 20:14:38 schrieb Jason Gunthorpe: > > CLASS-dev.c is a common idiom for Linux subsystems > > > > This pulls all the code related to the miscdev into tpm-dev.c and makes it > > static. The identical

Re: [PATCH 6/7] arm: dts: add tmp102 i2c sensor node on dra7-evm

2013-10-01 Thread Nishanth Menon
On 14:32-20131001, Eduardo Valentin wrote: > On dra7-evm there is an tmp102 temperature sensor on i2c bus 1. > This patch adds its device tree node. > > Signed-off-by: Eduardo Valentin > --- > arch/arm/boot/dts/dra7-evm.dts | 8 > 1 file changed, 8 insertions(+) &

Re: [PATCH] [workqueue] check values of pwq and wq in print_worker_info() before use

2013-10-01 Thread James Bottomley
On Wed, 2013-10-02 at 00:07 +0200, Helge Deller wrote: > On 10/01/2013 11:40 PM, James Bottomley wrote: > > On Tue, 2013-10-01 at 16:43 -0400, Tejun Heo wrote: > >> Hello, > >> > >> On Tue, Oct 01, 2013 at 10:35:20PM +0200, Helge Deller wrote: > >>> print_worker_info() includes no validity check

Re: [PATCH 08/13] tpm: Pull everything related to /dev/tpmX into tpm-dev.c

2013-10-01 Thread Peter Hüwe
Am Montag, 23. September 2013, 20:14:38 schrieb Jason Gunthorpe: > CLASS-dev.c is a common idiom for Linux subsystems > > This pulls all the code related to the miscdev into tpm-dev.c and makes it > static. The identical file_operation structs in the drivers are purged and > the tpm common code

Re: [PATCH 3/7] arm: dts: dra7: add bandgap entry

2013-10-01 Thread Nishanth Menon
On 14:32-20131001, Eduardo Valentin wrote: > This patch adds bandgap IP entry on DRA7 dtsi device tree file. > > Signed-off-by: Eduardo Valentin > --- > arch/arm/boot/dts/dra7.dtsi | 12 > 1 file changed, 12 insertions(+) > > diff --git a/arch/arm/boot/

Re: [PATCH] [workqueue] check values of pwq and wq in print_worker_info() before use

2013-10-01 Thread Tejun Heo
On Tue, Oct 01, 2013 at 06:40:23PM -0400, Tejun Heo wrote: > Because it is using probe_kernel_read() and such test wouldn't mean > anything? It may be NULL, it may be 1 or full Fs. NULL is just one > of many illegal pointers which may happen. Why add code which doesn't > achieve anything when

Re: [PATCH] OOM killer: wait for tasks with pending SIGKILL to exit

2013-10-01 Thread David Rientjes
On Tue, 1 Oct 2013, Sergey Dyasly wrote: > If you are ok with the first change in my patch regarding > fatal_signal_pending, > I can send new patch with just that change. > The entire patch is pointless, there's no need to give access to memory reserves simply because it is PF_EXITING. If it

Re: [BUG] Regression in 2fdac010 drivers/net/ethernet/via/via-velocity.c: update napi implementation

2013-10-01 Thread Francois Romieu
Julia Lawall : [...] > There has already been a discussion about this, and a patch has already > been proposed. It has to do with lock managament. I will look for the > email. The underlying problem has to do with disabled irq. netif_receive_skb assumes irq to be enabled. Current

Re: RFC: (re-)binding the VFIO platform driver to a platform device

2013-10-01 Thread Scott Wood
On Tue, 2013-10-01 at 16:59 -0500, Kim Phillips wrote: > On Tue, 1 Oct 2013 14:15:38 -0500 > Scott Wood wrote: > > > I think the ideal interface would be if you could write the sysfs device > > name into the vfio bind file (or some new file in the same directory), > > and have it claim that

Re: [PATCH] [workqueue] check values of pwq and wq in print_worker_info() before use

2013-10-01 Thread Tejun Heo
Hello, On Wed, Oct 02, 2013 at 12:34:53AM +0200, Helge Deller wrote: > Sure, probe_kernel_read() takes care that no segfaults will happen. > Nevertheless, if we know that "pwq" might become NULL, why access pwq->wq at > all? > struct pool_workqueue *pwq = NULL; > probe_kernel_read(, >wq,

Re: [PATCH 1/7] arm: dts: add dra7 DSPEVE thermal data

2013-10-01 Thread Nishanth Menon
On 14:32-20131001, Eduardo Valentin wrote: > This patch changes a dtsi file to contain the thermal data ^^ introduces? > for DSPEVE domain on DRA7 and later SoCs. This data will > enable a thermal shutdown at 125C. > > This thermal data can be reused across

Re: [tpmdd-devel] [PATCH 07/13] tpm: Remove tpm_show_caps_1_2

2013-10-01 Thread Peter Hüwe
Hi Jason, Am Mittwoch, 2. Oktober 2013, 00:21:13 schrieb Jason Gunthorpe: > On Wed, Oct 02, 2013 at 12:09:22AM +0200, Peter H?we wrote: > >Since the tpm_spi_stm_st33, tpm_i2c_nuvoton and tpm_i2c_atmel drivers > >are not yet merged and were heavily improved by you anyway, please > >

Re: [PATCH 2/7] arm: dts: add dra7 IVA thermal data

2013-10-01 Thread Nishanth Menon
On 14:32-20131001, Eduardo Valentin wrote: minor comments follow > This patch changes a dtsi file to contain the thermal data s/changes/introduces? > for IVA domain on DRA7 and later SoCs. This data will > enable a thermal shutdown at 125C. > > This thermal data can be reuse

  1   2   3   4   5   6   7   8   9   10   >