Re: [PATCH v2 1/5] uprobes: don't recheck vma/f_mapping in write_opcode()

2012-07-09 Thread Srikar Dronamraju
* Oleg Nesterov [2012-06-24 17:00:02]: > write_opcode() rechecks valid_vma() and ->f_mapping, this is pointless. > The caller, register_for_each_vma() or uprobe_mmap(), has already done > these checks under mmap_sem. > > To clarify, uprobe_mmap() checks valid_vma() only, but we can rely on >

[RFC PATCH v3 9/13] memory-hotplug : move register_page_bootmem_info_node and put_page_bootmem for sparse-vmemmap

2012-07-09 Thread Yasuaki Ishimatsu
For implementing register_page_bootmem_info_node of sparse-vmemmap, register_page_bootmem_info_node and put_page_bootmem are moved to memory_hotplug.c CC: David Rientjes CC: Jiang Liu CC: Len Brown CC: Benjamin Herrenschmidt CC: Paul Mackerras CC: Christoph Lameter Cc: Minchan Kim CC:

Re: [PATCH v2 3/5] uprobes: kill write_opcode()->lock_page(new_page)

2012-07-09 Thread Srikar Dronamraju
* Oleg Nesterov [2012-06-24 17:00:35]: > write_opcode() does lock_page(new_page) for no reason. Nobody can > see this page until __replace_page() exposes it under ptl lock, and > we do nothing with this page after pte_unmap_unlock(). > > If nothing else, the similar code in do_wp_page() doesn't

Re: [PATCH] sdio: Change pr_warning to pr_warn_ratelimited

2012-07-09 Thread Chris Ball
Hi, On Thu, Jun 14 2012, Liu, Chuansheng wrote: > From: liu chuansheng > Subject: [PATCH] sdio: Change pr_warning to pr_warn_ratelimited > > When debug one bad issue, got lots of pr_warning messages > "queuing unknown CIS tuple" which caused the printk storm > and flooded the console. > > This

[RFC PATCH v3 8/13] memory-hotplug : check page type in get_page_bootmem

2012-07-09 Thread Yasuaki Ishimatsu
There is a possibility that get_page_bootmem() is called to the same page many times. So when get_page_bootmem is called to the same page, the function only increments page->_count. CC: David Rientjes CC: Jiang Liu CC: Len Brown CC: Benjamin Herrenschmidt CC: Paul Mackerras CC: Christoph

Re: [PATCH v2 4/5] uprobes: cleanup and document write_opcode()->lock_page(old_page)

2012-07-09 Thread Srikar Dronamraju
* Oleg Nesterov [2012-06-24 17:00:53]: > The comment above write_opcode()->lock_page(old_page) tells about > the race with do_wp_page(). I don't really understand which exactly > race it means, but afaics this lock_page() was not enough to close > all races with do_wp_page(). > > Anyway, since

Re: [PATCH v2 2/5] uprobes: __replace_page() should not use page_address_in_vma()

2012-07-09 Thread Srikar Dronamraju
> -- > Subject: [PATCH v2 2/5] uprobes: __replace_page() should not use > page_address_in_vma() > > page_address_in_vma(old_page) in __replace_page() is ugly and wrong. > The caller already knows the correct virtual

[RFC PATCH v3 7/13] memory-hotplug : remove_memory calls __remove_pages

2012-07-09 Thread Yasuaki Ishimatsu
The patch adds __remove_pages() to remove_memory(). Then the range of phys_start_pfn argument and nr_pages argument in __remove_pagse() may have different zone. So zone argument is removed from __remove_pages() and __remove_pages() caluculates zone in each section. When CONFIG_SPARSEMEM_VMEMMAP

Re: [PATCH v2 0/5] uprobes: write_opcode() cleanups

2012-07-09 Thread Srikar Dronamraju
* Oleg Nesterov [2012-07-06 18:07:04]: > On 07/06, Ingo Molnar wrote: > > > > * Oleg Nesterov wrote: > > > > > Hello, > > > > > > write_opcode() cleanups resend + new minor fix. > > > > > > Changes: > > > > > > - document the new argument in 2/5. > > > > > > - drop the buggy 5/5, thanks

[RFC PATCH v3 6/13] memory-hotplug : add memory_block_release

2012-07-09 Thread Yasuaki Ishimatsu
When calling remove_memory_block(), the function shows following message at device_release(). Device 'memory528' does not have a release() function, it is broken and must be fixed. remove_memory_block() calls kfree(mem). I think it shouled be called from device_release(). So the patch implements

[RFC PATCH v3 5/13] memory-hotplug : does not release memory region in PAGES_PER_SECTION chunks

2012-07-09 Thread Yasuaki Ishimatsu
Since applying a patch(de7f0cba96786c), release_mem_region() has been changed as called in PAGES_PER_SECTION chunks because register_memory_resource() is called in PAGES_PER_SECTION chunks by add_memory(). But it seems firmware dependency. If CRS are written in the PAGES_PER_SECTION chunks in ACPI

Re: [PATCH 2/5] uprobes: suppress uprobe_munmap() from mmput()

2012-07-09 Thread Srikar Dronamraju
* Oleg Nesterov [2012-07-09 12:09:20]: > On 07/09, Peter Zijlstra wrote: > > > > On Sun, 2012-07-08 at 22:30 +0200, Oleg Nesterov wrote: > > > uprobe_munmap() does get_user_pages() and it is also called from > > > the final mmput()->exit_mmap() path. This slows down exit/mmput() > > > for no

[RFC PATCH v3 4/13] memory-hotplug : remove /sys/firmware/memmap/X sysfs

2012-07-09 Thread Yasuaki Ishimatsu
When (hot)adding memory into system, /sys/firmware/memmap/X/{end, start, type} sysfs files are created. But there is no code to remove these files. The patch implements the function to remove them. Note : The code does not free firmware_map_entry since there is no way to free memory which

[RFC PATCH v3 3/13] memory-hotplug : unify argument of firmware_map_add_early/hotplug

2012-07-09 Thread Yasuaki Ishimatsu
There are two ways to create /sys/firmware/memmap/X sysfs: - firmware_map_add_early When the system starts, it is calledd from e820_reserve_resources() - firmware_map_add_hotplug When the memory is hot plugged, it is called from add_memory() But these functions are called without

[RFC PATCH v3 2/13] memory-hotplug : add physical memory hotplug code to acpi_memory_device_remove

2012-07-09 Thread Yasuaki Ishimatsu
acpi_memory_device_remove() has been prepared to remove physical memory. But, the function only frees acpi_memory_device currentlry. The patch adds following functions into acpi_memory_device_remove(): - offline memory - remove physical memory (only return -EBUSY) - free acpi_memory_device

[PATCH] Trivial: Fix typo on Documentation/dynamic-debug-howto.txt

2012-07-09 Thread dl9pf
From: Jan-Simon Möller Signed-off-by: Jan-Simon Möller Cc: Rob Landley Cc: linux-...@vger.kernel.org Cc: linux-kernel@vger.kernel.org --- Documentation/dynamic-debug-howto.txt |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/Documentation/dynamic-debug-howto.txt

Re: [PATCH net-next 1/2] r8169: support RTL8106E

2012-07-09 Thread Francois Romieu
David Miller : > Francois, what would you like me to do with these two patches? I > haven't seen full ACKs from you yet. You have two options: - You can apply the first (8106) patch. I tested it. I'm ok with Hayes point. Consider it Acked. - You wait for a pull request with both patches + the

[RFC PATCH v3 1/13] memory-hotplug : rename remove_memory to offline_memory

2012-07-09 Thread Yasuaki Ishimatsu
remove_memory() does not remove memory but just offlines memory. The patch changes name of it to offline_memory(). CC: David Rientjes CC: Jiang Liu CC: Len Brown CC: Benjamin Herrenschmidt CC: Paul Mackerras CC: Christoph Lameter Cc: Minchan Kim CC: Andrew Morton CC: KOSAKI Motohiro CC:

[RFC PATCH v3 0/13] memory-hotplug : hot-remove physical memory

2012-07-09 Thread Yasuaki Ishimatsu
This patch series aims to support physical memory hot-remove. [RFC PATCH v3 1/13] memory-hotplug : rename remove_memory to offline_memory [RFC PATCH v3 2/13] memory-hotplug : add physical memory hotplug code to acpi_memory_device_remove [RFC PATCH v3 3/13] memory-hotplug : unify argument

Re: [PATCH 2/5] ubi: Limit amount of reserved eraseblocks for bad PEB handling

2012-07-09 Thread Richard Genoud
2012/7/4 Shmulik Ladkani : > diff --git a/drivers/mtd/ubi/misc.c b/drivers/mtd/ubi/misc.c > index f6a7d7a..e9dcb83 100644 > --- a/drivers/mtd/ubi/misc.c > +++ b/drivers/mtd/ubi/misc.c > @@ -98,10 +98,18 @@ int ubi_check_volume(struct ubi_device *ubi, int vol_id) > */ > void

Re: [PATCH 2/5] uprobes: suppress uprobe_munmap() from mmput()

2012-07-09 Thread Peter Zijlstra
On Mon, 2012-07-09 at 12:09 +0200, Oleg Nesterov wrote: > No, mmap/munmap do not participate in uprobe refcounting. This code > does put_uprobe() for each uprobe, yes, but only because the counter > was incremented in build_probe_list(). > > uprobe_munmap() is only needed to decrement

[PATCH] mm/hugetlb: split out is_hugetlb_entry_migration_or_hwpoison

2012-07-09 Thread Wanpeng Li
From: Wanpeng Li Code was duplicated in two functions, clean it up. Signed-off-by: Wanpeng Li --- mm/hugetlb.c | 20 +--- 1 files changed, 9 insertions(+), 11 deletions(-) diff --git a/mm/hugetlb.c b/mm/hugetlb.c index e198831..4f9ce3f 100644 --- a/mm/hugetlb.c +++

Re: [PATCH 2/5] uprobes: suppress uprobe_munmap() from mmput()

2012-07-09 Thread Oleg Nesterov
On 07/09, Peter Zijlstra wrote: > > On Sun, 2012-07-08 at 22:30 +0200, Oleg Nesterov wrote: > > uprobe_munmap() does get_user_pages() and it is also called from > > the final mmput()->exit_mmap() path. This slows down exit/mmput() > > for no reason, and I think it is simply dangerous/wrong to try

[PATCH] hwmon: (applesmc) Shorten minimum wait time

2012-07-09 Thread Henrik Rydberg
The 2012 series of MacBooks have a faster SMC, and the current driver timings do not work at all. Tests show that decreasing the minimum wait time, from 64 us to 16 us, works well. Since this is still larger than the original minimum of 10 us used before 2008, there is nothing inherently

Re: [Resend with Ack][PATCH v1] PCI: allow acpiphp to handle PCIe ports without native PCIe hotplug capability

2012-07-09 Thread Jiang Liu
Hi Bjorn and Yinghai, What's the policy to export a symbol by EXPORT_SYMBOL() or EXPORT_SYMBOL_GPL()? I know the legal difference, but don't know when I should mark a symbol as GPL. Thanks! -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of

Re: [PATCH 04/16] mm: allow PF_MEMALLOC from softirq context

2012-07-09 Thread Mel Gorman
On Sun, Jul 08, 2012 at 08:12:11PM +0200, Sebastian Andrzej Siewior wrote: > On Wed, Jun 27, 2012 at 09:26:14AM +0100, Mel Gorman wrote: > > > > diff --git a/mm/page_alloc.c b/mm/page_alloc.c > > > > index b6c0727..5c6d9c6 100644 > > > > --- a/mm/page_alloc.c > > > > +++ b/mm/page_alloc.c > > > >

Re: [PATCH RESEND v7 1/2] block: ioctl support for sanitize in eMMC 4.5

2012-07-09 Thread Girish K S
On 28 June 2012 14:02, Yaniv Gardi wrote: > Adding a new ioctl to support sanitize operation in eMMC > cards version 4.5. > The sanitize ioctl support helps performing this operation > via user application. > > Signed-off-by: Yaniv Gardi > > --- > block/blk-core.c | 15 ++-- >

Re: [PATCH V3 0/3] i2c-nomadik changes

2012-07-09 Thread Wolfram Sang
On Mon, Jun 11, 2012 at 11:37:15PM +0200, Linus Walleij wrote: > On Mon, Jun 11, 2012 at 10:56 PM, Alessandro Rubini wrote: > > > V3: > >        - fixed according to Linusw feedback (merged the patch he posted) > >        - added Tested-by: Linusw on his permission > > I'm happy with this

[PATCH v2] usb: host: Fix possible kernel crash

2012-07-09 Thread Venu Byravarasu
In functions itd_complete & sitd_complete, a pointer by name stream may get dereferenced after freeing it, when iso_stream_put is called with stream->refcount = 2. Hence fixing it. Signed-off-by: Venu Byravarasu --- In Patchset 1, modified parameter of iso_stream_put() to handle the crash.

[tip:timers/urgent] hrtimer: Update hrtimer base offsets each hrtimer_interrupt

2012-07-09 Thread tip-bot for John Stultz
Commit-ID: d7e2e7fef1f0f4e1e614353a6c5eef4e18d98d2d Gitweb: http://git.kernel.org/tip/d7e2e7fef1f0f4e1e614353a6c5eef4e18d98d2d Author: John Stultz AuthorDate: Thu, 5 Jul 2012 15:12:18 -0400 Committer: Thomas Gleixner CommitDate: Mon, 9 Jul 2012 11:35:38 +0200 hrtimer: Update hrtimer

[tip:timers/urgent] time: Fix leapsecond triggered hrtimer/ futex load spike issue

2012-07-09 Thread tip-bot for John Stultz
Commit-ID: bb88e92477def647976cd3d6964af98beceba900 Gitweb: http://git.kernel.org/tip/bb88e92477def647976cd3d6964af98beceba900 Author: John Stultz AuthorDate: Thu, 5 Jul 2012 15:12:17 -0400 Committer: Thomas Gleixner CommitDate: Mon, 9 Jul 2012 11:35:38 +0200 time: Fix leapsecond

[tip:timers/urgent] hrtimer: Fix clock_was_set so it is safe to call from irq context

2012-07-09 Thread tip-bot for John Stultz
Commit-ID: adf374cf61394dd41c027c74a81f9bef90f7a640 Gitweb: http://git.kernel.org/tip/adf374cf61394dd41c027c74a81f9bef90f7a640 Author: John Stultz AuthorDate: Thu, 5 Jul 2012 15:12:16 -0400 Committer: Thomas Gleixner CommitDate: Mon, 9 Jul 2012 11:35:38 +0200 hrtimer: Fix

Re: [PATCH v2] ext4: fix hole punch failure when depth is greater than 0

2012-07-09 Thread Lukáš Czerner
On Thu, 5 Jul 2012, Ashish Sangwan wrote: > Date: Thu, 5 Jul 2012 15:22:04 +0530 > From: Ashish Sangwan > To: Lukáš Czerner > Cc: sand...@redhat.com, Ted Tso , linux-kernel@vger.kernel.org, > linux-e...@vger.kernel.org, Namjae Jeon > Subject: Re: [PATCH v2] ext4: fix hole punch failure

[PATCH] ext4: fix double quota opts show

2012-07-09 Thread Dmitry Monakhov
Currently quota options showed twice, first time in _ext4_show_options() (inside main loop), and second time inside ext4_show_quota_options(). In my case it looks like follows: /dev/sdc /mnt ext4 rw,relatime,quota,usrquota,grpquota,data=ordered,usrquota,grpquota 0 0 Let's do not show quota's opts

[PATCH v4 5/5] sched: cpu_power: enable ARCH_POWER

2012-07-09 Thread Vincent Guittot
Heteregeneous ARM platform uses arch_scale_freq_power function to reflect the relative capacity of each core Signed-off-by: Vincent Guittot --- kernel/sched/features.h |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/kernel/sched/features.h b/kernel/sched/features.h index

[PATCH v4 4/5] sched, x86: Remove broken power estimation

2012-07-09 Thread Vincent Guittot
From: Peter Zijlstra The x86 sched power implementation has been broken forever and gets in the way of other stuff, remove it. For archaeological interest, fixing this code would require dealing with the cross-cpu calling of these functions and more importantly, we need to filter idle time out

[PATCH v4 3/5] ARM: topology: Update cpu_power according to DT information

2012-07-09 Thread Vincent Guittot
Use cpu compatibility field and clock-frequency field of DT to estimate the capacity of each core of the system and to update the cpu_power field accordingly. This patch enables to put more running tasks on big cores than on LITTLE ones. But this patch doesn't ensure that long running tasks will

[PATCH v4 2/5] ARM: topology: factorize the update of sibling masks

2012-07-09 Thread Vincent Guittot
This factorization has also been proposed in another patch that has not been merged yet: http://lists.infradead.org/pipermail/linux-arm-kernel/2012-January/080873.html So, this patch could be dropped depending of the state of the other one. Signed-off-by: Lorenzo Pieralisi Signed-off-by: Vincent

[PATCH v4 1/5] ARM: topology: Add arch_scale_freq_power function

2012-07-09 Thread Vincent Guittot
Add infrastructure to be able to modify the cpu_power of each core Signed-off-by: Vincent Guittot Reviewed-by: Namhyung Kim --- arch/arm/kernel/topology.c | 38 +- 1 file changed, 37 insertions(+), 1 deletion(-) diff --git a/arch/arm/kernel/topology.c

[PATCH v4 0/5] ARM: topology: set the capacity of each cores for big.LITTLE

2012-07-09 Thread Vincent Guittot
This patchset creates an arch_scale_freq_power function for ARM, which is used to set the relative capacity of each core of a big.LITTLE system. It also removes the broken power estimation of x86. Modification since v3: - Add comments - Add optimization for SMP system - Ensure that capacity

Re: [PATCH] apic: fix kvm build on UP without IOAPIC

2012-07-09 Thread Ingo Molnar
* Michael S. Tsirkin wrote: > On Fri, Jul 06, 2012 at 04:12:23PM +0200, Ingo Molnar wrote: > > > > * Marcelo Tosatti wrote: > > > > > On Fri, Jul 06, 2012 at 01:13:14PM +0200, Ingo Molnar wrote: > > > > > > > > * H. Peter Anvin wrote: > > > > > > > > > On 07/01/2012 08:05 AM, Michael S.

Re: 82571EB: Detected Hardware Unit Hang

2012-07-09 Thread Eric Dumazet
On Mon, 2012-07-09 at 16:51 +0800, Joe Jin wrote: > Hi list, > > I'm seeing a Unit Hang even with the latest e1000e driver 2.0.0 when doing > scp test. this issue is easy do reproduced on SUN FIRE X2270 M2, just copy > a big file (>500M) from another server will hit it at once. > > Would you

Re: [PATCH 0/6] staging: vt6655: Cleanup in usage of macros

2012-07-09 Thread Marcos Souza
2012/7/9 Joe Perches > > On Sun, 2012-07-08 at 23:51 -0300, Marcos Paulo de Souza wrote: > > Hi kernel guys! > > > > This patchset aims to clean all unused and commented macros. > > > > For this challenge, forgotten-macros tool helped us. > > Perhaps there may be false positives in your code.

Re: linux-next: comment on pm tree commit

2012-07-09 Thread preeti
On 07/09/2012 01:54 PM, Rafael J. Wysocki wrote: > On Monday, July 09, 2012, Stephen Rothwell wrote: >> Hi Rafael, >> >> I noticed commit b8eec56cd8e5 ("PM / cpuidle: System resume hang fix with >> cpuidle") in the pm tree needs some work (I noticed it because it was >> changed in a rebase ...).

Re: [PATCH 03/16] sched: aggregate load contributed by task entities on parenting cfs_rq

2012-07-09 Thread Ingo Molnar
* Peter Zijlstra wrote: > On Wed, 2012-07-04 at 17:28 +0200, Peter Zijlstra wrote: > > On Wed, 2012-06-27 at 19:24 -0700, Paul Turner wrote: > > > For a given task t, we can compute its contribution to load as: > > > task_load(t) = runnable_avg(t) * weight(t) > > > > > > On a parenting

Re: [PATCH v2 0/5] uprobes: write_opcode() cleanups

2012-07-09 Thread Ingo Molnar
* Oleg Nesterov wrote: > On 07/06, Oleg Nesterov wrote: > > > > On 07/06, Ingo Molnar wrote: > > > > > > * Oleg Nesterov wrote: > > > > > > > Hello, > > > > > > > > write_opcode() cleanups resend + new minor fix. > > > > > > > > Changes: > > > > > > > > - document the new argument in

Re: [PATCH] mm: Warn about costly page allocation

2012-07-09 Thread Mel Gorman
On Mon, Jul 09, 2012 at 05:46:57PM +0900, Minchan Kim wrote: > > > > > > +#if defined(CONFIG_DEBUG_VM) && !defined(CONFIG_COMPACTION) > > > +static inline void check_page_alloc_costly_order(unsigned int order) > > > +{ > > > + if (unlikely(order > PAGE_ALLOC_COSTLY_ORDER)) { > > > +

Re: WARNING: __GFP_FS allocations with IRQs disabled (kmemcheck_alloc_shadow)

2012-07-09 Thread Pekka Enberg
On Sun, 8 Jul 2012, David Rientjes wrote: > The correct fix is what I proposed at > http://marc.info/?l=linux-kernel=133754837703630 and was awaiting > testing. If Rus, Steven, or Fengguang could test this then we could add > it as a stable backport as well. Looks good to me. Care to send it

Re: [PATCH 3/7] mm/slub.c: remove invalid reference to list iterator variable

2012-07-09 Thread Pekka Enberg
On Sun, 8 Jul 2012, Julia Lawall wrote: > From: Julia Lawall > > If list_for_each_entry, etc complete a traversal of the list, the iterator > variable ends up pointing to an address at an offset from the list head, > and not a meaningful structure. Thus this value should not be used after >

Re: [PATCH RESEND v7 1/2] block: ioctl support for sanitize in eMMC 4.5

2012-07-09 Thread Girish K S
On 28 June 2012 14:02, Yaniv Gardi wrote: > Adding a new ioctl to support sanitize operation in eMMC > cards version 4.5. > The sanitize ioctl support helps performing this operation > via user application. > > Signed-off-by: Yaniv Gardi > > --- > block/blk-core.c | 15 ++-- >

[PATCH 2/2] KVM: X86: introduce set_mmio_exit_info

2012-07-09 Thread Xiao Guangrong
Introduce set_mmio_exit_info to cleanup the common code Signed-off-by: Xiao Guangrong --- arch/x86/kvm/x86.c | 33 + 1 files changed, 17 insertions(+), 16 deletions(-) diff --git a/arch/x86/kvm/x86.c b/arch/x86/kvm/x86.c index 7445545..7771f45 100644 ---

[PATCH 1/2] KVM: X86: remove read buffer for mmio read

2012-07-09 Thread Xiao Guangrong
After commit f78146b0f9230765c6315b2e14f56112513389ad: KVM: Fix page-crossing MMIO MMIO that are split across a page boundary are currently broken - the code does not expect to be aborted by the exit to userspace for the first MMIO fragment. This patch fixes the problem by

Re: [PATCHv2 3/3] watchdog: omap_wdt: add device tree support

2012-07-09 Thread Tony Lindgren
* Wim Van Sebroeck [120707 05:11]: > Hi Tony, > > > Hi Wim, > > > > * jgq...@gmail.com [120531 20:56]: > > > From: Xiao Jiang > > > > > > Add device table for omap_wdt to support dt. > > > > Care to ack this patch in the series? > > Yep. > Acked-by: Wim Van Sebroeck Thanks, I'll apply

Re: [PATCH 00/36] AArch64 Linux kernel port

2012-07-09 Thread Catalin Marinas
Hi Jon, On 9 July 2012 03:01, Jon Masters wrote: > On 07/08/2012 06:24 PM, Dennis Gilmore wrote: >> I know that the architecture really is new but thats not really clear >> by adding AArch32 into the mix to represent 32 bit arm as ARM has done >> or by calling it armv8. There is enough way to

82571EB: Detected Hardware Unit Hang

2012-07-09 Thread Joe Jin
Hi list, I'm seeing a Unit Hang even with the latest e1000e driver 2.0.0 when doing scp test. this issue is easy do reproduced on SUN FIRE X2270 M2, just copy a big file (>500M) from another server will hit it at once. Would you please help on this? device info: # lspci -s 05:00.0 05:00.0

Re: [PATCH] mm: Warn about costly page allocation

2012-07-09 Thread Minchan Kim
Hi Mel, On Mon, Jul 09, 2012 at 09:22:00AM +0100, Mel Gorman wrote: > On Mon, Jul 09, 2012 at 11:38:20AM +0900, Minchan Kim wrote: > > Since lumpy reclaim was introduced at 2.6.23, it helped higher > > order allocation. > > Recently, we removed it at 3.4 and we didn't enable compaction > >

Re: linux-next: comment on pm tree commit

2012-07-09 Thread Rafael J. Wysocki
On Monday, July 09, 2012, Stephen Rothwell wrote: > Hi Rafael, > > I noticed commit b8eec56cd8e5 ("PM / cpuidle: System resume hang fix with > cpuidle") in the pm tree needs some work (I noticed it because it was > changed in a rebase ...). > > diff --git a/include/linux/cpuidle.h

Re: [PATCH RESEND v7 1/2] block: ioctl support for sanitize in eMMC 4.5

2012-07-09 Thread Girish K S
On 28 June 2012 14:02, Yaniv Gardi wrote: > Adding a new ioctl to support sanitize operation in eMMC > cards version 4.5. > The sanitize ioctl support helps performing this operation > via user application. > > Signed-off-by: Yaniv Gardi > > --- > block/blk-core.c | 15 ++-- >

Re: [PATCH 4/5] uprobes: kill copy_vma()->uprobe_mmap()

2012-07-09 Thread Peter Zijlstra
On Sun, 2012-07-08 at 22:30 +0200, Oleg Nesterov wrote: > And why this uprobe_mmap() was added? I believe the intent was wrong. > Note that the caller is going to do move_page_tables(), all registered > uprobes are already faulted in, we only change the virtual addresses. > I think it was

Re: Infinite looping in omap2430.c USB driver

2012-07-09 Thread Tony Lindgren
* NeilBrown [120706 15:44]: > > Hello `./scripts/get_maintainer.pl -f drivers/usb/musb/omap2430.c` > > omap2430_musb_set_vbus in omap2430.c contains: > > while (musb_readb(musb->mregs, MUSB_DEVCTL) & 0x80) { > > cpu_relax(); > >

Re: [PATCH 2/5] uprobes: suppress uprobe_munmap() from mmput()

2012-07-09 Thread Peter Zijlstra
On Sun, 2012-07-08 at 22:30 +0200, Oleg Nesterov wrote: > uprobe_munmap() does get_user_pages() and it is also called from > the final mmput()->exit_mmap() path. This slows down exit/mmput() > for no reason, and I think it is simply dangerous/wrong to try to > fault-in a page into the dying mm.

Re: [PATCH] mm: Warn about costly page allocation

2012-07-09 Thread Mel Gorman
On Mon, Jul 09, 2012 at 11:38:20AM +0900, Minchan Kim wrote: > Since lumpy reclaim was introduced at 2.6.23, it helped higher > order allocation. > Recently, we removed it at 3.4 and we didn't enable compaction > forcingly[1]. The reason makes sense that compaction.o + migration.o > isn't trivial

Re: UBI fastmap updates

2012-07-09 Thread Richard Weinberger
Am 09.07.2012 09:37, schrieb Shmulik Ladkani: > Hi Richard, > > On Sun, 08 Jul 2012 14:07:41 +0200 Richard Weinberger wrote: >>> + /* TODO: in the new locking scheme, produce_free_peb is >>> +* called under wl_lock taken. >>> +* so when

Re: [RFC PATCH v2 4/13] memory-hotplug : remove /sys/firmware/memmap/X sysfs

2012-07-09 Thread Yasuaki Ishimatsu
Hi Wen, 2012/07/06 18:20, Wen Congyang wrote: > At 07/06/2012 04:27 PM, Yasuaki Ishimatsu Wrote: >> Hi Wen, >> >> 2012/07/04 19:01, Wen Congyang wrote: >>> At 07/04/2012 01:52 PM, Yasuaki Ishimatsu Wrote: Hi Wen, 2012/07/04 14:08, Wen Congyang wrote: > At 07/04/2012 12:45 PM,

Re: [PATCH] net: cgroup: fix out of bounds accesses

2012-07-09 Thread Gao feng
于 2012年07月09日 15:45, Eric Dumazet 写道: > From: Eric Dumazet > > dev->priomap is allocated by extend_netdev_table() called from > update_netdev_tables(). > And this is only called if write_priomap() is called. > > But if write_priomap() is not called, it seems we can have out of bounds > accesses

Re: Antw: Re: /sys and access(2): Correctly implemented?

2012-07-09 Thread Ulrich Windl
Hi! Still the problem seems to be related to the sysfs: # cd /tmp # touch testfile # chmod u=w,go= testfile # F=/tmp/testfile # test -r "$F" && cat "$F" So it seems access(2) works correctly for root and "normal" filesystems. That's why I came up with the issue here. Regards, Ulrich >>> Ryan

Re: [PATCH] CONFIG_CC_STACKPROTECTOR is no longer experimental

2012-07-09 Thread Jean Delvare
Hi all, Le vendredi 06 juillet 2012 à 22:19 +0200, Paul Bolle a écrit : > On Fri, 2012-07-06 at 10:58 -0700, Arjan van de Ven wrote: > > I rather just retire the whole concept of "Experimental". > > > > it's really utterly meaningless in practice anyway. > > See Russell King's quick survey in

Re: [PATCH RFC 0/2] kvm: Improving directed yield in PLE handler

2012-07-09 Thread Christian Borntraeger
On 09/07/12 08:20, Raghavendra K T wrote: > Currently Pause Looop Exit (PLE) handler is doing directed yield to a > random VCPU on PL exit. Though we already have filtering while choosing > the candidate to yield_to, we can do better. > > Problem is, for large vcpu guests, we have more

Re: Fwd: Hid over I2C and ACPI interaction

2012-07-09 Thread Mika Westerberg
On Mon, Jul 09, 2012 at 11:24:45AM +0800, Lan Tianyu wrote: > I think we can add new interface to get acpi specific resources. e.g > struct acpi_resource pnp_get_acpi_resource(...). When the pnp acpi devices > were initialized, put those acpi specific resources into a new resource list >

Re: [RFC][PATCH V2 2/3] pwm_backlight: use power sequences

2012-07-09 Thread Alex Courbot
Sorry, I just noticed a mistake in this patch I made while merging another one. The following also needs to be changed, otherwise the power-on sequence will never be executed: diff --git a/drivers/video/backlight/pwm_bl.c b/drivers/video/backlight/pwm_bl.c index 1a38953..4546d23 100644 ---

[PATCH] net: cgroup: fix out of bounds accesses

2012-07-09 Thread Eric Dumazet
From: Eric Dumazet dev->priomap is allocated by extend_netdev_table() called from update_netdev_tables(). And this is only called if write_priomap() is called. But if write_priomap() is not called, it seems we can have out of bounds accesses in cgrp_destroy(), read_priomap() & skb_update_prio()

Re: [PATCH 1/2] hw_random: mxc-rnga: Adapt clocks to new i.mx clock framework

2012-07-09 Thread Sascha Hauer
On Fri, Jul 06, 2012 at 05:20:19PM -0300, Fabio Estevam wrote: > Cc: Theodore Ts'o > Cc: Herbert Xu > Cc: > Signed-off-by: Fabio Estevam > --- > drivers/char/hw_random/mxc-rnga.c |8 > 1 files changed, 4 insertions(+), 4 deletions(-) > > diff --git

Re: UBI fastmap updates

2012-07-09 Thread Shmulik Ladkani
Hi Richard, On Sun, 08 Jul 2012 14:07:41 +0200 Richard Weinberger wrote: > > + /* TODO: in the new locking scheme, produce_free_peb is > > +* called under wl_lock taken. > > +* so when returning, should reacquire the lock > > +

Re: Fwd: Hid over I2C and ACPI interaction

2012-07-09 Thread Lan Tianyu
On 2012年07月09日 12:02, Moore, Robert wrote: > These are already defined in acpica - in the file acrestyp.h > > ACPI_RESOURCE_FIXED_DMA FixedDma; > > ACPI_RESOURCE_GPIO Gpio; > ACPI_RESOURCE_I2C_SERIALBUS I2cSerialBus; >

Re: [patch 04/11] mm: memcg: push down PageSwapCache check into uncharge entry functions

2012-07-09 Thread Johannes Weiner
On Mon, Jul 09, 2012 at 11:42:12AM +0900, Kamezawa Hiroyuki wrote: > (2012/07/05 9:44), Johannes Weiner wrote: > > @@ -3278,10 +3283,11 @@ void mem_cgroup_end_migration(struct mem_cgroup > > *memcg, > > unused = oldpage; > > } > > anon = PageAnon(used); > > -

Re: [PATCH v2 04/11] fbdev: export get_fb_info()/put_fb_info()

2012-07-09 Thread Geert Uytterhoeven
On Sun, Jul 8, 2012 at 11:56 PM, David Herrmann wrote: > --- a/drivers/video/fbmem.c > +++ b/drivers/video/fbmem.c > @@ -46,7 +46,7 @@ static DEFINE_MUTEX(registration_lock); > struct fb_info *registered_fb[FB_MAX] __read_mostly; > int num_registered_fb __read_mostly; > > -static struct fb_info

Re: [PATCH v2 11/11] MAINTAINERS: add fblog entry

2012-07-09 Thread Geert Uytterhoeven
On Sun, Jul 8, 2012 at 11:56 PM, David Herrmann wrote: > Add myself as maintainer for the fblog driver to the MAINTAINERS file. > > Signed-off-by: David Herrmann > --- > MAINTAINERS | 6 ++ > 1 file changed, 6 insertions(+) > > diff --git a/MAINTAINERS b/MAINTAINERS > index ae8fe46..249b02a

Re: pull request: wireless 2012-07-06

2012-07-09 Thread David Miller
From: "John W. Linville" Date: Fri, 6 Jul 2012 15:20:35 -0400 > Please let me know if there are problems! This indentation is not correct: commit 01f9cb073c827c60c43f769763b49a2026f1a897 Author: Thomas Huehn Date: Thu Jun 28 14:39:51 2012 -0700 mwl8k: fix possible race condition in

[PATCH] fs/ubifs/orphan.c: remove invalid reference to list iterator variable

2012-07-09 Thread Julia Lawall
From: Julia Lawall If list_for_each_entry, etc complete a traversal of the list, the iterator variable ends up pointing to an address at an offset from the list head, and not a meaningful structure. Thus this value should not be used after the end of the iterator. Replace a field access from

Re: Antw: Re: /sys and access(2): Correctly implemented?

2012-07-09 Thread Ryan Mallon
On 09/07/12 16:23, Ulrich Windl wrote: Ryan Mallon schrieb am 09.07.2012 um 01:24 in Nachricht > <4ffa16b6.9050...@gmail.com>: >> On 06/07/12 16:27, Ulrich Windl wrote: >>> Hi! >>> >>> Recently I found a problem with the command (kernel 3.0.34-0.7-default from >> SLES 11 SP2, run as root):

Re: [PATCH v2] cgroup: fix panic in netprio_cgroup

2012-07-09 Thread David Miller
From: Gao feng Date: Thu, 5 Jul 2012 17:28:40 +0800 > we set max_prioidx to the first zero bit index of prioidx_map in > function get_prioidx. > > So when we delete the low index netprio cgroup and adding a new > netprio cgroup again,the max_prioidx will be set to the low index. > > when we

Re: [PATCH] netdev/phy: Fixup lockdep warnings in mdio-mux.c

2012-07-09 Thread David Miller
From: David Daney Date: Wed, 4 Jul 2012 15:06:16 -0700 > From: David Daney > > With lockdep enabled we get: ... > This is a false positive, since we are indeed using 'nested' locking, > we need to use mutex_lock_nested(). > > Now in theory we can stack multiple MDIO multiplexers, but that

Re: [Question] sched/rt_mutex: re-enqueue_task on rt_mutex_setprio()

2012-07-09 Thread Namhyung Kim
On Mon, Jul 9, 2012 at 3:48 PM, Peter Zijlstra wrote: > On Mon, 2012-07-09 at 09:50 +0900, Namhyung Kim wrote: >> On Sat, 07 Jul 2012 21:29:19 -0400, Steven Rostedt wrote: >> > On Sat, 2012-07-07 at 14:44 +0900, Namhyung Kim wrote: >> >> Hi, >> >> >> >> I have a question on the code below: >> >>

[PATCH RESEND] Fix a dead loop in async_synchronize_full()

2012-07-09 Thread Li Zhong
This patch tries to fix a dead loop in async_synchronize_full(), which could be seen when preemption is disabled on a single cpu machine. void async_synchronize_full(void) { do { async_synchronize_cookie(next_cookie); } while (!list_empty(_running) || !

Re: [PATCH 2/2] x86, boot: Optimize the elf header handling.

2012-07-09 Thread Eric W. Biederman
Tejun Heo writes: > Hello, guys. > > On Sun, Jul 01, 2012 at 11:37:22AM -0700, H. Peter Anvin wrote: >> If we don't need it, I think we can use -z max-page-size=4096, but we >> use the PMD alignment for percpu on x86-64; Tejun, does that apply to >> the .data..percpu section in the executable as

Re: [PATCH 1/1] atl1c: fix issue of transmit queue 0 timed out

2012-07-09 Thread David Miller
From: Date: Wed, 4 Jul 2012 10:51:48 +0800 > some people report atl1c could cause system hang with following > kernel trace info: > --- > WARNING: at.../net/sched/sch_generic.c:258 dev_watchdog+0x1db/0x1d0() > ... > NETDEV WATCHDOG: eth0 (atl1c): transmit

Re: [PATCH 1/4] x86 boot: Jump to the entry point address in the elf header.

2012-07-09 Thread Eric W. Biederman
And Peter no rush on these. I have just finished testing and I am pushing the changes out before I forget them. Moving the Elf loader earlier to compile time makes the code a lot more robust. Eric -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a

Re: [PATCH 0/5] ubi: Fix bad PEBs reserve caclulation

2012-07-09 Thread Richard Genoud
2012/7/7 Shmulik Ladkani : > Many thanks for testing. > > Could you please verify the crash only occurs with the patch? > > Can you provide the vmlinux matching this oops, so I may analyze the > exact null dereferencing point? > It seems to be somewhere in ubi_wl_init, however the patch seems not

[PATCH 4/4] x86 boot: Tell ld the kernel doesn't want 2MB file offset alignment.

2012-07-09 Thread Eric W. Biederman
By default ld uses 2MB pages and aligns our 3 program segments in the file on 2MB boundaries, creating unnecessarily large uncompressed vmlinux files. Solve this by passing -z max-page-size 4096 to ld. In my test x86_64 SMP test configuration with CONFIG_DEBUG_RODATA enabled, this reduces the

[PATCH 3/4] x86 boot: When building vmlinux.bin properly precompute the memory image

2012-07-09 Thread Eric W. Biederman
The ELF loader in arch/x86/boot/compressed/misc.c is extremely fragile, as it copies the ELF executable over itself to put the code and data in their proper place. Squeezing unneeded space out of vmlinux by passing -z max-page-size 4096 to ld was enough to render the kernel unbootable. I

linux-next: Tree for July 9

2012-07-09 Thread Stephen Rothwell
Hi all, Changes since 20120706: I have not done the powerpc allyesconfig build today as it is too broken. Undropped tree: gpio-lw The jdelvare-hwmon tree lost its conflict. The v4l-dvb tree gained a build failure so I used the version from next-20120706. The infiniband tree lost its build

[PATCH 2/4] x86 boot: Optimize the elf header handling.

2012-07-09 Thread Eric W. Biederman
Create a space for the elf headers at the begginng of the kernels image in memory. - Rework arch/x86/kernel/vmlinux.lds.S so that we allow room for the ELF header in the loaded image. This removes the need in the ELF executalbe to insert padding between the ELf headers and the data of the

[PATCH 1/4] x86 boot: Jump to the entry point address in the elf header.

2012-07-09 Thread Eric W. Biederman
Since we have the kernel's entry point stored in the ELF header use it, and stop hardcoding the value. Signed-off-by: "Eric W. Biederman" --- arch/x86/boot/compressed/head_32.S |2 +- arch/x86/boot/compressed/head_64.S |2 +- arch/x86/boot/compressed/misc.c| 16 +---

Re: [Question] sched/rt_mutex: re-enqueue_task on rt_mutex_setprio()

2012-07-09 Thread Peter Zijlstra
On Mon, 2012-07-09 at 09:50 +0900, Namhyung Kim wrote: > On Sat, 07 Jul 2012 21:29:19 -0400, Steven Rostedt wrote: > > On Sat, 2012-07-07 at 14:44 +0900, Namhyung Kim wrote: > >> Hi, > >> > >> I have a question on the code below: > >> > >> void rt_mutex_setprio(struct task_struct *p, int prio) >

Re: [PATCH 2/2] x86, boot: Optimize the elf header handling.

2012-07-09 Thread Eric W. Biederman
"H. Peter Anvin" writes: > On 07/01/2012 01:40 PM, Eric W. Biederman wrote: >> >> So I have tracked down part of the crazyness. >> CONFIG_RODATA actually uses 2MB alignment, making >> -z max_page_size=4096 a bit questionable. >> > > Questionable how? It's not really like it matters since

Re: [PATCH v3] ieee802154: verify packet size before trying to allocate it

2012-07-09 Thread David Miller
From: Sasha Levin Date: Mon, 2 Jul 2012 13:29:55 +0200 > Currently when sending data over datagram, the send function will attempt to > allocate any size passed on from the userspace. > > We should make sure that this size is checked and limited. We'll limit it > to the MTU of the device,

Re: [PATCH net-next 1/2] r8169: support RTL8106E

2012-07-09 Thread David Miller
Francois, what would you like me to do with these two patches? I haven't seen full ACKs from you yet. Thanks. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at

Re: [PATCH] fat: Support fallocate on fat.

2012-07-09 Thread Namjae Jeon
Hi. Ogawa. 2012/7/8, OGAWA Hirofumi : > Namjae Jeon writes: > >> +/* >> + * preallocate space for a file. This implements fat's fallocate file >> + * operation, which gets called from sys_fallocate system call. User >> + * space requests len bytes at offset.If FALLOC_FL_KEEP_SIZE is set >> + * we

Re: [PATCH RFC 1/2] kvm vcpu: Note down pause loop exit

2012-07-09 Thread Raghavendra K T
On 07/09/2012 11:50 AM, Raghavendra K T wrote: Signed-off-by: Raghavendra K T Noting pause loop exited vcpu helps in filtering right candidate to yield. Yielding to same vcpu may result in more wastage of cpu. From: Raghavendra K T --- Oops. Sorry some how sign-off and from interchanged..

Re: [PATCHv3] pwm_backlight: pass correct brightness to callback

2012-07-09 Thread Thierry Reding
On Mon, Jul 09, 2012 at 03:04:23PM +0900, Alexandre Courbot wrote: > pwm_backlight_update_status calls the notify() and notify_after() > callbacks before and after applying the new PWM settings. However, if > brightness levels are used, the brightness value will be changed from > the index into

<    2   3   4   5   6   7   8   9   10   11   >