Re: [PATCH] staging: sbe-2t3e3: fix error handling in t3e3_init_channel()

2012-09-25 Thread Dan Carpenter
On Tue, Sep 25, 2012 at 04:56:02PM +0400, Alexey Khoroshilov wrote: > t3e3_init_channel() incorrectly handles errors in several places: > it returns zero and does not deallocate all required resources. > The patch fixes that places. > > Found by Linux Driver Verification project

Re: [PATCHv1 6/6] MAINTAINERS: add recently created files to dw_dmac section

2012-09-25 Thread Andy Shevchenko
On Tue, 2012-09-25 at 06:19 -0700, Joe Perches wrote: > On Tue, 2012-09-25 at 15:13 +0300, Andy Shevchenko wrote: > > Signed-off-by: Andy Shevchenko > > You also added yourself as a maintainer. > Congrats/sympathies, etc... Actually I prefer to be just a supporter, because I introduce few

perf tools regression testing was Re: [PATCH v3 1/9] perf hists: Move hists_init() from util/evsel.c to util/hist.c

2012-09-25 Thread Arnaldo Carvalho de Melo
Em Tue, Sep 25, 2012 at 09:59:02PM +0900, Namhyung Kim escreveu: > 2012-09-25 (화), 08:05 -0300, Arnaldo Carvalho de Melo: > > Em Tue, Sep 25, 2012 at 10:25:13AM +0900, Namhyung Kim escreveu: > > > On Mon, 24 Sep 2012 13:02:39 -0300, Arnaldo Carvalho de Melo wrote: > > > > Em Mon, Sep 24, 2012 at

Re: [PATCH] staging/gdm72xx: coding style fixes gdm_qos.c

2012-09-25 Thread Dan Carpenter
On Tue, Sep 25, 2012 at 04:53:52PM +0400, Alexey Khoroshilov wrote: > Fix checkpatch.pl warnings: > WARNING: Prefer pr_debug(... to printk(KERN_DEBUG, ... > WARNING: Prefer pr_warn(... to printk(KERN_WARNING, ... > WARNING: Prefer pr_err(... to printk(KERN_ERR, ... > Better to just replace the

Re: [PATCH 15/19] x86, irq: Move irq_remapped() check into free_remapped_irq

2012-09-25 Thread Joerg Roedel
On Sun, Aug 26, 2012 at 08:27:55PM +0200, Sebastian Andrzej Siewior wrote: > On Mon, Aug 20, 2012 at 03:56:01PM +0200, Joerg Roedel wrote: > > --- a/arch/x86/kernel/apic/io_apic.c > > +++ b/arch/x86/kernel/apic/io_apic.c > > @@ -2981,8 +2981,8 @@ void destroy_irq(unsigned int irq) > > > >

Re: [PATCH 1/1] perf, Add support for Xeon-Phi PMU

2012-09-25 Thread Cyrill Gorcunov
On Tue, Sep 25, 2012 at 02:01:26PM +0200, Peter Zijlstra wrote: > On Tue, 2012-09-25 at 15:42 +0400, Cyrill Gorcunov wrote: > > > Guys, letme re-read this whole mail thread first since I have no clue > > what this remapping about ;) > > x86_setup_perfctr() / set_ext_hw_attr() have special

Re: [PATCH] media: davinci: vpif: set device capabilities

2012-09-25 Thread Prabhakar Lad
Hi Hans/Laurent On Tue, Sep 25, 2012 at 5:24 PM, Hans Verkuil wrote: > On Tue 25 September 2012 13:49:16 Laurent Pinchart wrote: >> Hi Hans, >> >> On Tuesday 25 September 2012 13:43:36 Hans Verkuil wrote: >> > On Tue 25 September 2012 13:16:24 Prabhakar wrote: >> > > From: Lad, Prabhakar >> > >

[PATCH 1/2] PNP: Add a release method to system resource driver

2012-09-25 Thread Matthew Garrett
This could conceivably be hotpluggable, and we may want to displace it from devices under certain circustances, so add a release method to hand back the resources. Signed-off-by: Matthew Garrett --- drivers/pnp/system.c | 30 ++ 1 file changed, 26 insertions(+), 4

[PATCH 2/2] PNP: Unbind drivers if the new driver matches _HID rather than _CID

2012-09-25 Thread Matthew Garrett
ACPIPNP devices may have two levels of ID - the HID (a single string defining the hardware) and the CIDs (zero or more strings defining interfaces compatible with the hardware). If a driver matching a CID is bound first, it will be impossible to bind a driver that matches the HID despite it being

Re: 20% performance drop on PostgreSQL 9.2 from kernel 3.5.3 to 3.6-rc5 on AMD chipsets - bisected

2012-09-25 Thread Mel Gorman
On Mon, Sep 24, 2012 at 07:44:17PM +0200, Peter Zijlstra wrote: > On Mon, 2012-09-24 at 18:54 +0200, Peter Zijlstra wrote: > > But let me try and come up with the list thing, I think we've > > actually got that someplace as well. > > OK, I'm sure the below can be written better, but my brain is

Re: [PATCH 4/4] mfd: max8907: remove regulator-compatible from DT docs

2012-09-25 Thread Samuel Ortiz
Hi Stephen, On Mon, Sep 24, 2012 at 01:25:03PM -0600, Stephen Warren wrote: > From: Stephen Warren > > Commit "regulator: deprecate regulator-compatible DT property" deprecated > the use of the regulator-compatible DT property. Update the DT example in > the MAX8907 binding documentation to

Re: [PATCHv1 6/6] MAINTAINERS: add recently created files to dw_dmac section

2012-09-25 Thread Joe Perches
On Tue, 2012-09-25 at 15:13 +0300, Andy Shevchenko wrote: > Signed-off-by: Andy Shevchenko You also added yourself as a maintainer. Congrats/sympathies, etc... > diff --git a/MAINTAINERS b/MAINTAINERS [] > @@ -6007,10 +6007,13 @@ F:drivers/tty/serial > > SYNOPSYS DESIGNWARE DMAC DRIVER

Re: 20% performance drop on PostgreSQL 9.2 from kernel 3.5.3 to 3.6-rc5 on AMD chipsets - bisected

2012-09-25 Thread Borislav Petkov
On Tue, Sep 25, 2012 at 01:58:06PM +0200, Peter Zijlstra wrote: > On Mon, 2012-09-24 at 19:11 -0700, Linus Torvalds wrote: > > In the not-so-distant past, we had the intel "Dunnington" Xeon, which > > was iirc basically three Core 2 duo's bolted together (ie three > > clusters of two cores sharing

Re: [PATCH 3/3] DMA: PL330: Balance module remove function with probe

2012-09-25 Thread Jassi Brar
On Tue, Sep 25, 2012 at 2:27 PM, Inderpal Singh wrote: > Since peripheral channel resources are not being allocated at probe, > no need to flush the channels and free the resources in remove function. > > Signed-off-by: Inderpal Singh > --- > drivers/dma/pl330.c |8 +--- > 1 file

Re: [PATCH 10/19] x86, io_apic: Introduce function pointer for setup_timer_IRQ0_pin

2012-09-25 Thread Joerg Roedel
On Sun, Aug 26, 2012 at 08:16:03PM +0200, Sebastian Andrzej Siewior wrote: > On Mon, Aug 20, 2012 at 03:55:56PM +0200, Joerg Roedel wrote: > > diff --git a/drivers/iommu/irq_remapping.c b/drivers/iommu/irq_remapping.c > > index 827f8a1..71824fc 100644 > > --- a/drivers/iommu/irq_remapping.c > >

[PATCH v2 1/2] Ensure sched_domains_numa_levels safe in other functions.

2012-09-25 Thread Tang Chen
We should temporarily reset sched_domains_numa_levels to 0 after it is reset to 'level' in sched_init_numa(). If it fails to allocate memory for array sched_domains_numa_masks[][], the array will contain less then 'level' members. This could be dangerous when we use it to iterate array

Re: [PATCH 2/3] DMA: PL330: Change allocation method to properly free DMA descriptors

2012-09-25 Thread Jassi Brar
On Tue, Sep 25, 2012 at 2:27 PM, Inderpal Singh wrote: > In probe, memory for multiple DMA descriptors were being allocated at once > and then it was being split and added into DMA pool one by one. The address > of this memory allocation is not being saved anywhere. To free this memory, > the

Re: [PATCH v2 3/4] perf annotate: configure objdump path at compile time

2012-09-25 Thread Namhyung Kim
Hi Irina, 2012-09-23 (일), 22:27 +0300, Irina Tirdea: > From: Irina Tirdea > > The default name for objdump is "objdump". For cross-compiling the name of > objdump will be different (e.g. arm-eabi-objdump in Android). > > Set the default objdump name in the Makefile with DEFAULT_OBJDUMP_PATH.

[PATCH V3 07/10] Secure boot: Add a dummy kernel parameter that will switch on Secure Boot mode

2012-09-25 Thread Josh Boyer
This forcibly drops CAP_COMPROMISE_KERNEL from both cap_permitted and cap_bset in the init_cred struct, which everything else inherits from. This works on any machine and can be used to develop even if the box doesn't have UEFI. Signed-off-by: Josh Boyer --- Documentation/kernel-parameters.txt

Re: RTL8101E/RTL8102E PCI Express Fast Ethernet controller (rev 02)

2012-09-25 Thread Thanasis
on 09/25/2012 12:54 PM Francois Romieu wrote the following: > Thanasis : > [...] >> I don't know what's wrong, but I am getting those errors. >> Are you sure about the git tree? > > Yes. Replace 'git rev-list --all' with 'git rev-list --branches'. > > It should show something like: > $ git

Re: [PATCH 12/19] x86, irq: Add data structure to keep AMD specific irq remapping information

2012-09-25 Thread Joerg Roedel
On Sun, Aug 26, 2012 at 08:21:26PM +0200, Sebastian Andrzej Siewior wrote: > On Mon, Aug 20, 2012 at 03:55:58PM +0200, Joerg Roedel wrote: > > --- a/arch/x86/include/asm/hw_irq.h > > +++ b/arch/x86/include/asm/hw_irq.h > > @@ -101,6 +101,7 @@ static inline void set_io_apic_irq_attr(struct > >

Re: [PATCH 3/4] lib: vsprintf: Optimize put_dec_trunc8

2012-09-25 Thread Denys Vlasenko
On Tue, Sep 25, 2012 at 1:44 PM, George Spelvin wrote: > Thanks to Denys Vlasenko for sending me his benchmarking code. > > I went and hacked on it to ransomize the numbers being converted more, > since repeatedly converting the same number underestimates the number > of branch mispredictions. >

[PATCH] media: mt9p031/mt9t001/mt9v032: use V4L2_CID_TEST_PATTERN for test pattern control

2012-09-25 Thread Prabhakar
From: Lad, Prabhakar Signed-off-by: Lad, Prabhakar Signed-off-by: Manjunath Hadli --- drivers/media/i2c/mt9p031.c | 27 drivers/media/i2c/mt9t001.c | 33 +++--- drivers/media/i2c/mt9v032.c | 46 +-

Re: [PATCH v3 1/9] perf hists: Move hists_init() from util/evsel.c to util/hist.c

2012-09-25 Thread Namhyung Kim
2012-09-25 (화), 08:05 -0300, Arnaldo Carvalho de Melo: > Em Tue, Sep 25, 2012 at 10:25:13AM +0900, Namhyung Kim escreveu: > > On Mon, 24 Sep 2012 13:02:39 -0300, Arnaldo Carvalho de Melo wrote: > > > Em Mon, Sep 24, 2012 at 11:24:03PM +0800, Feng Tang escreveu: > > >> Which looks more natural > >

[PATCH] staging: sbe-2t3e3: fix error handling in t3e3_init_channel()

2012-09-25 Thread Alexey Khoroshilov
t3e3_init_channel() incorrectly handles errors in several places: it returns zero and does not deallocate all required resources. The patch fixes that places. Found by Linux Driver Verification project (linuxtesting.org). Signed-off-by: Alexey Khoroshilov --- drivers/staging/sbe-2t3e3/module.c

[PATCH] staging/gdm72xx: coding style fixes gdm_qos.c

2012-09-25 Thread Alexey Khoroshilov
Fix checkpatch.pl warnings: WARNING: Prefer pr_debug(... to printk(KERN_DEBUG, ... WARNING: Prefer pr_warn(... to printk(KERN_WARNING, ... WARNING: Prefer pr_err(... to printk(KERN_ERR, ... Signed-off-by: Alexey Khoroshilov --- drivers/staging/gdm72xx/gdm_qos.c |6 +++--- 1 file changed, 3

Re: [PATCH] v4l: change path of video drivers

2012-09-25 Thread Laurent Pinchart
Hi Prabhakar, Thanks for the patch. On Friday 14 September 2012 14:47:52 Prabhakar Lad wrote: > From: Lad, Prabhakar > > due to structure change for video drivers, change the > description with correct path. > > Signed-off-by: Lad, Prabhakar > Signed-off-by: Manjunath Hadli > Cc: Mauro

Re: [PATCH] block: makes bio_split support bio without data

2012-09-25 Thread Jens Axboe
On 09/24/2012 06:56 AM, NeilBrown wrote: > > Hi Jens, > this patch has been sitting in my -next tree for a little while and I was > hoping for it to go in for the next merge window. > It simply allows bio_split() to be used on bios without a payload, such as > 'discard'. > Are you happy with

[PATCH] x86: Unspaghettize do_trap()

2012-09-25 Thread Frederic Weisbecker
Cleanup the label maze in this function. Having a seperate function to first handle the traps that don't generate a signal makes it easier to convert into more readable conditional paths. Signed-off-by: Frederic Weisbecker Cc: Thomas Gleixner Cc: Ingo Molnar Cc: H. Peter Anvin ---

Re: [PATCH] block: export trace_block_unplug

2012-09-25 Thread Jens Axboe
On 09/25/2012 01:57 AM, NeilBrown wrote: > > Hi Jens, > is there any chance this can be in the next merge window? I'm > adding block tracing to md and found I need another export. No problem, applied. -- Jens Axboe -- To unsubscribe from this list: send the line "unsubscribe linux-kernel"

Re: [PATCH 1/3] DMA: PL330: Free memory allocated for peripheral channels

2012-09-25 Thread Jassi Brar
On Tue, Sep 25, 2012 at 2:27 PM, Inderpal Singh wrote: > The allocated memory for peripheral channels is not being freed upon > failure in probe and in module's remove funtion. It will lead to memory > leakage. Hence free the allocated memory. > > Signed-off-by: Inderpal Singh > --- >

Re: [PATCH 0/4] deprecate regulator-compatible DT property

2012-09-25 Thread Mark Brown
On Mon, Sep 24, 2012 at 01:24:59PM -0600, Stephen Warren wrote: > From: Stephen Warren > > As explained in patch 1, this series deprecates the regulator-compatible > DT property, and updates various binding documentation and examples for > this change. Applied regulator patches, thanks. -- To

[PATCH] UV: fix incorrect tlb flush all issue

2012-09-25 Thread Cliff Wickman
From: Cliff Wickman (this was sent as an ack on 9/13, but with incorrect title and sign-off) Ack. But with the adjustment below. The 'end' argument was not declared long. I tested the patch on a UV. It has the effect of either clearing 1 or all TLBs in a cpu. I added some debugging to test

Re: [PATCH 5/6] ARM: dove: Remove watchdog from DT

2012-09-25 Thread Arnd Bergmann
On Tuesday 25 September 2012, Arnd Bergmann wrote: > On Tuesday 25 September 2012, Andrew Lunn wrote: > > > Regarding mv78xx0, I agree that I'm not sure what to do. The number of > > > supported platforms is small. Should we simply mark mv78xx0 deprecated > > > now, wait a few release cycles to

Re: [RFT/PATCH] serial: omap: prevent resume if device is not suspended.

2012-09-25 Thread Jassi Brar
On 19 September 2012 17:29, Felipe Balbi wrote: > this is what I mean, actually. If we remove pm_runtime_get_sync() in > exchange for pm_runtime_set_active() before pm_runtime_enable(), it > works on PandaBoard, but breaks BeagleBoard. > Perhaps it suggests that OMAP4 (PandaBoard) serial port is

Re: [PATCH 5/6] ARM: dove: Remove watchdog from DT

2012-09-25 Thread Arnd Bergmann
On Tuesday 25 September 2012, Arnaud Patard wrote: > Arnd Bergmann writes: > > > > The next step would be to label mach-orion5x as deprecated in Kconfig for > > a release and change the help text so it tells people to move to mach-mvebu > > and submit dts files. > > > > You seem to imply that

[PATCH] tools lib traceevent: Fix error path on pevent_parse_event

2012-09-25 Thread Namhyung Kim
If __pevent_parse_format() succeeded but add_event() failed, 'ret' didn't have a proper error code. Set it to PEVENT_ERRNO__MEM_ALLOC_FAILED. In addition, at that point 'event' also has fields and format information and they all need to be freed. Call pevent_free_format() to handle it.

Re: [PATCH 1/1] perf, Add support for Xeon-Phi PMU

2012-09-25 Thread Cyrill Gorcunov
On Tue, Sep 25, 2012 at 04:22:29PM +0400, Cyrill Gorcunov wrote: > On Tue, Sep 25, 2012 at 02:05:58PM +0200, stephane eranian wrote: > > On Tue, Sep 25, 2012 at 2:01 PM, Peter Zijlstra > > wrote: > > > On Tue, 2012-09-25 at 15:42 +0400, Cyrill Gorcunov wrote: > > > > > >> Guys, letme re-read

Re: [PATCH 1/1] perf, Add support for Xeon-Phi PMU

2012-09-25 Thread Cyrill Gorcunov
On Tue, Sep 25, 2012 at 02:05:58PM +0200, stephane eranian wrote: > On Tue, Sep 25, 2012 at 2:01 PM, Peter Zijlstra > wrote: > > On Tue, 2012-09-25 at 15:42 +0400, Cyrill Gorcunov wrote: > > > >> Guys, letme re-read this whole mail thread first since I have no clue > >> what this remapping about

Re: [PATCH 0/1] dm-integrity: integrity protection device-mapper target

2012-09-25 Thread Milan Broz
On 09/24/2012 06:20 PM, Kasatkin, Dmitry wrote: >> So it can provide confidentiality but it CANNOT provide integrity protection. >> > Yes, it provides confidentiality and via encryption it provides > certain level of integrity protection. > Data cannot be modified without being detected. >

[PATCHv1 1/6] dmaengine: dw_dmac: Remove clk API dependency

2012-09-25 Thread Andy Shevchenko
From: Heikki Krogerus Not all platforms support clk API. Signed-off-by: Heikki Krogerus Signed-off-by: Andy Shevchenko --- drivers/dma/Kconfig|1 - drivers/dma/dw_dmac.c | 18 +++--- drivers/dma/dw_dmac_regs.h |1 - 3 files changed, 3 insertions(+), 17

[PATCHv1 4/6] avr32: at32ap700x: rename DMA controller

2012-09-25 Thread Andy Shevchenko
From: Heikki Krogerus There are no functional changes. Signed-off-by: Heikki Krogerus Signed-off-by: Andy Shevchenko --- arch/avr32/mach-at32ap/at32ap700x.c | 20 ++-- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/arch/avr32/mach-at32ap/at32ap700x.c

Re: [PATCHv3 0/7] rtc: convert to c99 format

2012-09-25 Thread Felipe Balbi
Hi, On Tue, Sep 25, 2012 at 05:01:59PM +0530, Shubhrajyoti D wrote: > The series tries to convert the i2c_msg to c99 struct. > This may avoid issues like below if someone tries to add a > structure. > http://www.mail-archive.com/linux-i2c@vger.kernel.org/msg08972.html > > Special thanks to Julia

[PATCHv1 6/6] MAINTAINERS: add recently created files to dw_dmac section

2012-09-25 Thread Andy Shevchenko
Signed-off-by: Andy Shevchenko --- MAINTAINERS |3 +++ 1 file changed, 3 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index 479a655..1a169f6 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -6007,10 +6007,13 @@ F: drivers/tty/serial SYNOPSYS DESIGNWARE DMAC DRIVER M:

[PATCHv1 5/6] MAINTAINERS: fix indentation for Viresh Kumar

2012-09-25 Thread Andy Shevchenko
Signed-off-by: Andy Shevchenko Cc: Viresh Kumar --- MAINTAINERS | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/MAINTAINERS b/MAINTAINERS index 8fa0255..479a655 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -572,7 +572,7 @@ F: drivers/net/appletalk/ F:

[PATCHv1 2/6] dmaengine: dw_dmac: add driver for Atmel AT32

2012-09-25 Thread Andy Shevchenko
From: Heikki Krogerus This driver should be usable on all platforms that depend on clk API. Signed-off-by: Heikki Krogerus Signed-off-by: Andy Shevchenko --- drivers/dma/Kconfig|9 +++ drivers/dma/Makefile |1 + drivers/dma/dw_dmac.c | 23 +--

[PATCHv1 3/6] dmaengine: dw_dmac: Add PCI part of the driver

2012-09-25 Thread Andy Shevchenko
From: Heikki Krogerus This is the PCI part of the DesignWare DMAC driver. The controller is usually used in the Intel hardware such as Medfield. Signed-off-by: Heikki Krogerus Signed-off-by: Andy Shevchenko --- drivers/dma/Kconfig|9 +++ drivers/dma/Makefile |1 +

[PATCHv1 0/6] dw_dmac: split the driver and introduce PCI part

2012-09-25 Thread Andy Shevchenko
This patchset is dedicated to support different platform devices via the same core driver. In our case the dw_dmac could be used as a PCI device, regular embedded device or something else. This split allows to support the controller connected to any bus by adding a little piece of code without

Re: 20% performance drop on PostgreSQL 9.2 from kernel 3.5.3 to 3.6-rc5 on AMD chipsets - bisected

2012-09-25 Thread Hillf Danton
On Tue, Sep 25, 2012 at 12:54 AM, Peter Zijlstra wrote: > On Mon, 2012-09-24 at 09:33 -0700, Linus Torvalds wrote: >> Sure, the "scan bits" bitops will return ">= nr_cpu_ids" for the "I >> couldn't find a bit" thing, but that doesn't mean that everything else >> should. > > Fair enough.. > > ---

[PATCH] prctl: Use access_ok instead of TASK_SIZE in prctl_set_mm

2012-09-25 Thread Cyrill Gorcunov
Eric pointed that better to use access_ok instead of TASK_SIZE for testing if address is allowed for use. Reported-by: "Eric W. Biederman" Signed-off-by: Cyrill Gorcunov CC: Mark Salter CC: Kees Cook CC: Andrew Morton CC: Pavel Emelyanov --- kernel/sys.c |2 +- 1 file changed, 1

[PATCH] aic94xx: fix handling of default CTRL-A settings

2012-09-25 Thread Paul Bolle
Compiling aic94xx_sds.o (part of the aic94xx driver) triggers this GCC warning: drivers/scsi/aic94xx/aic94xx_sds.c: In function 'asd_read_flash': drivers/scsi/aic94xx/aic94xx_sds.c:597:21: warning: 'offs' may be used uninitialized in this function [-Wmaybe-uninitialized]

[PATCH] GPIO: gpio-pxa: fix bug when get gpio value

2012-09-25 Thread Neil Zhang
We need to return 0 or 1 when get gpio value. Signed-off-by: Neil Zhang --- drivers/gpio/gpio-pxa.c |5 - 1 files changed, 4 insertions(+), 1 deletions(-) diff --git a/drivers/gpio/gpio-pxa.c b/drivers/gpio/gpio-pxa.c index 9cac88a..1f9a058 100644 --- a/drivers/gpio/gpio-pxa.c +++

Re: RCU idle CPU detection is broken in linux-next

2012-09-25 Thread Frederic Weisbecker
On Tue, Sep 25, 2012 at 01:10:27AM +0200, Sasha Levin wrote: > On 09/25/2012 01:06 AM, Frederic Weisbecker wrote: > > 2012/9/25 Sasha Levin : > >> On 09/25/2012 12:47 AM, Sasha Levin wrote: > >>> - While I no longer see the warnings I've originally noticed, if I run > >>> with Paul's last debug

Re: [PATCH 1/1] perf, Add support for Xeon-Phi PMU

2012-09-25 Thread stephane eranian
On Tue, Sep 25, 2012 at 2:01 PM, Peter Zijlstra wrote: > On Tue, 2012-09-25 at 15:42 +0400, Cyrill Gorcunov wrote: > >> Guys, letme re-read this whole mail thread first since I have no clue >> what this remapping about ;) > > x86_setup_perfctr() / set_ext_hw_attr() have special purposed 0 and -1

Re: [PATCH 1/1] perf, Add support for Xeon-Phi PMU

2012-09-25 Thread Peter Zijlstra
On Tue, 2012-09-25 at 15:42 +0400, Cyrill Gorcunov wrote: > Guys, letme re-read this whole mail thread first since I have no clue > what this remapping about ;) x86_setup_perfctr() / set_ext_hw_attr() have special purposed 0 and -1 config values to mean -ENOENT and -EINVAL resp. This means

Re: [PATCH] Update sched_domains_numa_masks when new cpus are onlined.

2012-09-25 Thread Tang Chen
Hi Peter~ I will make a v2 patch-set following your comments and resent it soon. :) Thanks. :) On 09/25/2012 07:50 PM, Peter Zijlstra wrote: On Tue, 2012-09-25 at 19:45 +0800, Tang Chen wrote: Let's have an example here. sched_init_numa() { ... // A loop set

Re: RCU idle CPU detection is broken in linux-next

2012-09-25 Thread Frederic Weisbecker
On Mon, Sep 24, 2012 at 09:04:20PM -0700, Paul E. McKenney wrote: > On Tue, Sep 25, 2012 at 01:41:18AM +0200, Frederic Weisbecker wrote: > > > > > > [ 168.703017] [ cut here ] > > > [ 168.708117] WARNING: at kernel/rcutree.c:502 > > > rcu_eqs_exit_common+0x4a/0x3a0() > >

Re: [PATCH] Revert "__d_unalias() should refuse to move mountpoints"

2012-09-25 Thread Maarten Lankhorst
Op 25-09-12 13:29, Eric W. Biederman schreef: > Maarten Lankhorst writes: > >>> Could you try the following patch? This should report what directories >>> cannot be renamed because one of them is a mount point and it gives some >>> real insight into what is going on. >> ls / >> __d_unalias: /dev

Re: 20% performance drop on PostgreSQL 9.2 from kernel 3.5.3 to 3.6-rc5 on AMD chipsets - bisected

2012-09-25 Thread Peter Zijlstra
On Mon, 2012-09-24 at 19:11 -0700, Linus Torvalds wrote: > In the not-so-distant past, we had the intel "Dunnington" Xeon, which > was iirc basically three Core 2 duo's bolted together (ie three > clusters of two cores sharing L2, and a fully shared L3). So that was > a true multi-core with fairly

[PATCH] gpu/drm/ttm: use copy_highpage

2012-09-25 Thread Akinobu Mita
Use copy_highpage() to copy from one page to another. Signed-off-by: Akinobu Mita Cc: David Airlie Cc: dri-de...@lists.freedesktop.org --- drivers/gpu/drm/ttm/ttm_tt.c | 16 ++-- 1 file changed, 2 insertions(+), 14 deletions(-) diff --git a/drivers/gpu/drm/ttm/ttm_tt.c

[PATCH] drbd: use copy_highpage

2012-09-25 Thread Akinobu Mita
Use copy_highpage() to copy from one page to another. Signed-off-by: Akinobu Mita Cc: drbd-...@lists.linbit.com --- drivers/block/drbd/drbd_bitmap.c | 7 +-- 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/drivers/block/drbd/drbd_bitmap.c b/drivers/block/drbd/drbd_bitmap.c

Re: [BUG] perf/x86: Intel uncore_pmu_to_box() local variable typo

2012-09-25 Thread Ingo Molnar
* Peter Zijlstra wrote: > On Tue, 2012-09-25 at 12:44 +0200, Stephane Eranian wrote: > > Hi, > > > > I don't understand why the local variable box needs to > > be declared static here: > > > > static struct intel_uncore_box * > > uncore_pmu_to_box(struct intel_uncore_pmu *pmu, int cpu) > > {

Re: [PATCH 5/6] ARM: dove: Remove watchdog from DT

2012-09-25 Thread Rtp
Arnd Bergmann writes: Hi, > On Tuesday 25 September 2012, Andrew Lunn wrote: >> On Tue, Sep 25, 2012 at 12:14:39PM +0200, Thomas Petazzoni wrote: >> > On Tue, 25 Sep 2012 11:46:10 +0200, Andrew Lunn wrote: >> > >> > > I principle, i agree. However, i'm not too sure about mach-orion5x & >> > >

Re: [PATCH] perf test: Fix build failure

2012-09-25 Thread Ingo Molnar
* Arnaldo Carvalho de Melo wrote: > Em Tue, Sep 25, 2012 at 11:20:28AM +0900, Namhyung Kim escreveu: > > From: Namhyung Kim > > > > The commit 6a6cd11d4e57 ("perf test: Add test for the sched tracepoint > > format fields") added following build error: > > > > CC builtin-test.o > >

Re: [Drbd-dev] [PATCH v3 06/26] block: Add bio_end_sector()

2012-09-25 Thread Lars Ellenberg
On Mon, Sep 24, 2012 at 03:34:46PM -0700, Kent Overstreet wrote: > Just a little convenience macro - main reason to add it now is preparing > for immutable bio vecs, it'll reduce the size of the patch that puts > bi_sector/bi_size/bi_idx into a struct bvec_iter. For the DRBD part: > diff --git

Re: [PATCH] media: davinci: vpif: set device capabilities

2012-09-25 Thread Hans Verkuil
On Tue 25 September 2012 13:49:16 Laurent Pinchart wrote: > Hi Hans, > > On Tuesday 25 September 2012 13:43:36 Hans Verkuil wrote: > > On Tue 25 September 2012 13:16:24 Prabhakar wrote: > > > From: Lad, Prabhakar > > > > > > Signed-off-by: Lad, Prabhakar > > > Signed-off-by: Manjunath Hadli >

Re: [PATCH 1/1] perf, Add support for Xeon-Phi PMU

2012-09-25 Thread Cyrill Gorcunov
On Tue, Sep 25, 2012 at 03:42:25PM +0400, Cyrill Gorcunov wrote: > On Tue, Sep 25, 2012 at 01:32:38PM +0200, Peter Zijlstra wrote: > > On Thu, 2012-09-20 at 13:03 -0400, Vince Weaver wrote: > > > One additional complication: some of the cache events map to > > > event "0". This causes problems

Re: [PATCH] Update sched_domains_numa_masks when new cpus are onlined.

2012-09-25 Thread Peter Zijlstra
On Tue, 2012-09-25 at 19:45 +0800, Tang Chen wrote: > Let's have an example here. > > sched_init_numa() > { > ... > // A loop set sched_domains_numa_levels to level.-1 > > // I set sched_domains_numa_levels to 0. > sched_domains_numa_levels =

Re: [PATCH] media: davinci: vpif: set device capabilities

2012-09-25 Thread Laurent Pinchart
Hi Hans, On Tuesday 25 September 2012 13:43:36 Hans Verkuil wrote: > On Tue 25 September 2012 13:16:24 Prabhakar wrote: > > From: Lad, Prabhakar > > > > Signed-off-by: Lad, Prabhakar > > Signed-off-by: Manjunath Hadli > > Cc: Hans Verkuil > > --- > > > >

Re: [PATCH] media: davinci: vpif: set device capabilities

2012-09-25 Thread Prabhakar Lad
Hi Hans, Thanks for the review. On Tue, Sep 25, 2012 at 5:13 PM, Hans Verkuil wrote: > On Tue 25 September 2012 13:16:24 Prabhakar wrote: >> From: Lad, Prabhakar >> >> Signed-off-by: Lad, Prabhakar >> Signed-off-by: Manjunath Hadli >> Cc: Hans Verkuil >> --- >>

Re: [PATCH 3/3] workqueue: Schedule work on non-idle cpu instead of current one

2012-09-25 Thread Peter Zijlstra
On Tue, 2012-09-25 at 13:40 +0200, Peter Zijlstra wrote: > On Tue, 2012-09-25 at 17:00 +0530, Viresh Kumar wrote: > > But this is what the initial idea during LPC we had. > > Yeah.. that's true. > > > Any improvements here you can suggest? > > We could uhm... /me tries thinking ... reuse some

Re: linux-next: build failure after merge of the pm tree

2012-09-25 Thread Rafael J. Wysocki
On Tuesday, September 25, 2012, Stephen Rothwell wrote: > Hi Rafael, > > After merging the pm tree, today's linux-next build (x86_64 allmodconfig) > failed like this: > > drivers/cpufreq/powernow-k8.c: In function 'transition_frequency_pstate': > drivers/cpufreq/powernow-k8.c:991:19: error:

Re: [PATCH 3/4] lib: vsprintf: Optimize put_dec_trunc8

2012-09-25 Thread George Spelvin
Thanks to Denys Vlasenko for sending me his benchmarking code. I went and hacked on it to ransomize the numbers being converted more, since repeatedly converting the same number underestimates the number of branch mispredictions. Then I tried computing the number of digits beforehand, as

Re: [PATCH] Update sched_domains_numa_masks when new cpus are onlined.

2012-09-25 Thread Tang Chen
Hi Peter~ Sorry about the confusing log, and thanks for the patient. :) Here, I want to say something more about the sched_domains_numa_levels to make myself more clear. :) Let's have an example here. sched_init_numa() { ... // A loop set sched_domains_numa_levels to

Re: [PATCH] media: davinci: vpif: set device capabilities

2012-09-25 Thread Hans Verkuil
On Tue 25 September 2012 13:16:24 Prabhakar wrote: > From: Lad, Prabhakar > > Signed-off-by: Lad, Prabhakar > Signed-off-by: Manjunath Hadli > Cc: Hans Verkuil > --- > drivers/media/platform/davinci/vpif_capture.c |4 +++- > drivers/media/platform/davinci/vpif_display.c |4 +++- > 2

Re: [PATCH 1/1] perf, Add support for Xeon-Phi PMU

2012-09-25 Thread Cyrill Gorcunov
On Tue, Sep 25, 2012 at 01:32:38PM +0200, Peter Zijlstra wrote: > On Thu, 2012-09-20 at 13:03 -0400, Vince Weaver wrote: > > One additional complication: some of the cache events map to > > event "0". This causes problems because the generic events code > > assumes "0" means not-available. I'm

Re: Backlight control broken between 3.6.0-rc1 and 3.6.0-rc4

2012-09-25 Thread Dan Carpenter
On Sun, Sep 16, 2012 at 04:25:10AM -0700, Grant wrote: > I have a Dell XPS 13 Ultrabook laptop. Backlight control used to be > broken, it works in 3.6.0-rc1, and it is broken again in 3.6.0-rc4. > I've filed a bug for this. https://bugzilla.kernel.org/show_bug.cgi?id=47941 Please add the

Re: [PATCH 3/3] workqueue: Schedule work on non-idle cpu instead of current one

2012-09-25 Thread Peter Zijlstra
On Tue, 2012-09-25 at 17:00 +0530, Viresh Kumar wrote: > But this is what the initial idea during LPC we had. Yeah.. that's true. > Any improvements here you can suggest? We could uhm... /me tries thinking ... reuse some of the NOHZ magic? Would that be sufficient, not waking a NOHZ cpu, or

Re: [PATCH v3] media: davinci: vpif: display: separate out subdev from output

2012-09-25 Thread Hans Verkuil
On Tue 25 September 2012 13:11:49 Prabhakar wrote: > From: Lad, Prabhakar > > vpif_display relied on a 1-1 mapping of output and subdev. This is not > necessarily the case. Separate the two. So there is a list of subdevs > and a list of outputs. Each output refers to a subdev and has routing >

Re: [PATCH 0/3] A driver for Si476x series of chips

2012-09-25 Thread Mauro Carvalho Chehab
Hi Andrey, Em Qui, 2012-09-13 às 15:40 -0700, Andrey Smirnov escreveu: This patchset contains a driver for a Silicon Laboratories 476x series > of radio tuners. The driver itself is implemented as an MFD devices > comprised of three parts: > 1. Core device that provides all the other devices

[PATCHv3] dw_dmac: autoconfigure data_width or get it via platform data

2012-09-25 Thread Andy Shevchenko
Not all of the controllers support the 64 bit data width. Make it configurable via platform data. The driver will try to get a value from the component parameters, otherwise it will use the platform data. Signed-off-by: Andy Shevchenko --- Since v2: - sometimes memory-to-memory test is failed,

Re: [PATCH 3/3] workqueue: Schedule work on non-idle cpu instead of current one

2012-09-25 Thread Vincent Guittot
On 25 September 2012 13:30, Viresh Kumar wrote: > On 25 September 2012 16:52, Peter Zijlstra wrote: >> On Tue, 2012-09-25 at 16:06 +0530, Viresh Kumar wrote: >>> @@ -1066,8 +1076,9 @@ int queue_work(struct workqueue_struct *wq, >>> struct work_struct *work) >>> { >>> int ret; >>> >>> -

[PATCH] ARM: ux500: Move regulator-name properties out to board DTS files

2012-09-25 Thread Lee Jones
Regulator supply names should be allocated by board rather than per SoC, as the same SoC could be wired differently on varying hardware. Here we push all regulator-name allocation out to the dbx5x0 subordinate board files; HREF and Snowball. Requested-by: Mark Brown Signed-off-by: Lee Jones ---

[PATCHv3 6/7] rtc: Convert struct i2c_msg initialization to C99 format

2012-09-25 Thread Shubhrajyoti D
Convert the struct i2c_msg initialization to C99 format. This makes maintaining and editing the code simpler. Also helps once other fields like transferred are added in future. Signed-off-by: Shubhrajyoti D --- drivers/rtc/rtc-isl1208.c | 20 1 files changed, 16

[PATCHv3 0/7] rtc: convert to c99 format

2012-09-25 Thread Shubhrajyoti D
The series tries to convert the i2c_msg to c99 struct. This may avoid issues like below if someone tries to add a structure. http://www.mail-archive.com/linux-i2c@vger.kernel.org/msg08972.html Special thanks to Julia Lawall for helping it automate. By the below script.

[PATCHv3 3/7] rtc: Convert struct i2c_msg initialization to C99 format

2012-09-25 Thread Shubhrajyoti D
Convert the struct i2c_msg initialization to C99 format. This makes maintaining and editing the code simpler. Also helps once other fields like transferred are added in future. Signed-off-by: Shubhrajyoti D --- drivers/rtc/rtc-s35390a.c | 13 +++-- 1 files changed, 11 insertions(+), 2

[PATCHv3 2/7] rtc: Convert struct i2c_msg initialization to C99 format

2012-09-25 Thread Shubhrajyoti D
Convert the struct i2c_msg initialization to C99 format. This makes maintaining and editing the code simpler. Also helps once other fields like transferred are added in future. Signed-off-by: Shubhrajyoti D --- drivers/rtc/rtc-x1205.c | 92 +++--- 1

[PATCHv3 4/7] rtc: Convert struct i2c_msg initialization to C99 format

2012-09-25 Thread Shubhrajyoti D
Convert the struct i2c_msg initialization to C99 format. This makes maintaining and editing the code simpler. Also helps once other fields like transferred are added in future. while at it also fix a checkpatch warn WARNING: sizeof rs5c->buf should be sizeof(rs5c->buf) Signed-off-by: Shubhrajyoti

[PATCHv3 5/7] rtc: Convert struct i2c_msg initialization to C99 format

2012-09-25 Thread Shubhrajyoti D
Convert the struct i2c_msg initialization to C99 format. This makes maintaining and editing the code simpler. Also helps once other fields like transferred are added in future. Signed-off-by: Shubhrajyoti D --- drivers/rtc/rtc-pcf8563.c | 13 +++-- 1 files changed, 11 insertions(+), 2

[PATCHv3 1/7] rtc: Convert struct i2c_msg initialization to C99 format

2012-09-25 Thread Shubhrajyoti D
Convert the struct i2c_msg initialization to C99 format. This makes maintaining and editing the code simpler. Also helps once other fields like transferred are added in future. Signed-off-by: Shubhrajyoti D --- drivers/rtc/rtc-ds1672.c | 26 ++ 1 files changed, 22

[PATCHv3 7/7] rtc: Convert struct i2c_msg initialization to C99 format

2012-09-25 Thread Shubhrajyoti D
Convert the struct i2c_msg initialization to C99 format. This makes maintaining and editing the code simpler. Also helps once other fields like transferred are added in future. Signed-off-by: Shubhrajyoti D --- drivers/rtc/rtc-em3027.c | 17 ++--- 1 files changed, 14

Re: [PATCH 1/1] perf, Add support for Xeon-Phi PMU

2012-09-25 Thread Peter Zijlstra
On Thu, 2012-09-20 at 13:03 -0400, Vince Weaver wrote: > One additional complication: some of the cache events map to > event "0". This causes problems because the generic events code > assumes "0" means not-available. I'm not sure the best way to address > that problem. For all except P4 we

Re: [RFT/PATCH] serial: omap: prevent resume if device is not suspended.

2012-09-25 Thread Russell King - ARM Linux
On Tue, Sep 25, 2012 at 02:12:43PM +0300, Felipe Balbi wrote: > I don't see any aggressive attitude towards what you suggested, > actually. Mailing list archives are available to check, but the one > cursing around was always yourself and THAT deserves an apology. Total rubbish. No apology,

Re: [PATCH 3/3] workqueue: Schedule work on non-idle cpu instead of current one

2012-09-25 Thread Viresh Kumar
On 25 September 2012 16:52, Peter Zijlstra wrote: > On Tue, 2012-09-25 at 16:06 +0530, Viresh Kumar wrote: >> @@ -1066,8 +1076,9 @@ int queue_work(struct workqueue_struct *wq, >> struct work_struct *work) >> { >> int ret; >> >> - ret = queue_work_on(get_cpu(), wq, work); >> -

Re: [PATCH] Revert "__d_unalias() should refuse to move mountpoints"

2012-09-25 Thread Eric W. Biederman
Maarten Lankhorst writes: >> Could you try the following patch? This should report what directories >> cannot be renamed because one of them is a mount point and it gives some >> real insight into what is going on. > > ls / > __d_unalias: /dev -> /dev > __d_unalias: /proc -> /proc >

Re: [PATCH 28/30] perf evsel: Provide a new constructor for tracepoints

2012-09-25 Thread Arnaldo Carvalho de Melo
Em Tue, Sep 25, 2012 at 01:26:46PM +0900, Namhyung Kim escreveu: > On Mon, 24 Sep 2012 12:59:42 -0300, Arnaldo Carvalho de Melo wrote: > > + if (size == alloc_size) { > > + alloc_size += BUFSIZ; > > + nbf = realloc(bf, alloc_size); > > +

Re: w/ current kernel (3.5.3) I need 2 attempts for s2disk in a row

2012-09-25 Thread Dan Carpenter
Hi, I have created a bug for this: https://bugzilla.kernel.org/show_bug.cgi?id=47931 Please add the following information: *) Last known good kernel version *) Complete dmesg regards, dan carpenter -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a

Re: [PATCH v3 7/9] perf header: Add check_perf_magic() func

2012-09-25 Thread Arnaldo Carvalho de Melo
Em Tue, Sep 25, 2012 at 04:21:40PM +0800, Feng Tang escreveu: > On Tue, 25 Sep 2012 11:07:15 +0900 > Namhyung Kim wrote: > > > On Mon, 24 Sep 2012 23:24:09 +0800, Feng Tang wrote: > > [snip] > > > +/* Return 0 if matched */ > > > +int check_perf_magic(u64 magic) > > > +{ > > > + if (!memcmp(,

Re: [PATCH 3/3] workqueue: Schedule work on non-idle cpu instead of current one

2012-09-25 Thread Peter Zijlstra
On Tue, 2012-09-25 at 16:06 +0530, Viresh Kumar wrote: > @@ -1066,8 +1076,9 @@ int queue_work(struct workqueue_struct *wq, > struct work_struct *work) > { > int ret; > > - ret = queue_work_on(get_cpu(), wq, work); > - put_cpu(); > + preempt_disable(); > + ret =

[PATCH] slab: Ignore internal flags in cache creation

2012-09-25 Thread Glauber Costa
Some flags are used internally by the allocators for management purposes. One example of that is the CFLGS_OFF_SLAB flag that slab uses to mark that the metadata for that cache is stored outside of the slab. No cache should ever pass those as a creation flags. We can just ignore this bit if it

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