linux.git/drivers/base/core.c, branch v3.7-rc2

device and dynamic_debug: Use dev_vprintk_emit and dev_printk_emit

2012-09-17T13:10:05+00:00

Convert direct calls of vprintk_emit and printk_emit to the
dev_ equivalents.

Make create_syslog_header static.

Signed-off-by: Joe Perches 
Acked-by: David S. Miller 
Tested-by: Jim Cromie 
Acked-by: Jason Baron 
Signed-off-by: Greg Kroah-Hartman

dev: Add dev_vprintk_emit and dev_printk_emit

2012-09-17T13:10:05+00:00

Add utility functions to consolidate the use of
create_syslog_header and vprintk_emit.

This allows conversion of logging functions that
call create_syslog_header and then call vprintk_emit
or printk_emit to the dev_ equivalents.

Signed-off-by: Joe Perches 
Acked-by: David S. Miller 
Tested-by: Jim Cromie 
Acked-by: Jason Baron 
Signed-off-by: Greg Kroah-Hartman

dev_dbg/dynamic_debug: Update to use printk_emit, optimize stack

2012-09-17T13:08:30+00:00

commit c4e00daaa9
("driver-core: extend dev_printk() to pass structured data")
changed __dev_printk and broke dynamic-debug's ability to control the
dynamic prefix of dev_dbg(dev,..).

commit af7f2158fd
("drivers-core: make structured logging play nice with dynamic-debug")
made a minimal correction.

The current dynamic debug code uses up to 3 recursion levels via %pV.
This can consume quite a bit of stack.  Directly call printk_emit to
reduce the recursion depth.

These changes include:

dev_dbg:
o Create and use function create_syslog_header to format the syslog
  header for printk_emit uses.
o Call create_syslog_header and neaten __dev_printk
o Make __dev_printk static not global
o Remove include header declaration of __dev_printk
o Remove now unused EXPORT_SYMBOL() of __dev_printk
o Whitespace neatening

dynamic_dev_dbg:
o Remove KERN_DEBUG from dynamic_emit_prefix
o Call create_syslog_header and printk_emit
o Whitespace neatening

Signed-off-by: Joe Perches 
Acked-by: David S. Miller 
Tested-by: Jim Cromie 
Acked-by: Jason Baron 
Signed-off-by: Greg Kroah-Hartman

Merge 3.6-rc3 into driver-core-next

2012-08-27T14:08:39+00:00

This picks up the printk fixes in 3.6-rc3 that are needed in this branch.

Signed-off-by: Greg Kroah-Hartman

dyndbg: fix for SOH in logging messages

2012-08-19T06:40:13+00:00

commit af7f2158fde was done against master, and clashed with structured
logging's change of KERN_LEVEL to SOH.

Bisected and fixed by Markus Trippelsdorf.

Reported-by: Markus Trippelsdorf 
Signed-off-by: Jim Cromie 
Cc: Linus Torvalds 
Cc: Andrew Morton 
Cc: Jason Baron 
Cc: stable 
Signed-off-by: Greg Kroah-Hartman

driver core: free devres in device_release

2012-08-16T17:32:07+00:00

device_del can happen anytime, so once it happens,
the devres of the device will be freed inside device_del, but
drivers can't know it has been deleted and may still add
resources into the device, so memory leak is caused.

This patch moves the devres_release_all to fix the problem.

Signed-off-by: Ming Lei 
Signed-off-by: Greg Kroah-Hartman

drivers-core: make structured logging play nice with dynamic-debug

2012-08-16T17:20:53+00:00

commit c4e00daaa96d3a0786f1f4fe6456281c60ef9a16 changed __dev_printk
in a way that broke dynamic-debug's ability to control the dynamic
prefix of dev_dbg(dev,..), but not dev_dbg(NULL,..) or pr_debug(..),
which is why it wasnt noticed sooner.

When dev==NULL, __dev_printk() just calls printk(), which just works.
But otherwise, it assumed that level was always a string like ""
and just plucked out the 'L', ignoring the rest.  However,
dynamic_emit_prefix() adds "[tid] module:func:line:" to the string,
those additions all got lost.

Signed-off-by: Jim Cromie 
Acked-by: Jason Baron 
Cc: stable 
Cc: Kay Sievers 
Signed-off-by: Greg Kroah-Hartman

driver-core: Use kobj_to_dev instead of re-implementing it

2012-07-17T01:05:45+00:00

Signed-off-by: Lars-Peter Clausen 
Signed-off-by: Greg Kroah-Hartman

driver core: fix shutdown races with probe/remove(v3)

2012-07-17T01:04:25+00:00

Firstly, .shutdown callback may touch a uninitialized hardware
if dev->driver is set and .probe is not completed.

Secondly, device_shutdown() may dereference a null pointer to cause
oops when dev->driver is cleared after it has been checked in
device_shutdown().

So just hold device lock and its parent lock(if it has) to
fix the races.

Cc: Alan Stern 
Signed-off-by: Ming Lei 
Signed-off-by: Greg Kroah-Hartman

driver core: always handle dpm_order

2012-06-11T22:58:04+00:00

If !dev->class, device_move() does not respect the dpm_order.
Fix it to do so.

Acked-by: Rafael J. Wysocki 
Signed-off-by: Rabin Vincent 
Reviewed-by: Srinidhi Kasagar 
[Fixed a small dangling label compile warning]
Signed-off-by: Linus Walleij 
Signed-off-by: Greg Kroah-Hartman