Re: + stupid-hack-to-make-mainline-build.patch added to -mm tree

[Dan Hecht]

On 03/07/2007 12:57 PM, Thomas Gleixner wrote:
> On Wed, 2007-03-07 at 12:11 -0800, Jeremy Fitzhardinge wrote:
> > Dan Hecht wrote:
> > > Jeremy, I saw you sent out the Xen version earlier, thanks.  Here's
> > > ours for reference (please excuse any formating issues); it's also
> > > lean. We'll send out a proper patch later after some more testing:
> > So the interrupt side of the clockevent comes through the virtual apic? 
> > Where does evt->handle_event get called?
>
>
> >         /* We use normal irq0 handler on cpu0. */
> >         time_init_hook();
>
> That's exactly the thing I ranted about before. We keep the historic
> view of emulated hardware and just wrap it into enough glue code instead
> of doing an abstract design, which just gets rid of those hardware
> assumptions at all. That's the big advantage of paravirtualization, but
> the current way on paravirt ops is just ignoring this.

Are you saying you would prefer we create our own irq handler something 
like this rather than using the standard i386 handlers?

irqreturn_t vmi_timer_interrupt(int irq, void *dev_id)
{
   local_event->event_handler(local_event);
   return IRQ_HANDLED;
}

??  That's fine with me.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/