On Tue, Mar 10, 2009 at 03:40:21PM +0800, Han, Weidong wrote:
> Simon Horman wrote:
> > In pt_irq_time_out() the following code loops through all used
> > guest_gsi:
> >
> > list_for_each_entry ( digl, &irq_map->digl_list, list )
> > {
> > guest_gsi = digl->gsi;
> > machine_gsi = dpci->girq[guest_gsi].machine_gsi;
> > ...
> > }
> >
> > And a little later on machine_gsi is used.
> > That is the last machine_gsi found is used,
> > rather than all of the machine_gsi that are found.
> >
> > This seems to be incorrect to me,
> > but I am unsure of how to test this.
> >
>
> Timer is set for each machine GSI, so all the machine_gsi found in loop
> are the same. More than one devices may share machine GSI, but assume
> they won't share guest gsi. digl_list contains all guest GSIs which are
> correspond to this a machine GSI.
Ok, thanks. I understand now.
>
> Now you want pass-throughed devices share guest gsi, you needs to change
> it obviously. Your below change looks fine for me.
Thanks
--
Simon Horman
VA Linux Systems Japan K.K., Sydney, Australia Satellite Office
H: www.vergenet.net/~horms/ W: www.valinux.co.jp/en
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel
|
Home