Re: [RFC][PATCH] fs: configfs: programmatically create config groups

2012-12-11 Thread Joel Becker
On Mon, Dec 10, 2012 at 03:17:34PM +0100, Andrzej Pietrasiewicz wrote: > @Joel in particular: please see my comment in the bottom. > I forgot to mention, representing udcs (USB Device Controllers) in > configfs is similar to interfaces/endpoints: the user needs to guess > what name to use in

Re: "attempt to access beyond end of device" on DVD

2012-12-11 Thread Bruno Prémont
On Wed, 12 December 2012 Jacek Poplawski wrote: > On 12/12/12, Bruno Prémont wrote: > > When it happens, does ejecting and re-inserting the CDs/DVDs also fix the > > problem or is a full reboot mandatory? > > The re-inserting doesn't fix the problem. > Even worse. When it starts - each next DVD

RE: [PATCH] xen/swiotlb: Exchange to contiguous memory for map_sg hook

2012-12-11 Thread Xu, Dongxiao
> -Original Message- > From: Konrad Rzeszutek Wilk [mailto:konrad.w...@oracle.com] > Sent: Wednesday, December 12, 2012 1:07 AM > To: Xu, Dongxiao > Cc: xen-de...@lists.xen.org; linux-kernel@vger.kernel.org > Subject: Re: [PATCH] xen/swiotlb: Exchange to contiguous memory for map_sg > hook

[PATCH RT 0/2] [ANNOUNCE] 3.0.55-rt80-rc1 stable review

2012-12-11 Thread Steven Rostedt
Dear RT Folks, This is the RT stable review cycle of patch 3.0.55-rt80-rc1. Please scream at me if I messed something up. Please test the patches too. The -rc release will be uploaded to kernel.org and will be deleted when the final release is out. This is just a review release (or release

[PATCH RT 1/2] sched: Queue RT tasks to head when prio drops

2012-12-11 Thread Steven Rostedt
From: Thomas Gleixner The following scenario does not work correctly: Runqueue of CPU1 contains two runnable and pinned tasks: T1: SCHED_FIFO, prio 80 T2: SCHED_FIFO, prio 80 T1 is on the cpu and executes the following syscalls (classic priority ceiling scenario):

[PATCH RT 2/2] Linux 3.0.55-rt80-rc1

2012-12-11 Thread Steven Rostedt
From: Steven Rostedt --- localversion-rt |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/localversion-rt b/localversion-rt index cf94a55..37bc36c 100644 --- a/localversion-rt +++ b/localversion-rt @@ -1 +1 @@ --rt79 +-rt80-rc1 -- 1.7.10.4 -- To unsubscribe from this

[PATCH RT 0/2] [ANNOUNCE] 3.2.35-rt53-rc1 stable release

2012-12-11 Thread Steven Rostedt
Dear RT Folks, This is the RT stable review cycle of patch 3.2.35-rt53-rc1. Please scream at me if I messed something up. Please test the patches too. The -rc release will be uploaded to kernel.org and will be deleted when the final release is out. This is just a review release (or release

[PATCH RT 1/2] sched: Queue RT tasks to head when prio drops

2012-12-11 Thread Steven Rostedt
From: Thomas Gleixner The following scenario does not work correctly: Runqueue of CPU1 contains two runnable and pinned tasks: T1: SCHED_FIFO, prio 80 T2: SCHED_FIFO, prio 80 T1 is on the cpu and executes the following syscalls (classic priority ceiling scenario):

[PATCH RT 2/2] Linux 3.2.35-rt53-rc1

2012-12-11 Thread Steven Rostedt
From: Steven Rostedt --- localversion-rt |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/localversion-rt b/localversion-rt index 66a5ed8..2dc212d 100644 --- a/localversion-rt +++ b/localversion-rt @@ -1 +1 @@ --rt52 +-rt53-rc1 -- 1.7.10.4 -- To unsubscribe from this

Re: [PATCH v3 3/5] page_alloc: Introduce zone_movable_limit[] to keep movable limit for nodes

2012-12-11 Thread Jiang Liu
On 2012-12-11 19:28, Simon Jeons wrote: > On Tue, 2012-12-11 at 11:32 +0800, Tang Chen wrote: >> On 12/11/2012 11:07 AM, Jianguo Wu wrote: >>> On 2012/12/11 10:33, Tang Chen wrote: >>> This patch introduces a new array zone_movable_limit[] to store the ZONE_MOVABLE limit from

Re: [PATCH] f2fs: fix up f2fs_get_parent issue to retrieve correct parent inode number

2012-12-11 Thread Namjae Jeon
2012/12/12, Jaegeuk Kim : > 2012-12-12 (수), 00:10 +0900, Namjae Jeon: >> From: Namjae Jeon >> >> Test Case: >> [NFS Client] >> ls -lR . >> >> [NFS Server] >> while [ 1 ] >> do >> echo 3 > /proc/sys/vm/drop_caches >> done >> >> Error on NFS Client: "No such file or directory" >> >> When cache is

[PATCH RT 0/2] [ANNOUNCE] 3.4.22-rt34-rc1 stable review

2012-12-11 Thread Steven Rostedt
Dear RT Folks, This is the RT stable review cycle of patch 3.4.22-rt34-rc1. Please scream at me if I messed something up. Please test the patches too. The -rc release will be uploaded to kernel.org and will be deleted when the final release is out. This is just a review release (or release

[PATCH RT 1/2] sched: Queue RT tasks to head when prio drops

2012-12-11 Thread Steven Rostedt
From: Thomas Gleixner The following scenario does not work correctly: Runqueue of CPU1 contains two runnable and pinned tasks: T1: SCHED_FIFO, prio 80 T2: SCHED_FIFO, prio 80 T1 is on the cpu and executes the following syscalls (classic priority ceiling scenario):

[PATCH RT 2/2] Linux 3.4.22-rt34-rc1

2012-12-11 Thread Steven Rostedt
From: Steven Rostedt --- localversion-rt |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/localversion-rt b/localversion-rt index e1d8362..c2c1097 100644 --- a/localversion-rt +++ b/localversion-rt @@ -1 +1 @@ --rt33 +-rt34-rc1 -- 1.7.10.4 -- To unsubscribe from this

Re: "attempt to access beyond end of device" on DVD

2012-12-11 Thread Jacek Poplawski
On 12/12/12, Bruno Prémont wrote: > When it happens, does ejecting and re-inserting the CDs/DVDs also fix the > problem or is a full reboot mandatory? The re-inserting doesn't fix the problem. Even worse. When it starts - each next DVD is also affected (until reboot). -- To unsubscribe from this

Re: [PATCH v4] 8042: Enable A20 using KBC to fix S3 resume on some MSI laptops

2012-12-11 Thread H. Peter Anvin
Looks good to me. Alan/Rafael/Dmitry, OK for me to commit this? -hpa On 12/11/2012 01:18 PM, Ondrej Zary wrote: > Some MSI laptop BIOSes are broken - INT 15h code uses port 92h to enable A20 > line but resume code assumes that KBC was used. > The laptop will not resume from S3 otherwise

Re: [PATCH 1/2] uio: add warning to documentation

2012-12-11 Thread Benedikt Spranger
On Tue, 11 Dec 2012 15:18:16 -0800 Greg KH wrote: > > -open(), you will probably also want a custom > > +release(), you will probably also want a > > custom release() function. > That sentance no longer makes sense. DUH! will fix... > > +CAVE: The release hook may be processed, even if a > >

Re: [GIT PULL] VFIO updates for v3.8

2012-12-11 Thread Stephen Rothwell
Hi Alex, On Tue, 11 Dec 2012 17:06:56 -0700 Alex Williamson wrote: > > Is that a bad thing? I can start tagging from my next branch if that's > preferred. Thanks, Linus has said many times to not rebase before sending a pull request. When you rebase your tree you effectively throw away your

Re: "attempt to access beyond end of device" on DVD

2012-12-11 Thread Bruno Prémont
Hello, On Mon, 10 December 2012 Jacek Poplawski wrote: > few years ago I noticed annoying bug on Linux, when I was copying > photos from old DVDs I realized that not all files were copied > correctly, there were errors on last files on DVD, you could say "well > that happens, DVD are faulty",

Re: [tip:x86/microcode] x86/microcode_intel_early.c: Early update ucode on Intel's CPU

2012-12-11 Thread H. Peter Anvin
On 12/11/2012 04:27 PM, Yinghai Lu wrote: > On Tue, Dec 11, 2012 at 3:57 PM, H. Peter Anvin wrote: >> Well, we could invoke it on the bootloader page tables, but as you say >> it may not be a good idea... depending on how much memory we may be >> talking about. One solution -- which I have to

Re: Read starvation by sync writes

2012-12-11 Thread Jan Engelhardt
On Monday 2012-12-10 23:12, Jan Kara wrote: > > I was looking into IO starvation problems where streaming sync writes (in >my case from kjournald but DIO would look the same) starve reads. This is >because reads happen in small chunks and until a request completes we don't >start reading further

Re: [PATCH v3] llist: add a safe version of llist_for_each_entry

2012-12-11 Thread Huang Ying
On Tue, 2012-12-11 at 12:25 +0100, Roger Pau Monne wrote: > Signed-off-by: Roger Pau Monné > Cc: Huang Ying > Cc: Konrad Rzeszutek Wilk > --- > Changes since v2: > * Allow to pass a NULL node as the first entry of deleted list >entries. > --- > include/linux/llist.h | 27

[PATCH] 8250/16?50: Add support for Broadcom TruManage redirected serial port

2012-12-11 Thread Stephen Hurd
From: Stephen Hurd Add support for the UART device present in Broadcom TruManage capable NetXtreme chips (ie: 5761m 5762, and 5725). This implementation has a hidden transmit FIFO, so running in single-byte interrupt mode results in too many interrupts. The UART_CAP_HFIFO capability was

[PATCH 1/5] staging: omap-thermal: fix error check in omap_thermal_expose_sensor() and in omap_thermal_register_cpu_cooling().

2012-12-11 Thread Cyril Roelandt
The omap_bandgap_get_sensor_data() function returns ERR_PTR(), so we need to use IS_ERR() rather than a NULL check. Signed-off-by: Cyril Roelandt --- drivers/staging/omap-thermal/omap-thermal-common.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

[PATCH 2/5] media: davinci: fix return value check in vpbe_display_reqbufs().

2012-12-11 Thread Cyril Roelandt
vb2_dma_contig_init_ctx() returns ERR_PTR and never returns NULL, so IS_ERR should be used instead of a NULL check. Signed-off-by: Cyril Roelandt --- drivers/media/platform/davinci/vpbe_display.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH 0/5] Use IS_ERR() rather than a NULL check when necessary.

2012-12-11 Thread Cyril Roelandt
The following five patches replace NULL checks by calls to IS_ERR() when checking the return values of functions that return ERR_PTR() on error. Regards, Cyril Roelandt. --- Cyril Roelandt (5): staging: omap-thermal: fix error check in omap_thermal_expose_sensor() and in

[PATCH 4/5] net: sfc: fix return value check in efx_ptp_probe_channel().

2012-12-11 Thread Cyril Roelandt
The ptp_clock_register() returns ERR_PTR() and never returns NULL. Replace the NULL check by a call to IS_ERR(). Signed-off-by: Cyril Roelandt --- drivers/net/ethernet/sfc/ptp.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/ethernet/sfc/ptp.c

[PATCH 3/5] c2port: fix return value check in duramar2150_c2port_init().

2012-12-11 Thread Cyril Roelandt
The c2port_device_register() function returns ERR_PTR() and never returns NULL, so the NULL check should be replaced by a call to IS_ERR(). Signed-off-by: Cyril Roelandt --- drivers/misc/c2port/c2port-duramar2150.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

Re: [PATCH] f2fs: fix up f2fs_get_parent issue to retrieve correct parent inode number

2012-12-11 Thread Jaegeuk Kim
2012-12-12 (수), 00:10 +0900, Namjae Jeon: > From: Namjae Jeon > > Test Case: > [NFS Client] > ls -lR . > > [NFS Server] > while [ 1 ] > do > echo 3 > /proc/sys/vm/drop_caches > done > > Error on NFS Client: "No such file or directory" > > When cache is dropped at the server, it results in

[PATCH 5/5] spi: fix return value check in hspi_probe().

2012-12-11 Thread Cyril Roelandt
According to its documentation, clk_get() returns a "valid IS_ERR() condition containing errno", so we should call IS_ERR() rather than a NULL check. Signed-off-by: Cyril Roelandt --- drivers/spi/spi-sh-hspi.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

Re: [tip:x86/microcode] x86/microcode_intel_early.c: Early update ucode on Intel's CPU

2012-12-11 Thread Yinghai Lu
On Tue, Dec 11, 2012 at 3:57 PM, H. Peter Anvin wrote: > Well, we could invoke it on the bootloader page tables, but as you say > it may not be a good idea... depending on how much memory we may be > talking about. One solution -- which I have to admit is starting to > sound really good -- is to

Re: [GIT PULL] VFIO updates for v3.8

2012-12-11 Thread Alex Williamson
On Wed, 2012-12-12 at 10:46 +1100, Stephen Rothwell wrote: > Hi Linus, > > On Tue, 11 Dec 2012 12:08:07 -0700 Alex Williamson > wrote: > > > > The following changes since commit 29594404d7fe73cd80eaa4ee8c43dcc53970c60e: > > > > Linux 3.7 (2012-12-10 19:30:57 -0800) > > > > are available in

Re: [[PATCH v9 3/3] 1/1] virtio_console: Remove buffers from out_vq at port removal

2012-12-11 Thread Rusty Russell
Amit Shah writes: > On (Tue) 11 Dec 2012 [09:39:41], Rusty Russell wrote: >> Amit Shah writes: >> >> > On (Fri) 16 Nov 2012 [11:22:09], Rusty Russell wrote: >> >> Amit Shah writes: >> >> > From: Sjur Brændeland >> >> > >> >> > Remove buffers from the out-queue when a port is removed.

Re: [tip:x86/microcode] x86/microcode_intel_early.c: Early update ucode on Intel's CPU

2012-12-11 Thread H. Peter Anvin
On 12/11/2012 03:53 PM, Yinghai Lu wrote: > On Tue, Dec 11, 2012 at 9:38 AM, H. Peter Anvin wrote: >> On 12/11/2012 09:15 AM, Yinghai Lu wrote: >>> >>> >>> No, that is not right place. initrd could be loaded anywhere like way >>> high by bootloader. >>> >> >> Only *after* your changes... the

Re: [tip:x86/microcode] x86/microcode_intel_early.c: Early update ucode on Intel's CPU

2012-12-11 Thread Yinghai Lu
On Tue, Dec 11, 2012 at 9:38 AM, H. Peter Anvin wrote: > On 12/11/2012 09:15 AM, Yinghai Lu wrote: >> >> >> No, that is not right place. initrd could be loaded anywhere like way >> high by bootloader. >> > > Only *after* your changes... the current protocol doesn't allow that. before

Re: [GIT PULL] VFIO updates for v3.8

2012-12-11 Thread Stephen Rothwell
Hi Linus, On Tue, 11 Dec 2012 12:08:07 -0700 Alex Williamson wrote: > > The following changes since commit 29594404d7fe73cd80eaa4ee8c43dcc53970c60e: > > Linux 3.7 (2012-12-10 19:30:57 -0800) > > are available in the git repository at: > > git://github.com/awilliam/linux-vfio.git

Re: pci_pm_runtime_suspend(): azx_runtime_suspend+0x0/0x50 [snd_hda_intel] returns -11

2012-12-11 Thread Rafael J. Wysocki
On Tuesday, December 11, 2012 06:55:08 PM Borislav Petkov wrote: > On Tue, Dec 11, 2012 at 06:48:23PM +0100, Rafael J. Wysocki wrote: > > Boris, please send the output of "lspci -vvv' from that box. > > Attached. So the audio is a Root Complex Integrated Endpoind and there shouldn't be any

Re: [REGRESSION] uml: signal.c build errors

2012-12-11 Thread ryanvade
Richard Weinberger nod.at> writes: > > On 24.10.2012 01:18, Matthew Leach wrote: > > Hi, > > > > When building UML against 3.7-rc2 I encounter the following build error: > > > > # make ARCH=uml > > scripts/kconfig/conf --silentoldconfig arch/x86/um/Kconfig > >CHK

Re: linux-next: some merging notes

2012-12-11 Thread Benjamin Herrenschmidt
On Wed, 2012-12-12 at 09:15 +1100, Stephen Rothwell wrote: > The powerpc tree > (git://git.kernel.org/pub/scm/linux/kernel/git/benh/powerpc.git#next) > contains a commit that breaks the building of > lib/pSeries-reconfig-notifier-error-inject.c. I applied a patch to > linux-next to disable

Re: [RFC v3] Support volatile range for anon vma

2012-12-11 Thread Minchan Kim
Hi John, On Tue, Dec 11, 2012 at 10:45:27AM -0800, John Stultz wrote: > On 12/10/2012 06:34 PM, Minchan Kim wrote: > >This still is [RFC v3] because just passed my simple test > >with TCMalloc tweaking. > > > >I hope more inputs from user-space allocator people and test patch > >with their

Re: [PATCH 2/2] uio: do not expose inode to uio open/release hooks

2012-12-11 Thread Greg KH
On Wed, Dec 12, 2012 at 12:12:02AM +0100, Benedikt Spranger wrote: > The inode parameter is unused by in kernel users of UIO. Ok. > Also the inode parameter makes it hard to resolve the existing open(), > mmap() and close() difficulty. I don't understand, what do you mean by this? What is this

Re: [PATCH 1/2] uio: add warning to documentation

2012-12-11 Thread Greg KH
On Wed, Dec 12, 2012 at 12:12:01AM +0100, Benedikt Spranger wrote: > The documentation has no clear statement to the POSIX 1003.1 mmap() > feature, wich allows open(), mmap(), close() while the mmaped pointer is > valid. > The release() hook inveigled driver programmer to activate owermanagement

epoll with ONESHOT possibly fails to deliver events

2012-12-11 Thread Andreas Voellmy
Hi list, I am using epoll for the Linux (version 3.4.0) implementation of the event notification subsystem of GHC's (Glasgow Haskell Compiler) RTS (runtime system). I am running into a bug that has only popped up using many cores (> 16) and under particular kind of load. I've been debugging

[PATCH 2/2] uio: do not expose inode to uio open/release hooks

2012-12-11 Thread Benedikt Spranger
The inode parameter is unused by in kernel users of UIO. Also the inode parameter makes it hard to resolve the existing open(), mmap() and close() difficulty. Signed-off-by: Benedikt Spranger --- Documentation/DocBook/uio-howto.tmpl |4 ++-- drivers/uio/uio.c|4 ++--

[PATCH 0/2] uio: open(), mmap(), close()

2012-12-11 Thread Benedikt Spranger
After open(), mmap(), close() POSIX 1003.1 mmap() declares the mmaped pointer valid. Add a warning to the documentation to inform UIO users keep this behaviour into account. While the inode parameter of in tree UIO kernel drivers is unused by open/release hooks, remove teh inode parateter. This

[PATCH 1/2] uio: add warning to documentation

2012-12-11 Thread Benedikt Spranger
The documentation has no clear statement to the POSIX 1003.1 mmap() feature, wich allows open(), mmap(), close() while the mmaped pointer is valid. The release() hook inveigled driver programmer to activate owermanagement functuonality in the release hook. This may harm. Signed-off-by: Benedikt

Re: [PATCH 0/2] ima: policy search speedup

2012-12-11 Thread Eric Paris
Linus made it clear he likes per-inode. Do a test with per-inode S_NOIMA. See how that compares to your 100,000 vs 10,000 lookups. Then we can discuss with facts if per sb is worth it or not. We still don't actually know if 100,000 lookups vs 10,000 lookups really matters, but at least we'll

[PATCH 02/37] perf ui: Always compile error printing code

2012-12-11 Thread Arnaldo Carvalho de Melo
From: Namhyung Kim It is used everywhere so always build it regardless of ui engine. Signed-off-by: Namhyung Kim Cc: Andi Kleen Cc: David Ahern Cc: Ingo Molnar Cc: Jiri Olsa Cc: Pekka Enberg Cc: Peter Zijlstra Cc: Steven Rostedt Link:

Re: [PATCH 0/2] ima: policy search speedup

2012-12-11 Thread Kasatkin, Dmitry
On Tue, Dec 11, 2012 at 10:08 PM, Eric Paris wrote: > S_PRIVATE is totally unacceptable as it has a meaning across all LSMs, > not just IMA. > > S_NOSEC means 'this is not setuid or setgid and we don't need to do > those checks on modify' > > You are going to need to use a S_NOIMA. > > Of

[PATCH 07/37] perf tools: Fix mmap limitations on 32-bit

2012-12-11 Thread Arnaldo Carvalho de Melo
From: David Miller This is a suggested patch to fix the bug I reported at: http://marc.info/?l=linux-kernel=135033028924652=2 Essentially, there is a hard requirement that when perf analyzes a trace, it must have the entire thing mmap()'d. Therefore the scheme used on 32-bit where we

[PATCH 34/37] perf evsel: Decode read_format and sample_type in perf_evsel__fprintf

2012-12-11 Thread Arnaldo Carvalho de Melo
From: Arnaldo Carvalho de Melo Before those fields showed just a number, now it decodes each bit: [root@sandy linux]# perf evlist -v cycles: sample_freq=4000, size: 96, sample_type: IP|TID|TIME|CPU|PERIOD, read_format: TOTAL_TIME_ENABLED|TOTAL_TIME_RUNNING|ID, disabled: 1, inherit: 1, mmap:

[PATCH 24/37] perf machine: Move more machine methods to machine.c

2012-12-11 Thread Arnaldo Carvalho de Melo
From: Arnaldo Carvalho de Melo Mechanical, no functional changes. Cc: David Ahern Cc: Frederic Weisbecker Cc: Jiri Olsa Cc: Mike Galbraith Cc: Namhyung Kim Cc: Paul Mackerras Cc: Peter Zijlstra Cc: Stephane Eranian Link:

[PATCH 32/37] perf evsel: Adopt fprintf routine from 'perf evlist'

2012-12-11 Thread Arnaldo Carvalho de Melo
From: Arnaldo Carvalho de Melo So that we can print all the details when debugging other tools, when we have just evlists and evsels, not a perf.data file. Cc: David Ahern Cc: Frederic Weisbecker Cc: Jiri Olsa Cc: Mike Galbraith Cc: Namhyung Kim Cc: Paul Mackerras Cc: Peter Zijlstra Cc:

[PATCH 36/37] perf record: Export the callchain parsing routine and help

2012-12-11 Thread Arnaldo Carvalho de Melo
From: Arnaldo Carvalho de Melo Will be used by perf top, that will first setup the symbol system to deal with callchains and then call these routines to ask the kernel for callchains. Cc: David Ahern Cc: Frederic Weisbecker Cc: Jiri Olsa Cc: Mike Galbraith Cc: Namhyung Kim Cc: Paul

[PATCH 20/37] perf buildid-list: We need to check if a file is ELF first

2012-12-11 Thread Arnaldo Carvalho de Melo
From: Arnaldo Carvalho de Melo I.e. before we try to use it as a perf.data file by calling perf_session__new, otherwise we lose the feature that shows the build id for the given ELF file, this one: [root@sandy redhat-perfdata-mtech-15]# perf buildid-list -i

[PATCH 04/37] perf tools: Don't check configuration on make clean

2012-12-11 Thread Arnaldo Carvalho de Melo
From: Namhyung Kim Current perf build process checks various system configuration on invocation to make. But this is not needed just for cleaning. To do that, move some of python related variables out of conditional since 'clean' target needs them. Normal path should not be affected by this.

[PATCH 03/37] perf ui/helpline: Introduce ui_helpline__vshow()

2012-12-11 Thread Arnaldo Carvalho de Melo
From: Namhyung Kim The ui_helpline__vshow() will be used for pr_* functions. Signed-off-by: Namhyung Kim Cc: Andi Kleen Cc: David Ahern Cc: Ingo Molnar Cc: Jiri Olsa Cc: Pekka Enberg Cc: Peter Zijlstra Cc: Steven Rostedt Link:

[PATCH 23/37] perf buildid-cache: Add option to show build ids that are missing in the cache

2012-12-11 Thread Arnaldo Carvalho de Melo
From: Arnaldo Carvalho de Melo This will allow to connect with services being put in place by distros such as Fedora, where one can retrieve DSOs by their build-id. Example usage: for buildid in $(perf buildid-cache --missing perf.data | cut -d' ' -f1) ; do echo

[PATCH 26/37] perf test: Fixup error reporting in basic mmap test

2012-12-11 Thread Arnaldo Carvalho de Melo
From: Arnaldo Carvalho de Melo In two cases this test could detect an error, bail out but return zero. Fix it by reporting -1 for failure. Cc: David Ahern Cc: Frederic Weisbecker Cc: Jiri Olsa Cc: Mike Galbraith Cc: Namhyung Kim Cc: Paul Mackerras Cc: Peter Zijlstra Cc: Stephane Eranian

[PATCH 30/37] perf evsel: No need to always ask for PERF_FORMAT_TOTAL_TIME_{ENABLED,RUNNING}

2012-12-11 Thread Arnaldo Carvalho de Melo
From: Arnaldo Carvalho de Melo This is needed, so far, just in 'perf stat', to scale counters, so don't unconditionally ask for them in the perf_evsel__config() method. Cc: David Ahern Cc: Frederic Weisbecker Cc: Jiri Olsa Cc: Mike Galbraith Cc: Namhyung Kim Cc: Paul Mackerras Cc: Peter

[PATCH 35/37] perf record: Pass perf_record_opts to the callchain cmdline parsing callback

2012-12-11 Thread Arnaldo Carvalho de Melo
From: Arnaldo Carvalho de Melo Its all it uses and makes the parsing callback suitable for use by 'perf top', which will happen in a followup patch. Cc: David Ahern Cc: Frederic Weisbecker Cc: Jiri Olsa Cc: Mike Galbraith Cc: Namhyung Kim Cc: Paul Mackerras Cc: Peter Zijlstra Cc:

[PATCH 31/37] perf evlist: Set the leader in the perf_evlist__config method

2012-12-11 Thread Arnaldo Carvalho de Melo
From: Arnaldo Carvalho de Melo Since we need to ensure the leader is set before configuring the evsel perf_event_attrs. Reducing the boilerplate needed by tools, helping, for instance, 'perf trace', that wasn't setting the leader. Cc: David Ahern Cc: Frederic Weisbecker Cc: Jiri Olsa Cc:

[PATCH 18/37] perf tools: Don't check configuration on make tags

2012-12-11 Thread Arnaldo Carvalho de Melo
From: Arnaldo Carvalho de Melo Doing the same thing done in: b059dee: perf tools: Don't check configuration on make clean Cc: David Ahern Cc: Frederic Weisbecker Cc: Jiri Olsa Cc: Mike Galbraith Cc: Namhyung Kim Cc: Paul Mackerras Cc: Peter Zijlstra Cc: Stephane Eranian Link:

[PATCH 29/37] perf evsel: No need to always ask for PERF_FORMAT_ID in read_format

2012-12-11 Thread Arnaldo Carvalho de Melo
From: Arnaldo Carvalho de Melo Instead make perf_evlist__confir_attrs use perf_evsel__set_sample_id() when having more than one event, that way only if we have multiple events we'll ask to have the event ids returned when we read its file descriptors. Cc: David Ahern Cc: Frederic Weisbecker

[PATCH 06/37] perf top: Add missing newline on pr_err call

2012-12-11 Thread Arnaldo Carvalho de Melo
From: Arnaldo Carvalho de Melo The perf_event__process_sample function, when not finding a machine associated with a sample, was calling pr_err without a newline, garbling the screen on TUI mode due to a problem introduced by a recent ui_helpline patch. On --stdio it would just concatenate the

[PATCH 37/37] perf top: Use perf_evlist__config()

2012-12-11 Thread Arnaldo Carvalho de Melo
From: Arnaldo Carvalho de Melo Using struct perf_record_opts to specify how to configure the evsel perf_event_attrs. This gets top closer to record in the way it sets up evsels, with the aim of sharing more and more to the point that both will be a single utility. In this direction top now

[PATCH 05/37] perf session: Free environment information when deleting session

2012-12-11 Thread Arnaldo Carvalho de Melo
From: Namhyung Kim The perf session environment information was saved (so allocated) during perf_session__open, but was not freed. As free(3) handles NULL pointer input properly it won't cause a issue for writing modes - e.g. perf record Signed-off-by: Namhyung Kim Cc: Andi Kleen Cc: Feng

[PATCH 08/37] perf hists: Introduce perf_hpp__list for period related columns

2012-12-11 Thread Arnaldo Carvalho de Melo
From: Jiri Olsa Adding perf_hpp__list list to register and contain all period related columns the command is interested in. This way we get rid of static array holding all possible columns and enable commands to register their own columns. It'll be handy for diff command in future to process

[PATCH 19/37] perf tools: Fix TUI helpline output

2012-12-11 Thread Arnaldo Carvalho de Melo
From: Arnaldo Carvalho de Melo In commit e2f4351 "perf ui/helpline: Introduce ui_helpline__vshow()" the test for the browser used made ui_helpline__vshow() to be called only for the GTK browser. The TUI one then was not used and vfprintf(stderr, ...) was used instead, making the TUI scroll the

Re: [patch v2 4/6] memcg: simplify mem_cgroup_iter

2012-12-11 Thread Ying Han
On Tue, Dec 11, 2012 at 8:01 AM, Michal Hocko wrote: > On Mon 10-12-12 20:35:20, Ying Han wrote: >> On Mon, Nov 26, 2012 at 10:47 AM, Michal Hocko wrote: >> > Current implementation of mem_cgroup_iter has to consider both css and >> > memcg to find out whether no group has been found (css==NULL

[PATCH 16/37] perf diff: Change compute methods to work with pair directly

2012-12-11 Thread Arnaldo Carvalho de Melo
From: Jiri Olsa Changing compute methods to operate over hist entry and its pair directly. This makes the code more obvious and readable, instead of all time checking for pair being != NULL. Signed-off-by: Jiri Olsa Acked-by: Namhyung Kim Cc: Corey Ashford Cc: Frederic Weisbecker Cc: Ingo

Re: [RFC] dynamic_debug: introduce debug_hex_dump()

2012-12-11 Thread Luis R. Rodriguez
On Tue, Dec 11, 2012 at 12:24:04PM -0800, Joe Perches wrote: > On Tue, 2012-12-11 at 12:12 -0800, Greg KH wrote: > > On Tue, Dec 11, 2012 at 03:08:31PM -0500, Jason Baron wrote: > > > On Tue, Dec 11, 2012 at 11:36:46AM -0800, Luis R. Rodriguez wrote: > > > > On Tue, Dec 04, 2012 at 11:28:26AM

[PATCH 21/37] perf diff: Remove displacement output option

2012-12-11 Thread Arnaldo Carvalho de Melo
From: Jiri Olsa It seems not very useful, because it's possible and event more convenient to lookup related symbol by name. Also the output value for both 'baseline' and 'new' data is quite apparent from diff output. And above all it complicates hist code factoring ;) Ditching out

[PATCH 17/37] perf diff: Change formula methods to work with pair directly

2012-12-11 Thread Arnaldo Carvalho de Melo
From: Jiri Olsa Changing formula methods to operate over hist entry and its pair directly. This makes the code more obvious and readable, instead of all time checking for pair being != NULL. Signed-off-by: Jiri Olsa Acked-by: Namhyung Kim Cc: Corey Ashford Cc: Frederic Weisbecker Cc: Ingo

[PATCH 22/37] perf symbols: Generalize filter in __fprintf_buildid methods

2012-12-11 Thread Arnaldo Carvalho de Melo
From: Arnaldo Carvalho de Melo We had that 'with_hits' filter to show just the build ids for DSOs that had samples, make that generic so that we can use it in the upcoming buildid-cache --missing feature, to show just the build ids that are not in the cache. Cc: David Ahern Cc: Frederic

[PATCH 15/37] perf diff: Remove displacement from struct hist_entry_diff

2012-12-11 Thread Arnaldo Carvalho de Melo
From: Jiri Olsa Removing displacement from struct hist_entry_diff, because it's not used. Displacement is not used for sorting, so there's no reason to pre-calculate it. Signed-off-by: Jiri Olsa Acked-by: Namhyung Kim Cc: Corey Ashford Cc: Frederic Weisbecker Cc: Ingo Molnar Cc: Namhyung

[PATCH 13/37] perf evsel: Set leader evsel's ->leader to itself

2012-12-11 Thread Arnaldo Carvalho de Melo
From: Namhyung Kim Currently only non-leader members are set ->leader to the leader evsel of the group and the leader has set NULL. Thus it requires special casing for leader evsels. Set ->leader to itself will remove this. Suggested-by: Arnaldo Carvalho de Melo Signed-off-by: Namhyung Kim

[PATCH 27/37] perf test: Use perf_evsel__newtp constructor in the tracepoint tests

2012-12-11 Thread Arnaldo Carvalho de Melo
From: Arnaldo Carvalho de Melo Removing one trace_event__id function, not used anymore. Cc: David Ahern Cc: Frederic Weisbecker Cc: Jiri Olsa Cc: Mike Galbraith Cc: Namhyung Kim Cc: Paul Mackerras Cc: Peter Zijlstra Cc: Stephane Eranian Link:

[PATCH 01/37] perf symbols: Fix dso__fprintf() print statement

2012-12-11 Thread Arnaldo Carvalho de Melo
From: Stephane Eranian Was ignoring the dso type (function vs. variable) and was therefore printing bogus information. Signed-off-by: Stephane Eranian Cc: Ingo Molnar Cc: Namhyung Kim Cc: Peter Zijlstra Link: http://lkml.kernel.org/r/20121120095101.GA5939@quad Signed-off-by: Arnaldo

[PATCH 14/37] perf evsel: Convert to _is_group_leader method

2012-12-11 Thread Arnaldo Carvalho de Melo
From: Namhyung Kim Convert perf_evsel__is_group_member to perf_evsel__is_group_leader. This is because the most usecases are using negative form to check whether the given evsel is a leader or not and it's IMHO somewhat ambiguous - leader also *is* a member of the group. Signed-off-by: Namhyung

[PATCH 33/37] perf tools: Add install-bin Makefile target

2012-12-11 Thread Arnaldo Carvalho de Melo
From: Arnaldo Carvalho de Melo Its too annoying to go over the Documentation install target while developing the tools. Cc: David Ahern Cc: Frederic Weisbecker Cc: Jiri Olsa Cc: Mike Galbraith Cc: Namhyung Kim Cc: Paul Mackerras Cc: Peter Zijlstra Cc: Stephane Eranian Link:

[PATCH 09/37] perf hists: Fix period symbol_conf.field_sep display

2012-12-11 Thread Arnaldo Carvalho de Melo
From: Jiri Olsa Currently we don't properly display hist data with symbol_conf.field_sep separator. We need to display either space or separator. Signed-off-by: Jiri Olsa Cc: Arnaldo Carvalho de Melo Cc: Peter Zijlstra Cc: Ingo Molnar Cc: Paul Mackerras Cc: Corey Ashford Cc: Frederic

[PATCH 10/37] perf symbols: Ignore ABS symbols when loading data maps

2012-12-11 Thread Arnaldo Carvalho de Melo
From: Namhyung Kim When loading symbols in a data mapping, ABS symbols (which has a value of SHN_ABS in its st_shndx) failed at elf_getscn(). And it marks the loading as a failure so already loaded symbols cannot be fixed up. I'm not sure what should be done. Just ignore them for now. :)

[PATCH 28/37] perf evsel: Introduce method to request IDs be used

2012-12-11 Thread Arnaldo Carvalho de Melo
From: Arnaldo Carvalho de Melo When mmaping multiple events we need to find the right evsel that matches an event in the ring buffer. For that we need to set the PERF_FORMAT_ID bit in perf_event_attr.read_format so that when we read the event fds we get that id to then hash it and be able later

[PATCH 12/37] perf hists: Link hist entry pairs to leader

2012-12-11 Thread Arnaldo Carvalho de Melo
From: Namhyung Kim Current hists__match/link() link a leader to its pair, so if multiple pairs were linked, the leader will lose pointer to previous pairs since it was overwritten. Fix it by making leader the list head. Signed-off-by: Namhyung Kim Cc: Andi Kleen Cc: Ingo Molnar Cc: Jiri

[GIT PULL 00/37] perf/core improvements and fixes

2012-12-11 Thread Arnaldo Carvalho de Melo
Hi Ingo, Please consider pulling. Jiri, tomorrow I'll go over the hists patches as I think you're fully ok with Namhyung latest patchset. Regards, - Arnaldo The following changes since commit cc1b39dbf9f55a438e8a21a694394c20e6a17129: Merge branch 'tip/perf/core' of

[PATCH 25/37] perf evsel: Update sample_size when setting sample_type bits

2012-12-11 Thread Arnaldo Carvalho de Melo
From: Arnaldo Carvalho de Melo We use evsel->sample_size to detect underflows in perf_evsel__parse_sample, but we were failing to update it after perf_evsel__init(), i.e. when we decide, after creating an evsel, that we want some extra field bit set. Fix it by introducing methods to set a bit

[PATCH 11/37] perf hists: Fix typo on hist__entry_add_pair

2012-12-11 Thread Arnaldo Carvalho de Melo
From: Namhyung Kim Fix a misplaced underscore. In this case, 'hist_entry' is the name of data structure and we usually put double underscores between data structure and actual function name. Signed-off-by: Namhyung Kim Acked-by: Jiri Olsa Cc: Andi Kleen Cc: Ingo Molnar Cc: Jiri Olsa , Cc:

Re: [PATCH] [RESEND] x86/kexec: crash_vmclear_local_vmcss needs __rcu

2012-12-11 Thread Marcelo Tosatti
On Tue, Dec 11, 2012 at 05:11:34PM +0800, Zhang Yanfei wrote: > This removes the sparse warning: > arch/x86/kernel/crash.c:49:32: sparse: incompatible types in comparison > expression (different address spaces) > > Reported-by: kbuild test robot > Signed-off-by: Zhang Yanfei > --- >

Re: [patch v2 3/6] memcg: rework mem_cgroup_iter to use cgroup iterators

2012-12-11 Thread Ying Han
On Tue, Dec 11, 2012 at 8:15 AM, Michal Hocko wrote: > On Tue 11-12-12 16:50:25, Michal Hocko wrote: >> On Sun 09-12-12 08:59:54, Ying Han wrote: >> > On Mon, Nov 26, 2012 at 10:47 AM, Michal Hocko wrote: >> [...] >> > > + /* >> > > +* Even if we found a group we

Re: [patch v2 3/6] memcg: rework mem_cgroup_iter to use cgroup iterators

2012-12-11 Thread Ying Han
On Tue, Dec 11, 2012 at 7:54 AM, Michal Hocko wrote: > On Sun 09-12-12 11:39:50, Ying Han wrote: >> On Mon, Nov 26, 2012 at 10:47 AM, Michal Hocko wrote: > [...] >> > if (reclaim) { >> > - iter->position = id; >> > + struct mem_cgroup

Re: [RFC v1 01/16] lib: devres: don't enclose pcim_*() functions in CONFIG_HAS_IOPORT

2012-12-11 Thread Arnd Bergmann
On Tuesday 11 December 2012, Russell King - ARM Linux wrote: > > On Tue, Dec 11, 2012 at 04:15:02PM +, Arnd Bergmann wrote: > > On Tuesday 11 December 2012, Thomas Petazzoni wrote: > > > On Tue, 11 Dec 2012 10:43:49 +, Arnd Bergmann wrote: > > > > On Friday 07 December 2012, Thomas

Re: [patch v2 3/6] memcg: rework mem_cgroup_iter to use cgroup iterators

2012-12-11 Thread Ying Han
On Tue, Dec 11, 2012 at 7:50 AM, Michal Hocko wrote: > On Sun 09-12-12 08:59:54, Ying Han wrote: >> On Mon, Nov 26, 2012 at 10:47 AM, Michal Hocko wrote: > [...] >> > + /* >> > +* Even if we found a group we have to make sure it is >> > alive. >> > +

Re: 3.7 XFS lockdep trace

2012-12-11 Thread Dave Chinner
On Tue, Dec 11, 2012 at 10:42:07AM -0500, Dave Jones wrote: > This says rc8+, but it's just missing the Makefile change, so it's still > there in 3.7 > Curious that firefox was the process mentioned here, as ~/.mozilla isn't on > xfs. > My only xfs partition is /data holding a kernel source tree

Re: [RFC v1 01/16] lib: devres: don't enclose pcim_*() functions in CONFIG_HAS_IOPORT

2012-12-11 Thread Arnd Bergmann
On Tuesday 11 December 2012, Alan Cox wrote: > > Plus, if you have IO space support, you must have some MMIO region for > > them to target - doing what many platforms have done to date and targetted > > ISA IO address 0 at virtual address 0 is just not on because as soon as > > you build a device

Re: linux-next: recent rebases

2012-12-11 Thread Stephen Rothwell
Hi Eric, On Tue, 11 Dec 2012 16:58:43 -0500 Eric Paris wrote: > > On Wed, 2012-12-12 at 08:41 +1100, Stephen Rothwell wrote: > > > > fsnotify: rebased to v3.6(!) to resolve a conflict. > > git://git.infradead.org/users/eparis/notify.git#for-next > > I planned to let it sit in -next until

Re: [RFC v1 01/16] lib: devres: don't enclose pcim_*() functions in CONFIG_HAS_IOPORT

2012-12-11 Thread Arnd Bergmann
On Tuesday 11 December 2012, Alan Cox wrote: > The "no I/O space" case really applies to things like the S/390 mainframe > which simply have no such concept on the system or the devices. In the > ARM case the bus has an I/O space and the bridge glues the processors > simpler model to the bridge

Re: [PATCH 00/19] MAINTAINERS: fix file patterns

2012-12-11 Thread Joe Perches
On Tue, 2012-12-11 at 19:49 -0200, Cesar Eduardo Barros wrote: > I could not find the maintainer for the MAINTAINERS file, so sending > directly to Linus. Sorry if I am mistaken. Thanks Cesar. Most maintainers patches are picked up by Andrew Morton (cc'd) -- To unsubscribe from this list: send

linux-next: some merging notes

2012-12-11 Thread Stephen Rothwell
Hi Linus, Just some notes about the current state of some of the merges in linux-next. The powerpc tree (git://git.kernel.org/pub/scm/linux/kernel/git/benh/powerpc.git#next) contains a commit that breaks the building of lib/pSeries-reconfig-notifier-error-inject.c. I applied a patch to

Re: [RFC PATCH 0/8] remove vm_struct list management

2012-12-11 Thread Dave Anderson
- Original Message - > On Mon, Dec 10, 2012 at 11:40:47PM +0900, JoonSoo Kim wrote: > > [..] > > > So without knowing details of both the data structures, I think if vmlist > > > is going away, then user space tools should be able to traverse > > > vmap_area_root > > > rb tree. I am

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