[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH 01/13] xen: fix output of xen_debug_interrupt



On Fri, 2013-02-01 at 11:06 +0000, David Vrabel wrote:
> On 31/01/13 14:46, Wei Liu wrote:
> > Four things are fixed:
> >  a) the test result of globaly masked event;
> >  b) make the per-cpu selector L1 to be consistent with description in
> >     __xen_evtchn_do_upcall's comment;
> >  c) the test result of L1 selector;
> >  d) add KERN_DEBUG in printk.
> 
> It's hard to pick out the correctness fixes with all the cosmetic
> KERN_DEBUG fixes in the same patch.
> 

Moved to separate changeset.

> > Signed-off-by: Wei Liu <wei.liu2@xxxxxxxxxx>
> > ---
> >  drivers/xen/events.c |   34 +++++++++++++++++-----------------
> >  1 file changed, 17 insertions(+), 17 deletions(-)
> > 
> > diff --git a/drivers/xen/events.c b/drivers/xen/events.c
> > index 7595581..2c94aad 100644
> > --- a/drivers/xen/events.c
> > +++ b/drivers/xen/events.c
> [...]
> > +                          !sync_test_bit(word_idx, &v->evtchn_pending_sel)
> > +                                        ? "" : " l1-clear",
> 
> This looks backwards now.
> 

Did you mean the "globally-masked" below? I think I made a mistake here,
because in global mask 1 means masked while in per cpu mask 0 means
masked...


Wei.

> > +                          sync_test_bit(i, sh->evtchn_mask)
> >                                          ? "" : " globally-masked",
> >                            sync_test_bit(i, cpu_evtchn)
> >                                          ? "" : " locally-masked");
> 
> David
> 



_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.