Re: [Xen-devel] [PATCH v8.1 15/27] xsplice, symbols: Implement fast symbol names -> virtual addresses lookup

2016-04-25 Thread Konrad Rzeszutek Wilk
On Mon, Apr 25, 2016 at 02:38:57AM -0600, Jan Beulich wrote: > >>> On 22.04.16 at 17:21, wrote: > > Here is what I came up using your idea. Do you want me to add > > 'Suggested-by' > > Jan on it? > > I'll leave that up to you; it was really just a vague idea that I gave... > > > --- a/xen/arch

Re: [Xen-devel] [PATCH v8.1 15/27] xsplice, symbols: Implement fast symbol names -> virtual addresses lookup

2016-04-25 Thread Jan Beulich
>>> On 22.04.16 at 17:21, wrote: > Here is what I came up using your idea. Do you want me to add 'Suggested-by' > Jan on it? I'll leave that up to you; it was really just a vague idea that I gave... > --- a/xen/arch/x86/Makefile > +++ b/xen/arch/x86/Makefile > @@ -74,6 +74,9 @@ efi-y := $(shell

Re: [Xen-devel] [PATCH v8.1 15/27] xsplice, symbols: Implement fast symbol names -> virtual addresses lookup

2016-04-22 Thread Konrad Rzeszutek Wilk
On Tue, Apr 19, 2016 at 01:52:33PM -0600, Jan Beulich wrote: > >>> Konrad Rzeszutek Wilk 04/14/16 12:02 AM >>> > >NEW CODE: > > > >To make that work we add three tables: > > Why three? Two (or a single one with element pairs) ought to be sufficient: > Afaict you could just have (symbol-address,sy

Re: [Xen-devel] [PATCH v8.1 15/27] xsplice, symbols: Implement fast symbol names -> virtual addresses lookup

2016-04-19 Thread Jan Beulich
>>> Konrad Rzeszutek Wilk 04/14/16 12:02 AM >>> >NEW CODE: > >To make that work we add three tables: Why three? Two (or a single one with element pairs) ought to be sufficient: Afaict you could just have (symbol-address,symbol-offset) pairs which you then sort and search by name (using symbols_ex

[Xen-devel] [PATCH v8.1 15/27] xsplice, symbols: Implement fast symbol names -> virtual addresses lookup

2016-04-13 Thread Konrad Rzeszutek Wilk
The current mechanism is geared towards fast virtual address -> symbol names lookup. This is fine for the normal use cases (BUG_ON, WARN_ON, etc), but for xSplice - where we need to find hypervisor symbols - it is slow. To understand this patch, a description of the existing method is explained fi