Re: [Xen-devel] [PATCH]: PVH: specify xen features strings cleany for PVH

[Konrad Rzeszutek Wilk <konrad.wilk@xxxxxxxxxx>]

On Thu, Jan 24, 2013 at 09:27:05AM +0000, Ian Campbell wrote:
> On Thu, 2013-01-24 at 09:16 +0000, Jan Beulich wrote:
> > This looks to be an incremental patch on top of something I
> > don't think I've seen - there's neither an embedded \0 nor any
> > redundancy being removed here, yet I'm sure that was so in the
> > prior patch, and these two were what I disliked most. So perhaps
> > sending out the original patch this one goes on top of was simply
> > never done?
> 
> The baseline of this patch wouldn't have worked because it has .asciz as
> the first (global) features, so that would be NULL terminated and the
> PVH features wouldn't appear.
> 
> This one one of the first revisions I think it's probably what is
> current in Konrad's tree.

Yup.
> 
> Ian.
> 

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