Re: [PATCH trivial] UAPI: Kbuild: add/modify comments for "uapi/Kbuild" and "uapi/linux/Kbuild"

2013-08-06 Thread Chen Gang
On 08/07/2013 01:31 AM, Joe Perches wrote: > On Tue, 2013-08-06 at 09:46 +0800, Chen Gang wrote: >> "include/uapi/" is the whole Linux kernel API, it is important enough >> to get more global explanations by comments. > > It'd probably be useful to have more descriptions > of uapi in the Documenta

Re: [PATCH 0/2] kernel/sys.c: for setfsgid(), return the current gid when error occurs

2013-08-06 Thread Chen Gang
On 08/07/2013 02:36 AM, Kees Cook wrote: > On Tue, Aug 6, 2013 at 1:00 AM, Chen Gang wrote: >> They are 2 related patches for setfsgid(). >> >> Patch 1 for bug fix: return the current gid when error occurs. >> Patch 2 for cleaning code: remove useless variable 'old_fsgid'. >> >> Signed-off-by: Che

Re: [PATCH] fs/nfs/inode.c: adjust code alignment

2013-08-06 Thread Dave Quigley
On 8/6/2013 2:04 PM, Steve Dickson wrote: Hello, On 05/08/13 10:59, Myklebust, Trond wrote: On Mon, 2013-08-05 at 16:47 +0200, Julia Lawall wrote: From: Julia Lawall Signed-off-by: Julia Lawall --- This patch adjusts the code so that the alignment matches the current semantics. I have no

Re: [PATCH 00/11] Add compression support to pstore

2013-08-06 Thread Aruna Balakrishnaiah
On Wednesday 07 August 2013 05:06 AM, Tony Luck wrote: On Mon, Aug 5, 2013 at 2:20 PM, Tony Luck wrote: Still have problems booting if there are any compressed images in ERST to be inflated. So I took another look at this part of the code ... and saw a couple of issues: while ((size

Re: [PATCH 04/12] KVM: MMU: log dirty page after marking spte writable

2013-08-06 Thread Marcelo Tosatti
On Tue, Jul 30, 2013 at 09:02:02PM +0800, Xiao Guangrong wrote: > Make sure we can see the writable spte before the dirt bitmap is visible > > We do this is for kvm_vm_ioctl_get_dirty_log() write-protects the spte based > on the dirty bitmap, we should ensure the writable spte can be found in rmap

Re: [PATCH v2 RESEND 11/18] x86, acpi: Try to find SRAT in firmware earlier.

2013-08-06 Thread Tang Chen
On 08/07/2013 07:33 AM, Toshi Kani wrote: On Fri, 2013-08-02 at 17:14 +0800, Tang Chen wrote: This patch introduce early_acpi_firmware_srat() to find the phys addr of SRAT provided by firmware. And call it in find_hotpluggable_memory(). Since we have initialized acpi_gbl_root_table_list earlier

Re: [PATCH 17/18] mm, hugetlb: retry if we fail to allocate a hugepage with use_reserve

2013-08-06 Thread Davidlohr Bueso
On Wed, 2013-08-07 at 11:03 +1000, David Gibson wrote: > On Tue, Aug 06, 2013 at 05:18:44PM -0700, Davidlohr Bueso wrote: > > On Mon, 2013-08-05 at 16:36 +0900, Joonsoo Kim wrote: > > > > Any mapping that doesn't use the reserved pool, not just > > > > MAP_NORESERVE. For example, if a process make

[PATCH] ARM: dts: Fix memory node in skeleton64.dtsi

2013-08-06 Thread Stepan Moskovchenko
Update the reg property of the memory node in skeleton64.dtsi to reflect the fact that the root node uses address-cells=2 and size-cells=2. Change-Id: Ie9b61166143969e020ceebc51e9a384405d8c0f2 Signed-off-by: Stepan Moskovchenko --- arch/arm/boot/dts/skeleton64.dtsi |2 +- 1 files changed, 1

Re: [PATCH 17/18] mm, hugetlb: retry if we fail to allocate a hugepage with use_reserve

2013-08-06 Thread David Gibson
On Tue, Aug 06, 2013 at 05:18:44PM -0700, Davidlohr Bueso wrote: > On Mon, 2013-08-05 at 16:36 +0900, Joonsoo Kim wrote: > > > Any mapping that doesn't use the reserved pool, not just > > > MAP_NORESERVE. For example, if a process makes a MAP_PRIVATE mapping, > > > then fork()s then the mapping is

Re: List corruption in hidraw_release in 3.11-rc4

2013-08-06 Thread Jiri Kosina
On Tue, 6 Aug 2013, Peter Wu wrote: > While debugging upowerd (with Logitech Unifying receiver via hidraw), > I came across this list corruption warning. Peter, does the patch below fix the problem you are seeing? --- drivers/hid/hidraw.c |2 +- 1 files changed, 1 insertions(+), 1 deletion

Re: [PATCH] proc: Add workaround for idle/iowait decreasing problem.

2013-08-06 Thread Frederic Weisbecker
On Tue, Jul 02, 2013 at 07:39:08PM +0900, Fernando Luis Vazquez Cao wrote: > On 2013年07月02日 12:56, Fernando Luis Vazquez Cao wrote: > >Hi Frederic, > > > >I'm sorry it's taken me so long to respond; I got sidetracked for > >a while. Comments follow below. > > > >On 2013/04/28 09:49, Frederic Weisbe

Re: Performance regression from switching lock to rw-sem for anon-vma tree

2013-08-06 Thread Tim Chen
On Tue, 2013-08-06 at 16:55 -0700, Davidlohr Bueso wrote: > I got good numbers, recovering the performance drop I noticed with the > i_mmap_mutex to rwsem patches. That's good. I remembered that the earlier version of the patch not only recovered the performance drop, but also provide some boost

[PATCH] regulator: fan53555: Prevent multiple include of fan53555.h

2013-08-06 Thread Axel Lin
Add define for __FAN53555_H__ to prevent multiple include of the header file. Signed-off-by: Axel Lin --- include/linux/regulator/fan53555.h | 1 + 1 file changed, 1 insertion(+) diff --git a/include/linux/regulator/fan53555.h b/include/linux/regulator/fan53555.h index 5c45c85..f13880e 100644

Re: [RFC] gcc feature request: Moving blocks into sections

2013-08-06 Thread Steven Rostedt
On Tue, 2013-08-06 at 20:45 -0400, Steven Rostedt wrote: > [3.387362] short jumps: 106 > [3.390277] long jumps: 330 > > Thus, approximately 25%. Not bad. Also, where these happen to be is probably even more important than how many. If all the short jumps happen in slow paths, it's rathe

Re: [PATCH] mm/mempolicy: return NULL if node is NUMA_NO_NODE in get_task_policy

2013-08-06 Thread Jianguo Wu
Hi Andrew, On 2013/8/7 5:03, Andrew Morton wrote: > On Tue, 6 Aug 2013 12:06:56 +0800 Jianguo Wu wrote: > >> If node == NUMA_NO_NODE, pol is NULL, we should return NULL instead of >> do "if (!pol->mode)" check. >> >> Signed-off-by: Jianguo Wu >> --- >> mm/mempolicy.c | 2 ++ >> 1 file changed

Re: [PATCH aio-next] aio: fix error handling and rcu usage in "convert the ioctx list to table lookup v3"

2013-08-06 Thread Benjamin LaHaise
On Tue, Aug 06, 2013 at 05:57:32PM -0400, Sasha Levin wrote: > Old error is gone, but now seeing this, which seems related. > > ctx = table->table[id]; > if (ctx->user_id == ctx_id) { <--- here > percpu_ref_get(&ctx->users); > ret = ctx; > }

Re: acpi_bus_trim does not detach devices in post order

2013-08-06 Thread Rafael J. Wysocki
On Wednesday, August 07, 2013 09:35:51 AM Yasuaki Ishimatsu wrote: > (2013/08/06 23:26), Rafael J. Wysocki wrote: > > On Tuesday, August 06, 2013 07:06:37 PM Yasuaki Ishimatsu wrote: > >> (2013/08/06 12:07), Yasuaki Ishimatsu wrote: > >>> > >>> I acked the following commit but I hit a problem by th

Re: [RFC] gcc feature request: Moving blocks into sections

2013-08-06 Thread Steven Rostedt
On Tue, 2013-08-06 at 16:43 -0400, Steven Rostedt wrote: > On Tue, 2013-08-06 at 16:33 -0400, Mathieu Desnoyers wrote: > > > Steve, perhaps you could add a mode to your binary rewriting program > > that counts the number of 2-byte vs 5-byte jumps found, and if possible > > get a breakdown of those

Re: acpi_bus_trim does not detach devices in post order

2013-08-06 Thread Yasuaki Ishimatsu
(2013/08/06 23:26), Rafael J. Wysocki wrote: On Tuesday, August 06, 2013 07:06:37 PM Yasuaki Ishimatsu wrote: (2013/08/06 12:07), Yasuaki Ishimatsu wrote: I acked the following commit but I hit a problem by the commit. So I report it. commit cecdb193c8d91a42d9489d00618cc3dfff92e55a Author: Ra

Re: [systemd-devel] [PATCH] Change CONFIG_FW_LOADER_USER_HELPER to default n and don't select it

2013-08-06 Thread Andy Lutomirski
On Tue, Aug 6, 2013 at 5:24 PM, Tom Gundersen wrote: > > On 6 Aug 2013 18:32, "Bryan Kadzban" wrote: >> >> On Tue, Aug 06, 2013 at 11:17:17AM +0200, Tom Gundersen wrote: >> > On Tue, Aug 6, 2013 at 11:11 AM, Tom Gundersen wrote: >> > > On Tue, Aug 6, 2013 at 10:20 AM, Maarten Lankhorst >> > > w

Re: [PATCH 17/18] mm, hugetlb: retry if we fail to allocate a hugepage with use_reserve

2013-08-06 Thread Davidlohr Bueso
On Mon, 2013-08-05 at 16:36 +0900, Joonsoo Kim wrote: > > Any mapping that doesn't use the reserved pool, not just > > MAP_NORESERVE. For example, if a process makes a MAP_PRIVATE mapping, > > then fork()s then the mapping is instantiated in the child, that will > > not draw from the reserved pool

Re: [PATCH] proc: Add workaround for idle/iowait decreasing problem.

2013-08-06 Thread Frederic Weisbecker
On Tue, Jul 02, 2013 at 12:56:04PM +0900, Fernando Luis Vazquez Cao wrote: > Hi Frederic, > > I'm sorry it's taken me so long to respond; I got sidetracked for > a while. Comments follow below. > > On 2013/04/28 09:49, Frederic Weisbecker wrote: > >On Tue, Apr 23, 2013 at 09:45:23PM +0900, Tetsuo

Re: [PATCH] block: support embedded device command line partition

2013-08-06 Thread Andrew Morton
On Tue, 6 Aug 2013 10:53:01 + Caizhiyong wrote: > > -Original Message- > > From: Andrew Morton [mailto:a...@linux-foundation.org] > > Sent: Tuesday, August 06, 2013 6:22 AM > > To: Caizhiyong > > Cc: Karel Zak; linux-kernel@vger.kernel.org; Wanglin (Albert); Quyaxin; > > Jens Axboe;

Re: [PATCH V2 00/11] CPUFreq: Fixes & Cleanups for 3.12

2013-08-06 Thread Rafael J. Wysocki
On Tuesday, August 06, 2013 10:53:02 PM Viresh Kumar wrote: > Hi Rafael, > > This is V2 of the patch series which was posted here earlier: > > http://www.gossamer-threads.com/lists/linux/kernel/1759514 > > This patchset tries to fix & cleanup many existing cpufreq core issues. First > four patch

Re: [PATCH 0/2] hugepage: optimize page fault path locking

2013-08-06 Thread Davidlohr Bueso
On Mon, 2013-07-29 at 15:18 +0900, Joonsoo Kim wrote: > On Fri, Jul 26, 2013 at 07:27:23AM -0700, Davidlohr Bueso wrote: > > This patchset attempts to reduce the amount of contention we impose > > on the hugetlb_instantiation_mutex by replacing the global mutex with > > a table of mutexes, selected

Re: [PATCH] MAINTAINERS: drivers/power: add entry for SmartReflex AVS drivers

2013-08-06 Thread Rafael J. Wysocki
On Tuesday, August 06, 2013 04:58:05 PM Kevin Hilman wrote: > "Rafael J. Wysocki" writes: > > > On Monday, August 05, 2013 09:29:19 AM Kevin Hilman wrote: > >> The SmartReflex AVS driver evolved out of the OMAP kernel and now > >> lives under drivers/power/avs. > >> > >> I've historically been m

Re: [PATCH] mutex: fix deadlock injection

2013-08-06 Thread Dave Airlie
On Tue, Jul 30, 2013 at 6:41 PM, Peter Zijlstra wrote: > On Tue, Jul 30, 2013 at 10:13:41AM +0200, Maarten Lankhorst wrote: >> The check needs to be for > 1, because ctx->acquired is already incremented. >> This will prevent ww_mutex_lock_slow from returning -EDEADLK and not locking >> the mutex.

Re: [PATCH] pktcdvd: Fix defective misuses of pkt_

2013-08-06 Thread Jiri Kosina
On Tue, 6 Aug 2013, Andrew Morton wrote: > > Andrew, I believe Jiri picked up this series and > > you can drop them from mm. Thanks. > > Shall do if/when they turn up in linux-next. Pull request to Jens is there [1], but his branch has been silent for two weeks ... [1] https://lkml.org/lkml/2

Re: [PATCH] MAINTAINERS: drivers/power: add entry for SmartReflex AVS drivers

2013-08-06 Thread Kevin Hilman
"Rafael J. Wysocki" writes: > On Monday, August 05, 2013 09:29:19 AM Kevin Hilman wrote: >> The SmartReflex AVS driver evolved out of the OMAP kernel and now >> lives under drivers/power/avs. >> >> I've historically been maintainer of this but Nishanth Menon is doing >> most of the heavy lifting

Re: [PATCH] MAINTAINERS: drivers/power: add entry for SmartReflex AVS drivers

2013-08-06 Thread Kevin Hilman
Anton Vorontsov writes: > On Mon, Aug 05, 2013 at 09:29:37AM -0700, Kevin Hilman wrote: >> The SmartReflex AVS driver evolved out of the OMAP kernel and now >> lives under drivers/power/avs. >> >> I've historically been maintainer of this but Nishanth Menon is doing >> most of the heavy lifting

Re: Performance regression from switching lock to rw-sem for anon-vma tree

2013-08-06 Thread Davidlohr Bueso
On Mon, 2013-07-29 at 17:13 -0700, Tim Chen wrote: > On Tue, 2013-07-23 at 11:53 +0200, Ingo Molnar wrote: > > * Peter Zijlstra wrote: > > > > > On Tue, Jul 23, 2013 at 11:45:13AM +0200, Ingo Molnar wrote: > > > > > > > Why not just try the delayed addition approach first? The spinning is > > >

Re: [PATCH] pinctrl: msm: Add support for MSM TLMM pinmux

2013-08-06 Thread Hanumant Singh
On 7/31/2013 5:17 PM, Hanumant Singh wrote: On 7/31/2013 2:06 PM, Stephen Warren wrote: On 07/31/2013 01:46 PM, Hanumant Singh wrote: On 7/30/2013 8:59 PM, Stephen Warren wrote: On 07/30/2013 06:13 PM, Hanumant Singh wrote: On 7/30/2013 5:08 PM, Stephen Warren wrote: On 07/30/2013 06:01 PM,

Re: [PATCH V2 5/6] cpuidle/powerpc: Backend-powerpc idle driver for powernv and pseries.

2013-08-06 Thread Scott Wood
On Wed, 2013-08-07 at 09:30 +1000, Benjamin Herrenschmidt wrote: > On Tue, 2013-08-06 at 18:08 -0500, Scott Wood wrote: > > Here's another example. get_lppaca() will only build on book3s -- and > > yet we get requests for e500 code to use this file. > > Indeed, Besides there is already accessors

Re: [PATCH 1/1] module: fix sprintf format specifier in param_get_byte()

2013-08-06 Thread Joe Perches
On Tue, 2013-08-06 at 23:12 +0200, Christoph Jaeger wrote: > In param_get_byte(), to which the macro STANDARD_PARAM_DEF(byte, ...) expands, > "%c" is used to print an unsigned char. So it gets printed as a character what > is not intended here. Use "%hhu" instead. > > Signed-off-by: Christoph Jaeg

Re: [PATCH 00/11] Add compression support to pstore

2013-08-06 Thread Tony Luck
On Mon, Aug 5, 2013 at 2:20 PM, Tony Luck wrote: > Still have problems booting if there are any compressed images in ERST > to be inflated. So I took another look at this part of the code ... and saw a couple of issues: while ((size = psi->read(&id, &type, &count, &time, &buf, &compresse

Re: [PATCH v2 RESEND 11/18] x86, acpi: Try to find SRAT in firmware earlier.

2013-08-06 Thread Toshi Kani
On Fri, 2013-08-02 at 17:14 +0800, Tang Chen wrote: > This patch introduce early_acpi_firmware_srat() to find the > phys addr of SRAT provided by firmware. And call it in > find_hotpluggable_memory(). > > Since we have initialized acpi_gbl_root_table_list earlier, > and store all the tables' phys

Re: [PATCH V2 5/6] cpuidle/powerpc: Backend-powerpc idle driver for powernv and pseries.

2013-08-06 Thread Benjamin Herrenschmidt
On Tue, 2013-08-06 at 18:08 -0500, Scott Wood wrote: > Here's another example. get_lppaca() will only build on book3s -- and > yet we get requests for e500 code to use this file. Indeed, Besides there is already accessors afaik for lppaca that compile to nothing on E (and if not they would be tri

Re: [PATCH v4 8/9] pci: Tune secondary bus reset timing

2013-08-06 Thread Alexander Duyck
On 08/05/2013 12:37 PM, Alex Williamson wrote: > The PCI spec indicates that with stable power, reset needs to be > asserted for a minimum of 1ms (Trst). Seems like we should be able > to assume power is stable for a runtime secondary bus reset. The > current code has always used 100ms with no ex

[PATCH] efifb: prevent null dereferences by removing unused array indices from dmi_list

2013-08-06 Thread James Bates
Hi all, The dmi_list array is initialized using gnu designated initializers, and therefore contains fewer explicitly defined entries as there are elements in it. This is because the enum above with M_blabla constants contains more items than the designated initializer. Those elements not explici

Re: kernel mailing list permalinks

2013-08-06 Thread Joe Perches
On Tue, 2013-08-06 at 16:05 -0700, H. Peter Anvin wrote: > On 08/06/2013 02:55 PM, Anders Larsen wrote: > > On 2013-08-06 23:23, H. Peter Anvin wrote: > >> On 08/06/2013 02:11 PM, Bjorn Helgaas wrote: > >> >> > >> >> The above link is also a good example of subtleness: + in a URL > >> means a > >>

Re: [PATCH 1/3] f2fs: add sysfs support for controlling the gc_thread

2013-08-06 Thread Namjae Jeon
2013/8/6, Jaegeuk Kim : > Hi, Namjae, Hi Jaeguek. > > I found and fixed a bug as follows. > > In f2fs_put_super, we should call kobject_del & kobject_put. > And, IMO, we'd better give -EINVAL instead of -EFAULT to users when > there is no background gc thread. Yes, Agree. Sorry to bother. > > If yo

Re: [PATCH V2 5/6] cpuidle/powerpc: Backend-powerpc idle driver for powernv and pseries.

2013-08-06 Thread Scott Wood
On Wed, 2013-07-31 at 08:29 +0530, Deepthi Dharwar wrote: > /* > - * pseries_idle_probe() > + * powerpc_idle_probe() > * Choose state table for shared versus dedicated partition > */ > -static int pseries_idle_probe(void) > +static int powerpc_idle_probe(void) > { > > +#ifndef PPC_POWERNV >

Re: [Ksummit-2013-discuss] [ARM ATTEND] arch/arm SoC organization

2013-08-06 Thread Christian Daudt
On Sun, Aug 4, 2013 at 11:21 PM, Tony Lindgren wrote: > * Christian Daudt [130802 16:13]: >> On Fri, Aug 2, 2013 at 1:33 AM, Tony Lindgren wrote: >> > * Jason Cooper [130731 07:25]: >> >> So, I'd like to propose we discuss some lessons learned and maybe arrive >> >> at some best practices. eg,

Re: kernel mailing list permalinks

2013-08-06 Thread H. Peter Anvin
On 08/06/2013 02:55 PM, Anders Larsen wrote: > On 2013-08-06 23:23, H. Peter Anvin wrote: >> On 08/06/2013 02:11 PM, Bjorn Helgaas wrote: >> >> >> >> The above link is also a good example of subtleness: + in a URL >> means a >> >> space character, it has to be escaped as %2B. Particularly a >> pro

Re: [PATCH v2 RESEND 04/18] acpi: Introduce acpi_verify_initrd() to check if a table is invalid.

2013-08-06 Thread Toshi Kani
On Fri, 2013-08-02 at 17:14 +0800, Tang Chen wrote: > In acpi_initrd_override(), it checks several things to ensure the > table it found is valid. In later patches, we need to do these check > somewhere else. So this patch introduces a common function > acpi_verify_initrd() to do all these checks,

Re: [Ksummit-2013-discuss] [ARM ATTEND] arch/arm SoC organization

2013-08-06 Thread Christian Daudt
On Fri, Aug 2, 2013 at 10:24 PM, Olof Johansson wrote: > On Fri, Aug 2, 2013 at 4:06 PM, Christian Daudt wrote: >> On Fri, Aug 2, 2013 at 1:33 AM, Tony Lindgren wrote: >>> * Jason Cooper [130731 07:25]: So, I'd like to propose we discuss some lessons learned and maybe arrive at some b

Re: [PATCH 0/5] ACPI: acpi_bind_one()/acpi_unbind_one() cleanups

2013-08-06 Thread Toshi Kani
On Wed, 2013-08-07 at 00:29 +0200, Rafael J. Wysocki wrote: > On Tuesday, August 06, 2013 03:15:36 PM Toshi Kani wrote: > > On Tue, 2013-08-06 at 02:22 +0200, Rafael J. Wysocki wrote: > > > Hi All, > > > > > > The following 5 patches clean up a little mess in acpi_bind_one() and > > > acpi_unbind_

[PATCH] thermal: new asus driver

2013-08-06 Thread Felipe Contreras
Simple driver to enable control of the fan in ASUS laptops. So far this has only been tested in ASUS Zenbook Prime UX31A, but according to some online reference [1], it should work in other models as well. Another source was a patch acpi4asus-user's mailing list [2]. [1] http://forum.notebookrevi

[PATCH] ACPI: Try harder to resolve _ADR collisions for bridges

2013-08-06 Thread Rafael J. Wysocki
From: Rafael J. Wysocki In theory, under a given ACPI namespace node there should be only one child device object with _ADR whose value matches a given bus address exactly. In practice, however, there are systems in which multiple child device objects under a given parent have _ADR matching exac

[patch 0/9] mm: thrash detection-based file cache sizing v3

2013-08-06 Thread Johannes Weiner
[ My apologies for the double send, I screwed up one of the recipient addresses the first time around and it got dropped by some MTAs. ] Changes in version 3: o Lazily remove inodes without shadow entries from the global list to reduce modifications of said list to an absolute minimum. Globa

[patch 6/9] mm + fs: provide shadow pages to page cache allocations

2013-08-06 Thread Johannes Weiner
In order to make informed placement and reclaim decisions, the page cache allocation requires the shadow information of refaulting pages. Every site that does a find_or_create()-style page cache allocation is converted to pass the shadow page found in the faulting slot of the radix tree to page_ca

[patch 5/9] mm + fs: store shadow entries in page cache

2013-08-06 Thread Johannes Weiner
Reclaim will be leaving shadow entries in the page cache radix tree upon evicting the real page. As those pages are found from the LRU, an iput() can lead to the inode being freed concurrently. At this point, reclaim must no longer install shadow pages because the inode freeing code needs to ensu

[patch 1/9] lib: radix-tree: radix_tree_delete_item()

2013-08-06 Thread Johannes Weiner
Provide a function that does not just delete an entry at a given index, but also allows passing in an expected item. Delete only if that item is still located at the specified index. This is handy when lockless tree traversals want to delete entries as well because they don't have to do an second

[patch 7/9] mm: make global_dirtyable_memory() available to other mm code

2013-08-06 Thread Johannes Weiner
Subsequent patches need a rough estimate of memory available for page cache. Signed-off-by: Johannes Weiner --- include/linux/writeback.h | 1 + mm/page-writeback.c | 2 +- 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/include/linux/writeback.h b/include/linux/writeback.h i

[patch 8/9] mm: thrash detection-based file cache sizing

2013-08-06 Thread Johannes Weiner
The VM maintains cached filesystem pages on two types of lists. One list holds the pages recently faulted into the cache, the other list holds pages that have been referenced repeatedly on that first list. The idea is to prefer reclaiming young pages over those that have shown to benefit from cach

[patch 3/9] mm: filemap: move radix tree hole searching here

2013-08-06 Thread Johannes Weiner
The radix tree hole searching code is only used for page cache, for example the readahead code trying to get a a picture of the area surrounding a fault. It sufficed to rely on the radix tree definition of holes, which is "empty tree slot". But this is about to change, though, as shadow page desc

[patch 9/9] mm: workingset: keep shadow entries in check

2013-08-06 Thread Johannes Weiner
Previously, page cache radix tree nodes were freed after reclaim emptied out their page pointers. But now reclaim stores shadow entries in their place, which are only reclaimed when the inodes themselves are reclaimed. This is problematic for bigger files that are still in use after they have a s

[patch 4/9] mm + fs: prepare for non-page entries in page cache radix trees

2013-08-06 Thread Johannes Weiner
shmem mappings already contain exceptional entries where swap slot information is remembered. To be able to store eviction information for regular page cache, prepare every site dealing with the radix trees directly to handle entries other than pages. The common lookup functions will filter out n

[patch 2/9] mm: shmem: save one radix tree lookup when truncating swapped pages

2013-08-06 Thread Johannes Weiner
Page cache radix tree slots are usually stabilized by the page lock, but shmem's swap cookies have no such thing. Because the overall truncation loop is lockless, the swap entry is currently confirmed by a tree lookup and then deleted by another tree lookup under the same tree lock region. Use ra

Re: [PATCH] MAINTAINERS: drivers/power: add entry for SmartReflex AVS drivers

2013-08-06 Thread Rafael J. Wysocki
On Monday, August 05, 2013 09:29:19 AM Kevin Hilman wrote: > The SmartReflex AVS driver evolved out of the OMAP kernel and now > lives under drivers/power/avs. > > I've historically been maintainer of this but Nishanth Menon is doing > most of the heavy lifting now. Add us both as co-maintainers.

Re: [PATCH 0/8] partitions/efi: detect hybrid mbrs

2013-08-06 Thread Davidlohr Bueso
On Tue, 2013-08-06 at 14:16 -0700, Andrew Morton wrote: > On Mon, 5 Aug 2013 22:21:08 -0700 Davidlohr Bueso wrote: > > > This patchset teaches the kernel about hybrid master boot records (MBRs), > > one of > > the most common issues with GUID partition tables, as a workaround to layout > > disk

Re: [PATCH v14 0/6] LSM: Multiple concurrent LSMs

2013-08-06 Thread Kees Cook
On Tue, Aug 6, 2013 at 3:25 PM, Casey Schaufler wrote: > On 8/5/2013 11:30 PM, Kees Cook wrote: >> On Thu, Jul 25, 2013 at 11:52 PM, Casey Schaufler >> wrote: >>> The /proc/*/attr interfaces are given to one LSM. This can be >>> done by setting CONFIG_SECURITY_PRESENT. Additional interfaces >>>

Re: [PATCH v2] cpufreq: loongson2: fix broken cpufreq

2013-08-06 Thread Rafael J. Wysocki
On Tuesday, August 06, 2013 04:40:12 PM Ralf Baechle wrote: > On Mon, Aug 05, 2013 at 09:27:12PM +0300, Aaro Koskinen wrote: > > > Commit 42913c799 (MIPS: Loongson2: Use clk API instead of direct > > dereferences) broke the cpufreq functionality on Loongson2 boards: > > clk_set_rate() is called be

Re: sl811h_suspend() and PM_EVENT_PRETHAW state handling

2013-08-06 Thread Shuah Khan
On 08/06/2013 04:28 PM, Rafael J. Wysocki wrote: > On Tuesday, August 06, 2013 09:38:36 PM Shuah Khan wrote: >> On 08/06/2013 03:22 PM, Alan Stern wrote: >>> On Tue, 6 Aug 2013, Shuah Khan wrote: >>> With the dev_pm_ops model, drivers have to provide interfaces for each one of these state

Re: sl811h_suspend() and PM_EVENT_PRETHAW state handling

2013-08-06 Thread Rafael J. Wysocki
On Tuesday, August 06, 2013 09:38:36 PM Shuah Khan wrote: > On 08/06/2013 03:22 PM, Alan Stern wrote: > > On Tue, 6 Aug 2013, Shuah Khan wrote: > > > >> With the dev_pm_ops model, drivers have to provide interfaces for each > >> one of these states. > > > > No, they don't. They can leave out inter

Re: [PATCH v5] x86, tboot: iomem fixes

2013-08-06 Thread H. Peter Anvin
On 07/20/2013 09:26 AM, Qiaowei Ren wrote: > > /* now map TXT heap */ > - heap_base = ioremap(*(u64 *)(config + TXTCR_HEAP_BASE), > - *(u64 *)(config + TXTCR_HEAP_SIZE)); > + heap_base = ioremap(readl(config + TXTCR_HEAP_BASE), > + rea

Re: [PATCH v14 0/6] LSM: Multiple concurrent LSMs

2013-08-06 Thread Casey Schaufler
On 8/5/2013 11:30 PM, Kees Cook wrote: > On Thu, Jul 25, 2013 at 11:52 PM, Casey Schaufler > wrote: >> The /proc/*/attr interfaces are given to one LSM. This can be >> done by setting CONFIG_SECURITY_PRESENT. Additional interfaces >> have been created in /proc/*/attr so that each LSM has its own

Re: [PATCH] mm, page_alloc: optimize batch count in free_pcppages_bulk()

2013-08-06 Thread Andrew Morton
On Tue, 6 Aug 2013 17:40:40 +0900 Joonsoo Kim wrote: > If we use a division operation, we can compute a batch count more closed > to ideal value. With this value, we can finish our job within > MIGRATE_PCPTYPES iteration. In addition, batching to free more pages > may be helpful to cache usage.

Re: [PATCH v4 2/4] mmc: dw_mmc: Add exynos resume_noirq callback to clear WAKEUP_INT

2013-08-06 Thread Tomasz Figa
On Tuesday 06 of August 2013 15:09:46 Doug Anderson wrote: > Tomasz, > > On Tue, Aug 6, 2013 at 2:58 PM, Tomasz Figa wrote: > >> +static int dw_mci_exynos_resume_noirq(struct device *dev) > >> +{ > >> + struct dw_mci *host = dev_get_drvdata(dev); > >> + u32 clksel; > >> + > >> + clks

Re: [PATCH 0/5] ACPI: acpi_bind_one()/acpi_unbind_one() cleanups

2013-08-06 Thread Rafael J. Wysocki
On Tuesday, August 06, 2013 03:15:36 PM Toshi Kani wrote: > On Tue, 2013-08-06 at 02:22 +0200, Rafael J. Wysocki wrote: > > Hi All, > > > > The following 5 patches clean up a little mess in acpi_bind_one() and > > acpi_unbind_one(). They are on top of current linux-next plus the patch > > at http

Re: [PATCH] kernel/sysctl_binary.c: improve the usage of return value 'result'

2013-08-06 Thread Eric W. Biederman
Andrew Morton writes: > On Tue, 06 Aug 2013 15:29:42 +0800 Chen Gang wrote: > >> Improve the usage of return value 'result', so not only can make code >> clearer to readers, but also can improve the performance. > > It used to be pervasive kernel style do to > > ret = -ENOMEM; > foo

Re: [PATCH] kernel/sysctl_binary.c: improve the usage of return value 'result'

2013-08-06 Thread Joe Perches
On Tue, 2013-08-06 at 14:43 -0700, Andrew Morton wrote: > On Tue, 06 Aug 2013 15:29:42 +0800 Chen Gang wrote: > > > Improve the usage of return value 'result', so not only can make code > > clearer to readers, but also can improve the performance. > > It used to be pervasive kernel style do to >

Re: [PATCH v4 2/4] mmc: dw_mmc: Add exynos resume_noirq callback to clear WAKEUP_INT

2013-08-06 Thread Doug Anderson
Tomasz, On Tue, Aug 6, 2013 at 2:58 PM, Tomasz Figa wrote: >> +static int dw_mci_exynos_resume_noirq(struct device *dev) >> +{ >> + struct dw_mci *host = dev_get_drvdata(dev); >> + u32 clksel; >> + >> + clksel = mci_readl(host, CLKSEL); >> + if (clksel & SDMMC_CLKSEL_WAKEUP_INT) >

Re: [PATCH 3/3 V2] EDAC, AMD64_EDAC: Add ECC decoding support for newer F15h models.

2013-08-06 Thread Borislav Petkov
On Tue, Aug 06, 2013 at 05:00:51PM -0500, Aravind Gopalakrishnan wrote: > Quick question: Shall I change all instances of > boot_cpu_data.[x86|x86_model] to use pvt->fam and pvt->model wherever > applicable as part of this patch or have it go in as a separate patch? Yes, a separate pre-patch would

Re: kernel mailing list permalinks

2013-08-06 Thread Anders Larsen
On 2013-08-06 23:23, H. Peter Anvin wrote: On 08/06/2013 02:11 PM, Bjorn Helgaas wrote: >> >> The above link is also a good example of subtleness: + in a URL means a >> space character, it has to be escaped as %2B. Particularly a problem in >> gmail message-ids. the link in question works

Re: [PATCH v4 4/4] mmc: dw_mmc: Set timeout to max upon resume

2013-08-06 Thread Tomasz Figa
On Tuesday 06 of August 2013 14:37:51 Doug Anderson wrote: > The TMOUT register is initted to 0x at probe time but isn't > initted after suspend/resume. Add an init of this value. > > No problems were observed without this (it will also get initted in > __dw_mci_start_request if there is

Re: [PATCH v4 3/4] mmc: dw_mmc: Always setup the bus after suspend/resume

2013-08-06 Thread Tomasz Figa
On Tuesday 06 of August 2013 14:37:50 Doug Anderson wrote: > After suspend/resume all of the dw_mmc registers are reset to > defaults. We restore most of them, but specifically don't setup the > clock registers after resume unless we've got a powered card. Things > still work because the core wil

Re: [PATCH 3/3 V2] EDAC, AMD64_EDAC: Add ECC decoding support for newer F15h models.

2013-08-06 Thread Aravind Gopalakrishnan
On 8/6/2013 3:55 PM, Aravind Gopalakrishnan wrote: On 8/6/2013 3:23 PM, Borislav Petkov wrote: On Fri, Aug 02, 2013 at 05:43:04PM -0500, Aravind Gopalakrishnan wrote: Adding support for handling ECC error decoding for new F15 models. On newer models, support has been included for upto 4 DCT's,

Re: [PATCH v4 1/4] mmc: dw_mmc: Invalidate cache of current_speed after suspend/resume

2013-08-06 Thread Tomasz Figa
On Tuesday 06 of August 2013 14:37:48 Doug Anderson wrote: > The dw_mmc driver keeps a cache of the current slot->clock in order to > avoid doing a whole lot of work every time set_ios() is called. > However, after suspend/resume the register values are bogus so we need > to ensure that the cached

Re: [PATCH v4 2/4] mmc: dw_mmc: Add exynos resume_noirq callback to clear WAKEUP_INT

2013-08-06 Thread Tomasz Figa
Hi Doug, See my comment inline. On Tuesday 06 of August 2013 14:37:49 Doug Anderson wrote: > If the WAKEUP_INT is asserted at wakeup and not cleared, we'll end up > looping around forever. This has been seen to happen on exynos5420 > silicon despite the fact that we haven't enabled any wakeup ev

Re: [PATCH] ioatdma: Fix compiler warning

2013-08-06 Thread Jon Mason
On Tue, Aug 06, 2013 at 11:52:35PM +0200, Paul Bolle wrote: > On Tue, 2013-08-06 at 14:36 -0700, Jon Mason wrote: > > GCC v4.6.3 complains: > > drivers/dma/ioat/dma_v3.c: In function ‘__ioat3_prep_pq16_lock’: > > drivers/dma/ioat/dma_v3.c:264:11: warning: array subscript is below array > > bounds

Re: [PATCH aio-next] aio: fix error handling and rcu usage in "convert the ioctx list to table lookup v3"

2013-08-06 Thread Sasha Levin
nked in: [ 542.185866] CPU: 2 PID: 22471 Comm: trinity-child36 Tainted: GW 3.11.0-rc4-next-20130806-sasha-2-gb144a3f #3977 [ 542.187428] task: 88020bc4 ti: 8801b6e7e000 task.ti: 8801b6e7e000 [ 542.188384] RIP: 0010:[] [] lookup_ioctx+0x8d/0xe0 [ 542.189408

Re: [PATCH 19/23] truncate: support huge pages

2013-08-06 Thread Dave Hansen
On 08/03/2013 07:17 PM, Kirill A. Shutemov wrote: > If a huge page is only partly in the range we zero out the part, > exactly like we do for partial small pages. What's the logic behind this behaviour? Seems like the kind of place that we would really want to be splitting pages. > + if (par

Re: [PATCH] ioatdma: Fix compiler warning

2013-08-06 Thread Paul Bolle
On Tue, 2013-08-06 at 14:36 -0700, Jon Mason wrote: > GCC v4.6.3 complains: > drivers/dma/ioat/dma_v3.c: In function ‘__ioat3_prep_pq16_lock’: > drivers/dma/ioat/dma_v3.c:264:11: warning: array subscript is below array > bounds [-Warray-bounds] > drivers/dma/ioat/dma_v3.c:264:11: warning: array su

[tip:x86/asmlinkage] x86: Fix sys_call_table type in asm/ syscall.h

2013-08-06 Thread tip-bot for Andi Kleen
Commit-ID: 1599e8fc84dd9c2954df0d4b2db52f7f6719f339 Gitweb: http://git.kernel.org/tip/1599e8fc84dd9c2954df0d4b2db52f7f6719f339 Author: Andi Kleen AuthorDate: Mon, 5 Aug 2013 15:02:35 -0700 Committer: H. Peter Anvin CommitDate: Tue, 6 Aug 2013 14:18:08 -0700 x86: Fix sys_call_table type

[tip:x86/asmlinkage] x86, asmlinkage: Make kprobes code visible and fix assembler code

2013-08-06 Thread tip-bot for Andi Kleen
Commit-ID: 04bb591ca74fb8ea06d5ab7fadfb7bf5b11fb28e Gitweb: http://git.kernel.org/tip/04bb591ca74fb8ea06d5ab7fadfb7bf5b11fb28e Author: Andi Kleen AuthorDate: Mon, 5 Aug 2013 15:02:41 -0700 Committer: H. Peter Anvin CommitDate: Tue, 6 Aug 2013 14:19:48 -0700 x86, asmlinkage: Make kprobe

[tip:x86/asmlinkage] x86, asmlinkage: Make dump_stack visible

2013-08-06 Thread tip-bot for Andi Kleen
Commit-ID: b6c035d04e80b4244a143e34e51c84cf2181bd94 Gitweb: http://git.kernel.org/tip/b6c035d04e80b4244a143e34e51c84cf2181bd94 Author: Andi Kleen AuthorDate: Mon, 5 Aug 2013 15:02:48 -0700 Committer: H. Peter Anvin CommitDate: Tue, 6 Aug 2013 14:21:01 -0700 x86, asmlinkage: Make dump_s

Re: [PATCH] kernel/sysctl_binary.c: improve the usage of return value 'result'

2013-08-06 Thread Eric W. Biederman
Chen Gang writes: Have you tested this code? Do you have anything that actually the uses sysctl binary interface? If you do have code that actually uses this interface please fix it not to use it. This code is fundamentally a stop gap measure and will bit-rot in time and then we will remove it

[PATCH v4 0/4] mmc: dw_mmc: fixes for suspend/resume on exynos

2013-08-06 Thread Doug Anderson
This series of patches addresses some suspend/resume problems with dw_mmc on exynos platforms, espeically exynos5420. Since suspend/resume is not fully working on ToT Linux (v3.11-rc4) on exynos5250-snow, this series was tested against the current ToT ChromeOS 3.8 tree. I have confirmed basic boo

[tip:x86/asmlinkage] x86, asmlinkage, power: Make various symbols used by the suspend asm code visible

2013-08-06 Thread tip-bot for Andi Kleen
Commit-ID: d6efc2f7240b4e55590df69d74f33fdb72ce934a Gitweb: http://git.kernel.org/tip/d6efc2f7240b4e55590df69d74f33fdb72ce934a Author: Andi Kleen AuthorDate: Mon, 5 Aug 2013 15:02:49 -0700 Committer: H. Peter Anvin CommitDate: Tue, 6 Aug 2013 14:21:03 -0700 x86, asmlinkage, power: Make

[tip:x86/asmlinkage] x86, asmlinkage, vdso: Mark vdso variables __visible

2013-08-06 Thread tip-bot for Andi Kleen
Commit-ID: 28596b6a8779b736829ad837f95fdc2e81bdd1ee Gitweb: http://git.kernel.org/tip/28596b6a8779b736829ad837f95fdc2e81bdd1ee Author: Andi Kleen AuthorDate: Mon, 5 Aug 2013 15:02:50 -0700 Committer: H. Peter Anvin CommitDate: Tue, 6 Aug 2013 14:21:08 -0700 x86, asmlinkage, vdso: Mark

[tip:x86/asmlinkage] x86, asmlinkage: Make various syscalls asmlinkage

2013-08-06 Thread tip-bot for Andi Kleen
Commit-ID: ff49103fdba59aa3f845ad2dcac0c26a9e4c5404 Gitweb: http://git.kernel.org/tip/ff49103fdba59aa3f845ad2dcac0c26a9e4c5404 Author: Andi Kleen AuthorDate: Mon, 5 Aug 2013 15:02:40 -0700 Committer: H. Peter Anvin CommitDate: Tue, 6 Aug 2013 14:18:33 -0700 x86, asmlinkage: Make variou

[tip:x86/asmlinkage] x86, asmlinkage: Make 64bit checksum functions visible

2013-08-06 Thread tip-bot for Andi Kleen
Commit-ID: 4a335c0695074c1fd2ede7f919b0064f53d35c3f Gitweb: http://git.kernel.org/tip/4a335c0695074c1fd2ede7f919b0064f53d35c3f Author: Andi Kleen AuthorDate: Mon, 5 Aug 2013 15:02:47 -0700 Committer: H. Peter Anvin CommitDate: Tue, 6 Aug 2013 14:20:59 -0700 x86, asmlinkage: Make 64bit

[tip:x86/asmlinkage] x86, asmlinkage: Make 32bit/ 64bit __switch_to visible

2013-08-06 Thread tip-bot for Andi Kleen
Commit-ID: 35ea7903b8a97162e38da9da3b560df74713321d Gitweb: http://git.kernel.org/tip/35ea7903b8a97162e38da9da3b560df74713321d Author: Andi Kleen AuthorDate: Mon, 5 Aug 2013 15:02:39 -0700 Committer: H. Peter Anvin CommitDate: Tue, 6 Aug 2013 14:18:30 -0700 x86, asmlinkage: Make 32bit/

[tip:x86/asmlinkage] x86, asmlinkage, paravirt: Add __visible/ asmlinkage to xen paravirt ops

2013-08-06 Thread tip-bot for Andi Kleen
Commit-ID: 9a55fdbe941e5cfb7a5c438db890a58dad82ef8e Gitweb: http://git.kernel.org/tip/9a55fdbe941e5cfb7a5c438db890a58dad82ef8e Author: Andi Kleen AuthorDate: Mon, 5 Aug 2013 15:02:46 -0700 Committer: H. Peter Anvin CommitDate: Tue, 6 Aug 2013 14:20:56 -0700 x86, asmlinkage, paravirt: A

Re: [PATCH] kernel/sysctl_binary.c: improve the usage of return value 'result'

2013-08-06 Thread Andrew Morton
On Tue, 06 Aug 2013 15:29:42 +0800 Chen Gang wrote: > Improve the usage of return value 'result', so not only can make code > clearer to readers, but also can improve the performance. It used to be pervasive kernel style do to ret = -ENOMEM; foo = alloc(...); if (!foo)

[tip:x86/asmlinkage] x86, asmlinkage: Make syscall tables visible

2013-08-06 Thread tip-bot for Andi Kleen
Commit-ID: e0e745e45d3d3c22f60dc9b2c17bcfc0d9f2bc82 Gitweb: http://git.kernel.org/tip/e0e745e45d3d3c22f60dc9b2c17bcfc0d9f2bc82 Author: Andi Kleen AuthorDate: Mon, 5 Aug 2013 15:02:44 -0700 Committer: H. Peter Anvin CommitDate: Tue, 6 Aug 2013 14:20:18 -0700 x86, asmlinkage: Make syscal

[tip:x86/asmlinkage] x86, asmlinkage: Make several variables used from assembler/linker script visible

2013-08-06 Thread tip-bot for Andi Kleen
Commit-ID: 277d5b40b7bf495d2d4193746181b17dd98441b2 Gitweb: http://git.kernel.org/tip/277d5b40b7bf495d2d4193746181b17dd98441b2 Author: Andi Kleen AuthorDate: Mon, 5 Aug 2013 15:02:43 -0700 Committer: H. Peter Anvin CommitDate: Tue, 6 Aug 2013 14:20:13 -0700 x86, asmlinkage: Make severa

[tip:x86/asmlinkage] x86, asmlinkage, apm: Make APM data structure used from assembler visible

2013-08-06 Thread tip-bot for Andi Kleen
Commit-ID: 54c2f3fdb941204cad136024c7b854b7ad112ab6 Gitweb: http://git.kernel.org/tip/54c2f3fdb941204cad136024c7b854b7ad112ab6 Author: Andi Kleen AuthorDate: Mon, 5 Aug 2013 15:02:45 -0700 Committer: H. Peter Anvin CommitDate: Tue, 6 Aug 2013 14:20:20 -0700 x86, asmlinkage, apm: Make A

[PATCH v4 2/4] mmc: dw_mmc: Add exynos resume_noirq callback to clear WAKEUP_INT

2013-08-06 Thread Doug Anderson
If the WAKEUP_INT is asserted at wakeup and not cleared, we'll end up looping around forever. This has been seen to happen on exynos5420 silicon despite the fact that we haven't enabled any wakeup events due to a silicon errata. It is safe to do on all exynos variants. Signed-off-by: Doug Anders

<    1   2   3   4   5   6   7   >