On Mon, 2007-03-26 at 01:14 -0800, Andrew Morton wrote: > arch/i386/lguest/lguest.c:496: warning: assignment from incompatible pointer type > arch/i386/lguest/lguest.c:497: warning: assignment from incompatible pointer type > arch/i386/lguest/lguest.c:498: warning: assignment from incompatible pointer type > > paravirt_ops.write_ldt_entry = lguest_write_ldt_entry; > paravirt_ops.write_gdt_entry = lguest_write_gdt_entry; > paravirt_ops.write_idt_entry = lguest_write_idt_entry; And here it is: == The prototype for paravirt_ops write_lgt_entry, write_gdt_entry and write_idt_entry was type-corrected, so change lguest to match. We can also use the "write_dt_entry" which was exported by that same cleanup patch, rather than write our own. Signed-off-by: Rusty Russell <rusty@xxxxxxxxxxxxxxx> diff -r f52fef6a1f2d arch/i386/lguest/lguest.c --- a/arch/i386/lguest/lguest.c Mon Mar 26 10:16:33 2007 +1000 +++ b/arch/i386/lguest/lguest.c Tue Mar 27 10:59:21 2007 +1000 @@ -362,30 +362,24 @@ static void __init lguest_init_IRQ(void) irq_ctx_init(smp_processor_id()); } -static inline void native_write_dt_entry(void *dt, int entry, u32 entry_low, u32 entry_high) -{ - u32 *lp = (u32 *)((char *)dt + entry*8); - lp[0] = entry_low; - lp[1] = entry_high; -} - -static fastcall void lguest_write_ldt_entry(void *dt, int entrynum, u32 low, u32 high) +static fastcall void lguest_write_ldt_entry(struct desc_struct *dt, + int entrynum, u32 low, u32 high) { /* FIXME: Allow this. */ BUG(); } -static fastcall void lguest_write_gdt_entry(void *dt, int entrynum, - u32 low, u32 high) -{ - native_write_dt_entry(dt, entrynum, low, high); +static fastcall void lguest_write_gdt_entry(struct desc_struct *dt, + int entrynum, u32 low, u32 high) +{ + write_dt_entry(dt, entrynum, low, high); hcall(LHCALL_LOAD_GDT, __pa(dt), GDT_ENTRIES, 0); } -static fastcall void lguest_write_idt_entry(void *dt, int entrynum, - u32 low, u32 high) -{ - native_write_dt_entry(dt, entrynum, low, high); +static fastcall void lguest_write_idt_entry(struct desc_struct *dt, + int entrynum, u32 low, u32 high) +{ + write_dt_entry(dt, entrynum, low, high); hcall(LHCALL_LOAD_IDT_ENTRY, entrynum, low, high); } _______________________________________________ Virtualization mailing list Virtualization@xxxxxxxxxxxxxxxxxxxxxxxxxx https://lists.linux-foundation.org/mailman/listinfo/virtualization