RE: [Xen-ia64-devel] RE: Patches for ia64 arch_vcpu_info_t merge

["Magenheimer, Dan (HP Labs Fort Collins)" <dan.magenheimer@xxxxxx>]

> > 2) Checked in the asm-xsi-offsets.c file and the changes
> >    to arch/ia64/Makefile to build it.  (NOTE: It appears
> >    that this change results in a complete rebuild for
> >    every file change?  Please confirm.) 
> We will validate it tonight and provide a patch for it. 

Thanks, it seems to work.

> > 2) Please start using the Intel hg staging tree.
> We are definitely to use for our development

Looks good so far!

Since you don't have an Intel xenlinux tree, please send
that patch separately and I will apply it and test.

As I've been discussing with Eddie, I'm rather unhappy with
the way the merge has turned arch-ia64.h into such a mess.
I had planned on simplifying it rather than making it more
complex.  But I guess we can go with this merge for now.
 
> > 3) Please separate unrelated changes into separate changesets.
> Can you share more light on this?  Thanks.

It appears that you are doing that already now.

Looking over the last changeset, I am concerned that the
console default may have switched to tiger4.  From what I
understand, this box is only used inside of Intel.  It would
probably be best if the default is left to be the box
that most developers are using.  Or, alternately, perhaps
someone in Intel can study the existing Linux code for
serial init... the best long-term default is probably
whatever Linux does.

Dan

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