RE: Dangerous devm_request_irq() conversions

2013-02-21 Thread Jingoo Han
On Friday, February 22, 2013 4:27 PM, Dmitry Torokhov wrote: > On Fri, Feb 22, 2013 at 04:12:36PM +0900, Jingoo Han wrote: > > On Friday, February 22, 2013 3:54 PM, Dmitry Torokhov wrote: > > > > > > Hi, > > > > > > It looks like a whole slew of devm_request_irq() conversions just got > > >

Re: [PATCH 01/35] mfd: ab8500-gpadc: Implemented suspend/resume

2013-02-21 Thread Lee Jones
On Thu, 21 Feb 2013, Ulf Hansson wrote: > On 20 February 2013 14:19, Mark Brown > wrote: > > On Fri, Feb 15, 2013 at 12:56:32PM +, Lee Jones wrote: > > > >> +static int ab8500_gpadc_suspend(struct device *dev) > >> +{ > >> + struct ab8500_gpadc *gpadc = dev_get_drvdata(dev); > >> + > >>

[PATCH] PM: align buffers for LZO compression

2013-02-21 Thread Markus F.X.J. Oberhumer
Hi, for performance reasons I'd strongly suggest that you explicitly align all buffers passed to the LZO compress and decompress functions. Below is a small (and completely untested!) patch, but I think you get the idea. BTW, it might be even more beneficial (esp. for NUMA systems) to align

Re: [PATCH 1/2] cpustat: use accessor functions for get/set/add

2013-02-21 Thread Viresh Kumar
On 22 February 2013 12:47, Amit Kucheria wrote: > On Fri, Feb 22, 2013 at 11:51 AM, Viresh Kumar > wrote: >> BTW, i don't see kcpustat_cpu() used in >> >> kernel/sched/core.c| 12 +--- >> kernel/sched/cputime.c | 29 +-- >> >> I

[PATCH] acpi: sleep: Avoid interleaved message on errors

2013-02-21 Thread Joe Perches
Got this dmesg log on an Acer Aspire 725. [0.256351] ACPI: (supports S0ACPI Exception: AE_NOT_FOUND, While evaluating Sleep State [\_S1_] (20130117/hwxface-568) [0.256373] ACPI Exception: AE_NOT_FOUND, While evaluating Sleep State [\_S2_] (20130117/hwxface-568) [0.256391] S3 S4 S5)

[PATCH 4/8] ARM: PRIMA2: Divorce timer-marco from local timer API

2013-02-21 Thread Stephen Boyd
Separate the marco local timers from the local timer API. This will allow us to remove ARM local timer support in the near future and gets us closer to moving this driver to drivers/clocksource. Cc: Barry Song Signed-off-by: Stephen Boyd --- arch/arm/mach-prima2/timer-marco.c | 98

Re: [PATCH v4] mfd: syscon: Add non-DT support

2013-02-21 Thread Thierry Reding
On Fri, Feb 22, 2013 at 03:13:12PM +0800, Dong Aisheng wrote: > On Fri, Feb 22, 2013 at 11:01:18AM +0400, Alexander Shiyan wrote: > > > On Thu, Feb 21, 2013 at 07:29:02PM +0400, Alexander Shiyan wrote: > > > > This patch allow using syscon driver from the platform data, i.e. > > > > possibility

[PATCH 5/8] ARM: MSM: Divorce msm_timer from local timer API

2013-02-21 Thread Stephen Boyd
Separate the msm_timer from the local timer API. This will allow us to remove ARM local timer support in the near future and gets us closer to moving this driver to drivers/clocksource. Cc: David Brown Cc: Daniel Walker Cc: Bryan Huntsman Signed-off-by: Stephen Boyd ---

[PATCH 3/8] ARM: EXYNOS4: Divorce mct from local timer API

2013-02-21 Thread Stephen Boyd
Separate the mct local timers from the local timer API. This will allow us to remove ARM local timer support in the near future and gets us closer to moving this driver to drivers/clocksource. Cc: Kukjin Kim Signed-off-by: Stephen Boyd --- arch/arm/mach-exynos/mct.c | 53

[PATCH 2/8] ARM: smp_twd: Divorce smp_twd from local timer API

2013-02-21 Thread Stephen Boyd
Separate the smp_twd timers from the local timer API. This will allow us to remove ARM local timer support in the near future and gets us closer to moving this driver to drivers/clocksource. Cc: Russell King Signed-off-by: Stephen Boyd --- arch/arm/kernel/smp_twd.c | 48

[PATCH 8/8] ARM: smp: Remove local timer API

2013-02-21 Thread Stephen Boyd
There are no more users of this API, remove it. Cc: Russell King Signed-off-by: Stephen Boyd --- arch/arm/Kconfig | 12 +-- arch/arm/include/asm/localtimer.h | 34 arch/arm/kernel/smp.c | 67 ++-

[PATCH 6/8] clocksource: time-armada-370-xp: Fix sparse warning

2013-02-21 Thread Stephen Boyd
drivers/clocksource/time-armada-370-xp.c:217:13: warning: symbol 'armada_370_xp_timer_init' was not declared. Should it be static? Also remove the __init marking in the prototype as it's unnecessary and drop the init.h file. Cc: Gregory CLEMENT Signed-off-by: Stephen Boyd ---

[PATCH 7/8] clocksource: time-armada-370-xp: Divorce from local timer API

2013-02-21 Thread Stephen Boyd
Separate the armada 370xp local timers from the local timer API. This will allow us to remove ARM local timer support in the near future and makes this driver multi-architecture friendly. Cc: Gregory CLEMENT Signed-off-by: Stephen Boyd --- drivers/clocksource/time-armada-370-xp.c | 85

[PATCH 1/8] ARM: smp: Lower rating of dummy broadcast device

2013-02-21 Thread Stephen Boyd
In the near future the dummy broadcast device will always be registered with the clockevent core. If the rating of the dummy is higher than the rating of the real clockevent the clockevents core will try to replace the real clockevent with the dummy broadcast. We don't want this to happen, so

[PATCH 0/8] Remove ARM local timer API

2013-02-21 Thread Stephen Boyd
and SMP situations. Right now these drivers have to ignore the timer setup callback on the boot CPU to avoid registering clockevents twice. This is not very symmetric and causes convuluted code that does the same thing in two places. Patches based on linux-next-20130221. Mostly compile tested as I

Re: [PATCH v2] memcg: Add memory.pressure_level events

2013-02-21 Thread Minchan Kim
On Thu, Feb 21, 2013 at 10:55:52PM -0800, Anton Vorontsov wrote: > On Fri, Feb 22, 2013 at 08:56:08AM +0900, Minchan Kim wrote: > > [...] The my point is that you have a plan to support? Why I have a > > question is that you said your goal is to replace lowmemory killer > > In short: yes, of

Re: Dangerous devm_request_irq() conversions

2013-02-21 Thread Dmitry Torokhov
On Fri, Feb 22, 2013 at 04:12:36PM +0900, Jingoo Han wrote: > On Friday, February 22, 2013 3:54 PM, Dmitry Torokhov wrote: > > > > Hi, > > > > It looks like a whole slew of devm_request_irq() conversions just got > > applied to mainline and many of them are quite broken. > > > > Consider

Re: [PATCH 1/2] cpustat: use accessor functions for get/set/add

2013-02-21 Thread Amit Kucheria
On Fri, Feb 22, 2013 at 11:51 AM, Viresh Kumar wrote: > On Fri, Feb 22, 2013 at 11:26 AM, Kevin Hilman wrote: >> Add some accessor functions in order to facilitate the conversion to >> atomic reads/writes of cpustat values. >> >> Signed-off-by: Kevin Hilman >> --- >>

Re: [PATCH v4] mfd: syscon: Add non-DT support

2013-02-21 Thread Dong Aisheng
On Fri, Feb 22, 2013 at 11:01:18AM +0400, Alexander Shiyan wrote: > > On Thu, Feb 21, 2013 at 07:29:02PM +0400, Alexander Shiyan wrote: > > > This patch allow using syscon driver from the platform data, i.e. > > > possibility using driver on systems without oftree support. > > > For search syscon

Re: [PATCH 2/7] ksm: treat unstable nid like in stable tree

2013-02-21 Thread Ric Mason
On 02/21/2013 04:20 PM, Hugh Dickins wrote: An inconsistency emerged in reviewing the NUMA node changes to KSM: when meeting a page from the wrong NUMA node in a stable tree, we say that it's okay for comparisons, but not as a leaf for merging; whereas when meeting a page from the wrong NUMA

Re: Dangerous devm_request_irq() conversions

2013-02-21 Thread Jingoo Han
On Friday, February 22, 2013 3:54 PM, Dmitry Torokhov wrote: > > Hi, > > It looks like a whole slew of devm_request_irq() conversions just got > applied to mainline and many of them are quite broken. > > Consider fd5231ce336e038037b4f0190a6838bdd6e17c6d or >

Re: [PATCH RFC 4/5] UBIFS: Add security.* XATTR support for the UBIFS

2013-02-21 Thread Artem Bityutskiy
OK, the lockdep warnings clearly tell the reason: CPU0CPU1 lock(>ui_mutex); lock(>s_type->i_mutex_key#10); lock(>ui_mutex); lock(>s_type->i_mutex_key#10); And

[PATCH] fusb300_udc: modify stall clear and idma reset procedure

2013-02-21 Thread Yuan-Hsin Chen
From: Yuan-Hsin Chen Due to fusb300 controller modification, stall clear procedure should be modified consistantly. This patch also fixes software bugs: only enter IDMA_RESET when the condition matched and disable corresponding PRD interrupt in IDMA_RESET. Signed-off-by: Yuan-Hsin Chen ---

Re[2]: [PATCH v4] mfd: syscon: Add non-DT support

2013-02-21 Thread Alexander Shiyan
> On Thu, Feb 21, 2013 at 07:29:02PM +0400, Alexander Shiyan wrote: > > This patch allow using syscon driver from the platform data, i.e. > > possibility using driver on systems without oftree support. > > For search syscon device from the client drivers, > > "syscon_regmap_lookup_by_pdevname"

Re: [PATCH v2] memcg: Add memory.pressure_level events

2013-02-21 Thread Anton Vorontsov
On Fri, Feb 22, 2013 at 08:56:08AM +0900, Minchan Kim wrote: > [...] The my point is that you have a plan to support? Why I have a > question is that you said your goal is to replace lowmemory killer In short: yes, of course, if the non-memcg interface will be in demand. > but android don't have

[PATCH] ARM: EXYNOS: Keep USB related LDOs always active on Origen

2013-02-21 Thread Tushar Behera
LDO3 and LDO8 are used for powering both device and host phy controllers. These regulators are not handled in USB host driver. Hence we get unexpected behaviour when the regulators are disabled elsewhere. It would be best to keep these regulators always on. Signed-off-by: Tushar Behera ---

[GIT PULL] arch/arc for v3.9-rc1

2013-02-21 Thread Vineet Gupta
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 Hi Linus, I would like to introduce the Linux port to ARC Processors (from Synopsys) for 3.9-rc1. The patch-set has been discussed on the public lists since Nov and has received a fair bit of review, specially from Arnd, tglx, Al and other subsystem

Re: [PATCH v4] mfd: syscon: Add non-DT support

2013-02-21 Thread Dong Aisheng
On Thu, Feb 21, 2013 at 07:29:02PM +0400, Alexander Shiyan wrote: > This patch allow using syscon driver from the platform data, i.e. > possibility using driver on systems without oftree support. > For search syscon device from the client drivers, > "syscon_regmap_lookup_by_pdevname" function was

Re: Origen board hang with functionfs

2013-02-21 Thread Tushar Behera
On 02/22/2013 12:03 AM, John Stultz wrote: > On 02/20/2013 06:01 PM, John Stultz wrote: >> Hey Kukjin, Andrzej, >> I recently started playing around with functionfs, and have >> noticed some strange behavior with my origen board. >> >> If I enable the FunctionFS gadget driver, I see the board

Dangerous devm_request_irq() conversions

2013-02-21 Thread Dmitry Torokhov
Hi, It looks like a whole slew of devm_request_irq() conversions just got applied to mainline and many of them are quite broken. Consider fd5231ce336e038037b4f0190a6838bdd6e17c6d or c1879fe80c61f3be6f2ddb82509c2e7f92a484fe: the drivers udsed first to free IRQ and then unregister the

Re: [GIT PATCH] USB patches for 3.9-rc1

2013-02-21 Thread Thierry Reding
On Thu, Feb 21, 2013 at 01:58:39PM -0800, Greg KH wrote: > On Thu, Feb 21, 2013 at 12:25:24PM -0800, Linus Torvalds wrote: > > On Thu, Feb 21, 2013 at 10:40 AM, Greg KH > > wrote: > > > > > > USB patches for 3.9-rc1 > > > > > > Here's the big USB merge for 3.9-rc1 > > > > > > Nothing major, lots

Re: [RFC PATCH v3 0/3] sched: simplify the select_task_rq_fair()

2013-02-21 Thread Michael Wang
On 02/22/2013 01:02 PM, Mike Galbraith wrote: > On Fri, 2013-02-22 at 10:36 +0800, Michael Wang wrote: >> On 02/21/2013 05:43 PM, Mike Galbraith wrote: >>> On Thu, 2013-02-21 at 17:08 +0800, Michael Wang wrote: >>> But is this patch set really cause regression on your Q6600? It may

Re: sched: Fix signedness bug in yield_to()

2013-02-21 Thread Raghavendra KT
On Fri, Feb 22, 2013 at 4:56 AM, Marcelo Tosatti wrote: > On Thu, Feb 21, 2013 at 09:56:54AM +0100, Ingo Molnar wrote: >> >> * Shuah Khan wrote: >> >> > On Tue, Feb 19, 2013 at 7:27 PM, Linux Kernel Mailing List >> > wrote: >> > > Gitweb: >> > >

Resend: [patch] hsi : Avoid race condition between HSI controller and HSI client when system restart and power down

2013-02-21 Thread xtu4
Avoid race condition between HSI controller and HSI client when system restart and power down hsi_isr_tasklet disabled in HSI_controller exit, but before HSI controller exit, HSI client will cleanup, this cleanup will destroy the spinlock used by the hsi_isr_tasklet,so if after HSI client

Re: Re: [PATCH] PM / devfreq: fix missing unlock on error in exynos4_busfreq_pm_notifier_event()

2013-02-21 Thread MyungJoo Ham
> On 12:33-20130222, Wei Yongjun wrote: > > From: Wei Yongjun > > > > Add the missing unlock before return from function > > exynos4_busfreq_pm_notifier_event() in the error > > handling case. > > > > This issue introduced by commit 8fa938 > > (PM / devfreq: exynos4_bus: honor RCU lock usage) >

[patch] hsi : Avoid race condition between HSI controller and HSI client when system restart and power down

2013-02-21 Thread xtu4
Avoid race condition between HSI controller and HSI client when system restart and power down hsi_isr_tasklet disabled in HSI_controller exit, but before HSI controller exit, HSI client will cleanup, this cleanup will destroy the spinlock used by the hsi_isr_tasklet,so if after HSI client

Re: [PATCH v2] staging: comedi: drivers: usbduxsigma.c: fix DMA buffers on stack

2013-02-21 Thread Dan Carpenter
Looks good. Reviewed-by: Dan Carpenter regards, dan carpenter -- 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 http://vger.kernel.org/majordomo-info.html Please read the FAQ at

Re: [PATCH 1/2] cpustat: use accessor functions for get/set/add

2013-02-21 Thread Viresh Kumar
On Fri, Feb 22, 2013 at 11:26 AM, Kevin Hilman wrote: > Add some accessor functions in order to facilitate the conversion to > atomic reads/writes of cpustat values. > > Signed-off-by: Kevin Hilman > --- > drivers/cpufreq/cpufreq_governor.c | 18 - >

Re: [RFC PATCH v3 0/3] sched: simplify the select_task_rq_fair()

2013-02-21 Thread Mike Galbraith
On Fri, 2013-02-22 at 14:06 +0800, Michael Wang wrote: > On 02/22/2013 01:08 PM, Mike Galbraith wrote: > > On Fri, 2013-02-22 at 10:37 +0800, Michael Wang wrote: > > > >> According to the testing result, I could not agree this purpose of > >> wake_affine() benefit us, but I'm sure that

Re: [RFC PATCH v3 0/3] sched: simplify the select_task_rq_fair()

2013-02-21 Thread Mike Galbraith
On Fri, 2013-02-22 at 13:26 +0800, Michael Wang wrote: > Just confirm that I'm not on the wrong way, did the 1:N mode here means > 1 task forked N threads, and child always talk with father? Yes, one server, many clients. -Mike -- To unsubscribe from this list: send the line "unsubscribe

[PATCH v2] staging: comedi: drivers: usbduxsigma.c: fix DMA buffers on stack

2013-02-21 Thread Kumar Amit Mehta
This patch fixes an instance of DMA buffer on stack(being passed to usb_control_msg)for the USB-DUXsigma Board driver. Found using smatch. Signed-off-by: Kumar Amit Mehta --- drivers/staging/comedi/drivers/usbduxsigma.c | 27 -- 1 file changed, 17 insertions(+), 10

[patch] remoteproc: off by one in rproc_virtio_new_vringh()

2013-02-21 Thread Dan Carpenter
It should be >= ARRAY_SIZE() instead of > ARRAY_SIZE() because it is an index. Signed-off-by: Dan Carpenter diff --git a/drivers/remoteproc/remoteproc_virtio.c b/drivers/remoteproc/remoteproc_virtio.c index dba33ff..b5e3af5 100644 --- a/drivers/remoteproc/remoteproc_virtio.c +++

Re: [RFC PATCH v3 0/3] sched: simplify the select_task_rq_fair()

2013-02-21 Thread Michael Wang
On 02/22/2013 01:08 PM, Mike Galbraith wrote: > On Fri, 2013-02-22 at 10:37 +0800, Michael Wang wrote: > >> According to the testing result, I could not agree this purpose of >> wake_affine() benefit us, but I'm sure that wake_affine() is a terrible >> performance killer when system is busy. > >

Re: [RFC/PATCH 2/5] kernel_cpustat: convert to atomic 64-bit accessors

2013-02-21 Thread Kevin Hilman
Frederic Weisbecker writes: > 2013/2/21 Frederic Weisbecker : >> 2013/2/21 Kevin Hilman : >>> Subject: [PATCH 2/5] kernel_cpustat: convert to atomic 64-bit accessors >>> >>> Use the atomic64_* accessors for all the kernel_cpustat fields to >>> ensure atomic access on non-64 bit platforms. >>>

[PATCH 1/2] cpustat: use accessor functions for get/set/add

2013-02-21 Thread Kevin Hilman
Add some accessor functions in order to facilitate the conversion to atomic reads/writes of cpustat values. Signed-off-by: Kevin Hilman --- arch/s390/appldata/appldata_os.c | 16 +++ drivers/cpufreq/cpufreq_governor.c | 18 - drivers/cpufreq/cpufreq_ondemand.c | 2

Re: [PATCH v3 linux-next] cpufreq: ondemand: Calculate gradient of CPU load to early increase frequency

2013-02-21 Thread Viresh Kumar
On Fri, Feb 22, 2013 at 7:26 AM, Viresh Kumar wrote: > On 21 February 2013 23:09, Stratos Karafotis wrote: >> Instead of checking only the absolute value of CPU load_freq to increase >> frequency, we detect forthcoming CPU load rise and increase frequency >> earlier. >> >> Every sampling rate,

[PATCH 2/2] cpustat: convert to atomic operations

2013-02-21 Thread Kevin Hilman
For non 64-bit platforms, convert cpustat fields to atomic64 type so reads and udpates of cpustats are atomic on those platforms as well. For 64-bit platforms, the cpustat field is left as u64 because on 64-bit, using atomic64_add will have the additional overhead of a lock. We could also have

[PATCH 0/2] cpustat: use atomic operations to read/update stats

2013-02-21 Thread Kevin Hilman
On 64-bit platforms, reads/writes of the various cpustat fields are atomic due to native 64-bit loads/stores. However, on non 64-bit platforms, reads/writes of the cpustat fields are not atomic and could lead to inconsistent statistics. This problem was originally reported by Frederic Weisbecker

[rebased][PATCH 0/4] acpi: do some changes for numa info

2013-02-21 Thread liguang
just do some trivial changes to make acpi's numa info operation more cleaner. ChangeLog v3->v4 1.fix srat_disabled function spotted by Yasuaki Ishimatsu v2->v3 1. rebase on linux-next 2. bring back lost Makefile changes spotted by David Rientjes spotted by Yasuaki Ishimatsu v1->v2 1.

[rebased][PATCH 2/4] numa: avoid export acpi_numa variable

2013-02-21 Thread liguang
acpi_numa is used to prevent srat table being parsed, seems a little miss-named, if 'noacpi' was specified by cmdline and CONFIG_ACPI_NUMA was enabled, acpi_numa will be operated directly from everywhere it needed to disable/enable numa in acpi mode which was a bad thing, so, try to export a

[rebased][PATCH 4/4] remove include asm/acpi.h in process_driver.c

2013-02-21 Thread liguang
process_driver.c include linux/acpi.h which already include asm/acpi.h, so remove it. Reviewed-by: Yasuaki Ishimatsu Acked-by: David Rientjes Signed-off-by: liguang --- drivers/acpi/processor_driver.c |1 - 1 files changed, 0 insertions(+), 1 deletions(-) diff --git

[rebased][PATCH 1/4] acpi: move x86/mm/srat.c to x86/kernel/acpi/srat.c

2013-02-21 Thread liguang
srat table should present only on acpi domain, seems mm/ is not the right place for it. Reviewed-by: Yasuaki Ishimatsu Signed-off-by: liguang --- arch/x86/kernel/acpi/Makefile |1 + arch/x86/kernel/acpi/srat.c | 278 + arch/x86/mm/Makefile

[rebased][PATCH 3/4] acpi: add clock_domain field to acpi_srat_cpu_affinity

2013-02-21 Thread liguang
according to ACPI SPEC v5.0, page 152, 5.2.16.1 Processor Local APIC/SAPIC Affinity Structure, the last member of it is clock_domain. Reviewed-by: Yasuaki Ishimatsu Acked-by: David Rientjes Signed-off-by: liguang --- include/acpi/actbl1.h |2 +- 1 files changed, 1 insertions(+), 1

[PATCH] lib: devres: Fix misplaced #endif

2013-02-21 Thread Jingoo Han
A misplaced #endif causes link errors related to pcim_*() functions. Signed-off-by: Jingoo Han --- lib/devres.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/lib/devres.c b/lib/devres.c index 88ad759..8235331 100644 --- a/lib/devres.c +++ b/lib/devres.c @@ -227,6

Re: thermal governor: does it actually work??

2013-02-21 Thread Peter Feuerer
Adding Boris, sorry, I can't do anything currently, I'm down with influenza. kind regards, --peter; Zhang Rui writes: On Thu, 2013-02-14 at 16:32 +0100, Andreas Mohr wrote: For me after having loaded acerhdf the fan never stops (with kernelmode active), despite staying safely below trip

[GIT PULL] Update LZO compression code for v3.9

2013-02-21 Thread Markus F.X.J. Oberhumer
Hi Linus, please pull my "lzo-update" branch from git://github.com/markus-oberhumer/linux.git lzo-update You can also browse the branch at https://github.com/markus-oberhumer/linux/compare/lzo-update The LZO update actually had been approved by akpm for a 3.7 merge and is available in

Re: [RFC PATCH v3 0/3] sched: simplify the select_task_rq_fair()

2013-02-21 Thread Michael Wang
On 02/22/2013 01:02 PM, Mike Galbraith wrote: > On Fri, 2013-02-22 at 10:36 +0800, Michael Wang wrote: >> On 02/21/2013 05:43 PM, Mike Galbraith wrote: >>> On Thu, 2013-02-21 at 17:08 +0800, Michael Wang wrote: >>> But is this patch set really cause regression on your Q6600? It may

Re: linux-next: build failure after merge of the final tree (drm tree related)

2013-02-21 Thread Rob Clark
Hmm, maybe DRM_GEM_CMA_HELPER should depend on ARM (or !PPC)? Or maybe there is an alternative fxn to use on other archs? In truth, it is fine to make TILCDC depend on ARM, as it wouldn't be used on any other platform (today.. until TI comes up with some crazy chip w/ some TI DSP plus display

Re: [RFC PATCH v3 0/3] sched: simplify the select_task_rq_fair()

2013-02-21 Thread Mike Galbraith
On Fri, 2013-02-22 at 10:37 +0800, Michael Wang wrote: > According to the testing result, I could not agree this purpose of > wake_affine() benefit us, but I'm sure that wake_affine() is a terrible > performance killer when system is busy. (hm, result is singular.. pgbench in 1:N mode only?) --

Re: [RFC PATCH v3 1/3] sched: schedule balance map foundation

2013-02-21 Thread Michael Wang
On 02/22/2013 12:46 PM, Alex Shi wrote: > On 02/22/2013 12:19 PM, Michael Wang wrote: >> Why not seek other way to change O(n^2) to O(n)? Access 2G memory is unbelievable performance cost. >> Not access 2G memory, but (2G / 16K) memory, the sbm size is O(N). >> >> And please notice

Re: [RFC PATCH v3 0/3] sched: simplify the select_task_rq_fair()

2013-02-21 Thread Mike Galbraith
On Fri, 2013-02-22 at 10:36 +0800, Michael Wang wrote: > On 02/21/2013 05:43 PM, Mike Galbraith wrote: > > On Thu, 2013-02-21 at 17:08 +0800, Michael Wang wrote: > > > >> But is this patch set really cause regression on your Q6600? It may > >> sacrificed some thing, but I still think it will

[GIT PULL] x86/microcode for v3.9-rc1

2013-02-21 Thread H. Peter Anvin
Hi Linus, This patchset lets us update the CPU microcode very, very early in initialization if the BIOS fails to do so (never happens, right?) This is handy for dealing with things like the Atom erratum where we have to run without PSE because microcode loading happens too late. As I mentioned

[PATCH] kexec: avoid freeing NULL pointer in function kimage_crash_alloc

2013-02-21 Thread Zhang Yanfei
Though there is no error if we free a NULL pointer, I think we could avoid this behaviour. Change the code a little in kimage_crash_alloc() could avoid this kind of unnecessary free. Cc: "Eric W. Biederman" Cc: Andrew Morton Signed-off-by: Zhang Yanfei --- kernel/kexec.c | 15

Re: [PATCH] PM / devfreq: fix missing unlock on error in exynos4_busfreq_pm_notifier_event()

2013-02-21 Thread Nishanth Menon
On 12:33-20130222, Wei Yongjun wrote: > From: Wei Yongjun > > Add the missing unlock before return from function > exynos4_busfreq_pm_notifier_event() in the error > handling case. > > This issue introduced by commit 8fa938 > (PM / devfreq: exynos4_bus: honor RCU lock usage) > > Signed-off-by:

Re: [RFC PATCH v3 1/3] sched: schedule balance map foundation

2013-02-21 Thread Alex Shi
On 02/22/2013 12:19 PM, Michael Wang wrote: > >> > Why not seek other way to change O(n^2) to O(n)? >> > >> > Access 2G memory is unbelievable performance cost. > Not access 2G memory, but (2G / 16K) memory, the sbm size is O(N). > > And please notice that on 16k cpus system, topology will be

[PATCH] kexec: fix memory leak in function kimage_normal_alloc

2013-02-21 Thread Zhang Yanfei
If kimage_normal_alloc() fails to alloc pages for image->swap_page, it should call kimage_free_page_list() to free allocated pages in image->control_pages list before it frees image. Cc: "Eric W. Biederman" Cc: Andrew Morton Cc: Sasha Levin Signed-off-by: Zhang Yanfei --- kernel/kexec.c |

[PATCH] PM / devfreq: fix missing unlock on error in exynos4_busfreq_pm_notifier_event()

2013-02-21 Thread Wei Yongjun
From: Wei Yongjun Add the missing unlock before return from function exynos4_busfreq_pm_notifier_event() in the error handling case. This issue introduced by commit 8fa938 (PM / devfreq: exynos4_bus: honor RCU lock usage) Signed-off-by: Wei Yongjun --- drivers/devfreq/exynos4_bus.c | 1 + 1

[GIT PULL] slave-dmaengine updates

2013-02-21 Thread Vinod Koul
Hi Linus, Here is the slave-dmaengine updates. This is fairly big pull by my standards as I had missed last merge window. So we have the support for device tree for slave-dmaengine, large updates to dw_dmac driver from Andy for reusing on different architectures. Along with this we have fixes

Re: [PATCH 1/7] ksm: add some comments

2013-02-21 Thread Ric Mason
On 02/21/2013 04:19 PM, Hugh Dickins wrote: Added slightly more detail to the Documentation of merge_across_nodes, a few comments in areas indicated by review, and renamed get_ksm_page()'s argument from "locked" to "lock_it". No functional change. Signed-off-by: Hugh Dickins ---

Re: [RFC PATCH v3 1/3] sched: schedule balance map foundation

2013-02-21 Thread Michael Wang
On 02/22/2013 11:33 AM, Alex Shi wrote: > On 02/22/2013 10:53 AM, Michael Wang wrote: >> And the final cost is 3000 int and 103 pointer, and some padding, >> but won't bigger than 10M, not a big deal for a system with 1000 cpu >> too. Maybe, but quadric stuff should

linux-next: build failure after merge of the final tree (drm tree related)

2013-02-21 Thread Stephen Rothwell
Hi all, After merging the final tree, today's linux-next build (powerpc allyesconfig) failed like this: drivers/gpu/drm/drm_gem_cma_helper.c: In function 'drm_gem_cma_buf_destroy': drivers/gpu/drm/drm_gem_cma_helper.c:38:2: error: implicit declaration of function 'dma_free_writecombine'

Re: [PATCH] staging/zcache: Fix/improve zcache writeback code, tie to a config option

2013-02-21 Thread Ric Mason
On 02/07/2013 02:27 AM, Dan Magenheimer wrote: It was observed by Andrea Arcangeli in 2011 that zcache can get "full" and there must be some way for compressed swap pages to be (uncompressed and then) sent through to the backing swap disk. A prototype of this functionality, called "unuse", was

Re: [PATCH 0/4] dcache: make Oracle more scalable on large systems

2013-02-21 Thread Waiman Long
On 02/21/2013 07:13 PM, Andi Kleen wrote: Dave Chinner writes: On Tue, Feb 19, 2013 at 01:50:55PM -0500, Waiman Long wrote: It was found that the Oracle database software issues a lot of call to the seq_path() kernel function which translates a (dentry, mnt) pair to an absolute path. The

RE: [PATCH RFC] video: Add Hyper-V Synthetic Video Frame Buffer Driver

2013-02-21 Thread Haiyang Zhang
> From: Olaf Hering > Sent: Thursday, February 21, 2013 10:53 AM > To: Haiyang Zhang > Cc: florianschandi...@gmx.de; linux-fb...@vger.kernel.org; KY Srinivasan; > jasow...@redhat.com; linux-kernel@vger.kernel.org; > de...@linuxdriverproject.org > Subject: Re: [PATCH RFC] video: Add Hyper-V

Re: [PATCH] kexec: prevent double free on image allocation failure

2013-02-21 Thread Zhang Yanfei
于 2013年02月22日 11:41, Sasha Levin 写道: > On 02/21/2013 09:46 PM, Zhang Yanfei wrote: >> 于 2013年02月22日 09:55, Eric W. Biederman 写道: >>> Sasha Levin writes: >>> If kimage_normal_alloc() fails to initialize an allocated kimage, it will free the image but would still set 'rimage', as a

Re: [PATCH] iommu: making IOMMU sysfs nodes API public

2013-02-21 Thread Alex Williamson
On Fri, 2013-02-22 at 11:04 +1100, David Gibson wrote: > On Tue, Feb 19, 2013 at 01:11:51PM -0700, Alex Williamson wrote: > > On Tue, 2013-02-19 at 18:38 +1100, David Gibson wrote: > > > On Mon, Feb 18, 2013 at 10:24:00PM -0700, Alex Williamson wrote: > > > > On Mon, 2013-02-18 at 17:15 +1100,

Re: [GIT] Networking

2013-02-21 Thread Mark Lord
On 13-02-21 09:26 PM, Paul Gortmaker wrote: > On Thu, Feb 21, 2013 at 9:37 AM, Mark Lord wrote: >> On 13-02-20 10:05 PM, Linus Torvalds wrote: >>> On Wed, Feb 20, 2013 at 2:09 PM, David Miller wrote: .. >>> Nooo You killed the 3c501 and 3c503 drivers! Snif. >>> >>> I wonder if they still

Re: [PATCH] staging/zcache: Fix/improve zcache writeback code, tie to a config option

2013-02-21 Thread Ric Mason
On 02/07/2013 02:27 AM, Dan Magenheimer wrote: It was observed by Andrea Arcangeli in 2011 that zcache can get "full" and there must be some way for compressed swap pages to be (uncompressed and then) sent through to the backing swap disk. A prototype of this functionality, called "unuse", was

linux-next: manual merge of the akpm-current tree with the tree

2013-02-21 Thread Stephen Rothwell
Hi Andrew, Today's linux-next merge of the akpm-current tree got a conflict in fs/btrfs/inode.cfs/btrfs/file.c between commit 55e301fd57a6 ("Btrfs: move fs/btrfs/ioctl.h to include/uapi/linux/btrfs.h") from the btrfs tree and commit "aio: don't include aio.h in sched.h" from the akpm-current

Re: [PATCH 0/7] ksm: responses to NUMA review

2013-02-21 Thread Ric Mason
On 02/21/2013 04:17 PM, Hugh Dickins wrote: Here's a second KSM series, based on mmotm 2013-02-19-17-20: partly in response to Mel's review feedback, partly fixes to issues that I found myself in doing more review and testing. None of the issues fixed are truly show-stoppers, though I would

Re: [PATCH] kexec: prevent double free on image allocation failure

2013-02-21 Thread Sasha Levin
On 02/21/2013 09:46 PM, Zhang Yanfei wrote: > 于 2013年02月22日 09:55, Eric W. Biederman 写道: >> Sasha Levin writes: >> >>> If kimage_normal_alloc() fails to initialize an allocated kimage, it will >>> free >>> the image but would still set 'rimage', as a result kexec_load will try >>> to free it

Re: [RFC] arm: use built-in byte swap function

2013-02-21 Thread Nicolas Pitre
On Thu, 21 Feb 2013, Kim Phillips wrote: > Here's the asm version I'm working on now, based on compiler > output of the C version. Haven't tested beyond defconfig builds, > which pass ok. > > Is there anything I have to do for thumb mode? If so, how to test? You just need to pick a config

Re: [RFC PATCH v3 1/3] sched: schedule balance map foundation

2013-02-21 Thread Alex Shi
On 02/22/2013 10:53 AM, Michael Wang wrote: >> > >>> >> And the final cost is 3000 int and 103 pointer, and some padding, >>> >> but won't bigger than 10M, not a big deal for a system with 1000 cpu >>> >> too. >> > >> > Maybe, but quadric stuff should be frowned upon at all times, these >> >

Re: PROBLEM: Crash cgdeleting empty memory cgroups with memory.kmem.limit_in_bytes set

2013-02-21 Thread Kamezawa Hiroyuki
(2013/02/21 17:34), Glauber Costa wrote: On 02/21/2013 03:00 AM, Tejun Heo wrote: (cc'ing cgroup / memcg people and quoting whole body) Looks like something is going wrong with memcg cache destruction. Glauber, any ideas? Also, can we please not use names as generic as

Re: Questin about swap_slot free and invalidate page

2013-02-21 Thread Ric Mason
On 02/22/2013 05:42 AM, Dan Magenheimer wrote: From: Ric Mason [mailto:ric.mas...@gmail.com] Subject: Re: Questin about swap_slot free and invalidate page On 02/19/2013 11:27 PM, Dan Magenheimer wrote: From: Ric Mason [mailto:ric.mas...@gmail.com] Hugh is right that handling the possibility

Re: [PATCH] sched: Skip looking at skip if next or last is set

2013-02-21 Thread Michael Wang
On 02/22/2013 12:06 AM, Srikar Dronamraju wrote: > * Peter Zijlstra [2013-02-20 09:46:25]: > >> On Mon, 2013-02-18 at 18:31 +0530, Srikar Dronamraju wrote: >>> pick_next_entity() prefers next, then last. However code checks if the >>> left entity can be skipped even if next / last is set. >>>

Re: [RFC] arm: use built-in byte swap function

2013-02-21 Thread Kim Phillips
On Thu, 21 Feb 2013 11:40:54 -0500 Nicolas Pitre wrote: > On Thu, 21 Feb 2013, Kim Phillips wrote: > > > On Wed, 20 Feb 2013 23:29:58 -0500 > > Nicolas Pitre wrote: > > > > > On Wed, 20 Feb 2013, Kim Phillips wrote: > > > > > > > On Wed, 20 Feb 2013 10:43:18 -0500 > > > > Nicolas Pitre

Re: [PATCH] kexec: prevent double free on image allocation failure

2013-02-21 Thread Zhang Yanfei
于 2013年02月22日 09:55, Eric W. Biederman 写道: > Sasha Levin writes: > >> If kimage_normal_alloc() fails to initialize an allocated kimage, it will >> free >> the image but would still set 'rimage', as a result kexec_load will try >> to free it again. >> >> This would explode as part of the freeing

Re: [PATCHv5 2/8] zsmalloc: add documentation

2013-02-21 Thread Ric Mason
On 02/21/2013 11:50 PM, Seth Jennings wrote: On 02/21/2013 02:49 AM, Ric Mason wrote: On 02/19/2013 03:16 AM, Seth Jennings wrote: On 02/16/2013 12:21 AM, Ric Mason wrote: On 02/14/2013 02:38 AM, Seth Jennings wrote: This patch adds a documentation file for zsmalloc at

Re: [PATCHv5 2/8] zsmalloc: add documentation

2013-02-21 Thread Ric Mason
On 02/21/2013 11:50 PM, Seth Jennings wrote: On 02/21/2013 02:49 AM, Ric Mason wrote: On 02/19/2013 03:16 AM, Seth Jennings wrote: On 02/16/2013 12:21 AM, Ric Mason wrote: On 02/14/2013 02:38 AM, Seth Jennings wrote: This patch adds a documentation file for zsmalloc at

Re: [RFC PATCH v3 1/3] sched: schedule balance map foundation

2013-02-21 Thread Michael Wang
On 02/21/2013 07:37 PM, Peter Zijlstra wrote: > On Thu, 2013-02-21 at 12:58 +0800, Michael Wang wrote: >> >> You are right, it cost space in order to accelerate the system, I've >> calculated the cost once before (I'm really not good at this, please >> let >> me know if I make any silly

Re: [PATCH V2 0/4] CPUFreq: Implement per policy instances of governors

2013-02-21 Thread Viresh Kumar
On 11 February 2013 13:19, Viresh Kumar wrote: > This is targetted for 3.10-rc1 or linux-next just after the merge window. Hi Rafael, I have pushed this patch again with the modifications/fixups i posted: cpufreq-for-3.10 Also i have swapped patch 3 & 4, in case you decide to drop that Kconfig

Re: [PATCH V2 3/4] cpufreq: Add Kconfig option to enable/disable have_multiple_policies

2013-02-21 Thread Viresh Kumar
On 22 February 2013 07:59, Rafael J. Wysocki wrote: > On Friday, February 22, 2013 07:44:23 AM Viresh Kumar wrote: >> If you don't like this one then we can add another entry >> into struct policy like: gov_sysfs_parent. > > I don't know. This is going to look kind of ugly this way or another I

[GIT PULL] Blackfin updates for 3.9

2013-02-21 Thread Bob Liu
Hi Linus, The following changes since commit 19f949f52599ba7c3f67a5897ac6be14bfcb1200: Linux 3.8 (2013-02-18 15:58:34 -0800) are available in the git repository at: git://git.kernel.org/pub/scm/linux/kernel/git/lliubbo/blackfin.git for-linus for you to fetch changes up to

Re: [RFC PATCH v3 0/3] sched: simplify the select_task_rq_fair()

2013-02-21 Thread Michael Wang
On 02/21/2013 06:20 PM, Peter Zijlstra wrote: > On Thu, 2013-02-21 at 12:51 +0800, Michael Wang wrote: >> The old logical when locate affine_sd is: >> >> if prev_cpu != curr_cpu >> if wake_affine() >> prev_cpu = curr_cpu >> new_cpu =

linux-next: manual merge of the writeback tree with the btrfs tree

2013-02-21 Thread Stephen Rothwell
Hi Wu, Today's linux-next merge of the writeback tree got a conflict in fs/btrfs/extent-tree.c between commit da633a421701 ("Btrfs: flush all dirty inodes if writeback can not start") from the btrfs tree and commit 10ee27a06cc8 ("vfs: re-implement writeback_inodes_sb(_nr)_if_idle() and rename

Re: [RFC PATCH v3 0/3] sched: simplify the select_task_rq_fair()

2013-02-21 Thread Michael Wang
On 02/21/2013 05:43 PM, Mike Galbraith wrote: > On Thu, 2013-02-21 at 17:08 +0800, Michael Wang wrote: > >> But is this patch set really cause regression on your Q6600? It may >> sacrificed some thing, but I still think it will benefit far more, >> especially on huge systems. > > We spread on

Re: [PATCH] kexec: prevent double free on image allocation failure

2013-02-21 Thread Sasha Levin
On 02/21/2013 08:55 PM, ebied...@xmission.com wrote: > Sasha Levin writes: > >> If kimage_normal_alloc() fails to initialize an allocated kimage, it will >> free >> the image but would still set 'rimage', as a result kexec_load will try >> to free it again. >> >> This would explode as part of

Re: [PATCH V2 4/4] cpufreq: Get rid of "struct global_attr"

2013-02-21 Thread Viresh Kumar
On 22 February 2013 08:03, Rafael J. Wysocki wrote: > On Friday, February 22, 2013 07:47:44 AM Viresh Kumar wrote: >> On 22 February 2013 05:15, Rafael J. Wysocki wrote: >> > Why did you change all of the lines of this macro instead of changing just >> > the >> > one line you needed to change?

Re: [PATCH V2 1/4] cpufreq: Add per policy governor-init/exit infrastructure

2013-02-21 Thread Viresh Kumar
On 22 February 2013 05:05, Rafael J. Wysocki wrote: > Why don't you use different values here? > > If you need only one value, one #define should be sufficient. This is the fixup i have for this, I will push all patches again to cpufreq-for-3.10 branch:

  1   2   3   4   5   6   7   8   9   10   >