Re: 2.6.18-rc3-g3b445eea BUG: warning at /usr/src/linux-git/kernel/cpu.c:51

["Jan Beulich" <jbeulich@novell.com>]

>>> Chuck Ebbert <76306.1226@compuserve.com> 08.08.06 02:39 >>>
>In-Reply-To: <44D7136E.76E4.0078.0@novell.com>
>
>On Mon, 07 Aug 2006 09:18:22 +0100, Jan Beulich wrote:
>
>> >Most likely the CFI annotation for that sysenter path is not complete.
>>
>> Correct, the return point of sysexit (SYSENTER_RETURN) is still in kernel space,
>> but its annotations are invisible to the unwinder. We should make the VDSO be
>> treated as user-mode code despite living above PAGE_OFFSET.
>
>Umm, that's already been done?
>
>include/asm-i386/unwind.h::arch_unw_user_mode():
>        return info->regs.eip < PAGE_OFFSET
>               || (info->regs.eip >= __fix_to_virt(FIX_VDSO)
>                    && info->regs.eip < __fix_to_virt(FIX_VDSO) + PAGE_SIZE)
>               || info->regs.esp < PAGE_OFFSET;

Hmm, indeed. Then I'm unclear what the problem might be here.

>Could this be the problem?
>
>|ENTRY(sysenter_entry)
>|        CFI_STARTPROC simple
>|        CFI_DEF_CFA esp, 0
>|==>     CFI_REGISTER esp, ebp
>|        movl TSS_sysenter_esp0(%esp),%esp
>|sysenter_past_esp:

Clearly not. That is the way the user stack gets communicated to the kernel,
and it would cause problems at earliest in the next outer frame (which is in
user mode, so we don't care anyway). And I know I saw it unwind properly
through the sysenter code in other cases in the past.

>What does that do?  .cfi_register is not documented anywhere, not
>even in the gnu.org online documentation for gas.  (I spent 10
>minutes googling and found nothing other than the changeset that
>added it to gas.)

.cfi_register is the directive equivalent to DW_CFA_register, saying that
on register is spilled to another (rather than to memory) - see the Dwarf2
(or Dwarf3) specification for details.

Jan