Hello, Lai.
On Sun, Sep 09, 2012 at 12:09:58PM +0800, Lai Jiangshan wrote:
> > mutex_lock(&pool->manager_mutex);
> > if (worker_maybe_bind_and_lock(worker))
> > worker_clr_flags(worker, WORKER_UNBOUND);
> > ret = true;
> >
>
>
> This code is correct. worke
Yes that fix was already sent by me.
On Sep 8, 2012, at 22:42, Hugh Dickins wrote:
> Latest mmotm rarely boots if SLUB is enabled: earlyprintk=vga shows
> it crashing with various backtraces. The memset has now been removed
> from kmem_cache_open(), so kmem_cache_init() needs to zero its page
What's new?
I liked you and that u know how to make women happy. you know, I was studying
psychology in college and that's why I know tons about physiognomy! So, I saw
ur photo shots, made an analysis of them and decided that u are really cool.
That's why I'm writing you this mail.
Oh, some of
On Sun, Sep 9, 2012 at 3:02 AM, Tejun Heo wrote:
> Hello, Lai.
>
> On Sun, Sep 09, 2012 at 02:34:02AM +0800, Lai Jiangshan wrote:
>> in 3.6 busy_worker_rebind() handle WORKER_REBIND bit,
>> not WORKER_UNBOUND bit.
>>
>> busy_worker_rebind() takes struct work_struct *work argument, we have to
>> ad
On 09/07/2012 07:37 AM, Konrad Rzeszutek Wilk wrote:
significant design challenges exist, many of which are already resolved in
the new codebase ("zcache2"). These design issues include:
.. snip..
Before other key mm maintainers read and comment on zcache, I think
it would be most wise to move
Latest mmotm rarely boots if SLUB is enabled: earlyprintk=vga shows
it crashing with various backtraces. The memset has now been removed
from kmem_cache_open(), so kmem_cache_init() needs to zero its page.
This gets SLUB booting reliably again.
Signed-off-by: Hugh Dickins
---
mm/slub.c |2
On Sat, Sep 08, 2012 at 03:53:25AM -0700, Yunfan Zhang wrote:
> The primary handler will NOT be called if the interrupt nests into
> another interrupt thread. Remove it to avoid confusing.
Applied, thanks.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a
On Sun, Sep 09, 2012 at 07:25:15AM +0800, Ming Lei wrote:
> Hi,
>
> This patchset implements asynchronous driver probe for driver_register,
> and try to address the below problems about driver init during
> kernel boot:
> - help to solve some dependency problem during kernel boot
> (su
Looks OK to me, thanks.
On Sep 7, 2012, at 11:59 PM, "Dan Carpenter" wrote:
> We changed this recently so we can just use kzalloc() here instead of
> kcalloc(1, ...). Kernel style prefers sizeof(*t) over sizeof *t. The
> kfree(t) is a no-op now as well so that can be removed.
>
> Signed-off-b
Removed 3 checkpatch.sh warnings and 1 error in
drivers/usb/gadget/serial.c.
-sizeof brackets x2
-remove initialise to 0
-pr_warning to pr_warn
Signed-off-by: Ben Minerds
---
drivers/usb/gadget/serial.c |9 -
1 files changed, 4 insertions(+), 5 deletions(-)
diff --git a/drivers/usb/
Hey, we're back to the *regular* release schedule, with weekly -rc's and all.
So 3.6-rc5 is out there, and everything is looking fairly calm. Too
calm, in fact, I'm waiting for the other shoe to drop, when Greg
finally crawls his way out from under his mailbox after the kernel
summit and kayaking.
Please pull from pekkas tree on github or use next (check if the back and forth
with patchset in and out has been resolved first).
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.
The check is not necessary since driver_attach should return 0
always.
Signed-off-by: Ming Lei
---
include/linux/device.h |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/include/linux/device.h b/include/linux/device.h
index 7222c0b..dbb5efd 100644
--- a/include/linux/devic
platform_driver_probe supposes that the driver's probe is always called
inside platform_driver_probe, so apply driver_register_sync for it.
Signed-off-by: Ming Lei
---
drivers/base/platform.c | 11 +++
include/linux/platform_device.h | 12 +++-
2 files changed, 18 ins
This patch introduces driver_register_sync helper to make
driver's probe called synchronously inside driver_register
path.
platform_driver_probe should use driver_register_sync.
Signed-off-by: Ming Lei
---
drivers/base/driver.c |5 +++--
include/linux/device.h | 13 -
2 files
It should be correct to probe driver at the last minute in
driver_register since it will be so in !auto_probe case and
driver_register on no matched devices case.
Signed-off-by: Ming Lei
---
drivers/base/bus.c|5 -
drivers/base/driver.c |4
2 files changed, 4 insertions(+),
Signed-off-by: Ming Lei
---
drivers/base/driver.c | 16 ++--
1 file changed, 14 insertions(+), 2 deletions(-)
diff --git a/drivers/base/driver.c b/drivers/base/driver.c
index 9365076..67fb63c 100644
--- a/drivers/base/driver.c
+++ b/drivers/base/driver.c
@@ -15,6 +15,7 @@
#includ
Hi,
This patchset implements asynchronous driver probe for driver_register,
and try to address the below problems about driver init during
kernel boot:
- help to solve some dependency problem during kernel boot
(such as, request_firmware is called inside probe when driver
i
Christoph,
On Sat, Sep 8, 2012 at 7:30 PM, Christoph Lameter wrote:
> On Sat, 8 Sep 2012, Ezequiel Garcia wrote:
>
>> This is the second spin of my patchset to clean SLxB and improve kmem
>> trace events accuracy.
>
> Please redo the patches on top of the patchsets that create
> mm/slab_common.c.
On Sat, Sep 8, 2012 at 12:13 PM, Pekka Enberg wrote:
> Hi Irina,
>
Hi Pekka,
> I commented on some of the patches but overall, I think the series makes
> sense:
>
> Acked-by: Pekka Enberg
Thanks for reviewing it!
Irina
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel
On Sat, Sep 08, 2012 at 02:36:00PM +0930, Rusty Russell wrote:
> "Michael S. Tsirkin" writes:
> > On Fri, Sep 07, 2012 at 04:09:50PM +0930, Rusty Russell wrote:
> >> > So it looks like a bug: we should teach driver to tell host first on
> >> > leak?
> >> > Yan, Vadim, can you comment please?
> >>
On Sat, 8 Sep 2012, Ezequiel Garcia wrote:
> This is the second spin of my patchset to clean SLxB and improve kmem
> trace events accuracy.
Please redo the patches on top of the patchsets that create
mm/slab_common.c. You will be able to extract a lot more common code and
help the goal of having
> Irina: Would you verify perf still compiles cleanly for Android. Thanks.
>
With these 3 patches perf still compiles cleanly for Android.
Thanks for making the changes.
Irina
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.ker
The IDE layer does not handle shared interrupts properly at all,
and it never will be rearchitected to do so.
This is a well known issue.
Please disable IDE in your configuration and use the PATA drivers
instead, which are able to handle shared interrupts properly.
Thanks.
--
To unsubscribe fro
I upgraded my workstation to 3.4.10 from something like 3.2 or so, thus
I don't know when this started causing issues. But after I did, my 2TB
WD external USB hard drive stopped showing up.
I looked at my dmesg and saw this:
[1.372997]
/home/rostedt/work/git/nobackup/linux-build.git/drivers/
On Fri, Sep 07, 2012 at 04:44:40PM +0200, Paolo Bonzini wrote:
> Il 07/09/2012 16:25, Michael S. Tsirkin ha scritto:
> >> > Silent deflate is deflating just by using the page, without using the
> >> > deflateq at all. So it can be done even from GFP_ATOMIC context.
> > BTW I don't see a need to av
>> +#ifndef __WORDSIZE
>> +#if defined(__x86_64__)
>> +# define __WORDSIZE 64
>> +#endif
>> +#if defined(__i386__) || defined(__arm__)
>> +# define __WORDSIZE 32
>> +#endif
>> +#endif
>
> Why not use "sizeof(unsigned long) * 8" ?
I tried to do it this, but the compilation crashes because this valu
On Thursday, August 30, 2012, MyungJoo Ham wrote:
> Even if the performance of a device is controlled properly with devfreq,
> sometimes, we still need to get PM-QoS inputs in order to meet the
> required performance.
>
> In our testbed of Exynos4412, which has on-chip various DMA devices, the
> m
The ACPI BGRT driver accesses the BIOS logo image when it initializes.
However, ACPI 5.0 (which introduces the BGRT) recommends putting the
logo image in EFI boot services memory, so that the OS can reclaim that
memory. Production systems follow this recommendation, breaking the
ACPI BGRT driver.
The EFI initialization creates virtual mappings for EFI boot services
memory, so if a driver wants to access EFI boot services memory, it
cannot call ioremap itself; doing so will trip the WARN about mapping
RAM twice. Thus, a driver accessing EFI boot services memory must do so
via the existing m
Some new ACPI 5.0 tables reference resources stored in boot services
memory, so keep that memory around until we have ACPI and can extract
data from it.
Signed-off-by: Josh Triplett
---
arch/x86/platform/efi/efi.c | 31 ++-
include/linux/efi.h |2 ++
ini
This eliminates an ifdef in init/main.c.
Signed-off-by: Josh Triplett
---
include/linux/efi.h |4
init/main.c |2 --
2 files changed, 4 insertions(+), 2 deletions(-)
diff --git a/include/linux/efi.h b/include/linux/efi.h
index ec45ccd..52fbedf 100644
--- a/include/linux/efi
The ACPI BGRT lets the OS access the BIOS logo image and its position on the
screen at boot time, allowing it to maintain that image on the screen until
ready to display something else, making boot more seamless. This series fixes
support for accessing the boot logo image via the BGRT when the BIO
On 09/08/2012 04:50 PM, Alan Stern wrote:
On Sat, 8 Sep 2012, Rafael J. Wysocki wrote:
On Saturday, September 08, 2012, Bjorn Helgaas wrote:
+cc Rafael, Huang, linux-pm
On Fri, Sep 7, 2012 at 3:44 PM, Larry Finger wrote:
Hi,
On occasion when I unload and reload driver rtl8192ce, I get the
Hi John,
A few more fixes to 3.6, here we have four important fix to the MGMT interface,
two from Johan Hedberg and Andrzej Kaczmarek. Andrei fixed a free on
uninitialized memory and I added support for Broadcom/Foxconn devices.
Please pull, or let me know of any problems. Thanks.
Gustav
On Sat, 8 Sep 2012, Rafael J. Wysocki wrote:
> On Saturday, September 08, 2012, Bjorn Helgaas wrote:
> > +cc Rafael, Huang, linux-pm
> >
> > On Fri, Sep 7, 2012 at 3:44 PM, Larry Finger
> > wrote:
> > > Hi,
> > >
> > > On occasion when I unload and reload driver rtl8192ce, I get the message
> >
On Sat, Sep 08, 2012 at 05:47:50PM -0300, Ezequiel Garcia wrote:
> As its name suggest this option prevents gcc from auto inlining
> small functions. This is very important if one wants to obtain
> traces with accurate call sites.
>
> Without this option, gcc will collapse some small functions,
>
Hi Peter,
* Peter Senna Tschudin [2012-09-07 17:24:48 +0200]:
> From: Peter Senna Tschudin
>
> removes unnecessary semicolon
>
> Found by Coccinelle: http://coccinelle.lip6.fr/
>
> Signed-off-by: Peter Senna Tschudin
>
> ---
> drivers/bluetooth/btuart_cs.c |2 +-
> 1 file changed, 1 i
Hi Peter,
* Peter Senna Tschudin [2012-09-07 17:24:47 +0200]:
> From: Peter Senna Tschudin
>
> removes unnecessary semicolon
>
> Found by Coccinelle: http://coccinelle.lip6.fr/
>
> Signed-off-by: Peter Senna Tschudin
>
> ---
> drivers/bluetooth/hci_vhci.c |2 +-
> 1 file changed, 1 in
Hi Peter,
* Peter Senna Tschudin [2012-09-07 17:24:42 +0200]:
> From: Peter Senna Tschudin
>
> removes unnecessary semicolon
>
> Found by Coccinelle: http://coccinelle.lip6.fr/
>
> Signed-off-by: Peter Senna Tschudin
>
> ---
> drivers/bluetooth/hci_ll.c |2 +-
> 1 file changed, 1 inse
Hi Peter,
* Peter Senna Tschudin [2012-09-07 17:24:39 +0200]:
> From: Peter Senna Tschudin
>
> removes unnecessary semicolon
>
> Found by Coccinelle: http://coccinelle.lip6.fr/
>
> Signed-off-by: Peter Senna Tschudin
Patch has been applied to the bluetooth-next tree. Thanks.
Gusta
Hi Peter,
* Peter Senna Tschudin [2012-09-07 17:24:40 +0200]:
> From: Peter Senna Tschudin
>
> removes unnecessary semicolon
>
> Found by Coccinelle: http://coccinelle.lip6.fr/
>
> Signed-off-by: Peter Senna Tschudin
>
> ---
> drivers/bluetooth/bluecard_cs.c |2 +-
> 1 file changed, 1
On Sat, Sep 08, 2012 at 11:40:52AM -0700, Yinghai Lu wrote:
> On Sat, Sep 8, 2012 at 8:34 AM, Feng Tang wrote:
> >> This makes lspci work again on my side. The caveat is, kzalloc() will
> >> zero out all data while the new local variable leaves some data
> >> uninitialized.
> >
> > Yes, thanks for
This patch does not fix anything, and its only goal is to enable us
to obtain some common code between SLAB and SLUB.
Neither behavior nor produced code is affected.
Cc: Christoph Lameter
Signed-off-by: Ezequiel Garcia
---
mm/slub.c | 24 +++-
1 files changed, 15 insertion
As its name suggest this option prevents gcc from auto inlining
small functions. This is very important if one wants to obtain
traces with accurate call sites.
Without this option, gcc will collapse some small functions,
even when not marked as 'inline' thus making impossible to
correlate the trac
Currently slob falls back to regular kmalloc for this case.
With this patch kmalloc_track_caller() is correctly implemented,
thus tracing the specified caller.
This is important to trace accurately allocations performed by
krealloc, kstrdup, kmemdup, etc.
Cc: Pekka Enberg
Signed-off-by: Ezequiel
This long (seemingly unnecessary) patch does not fix anything and
its only goal is to produce common code between SLAB and SLUB.
Cc: Pekka Enberg
Cc: Christoph Lameter
Signed-off-by: Ezequiel Garcia
---
include/linux/slab_def.h |7 +++
mm/slab.c| 10 +-
2 file
Several interfaces are now exactly the same in both SLAB and SLUB,
and it's very easy to factor them out.
To make this work slab_alloc() and slab_alloc_node() are made non-static.
This has the benefit of putting most of the tracing code in mm/slab_common.c,
making it harder to produce inconsistent
This patch does not fix anything and its only goal is to
produce common code between SLAB and SLUB.
Cc: Pekka Enberg
Cc: Christoph Lameter
Signed-off-by: Ezequiel Garcia
---
mm/slab.c | 14 +++---
1 files changed, 7 insertions(+), 7 deletions(-)
diff --git a/mm/slab.c b/mm/slab.c
in
This allows to use _RET_IP_ instead of builtin_address(0), thus
achiveing implementation consistency in all three allocators.
Though maybe a nitpick, the real goal behind this patch is
to be able to obtain common code between SLAB and SLUB.
Cc: Pekka Enberg
Signed-off-by: Ezequiel Garcia
---
mm
This function is seldom used, and can be simply replaced with cachep->size.
Cc: Pekka Enberg
Signed-off-by: Ezequiel Garcia
---
include/linux/slab_def.h |5 -
mm/slab.c| 12 ++--
2 files changed, 2 insertions(+), 15 deletions(-)
diff --git a/include/linux/slab
Previously the strndup_user allocation was being done through memdup_user,
and the caller was wrongly traced as being strndup_user
(the correct trace must report the caller of strndup_user).
This is a common problem: in order to get accurate callsite tracing,
a utils function can't allocate throug
Cc: Pekka Enberg
Signed-off-by: Ezequiel Garcia
---
include/linux/slob_def.h |6 --
mm/slob.c|6 +++---
2 files changed, 7 insertions(+), 5 deletions(-)
diff --git a/include/linux/slob_def.h b/include/linux/slob_def.h
index 0ec00b3..f28e14a 100644
--- a/include/linux
Hi everyone,
This is the second spin of my patchset to clean SLxB and improve kmem
trace events accuracy.
For this v2, the most relevant stuff is:
I've dropped two patches that were not very well received:
Namely this two are now gone:
mm, slob: Use only 'ret' variable for both slob object and
On Friday, August 10, 2012, 함명주 wrote:
> > + Myungjoo Ham,
> >
> > It used at devfreq. Mr. Ham can you explain it in detail?
> >
> > Thank you,
> > Kyungmin Park
> > ,
> > On 8/9/12, Rafael J. Wysocki wrote:
> > > On Wednesday, August 08, 2012, Jonghwa Lee wrote:
> > >> Bus throughput metric is
From: Irina Tirdea
When NDEBUG is defined, the assert macro will be expanded to nothing.
Some assert calls used in perf are also including some functionality
(e.g. system calls), not only validity checks. Therefore, if NDEBUG is
defined, this functionality will be removed along with the assert.
From: Arnaldo Carvalho de Melo
Hi Ingo,
Please consider pulling,
Thanks,
- Arnaldo
The following changes since commit ef34eb4da3eb62a1511592adf7c76d74faca0b14:
Merge tag 'perf-core-for-mingo' of
git://git.kernel.org/pub/scm/linux/kernel/git/acme/linux into perf/core
(2012-09-08 1
From: Namhyung Kim
Current hist print functions are messy because it has to consider many
of command line options and the code doing that is scattered around to
places. So when someone wants to add an option to manipulate the hist
output it'd very easy to miss to update all of them in sync. And t
From: Namhyung Kim
Now we can support color using pango markup with this change.
Signed-off-by: Namhyung Kim
Acked-by: Pekka Enberg
Cc: Ingo Molnar
Cc: Paul Mackerras
Cc: Peter Zijlstra
Link:
http://lkml.kernel.org/r/1346640790-17197-6-git-send-email-namhy...@kernel.org
Signed-off-by: Arna
From: Namhyung Kim
Signed-off-by: Namhyung Kim
Cc: Ingo Molnar
Cc: Paul Mackerras
Cc: Peter Zijlstra
Link:
http://lkml.kernel.org/r/1346640790-17197-4-git-send-email-namhy...@kernel.org
Signed-off-by: Arnaldo Carvalho de Melo
---
tools/perf/ui/hist.c | 27 +++
t
From: David Ahern
The basename function may modify the string passed to it, so the string
should not be marked const.
Signed-off-by: David Ahern
Acked-by: Pekka Enberg
Cc: Irina Tirdea
Cc: Pekka Enberg
Link:
http://lkml.kernel.org/r/1347116812-93646-2-git-send-email-dsah...@gmail.com
Signed
From: Namhyung Kim
Override hpp->color functions for TUI. Because line coloring is done
outside of the function, it just sets the percent value and pass it.
Signed-off-by: Namhyung Kim
Cc: Ingo Molnar
Cc: Paul Mackerras
Cc: Peter Zijlstra
Link:
http://lkml.kernel.org/r/1346640790-17197-5-gi
From: Namhyung Kim
When a field separator is given, the output format doesn't need to be
fancy like aligning to column length, coloring the percent value and so
on. And since there's a slight difference to normal format, fix it not
to break backward compatibility.
Signed-off-by: Namhyung Kim
C
From: David Ahern
The basename function may modify the string passed to it, so the string
should not be marked const.
Signed-off-by: David Ahern
Acked-by: Pekka Enberg
Cc: Irina Tirdea
Cc: Pekka Enberg
Cc: Srikar Dronamraju
Link:
http://lkml.kernel.org/r/1347116812-93646-3-git-send-email-d
From: David Ahern
Now that the 2 offenders are fixed, the BIONIC conditional around
libgen.h can be removed.
Signed-off-by: David Ahern
Acked-by: Pekka Enberg
Cc: Irina Tirdea
Cc: Pekka Enberg
Link:
http://lkml.kernel.org/r/1347116812-93646-4-git-send-email-dsah...@gmail.com
Signed-off-by:
On Saturday, September 08, 2012, Bjorn Helgaas wrote:
> +cc Rafael, Huang, linux-pm
>
> On Fri, Sep 7, 2012 at 3:44 PM, Larry Finger
> wrote:
> > Hi,
> >
> > On occasion when I unload and reload driver rtl8192ce, I get the message
> >
> > "Refused to change power state, currently in D3"
> >
> >
This patch splits trace event initialization in two stages:
* ftrace enable
* sysfs event entry creation
This allows to capture trace events from an earlier point
by using 'trace_event' kernel parameter and is important
to trace boot-up allocations.
Note that, in order to enable events at core_
On Fri, Sep 7, 2012 at 4:54 PM, Suresh Siddha wrote:
>
> Essentially the user is trying to mmap at a very large offset (from the
> oops it appears "vma->vm_pgoff << PAGE_SHIFT + start" ends up to
> "0xf000").
Ok, Sasha confirmed that.
> So it appears that the condition "(vma->vm_end
On Wed, Aug 29, 2012 at 6:14 PM, Or Gerlitz wrote:
> Hi Roland,
>
> This short series include two more fixes from Shlomo to the newly
> introduced IPoIB neighbour table, and a fix for Yishai that completes
> the support for memory registration of up to 8TB.
Roland,
AFAIK 3.6-rc5 should be out b
From: Andi Kleen
For large kernel configurations (like a distribution kernel)
targz-pkg takes a quite long time to just do the compression.
I clocked it at 15+mins for a SUSE kernel like config on a fast
system. And tarxz and bzip2 are even slower.
The main reason is that the script that is doi
From: anish kumar
This commit- 314be14bb renamed the _st_ functions to loose the bit
that was meant for staging version but forgot to change
the documentation which still have _st_ sprinkled in some of the
places.
Signed-off-by: anish kumar
---
drivers/staging/iio/Documentation/inkernel.txt |
Hello, Kent.
On Fri, Sep 07, 2012 at 03:59:11PM -0700, Kent Overstreet wrote:
> Random assortment of refactoring and trivial cleanups.
>
> Immutable bio vecs and efficient bio splitting require auditing and removing
> pretty much all bi_idx uses, among other things.
>
> The reason is that with i
(Restoring cc list from the previous discussion. Please retain the cc
list of the people who discussed in the previous postings.)
On Fri, Sep 07, 2012 at 03:12:53PM -0700, Kent Overstreet wrote:
> But this is tricky and not a generic solution. This patch solves it for
> all users by inverting the
Hi,
On Fri, Sep 07, 2012 at 01:53:11PM -0700, Kevin Hilman wrote:
> Felipe Balbi writes:
>
> > Hi,
> >
> > On Thu, Sep 06, 2012 at 03:44:13PM -0700, Kevin Hilman wrote:
> >> Felipe Balbi writes:
> >>
> >> > Hi guys,
> >> >
> >> > here's v4 of the omap uart patchset. No changes other than a reb
Hello, Lai.
On Sun, Sep 09, 2012 at 02:34:02AM +0800, Lai Jiangshan wrote:
> in 3.6 busy_worker_rebind() handle WORKER_REBIND bit,
> not WORKER_UNBOUND bit.
>
> busy_worker_rebind() takes struct work_struct *work argument, we have to
> add new patch to add a helper and restruct it at first.
What'
A new stable release 1.2.0 of the Sysprof system-level sampling profiler
for Linux is now available. News since 1.0:
- User interface improvements
- Improved tracking of time spent in kernel
- Command line version
- Performance improvements
- Now based on perf_event_open system call
Where to get
Peter,
On Sat, Sep 8, 2012 at 11:01 AM, Peter Senna Tschudin
wrote:
> From: Peter Senna Tschudin
>
> Convert a nonnegative error return code to a negative one, as returned
> elsewhere in the function.
>
> A simplified version of the semantic match that finds this problem is as
> follows: (http:/
Just a friendly reminder that I'm still seeing this NULL pointer
dereference on boot with 3.6-rc4.
On Sat, Aug 18, 2012 at 11:11:21PM -0500, Shaun Ruffell wrote:
> With kernel version 3.6-rc2 on a Dell Poweredge 2600 I experienced a NULL
> pointer dereference that did not occur with on 3.5. I beli
On Sat, Sep 8, 2012 at 8:34 AM, Feng Tang wrote:
>> This makes lspci work again on my side. The caveat is, kzalloc() will
>> zero out all data while the new local variable leaves some data
>> uninitialized.
>
> Yes, thanks for the quick root cause and fix to the bug in my code.
Can you resubmit y
On Sun, Sep 9, 2012 at 2:11 AM, Tejun Heo wrote:
> On Sun, Sep 09, 2012 at 02:07:50AM +0800, Lai Jiangshan wrote:
>> when we release gcwq->lock and then grab it, we leave a hole that things
>> can be changed.
>>
>> I don't want to open a hole. if the hole has bug we have to fix it.
>> if the hole
Hi,
I get a lot of those and was wondering if it is a generic fscache issues
or in some way related to the RT patchset. Perhaps that the two do not
play well together?
Traces typically look like this:
BUG: scheduling while atomic: kworker/u:2/4151/0x0002
Modules linked in: snd_usb_aud
>
> Hmmm... so, I'm having some difficulty communicating with you. We
> need two separate patch series. One for for-3.6-fixes and the other
> for restructuring on top of for-3.7 after the fixes are merged into
> it.
>
> As you currently posted, the patches are based on for-3.7 and fixes
> and res
On Sun, Sep 09, 2012 at 02:07:50AM +0800, Lai Jiangshan wrote:
> when we release gcwq->lock and then grab it, we leave a hole that things
> can be changed.
>
> I don't want to open a hole. if the hole has bug we have to fix it.
> if the hole has no bug, we have to add lot of comments to explain it
On Sun, Sep 9, 2012 at 1:53 AM, Tejun Heo wrote:
> Hello,
>
> On Sun, Sep 09, 2012 at 01:50:41AM +0800, Lai Jiangshan wrote:
>> >> + if (worker_maybe_bind_and_lock(manager))
>> >> + worker_clr_flags(manager, WORKER_UNBOUND);
>> >> + }
>> >> +}
>> >
>> > We can r
Convert omap_wdt to new watchdog core. On OMAP boards, there are usually
multiple watchdogs. Since the new watchdog core supports multiple
watchdogs, all watchdog drivers used on OMAP should be converted.
The legacy watchdog device node is still created, so this should not
break existing users.
S
Adding a missing copyright header to parse-utils.c. Assuminng that the
license is LGPL like the rest of the trace-cmd library code.
Signed-off-by: Jon Stanley
Cc: Arnaldo Carvalho de Melo
Cc: Steven Rostedt
---
* Changes for v2 -
I seem to have forgotten to add the Signed-off-by abnd CC lines.
On Sun, Sep 09, 2012 at 01:56:47AM +0800, Lai Jiangshan wrote:
> > So, your worry was incorrect and the above is what we're gonna do, no?
>
> NO BUG found for PATCH4 even idle rebinding is synchronous.
Hmmm... so, I'm having some difficulty communicating with you. We
need two separate patch seri
On Sun, Sep 9, 2012 at 1:50 AM, Tejun Heo wrote:
> Hello, Lai.
>
> On Sun, Sep 09, 2012 at 01:46:59AM +0800, Lai Jiangshan wrote:
>> > * Instead of MANAGING, add pool->manager.
>> >
>> > * Fix the idle depletion bug by using pool->manager for exclusion and
>> > always grabbing pool->manager_mute
Hello,
On Sun, Sep 09, 2012 at 01:50:41AM +0800, Lai Jiangshan wrote:
> >> + if (worker_maybe_bind_and_lock(manager))
> >> + worker_clr_flags(manager, WORKER_UNBOUND);
> >> + }
> >> +}
> >
> > We can reuse busy_worker_rebind_fn(), right?
>
> busy_worker_rebind_
On Sat, Sep 08, 2012 at 10:50:07AM -0700, Tejun Heo wrote:
> Please create two patches, first introducing pool->manager_mutex and
^ pool->manager
--
tejun
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a me
On Sun, Sep 9, 2012 at 1:40 AM, Tejun Heo wrote:
> Hello, Lai.
>
> On Sun, Sep 09, 2012 at 01:12:53AM +0800, Lai Jiangshan wrote:
>> +/* does the manager need to be rebind after we just release gcwq->lock */
>> +static void maybe_rebind_manager(struct worker *manager)
>> +{
>> + struct global_
Hello, Lai.
On Sun, Sep 09, 2012 at 01:46:59AM +0800, Lai Jiangshan wrote:
> > * Instead of MANAGING, add pool->manager.
> >
> > * Fix the idle depletion bug by using pool->manager for exclusion and
> > always grabbing pool->manager_mutex. Hotplug can use pool->manager
> > to schedule rebind
On Sun, Sep 9, 2012 at 1:37 AM, Tejun Heo wrote:
> Hello, Lai.
>
> On Sun, Sep 09, 2012 at 01:27:37AM +0800, Lai Jiangshan wrote:
>> On Sun, Sep 9, 2012 at 1:12 AM, Lai Jiangshan wrote:
>> > The patch set is based on 3b07e9ca26866697616097044f25fbe53dbab693 of
>> > wq.git
>> >
>> > Patch 1,2 are
On Sun, Sep 09, 2012 at 01:40:19AM +0800, Lai Jiangshan wrote:
> On Sun, Sep 9, 2012 at 1:32 AM, Tejun Heo wrote:
> > On Sat, Sep 08, 2012 at 10:29:50AM -0700, Tejun Heo wrote:
> >> > hotplug code can't iterate manager. not rebind_work() nor UNBOUND for
> >> > manager.
> >>
> >> Ah, right. It i
Hello, Lai.
On Sun, Sep 09, 2012 at 01:12:53AM +0800, Lai Jiangshan wrote:
> +/* does the manager need to be rebind after we just release gcwq->lock */
> +static void maybe_rebind_manager(struct worker *manager)
> +{
> + struct global_cwq *gcwq = manager->pool->gcwq;
> + bool assoc = !(gcw
On Sun, Sep 9, 2012 at 1:32 AM, Tejun Heo wrote:
> On Sat, Sep 08, 2012 at 10:29:50AM -0700, Tejun Heo wrote:
>> > hotplug code can't iterate manager. not rebind_work() nor UNBOUND for
>> > manager.
>>
>> Ah, right. It isn't either on idle or busy list. Maybe have
>> pool->manager pointer?
>
>
Hello, Lai.
On Sun, Sep 09, 2012 at 01:27:37AM +0800, Lai Jiangshan wrote:
> On Sun, Sep 9, 2012 at 1:12 AM, Lai Jiangshan wrote:
> > The patch set is based on 3b07e9ca26866697616097044f25fbe53dbab693 of wq.git
> >
> > Patch 1,2 are accepted. Patch 1 goes to 3.6. tj has a replacement goes
> > to
On Sun, Sep 9, 2012 at 1:12 AM, Lai Jiangshan wrote:
> The patch set is based on 3b07e9ca26866697616097044f25fbe53dbab693 of wq.git
>
> Patch 1,2 are accepted. Patch 1 goes to 3.6. tj has a replacement goes
> to 3.6 instead of Patch 2. so Patch2 will go to 3.7. Patch2 will need
> to be rebased if
On Sun, Sep 9, 2012 at 1:27 AM, Lai Jiangshan wrote:
> On Sun, Sep 9, 2012 at 1:12 AM, Lai Jiangshan wrote:
>> The patch set is based on 3b07e9ca26866697616097044f25fbe53dbab693 of wq.git
>>
>> Patch 1,2 are accepted. Patch 1 goes to 3.6. tj has a replacement goes
>> to 3.6 instead of Patch 2. so
On Sat, Sep 08, 2012 at 10:29:50AM -0700, Tejun Heo wrote:
> > hotplug code can't iterate manager. not rebind_work() nor UNBOUND for
> > manager.
>
> Ah, right. It isn't either on idle or busy list. Maybe have
> pool->manager pointer?
Ooh, this is what you did with the new patchset, right?
-
1 - 100 of 211 matches
Mail list logo
