[PATCH net] net: rename and move busy poll mib counter

2013-08-06 Thread Eliezer Tamir
Move the low latency mib counter to the ip section. Rename it from low latency to busy poll. Reported-by: Shawn Bohrer Signed-off-by: Eliezer Tamir --- include/net/busy_poll.h |4 ++-- include/uapi/linux/snmp.h |2 +- net/ipv4/proc.c |2 +- 3 files changed, 4

RE: [PATCH] iommu/exynos: remove dead code (set_prefbuf)

2013-08-06 Thread Cho KyongHo
> -Original Message- > From: grund...@google.com [mailto:grund...@google.com] On Behalf Of Grant > Grundler > Sent: Tuesday, August 06, 2013 1:40 AM > > Ping? > > Adding linux-iommu ML. > > thanks, > grant > > On Tue, Jul 2, 2013 at 9:08 AM, Grant Grundler wrote: > >

Re: [PATCH 1/5] pdev: Fix platform device resource linking

2013-08-06 Thread Pantelis Antoniou
Hi Greg, On Aug 6, 2013, at 12:36 PM, Greg Kroah-Hartman wrote: > On Tue, Aug 06, 2013 at 10:53:40AM +0300, Pantelis Antoniou wrote: >> Platform device removal uncovered a number of problems with >> the way resources are handled in the core platform code. >> >> Resources now form child/parent

Re: [PATCH 001/001] CHAR DRIVERS: a simple device to give daemons a /sys-like interface

2013-08-06 Thread Greg Kroah-Hartman
On Mon, Aug 05, 2013 at 04:46:32PM -0700, Bob Smith wrote: > Greg > Thanks for discussing the module with me. I think I'm now > closer to distilling it down to its essence. > > > GOAL: > The goal of this module is to give user space programs an > interface similar to that enjoyed by the kernel

Re: [PATCH] mm: zcache: zcache_cleancache_flush_fs fix

2013-08-06 Thread Bob Liu
Hi Piotr, On 08/06/2013 05:36 PM, Piotr Sarna wrote: > This patch fixes "mm: zcache: core functions added" patch, > available at https://lkml.org/lkml/2013/7/20/90. > It regards incorrect implementation of zcache_cleancache_flush_fs(). > > Function above should be effective only if cleancache

Re: [PATCH] warning-elimination: android: binder

2013-08-06 Thread Greg KH
On Tue, Aug 06, 2013 at 11:36:19AM +0800, Andy Green wrote: > From: Andy Green > > warning-elimination: android: binder > > This commit in mainline (now) causes a couple of warnings What specifically are those warnings? I don't see any here, so I'm guessing your patch isn't really needed :)

Re: [PATCH 5/5] arm: omap: Proper cleanups for omap_device

2013-08-06 Thread Pantelis Antoniou
Hi Greg, On Aug 6, 2013, at 12:33 PM, Greg Kroah-Hartman wrote: > On Tue, Aug 06, 2013 at 10:53:44AM +0300, Pantelis Antoniou wrote: >> Removing any omap device always resulted in a crash; turns out >> BUS_NOTIFY_DEL_DEVICE is not the last notifier event sent in the >> course of removing the

Re: [PATCH 02/22] ARM: nomadik: Remove '0x's from nomadik stn8815 DTS file

2013-08-06 Thread Linus Walleij
On Mon, Jul 22, 2013 at 12:52 PM, Lee Jones wrote: > Signed-off-by: Lee Jones Applied. Yours, Linus Walleij -- 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

[PATCH] mm: zcache: zcache_cleancache_flush_fs fix

2013-08-06 Thread Piotr Sarna
This patch fixes "mm: zcache: core functions added" patch, available at https://lkml.org/lkml/2013/7/20/90. It regards incorrect implementation of zcache_cleancache_flush_fs(). Function above should be effective only if cleancache pool referred by pool_id is valid. This issue is checked by

Re: [PATCH 1/5] pdev: Fix platform device resource linking

2013-08-06 Thread Greg Kroah-Hartman
On Tue, Aug 06, 2013 at 10:53:40AM +0300, Pantelis Antoniou wrote: > Platform device removal uncovered a number of problems with > the way resources are handled in the core platform code. > > Resources now form child/parent linkages and this requires > proper linking of the resources. On top of

Re: [PATCH 5/5] arm: omap: Proper cleanups for omap_device

2013-08-06 Thread Greg Kroah-Hartman
On Tue, Aug 06, 2013 at 10:53:44AM +0300, Pantelis Antoniou wrote: > Removing any omap device always resulted in a crash; turns out > BUS_NOTIFY_DEL_DEVICE is not the last notifier event sent in the > course of removing the device, the correct event is > BUS_NOTIFY_UNBOUND_DRIVER, which still is

Re: [Xen-devel] [PATCH 3/3] MAINTAINERS: Add in two extra co-maintainers of the Xen tree.

2013-08-06 Thread David Vrabel
On 05/08/13 19:05, Konrad Rzeszutek Wilk wrote: > Both Boris and David have graciously volunteered to help in > maintaining the Xen subsystem tree. Cementing this in the > MAINTAINERS file so they are copied on Xen related patches. Acked-by: David Vrabel > --- > MAINTAINERS | 2 ++ > 1 file

Re: [RFC PATCH 1/4] zbud: use page ref counter for zbud pages

2013-08-06 Thread Krzysztof Kozlowski
Hi Bob, Thank you for review. On wto, 2013-08-06 at 17:00 +0800, Bob Liu wrote: > Nit picker, how about change the name to adjust_lists() or something > like this because we don't do any rebalancing. OK, I'll change it. Best regards, Krzysztof -- To unsubscribe from this list: send the line

Re: [PATCH v2 3/3] sched: clean-up struct sd_lb_stat

2013-08-06 Thread Preeti U Murthy
Hi Joonsoo, On 08/05/2013 01:02 PM, Joonsoo Kim wrote: >>> + if (busiest->group_imb) { >>> + busiest->sum_weighted_load = >>> + min(busiest->sum_weighted_load, sds->sd_avg_load); >> >> Right here we get confused as to why the total load is being compared >> against

Re: [3.10.4] NFS locking panic, plus persisting NFS shutdown panic from 3.9.*

2013-08-06 Thread Jeff Layton
On Tue, 6 Aug 2013 02:21:35 + "Myklebust, Trond" wrote: > On Mon, 2013-08-05 at 14:33 -0400, Jeff Layton wrote: > > On Mon, 5 Aug 2013 18:18:03 + > > "Myklebust, Trond" wrote: > > > > > On Mon, 2013-08-05 at 13:37 -0400, Jeff Layton wrote: > > > > On Mon, 5 Aug 2013 16:15:01 + > >

[Help Test] kdump, x86, acpi: Reproduce CPU0 SMI corruption issue after unsetting BSP flag

2013-08-06 Thread HATAYAMA Daisuke
Hello, I've addressing kdump restriction that there's only one cpu available on the kdump 2nd kernel. Now I need to check if the following CPU0 SMI corruption issue fixed in the following commit can again be reproduced by unsetting BSP flag of the boot cpu: commit

Re: Enable arm_global_timer for Zynq brakes boot

2013-08-06 Thread Michal Simek
On 08/06/2013 10:46 AM, Daniel Lezcano wrote: > On 08/06/2013 03:28 AM, Sören Brinkmann wrote: >> Hi Daniel, >> >> On Thu, Aug 01, 2013 at 07:48:04PM +0200, Daniel Lezcano wrote: >>> On 08/01/2013 07:43 PM, Sören Brinkmann wrote: On Thu, Aug 01, 2013 at 07:29:12PM +0200, Daniel Lezcano wrote:

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

2013-08-06 Thread Tom Gundersen
On Tue, Aug 6, 2013 at 11:11 AM, Tom Gundersen wrote: > On Tue, Aug 6, 2013 at 10:20 AM, Maarten Lankhorst > wrote: >> Op 05-08-13 18:29, Andy Lutomirski schreef: >>> The systemd commit below can delay firmware loading by multiple >>> minutes if CONFIG_FW_LOADER_USER_HELPER=y. Unfortunately no

Re: [RFC PATCH 0/4] mm: reclaim zbud pages on migration and compaction

2013-08-06 Thread Bob Liu
On 08/06/2013 02:42 PM, Krzysztof Kozlowski wrote: > Hi, > > Currently zbud pages are not movable and they cannot be allocated from CMA > region. These patches try to address the problem by: > 1. Adding a new form of reclaim of zbud pages. > 2. Reclaiming zbud pages during migration and

Re: [PATCH 1/2] ARM: msm: Add support for MSM8974 Dragonboard

2013-08-06 Thread Ivan T. Ivanov
On Mon, 2013-08-05 at 16:02 -0700, Rohit Vaswani wrote: > This patch adds basic board support for MSM8974 Dragonboard > which belongs to the Snapdragon 800 family. > For now, just support a basic machine with device tree. > > Signed-off-by: Rohit Vaswani > --- > arch/arm/boot/dts/Makefile

[PATCH 3/3] pinctrl: sirf: add missing __iomem annotation

2013-08-06 Thread Jingoo Han
Added missing __iomem annotation in order to fix the following sparse warnings: drivers/pinctrl/sirf/pinctrl-sirf.c:846:14: warning: incorrect type in assignment (different address spaces) drivers/pinctrl/sirf/pinctrl-sirf.c:846:14:expected void *regs

[PATCH 2/3] pinctrl: st: Staticize local symbols

2013-08-06 Thread Jingoo Han
These local symbols are used only in this file. Fix the following sparse warning: drivers/pinctrl/pinctrl-st.c:291:14: warning: symbol 'stih415_input_delays' was not declared. Should it be static? drivers/pinctrl/pinctrl-st.c:292:14: warning: symbol 'stih415_output_delays' was not declared.

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

2013-08-06 Thread Tom Gundersen
On Tue, Aug 6, 2013 at 10:20 AM, Maarten Lankhorst wrote: > Op 05-08-13 18:29, Andy Lutomirski schreef: >> The systemd commit below can delay firmware loading by multiple >> minutes if CONFIG_FW_LOADER_USER_HELPER=y. Unfortunately no one >> noticed that the systemd-udev change would break new

[PATCH 1/3] pinctrl: pinconf: fix comparison of different types

2013-08-06 Thread Jingoo Han
Fix the following sparse warning: drivers/pinctrl/pinconf.c:521:20: error: incompatible types in comparison expression (different type sizes) Signed-off-by: Jingoo Han --- drivers/pinctrl/pinconf.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

Re: About TRB_TO_EP_INDEX() macro using

2013-08-06 Thread 'gre...@linuxfoundation.org'
On Tue, Aug 06, 2013 at 08:59:32AM +, Du, ChangbinX wrote: > > On Mon, Aug 05, 2013 at 08:07:25AM +, Du, ChangbinX wrote: > > > Recently when I check xHCI code, find that some functions try to get EP > > > index > > > from a Command Completion Event TRB via TRB_TO_EP_INDEX() macro. > > >

Re: [PATCH v4 14/17] clocksource: time-armada-370-xp: Switch to sched_clock_register()

2013-08-06 Thread Gregory CLEMENT
On 19/07/2013 01:21, Stephen Boyd wrote: > The 32 bit sched_clock interface now supports 64 bits. Upgrade to > the 64 bit function to allow us to remove the 32 bit registration > interface. Acked-by: Gregory CLEMENT > > Cc: Gregory CLEMENT > Signed-off-by: Stephen Boyd > --- >

Re: [PATCH v3 05/16] ARM: mvebu: remove device tree parsing for cpu nodes

2013-08-06 Thread Sudeep KarkadaNagesha
On 06/08/13 09:25, Gregory CLEMENT wrote: > On 05/08/2013 18:28, Sudeep KarkadaNagesha wrote: >> On 01/08/13 13:02, Jason Cooper wrote: >>> Sudeep, >>> >>> On Thu, Aug 01, 2013 at 10:54:44AM +0100, Sudeep KarkadaNagesha wrote: On 22/07/13 12:32, Sudeep KarkadaNagesha wrote: > From: Sudeep

Re: [PATCH v2 0/3] ARM/ARM64 architected timer updates

2013-08-06 Thread Sudeep KarkadaNagesha
On 06/08/13 09:09, Daniel Lezcano wrote: > On 08/05/2013 06:25 PM, Sudeep KarkadaNagesha wrote: >> Hi Daniel, >> >> On 02/08/13 13:12, Daniel Lezcano wrote: >>> On 08/02/2013 11:55 AM, Sudeep KarkadaNagesha wrote: On 02/08/13 10:48, Daniel Lezcano wrote: > On 08/02/2013 11:32 AM, Sudeep

Re: [RFC PATCH 1/4] zbud: use page ref counter for zbud pages

2013-08-06 Thread Bob Liu
Hi Krzysztof, On 08/06/2013 02:42 PM, Krzysztof Kozlowski wrote: > Use page reference counter for zbud pages. The ref counter replaces > zbud_header.under_reclaim flag and ensures that zbud page won't be freed > when zbud_free() is called during reclaim. It allows implementation of > additional

RE: About TRB_TO_EP_INDEX() macro using

2013-08-06 Thread Du, ChangbinX
> On Mon, Aug 05, 2013 at 08:07:25AM +, Du, ChangbinX wrote: > > Recently when I check xHCI code, find that some functions try to get EP > > index > > from a Command Completion Event TRB via TRB_TO_EP_INDEX() macro. > > > > This is totally wrong. The macro definition is: > > > > #define

Re: [PATCH 2/2] kernel/sys.c: remove useless variable 'old_fsgid' for setfsgid()

2013-08-06 Thread Chen Gang
On 08/06/2013 04:02 PM, Chen Gang wrote: > Remove useless variable 'old_fsgid' to make code clearer for readers. > > Signed-off-by: Chen Gang > --- > kernel/sys.c |9 ++--- > 1 files changed, 2 insertions(+), 7 deletions(-) > > diff --git a/kernel/sys.c b/kernel/sys.c > index

Re: [PATCH 2/2] kernel/sys.c: remove useless variable 'old_fsuid' for setfsuid()

2013-08-06 Thread Chen Gang
On 08/06/2013 04:16 PM, Chen Gang wrote: > Remove useless variable 'old_fsuid' to make code clearer for readers. > > Signed-off-by: Chen Gang > --- > kernel/sys.c | 14 ++ > 1 files changed, 6 insertions(+), 8 deletions(-) > > diff --git a/kernel/sys.c b/kernel/sys.c > index

Re: [PATCH RFC 51/51] ARM: 7805/1: mm: change max*pfn to include the physical offset of memory

2013-08-06 Thread Russell King - ARM Linux
On Mon, Aug 05, 2013 at 10:15:11PM -0500, Rob Herring wrote: > On Thu, Aug 1, 2013 at 5:25 PM, Santosh Shilimkar > wrote: > > +/* The ARM override for dma_max_pfn() */ > > +static inline unsigned long dma_max_pfn(struct device *dev) > > +{ > > + return PHYS_PFN_OFFSET + dma_to_pfn(dev,

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

2013-08-06 Thread Joonsoo Kim
On Tue, Aug 06, 2013 at 05:43:40PM +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

Re: Enable arm_global_timer for Zynq brakes boot

2013-08-06 Thread Daniel Lezcano
On 08/06/2013 03:28 AM, Sören Brinkmann wrote: > Hi Daniel, > > On Thu, Aug 01, 2013 at 07:48:04PM +0200, Daniel Lezcano wrote: >> On 08/01/2013 07:43 PM, Sören Brinkmann wrote: >>> On Thu, Aug 01, 2013 at 07:29:12PM +0200, Daniel Lezcano wrote: On 08/01/2013 01:38 AM, Sören Brinkmann wrote:

[PATCH 2/4] mm, rmap: allocate anon_vma_chain before starting to link anon_vma_chain

2013-08-06 Thread Joonsoo Kim
If we allocate anon_vma_chain before starting to link, we can reduce the lock hold time. This patch implement it. Signed-off-by: Joonsoo Kim diff --git a/mm/rmap.c b/mm/rmap.c index c2f51cb..1603f64 100644 --- a/mm/rmap.c +++ b/mm/rmap.c @@ -240,18 +240,21 @@ int anon_vma_clone(struct

[PATCH 1/4] mm, rmap: do easy-job first in anon_vma_fork

2013-08-06 Thread Joonsoo Kim
If we fail due to some errorous situation, it is better to quit without doing heavy work. So changing order of execution. Signed-off-by: Joonsoo Kim diff --git a/mm/rmap.c b/mm/rmap.c index a149e3a..c2f51cb 100644 --- a/mm/rmap.c +++ b/mm/rmap.c @@ -278,19 +278,19 @@ int anon_vma_fork(struct

[PATCH] kernel/sys.c: improve the usage of return value

2013-08-06 Thread Chen Gang
Improve the usage of return value, so not only can make code clearer to readers, but also can improve the performance. Assign the return error code only when error occurs, so can save some instructions. For getcpu(), uname(), newuname(), and override_release(), can remove the return variable to

Re: [PATCH V2] kexec: remove unnecessary return

2013-08-06 Thread Zhang Yanfei
于 2013/8/6 16:35, Xishi Qiu 写道: > Code can not run here forever, so remove the unnecessary return. > > Signed-off-by: Xishi Qiu > Suggested-by: Zhang Yanfei Reviewed-by: Zhang Yanfei > --- > kernel/kexec.c |5 + > 1 files changed, 1 insertions(+), 4 deletions(-) > > diff --git

[PATCH 3/4] mm, rmap: minimize lock hold when unlink_anon_vmas

2013-08-06 Thread Joonsoo Kim
Currently, we free the avc objects with holding a lock. To minimize lock hold time, we just move the avc objects to another list with holding a lock. Then, iterate them and free objects without holding a lock. This makes lock hold time minimized. Signed-off-by: Joonsoo Kim diff --git

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

2013-08-06 Thread Joonsoo Kim
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. Signed-off-by: Joonsoo Kim diff --git a/mm/page_alloc.c

Re: [PATCH 2/2] pinctrl: add generic pins and functions properties

2013-08-06 Thread Mark Rutland
On Mon, Aug 05, 2013 at 10:56:00PM +0100, Stephen Warren wrote: > From: Stephen Warren > > pinctrl bindings can benefit from generic property names that define > which pins a "pin configuration node" affects, and which mux function > to select onto those pins. Document new properties for this

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

2013-08-06 Thread Joonsoo Kim
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. Signed-off-by: Joonsoo Kim diff --git a/mm/page_alloc.c

[PATCH v3 1/3] sched: remove one division operation in find_buiest_queue()

2013-08-06 Thread Joonsoo Kim
Remove one division operation in find_buiest_queue(). Signed-off-by: Joonsoo Kim diff --git a/kernel/sched/fair.c b/kernel/sched/fair.c index 9565645..52898dc 100644 --- a/kernel/sched/fair.c +++ b/kernel/sched/fair.c @@ -4968,7 +4968,7 @@ static struct rq *find_busiest_queue(struct lb_env

Re: [PATCH V2] kexec: remove unnecessary return

2013-08-06 Thread Simon Horman
On Tue, Aug 06, 2013 at 04:35:00PM +0800, Xishi Qiu wrote: > Code can not run here forever, so remove the unnecessary return. > > Signed-off-by: Xishi Qiu > Suggested-by: Zhang Yanfei Reviewed-by: Simon Horman > --- > kernel/kexec.c |5 + > 1 files changed, 1 insertions(+), 4

[PATCH v2 mmotm 2/3] mm: move pgtable related functions to right place

2013-08-06 Thread Joonsoo Kim
pgtable related functions are mostly in pgtable-generic.c. So move remaining functions from memory.c to pgtable-generic.c. Signed-off-by: Joonsoo Kim diff --git a/mm/memory.c b/mm/memory.c index f2ab2a8..8fd4d42 100644 --- a/mm/memory.c +++ b/mm/memory.c @@ -374,30 +374,6 @@ void

[PATCH v2 mmotm 1/3] mm, page_alloc: add unlikely macro to help compiler optimization

2013-08-06 Thread Joonsoo Kim
We rarely allocate a page with ALLOC_NO_WATERMARKS and it is used in slow path. For helping compiler optimization, add unlikely macro to ALLOC_NO_WATERMARKS checking. This patch doesn't have any effect now, because gcc already optimize this properly. But we cannot assume that gcc always does

[PATCH v3 2/3] sched: factor out code to should_we_balance()

2013-08-06 Thread Joonsoo Kim
Now checking whether this cpu is appropriate to balance or not is embedded into update_sg_lb_stats() and this checking has no direct relationship to this function. There is not enough reason to place this checking at update_sg_lb_stats(), except saving one iteration for sched_group_cpus. In this

[PATCH v2 mmotm 3/3] swap: clean-up #ifdef in page_mapping()

2013-08-06 Thread Joonsoo Kim
PageSwapCache() is always false when !CONFIG_SWAP, so compiler properly discard related code. Therefore, we don't need #ifdef explicitly. Acked-by: Johannes Weiner Signed-off-by: Joonsoo Kim diff --git a/include/linux/swap.h b/include/linux/swap.h index 24db914..c03c139 100644 ---

Re: [PATCH v3 05/16] ARM: mvebu: remove device tree parsing for cpu nodes

2013-08-06 Thread Gregory CLEMENT
On 05/08/2013 18:28, Sudeep KarkadaNagesha wrote: > On 01/08/13 13:02, Jason Cooper wrote: >> Sudeep, >> >> On Thu, Aug 01, 2013 at 10:54:44AM +0100, Sudeep KarkadaNagesha wrote: >>> On 22/07/13 12:32, Sudeep KarkadaNagesha wrote: From: Sudeep KarkadaNagesha Currently

[PATCH v3 3/3] sched: clean-up struct sd_lb_stat

2013-08-06 Thread Joonsoo Kim
There is no reason to maintain separate variables for this_group and busiest_group in sd_lb_stat, except saving some space. But this structure is always allocated in stack, so this saving isn't really benificial. This patch unify these variables, so IMO, readability may be improved.

[PATCH V2] kexec: remove unnecessary return

2013-08-06 Thread Xishi Qiu
Code can not run here forever, so remove the unnecessary return. Signed-off-by: Xishi Qiu Suggested-by: Zhang Yanfei --- kernel/kexec.c |5 + 1 files changed, 1 insertions(+), 4 deletions(-) diff --git a/kernel/kexec.c b/kernel/kexec.c index 59f7b55..2a74f30 100644 ---

[PATCH v3 0/3] optimization, clean-up about fair.c

2013-08-06 Thread Joonsoo Kim
Patch 1 is for removing one division operation with multiplication. Patch 2,3 is for clean-up related to load_balance(), there is improvement in terms of code size and readability may be also improved. Feel free to give a comment for this patchset. It's tested on v3.10. On v3.11-rc3, it can be

Re: Warnings from silentoldconfig (Re: [GIT PULL] PCI updates for v3.11)

2013-08-06 Thread Michal Marek
On 6.8.2013 00:58, Yann E. MORIN wrote: > On 2013-08-05 10:39 +0200, Michal Marek spake thusly: >> Added Yann and the linux-kbuild list to CC. Reproducer: >> >> git checkout 1fe0135 >> make mrproper >> make allmodconfig >> make silentoldconfig >> git checkout aa8032b >> make

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

2013-08-06 Thread Maarten Lankhorst
Op 05-08-13 18:29, Andy Lutomirski schreef: > The systemd commit below can delay firmware loading by multiple > minutes if CONFIG_FW_LOADER_USER_HELPER=y. Unfortunately no one > noticed that the systemd-udev change would break new kernels as well > as old kernels. > > Since the kernel apparently

Re: [PATCH] kexec: remove unnecessary return

2013-08-06 Thread Zhang Yanfei
于 2013/8/6 16:05, Xishi Qiu 写道: > Code can not run here forever, so remove the unnecessary return. > > Signed-off-by: Xishi Qiu > --- > kernel/kexec.c |2 -- > 1 files changed, 0 insertions(+), 2 deletions(-) > > diff --git a/kernel/kexec.c b/kernel/kexec.c > index 59f7b55..c14e3ea 100644

[PATCH 2/2] kernel/sys.c: remove useless variable 'old_fsuid' for setfsuid()

2013-08-06 Thread Chen Gang
Remove useless variable 'old_fsuid' to make code clearer for readers. Signed-off-by: Chen Gang --- kernel/sys.c | 14 ++ 1 files changed, 6 insertions(+), 8 deletions(-) diff --git a/kernel/sys.c b/kernel/sys.c index 558ccdb..e3983b2 100644 --- a/kernel/sys.c +++ b/kernel/sys.c

[PATCH 1/2] kernel/sys.c: return the current uid when error occurs

2013-08-06 Thread Chen Gang
According to the API definition, when error occurs, need return current fsuid instead of the previous one. The related informations ("man setfsuid"): RETURN VALUE On success, the previous value of fsuid is returned. On error, the current value of fsuid is returned. Signed-off-by:

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

2013-08-06 Thread Chen Gang
They are 2 related patches for setfsuid(). Patch 1 for bug fix: return the current uid when error occurs. Patch 2 for cleaning code: remove useless variable 'old_fsuid'. Signed-off-by: Chen Gang -- kernel/sys.c | 14 ++ 1 files changed, 6 insertions(+), 8 deletions(-) -- To

Re: kernel/sys.c: for setfsuid(), return the current uid when error occurs

2013-08-06 Thread Chen Gang
Please skip it, sorry for bothering you with waste information. On 08/06/2013 04:13 PM, Chen Gang wrote: > They are 2 related patches for setfsuid(). > > Patch 1 for bug fix: return the current uid when error occurs. > Patch 2 for cleaning code: remove useless variable 'old_fsuid'. > >

kernel/sys.c: for setfsuid(), return the current uid when error occurs

2013-08-06 Thread Chen Gang
They are 2 related patches for setfsuid(). Patch 1 for bug fix: return the current uid when error occurs. Patch 2 for cleaning code: remove useless variable 'old_fsuid'. Signed-off-by: Chen Gang -- kernel/sys.c | 14 ++ 1 files changed, 6 insertions(+), 8 deletions(-) -- To

Re: [PATCH v2 0/3] ARM/ARM64 architected timer updates

2013-08-06 Thread Daniel Lezcano
On 08/05/2013 06:25 PM, Sudeep KarkadaNagesha wrote: > Hi Daniel, > > On 02/08/13 13:12, Daniel Lezcano wrote: >> On 08/02/2013 11:55 AM, Sudeep KarkadaNagesha wrote: >>> On 02/08/13 10:48, Daniel Lezcano wrote: On 08/02/2013 11:32 AM, Sudeep KarkadaNagesha wrote: > Hi Daniel, >

[PATCH] kexec: remove unnecessary return

2013-08-06 Thread Xishi Qiu
Code can not run here forever, so remove the unnecessary return. Signed-off-by: Xishi Qiu --- kernel/kexec.c |2 -- 1 files changed, 0 insertions(+), 2 deletions(-) diff --git a/kernel/kexec.c b/kernel/kexec.c index 59f7b55..c14e3ea 100644 --- a/kernel/kexec.c +++ b/kernel/kexec.c @@

RE: [PATCH 1/1] checkpatch: fix some whitespace issues caused by --fix

2013-08-06 Thread Phil Carmody
> On Mon, 2013-08-05 at 14:08 +0300, Phil Carmody wrote: > > Lines with incorrect spacing around an operator, such as: > > bystander, correct,incorrect > > would get "fixed" to > > bystander,correct, incorrect > > as the correct argument as well as the incorrectly-spaced operator > > were both

[PATCH 2/2] kernel/sys.c: remove useless variable 'old_fsgid' for setfsgid()

2013-08-06 Thread Chen Gang
Remove useless variable 'old_fsgid' to make code clearer for readers. Signed-off-by: Chen Gang --- kernel/sys.c |9 ++--- 1 files changed, 2 insertions(+), 7 deletions(-) diff --git a/kernel/sys.c b/kernel/sys.c index 9356dc8..fc169d8 100644 --- a/kernel/sys.c +++ b/kernel/sys.c @@

[PATCH 1/2] kernel/sys.c: return the current gid when error occurs

2013-08-06 Thread Chen Gang
According to the API definition, when error occurs, need return current fsgid instead of the previous one. The related informations ("man setfsgid"): RETURN VALUE On success, the previous value of fsgid is returned. On error, the current value of fsgid is returned. Signed-off-by:

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

2013-08-06 Thread Chen Gang
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: Chen Gang -- kernel/sys.c | 15 +-- 1 files changed, 5 insertions(+), 10 deletions(-) -- To

Re: [PATCH RESEND] ARM: dts: Add USB host node for Exynos4

2013-08-06 Thread Tomasz Figa
Hi Dongjin, On Tuesday 06 of August 2013 03:09:19 Dongjin Kim wrote: > This patch adds EHCI and OHCI host device nodes for Exynos4. > > CC: Jingoo Han > Signed-off-by: Dongjin Kim > --- > arch/arm/boot/dts/exynos4.dtsi | 18 ++ > 1 file changed, 18 insertions(+) > > diff

Re: [PATCH V3]hrtimer: Fix a performance regression by disable reprogramming in remove_hrtimer

2013-08-06 Thread Mike Galbraith
(CCs Intel folks) On Tue, 2013-08-06 at 09:29 +0200, Mike Galbraith wrote: > On Tue, 2013-07-30 at 11:35 +0200, Peter Zijlstra wrote: > > > It would be good if you could do what Thomas suggested and look at which > > timer is actually active during your workload. > > Rebuilding regression test

[PATCH v3 3/3] selftests: Add test of PMU instruction counting on powerpc

2013-08-06 Thread Michael Ellerman
This commit adds a test of instruction counting using the PMU on powerpc. Although the bulk of the code is architecture agnostic, the code needs to run a precisely sized loop which is implemented in assembler. Signed-off-by: Michael Ellerman --- v2,3: No change

[PATCH v3 1/3] selftests: Add infrastructure for powerpc selftests

2013-08-06 Thread Michael Ellerman
This commit adds a powerpc subdirectory to tools/testing/selftests, for tests that are powerpc specific. On other architectures nothing is built. The makefile supports cross compilation if the user sets ARCH and CROSS_COMPILE. Signed-off-by: Michael Ellerman --- v3: Add tags v2: No change

[PATCH v3 2/3] selftests: Add support files for powerpc tests

2013-08-06 Thread Michael Ellerman
This commit adds support code used by upcoming powerpc tests. Signed-off-by: Michael Ellerman --- v3: Print when the child dies due to a signal v2: Put back the SIGALARM handler to make the hang logic work. tools/testing/selftests/powerpc/harness.c | 105 ++

Re: [PATCH] Use gcc alias instead of assembler aliases for syscalls

2013-08-06 Thread Al Viro
On Tue, Aug 06, 2013 at 08:42:01AM +0200, Andi Kleen wrote: > On Mon, Aug 05, 2013 at 10:59:49PM -0700, Andrew Morton wrote: > > On Mon, 5 Aug 2013 15:09:35 -0700 Andi Kleen wrote: > > > > > From: Andi Kleen > > > > > > Use standard gcc __attribute__((alias(foo))) to define > > > the syscall

[PATCH -tip ] x86/insn: Add Haswell-introduced opcodes

2013-08-06 Thread Masami Hiramatsu
Add TSX-NI related instructions and new instructions to x86-opcode-map.txt according to the Intel(R) 64 and IA-32 Architectures Software Developer's Manual Vol2C (June, 2013). This also includes below updates. - Fix a typo of MWAIT (the lack of (11B)). - Change NOP Ev to prefetchw Ev - Add

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

2013-08-06 Thread Chen Gang
Improve the usage of return value 'result', so not only can make code clearer to readers, but also can improve the performance. Assign the return error code only when error occurs, so can save some instructions (some of them are inside looping). Rewrite the sysctl_getname() to make code clearer

Re: [PATCH V3]hrtimer: Fix a performance regression by disable reprogramming in remove_hrtimer

2013-08-06 Thread Mike Galbraith
On Tue, 2013-07-30 at 11:35 +0200, Peter Zijlstra wrote: > It would be good if you could do what Thomas suggested and look at which > timer is actually active during your workload. Rebuilding regression test trees, some pipe-test results... I'm missing mwait_idle() rather a lot on Q6600, and at

Re: [edk2] Corrupted EFI region

2013-08-06 Thread Laszlo Ersek
On 08/06/13 00:52, James Bottomley wrote: > On Mon, 2013-08-05 at 23:55 +0200, Laszlo Ersek wrote: >> On 08/05/13 23:41, Borislav Petkov wrote: >>> On Mon, Aug 05, 2013 at 02:37:08PM -0700, H. Peter Anvin wrote: All of this would be a non-problem if there weren't buggy implementations

Re: [PATCH v2 08/23] cgroup: pass around cgroup_subsys_state instead of cgroup in subsystem methods

2013-08-06 Thread Daniel Wagner
On 08/02/2013 10:24 PM, Tejun Heo wrote: cgroup is currently in the process of transitioning to using struct cgroup_subsys_state * as the primary handle instead of struct cgroup * in subsystem implementations for the following reasons. * With unified hierarchy, subsystems will be dynamically

Re: [PATCH v3 07/22] ARM: dts: Remove '0x's from Exynos4210 DTSI file

2013-08-06 Thread Lee Jones
On Tue, 06 Aug 2013, Kukjin Kim wrote: > On 07/25/13 00:09, Lee Jones wrote: > >... for the sake of consistency and assumed convention. > > > >Cc: Kukjin Kim > >Cc: linux-samsung-...@vger.kernel.org > >Signed-off-by: Lee Jones > > > >diff --git a/arch/arm/boot/dts/exynos4210.dtsi >

[3.10 regression] mei endless recursion

2013-08-06 Thread Stanislaw Gruszka
Hi We have a bug report when system fail to boot on 3.10 . It happens due to problem in mei which do endless calls: mei_reset -> mei_hbm_start_req -> mei_reset -> mei_hbm_start_req -> ... It did not happen on 3.9. Report is here: https://bugzilla.redhat.com/show_bug.cgi?id=991571 Stanislaw --

Re: [PATCH 22/23] cgroup: make cgroup_taskset deal with cgroup_subsys_state instead of cgroup

2013-08-06 Thread Daniel Wagner
Hi Tejun, On 08/01/2013 11:50 PM, Tejun Heo wrote: cgroup is in the process of converting to css (cgroup_subsys_state) from cgroup as the principal subsystem interface handle. This is mostly to prepare for the unified hierarchy support where css's will be created and destroyed dynamically but

Re: [PATCH 12/23] cgroup: pass around cgroup_subsys_state instead of cgroup in file methods

2013-08-06 Thread Daniel Wagner
Hi Tejun, On 08/01/2013 11:49 PM, Tejun Heo wrote: cgroup is currently in the process of transitioning to using struct cgroup_subsys_state * as the primary handle instead of struct cgroup. Please see the previous commit which converts the subsystem methods for rationale. This patch converts

[RFC PATCH 3/4] mm: add zbud flag to page flags

2013-08-06 Thread Krzysztof Kozlowski
Add PageZbud flag to page flags to distinguish pages allocated in zbud. Currently these pages do not have any flags set. Signed-off-by: Krzysztof Kozlowski --- include/linux/page-flags.h | 12 mm/page_alloc.c|3 +++ mm/zbud.c |4 3 files

[RFC PATCH 4/4] mm: reclaim zbud pages on migration and compaction

2013-08-06 Thread Krzysztof Kozlowski
Reclaim zbud pages during migration and compaction by unusing stored data. This allows adding__GFP_RECLAIMABLE flag when allocating zbud pages and effectively CMA pool can be used for zswap. zbud pages are not movable and are not stored under any LRU (except zbud's LRU). PageZbud flag is used in

[RFC PATCH 0/4] mm: reclaim zbud pages on migration and compaction

2013-08-06 Thread Krzysztof Kozlowski
Hi, Currently zbud pages are not movable and they cannot be allocated from CMA region. These patches try to address the problem by: 1. Adding a new form of reclaim of zbud pages. 2. Reclaiming zbud pages during migration and compaction. 3. Allocating zbud pages with __GFP_RECLAIMABLE flag. This

[RFC PATCH 2/4] mm: split code for unusing swap entries from try_to_unuse

2013-08-06 Thread Krzysztof Kozlowski
Move out the code for unusing swap entries from loop in try_to_unuse() to separate function: try_to_unuse_swp_entry(). Export this new function in swapfile.h just like try_to_unuse() is exported. Signed-off-by: Krzysztof Kozlowski --- include/linux/swapfile.h |2 + mm/swapfile.c

[RFC PATCH 1/4] zbud: use page ref counter for zbud pages

2013-08-06 Thread Krzysztof Kozlowski
Use page reference counter for zbud pages. The ref counter replaces zbud_header.under_reclaim flag and ensures that zbud page won't be freed when zbud_free() is called during reclaim. It allows implementation of additional reclaim paths. The page count is incremented when: - a handle is created

Re: [PATCH] Use gcc alias instead of assembler aliases for syscalls

2013-08-06 Thread Andi Kleen
On Mon, Aug 05, 2013 at 10:59:49PM -0700, Andrew Morton wrote: > On Mon, 5 Aug 2013 15:09:35 -0700 Andi Kleen wrote: > > > From: Andi Kleen > > > > Use standard gcc __attribute__((alias(foo))) to define > > the syscall aliases instead of custom assembler macros. > > > > This is far cleaner,

linux-next: Tree for Aug 6

2013-08-06 Thread Stephen Rothwell
Hi all, Changes since 20130805: The ext4 tree still has its build failure so I used the version from next-20130726. The akpm tree gained a conflict against the drm-intel tree. I have created today's linux-next tree

Re: [patch 1/7] arch: mm: remove obsolete init OOM protection

2013-08-06 Thread Vineet Gupta
Hi Johannes, Thk for the cleanup. On 08/03/2013 10:29 PM, Johannes Weiner wrote: > Back before smart OOM killing, when faulting tasks where killed > directly on allocation failures, the arch-specific fault handlers > needed special protection for the init process. > > Now that all fault

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

2013-08-06 Thread Kees Cook
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 > named interfaces. The name of the

[PATCH] UBI: Fastmap: Fix memory leak

2013-08-06 Thread Richard Weinberger
From: Thomas Weber Tested with linux-v3.10-rc3 on Devkit8000. The discussion about this patch can be found: https://lkml.org/lkml/2013/5/2/118 This patches fixes the following bug for me: When CONFIG_DEBUG_VM is active and making ubiattach ... ; ubidetach...; ubiattach ... the second ubiattach

Re: [PATCH v2 1/5] ia64: add early_memremap() alias for early_ioremap()

2013-08-06 Thread Matt Fleming
On Mon, 05 Aug, at 09:10:19AM, Tony Luck wrote: > Acked-by: Tony Luck > > [Cut & paste this ack to other parts of the series that touch ia64] Will do, thanks Tony! -- Matt Fleming, Intel Open Source Technology Center -- To unsubscribe from this list: send the line "unsubscribe linux-kernel"

Re: [PATCH] RFC: perf, tools: Move gtk browser into separate perfgtk executable

2013-08-06 Thread Christoph Hellwig
On Mon, Aug 05, 2013 at 11:08:57AM +0200, Ingo Molnar wrote: > You never replied to the original counter-arguments, such as this one from > Linus: > > http://article.gmane.org/gmane.linux.kernel/849965 The only thing Linus sais is that it's trivial to generate a subpackage, and that opofile

Re: [PATCH] ipvs: fixed style errors in ip_vs_dh

2013-08-06 Thread Simon Horman
On Thu, Jul 11, 2013 at 10:15:44PM +0300, Dragos Foianu wrote: > found with the help of checkpatch.pl Thanks for your contribution and sorry for taking so long to look at it. While in principle I have no problem with these changes. I would like them to be be accompanied by more descriptive

Re: [PATCH] ipvs: fixed spacing at for statements

2013-08-06 Thread Simon Horman
On Thu, Jul 11, 2013 at 09:45:42AM +0300, Dragos Foianu wrote: > found using checkpatch.pl This seems reasonable to me. I will queue it up in the ipvs-next tree. > > Signed-off-by: Dragos Foianu > --- > net/netfilter/ipvs/ip_vs_lblcr.c |8 > 1 file changed, 4 insertions(+), 4

Re: [PATCH RESEND] ARM: dts: Add USBPHY nodes to Exynos4x12

2013-08-06 Thread Sachin Kamat
Hi Dongjin, On 5 August 2013 23:48, Dongjin Kim wrote: > This patch adds device nodes for USBPHY to Exynos4x12. > > CC: Sachin Kamat > Signed-off-by: Dongjin Kim > --- > arch/arm/boot/dts/exynos4x12.dtsi | 18 ++ > 1 file changed, 18 insertions(+) > > diff --git

Re: [PATCH] Use gcc alias instead of assembler aliases for syscalls

2013-08-06 Thread Andrew Morton
On Mon, 5 Aug 2013 15:09:35 -0700 Andi Kleen wrote: > From: Andi Kleen > > Use standard gcc __attribute__((alias(foo))) to define > the syscall aliases instead of custom assembler macros. > > This is far cleaner, and also fixes my LTO kernel build. I wonder what gcc version this was added

Re: [PATCH] Use gcc alias instead of assembler aliases for syscalls

2013-08-06 Thread Andrew Morton
On Mon, 5 Aug 2013 15:09:35 -0700 Andi Kleen a...@firstfloor.org wrote: From: Andi Kleen a...@linux.intel.com Use standard gcc __attribute__((alias(foo))) to define the syscall aliases instead of custom assembler macros. This is far cleaner, and also fixes my LTO kernel build. I wonder

Re: [PATCH RESEND] ARM: dts: Add USBPHY nodes to Exynos4x12

2013-08-06 Thread Sachin Kamat
Hi Dongjin, On 5 August 2013 23:48, Dongjin Kim tobet...@gmail.com wrote: This patch adds device nodes for USBPHY to Exynos4x12. CC: Sachin Kamat sachin.ka...@linaro.org Signed-off-by: Dongjin Kim tobet...@gmail.com --- arch/arm/boot/dts/exynos4x12.dtsi | 18 ++ 1 file

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