Bug 456559 - anaconda: i386 xen installer images creation
Summary: anaconda: i386 xen installer images creation
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: anaconda
Version: rawhide
Hardware: All
OS: Linux
low
low
Target Milestone: ---
Assignee: Mark McLoughlin
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: F10_XenPvOps F10Beta, F10BetaBlocker
TreeView+ depends on / blocked
 
Reported: 2008-07-24 17:21 UTC by Mark McLoughlin
Modified: 2008-09-10 10:49 UTC (History)
4 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2008-09-10 10:49:25 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)
mk-images-replace-kernel-xen-with-pv_ops-kernel.patch (5.52 KB, patch)
2008-08-06 17:23 UTC, Mark McLoughlin
no flags Details | Diff

Description Mark McLoughlin 2008-07-24 17:21:08 UTC
Now that xen guest support has been moved to kernel-PAE.i686 and kernel.x86_64,
we need to drop any remnants of code to handle kernel-xen.

Should be trivial, just filing this to remind myself

Comment 1 Jeremy Katz 2008-07-24 17:48:56 UTC
Hrmm -- the fact that it's in the PAE kernel for x86 adds a couple of wrinkles
1) We don't install the PAE kernel unless you have memory above 4 gigs.  
2) We don't ever use the PAE kernel as the installer kernel, instead using the
i586 kernel in Fedora, i686 kernel in RHEL

Comment 2 Daniel Berrangé 2008-07-24 17:54:55 UTC
Fortunately the '.treeinfo' file could help us here. Currently it says:

[images-i386]
kernel = images/pxeboot/vmlinuz
initrd = images/pxeboot/initrd.img
boot.iso = images/boot.iso
[images-xen]
kernel = images/xen/vmlinuz
initrd = images/xen/initrd.img

We could have the 'images/xen' directory contain the PAE variant of 'kernel', or
have a new  images/pxeboot/vmlinux-PAE and 'images/pxeboot/initrd-PAE.img' and
make the 'images-xen' section refer to those.

Since F8 virt-install and virt-manager will both look at the $URL/.treeinfo file
when picking the kernel for guest installs.


Comment 3 Jeremy Katz 2008-07-24 18:17:40 UTC
Yeah, but we still then have to have double sets of images lying around which is
unfortunate.  And we have to switch the logic which does kernel-xen to instead
do kernel-PAE.  Which means that we don't really gain much in the way of
simplifying logic or life.

Comment 4 Bill Nottingham 2008-07-25 14:32:41 UTC
Marking as blocking the beta - whatever we decide to do should be sorted out
then for testing.

Comment 5 Mark McLoughlin 2008-08-06 17:23:46 UTC
Created attachment 313624 [details]
mk-images-replace-kernel-xen-with-pv_ops-kernel.patch

Here's a mk-images patch which I'm still testing

I've also got a bunch of small patches to kill off obsolete xen hacks in anaconda proper; I'll send those out tomorrow

Comment 6 Mark McLoughlin 2008-08-07 09:29:03 UTC
*** Bug 458164 has been marked as a duplicate of this bug. ***

Comment 7 Jeremy Katz 2008-08-07 13:31:34 UTC
Applied, thanks

Comment 8 Daniel Berrangé 2008-08-12 09:31:02 UTC
*** Bug 458164 has been marked as a duplicate of this bug. ***

Comment 9 Mark McLoughlin 2008-08-12 10:18:10 UTC
Seems to be working fine on x86_64 now, but i386 doesn't have .treeinfo/[images-xen] or images/pxeboot/vmlinuz-PAE

Comment 10 Mark McLoughlin 2008-08-14 11:20:39 UTC
Looking at e.g.:

  http://kojipkgs.fedoraproject.org/mash/rawhide-20080812/logs/i386.log

  Wrote /tmp/treebuild.20080812/output/development/i386/os/images/pxeboot/efiboot.img (39159844k free)
  Building stage2.img

I'd expect to it to say "Building initrd-PAE.img" before going onto building stage2.img

I can't see anything obviously wrong with the code and because of bug #458541, I can't easily debug it either

Comment 11 Mark McLoughlin 2008-09-10 10:49:25 UTC
Don't know what's changed but:

  http://kojipkgs.fedoraproject.org/mash/rawhide-20080910/logs/i386.log

shows:

  Building initrd-PAE.img

and:

  http://download.fedora.redhat.com/pub/fedora/linux/development/i386/os/.treeinfo

contains:

  [images-xen]
  initrd = images/pxeboot/initrd-PAE.img
  kernel = images/pxeboot/vmlinuz-PAE


Note You need to log in before you can comment on or make changes to this bug.