From: Ben Pfaff Date: Mon, 10 Jan 2005 07:18:03 +0000 (+0000) Subject: Fix order of edi, esi, ebp, ebx in switch_threads_frame. Doesn't X-Git-Url: https://pintos-os.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=f329c5c10351234a6a5e2757c5f070f02b73639c;p=pintos-anon Fix order of edi, esi, ebp, ebx in switch_threads_frame. Doesn't matter because they're never used in the C code, but it's good to have them correct. Thanks to Howie Xu for reporting this bug. --- diff --git a/src/threads/switch.h b/src/threads/switch.h index d79af5d..cc156b6 100644 --- a/src/threads/switch.h +++ b/src/threads/switch.h @@ -5,10 +5,10 @@ /* switch_thread()'s stack frame. */ struct switch_threads_frame { - uint32_t ebx; /* 0: Saved %ebx. */ - uint32_t ebp; /* 4: Saved %ebp. */ - uint32_t esi; /* 8: Saved %esi. */ - uint32_t edi; /* 12: Saved %edi. */ + uint32_t edi; /* 0: Saved %edi. */ + uint32_t esi; /* 4: Saved %esi. */ + uint32_t ebp; /* 8: Saved %ebp. */ + uint32_t ebx; /* 12: Saved %ebx. */ void (*eip) (void); /* 16: Return address. */ struct thread *cur; /* 20: switch_threads()'s CUR argument. */ struct thread *next; /* 24: switch_threads()'s NEXT argument. */