On Monday 07 August 2006 09:50, Rusty Russell wrote:

On Mon, 2006-08-07 at 07:40 +0200, Andi Kleen wrote:

quoted

On Monday 07 August 2006 06:45, Rusty Russell wrote:

quoted

Unfortunately, due to include cycles, we can't put these in
paravirt.h: we use a separate header for these.

The implementation comes from Zach's [RFC, PATCH 10/24] i386 Vmi descriptor changes:

  Descriptor and trap table cleanups.  Add cleanly written accessors for
  IDT and GDT gates so the subarch may override them.  Note that this
  allows the hypervisor to transparently tweak the DPL of the descriptors
  as well as the RPL of segments in those descriptors, with no unnecessary
  kernel code modification.  It also allows the hypervisor implementation
  of the VMI to tweak the gates, allowing for custom exception frames or
  extra layers of indirection above the guest fault / IRQ handlers.

Nice cleanup. The old assembly mess was ripe to be killed for a long time.

OK, here's that patch extracted out.

Is there something wrong with your mailer? This one doesn't apply either:

Applying patch patches/paravirt_desc.h-for-native
patching file arch/i386/kernel/traps.c
Hunk #1 FAILED at 1112.
1 out of 1 hunk FAILED -- rejects in file arch/i386/kernel/traps.c
missing header for unified diff at line 78 of patch
can't find file to patch at input line 78
Perhaps you used the wrong -p or --strip option?
The text leading up to this was:
--------------------------
|  * thus use the _nonmapped_ version of the IDT, as the
--------------------------
No file to patch.  Skipping patch.
1 out of 1 hunk ignored
missing header for unified diff at line 88 of patch
can't find file to patch at input line 88
Perhaps you used the wrong -p or --strip option?
The text leading up to this was:
--------------------------
| /*
--------------------------


-Andi