[PATCH] MAINTAINERS: drivers/power: add entry for SmartReflex AVS drivers

2013-08-05 Thread Kevin Hilman
The SmartReflex AVS driver evolved out of the OMAP kernel and now lives under drivers/power/avs. I've historically been maintainer of this but Nishanth Menon is doing most of the heavy lifting now. Add us both as co-maintainers. Cc: Anton Vorontsov Cc: David Woodhouse Cc: Rafael J. Wysocki

Re: [PATCHSET cgroup/for-3.12] cgroup: make cgroup_event specific to memcg

2013-08-05 Thread Tejun Heo
Hello, Michal. On Mon, Aug 05, 2013 at 06:01:07PM +0200, Michal Hocko wrote: > Could you be more specific about what is so "overboard" about this > interface? I am not familiar with internals much, so I cannot judge the > complexity part, but I thought that eventfd was intended for this kind > of

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

2013-08-05 Thread Andy Lutomirski
The systemd commit below can delay firmware loading by multiple minutes if CONFIG_FW_LOADER_USER_HELPER=y. Unfortunately no one noticed that the systemd-udev change would break new kernels as well as old kernels. Since the kernel apparently can't count on reasonable userspace support, turn this

Re: [PATCH] trivial: adjust code alignment

2013-08-05 Thread walter harms
Hello Julia, IMHO keep the patch as it is. It does not change any code that is good. Suspicious code that comes up here can be addressed in a separate patch. just my 2 cents, re, wh Am 05.08.2013 18:19, schrieb Julia Lawall: > On Mon, 5 Aug 2013, Dan Carpenter wrote: > >> On Mon, Aug 05, 2013

Re: [PATCH] trivial: adjust code alignment

2013-08-05 Thread Dan Carpenter
On Mon, Aug 05, 2013 at 09:17:07AM -0700, Joe Perches wrote: > On Mon, 2013-08-05 at 19:06 +0300, Dan Carpenter wrote: > > On Mon, Aug 05, 2013 at 04:47:39PM +0200, Julia Lawall wrote: > > > diff --git a/drivers/media/i2c/ov7670.c b/drivers/media/i2c/ov7670.c > [] > > > @@ -1369,8 +1369,8 @@

[PATCH v3 0/10] enable swiotlb-xen on arm and arm64

2013-08-05 Thread Stefano Stabellini
Hi all, this patch series enables xen-swiotlb on arm and arm64. Considering that all guests, including dom0, run on xen on arm with second stage translation enabled, it follows that without an IOMMU no guests could actually drive the hardware. The solution for platforms without an IOMMU is to

Re: [PATCH RESEND] PM / AVS: SmartReflex: remove redundant dev_err call in omap_sr_probe()

2013-08-05 Thread Kevin Hilman
Wei Yongjun writes: > There is a error message within devm_ioremap_resource > already, so remove the dev_err call to avoid redundant > error message. > > Signed-off-by: Wei Yongjun > Acked-by: Nishanth Menon > --- > cc to linux-pm with Nishanth Menon's ACK Thanks, queued for v3.12. Kevin >

Re: [edk2] Corrupted EFI region

2013-08-05 Thread Laszlo Ersek
Apologies in advance for my response because it diverges from the technical stuff. On 08/05/13 17:34, James Bottomley wrote: > On Mon, 2013-08-05 at 17:15 +0200, Laszlo Ersek wrote: >> On 08/05/13 16:40, Borislav Petkov wrote: >>> On Mon, Aug 05, 2013 at 04:27:44PM +0200, Laszlo Ersek wrote:

Re: [PATCH v2 3/3] dma: Add Freescale eDMA engine driver support

2013-08-05 Thread Vinod Koul
On Mon, Aug 05, 2013 at 09:53:45AM +0200, Lothar Waßmann wrote: > Hi, > > Jingchang Lu writes: > > Add Freescale enhanced direct memory(eDMA) controller support. > > The eDMA controller deploys DMAMUXs routing DMA request sources(slot) > > to eDMA channels. > > This module can be found on Vybrid

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

2013-08-05 Thread Sudeep KarkadaNagesha
Hi Daniel, On 02/08/13 13:12, Daniel Lezcano wrote: > On 08/02/2013 11:55 AM, Sudeep KarkadaNagesha wrote: >> On 02/08/13 10:48, Daniel Lezcano wrote: >>> On 08/02/2013 11:32 AM, Sudeep KarkadaNagesha wrote: Hi Daniel, On 22/07/13 12:21, Sudeep KarkadaNagesha wrote: > From:

Re: [PATCH] trivial: adjust code alignment

2013-08-05 Thread Julia Lawall
On Mon, 5 Aug 2013, walter harms wrote: Hello Julia, IMHO keep the patch as it is. It does not change any code that is good. Suspicious code that comes up here can be addressed in a separate patch. OK, thanks! julia -- To unsubscribe from this list: send the line "unsubscribe linux-kernel"

Re: [PATCH] zram: bug fix: delay lock holding in zram_slot_free_noity

2013-08-05 Thread Minchan Kim
On Mon, Aug 05, 2013 at 04:18:34PM +0900, Minchan Kim wrote: > I was preparing to promote zram and it was almost done. > Before sending patch, I tried to test and eyebrows went up. > > [1] introduced down_write in zram_slot_free_notify to prevent race > between zram_slot_free_notify and

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

2013-08-05 Thread Sudeep KarkadaNagesha
On 01/08/13 13:02, Jason Cooper wrote: > Sudeep, > > On Thu, Aug 01, 2013 at 10:54:44AM +0100, Sudeep KarkadaNagesha wrote: >> On 22/07/13 12:32, Sudeep KarkadaNagesha wrote: >>> From: Sudeep KarkadaNagesha >>> >>> Currently set_secondary_cpus_clock assume the CPU logical ordering >>> and the

[PATCH V9 08/14] perf tools: move struct ip_event

2013-08-05 Thread Adrian Hunter
struct ip_event assumes fixed positions for ip, pid and tid. That is no longer true with the addition of PERF_SAMPLE_IDENTIFIER. struct ip_event is no longer used except by hists_link.c. Move it there. Signed-off-by: Adrian Hunter --- tools/perf/tests/hists_link.c | 27

[PATCH V9 07/14] perf tools: remove references to struct ip_event

2013-08-05 Thread Adrian Hunter
struct ip_event assumes fixeed positions for ip, pid and tid. That is no longer true with the addition of PERF_SAMPLE_IDENTIFIER. The information is anyway in struct sample, so use that instead. Signed-off-by: Adrian Hunter --- tools/perf/builtin-inject.c | 4 ++-- tools/perf/builtin-kmem.c

[PATCH V9 09/14] perf: make events stream always parsable

2013-08-05 Thread Adrian Hunter
The event stream is not always parsable because the format of a sample is dependent on the sample_type of the selected event. When there is more than one selected event and the sample_types are not the same then parsing becomes problematic. A sample can be matched to its selected event using the

[PATCH V9 05/14] perf tools: tidy up sample parsing overflow checking

2013-08-05 Thread Adrian Hunter
The size of data retrieved from a sample event must be validated to ensure it does not go past the end of the event. That was being done sporadically and without considering integer overflows. Signed-off-by: Adrian Hunter --- tools/perf/util/evsel.c | 112

[PATCH V9 10/14] perf tools: move perf_evlist__config() to a new source file

2013-08-05 Thread Adrian Hunter
perf_evlist__config() must be moved to a separate source file to avoid Python link errors when adding support for PERF_SAMPLE_IDENTIFIER. It is appropriate to do this because perf_evlist__config() is a helper function for event recording. It is used by tools to apply recording options to

next-20130805 - compile errors in drivers/staging/lustre/lustre/llite/file.c

2013-08-05 Thread Valdis Kletnieks
Not sure how this train wreck happened - 'git log' shows the last patch to touch include/linux/aio.h was some stuff of Kent's back in May, and 'git blame' says the 2 lines of code that died were added on 2013-05-02. It compiled when I build next-0730. And yet, this morning, I get this building

[PATCH V9 14/14] perf tools: add a sample parsing test

2013-08-05 Thread Adrian Hunter
Add a test that checks that sample parsing is correctly implemented. Signed-off-by: Adrian Hunter --- tools/perf/Makefile | 1 + tools/perf/tests/builtin-test.c | 4 + tools/perf/tests/sample-parsing.c | 294 ++ tools/perf/tests/tests.h

[PATCH V9 04/14] perf tools: change machine__findnew_thread() to set thread pid

2013-08-05 Thread Adrian Hunter
Add a new parameter for 'pid' to machine__findnew_thread(). Change callers to pass 'pid' when it is known. Signed-off-by: Adrian Hunter --- tools/perf/builtin-inject.c | 2 +- tools/perf/builtin-kmem.c | 3 ++- tools/perf/builtin-kvm.c | 3 ++- tools/perf/builtin-lock.c | 3

[PATCH V9 11/14] perf tools: add support for PERF_SAMPLE_IDENTFIER

2013-08-05 Thread Adrian Hunter
Enable parsing of samples with sample format bit PERF_SAMPLE_IDENTFIER. In addition, if the kernel supports it, prefer it to selecting PERF_SAMPLE_ID thereby avoiding the need to force compatible sample types. Signed-off-by: Adrian Hunter --- tools/perf/tests/mmap-basic.c | 2 +-

[PATCH V9 12/14] perf tools: expand perf_event__synthesize_sample()

2013-08-05 Thread Adrian Hunter
Expand perf_event__synthesize_sample() to handle all sample format bits. Signed-off-by: Adrian Hunter --- tools/perf/builtin-inject.c | 4 +- tools/perf/util/event.h | 1 + tools/perf/util/evsel.c | 94 - 3 files changed, 97 insertions(+), 2

[PATCH V9 06/14] perf tools: remove unnecessary callchain validation

2013-08-05 Thread Adrian Hunter
Now that the sample parsing correctly checks data sizes there is no reason for it to be done again for callchains. Signed-off-by: Adrian Hunter --- tools/perf/util/callchain.c | 8 tools/perf/util/callchain.h | 5 - tools/perf/util/session.c | 20 3 files

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

2013-08-05 Thread Jeff Layton
On Mon, 05 Aug 2013 16:50:37 +0100 Nix wrote: > On 5 Aug 2013, Jeff Layton said: > > > On Mon, 5 Aug 2013 11:04:27 -0400 > > Jeff Layton wrote: > > > >> On Mon, 05 Aug 2013 15:48:01 +0100 > >> Nix wrote: > >> > >> > On 5 Aug 2013, Jeff Layton stated: > >> > > >> > > On Sun, 04 Aug 2013

Re: [PATCH v1 0/2] PM / AVS: interrupt handling fixes

2013-08-05 Thread Kevin Hilman
Andrii Tseglytskyi writes: > Hi Kevin, > > Could you please take a look to the following patch series. > It consists of two patches, which are needed for proper > SmartReflex Interrupt handling. Thanks. It doesn't appear that these are regression fixes, so I'm queuing for v3.12. Kevin

[PATCH V9 13/14] perf tools: add a function to calculate sample event size

2013-08-05 Thread Adrian Hunter
Add perf_event__sample_event_size() which can be used when synthesizing sample events to determine how big the resulting event will be, and therefore how much memory to allocate. Signed-off-by: Adrian Hunter --- tools/perf/util/event.h | 2 ++ tools/perf/util/evsel.c | 92

Re: [alsa-devel] [PATCH] ASoC: codecs: da9055: Update driver name to fix breakage due to pmic driver with same name

2013-08-05 Thread Mark Brown
On Mon, Aug 05, 2013 at 09:21:37PM +0530, Ashish Chavan wrote: > On Mon, 2013-08-05 at 15:42 +0100, Mark Brown wrote: > > No, I'm talking about the same thing I was talking about originally. > Thanks for confirming it. From our view point, we still feel that it's > not a good design which

Re: [RFC PATCH] cfq-iosched: limit slice_idle when many busy queues are in idle window

2013-08-05 Thread Tomoki Sekiyama
On 8/1/13 17:04 , "Jens Axboe" wrote: >On 08/01/2013 02:28 PM, Tomoki Sekiyama wrote: >> On 7/30/13 10:09 PM, Shaohua Li wrote: >>> On Tue, Jul 30, 2013 at 03:30:33PM -0400, Tomoki Sekiyama wrote: Hi, When some application launches several hundreds of processes that issue

[PATCH V9 02/14] perf tools: allow non-matching sample types

2013-08-05 Thread Adrian Hunter
Sample types need not be identical to determine the sample id from the event. Only the position of the sample id needs to be the same. Compatible sample types are ones in which the bits defined by PERF_COMPAT_MASK are the same. 'perf_evlist__config()' forces sample types to be compatible on that

[PATCH V9 01/14] perf tools: add debug prints

2013-08-05 Thread Adrian Hunter
It is useful to see the arguments to perf_event_open and whether the perf events ring buffer was mmapped per-cpu or per-thread. That information will now be displayed when verbose is 2 i.e option -vv Signed-off-by: Adrian Hunter --- tools/perf/util/evlist.c | 3 +++ tools/perf/util/evsel.c |

[PATCH v3 02/12] ARM: edma: Don't clear EMR of channel in edma_stop

2013-08-05 Thread Joel Fernandes
We certainly don't want error conditions to be cleared any other place but the EDMA error handler, as this will make us 'forget' about missed events we might need to know errors have occurred. This fixes a race condition where the EMR was being cleared by the transfer completion interrupt

[PATCH v3 09/12] dma: edma: Implement multiple linked sets for continuity

2013-08-05 Thread Joel Fernandes
Here we implement splitting up of the total MAX number of slots available for a channel into 2 cyclically linked sets. Transfer completion Interrupts are enabled on both linked sets and respective handler recycles them on completion to process the next linked set. Both linked sets are cyclically

[PATCH V9 03/14] perf tools: add pid to struct thread

2013-08-05 Thread Adrian Hunter
Record pid on struct thread. The member is named 'pid_' to avoid confusion with the 'tid' member which was previously named 'pid'. Signed-off-by: Adrian Hunter --- tools/perf/util/machine.c | 16 +++- tools/perf/util/thread.c | 3 ++- tools/perf/util/thread.h | 3 ++- 3 files

[PATCH v3 01/12] dma: edma: Setup parameters to DMA MAX_NR_SG at a time

2013-08-05 Thread Joel Fernandes
Changes are made here for configuring existing parameters to support DMA'ing them out in batches as needed. Also allocate as many as slots as needed by the SG list, but not more than MAX_NR_SG. Then these slots will be reused accordingly. For ex, if MAX_NR_SG=10, and number of SG entries is 40,

Re: [PATCH] trivial: adjust code alignment

2013-08-05 Thread Julia Lawall
On Mon, 5 Aug 2013, Dan Carpenter wrote: On Mon, Aug 05, 2013 at 04:47:39PM +0200, Julia Lawall wrote: diff --git a/drivers/media/i2c/ov7670.c b/drivers/media/i2c/ov7670.c index e8a1ce2..4a5a5dc 100644 --- a/drivers/media/i2c/ov7670.c +++ b/drivers/media/i2c/ov7670.c @@ -1369,8 +1369,8 @@

[PATCH v3 07/12] dma: edma: Add function to dump a PaRAM set from PaRAM

2013-08-05 Thread Joel Fernandes
Previously, such a dump function was used but it wasn't reading from the PaRAM, rather just from a edmacc_param structure, we add a helpful function for debugging that directly reads from the PaRAM and gives the current state correctly (including links and interrupt information). Signed-off-by:

[PATCH v3 00/12] edma: Add support for SG lists of any length

2013-08-05 Thread Joel Fernandes
Here is a more improved approach for DMA support of SG lists of any length in the EDMA DMA Engine driver. In the previous approach [1] we depended on error interrupts to detect missed events and manually retrigger them, however as discussed in [2], there are concerns this can be trouble some for

[PATCH v3 11/12] dma: edma: Keep tracking of Pending interrupts (pending_acks)

2013-08-05 Thread Joel Fernandes
The total_processed variable was being used to keep track of when DMA is completed for a particular descriptor. This doesn't work anymore, as the interrupt for completion can come in much later than when the total_processed variable is updated to reflect that all SG entries have been issues.

[PATCH v3 04/12] dma: edma: Write out and handle MAX_NR_SG at a given time

2013-08-05 Thread Joel Fernandes
Process SG-elements in batches of MAX_NR_SG if they are greater than MAX_NR_SG. Due to this, at any given time only those many slots will be used in the given channel no matter how long the scatter list is. We keep track of how much has been written inorder to process the next batch of elements in

[PATCH v3 03/12] dma: edma: remove limits on number of slots

2013-08-05 Thread Joel Fernandes
With this series, this check is no longer required and we shouldn't need to reject drivers DMA'ing more than the MAX number of slots. Signed-off-by: Joel Fernandes --- drivers/dma/edma.c |6 -- 1 file changed, 6 deletions(-) diff --git a/drivers/dma/edma.c b/drivers/dma/edma.c index

Re: [PATCH] trivial: adjust code alignment

2013-08-05 Thread Joe Perches
On Mon, 2013-08-05 at 19:06 +0300, Dan Carpenter wrote: > On Mon, Aug 05, 2013 at 04:47:39PM +0200, Julia Lawall wrote: > > diff --git a/drivers/media/i2c/ov7670.c b/drivers/media/i2c/ov7670.c [] > > @@ -1369,8 +1369,8 @@ static int ov7670_s_exp(struct v4l2_subdev *sd, int > > value) > >

[PATCH v3 10/12] dma: edma: Check if MAX_NR_SG is even in prep function

2013-08-05 Thread Joel Fernandes
Splitting of MAX available slots into 2 sets of size MAX_NR_LS requires to the MAX_NR_SG function to be even. We ensure the same in prep function. Signed-off-by: Joel Fernandes --- drivers/dma/edma.c |3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/dma/edma.c

[PATCH v3 12/12] dma: edma: Return if nothing left todo in edma_execute

2013-08-05 Thread Joel Fernandes
It is possible edma_execute is called even when all the SG elements have been submitted for transmission, we add a check for the same and avoid executing the rest of the function. Signed-off-by: Joel Fernandes --- drivers/dma/edma.c |3 +++ 1 file changed, 3 insertions(+) diff --git

[PATCH v3 06/12] ARM: edma: Add pr_debug in edma_link

2013-08-05 Thread Joel Fernandes
Useful for visualizing linking of PaRAM slots Signed-off-by: Joel Fernandes --- arch/arm/common/edma.c |2 ++ 1 file changed, 2 insertions(+) diff --git a/arch/arm/common/edma.c b/arch/arm/common/edma.c index aa43c49..34d3fc9 100644 --- a/arch/arm/common/edma.c +++ b/arch/arm/common/edma.c

[PATCH v3 05/12] ARM: edma: Add function to enable interrupt for a PaRAM slot

2013-08-05 Thread Joel Fernandes
To prevent common programming errors, add a function to enable interrupts correctly. Also keeps calling code more readable. Signed-off-by: Joel Fernandes --- arch/arm/common/edma.c | 15 +++ include/linux/platform_data/edma.h |1 + 2 files changed, 16 insertions(+)

[PATCH v3 08/12] dma: edma: Add one more required slot to MAX slots

2013-08-05 Thread Joel Fernandes
We'd now need a separate slot just for the channel and separate ones for the 2 linked sets, so we make adjustments to allocate an extra channel accordingly. Signed-off-by: Joel Fernandes --- drivers/dma/edma.c |5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git

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

2013-08-05 Thread Myklebust, Trond
On Mon, 2013-08-05 at 16:50 +0100, Nix wrote: > On 5 Aug 2013, Jeff Layton said: > > > On Mon, 5 Aug 2013 11:04:27 -0400 > > Jeff Layton wrote: > > > >> On Mon, 05 Aug 2013 15:48:01 +0100 > >> Nix wrote: > >> > >> > On 5 Aug 2013, Jeff Layton stated: > >> > > >> > > On Sun, 04 Aug 2013

Re: [PATCH] sound/sgtl5000: fix codec clock source transition to avoid clockless moment

2013-08-05 Thread Mark Brown
On Mon, Aug 05, 2013 at 07:36:02AM +, Oskar Schirmer wrote: > Powering down PLL before switching to a mode that does not use it > is a bad idea. It would cause the SGTL5000 be without internal > clock supply, especially on the I2C interface, which would make > subsequent access to it fail.

Re: [PATCH] trivial: adjust code alignment

2013-08-05 Thread Dan Carpenter
On Mon, Aug 05, 2013 at 04:47:39PM +0200, Julia Lawall wrote: > diff --git a/drivers/media/i2c/ov7670.c b/drivers/media/i2c/ov7670.c > index e8a1ce2..4a5a5dc 100644 > --- a/drivers/media/i2c/ov7670.c > +++ b/drivers/media/i2c/ov7670.c > @@ -1369,8 +1369,8 @@ static int ov7670_s_exp(struct

[PATCH] memcg: don't initialize kmem-cache destroying work for root caches

2013-08-05 Thread Andrey Vagin
struct memcg_cache_params has a union. Different parts of this union are used for root and non-root caches. A part with destroying work is used only for non-root caches. I fixed the same problem in another place v3.9-rc1-16204-gf101a94, but didn't notice this one. Cc: Glauber Costa Cc: Johannes

Re: [PATCH 1/2] input: ti_tsc: Enable shared IRQ for TSC

2013-08-05 Thread Dmitry Torokhov
On Sun, Aug 04, 2013 at 12:08:11PM +0100, Jonathan Cameron wrote: > On 07/27/13 00:51, Zubair Lutfullah wrote: > > From: "Patil, Rachna" > > > > Touchscreen and ADC share the same IRQ line from parent MFD core. > > Previously only Touchscreen was interrupt based. > > With continuous mode support

Re: [edk2] Corrupted EFI region

2013-08-05 Thread Borislav Petkov
On Mon, Aug 05, 2013 at 05:15:38PM +0200, Laszlo Ersek wrote: > The current implementation (how pointers are converted) probably doesn't > accommodate a second call. > > Of course you want to know why SetVirtualAddressMap() was designed like > that... I didn't participate in the design so I don't

Re: [PATCH 2/5] tools, perf: Add support to evsel for enabling counters

2013-08-05 Thread Arnaldo Carvalho de Melo
Em Mon, Aug 05, 2013 at 05:28:14PM +0900, Namhyung Kim escreveu: > Hi Andi, > > On Fri, 2 Aug 2013 17:41:10 -0700, Andi Kleen wrote: > > From: Andi Kleen > > > > Add support for enabling already set up counters by using an > > ioctl. I share some code with the filter setup. > > > >

Re: [systemd-devel] Slow firmware timeouts again (Re: [3.11 regression?] iwlwifi firmware takes two minutes to load)

2013-08-05 Thread Andy Lutomirski
On Mon, Aug 5, 2013 at 4:18 AM, Kay Sievers wrote: > On Fri, Aug 2, 2013 at 6:28 PM, Zbigniew Jędrzejewski-Szmek > wrote: >> On Fri, Aug 02, 2013 at 09:04:44AM -0700, Andy Lutomirski wrote: >>> CONFIG_FW_LOADER_USER_HELPER=y >> Do you need this? Unsetting this should help. >> >> """This option

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

2013-08-05 Thread Tony Luck
On Mon, Aug 5, 2013 at 3:56 AM, Matt Fleming wrote: >> @@ -424,6 +424,7 @@ extern void __iomem * ioremap(unsigned long offset, >> unsigned long size); >> extern void __iomem * ioremap_nocache (unsigned long offset, unsigned long >> size); >> extern void iounmap (volatile void __iomem *addr);

Re: [alsa-devel] [PATCH v4] ASoC: core: init delayed_work for codec-codec links

2013-08-05 Thread Mark Brown
On Mon, Aug 05, 2013 at 01:17:28PM +0100, Richard Fitzgerald wrote: > We must init the delayed_work for codec-codec links > otherwise shutting down the DAI chain will fault when > calling flush_delayed_work_sync() on the linked DAI. Applied, thanks - please send using a normal patch subject line

Re: aio: kernel BUG at fs/aio.c:646!

2013-08-05 Thread Benjamin LaHaise
Hi Sasha, On Mon, Aug 05, 2013 at 09:57:08AM -0400, Sasha Levin wrote: > Hi all, > > While fuzzing with trinity inside a KVM tools guest running latest -next > kernel, > I've stumbled on the following spew caused by a new BUG() added in "aio: fix > io_destroy() regression by using call_rcu()".

Re: [PATCHSET cgroup/for-3.12] cgroup: make cgroup_event specific to memcg

2013-08-05 Thread Michal Hocko
On Sun 04-08-13 12:07:21, Tejun Heo wrote: > Hello, Hi Tejun, > Like many other things in cgroup, cgroup_event is way too flexible and > complex - it strives to provide completely flexible event monitoring > facility in cgroup proper which allows any number of users to monitor > custom events.

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

2013-08-05 Thread Nix
On 5 Aug 2013, Jeff Layton said: > On Mon, 5 Aug 2013 11:04:27 -0400 > Jeff Layton wrote: > >> On Mon, 05 Aug 2013 15:48:01 +0100 >> Nix wrote: >> >> > On 5 Aug 2013, Jeff Layton stated: >> > >> > > On Sun, 04 Aug 2013 16:40:58 +0100 >> > > Nix wrote: >> > > >> > >> I just got this panic on

Re: [PATCH] WIP: HACK: LPAE, BOOTMEM and NO_BOOTMEM

2013-08-05 Thread Santosh Shilimkar
On Monday 05 August 2013 11:38 AM, Tejun Heo wrote: > Hello, > > On Mon, Aug 05, 2013 at 11:29:46AM -0400, Santosh Shilimkar wrote: >> So looks like I am bit confused here. The current memblock_alloc() >> API just returns the physical address which not mapped memory. > > The memory returned by

Re: [PATCH 4/9] perf kvm: split out tracepoints from record args

2013-08-05 Thread Arnaldo Carvalho de Melo
Em Mon, Aug 05, 2013 at 01:09:47PM +0800, Xiao Guangrong escreveu: > On 08/03/2013 04:05 AM, David Ahern wrote: > > Needed by kvm live command. Make record_args a local while we are > > messing with the args. > > Reviewed-by: Xiao Guangrong Merged up to this one, waiting for the rest with

Re: [PATCH 2/5] cgroup: export __cgroup_from_dentry() and __cgroup_dput()

2013-08-05 Thread Tejun Heo
On Mon, Aug 05, 2013 at 10:58:13AM +0800, Li Zefan wrote: > > +struct cgroup *__cgroup_from_dentry(struct dentry *dentry, struct cftype > > **cftp) > > { > > - if (file_inode(file)->i_fop != _file_operations) > > - return ERR_PTR(-EINVAL); > > - return __d_cft(file->f_dentry); > >

Re: [PATCH RFC V11 15/18] kvm : Paravirtual ticketlocks support for linux guests running on KVM hypervisor

2013-08-05 Thread Konrad Rzeszutek Wilk
On Mon, Aug 05, 2013 at 11:46:03AM +0200, Ingo Molnar wrote: > > * Gleb Natapov wrote: > > > On Fri, Aug 02, 2013 at 11:25:39AM +0200, Ingo Molnar wrote: > > > > Ingo, > > > > > > > > Do you have any concerns reg this series? please let me know if this > > > > looks good now to you. > > > >

Re: [PATCH] WIP: HACK: LPAE, BOOTMEM and NO_BOOTMEM

2013-08-05 Thread Tejun Heo
Hello, On Mon, Aug 05, 2013 at 11:29:46AM -0400, Santosh Shilimkar wrote: > So looks like I am bit confused here. The current memblock_alloc() > API just returns the physical address which not mapped memory. The memory returned by memblock is mapped if the memory area being allocated can be

[PATCH] ARM: DTS: SAMA5: Add PMU support

2013-08-05 Thread Alexandre Belloni
ARM Performance Monitor Units are available on the sama5d3, add the support in the dtsi. Tested with perf and oprofile on the sama5d31ek. Signed-off-by: Alexandre Belloni --- arch/arm/boot/dts/sama5d3.dtsi | 5 + 1 file changed, 5 insertions(+) diff --git a/arch/arm/boot/dts/sama5d3.dtsi

Re: [edk2] Corrupted EFI region

2013-08-05 Thread James Bottomley
On Mon, 2013-08-05 at 17:15 +0200, Laszlo Ersek wrote: > On 08/05/13 16:40, Borislav Petkov wrote: > > On Mon, Aug 05, 2013 at 04:27:44PM +0200, Laszlo Ersek wrote: > >> I wouldn't call the design of SetVirtualAddressMap() braindead. > > > > Ok, I've always wondered and you could probably shed

Re: [PATCH 6/7] Add EFI stub for ARM

2013-08-05 Thread Leif Lindholm
On Mon, Aug 05, 2013 at 03:11:49PM +0100, Dave Martin wrote: > > diff --git a/arch/arm/boot/compressed/head.S > > b/arch/arm/boot/compressed/head.S > > index 75189f1..4c70b9e 100644 > > --- a/arch/arm/boot/compressed/head.S > > +++ b/arch/arm/boot/compressed/head.S > > @@ -122,19 +122,106 @@ > >

Re: [PATCH] WIP: HACK: LPAE, BOOTMEM and NO_BOOTMEM

2013-08-05 Thread Santosh Shilimkar
On Monday 05 August 2013 11:01 AM, Tejun Heo wrote: > Hello, > > On Fri, Aug 02, 2013 at 05:06:02PM -0400, Santosh Shilimkar wrote: >> Looking at the situation, how about proceeding with patch updating >> the bootmem API signatures to use phys_addr_t which can unblock me >> to get my machine

Re: [PATCH cgroup/for-3.12] cgroup: make css_for_each_descendant() and friends include the origin css in the iteration

2013-08-05 Thread Vivek Goyal
On Sun, Aug 04, 2013 at 07:07:03PM -0400, Tejun Heo wrote: > Hello, > > I've been wanting to do this for some time and given all the recent > API updates now seems like a pretty good opportunity. Verified > freezer and blkcg. The conversions are mostly straight forward but > I'd much appreciate

Re: [PATCH] ARM: fix wrong address when loading PRM_FRAC_INCREMENTOR_DENUMERATOR_RELOAD

2013-08-05 Thread Chen Baozi
ping? On Aug 1, 2013, at 7:27 PM, Chen Baozi wrote: > The denominator should be load from INCREMENTOR_DENUMERATOR_RELOAD_OFFSET > rather than INCREMENTER_NUMERATOR_OFFSET. > > Signed-off-by: Chen Baozi > --- > arch/arm/mach-omap2/timer.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-)

[PATCH v3] macvlan: validate flags

2013-08-05 Thread Michael S. Tsirkin
commit df8ef8f3aaa6692970a436204c4429210addb23a macvlan: add FDB bridge ops and macvlan flags added a flags field to macvlan, which can be controlled from userspace. The idea is to make the interface future-proof so we can add flags and not new fields. However, flags value isn't validated, as

Re: [PATCH V2 4/4] x86: correctly detect hypervisor

2013-08-05 Thread H. Peter Anvin
On 08/05/2013 07:34 AM, Konrad Rzeszutek Wilk wrote: > > Could you provide me with a git branch so I can test it overnight please? > Pull tip:x86/paravirt. -hpa -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to

Re: [PATCH v2 RESEND 13/18] x86, numa, mem_hotplug: Skip all the regions the kernel resides in.

2013-08-05 Thread Zhang Yanfei
Hi tj, On 08/05/2013 10:52 PM, Tejun Heo wrote: > On Mon, Aug 05, 2013 at 02:22:47PM +0800, Tang Chen wrote: >> I have resent the v2 patch-set. Would you please give some more >> comments about the memblock and x86 booting code modification ? > > Patch 13 still seems corrupt. Is it a problem on

Lets work together broda

2013-08-05 Thread Lets work together broda
hello bro..my name is steve and we can work together. I dey get company account for hong kong, china, india, thailand, and malaysia both company and personal account. make we share money na online here good contacts dey come from. you can contact me through bgee3...@gmail.com Steve from China

Re: [edk2] Corrupted EFI region

2013-08-05 Thread Laszlo Ersek
On 08/05/13 16:40, Borislav Petkov wrote: > On Mon, Aug 05, 2013 at 04:27:44PM +0200, Laszlo Ersek wrote: >> I wouldn't call the design of SetVirtualAddressMap() braindead. > > Ok, I've always wondered and you could probably shed some light on the > matter: why is SetVirtualAddressMap() a

Re: [PATCHv2] Add per-process flag to control thp

2013-08-05 Thread Alex Thorlton
> It would be good to describe which jobs and why thp cannot be fixed for > these. I'm going to see if I can get the benchmarks that are causing trouble up and running on my own to get some more information (problem was originally identified by our benchmarking team). Thanks for the input! -

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

2013-08-05 Thread Jeff Layton
On Mon, 5 Aug 2013 11:04:27 -0400 Jeff Layton wrote: > On Mon, 05 Aug 2013 15:48:01 +0100 > Nix wrote: > > > On 5 Aug 2013, Jeff Layton stated: > > > > > On Sun, 04 Aug 2013 16:40:58 +0100 > > > Nix wrote: > > > > > >> I just got this panic on 3.10.4, in the middle of a large parallel > > >>

Re: [PATCHv2] Add per-process flag to control thp

2013-08-05 Thread Rik van Riel
On 08/03/2013 01:01 PM, Oleg Nesterov wrote: +#ifdef CONFIG_TRANSPARENT_HUGEPAGE + p->thp_disabled = current->thp_disabled; +#endif Unneeded. It will be copied by dup_task_struct() automagically. But I simply can't understand why this flag is per-thread. It should be mm flag, no? It

Re: [PATCH] Add per-process flag to control thp

2013-08-05 Thread Alex Thorlton
> Please try it with the vclear patches. Thanks, Andi. I'll give that a shot and see if it makes any difference. - Alex -- 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

Re: [PATCHv2] Add per-process flag to control thp

2013-08-05 Thread Alex Thorlton
think the changelog should explain why madvise() is bad. No problem. I wanted to keep it simple for the original submission, but that's probably something that should be included. > But I simply can't understand why this flag is per-thread. It should > be > mm flag, no? This is something that

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

2013-08-05 Thread Jeff Layton
On Mon, 05 Aug 2013 15:48:01 +0100 Nix wrote: > On 5 Aug 2013, Jeff Layton stated: > > > On Sun, 04 Aug 2013 16:40:58 +0100 > > Nix wrote: > > > >> I just got this panic on 3.10.4, in the middle of a large parallel > >> compilation (of Chromium, as it happens) over NFSv3: > >> > >>

Re: [PATCH] WIP: HACK: LPAE, BOOTMEM and NO_BOOTMEM

2013-08-05 Thread Tejun Heo
Hello, On Fri, Aug 02, 2013 at 05:06:02PM -0400, Santosh Shilimkar wrote: > Looking at the situation, how about proceeding with patch updating > the bootmem API signatures to use phys_addr_t which can unblock me > to get my machine working. I'm not sure about that. No matter how you play it,

Re: [PATCH 8/9] perf kvm: debug for missing vmexit/vmentry event

2013-08-05 Thread David Ahern
On 8/5/13 2:53 AM, Xiao Guangrong wrote: On 08/03/2013 04:05 AM, David Ahern wrote: Expected to have missing events for each vcpu when perf is started. After that should not have missing events. Sorry, i can not understand what's this info used for. For debugging. Things that should not

Re: [PATCH] fs/nfs/inode.c: adjust code alignment

2013-08-05 Thread Myklebust, Trond
On Mon, 2013-08-05 at 16:47 +0200, Julia Lawall wrote: > From: Julia Lawall > > Signed-off-by: Julia Lawall > > --- > > This patch adjusts the code so that the alignment matches the current > semantics. I have no idea if it is the intended semantics, though. Should > the call to

Re: [PATCH 9/9] perf kvm stat report: Add option to analyze specific VM

2013-08-05 Thread David Ahern
On 8/5/13 2:57 AM, Xiao Guangrong wrote: On 08/03/2013 04:05 AM, David Ahern wrote: Add an option to analyze a specific VM within a data file. This allows the collection of kvm events for all VMs and then analyze data for each VM (or set of VMs) individually. Interesting. But how can we know

Re: percpu: kernel BUG at mm/percpu.c:579!

2013-08-05 Thread Tejun Heo
- > [ 274.821320] kernel BUG at mm/percpu.c:579! Looks like double free. > [ 274.821848] invalid opcode: [#1] PREEMPT SMP DEBUG_PAGEALLOC > [ 274.822467] Modules linked in: > [ 274.823240] CPU: 13 PID: 58 Comm: rcuos/13 Tainted: G W > 3.11.0-rc4-next-20130805-sasha-

Re: [PATCH RFC V11 15/18] kvm : Paravirtual ticketlocks support for linux guests running on KVM hypervisor

2013-08-05 Thread Paolo Bonzini
> Only thing I am thinking is about KVM_FEATURE_PV_UNHALT, and > KVM_HC_KICK_CPU definition in the below hunk, that is needed by guest > as well. may be this header file change can be a separate patch so that > duplicate can be handled easily during merge? Sure, good idea. Paolo > I do testing

Re: [PATCH v2 RESEND 13/18] x86, numa, mem_hotplug: Skip all the regions the kernel resides in.

2013-08-05 Thread Tejun Heo
On Mon, Aug 05, 2013 at 02:22:47PM +0800, Tang Chen wrote: > I have resent the v2 patch-set. Would you please give some more > comments about the memblock and x86 booting code modification ? Patch 13 still seems corrupt. Is it a problem on my side maybe? Nope, gmane raw message is corrupt too.

Re: [GIT PULL] tracing: final fixes for events and some

2013-08-05 Thread Steven Rostedt
On Mon, 2013-08-05 at 16:32 +0200, Ingo Molnar wrote: > * Steven Rostedt wrote: > > > Linus, > > > > Oleg Nesterov has been working hard in closing all the holes that can > > lead to race conditions between deleting an event and accessing an event > > debugfs file. This included a fix to the

Re: [PATCH] regmap: cache: Make sure to sync the last register in a block

2013-08-05 Thread Mark Brown
On Mon, Aug 05, 2013 at 11:21:29AM +0200, Lars-Peter Clausen wrote: > regcache_sync_block_raw_flush() expects the address of the register after last > register that needs to be synced as its parameter. But the last call to > regcache_sync_block_raw_flush() in regcache_sync_block_raw() passes the

Re: [PATCH 2/2] HID: hid-logitech-dj, querying_devices was never set

2013-08-05 Thread Benjamin Tissoires
On Mon, Aug 5, 2013 at 3:22 PM, Jiri Kosina wrote: > On Fri, 2 Aug 2013, Benjamin Tissoires wrote: > >> > Could you please elaborate? (and put an elaborate description to revert >> > commit log perhaps?) >> >> Sure, so here is the revert commit log: >> >> -- >> >> Commit "HID: hid-logitech-dj,

Re: [PATCH 7/9] perf kvm: option to print events that exceed a threshold

2013-08-05 Thread David Ahern
On 8/5/13 2:39 AM, Xiao Guangrong wrote: On 08/03/2013 04:05 AM, David Ahern wrote: This is useful to spot high latency blips. Yes, it is a good idea. Signed-off-by: David Ahern Cc: Arnaldo Carvalho de Melo Cc: Ingo Molnar Cc: Frederic Weisbecker Cc: Peter Zijlstra Cc: Jiri Olsa Cc:

percpu: kernel BUG at mm/percpu.c:579!

2013-08-05 Thread Sasha Levin
[ 274.822467] Modules linked in: [ 274.823240] CPU: 13 PID: 58 Comm: rcuos/13 Tainted: GW 3.11.0-rc4-next-20130805-sasha-2-gf6cc217 #3975 [ 274.824464] task: 880220cb3000 ti: 880220cba000 task.ti: 880220cba000 [ 274.825442] RIP: 0010:[] [] pcpu_free_area+0xd8/0x1e0

[PATCH] drivers/scsi/lpfc/lpfc_init.c: adjust code alignment

2013-08-05 Thread Julia Lawall
From: Julia Lawall Adjust code alignment so that each statement is lined up with its neighbor. In the second case, it could be desirable to put the call to lpfc_destroy_vport_work_array under the if. The call, is, however, safe in case vports is NULL, so the patch just adjusts the indentation

[PATCH] net/vmw_vsock/af_vsock.c: drop unneeded semicolon

2013-08-05 Thread Julia Lawall
From: Julia Lawall Drop the semicolon at the end of the list_for_each_entry loop header. Signed-off-by: Julia Lawall --- Not tested, but I can't imagine how the current code could work, since vsk should end up pointing to a dummy value. net/vmw_vsock/af_vsock.c |2 +- 1 file changed, 1

[PATCH] fs/nfs/inode.c: adjust code alignment

2013-08-05 Thread Julia Lawall
From: Julia Lawall Signed-off-by: Julia Lawall --- This patch adjusts the code so that the alignment matches the current semantics. I have no idea if it is the intended semantics, though. Should the call to nfs_setsecurity also be under the else? fs/nfs/inode.c |2 +- 1 file changed,

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

2013-08-05 Thread Nix
On 5 Aug 2013, Jeff Layton stated: > On Sun, 04 Aug 2013 16:40:58 +0100 > Nix wrote: > >> I just got this panic on 3.10.4, in the middle of a large parallel >> compilation (of Chromium, as it happens) over NFSv3: >> >> [16364.527516] BUG: unable to handle kernel NULL pointer dereference at >>

[PATCH] adjust code alignment

2013-08-05 Thread Julia Lawall
These patches result from the following semantic patch (http://coccinelle.lip6.fr/), which checks for successive statements that are not aligned. @bad@ statement S; expression e; position p1,p; @@ S@p1 e@p; @script:ocaml@ p1 << bad.p1; p << bad.p; @@ if not ((List.hd p1).line = (List.hd

Re: [PATCHv2] Add per-process flag to control thp

2013-08-05 Thread Peter Zijlstra
On Sat, Aug 03, 2013 at 07:01:02PM +0200, Oleg Nesterov wrote: > On 08/02, Alex Thorlton wrote: > > > > This patch implements functionality to allow processes to disable the use of > > transparent hugepages through the prctl syscall. > > > > We've determined that some jobs perform significantly

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