Re: [Xen-devel] [OSSTEST PATCH v2 08/12] ts-xen-install: install Xen with XSM support if requested

[Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>]

Wei Liu writes ("Re: [OSSTEST PATCH v2 08/12] ts-xen-install: install Xen with 
XSM support if requested"):
> On Fri, Oct 10, 2014 at 04:51:12PM +0100, Ian Jackson wrote:
> > Secondly, overlay/etc/grub.d/20_linux_xen already exists (mentioning
> > Debian bug #633127).  This is automatically copied into the installed
> > system.  So any change should be made there.
> 
> OK, I missed the overlay file. No wonder the patch I wrote for my dev
> machine did work on live OSSTest host!

Heh.

> > Thirdly, you should mention the bug report that you have filed (you've
> > filed one, right?) about the need to patch in this support.
> 
> No, I haven't. My plan was to file a bug after this series is merged.
> 
> But it looks like you would like me to file a bug first. I can do this
> as well.

Yes, please.

> > Surely the right thing here is just to carry on.  Perhaps the file was
> > improved and renamed or something.  (If this it is to be done like
> > this at all, which I doubt - see above.)
> 
> If I'm to change the overlay file there won't be need to patch it
> anymore.

Right.

Ina.

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