Re: [PATCH v2 09/20] mm, hugetlb: protect region tracking via newly introduced resv_map lock

2013-08-22 Thread Joonsoo Kim
On Wed, Aug 21, 2013 at 03:43:27PM +0530, Aneesh Kumar K.V wrote: > > static long region_chg(struct resv_map *resv, long f, long t) > > { > > struct list_head *head = &resv->regions; > > - struct file_region *rg, *nrg; > > + struct file_region *rg, *nrg = NULL; > > long chg = 0; > >

Re: Regression: x86/mm: new _PTE_SWP_SOFT_DIRTY bit conflicts with existing use

2013-08-22 Thread Cyrill Gorcunov
On Thu, Aug 22, 2013 at 07:56:26AM +0100, Jan Beulich wrote: > >>> On 21.08.13 at 18:19, Cyrill Gorcunov wrote: > > On Wed, Aug 21, 2013 at 05:03:13PM +0100, Jan Beulich wrote: > >> > > >> > Only to non-present ptes, as far as I know. > >> > >> That's not really any guarantee. And the accessor f

[GIT PULL] ARM: SoC fixes for 3.11-rc

2013-08-22 Thread Olof Johansson
Hi Linus, The following changes since commit 1154f858ab5cd3c5facde7cb9ac31c889eb780a8: MAINTAINERS: add TI Keystone ARM platform (2013-08-07 10:30:29 -0700) are available in the git repository at: git://git.kernel.org/pub/scm/linux/kernel/git/arm/arm-soc.git tags/fixes-for-linus for you t

Re: [PATCH 1/3 v3] pinctrl: ADI PIN control driver for the GPIO controller on bf54x and bf60x.

2013-08-22 Thread Sonic Zhang
Hi Stephen, On Thu, Aug 22, 2013 at 2:45 AM, Stephen Warren wrote: > On 08/21/2013 12:30 AM, Sonic Zhang wrote: >> From: Sonic Zhang >> >> The new ADI GPIO2 controller was introduced since the BF548 and BF60x >> processors. It differs a lot from the old one on BF5xx processors. So, >> create a p

Re: [PATCH v2 03/20] mm, hugetlb: fix subpool accounting handling

2013-08-22 Thread Aneesh Kumar K.V
Joonsoo Kim writes: > Hello, Aneesh. > > First of all, thank you for review! > > On Wed, Aug 21, 2013 at 02:58:20PM +0530, Aneesh Kumar K.V wrote: >> Joonsoo Kim writes: >> >> > If we alloc hugepage with avoid_reserve, we don't dequeue reserved one. >> > So, we should check subpool counter when

Re: [RFC PATCH next]module: Fix mod->mkobj.kobj potentially freed too early

2013-08-22 Thread Rusty Russell
Greg KH writes: > On Wed, Aug 21, 2013 at 05:49:58PM +0800, Li Zhong wrote: > > struct kobj_type module_ktype = { > > + .release =module_kobj_release, > > .sysfs_ops =&module_sysfs_ops, > > }; > > Wait, as there is no release function here for the kobject (a different > problem),

Re: [Xen-devel] [PATCH] xen/hvc: If we use xen_raw_printk let it also work on HVM guests.

2013-08-22 Thread Ian Campbell
On Wed, 2013-08-21 at 06:44 -0400, Konrad Rzeszutek Wilk wrote: > Ian Campbell wrote: > >On Tue, 2013-08-20 at 15:35 -0400, Konrad Rzeszutek Wilk wrote: > >> void xen_raw_console_write(const char *str) > >> { > >> - dom0_write_console(0, str, strlen(str)); > >> + ssize_t len = strlen(str); > >

[PATCH] clk: handle NULL struct clk gracefully

2013-08-22 Thread Mike Turquette
At some point changes to clk_set_rate and clk_set_parent introduced a bug whereby NULL struct clk pointers were treated as an error. This is in violation of the API in include/linux/clk.h. Reintroduce graceful handling of NULL clk's by bailing from clk_set_rate and clk_set_parent with return codes

Re: [PATCH 05/13] ARM: ux500: Apply a ste-* prefix onto snowball.dts

2013-08-22 Thread Lee Jones
On Wed, 21 Aug 2013, Olof Johansson wrote: > On Wed, Aug 21, 2013 at 3:37 PM, Linus Walleij > wrote: > > On Wed, Aug 21, 2013 at 2:31 AM, Olof Johansson wrote: > > > >>> diff --git a/arch/arm/boot/dts/snowball.dts > >>> b/arch/arm/boot/dts/ste-snowball.dts > >>> similarity index 100% > >>> ren

Re: [PATCH] kernel/padata.c: always check the return value of __padata_remove_cpu() and __padata_add_cpu()

2013-08-22 Thread Chen Gang
On 08/22/2013 02:05 PM, Steffen Klassert wrote: > On Thu, Aug 22, 2013 at 01:27:16PM +0800, Chen Gang wrote: >> On 08/22/2013 01:11 PM, Steffen Klassert wrote: >>> On Tue, Aug 20, 2013 at 11:44:31AM +0800, Chen Gang wrote: If this patch is correct, better to let CPU_ONLINE and CPU_DOWN_FA

[PATCH v2] kernel/padata.c: share code between CPU_ONLINE and CPU_DOWN_FAILED, same to CPU_DOWN_PREPARE and CPU_UP_CANCELED

2013-08-22 Thread Chen Gang
Share code between CPU_ONLINE and CPU_DOWN_FAILED, same to CPU_DOWN_PREPARE and CPU_UP_CANCELED. It will fix 2 bugs: "not check the return value of __padata_remove_cpu() and __padata_add_cpu()". "need add 'break' between CPU_UP_CANCELED and CPU_DOWN_FAILED". Signed-off-by: Chen Gang --- k

[tip:x86/asmlinkage] x86/asmlinkage: Fix warning in xen asmlinkage change

2013-08-22 Thread tip-bot for Andi Kleen
Commit-ID: eb86b5fd505cb97743d84226140cf247d91a2f03 Gitweb: http://git.kernel.org/tip/eb86b5fd505cb97743d84226140cf247d91a2f03 Author: Andi Kleen AuthorDate: Wed, 21 Aug 2013 13:07:42 -0700 Committer: Ingo Molnar CommitDate: Thu, 22 Aug 2013 09:17:19 +0200 x86/asmlinkage: Fix warning i

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

2013-08-22 Thread Johannes Weiner
On Tue, Aug 20, 2013 at 01:59:10PM -0700, Andrew Morton wrote: > On Sat, 17 Aug 2013 15:31:18 -0400 Johannes Weiner wrote: > > > shmem mappings already contain exceptional entries where swap slot > > information is remembered. > > > > To be able to store eviction information for regular page cac

Re: [PATCH 2/2] powerpc/iommu: check dev->iommu_group before remove a device from iommu_group

2013-08-22 Thread Alexey Kardashevskiy
On 08/19/2013 11:55 AM, Wei Yang wrote: > On Mon, Aug 19, 2013 at 11:39:49AM +1000, Alexey Kardashevskiy wrote: >> On 08/19/2013 11:29 AM, Wei Yang wrote: >>> On Fri, Aug 16, 2013 at 08:15:36PM +1000, Alexey Kardashevskiy wrote: On 08/16/2013 08:08 PM, Wei Yang wrote: > --- > arch/pow

Re: linux-next: Tree for Aug 21 [ screen corruption in graphical mode ]

2013-08-22 Thread Daniel Vetter
On Thu, Aug 22, 2013 at 08:32:47AM +0200, Sedat Dilek wrote: > It's independent of the applied patch. > My problem goes away with SNA but still exists with UXA. > > As said in my previous analysis... switching back to Ubuntu's graphics > did not show the symptoms, too. > > It's interesting to see

Re: [PATCH v2 10/20] mm, hugetlb: remove resv_map_put()

2013-08-22 Thread Joonsoo Kim
On Wed, Aug 21, 2013 at 04:19:20PM +0530, Aneesh Kumar K.V wrote: > Joonsoo Kim writes: > > > In following patch, I change vma_resv_map() to return resv_map > > for all case. This patch prepares it by removing resv_map_put() which > > doesn't works properly with following change, because it works

Re: [RFC 02/17] ARM: call clk_of_init from time_init

2013-08-22 Thread Mike Turquette
Quoting Arnd Bergmann (2013-08-21 11:54:10) > On Tuesday 20 August 2013, Sebastian Hesselbarth wrote: > > Perhaps Tegra is the common case but other SoC haven't dug deep enough? > > IMHO from a HW point-of-view clocks are really among the essential > > things that need to be running before you can

Re: [PATCH v2 11/20] mm, hugetlb: make vma_resv_map() works for all mapping type

2013-08-22 Thread Joonsoo Kim
On Wed, Aug 21, 2013 at 04:07:36PM +0530, Aneesh Kumar K.V wrote: > Joonsoo Kim writes: > > > Util now, we get a resv_map by two ways according to each mapping type. > > This makes code dirty and unreadable. So unfiying it. > > > > Signed-off-by: Joonsoo Kim > > > > diff --git a/mm/hugetlb.c b/m

Re: Regression: x86/mm: new _PTE_SWP_SOFT_DIRTY bit conflicts with existing use

2013-08-22 Thread Jan Beulich
>>> On 22.08.13 at 09:03, Cyrill Gorcunov wrote: > Ok, how about this? > > static inline pte_t pte_swp_mksoft_dirty(pte_t pte) > { > BUG_ON(pte_present(pte)); > return pte_set_flags(pte, _PAGE_SWP_SOFT_DIRTY); > } Sure, fine with me. Perhaps VM_BUG_ON() or some other similar construc

RE: [PATCH 4/4] Documentation: Add device tree bindings for Freescale FTM PWM

2013-08-22 Thread Xiubo Li-B47053
Hi Sascha, > > > Could you explain meaning of this property more precisely? I'm > > > interested especially how is this related to the PWM IP block and > boards. > > > > > > > Yes. > > There are 8 channels most. While the pinctrls of 4th and 5th channels > > could be used by uart's Rx and Tx, then

Re: bcache: Fix a writeback performance regression

2013-08-22 Thread Stefan Priebe - Profihost AG
great! Everything seems to work fine now! Except read_dirty always going to negative values after a reboot. Stefan Am 22.08.2013 08:02, schrieb Kent Overstreet: > On Thu, Aug 22, 2013 at 07:59:04AM +0200, Stefan Priebe wrote: >> >>> schedule_timeout() is not the same as >>> schedule_timeout_inte

[RFC PATCH v2 next]module: Fix mod->mkobj.kobj potentially freed too early

2013-08-22 Thread Li Zhong
DEBUG_KOBJECT_RELEASE helps to find the issue attached below. After some investigation, it seems the reason is: The mod->mkobj.kobj(a01600d0 below) is freed together with mod itself by module_free(mod, mod->module_core) in free_module(). However, its children still hold references to it, a

Re: [PATCH] documentation/devicetree: Move DT bindigns from gpio to watchdog

2013-08-22 Thread Johannes Thumshirn
On Wed, Aug 21, 2013 at 11:34:00AM -0600, Stephen Warren wrote: > On 08/21/2013 06:42 AM, Johannes Thumshirn wrote: > > I accidently put the devicetree bindings for the MEN A21 watchdog driver in > > Documentation/devicetree/bindings/gpio instead of > > Documentation/devicetree/bindings/watchdog, t

Re: [PATCH] iio: adc: Add bindigs documentation for twl6030 GPADC

2013-08-22 Thread Jonathan Cameron
Just realised that we have dropped linux-iio (could have sworn it was there back at the beginning of this thread) from the cc list so put it back. Might be worth adding lmsensors as well. Will leave that up to Guenter. A follow up below. On 21/08/13 22:03, Jonathan Cameron wrote: On 08/21/13 18

Re: bcache: Fix a writeback performance regression

2013-08-22 Thread Kent Overstreet
On Thu, Aug 22, 2013 at 09:32:52AM +0200, Stefan Priebe - Profihost AG wrote: > great! > > Everything seems to work fine now! Except read_dirty always going to > negative values after a reboot. That one's fixed in 3.11 :) -- To unsubscribe from this list: send the line "unsubscribe linux-kernel"

Re: [PATCH] of: add vendor prefix for Microchip Technology Inc

2013-08-22 Thread Lars Poeschel
On Wednesday 21 August 2013 at 19:40:56, Stephen Warren wrote: > On 08/21/2013 09:19 AM, Lars Poeschel wrote: > > Trivial patch to add Microchip Technology Inc. to the list > > of devicetree vendor prefixes, as it is already used in > > Documentation/devicetree/bindings/gpio/gpio-mcp23s08.txt > >

Re: [PATCH v2 03/20] mm, hugetlb: fix subpool accounting handling

2013-08-22 Thread Joonsoo Kim
On Thu, Aug 22, 2013 at 12:38:12PM +0530, Aneesh Kumar K.V wrote: > Joonsoo Kim writes: > > > Hello, Aneesh. > > > > First of all, thank you for review! > > > > On Wed, Aug 21, 2013 at 02:58:20PM +0530, Aneesh Kumar K.V wrote: > >> Joonsoo Kim writes: > >> > >> > If we alloc hugepage with avoid

Re: Regression: x86/mm: new _PTE_SWP_SOFT_DIRTY bit conflicts with existing use

2013-08-22 Thread Jan Beulich
>>> On 22.08.13 at 01:04, Linus Torvalds wrote: > On Wed, Aug 21, 2013 at 12:03 PM, Cyrill Gorcunov wrote: >> >> I personally don't see bug here because >> >> - this swapped page soft dirty bit is set for non-present entries only, >>never for present ones, just at moment we form swap pte ent

[PATCH 00/11] sched/cleanup: scheduler related cleanup and refine

2013-08-22 Thread Michael wang
This patch-set will cleanup and refine the code related to scheduler and save few lines. Rough tested (random benchmark) with latest tip/master on a desktop. Michael Wang (11): [PATCH 01/11] sched/cleanup: remove the extra line in init_sched_fair_class() [PATCH 02/11] sched/clean

[PATCH] iscsi-target: Fix ImmediateData=Yes failure regression in >= v3.10

2013-08-22 Thread Nicholas A. Bellinger
From: Nicholas Bellinger This patch addresses a regression bug within ImmediateData=Yes failure handling that ends up triggering an OOPs within >= v3.10 iscsi-target code. The problem occurs when iscsit_process_scsi_cmd() does the call to target_put_sess_cmd(), and once again in iscsit_get_immed

Re: [RFC PATCH next]module: Fix mod->mkobj.kobj potentially freed too early

2013-08-22 Thread Li Zhong
On Thu, 2013-08-22 at 16:30 +0930, Rusty Russell wrote: > Greg KH writes: > > On Wed, Aug 21, 2013 at 05:49:58PM +0800, Li Zhong wrote: > > > struct kobj_type module_ktype = { > > > + .release =module_kobj_release, > > > .sysfs_ops =&module_sysfs_ops, > > > }; > > > > Wait, as there

[PATCH 01/11] sched/cleanup: remove the extra line in init_sched_fair_class()

2013-08-22 Thread Michael wang
Remove the extra line. CC: Ingo Molnar CC: Peter Zijlstra Signed-off-by: Michael Wang --- kernel/sched/fair.c |1 - 1 file changed, 1 deletion(-) diff --git a/kernel/sched/fair.c b/kernel/sched/fair.c index aaaed57..a925e17 100644 --- a/kernel/sched/fair.c +++ b/kernel/sched/fair.c @@ -62

[PATCH 02/11] sched/cleanup: remove the extra parm of alloc_thread_info_node()

2013-08-22 Thread Michael wang
Parm 'tsk' has not been used, remove it to make code clean. CC: Andrew Morton CC: Al Viro CC: "Eric W. Biederman" CC: Oleg Nesterov CC: Srikar Dronamraju CC: Ingo Molnar CC: Peter Zijlstra Signed-off-by: Michael Wang --- kernel/fork.c |8 +++- 1 file changed, 3 insertions(+), 5 de

[PATCH 03/11] sched/cleanup: remove the extra parm of copy_flags()

2013-08-22 Thread Michael wang
Parm 'clone_flags' has not been used, remove it to make code clean. CC: Andrew Morton CC: Al Viro CC: "Eric W. Biederman" CC: Oleg Nesterov CC: Srikar Dronamraju CC: Ingo Molnar CC: Peter Zijlstra Signed-off-by: Michael Wang --- kernel/fork.c |4 ++-- 1 file changed, 2 insertions(+),

Re: [PATCH 2/2] powerpc/iommu: check dev->iommu_group before remove a device from iommu_group

2013-08-22 Thread Wei Yang
On Thu, Aug 22, 2013 at 05:23:34PM +1000, Alexey Kardashevskiy wrote: >On 08/19/2013 11:55 AM, Wei Yang wrote: >> On Mon, Aug 19, 2013 at 11:39:49AM +1000, Alexey Kardashevskiy wrote: >>> On 08/19/2013 11:29 AM, Wei Yang wrote: On Fri, Aug 16, 2013 at 08:15:36PM +1000, Alexey Kardashevskiy wro

Re: [PATCH] of: add vendor prefix for Microchip Technology Inc

2013-08-22 Thread Florian Fainelli
Hello, 2013/8/22 Lars Poeschel : > On Wednesday 21 August 2013 at 19:40:56, Stephen Warren wrote: >> On 08/21/2013 09:19 AM, Lars Poeschel wrote: >> > Trivial patch to add Microchip Technology Inc. to the list >> > of devicetree vendor prefixes, as it is already used in >> > Documentation/devicetr

Re: Regression: x86/mm: new _PTE_SWP_SOFT_DIRTY bit conflicts with existing use

2013-08-22 Thread Jan Beulich
>>> On 21.08.13 at 19:28, Andy Lutomirski wrote: > On Wed, Aug 21, 2013 at 6:48 AM, David Vrabel wrote: >> All, >> >> 179ef71c (mm: save soft-dirty bits on swapped pages) introduces a new >> PTE bit on x86 _PTE_SWP_SOFT_DIRTY which has the same value as _PTE_PSE >> and _PTE_PAT. >> >> With a Xen

[PATCH 04/11] sched/cleanup: refine rt_policy()

2013-08-22 Thread Michael wang
Refine the rt_policy() to save few lines. CC: Ingo Molnar CC: Peter Zijlstra Signed-off-by: Michael Wang --- kernel/sched/sched.h |4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/kernel/sched/sched.h b/kernel/sched/sched.h index 4c1cb80..b557f39 100644 --- a/kernel/sch

[PATCH 05/11] sched/cleanup: remove the extra parm of wakeup_gran()

2013-08-22 Thread Michael wang
Parm 'curr' has not been used, remove it to make code clean. CC: Ingo Molnar CC: Peter Zijlstra Signed-off-by: Michael Wang --- kernel/sched/fair.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/kernel/sched/fair.c b/kernel/sched/fair.c index a925e17..05643d6 100644

[PATCH 06/11] sched/cleanup: refine __setup_irq()

2013-08-22 Thread Michael wang
Refine the __setup_irq() to save few lines. CC: Thomas Gleixner CC: Ingo Molnar CC: Peter Zijlstra Signed-off-by: Michael Wang --- kernel/irq/manage.c |6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/kernel/irq/manage.c b/kernel/irq/manage.c index 514bcfd..46941e7

[PATCH 07/11] sched/cleanup: remove the extra parm of irq_set_thread_affinity()

2013-08-22 Thread Michael wang
Parm 'force' has been ignored, and currently no one need a 'true' option, so remove it to make code clean. CC: Thomas Gleixner CC: Ingo Molnar CC: Peter Zijlstra Signed-off-by: Michael Wang --- kernel/irq/internals.h |2 +- kernel/irq/manage.c|7 +++ kernel/irq/migration.c |

Re: oops during boot with CONFIG_SND_DYNAMIC_MINORS not set

2013-08-22 Thread Takashi Iwai
At Thu, 22 Aug 2013 00:42:41 +0300, Stratos Karafotis wrote: > > Hi, > > I get the following oops during boot when build with CONFIG_SND_DYNAMIC_MINORS > not set (3.11-rc6). > The issue is vanished building the kernel with CONFIG_SND_DYNAMIC_MINORS=y > as suggested in printk message. > > Regards

[PATCH 08/11] sched/cleanup: remove the extra parm of sched_clock_idle_wakeup_event()

2013-08-22 Thread Michael wang
Parm 'delta_ns' has not been used, remove it to make code clean. CC: Thomas Gleixner CC: "H. Peter Anvin" CC: Len Brown CC: "Rafael J. Wysocki" CC: John Stultz CC: Paul Gortmaker CC: Feng Tang CC: Ingo Molnar CC: Peter Zijlstra Signed-off-by: Michael Wang --- arch/x86/kernel/tsc.c

[PATCH 09/11] sched/cleanup: remove the extra parm of tick_nohz_start_idle()

2013-08-22 Thread Michael wang
Parm 'cpu' has not been used, remove it to make code clean. CC: Thomas Gleixner CC: Ingo Molnar CC: Peter Zijlstra Signed-off-by: Michael Wang --- kernel/time/tick-sched.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/kernel/time/tick-sched.c b/kernel/time/tick-sc

[PATCH 11/11] sched/cleanup: remove the extra parm of find_new_ilb()

2013-08-22 Thread Michael wang
Parm 'call_cpu' has not been used, remove it to make code clean. CC: Ingo Molnar CC: Peter Zijlstra Signed-off-by: Michael Wang --- kernel/sched/fair.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/kernel/sched/fair.c b/kernel/sched/fair.c index 05643d6..29ac8dd 10

[PATCH 10/11] sched/cleanup: remove the extra parm of retrigger_next_event()

2013-08-22 Thread Michael wang
Parm 'arg' has not been used, remove it to make code clean. CC: Thomas Gleixner CC: Ingo Molnar CC: Peter Zijlstra Signed-off-by: Michael Wang --- kernel/hrtimer.c |8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/kernel/hrtimer.c b/kernel/hrtimer.c index 383319b.

Re: [PATCH] of: reduce the number of PROBE_DEFERs

2013-08-22 Thread Jean-Francois Moine
On Tue, 20 Aug 2013 11:13:24 +0100 Grant Likely wrote: > On Tue, Aug 20, 2013 at 11:01 AM, Jean-Francois Moine wrote: > > This patch populates the platform from the device tree into two steps: > > the first step creates the nodes that are referenced by a phandle, > > the second step creates the

linux-next: Tree for Aug 22

2013-08-22 Thread Stephen Rothwell
Hi all, There will be no linux-next trees on Aug 23 or 26. Changes since 20130821: The xfs tree lost its build failure. The workqueues tree gained a conflict against the trivial tree. The driver-core tree gained a build failure so I used the version from next-20130821. I added supplied semant

Re: ADSL/ATM linklayer tc shaping regression fix commits for stable

2013-08-22 Thread Jesper Dangaard Brouer
On Wed, 21 Aug 2013 11:20:23 -0700 (PDT) David Miller wrote: > From: Jesper Dangaard Brouer > Date: Wed, 21 Aug 2013 14:03:50 +0200 > > > > > On Tue, 20 Aug 2013 08:16:50 -0700 > > Greg Kroah-Hartman wrote: > >> On Tue, Aug 20, 2013 at 01:26:29PM +0200, Jesper Dangaard Brouer wrote: > > [...]

Re: [PATCH v3 2/5] ARM: dts: add reference voltage property for MXS LRADC

2013-08-22 Thread Hector Palacios
Dear Alexandre, On 08/22/2013 12:13 AM, Alexandre Belloni wrote: Hi Pawel, On 14/08/2013 16:44, Pawel Moll wrote: On Tue, 2013-08-13 at 22:23 +0100, Jonathan Cameron wrote: On 07/22/13 15:04, Hector Palacios wrote: Some LRADC channels have fixed pre-dividers so they can measure different vol

Re: [PATCH 08/11] sched/cleanup: remove the extra parm of sched_clock_idle_wakeup_event()

2013-08-22 Thread Michael wang
On 08/22/2013 03:57 PM, Michael wang wrote: > Parm 'delta_ns' has not been used, remove it to make code clean. [snip] > > -extern void sched_clock_idle_sleep_event(void); > -extern void sched_clock_idle_wakeup_event(u64 delta_ns); Also remove the extra declaration here. Regards, Michael Wang >

Re: [PATCH 7/7] drivers: base: refactor add_memory_section() to add_memory_block()

2013-08-22 Thread Yasuaki Ishimatsu
(2013/08/21 2:13), Seth Jennings wrote: > Right now memory_dev_init() maintains the memory block pointer > between iterations of add_memory_section(). This is nasty. > > This patch refactors add_memory_section() to become add_memory_block(). > The refactoring pulls the section scanning out of mem

Re: ADSL/ATM linklayer tc shaping regression fix commits for stable

2013-08-22 Thread David Miller
From: Jesper Dangaard Brouer Date: Thu, 22 Aug 2013 10:04:26 +0200 > So, for future reference: > > Stable patches for the networking tree, I should: > 1) check http://patchwork.ozlabs.org/bundle/davem/stable/?state=* >to see if my patch is already on your stable queue > 2) if not, ask you an

Re: [PATCH 4/4] Documentation: Add device tree bindings for Freescale FTM PWM

2013-08-22 Thread Tomasz Figa
On Thursday 22 of August 2013 02:55:42 Xiubo Li-B47053 wrote: > Hi Tomasz, > > Thanks for your comments. > > > > +- #pwm-cells: Should be 3. Number of cells being used to specify > > > PWM > > > property. > > > + First cell specifies the per-chip channel index of the PWM > > > to use, the > > >

Re: [PATCH 7/7] drivers: base: refactor add_memory_section() to add_memory_block()

2013-08-22 Thread Yasuaki Ishimatsu
(2013/08/22 17:20), Yasuaki Ishimatsu wrote: > (2013/08/21 2:13), Seth Jennings wrote: >> Right now memory_dev_init() maintains the memory block pointer >> between iterations of add_memory_section(). This is nasty. >> >> This patch refactors add_memory_section() to become add_memory_block(). >> Th

Re: [PATCH v2 1/3] HID: Use hid_parser for pre-scanning the report descriptors

2013-08-22 Thread Benjamin Tissoires
Hi Henrik, On Wed, Aug 21, 2013 at 8:15 PM, Henrik Rydberg wrote: > Hi Benjamin, > > this looks pretty good to me, just a few nitpicks below. > >> hid_scan_report() implements its own HID report descriptor parsing. It is >> going to be really bad with the detection of Win 8 certified touchscreen,

[PATCH v4] extcon: palmas: Added a new compatible type *ti,palmas-usb-vid*

2013-08-22 Thread Kishon Vijay Abraham I
The Palmas device contains only a USB VBUS-ID detector, so added a compatible type *ti,palmas-usb-vid*. Didn't remove the existing compatible types for backward compatibility. Signed-off-by: Kishon Vijay Abraham I --- Changes from v3: * Fixed commit log message (VID to VBUS-ID). * Fixed the Docum

Re: [PATCH v2 0/3] HID: Win 8 multitouch panels detection in core

2013-08-22 Thread Benjamin Tissoires
On Wed, Aug 21, 2013 at 11:24 PM, Srinivas Pandruvada wrote: > On 08/21/2013 08:40 AM, Benjamin Tissoires wrote: >> >> Hi guys, >> >> this is the v2 of the rework of the pre-scanning of the hid report >> descriptors. >> This allows us to be able to detect Win 8 multitouch panels. >> I tried to tak

Re: Re: [PATCH] [BUGFIX] crash/ioapic: Prevent crash_kexec() from deadlocking of ioapic_lock

2013-08-22 Thread Yoshihiro YUNOMAE
(2013/08/20 23:27), Don Zickus wrote: On Tue, Aug 20, 2013 at 03:12:32AM -0700, Eric W. Biederman wrote: Yoshihiro YUNOMAE writes: Hi Ingo, Thank you for fixing typos! OK, I'll fix them and rename to ioapic_zap_locks(). Thank you again! The better fix for this would be to remove the disa

Re: ADSL/ATM linklayer tc shaping regression fix commits for stable

2013-08-22 Thread Jesper Dangaard Brouer
On Thu, 22 Aug 2013 01:23:48 -0700 (PDT) David Miller wrote: > From: Jesper Dangaard Brouer > Date: Thu, 22 Aug 2013 10:04:26 +0200 > > > So, for future reference: > > > > Stable patches for the networking tree, I should: > > 1) check http://patchwork.ozlabs.org/bundle/davem/stable/?state=* >

Re: [PATCH v2 4/8] drm/i2c: tda998x: prepare for video input configuration

2013-08-22 Thread Russell King - ARM Linux
On Thu, Aug 22, 2013 at 08:53:13AM +0200, Jean-Francois Moine wrote: > On Wed, 21 Aug 2013 23:36:05 +0100 > Russell King - ARM Linux wrote: > > > > AFAIK, the TI boards have no "pin-swapped", nor has the Cubox (there is > > > no need to set the bit CFG_GRA_SWAPRB of the register LCD_SPU_DMA_CTRL0

Re: kmemleak in apei_res_add

2013-08-22 Thread Huang Ying
Hi, Alexandra, On Wed, 2013-08-21 at 17:44 +0400, Alexandra N. Kossovsky wrote: > On Aug 21 10:38, Catalin Marinas wrote: > > On 12 August 2013 03:45, Alexandra N. Kossovsky > > wrote: > > > When running 3.10.3 with kmemleak enabled, I see following warnings from > > > kmemleak: > > > > > > unref

[PATCH 06/16] slab: put forward freeing slab management object

2013-08-22 Thread Joonsoo Kim
We don't need to free slab management object in rcu context, because, from now on, we don't manage this slab anymore. So put forward freeing. Signed-off-by: Joonsoo Kim diff --git a/mm/slab.c b/mm/slab.c index b378f91..607a9b8 100644 --- a/mm/slab.c +++ b/mm/slab.c @@ -1820,8 +1820,6 @@ static v

Re: [PATCH 1/7] drivers: base: move mutex lock out of add_memory_section()

2013-08-22 Thread Yasuaki Ishimatsu
(2013/08/21 2:24), Seth Jennings wrote: Gah! Forgot the cover letter. This patchset just seeks to clean up and refactor some things in memory.c for better understanding and possibly better performance due do a decrease in mutex acquisitions and refcount churn at boot time. No functional change

[PATCH 00/16] slab: overload struct slab over struct page to reduce memory usage

2013-08-22 Thread Joonsoo Kim
There is two main topics in this patchset. One is to reduce memory usage and the other is to change a management method of free objects of a slab. The SLAB allocate a struct slab for each slab. The size of this structure except bufctl array is 40 bytes on 64 bits machine. We can reduce memory wast

Re: [PATCH v2 12/20] mm, hugetlb: remove vma_has_reserves()

2013-08-22 Thread Aneesh Kumar K.V
Joonsoo Kim writes: > vma_has_reserves() can be substituted by using return value of > vma_needs_reservation(). If chg returned by vma_needs_reservation() > is 0, it means that vma has reserves. Otherwise, it means that vma don't > have reserves and need a hugepage outside of reserve pool. This d

Re: Re: [PATCH-SR9700] Merge USB 1.1 Ethernet Adapter SR9700 DeviceDriver into the Linux Kernel

2013-08-22 Thread Simon Horman
On Wed, Aug 21, 2013 at 06:02:45PM +0800, liujunliang_ljl wrote: > Dear Ben : > > 1, please give me email address of David Miller, and Thanks a > lot. You can find him in the MAINTAINERS file. He should also show up if you use scripts/get_maintainer.pl -- To unsubscribe from this

[PATCH 03/16] slab: remove colouroff in struct slab

2013-08-22 Thread Joonsoo Kim
Now there is no user colouroff, so remove it. Signed-off-by: Joonsoo Kim diff --git a/mm/slab.c b/mm/slab.c index 180f532..d9f81a0 100644 --- a/mm/slab.c +++ b/mm/slab.c @@ -219,7 +219,6 @@ struct slab { union { struct { struct list_head list; -

[PATCH 01/16] slab: correct pfmemalloc check

2013-08-22 Thread Joonsoo Kim
We checked pfmemalloc by slab unit, not page unit. You can see this in is_slab_pfmemalloc(). So other pages don't need to be set/cleared pfmemalloc. And, therefore we should check pfmemalloc in page flag of first page, but current implementation don't do that. virt_to_head_page(obj) just return 's

[PATCH 14/16] slab: use struct page for slab management

2013-08-22 Thread Joonsoo Kim
Now, there are a few field in struct slab, so we can overload these over struct page. This will save some memory and reduce cache footprint. After this change, slabp_cache and slab_size no longer related to a struct slab, so rename them as freelist_cache and freelist_size. These changes are just

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

2013-08-22 Thread Johannes Weiner
On Tue, Aug 20, 2013 at 01:59:20PM -0700, Andrew Morton wrote: > On Sat, 17 Aug 2013 15:31:22 -0400 Johannes Weiner wrote: > > > 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

[PATCH] x86: ioapic needs check attr when programmed more than once

2013-08-22 Thread Liu Ping Fan
When programming ioapic pinX more than once, current code does not check whether the later attr (trigger&polarity) is the same as the former or not. This causes a broken semantic. Fix it by reporting -EBUSY, when attr is different. Signed-off-by: Liu Ping Fan --- arch/x86/kernel/acpi/boot.c

[PATCH 02/16] slab: change return type of kmem_getpages() to struct page

2013-08-22 Thread Joonsoo Kim
It is more understandable that kmem_getpages() return struct page. And, with this, we can reduce one translation from virt addr to page and makes better code than before. Below is a change of this patch. * Before textdata bss dec hex filename 22123 23434 4 45561b

[PATCH 16/16] slab: rename slab_bufctl to slab_freelist

2013-08-22 Thread Joonsoo Kim
Now, bufctl is not proper name to this array. So change it. Signed-off-by: Joonsoo Kim diff --git a/mm/slab.c b/mm/slab.c index 6abc069..e8ec4c5 100644 --- a/mm/slab.c +++ b/mm/slab.c @@ -2538,7 +2538,7 @@ static struct freelist *alloc_slabmgmt(struct kmem_cache *cachep, return freelist

[PATCH 15/16] slab: remove useless statement for checking pfmemalloc

2013-08-22 Thread Joonsoo Kim
Now, virt_to_page(page->s_mem) is same as the page, because slab use this structure for management. So remove useless statement. Signed-off-by: Joonsoo Kim diff --git a/mm/slab.c b/mm/slab.c index cf39309..6abc069 100644 --- a/mm/slab.c +++ b/mm/slab.c @@ -750,9 +750,7 @@ static struct array_cac

[PATCH 12/16] slab: remove SLAB_LIMIT

2013-08-22 Thread Joonsoo Kim
It's useless now, so remove it. Signed-off-by: Joonsoo Kim diff --git a/mm/slab.c b/mm/slab.c index 7216ebe..98257e4 100644 --- a/mm/slab.c +++ b/mm/slab.c @@ -163,8 +163,6 @@ */ static bool pfmemalloc_active __read_mostly; -#defineSLAB_LIMIT (((unsigned int)(~0U))-1) - /* *

[PATCH 04/16] slab: remove nodeid in struct slab

2013-08-22 Thread Joonsoo Kim
We can get nodeid using address translation, so this field is not useful. Therefore, remove it. Signed-off-by: Joonsoo Kim diff --git a/mm/slab.c b/mm/slab.c index d9f81a0..69dc25a 100644 --- a/mm/slab.c +++ b/mm/slab.c @@ -222,7 +222,6 @@ struct slab { void *s_mem;

[PATCH 13/16] slab: replace free and inuse in struct slab with newly introduced active

2013-08-22 Thread Joonsoo Kim
Now, free in struct slab is same meaning as inuse. So, remove both and replace them with active. Signed-off-by: Joonsoo Kim diff --git a/mm/slab.c b/mm/slab.c index 98257e4..9dcbb22 100644 --- a/mm/slab.c +++ b/mm/slab.c @@ -174,8 +174,7 @@ struct slab { struct { struct l

[PATCH 10/16] slab: change the management method of free objects of the slab

2013-08-22 Thread Joonsoo Kim
Current free objects management method of the slab is weird, because it touch random position of the array of kmem_bufctl_t when we try to get free object. See following example. struct slab's free = 6 kmem_bufctl_t array: 1 END 5 7 0 4 3 2 To get free objects, we access this array with following

[PATCH 11/16] slab: remove kmem_bufctl_t

2013-08-22 Thread Joonsoo Kim
Now, we changed the management method of free objects of the slab and there is no need to use special value, BUFCTL_END, BUFCTL_FREE and BUFCTL_ACTIVE. So remove them. Signed-off-by: Joonsoo Kim diff --git a/mm/slab.c b/mm/slab.c index 4551d57..7216ebe 100644 --- a/mm/slab.c +++ b/mm/slab.c @@ -

[PATCH 05/16] slab: remove cachep in struct slab_rcu

2013-08-22 Thread Joonsoo Kim
We can get cachep using page in struct slab_rcu, so remove it. Signed-off-by: Joonsoo Kim diff --git a/mm/slab.c b/mm/slab.c index 69dc25a..b378f91 100644 --- a/mm/slab.c +++ b/mm/slab.c @@ -204,7 +204,6 @@ typedef unsigned int kmem_bufctl_t; */ struct slab_rcu { struct rcu_head head;

[PATCH 08/16] slab: use well-defined macro, virt_to_slab()

2013-08-22 Thread Joonsoo Kim
This is trivial change, just use well-defined macro. Signed-off-by: Joonsoo Kim diff --git a/mm/slab.c b/mm/slab.c index 9e98ee0..ee03eba 100644 --- a/mm/slab.c +++ b/mm/slab.c @@ -2853,7 +2853,6 @@ static inline void verify_redzone_free(struct kmem_cache *cache, void *obj) static void *cache_

[PATCH 07/16] slab: overloading the RCU head over the LRU for RCU free

2013-08-22 Thread Joonsoo Kim
With build-time size checking, we can overload the RCU head over the LRU of struct page to free pages of a slab in rcu context. This really help to implement to overload the struct slab over the struct page and this eventually reduce memory usage and cache footprint of the SLAB. Signed-off-by: Joo

[PATCH 09/16] slab: use __GFP_COMP flag for allocating slab pages

2013-08-22 Thread Joonsoo Kim
If we use 'struct page' of first page as 'struct slab', there is no advantage not to use __GFP_COMP. So use __GFP_COMP flag for all the cases. Signed-off-by: Joonsoo Kim diff --git a/mm/slab.c b/mm/slab.c index ee03eba..855f481 100644 --- a/mm/slab.c +++ b/mm/slab.c @@ -1712,15 +1712,6 @@ static

Re: Proposed stable release changes

2013-08-22 Thread Borislav Petkov
On Wed, Aug 21, 2013 at 01:54:01PM -0700, Tony Luck wrote: > On Wed, Aug 21, 2013 at 1:00 PM, Borislav Petkov wrote: > > We don't want to run daily snapshots of your tree though, right? Only > > -rcs because the daily states are kinda arbitrary and they can be broken > > in various ways. Or are we

Re: [PATCH 01/10] sched: Remove one division operation in find_busiest_queue()

2013-08-22 Thread Paul Turner
On Mon, Aug 19, 2013 at 9:00 AM, Peter Zijlstra wrote: > From: Joonsoo Kim > > Remove one division operation in find_busiest_queue() by using > crosswise multiplication: > > wl_i / power_i > wl_j / power_j := > wl_i * power_j > wl_j * power_i > > Signed-off-by: Joonsoo Kim > [pet

Re: [PATCH] pinctrl: rockchip: Simplify for loop iteration

2013-08-22 Thread Heiko Stübner
Am Mittwoch, 21. August 2013, 04:28:50 schrieb Axel Lin: > Just return once a match found makes the code simpler and shorter. > > Signed-off-by: Axel Lin as I'm a bit late for an Ack it seems, I can only offer a thanks for making this nicer :-) Heiko > --- > drivers/pinctrl/pinctrl-rockchi

Re: [PATCH v2] gpio: interrupt consistency check for OF GPIO IRQs

2013-08-22 Thread Lars Poeschel
On Thursday 22 August 2013 at 01:10:27, Stephen Warren wrote: > On 08/21/2013 03:49 PM, Tomasz Figa wrote: > >> diff --git a/drivers/gpio/gpiolib-of.c b/drivers/gpio/gpiolib-of.c > >> > >> +static void of_gpio_scan_irq_lines(const struct device_node *const > >> > >> + for (i = 0; i < int

There is strange packets dropping on new kernels

2013-08-22 Thread Alexey Vlasov
Hi. I noticed that tcpdump shows many dropped packets on new kernels. It looks like that: # tcpdump -n -i eth0 -c 10 ... 10 packets captured 10 packets received by filter 0 packets dropped by kernel 4294967268 packets dropped by interface # uname -r 3.10.7 On old kernels the same command doesn't

Re: [PATCH] of: add vendor prefix for Microchip Technology Inc

2013-08-22 Thread Lars Poeschel
On Thursday 22 August 2013 at 09:52:49, Florian Fainelli wrote: > Hello, > > 2013/8/22 Lars Poeschel : > > On Wednesday 21 August 2013 at 19:40:56, Stephen Warren wrote: > >> On 08/21/2013 09:19 AM, Lars Poeschel wrote: > >> > Trivial patch to add Microchip Technology Inc. to the list > >> > of de

Re: [RFC V2 PATCH 3/6] cpuidle/ppc: Add timer offload framework to support deep idle states

2013-08-22 Thread Preeti U Murthy
Hi Ben, On 08/22/2013 08:57 AM, Benjamin Herrenschmidt wrote: > On Wed, 2013-08-14 at 17:26 +0530, Preeti U Murthy wrote: > >> static irqreturn_t timer_action(int irq, void *data) >> { >> -timer_interrupt(); >> +decrementer_timer_interrupt(); >> return IRQ_HANDLED; >> } > > I don

Re: [RFC V2 PATCH 4/6] cpuidle/ppc: Add longnap state to the idle states on powernv

2013-08-22 Thread Preeti U Murthy
Hi Ben, On 08/22/2013 08:58 AM, Benjamin Herrenschmidt wrote: > On Wed, 2013-08-14 at 17:26 +0530, Preeti U Murthy wrote: >> This patch hooks into the existing broadcast framework along with the support >> that this patchset introduces for ppc, and the cpuidle driver backend >> for powernv(posted

Re: [PATCH v2 12/20] mm, hugetlb: remove vma_has_reserves()

2013-08-22 Thread Joonsoo Kim
On Thu, Aug 22, 2013 at 02:14:38PM +0530, Aneesh Kumar K.V wrote: > Joonsoo Kim writes: > > > vma_has_reserves() can be substituted by using return value of > > vma_needs_reservation(). If chg returned by vma_needs_reservation() > > is 0, it means that vma has reserves. Otherwise, it means that v

Re: [PATCH 21/33] clk: ux500: Add Device Tree support for the PRCC Kernel clock

2013-08-22 Thread Lee Jones
On Thu, 22 Aug 2013, Linus Walleij wrote: > On Wed, Aug 21, 2013 at 12:14 PM, Lee Jones wrote: > > On Wed, 21 Aug 2013, Linus Walleij wrote: > > >> Isn't it possible to fork a function u8500_clk_init_dt() to add all the > >> clocks in the DT probe path and keep this function > >> u8500_clk_init(

Re: [PATCH 09/33] ARM: ux500: Supply the I2C clocks lookup to the DBX500 DT

2013-08-22 Thread Lee Jones
On Tue, 20 Aug 2013, Linus Walleij wrote: > On Thu, Jun 6, 2013 at 2:16 PM, Lee Jones wrote: > > > +++ b/arch/arm/boot/dts/dbx5x0.dtsi > > @@ -572,6 +572,8 @@ > > v-i2c-supply = <&db8500_vape_reg>; > > > > clock-frequency = <40>; > > +

Re: [PATCH 4/4] mfd: as3722: introduce regmap support

2013-08-22 Thread Lee Jones
On Wed, 21 Aug 2013, Mark Brown wrote: > On Tue, Aug 20, 2013 at 01:08:00PM +0100, Lee Jones wrote: > > On Tue, 20 Aug 2013, Florian Lobmaier wrote: > > > > Signed-off-by: Florian Lobmaier > > > --- > > > drivers/mfd/as3722-regmap.c | 417 > > > +++ > >

Re: [PATCH 2/2] mm: add overcommit_kbytes sysctl variable

2013-08-22 Thread Jerome Marchand
On 08/21/2013 06:23 PM, Dave Hansen wrote: > On 08/21/2013 08:22 AM, Jerome Marchand wrote: Instead of introducing yet another tunable, why don't we just make the ratio that comes in from the user more fine-grained? sysctl overcommit_ratio=0.2 We change the internal

Re: [PATCH] mm, fs: avoid page allocation beyond i_size on read

2013-08-22 Thread Steven Whitehouse
Hi, On Wed, 2013-08-21 at 13:58 -0700, Andrew Morton wrote: > On Wed, 21 Aug 2013 17:42:12 +0100 Steven Whitehouse > wrote: > > > > I don't think the change is harmful. The worst case scenario is race with > > > write or truncate, but it's valid to return EOF in this case. > > > > > > What sce

Re: Regression: x86/mm: new _PTE_SWP_SOFT_DIRTY bit conflicts with existing use

2013-08-22 Thread David Vrabel
On 22/08/13 00:04, Linus Torvalds wrote: > On Wed, Aug 21, 2013 at 12:03 PM, Cyrill Gorcunov wrote: >> >> I personally don't see bug here because >> >> - this swapped page soft dirty bit is set for non-present entries only, >>never for present ones, just at moment we form swap pte entry >> >>

Re: [PATCH v4 2/5] mmc: atmel-mci: prepare clk before calling enable

2013-08-22 Thread boris brezillon
Hello Chris, Could you take this patch ? It has been acked by Ludovic, and I need it in order to gracefully handle at91's SoCs migration to common clk framework. Thanks. Best Regards, Boris On 18/07/2013 09:38, Boris BREZILLON wrote: Replace clk_enable/disable with clk_prepare_enable/disabl

  1   2   3   4   5   6   7   >