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

Re: [Xen-ia64-devel] PATCH: mini-cleanup in vtlb.c

To: tgingold@xxxxxxx
From: Aron Griffis <aron@xxxxxx>
Date: Sun, 16 Sep 2007 00:40:56 -0400
Cc: Alex Williamson <alex.williamson@xxxxxx>, Xen-ia64-devel <xen-ia64-devel@xxxxxxxxxxxxxxxxxxx>
Delivery-date: Sat, 15 Sep 2007 21:46:38 -0700
List-id: Discussion of the ia64 port of Xen <xen-ia64-devel.lists.xensource.com>
Mail-followup-to: tgingold@xxxxxxx, Xen-ia64-devel <xen-ia64-devel@xxxxxxxxxxxxxxxxxxx>, Alex Williamson <alex.williamson@xxxxxx>

Hi Tristan,

tgingold@xxxxxxx wrote:  [Sat Sep 15 2007, 11:04:22PM EDT]
> -    rid = rid&RR_RID_MASK;;
> +    rid = rid & RR_RID_MASK;;

This appears twice in the cleanup, verbatim...  Couldn't it just be
rid &= RR_RID_MASK?  Also, why the double-semicolon?  I don't think
that means anything in C.

Thanks,
Aron

_______________________________________________
Xen-ia64-devel mailing list
Xen-ia64-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-ia64-devel

Follow-Ups:
- Re: [Xen-ia64-devel] PATCH: mini-cleanup in vtlb.c
  - From: Tristan Gingold

References:
- [Xen-ia64-devel] PATCH: mini-cleanup in vtlb.c
  - From: tgingold

Prev by Date: [Xen-ia64-devel] PATCH: fill more registers for vcpu_getcontext
Next by Date: Re: [Xen-ia64-devel] Shall we put open source firmware source in XEN/IA64 tree
Previous by thread: [Xen-ia64-devel] PATCH: mini-cleanup in vtlb.c
Next by thread: Re: [Xen-ia64-devel] PATCH: mini-cleanup in vtlb.c
Index(es):
- Date
- Thread

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