On 2015-12-03 at 15:48 "'Davide Libenzi' via Akaros"
<[email protected]> wrote:
> Sorry, forgot to mention: include/generated/asm-offsets.h

The generation of this looks a little intense:

#####
# 3) Generate asm-offsets.h
#

offsets-file := include/generated/asm-offsets.h

always  += $(offsets-file)
targets += arch/$(SRCARCH)/kernel/asm-offsets.s

# We use internal kbuild rules to avoid the "is up to date" message from make
arch/$(SRCARCH)/kernel/asm-offsets.s: arch/$(SRCARCH)/kernel/asm-offsets.c \
                                      $(obj)/$(timeconst-file) 
$(obj)/$(bounds-file) FORCE
    $(Q)mkdir -p $(dir $@)
    $(call if_changed_dep,cc_s_c)

$(obj)/$(offsets-file): arch/$(SRCARCH)/kernel/asm-offsets.s FORCE
    $(call filechk,offsets,__ASM_OFFSETS_H__)


we don't have the timeconst, bounds-file, or any of the stuff that
asm-offsets.c needs.

That being said, all of the .S code is tightly coupled with the
trapframe code and I don't see the payoff for doing the asm-offsets at
this point.

I actually have a static_assert in place for that 0x90.  I'll add one
for the 0xac as well.

Barret


-- 
You received this message because you are subscribed to the Google Groups 
"Akaros" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To post to this group, send email to [email protected].
For more options, visit https://groups.google.com/d/optout.

Reply via email to