[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH] common/symbols: Drop '+0/<len>' when printing function pointer symbols.



>>> On 04.10.13 at 12:02, Andrew Cooper <andrew.cooper3@xxxxxxxxxx> wrote:
> Introduce print_function() with the same semantics as print_symbol().  The
> underlying __print_symbol() now takes an extra boolean indicating whether we
> are expecting to print a function pointer.   In the case that we are 
> expecting
> a function pointer, and the offset is 0, drop the offset and length.
> 
> The requirement for offset being 0 is for the (hopefully never, but we 
> really
> want to know if it does happen) case where a Xen function pointer is not
> actually pointing at the start of a function.
> 
> The relevent print_symbol() functions are updated to print_function()

There's no reason why the same couldn't apply to data symbols.
Rather than doing it this way (and with a mis-named function), I'd
much prefer going the printk() format string extensions route that
Linux went. This would at once allow re-combining the broken up
printing when symbols are involved into single invocations of printk().
This has been on my (mental) to-do list for quite some time, but
hasn't been important enough for me to ever get to it.

Jan


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.