[PATCH 0/9] objtool: Rewrite annotations

Peter Zijlstra posted 9 patches 1 year, 2 months ago
There is a newer version of this series
[PATCH 0/9] objtool: Rewrite annotations
Posted by Peter Zijlstra 1 year, 2 months ago
Just the objtool annotation rewrite bits.

Changes since last time:

 - split from the x86 and kvm patches
 - s/ANNOTYPE_INTRA_FUNCTION_CALLS/ANNOTYPE_INTRA_FUNCTION_CALL/g
 - made __ASM_ANNOTATE() take a full label name (no longer appends 'b')
 - added a patch that moves all the annotations to objtool.h
 - some changes to the first patch
Re: [PATCH 0/9] objtool: Rewrite annotations
Posted by Josh Poimboeuf 1 year, 2 months ago
On Fri, Nov 22, 2024 at 01:10:16PM +0100, Peter Zijlstra wrote:
> Just the objtool annotation rewrite bits.
> 
> Changes since last time:
> 
>  - split from the x86 and kvm patches
>  - s/ANNOTYPE_INTRA_FUNCTION_CALLS/ANNOTYPE_INTRA_FUNCTION_CALL/g
>  - made __ASM_ANNOTATE() take a full label name (no longer appends 'b')
>  - added a patch that moves all the annotations to objtool.h
>  - some changes to the first patch

For Valentin's thing we'll be adding annotations for static keys.  Those
will be symbol specific, like STACK_FRAME_NON_STANDARD().  In which case
we could have a generic .discard.annotate_sym.  And then rename
.discard.annotate to .discard.annotate_insn?

-- 
Josh
Re: [PATCH 0/9] objtool: Rewrite annotations
Posted by Peter Zijlstra 1 year, 2 months ago
On Fri, Nov 22, 2024 at 09:40:56AM -0800, Josh Poimboeuf wrote:
> On Fri, Nov 22, 2024 at 01:10:16PM +0100, Peter Zijlstra wrote:
> > Just the objtool annotation rewrite bits.
> > 
> > Changes since last time:
> > 
> >  - split from the x86 and kvm patches
> >  - s/ANNOTYPE_INTRA_FUNCTION_CALLS/ANNOTYPE_INTRA_FUNCTION_CALL/g
> >  - made __ASM_ANNOTATE() take a full label name (no longer appends 'b')
> >  - added a patch that moves all the annotations to objtool.h
> >  - some changes to the first patch
> 
> For Valentin's thing we'll be adding annotations for static keys.  Those
> will be symbol specific, like STACK_FRAME_NON_STANDARD().  In which case
> we could have a generic .discard.annotate_sym.  And then rename
> .discard.annotate to .discard.annotate_insn?

Done.