Re: perf/tracepoint: another fuzzer generated lockup

2013-11-16 Thread Masami Hiramatsu
(2013/11/15 23:28), Frederic Weisbecker wrote: > On Fri, Nov 15, 2013 at 09:15:21AM -0500, Steven Rostedt wrote: >> On Fri, 15 Nov 2013 13:28:33 +0100 >> Peter Zijlstra wrote: >> >>> On Fri, Nov 15, 2013 at 10:16:18AM +0900, Masami Hiramatsu wrote: Kprobes itself can detect nested call by

linux-kernel@vger.kernel.org

2013-11-16 Thread

Re: [GIT PULL] A minor amd64_edac fix for 3.13

2013-11-16 Thread Borislav Petkov
On Sun, Nov 17, 2013 at 02:22:15AM +0100, Frank Becker wrote: > Still, Google made a mistake by tagging Boris mail as SPAM. As you > already wrote it probably was triggered by the content filter. They > don't tell. To get those filters right is probably impossible. BTW, I > liked your idea to tag

Re: [PATCH 3.4 11/26] md: Fix skipping recovery for read-only arrays.

2013-11-16 Thread NeilBrown
On Sun, 17 Nov 2013 04:11:19 + Ben Hutchings wrote: > On Fri, 2013-11-08 at 22:51 -0800, Greg Kroah-Hartman wrote: > > 3.4-stable review patch. If anyone has any objections, please let me know. > > > > -- > > > > From: Lukasz Dorau > > > > commit

linux-kernel@vger.kernel.org

2013-11-16 Thread

Re: [tip:x86/asm] x86-64, copy_user: Remove zero byte check before copy user buffer.

2013-11-16 Thread H. Peter Anvin
On 11/16/2013 10:44 PM, Linus Torvalds wrote: > So this doesn't do the 32-bit truncation in the error path of the generic > string copy. Oversight? > >Linus Indeed... although in the kernel it seems to be taken as an invariant that copy lengths over 4G is simply prohibited. There are

[tip:x86/asm] x86-64, copy_user: Remove zero byte check before copy user buffer.

2013-11-16 Thread tip-bot for Fenghua Yu
Commit-ID: f4cb1cc18f364d761d5614eb62936647f259 Gitweb: http://git.kernel.org/tip/f4cb1cc18f364d761d5614eb62936647f259 Author: Fenghua Yu AuthorDate: Sat, 16 Nov 2013 12:37:01 -0800 Committer: H. Peter Anvin CommitDate: Sat, 16 Nov 2013 18:00:58 -0800 x86-64, copy_user: Remove

Re: [PATCH] cpufreq: cpufreq-cpu0: Use a sane boot frequency when booting with a mismatched bootloader configuration

2013-11-16 Thread Viresh Kumar
On 16 November 2013 19:14, Shawn Guo wrote: > No, it's not a kernel bug. > > OPP is not a definition that belongs to kernel. Instead, it's > characteristics of hardware, and that's why we can naturally put the > definition into device tree. Bear it in mind that device tree is a > hardware

linux-kernel@vger.kernel.org

2013-11-16 Thread

linux-kernel@vger.kernel.org

2013-11-16 Thread

linux-kernel@vger.kernel.org

2013-11-16 Thread

linux-kernel@vger.kernel.org

2013-11-16 Thread

linux-kernel@vger.kernel.org

2013-11-16 Thread

linux-kernel@vger.kernel.org

2013-11-16 Thread

Re: [PATCH 9/9] KEYS: Fix encrypted key type update method

2013-11-16 Thread Mimi Zohar
On Thu, 2013-11-14 at 17:59 +, David Howells wrote: > Mimi Zohar wrote: > > > Is there a keyutils git repo with a version of keyctl that supports the > > control option? > > http://git.kernel.org/cgit/linux/kernel/git/dhowells/keyutils.git/log/?h=development Thanks! > > - type size_t is

Re: [BUG] perf stat: explicit grouping yields unexpected results

2013-11-16 Thread Andi Kleen
> I'd say that the default behavior should be what Jiri implemented: get > the most out of the situation and inform. But you are right in that > 'forcing' all elements of a group to be valid should be possible as > well - if a special perf stat option or event format is used. When something is

Re: [PATCH] sysfs: use a separate locking class for open files depending on mmap

2013-11-16 Thread Tejun Heo
On Sat, Nov 16, 2013 at 10:21:19PM -0500, Dave Jones wrote: > On Sun, Nov 17, 2013 at 11:17:36AM +0900, Tejun Heo wrote: > > > > + if (has_mmap) > > + mutex_init(>mutex); > > + else > > + mutex_init(>mutex); > > ummm... Supposed to look that way. It'll give two

[GIT PULL] MMC updates for 3.13-rc1

2013-11-16 Thread Chris Ball
Hi Linus, Please pull from: git://git.kernel.org/pub/scm/linux/kernel/git/cjb/mmc.git tags/mmc-updates-for-3.13-rc1 to receive the MMC merge for 3.13. These patches have been tested in linux-next, and there's a conflict with the resolution below. Thanks. diff --cc

Re: [PATCH] sysfs: use a separate locking class for open files depending on mmap

2013-11-16 Thread Dave Jones
On Sun, Nov 17, 2013 at 11:17:36AM +0900, Tejun Heo wrote: > +if (has_mmap) > +mutex_init(>mutex); > +else > +mutex_init(>mutex); ummm... Dave -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to

[PATCH] Fix a trivial typo in comments.

2013-11-16 Thread Shigeru Yoshida
Fix a trivial typo in rq_attach_root. Signed-off-by: Shigeru Yoshida --- kernel/sched/core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/kernel/sched/core.c b/kernel/sched/core.c index 5ac63c9..2bc3bc0 100644 --- a/kernel/sched/core.c +++ b/kernel/sched/core.c @@ -4985,7

Fwd: About deadline IO scheduler in kernel

2013-11-16 Thread 韩磊
-- Forwarded message -- From: 韩磊 Date: 2013/11/16 Subject: About deadline IO scheduler in kernel To: Linux Kernel Mailing List In deadline scheduler, in a FIFO list when a request insert it and other request remove it in the same time,whether it make conflict??? Whether it

Re: Fwd: [PATCH 8/8] arm: dts: keystone: add watchdog entry

2013-11-16 Thread Guenter Roeck
On 11/06/2013 03:33 AM, ivan.khoronzhuk wrote: Add watchdog entry to keystone device tree. Signed-off-by: Ivan Khoronzhuk Acked-by: Guenter Roeck --- arch/arm/boot/dts/keystone.dts |6 ++ 1 file changed, 6 insertions(+) diff --git a/arch/arm/boot/dts/keystone.dts

Re: Fwd: [PATCH 7/8] watchdog: davinci: add "clocks" property

2013-11-16 Thread Guenter Roeck
On 11/06/2013 03:32 AM, ivan.khoronzhuk wrote: The Keystone arch is using clocks in DT and source clock for watchdog has to be specified, so add this to binding. Signed-off-by: Ivan Khoronzhuk --- .../devicetree/bindings/watchdog/davinci-wdt.txt |5 + 1 file changed, 5

Re: Fwd: [PATCH 6/8] watchdog: davinci: reuse driver for keystone arch

2013-11-16 Thread Guenter Roeck
On 11/06/2013 03:32 AM, ivan.khoronzhuk wrote: The keystone arch use the same IP watchdog, so add "ti,keystone-wdt" compatible and correct identity. Signed-off-by: Ivan Khoronzhuk --- .../devicetree/bindings/watchdog/davinci-wdt.txt | 11 +-- drivers/watchdog/Kconfig

Re: Fwd: [PATCH 5/8] watchdog: davinci: add "timeout-sec" property

2013-11-16 Thread Guenter Roeck
On 11/06/2013 03:32 AM, ivan.khoronzhuk wrote: Since Davinci WDT has been switched to use WDT core, it became able to support timeout-sec property, so add it to it's binding description. Signed-off-by: Ivan Khoronzhuk --- .../devicetree/bindings/watchdog/davinci-wdt.txt |5 + 1

Re: Fwd: [PATCH 4/8] watchdog: davinci: add GET_STATUS option support

2013-11-16 Thread Guenter Roeck
On 11/06/2013 03:32 AM, ivan.khoronzhuk wrote: When watchdog timer is expired we can know about it thought thought -> through or with GET_STATUS ioctl option. Signed-off-by: Ivan Khoronzhuk --- drivers/watchdog/davinci_wdt.c | 13 + 1 file changed, 13 insertions(+) diff

Re: Fwd: [PATCH 3/8] watchdog: davinci: add GET_TIMELEFT option support

2013-11-16 Thread Guenter Roeck
On 11/06/2013 03:31 AM, ivan.khoronzhuk wrote: Currently, the davinci watchdog can be read while counting, so we can add ability to report the remaining time before the system will reboot. Signed-off-by: Ivan Khoronzhuk --- drivers/watchdog/davinci_wdt.c | 28

Re: Fwd: [PATCH 2/8] watchdog: davinci: use davinci_wdt_device structure to hold device data

2013-11-16 Thread Guenter Roeck
On 11/06/2013 03:31 AM, ivan.khoronzhuk wrote: Some SoCs, like Keystone 2, can support more than one WDT and each watchdog device has to use it's own base address, clock source, wdd device, so add new davinci_wdt_device structure to hold device data. Signed-off-by: Ivan Khoronzhuk

[PATCH] sysfs: use a separate locking class for open files depending on mmap

2013-11-16 Thread Tejun Heo
The following two commits implemented mmap support in the regular file path and merged bin file support into the regular path. 73d9714627ad ("sysfs: copy bin mmap support from fs/sysfs/bin.c to fs/sysfs/file.c") 3124eb1679b2 ("sysfs: merge regular and bin file handling") After the merge, the

Re: xhci_hcd 3.12 regression

2013-11-16 Thread Nikolaus Meine
Nikolaus Meine tnt.uni-hannover.de> writes: > > After upgrading from kernel 3.11 to 3.12 my USB 3.0 card reader > (Transcend RDF8) stopped working with the same error. > > I managed to find the code line which triggered the erroneous behaviour: > > It's the definition of the macro

Re: [tpmdd-devel] [PATCH 5/5 v2] tpm: Make tpm-dev allocate a per-file structure

2013-11-16 Thread Ashley Lai
On Sun, 2013-11-03 at 20:38 -0700, Jason Gunthorpe wrote: > This consolidates everything that is only used within tpm-dev.c > into tpm-dev.c and out of the publicly visible struct tpm_chip. > > The per-file allocation lays the ground work for someday fixing the > strange forced O_EXCL behaviour

Re: xhci_hcd 3.12 regression

2013-11-16 Thread Nikolaus Meine
M G Berberich oss.m-berberich.de> writes: > > Hello, > > since 3.12 dmesg (and logfiles) are filled up with: > > xhci_hcd :00:14.0: ep 0x81 - asked for 96 bytes, 78 bytes untransferred > xhci_hcd :00:14.0: Giveback URB 8807f82ec240, len = 18, expected = 96, status = -121 >

Re: [GIT PULL] A minor amd64_edac fix for 3.13

2013-11-16 Thread Frank Becker
Linus Torvalds [2013-11-17, 00:52 +0100]: > On Nov 16, 2013 3:34 PM, "Frank Becker" wrote: Hi, > > Blaming the server provider is nonsense. > > BS. :-) Looks like an authentic mail. Who needs crypto-signatures? > If the service provider allows spammers, the service provider is shit. It >

HI

2013-11-16 Thread BECCA JACLYN
Hello! My name is Becca Jaclyn from England. I saw your profile today and picked interest to discuss an important business proposal. Please reply to this mail so i can give you more details and also send my photos. Thanks Best regards Becca -- To unsubscribe from this list: send the line

Re: [tpmdd-devel] [PATCH 4/5 v2] tpm: Use the ops structure instead of a copy in tpm_vendor_specific

2013-11-16 Thread Ashley Lai
On Sun, 2013-11-03 at 20:38 -0700, Jason Gunthorpe wrote: > This builds on the last commit to use the ops structure in the core > and reduce the size of tpm_vendor_specific. > > Signed-off-by: Jason Gunthorpe > Reviewed-by: Joel Schopp > --- Reviewed-by: Ashley Lai Acked-by: Ashley Lai

Re: [tpmdd-devel] [PATCH 3/5 v2] tpm: Create a tpm_class_ops structure and use it in the drivers

2013-11-16 Thread Ashley Lai
On Sun, 2013-11-03 at 20:38 -0700, Jason Gunthorpe wrote: > This replaces the static initialization of a tpm_vendor_specific > structure in the drivers with the standard Linux idiom of providing > a const structure of function pointers. > > Signed-off-by: Jason Gunthorpe > Reviewed-by: Joel

Re: [tpmdd-devel] [PATCH 2/5 v2] tpm: Pull everything related to sysfs into tpm-sysfs.c

2013-11-16 Thread Jason Gunthorpe
On Sat, Nov 16, 2013 at 05:53:19PM -0600, Ashley Lai wrote: > > +/* XXX for now this helper is duplicated in tpm.c */ > > I think you mean this is duplicated in tpm-interface.c. Is there a > reason why we cannot add this to tpm.h to avoid this duplication? Indeed, it was called tpm.c when I

linux-kernel@vger.kernel.org

2013-11-16 Thread

linux-kernel@vger.kernel.org

2013-11-16 Thread

Re: [tpmdd-devel] [PATCH 1/5 v2] tpm: Pull everything related to /dev/tpmX into tpm-dev.c

2013-11-16 Thread Jason Gunthorpe
On Sat, Nov 16, 2013 at 04:18:55PM -0600, Ashley Lai wrote: > Nice clean up!!! The code looks much more clean after removing the > duplicated code in all drivers. Thanks. > > +++ b/drivers/char/tpm/tpm-dev.c > > + * Copyright (C) 2013 Obsidian Reearch Corp > > Typo Research? Yes, thanks :) >

Re: [tpmdd-devel] [PATCH 2/5 v2] tpm: Pull everything related to sysfs into tpm-sysfs.c

2013-11-16 Thread Ashley Lai
> diff --git a/drivers/char/tpm/tpm-sysfs.c b/drivers/char/tpm/tpm-sysfs.c > new file mode 100644 > index 000..3bcfed0 > --- /dev/null > +++ b/drivers/char/tpm/tpm-sysfs.c > @@ -0,0 +1,318 @@ > + * Copyright (C) 2013 Obsidian Reearch Corp Typo Research? > + * Jason Gunthorpe > + * >

Re: [GIT PULL] A minor amd64_edac fix for 3.13

2013-11-16 Thread Frank Becker
Borislav Petkov [2013-11-16, 19:23 +0100]: > On Sat, Nov 16, 2013 at 10:10:05AM -0800, Linus Torvalds wrote: Hi, > > Ahh. Yes. Googling for "hetzner online spam", and there's a *lot* of > > complaints. See for example > > > > > >

Re: [PATCH] aio: fix D-cache aliasing issues

2013-11-16 Thread John David Anglin
On 16-Nov-13, at 5:37 PM, James Bottomley wrote: On Sat, 2013-11-16 at 17:32 -0500, John David Anglin wrote: On 16-Nov-13, at 5:06 PM, James Bottomley wrote: On Sat, 2013-11-16 at 21:07 +0100, Simon Baatz wrote: On Fri, Nov 15, 2013 at 02:42:05PM -0800, James Bottomley wrote: On Fri,

Re: [PATCH] aio: fix D-cache aliasing issues

2013-11-16 Thread John David Anglin
On 16-Nov-13, at 5:06 PM, James Bottomley wrote: On Sat, 2013-11-16 at 21:07 +0100, Simon Baatz wrote: On Fri, Nov 15, 2013 at 02:42:05PM -0800, James Bottomley wrote: On Fri, 2013-11-15 at 23:05 +0100, Helge Deller wrote: When a user page mapping is released via kunmap*() functions, the

Re: [PATCH] aio: fix D-cache aliasing issues

2013-11-16 Thread James Bottomley
On Sat, 2013-11-16 at 17:32 -0500, John David Anglin wrote: > On 16-Nov-13, at 5:06 PM, James Bottomley wrote: > > > On Sat, 2013-11-16 at 21:07 +0100, Simon Baatz wrote: > >> On Fri, Nov 15, 2013 at 02:42:05PM -0800, James Bottomley wrote: > >>> On Fri, 2013-11-15 at 23:05 +0100, Helge Deller

Re: [tpmdd-devel] [PATCH 1/5 v2] tpm: Pull everything related to /dev/tpmX into tpm-dev.c

2013-11-16 Thread Ashley Lai
Nice clean up!!! The code looks much more clean after removing the duplicated code in all drivers. Thanks. > diff --git a/drivers/char/tpm/tpm-dev.c b/drivers/char/tpm/tpm-dev.c > new file mode 100644 > index 000..8d94e97 > --- /dev/null > +++ b/drivers/char/tpm/tpm-dev.c > + * Copyright

Re: [PATCH] aio: fix D-cache aliasing issues

2013-11-16 Thread James Bottomley
On Sat, 2013-11-16 at 21:07 +0100, Simon Baatz wrote: > On Fri, Nov 15, 2013 at 02:42:05PM -0800, James Bottomley wrote: > > On Fri, 2013-11-15 at 23:05 +0100, Helge Deller wrote: > > > When a user page mapping is released via kunmap*() functions, the D-cache > > > needs > > > to be flushed via

[PROBLEM] possible divide by 0 in kernel/sched/cputime.c scale_stime()

2013-11-16 Thread Christian Engelmayer
Since upgrading from v3.8 to v3.12 I see random crashes in function scale_stime() in kernel/sched/cputime.c: divide error: [#1] SMP Modules linked in: parport_pc(F) ppdev(F) bnep rfcomm bluetooth binfmt_misc(F) zl10353 cx88_dvb cx88_vp3054_i2c videobuf_dvb dvb_core

[GIT PULL] Please pull NFS client bugfixes

2013-11-16 Thread Myklebust, Trond
Hi Linus The following changes since commit fab99ebe39fe7d11fbd9b5fb84f07432af9ba36f: NFSv4.2: Remove redundant checks in nfs_setsecurity+nfs4_label_init_security (2013-11-04 16:42:52 -0500) are available in the git repository at: git://git.linux-nfs.org/projects/trondmy/linux-nfs.git

linux-kernel@vger.kernel.org

2013-11-16 Thread

[PATCH/RFC] wait_for_completion_timeout() considered harmful.

2013-11-16 Thread NeilBrown
It would be reasonable to assume that wait_for_completion_timeout(>auxadc_done, msecs_to_jiffies(5)); would wait at least 5 msecs for the auxadc_done to complete. But it does not. With a HZ of 200 or less, msecs_to_jiffies(5) has value '1', and so this will only wait until the next

Re: [PATCH 2/2] mm: create a separate slab for page->ptl allocation

2013-11-16 Thread Geert Uytterhoeven
On Thu, Nov 7, 2013 at 3:14 PM, Kirill A. Shutemov wrote: > --- a/include/linux/mm.h > +++ b/include/linux/mm.h > +static inline void pgtable_init(void) > +{ > + ptlock_cache_init(); > + pgtable_cache_init(); > +} sparc64defconfig: include/linux/mm.h:1391:2: error: implicit

[PATCH] x86/copy_user.S: Remove zero byte check before copy user buffer.

2013-11-16 Thread Fenghua Yu
From: Fenghua Yu Operation of rep movsb instruction handles zero byte copy. As pointed out by Linus, there is no need to check zero size in kernel. Removing this redundant check saves a few cycles in copy user functions. Signed-off-by: Fenghua Yu --- arch/x86/lib/copy_user_64.S | 8 ++--

[PATCH] afs: dir: remove unused label out_skip

2013-11-16 Thread Levente Kurusa
The out_skip label is not used, so remove it. Signed-off-by: Levente Kurusa --- fs/afs/dir.c |1 - 1 file changed, 1 deletion(-) diff --git a/fs/afs/dir.c b/fs/afs/dir.c index 3756d4f..0e10b14 100644 --- a/fs/afs/dir.c +++ b/fs/afs/dir.c @@ -669,7 +669,6 @@ static int

[PATCH] Fix NX-related Oops in wistron_btns module

2013-11-16 Thread Jakub Bogusz
The attached patch fixes "kernel tried to execute NX-protected page" Oops when loading wistron_btns module, occurring since at least 3.4.x; still applies to Linux 3.12. -- Jakub Boguszhttp://qboosh.pl/ This patch fixes "kernel tried to execute NX-protected page" oops when loading

Re: Fwd: [PATCH 1/8] watchdog: davinci: change driver to use WDT core

2013-11-16 Thread Guenter Roeck
On 11/06/2013 03:31 AM, ivan.khoronzhuk wrote: To reduce code duplicate and increase code readability use WDT core code to handle WDT interface. Remove io_lock as the WDT core uses mutex to lock each wdt device. Remove wdt_state as the WDT core track state with its own variable. The

Re: [PATCH 3/5] cmdline: declare exported symbols immediately

2013-11-16 Thread Felipe Contreras
On Sat, Nov 16, 2013 at 2:21 PM, Levente Kurusa wrote: > 2013-11-16 18:32 keltezéssel, Felipe Contreras írta: >> WARNING: EXPORT_SYMBOL(foo); should immediately follow its function/variable >> +EXPORT_SYMBOL(memparse); >> >> WARNING: EXPORT_SYMBOL(foo); should immediately follow its

Re: [PATCH v2] panic: setup panic_timeout early

2013-11-16 Thread Levente Kurusa
2013-11-16 19:46 keltezéssel, Felipe Contreras írta: > On Sat, Nov 16, 2013 at 11:45 AM, Ingo Molnar wrote: >> * Felipe Contreras wrote: > >> Anyway, the fact that you are argumentative even with Linus gives me >> little hope that you will improve your communication patterns with >> _me_, so

Re: [PATCH 3/5] cmdline: declare exported symbols immediately

2013-11-16 Thread Levente Kurusa
2013-11-16 18:32 keltezéssel, Felipe Contreras írta: > WARNING: EXPORT_SYMBOL(foo); should immediately follow its function/variable > +EXPORT_SYMBOL(memparse); > > WARNING: EXPORT_SYMBOL(foo); should immediately follow its function/variable > +EXPORT_SYMBOL(get_option); > > WARNING:

Re: [PATCH 1/5] kstrtox: remove redundant cleanup

2013-11-16 Thread Levente Kurusa
2013-11-16 18:32 keltezéssel, Felipe Contreras írta: > We can't reach the cleanup code unless the flag KSTRTOX_OVERFLOW is not > set, so there's not no point in clearing a bit that we know is not set. > > Signed-off-by: Felipe Contreras Acked-by: Levente Kurusa Legit one. To be honest, I don't

Re: [PATCH tip/core/rcu 4/4] documentation: Update circular buffer for load-acquire/store-release

2013-11-16 Thread Paul E. McKenney
On Sat, Nov 16, 2013 at 05:03:40PM +0100, Peter Zijlstra wrote: > On Fri, Nov 15, 2013 at 04:10:30PM -0800, Paul E. McKenney wrote: > > From: "Paul E. McKenney" > > > > This commit replaces full barriers by targeted use of load-acquire and > > store-release. > > I guess I'd better hurry up

Re: [PATCH v2] panic: setup panic_timeout early

2013-11-16 Thread Levente Kurusa
2013-11-16 19:46 keltezéssel, Felipe Contreras írta: > On Sat, Nov 16, 2013 at 11:45 AM, Ingo Molnar wrote: >> * Felipe Contreras wrote: > >> Anyway, the fact that you are argumentative even with Linus gives me >> little hope that you will improve your communication patterns with >> _me_, so

Re: [PATCH] aio: fix D-cache aliasing issues

2013-11-16 Thread Benjamin LaHaise
On Sat, Nov 16, 2013 at 09:07:18PM +0100, Simon Baatz wrote: > On Fri, Nov 15, 2013 at 02:42:05PM -0800, James Bottomley wrote: > > On Fri, 2013-11-15 at 23:05 +0100, Helge Deller wrote: > > > When a user page mapping is released via kunmap*() functions, the D-cache > > > needs > > > to be

Re: [PATCH 2/2] of: remove /proc/device-tree

2013-11-16 Thread Geert Uytterhoeven
On Thu, Mar 21, 2013 at 1:36 PM, Benjamin Herrenschmidt wrote: > On Thu, 2013-03-21 at 08:16 +, Grant Likely wrote: >> On Thu, Mar 21, 2013 at 7:43 AM, Benjamin Herrenschmidt >> wrote: >> > On Thu, 2013-03-21 at 07:35 +, Grant Likely wrote: >> >> > Shouldn't we have the symlink just be a

Re: [PATCH] aio: fix D-cache aliasing issues

2013-11-16 Thread Simon Baatz
On Fri, Nov 15, 2013 at 02:42:05PM -0800, James Bottomley wrote: > On Fri, 2013-11-15 at 23:05 +0100, Helge Deller wrote: > > When a user page mapping is released via kunmap*() functions, the D-cache > > needs > > to be flushed via flush_dcache_page() to avoid D-cache aliasing issues. > > > >

Re: [PATCH 04/40] staging/lustre/llite: Access to released file trigs a restore

2013-11-16 Thread Greg Kroah-Hartman
On Sat, Nov 16, 2013 at 10:36:18AM +, Dilger, Andreas wrote: > >So, sorry, I have to stop here at this series. I've applied the first 3 > >to the opw-next branch of staging.git so they can live somewhere until > >3.13-rc1 is out. > > > >I know you spent a lot of time making these 120 patches

[PATCH 0/3] get/set_dumpable() cleanups and theoretical fix

2013-11-16 Thread Oleg Nesterov
On 11/15, Kees Cook wrote: > > On Fri, Nov 15, 2013 at 12:36 PM, Oleg Nesterov wrote: > > > > unless I missed something, this is the fix, not cleanup ? > > > > If commit_creds()->set_dumpable(SUID_DUMP_ROOT) races with > > sys_prctl()->set_dumpable(SUID_DUMP_DISABLE), we can get > >

linux-kernel@vger.kernel.org

2013-11-16 Thread

linux-kernel@vger.kernel.org

2013-11-16 Thread

linux-kernel@vger.kernel.org

2013-11-16 Thread

[PATCH 1/3] set_dumpable: fix the theoretical race with itself

2013-11-16 Thread Oleg Nesterov
set_dumpable() updates MMF_DUMPABLE_MASK in a non-trivial way to ensure that get_dumpable() can't observe the intermediate state, but this all can't help if multiple threads call set_dumpable() at the same time. And in theory commit_creds()->set_dumpable(SUID_DUMP_ROOT) racing with

[PATCH 2/3] kill MMF_DUMPABLE and MMF_DUMP_SECURELY

2013-11-16 Thread Oleg Nesterov
Nobody actually needs MMF_DUMPABLE/MMF_DUMP_SECURELY, there are only used to enforce the encoding of SUID_DUMP_* enum in mm->flags & MMF_DUMPABLE_MASK. Now that set_dumpable() updates both bits atomically we can kill them and simply store the value "as is" in 2 lower bits. Signed-off-by: Oleg

[PATCH 3/3] make __get_dumpable/get_dumpable inline, kill fs/coredump.h

2013-11-16 Thread Oleg Nesterov
1. Remove fs/coredump.h. It is not clear why do we need it, it only declares __get_dumpable(), signal.c includes it for no reason. 2. Now that get_dumpable() and __get_dumpable() are really trivial make them inline in linux/sched.h. Signed-off-by: Oleg Nesterov --- fs/coredump.c

Re: [PATCH v2] panic: setup panic_timeout early

2013-11-16 Thread Felipe Contreras
On Sat, Nov 16, 2013 at 11:45 AM, Ingo Molnar wrote: > * Felipe Contreras wrote: > Anyway, the fact that you are argumentative even with Linus gives me > little hope that you will improve your communication patterns with > _me_, so I'm personally done arguing with you. How am I being

Re: [GIT PULL] A minor amd64_edac fix for 3.13

2013-11-16 Thread Borislav Petkov
On Sat, Nov 16, 2013 at 10:10:05AM -0800, Linus Torvalds wrote: > Ahh. Yes. Googling for "hetzner online spam", and there's a *lot* of > complaints. See for example > > > http://www.spamrankings.net/rankv2/2013/09/01/monthly/world/volume/cbl/all/regular/ Yowza, this is just great! :-( >

Re: [GIT PULL] A minor amd64_edac fix for 3.13

2013-11-16 Thread Linus Torvalds
On Sat, Nov 16, 2013 at 3:12 AM, Borislav Petkov wrote: > On Sat, Nov 16, 2013 at 02:50:49AM -0800, Linus Torvalds wrote: > >> The most common reason seems to be that your email provide is >> associated with spam, sometimes just because of a shared ISP. > > Sure, it is one:

Re: [PATCH v5 2/3] of/selftest: Add self tests for manipulation of properties

2013-11-16 Thread Pantelis Antoniou
Hi Grant, On Nov 15, 2013, at 7:46 PM, Grant Likely wrote: > Adds a few simple test cases to ensure that addition, update and removal > of device tree node properties works correctly. > > Signed-off-by: Grant Likely > Cc: Rob Herring > Cc: Benjamin Herrenschmidt > Cc: David S. Miller > Cc:

Re: [PATCH v2] panic: setup panic_timeout early

2013-11-16 Thread Ingo Molnar
* Felipe Contreras wrote: > On Fri, Nov 15, 2013 at 2:15 PM, Linus Torvalds > wrote: > > On Fri, Nov 15, 2013 at 12:10 PM, Felipe Contreras > > wrote: > >> > >> I haven't seen a single complaint about this commit message, so I > >> don't see what is your point. > > > > My point is that I

[PATCH 4/5] kstrtox: remove redundant casts

2013-11-16 Thread Felipe Contreras
The temporary variable is of the same type as the cast, so it's redundant. Signed-off-by: Felipe Contreras --- lib/kstrtox.c | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/lib/kstrtox.c b/lib/kstrtox.c index ec8da78..649b74b 100644 --- a/lib/kstrtox.c +++

[PATCH 2/5] cmdline: fix style issues

2013-11-16 Thread Felipe Contreras
WARNING: space prohibited between function name and open parenthesis '(' +int get_option (char **str, int *pint) WARNING: space prohibited between function name and open parenthesis '(' + *pint = simple_strtol (cur, str, 0); ERROR: trailing whitespace + $ WARNING: please, no spaces at the

[PATCH 3/5] cmdline: declare exported symbols immediately

2013-11-16 Thread Felipe Contreras
WARNING: EXPORT_SYMBOL(foo); should immediately follow its function/variable +EXPORT_SYMBOL(memparse); WARNING: EXPORT_SYMBOL(foo); should immediately follow its function/variable +EXPORT_SYMBOL(get_option); WARNING: EXPORT_SYMBOL(foo); should immediately follow its function/variable

[PATCH 5/5] params: improve standard definitions

2013-11-16 Thread Felipe Contreras
We are repeating the functionality of kstrtol in param_set_long, and the same for kstrtoint. We can get rid of the extra code by using the right functions. Signed-off-by: Felipe Contreras --- kernel/params.c | 25 + 1 file changed, 9 insertions(+), 16 deletions(-) diff

[PATCH 1/5] kstrtox: remove redundant cleanup

2013-11-16 Thread Felipe Contreras
We can't reach the cleanup code unless the flag KSTRTOX_OVERFLOW is not set, so there's not no point in clearing a bit that we know is not set. Signed-off-by: Felipe Contreras --- lib/kstrtox.c | 1 - 1 file changed, 1 deletion(-) diff --git a/lib/kstrtox.c b/lib/kstrtox.c index

[PATCH 0/5] Command line related cleanups

2013-11-16 Thread Felipe Contreras
Hi, These became apparent in the review process of a new command line parameter. Felipe Contreras (5): kstrtox: remove redundant cleanup cmdline: fix style issues cmdline: declare exported symbols immediately kstrtox: remove redundant casts params: improve standard definitions

[GIT PULL] sound fixes for 3.13-rc1

2013-11-16 Thread Takashi Iwai
Linus, please pull sound fixes for v3.13-rc1 from: git://git.kernel.org/pub/scm/linux/kernel/git/tiwai/sound.git tags/sound-fix-3.13-rc1 The topmost commit is abfe69dd2e313d0c8226ca4a12329e3d829cfd7c sound fixes for 3.13-rc1

Re: [RFC 16/23] ASoC: omap: mcbsp, mcpdm, dmic: raw read and write endian fix

2013-11-16 Thread Takashi Iwai
At Sat, 16 Nov 2013 18:09:51 +0200, Jarkko Nikula wrote: > > On 11/16/2013 02:01 AM, Taras Kondratiuk wrote: > > From: Victor Kamensky > > > > All OMAP IP blocks expect LE data, but CPU may operate in BE mode. > > Need to use endian neutral functions to read/write h/w registers. > > I.e instead

Re: [PATCH] i2c: Fallback to of_node of parent

2013-11-16 Thread Florian Meier
I have looked through all bus drivers and in most cases they have a corresponding line that could be removed. Although, this patch would break i2c-powermac, because it relies on the fact that of_node stays NULL. Any idea how to handle that? Greetings, Florian On 16.11.2013 17:11, Florian Meier

Re: [RFC 16/23] ASoC: omap: mcbsp, mcpdm, dmic: raw read and write endian fix

2013-11-16 Thread Jarkko Nikula
On 11/16/2013 02:01 AM, Taras Kondratiuk wrote: > From: Victor Kamensky > > All OMAP IP blocks expect LE data, but CPU may operate in BE mode. > Need to use endian neutral functions to read/write h/w registers. > I.e instead of __raw_read[lw] and __raw_write[lw] functions code > need to use

Re: xen drivers fail to link on ARM with v3.12-9888-gf63c482

2013-11-16 Thread Josh Boyer
On Sat, Nov 16, 2013 at 9:56 AM, Josh Boyer wrote: > Hi All, > > The xen-gntalloc, xen-netfront, xen-blkfront, and xen-netback drivers > fail to link on ARM today with the following error: > > ERROR: "phys_to_mach" [drivers/xen/xen-gntalloc.ko] undefined! > ERROR: "phys_to_mach"

Re: [PATCH] i2c: Fallback to of_node of parent

2013-11-16 Thread Florian Meier
Ok, I will try to find all relevant lines. Where is the best place to document this? Greetings, Florian 2013/11/16 Wolfram Sang : > >> + if (!dev->of_node && dev->parent) >> + dev->of_node = dev->parent->of_node; >> + > > That is not enough. Current drivers could then have the

Re: [PATCH] i2c: Fallback to of_node of parent

2013-11-16 Thread Wolfram Sang
> + if (!dev->of_node && dev->parent) > + dev->of_node = dev->parent->of_node; > + That is not enough. Current drivers could then have the assignment removed and even more important, this behaviour should be documented. Regards, Wolfram signature.asc Description: Digital

Re: [PATCH tip/core/rcu 4/4] documentation: Update circular buffer for load-acquire/store-release

2013-11-16 Thread Peter Zijlstra
On Fri, Nov 15, 2013 at 04:10:30PM -0800, Paul E. McKenney wrote: > From: "Paul E. McKenney" > > This commit replaces full barriers by targeted use of load-acquire and > store-release. I guess I'd better hurry up merging the patches that introduce these thingies someplace :-) -- To unsubscribe

Re: [patch 1/2] autofs4: allow autofs to work outside the initial PID namespace

2013-11-16 Thread Oleg Nesterov
On 11/15, Andrew Morton wrote: > > Enable autofs4 to work in a "container". oz_pgrp is converted from pid_t > to struct pid and this is stored at mount time based on the "pgrp=" option > or if the option is missing then the current pgrp. I don't understand this code, so I am probably wrong. And

Re: vmstat: On demand vmstat workers V3

2013-11-16 Thread Frederic Weisbecker
On Thu, Oct 03, 2013 at 05:40:40PM +, Christoph Lameter wrote: > V2->V3: > - Introduce a new tick_get_housekeeping_cpu() function. Not sure > if that is exactly what we want but it is a start. Thomas? Not really. Thomas suggested an infrastructure to move CPU-local periodic jobs handling to

Re: [PATCH tip/core/rcu 4/4] documentation: Update circular buffer for load-acquire/store-release

2013-11-16 Thread Paul E. McKenney
On Sat, Nov 16, 2013 at 11:58:45AM +, David Howells wrote: > Paul E. McKenney wrote: > > > - /* read index before reading contents at that index */ > > > - smp_mb(); /* finish reading descriptor before incrementing tail > > */ > > I'd rather you didn't remove these

[PATCH] i2c: Fallback to of_node of parent

2013-11-16 Thread Florian Meier
Many busses (e.g. tegra, omap, bcm2835) need to set the of_node of the adapter device to the one of the parent device, i.e. adap->dev.of_node = pdev->dev.of_node; As suggested by Stephen Warren, this could also be done in the i2c core and it is a common mistake to forget this line: I2C:

[PATCH V2] Cpufreq: Make governor data on nonboot cpus across system suspend/resume

2013-11-16 Thread Lan Tianyu
Currently, governor of nonboot cpus will be put to EXIT when system suspend. Since all these cpus will be unplugged and the governor usage_count decreases to zero. The governor data and its sysfs interfaces will be freed or released. This makes user config of these governors loss during suspend

Re: [PATCH] power: Change device_wakeup_enable() to check for null dev_name(dev)

2013-11-16 Thread Shuah Khan
On 11/15/2013 05:25 PM, Greg KH wrote: On Fri, Nov 15, 2013 at 05:16:31PM -0700, Shuah Khan wrote: On 11/15/2013 05:21 PM, Rafael J. Wysocki wrote: On Friday, November 15, 2013 05:03:57 PM Shuah Khan wrote: device_wakeup_enable() uses dev_name(dev) as the wakeup source name. When it gets

Re: [Update PATCH 1/1] Cpufreq: Make governor data on nonboot cpus across system suspend/resume

2013-11-16 Thread Lan Tianyu
On 11/16/2013 10:41 PM, Rafael J. Wysocki wrote: On Saturday, November 16, 2013 11:59:59 AM Lan Tianyu wrote: On 11/16/2013 08:38 AM, Rafael J. Wysocki wrote: On Friday, November 15, 2013 04:15:34 PM Lan Tianyu wrote: Currently, governor of nonboot cpus will be put to EXIT when system

Re: [PATCH tip/core/rcu 11/14] bonding/bond_main: Apply ACCESS_ONCE() to avoid sparse false positive

2013-11-16 Thread Paul E. McKenney
On Sat, Nov 16, 2013 at 12:32:16PM +0800, Ding Tianhong wrote: > 于 2013/11/16 8:40, Paul E. McKenney 写道: > > From: "Paul E. McKenney" > > > > The sparse checking for rcu_assign_pointer() was recently upgraded > > to reject non-__kernel address spaces. This also rejects __rcu, > > which is almost

  1   2   3   >