FAQ Search Today's Posts Mark Forums Read
» Video Reviews

» Linux Archive

Linux-archive is a website aiming to archive linux email lists and to make them easily accessible for linux users/developers.


» Sponsor

» Partners

» Sponsor

Go Back   Linux Archive > Redhat > Fedora Development

 
 
LinkBack Thread Tools
 
Old 02-09-2010, 02:49 PM
 
Default Cut the size of the boot.img for ia64 in half (#556976).

From: Ales Kozumplik <akozumpl@redhat.com>

This is by removing (what I suppose is) redundant copy of the efi/boot contents
in the root of the image.
---
scripts/mk-images.ia64 | 3 ---
1 files changed, 0 insertions(+), 3 deletions(-)

diff --git a/scripts/mk-images.ia64 b/scripts/mk-images.ia64
index 5843aa8..0ac748f 100644
--- a/scripts/mk-images.ia64
+++ b/scripts/mk-images.ia64
@@ -102,9 +102,6 @@ image=vmlinuz
read-only
initrd=initrd.img
EOF
-
- # make a copy in the root of the image
- cp $MBD_BOOTTREE_TMP/EFI/boot/* $MBD_BOOTTREE_TMP
}

makeBootImages() {
--
1.6.6

_______________________________________________
Anaconda-devel-list mailing list
Anaconda-devel-list@redhat.com
https://www.redhat.com/mailman/listinfo/anaconda-devel-list
 
Old 02-10-2010, 07:40 AM
Ales Kozumplik
 
Default Cut the size of the boot.img for ia64 in half (#556976).

On 02/10/2010 12:31 AM, Jeremy Katz wrote:

It's not actually redundant; it is/was required for some IA64 EFI
implementations

- Jeremy



Doug Chapman (https://bugzilla.redhat.com/show_bug.cgi?id=556976#c65)
says that removing one of the images is the right thing to do, but
perhaps he was not aware of this. Jeremy, does the IA64 implementations
you are referring to actually need *both* the copies? (or one needs the
root copy and the other one needs the efi/boot copy). Because then we
could split the image in two (and say it in the release notes).


Otherwise, would putting relative paths in elilo.conf help?

Ales

_______________________________________________
Anaconda-devel-list mailing list
Anaconda-devel-list@redhat.com
https://www.redhat.com/mailman/listinfo/anaconda-devel-list
 
Old 02-15-2010, 03:11 PM
Ales Kozumplik
 
Default Cut the size of the boot.img for ia64 in half (#556976).

This is by removing (what I suppose is) redundant copy of the efi/boot's initrd
and vmlinuz. As a fallback, the elilo.conf in the root is still pointing at the
right files under efi/boot.
---
scripts/mk-images.ia64 | 15 +++++++++++++--
1 files changed, 13 insertions(+), 2 deletions(-)

diff --git a/scripts/mk-images.ia64 b/scripts/mk-images.ia64
index 5843aa8..3e3f52f 100644
--- a/scripts/mk-images.ia64
+++ b/scripts/mk-images.ia64
@@ -102,9 +102,20 @@ image=vmlinuz
read-only
initrd=initrd.img
EOF
+ # now crate a similar copy in the root of the image
+ # (this is arguably needed for certain EFI implementations)
+ cp -a $BOOTDISKDIR/* $MBD_BOOTTREE_TMP/
+ cp $MBD_BOOTTREE_TMP/elilo.efi $MBD_BOOTTREE_TMP/bootia64.efi
+ cat > $MBD_BOOTTREE_TMP/elilo.conf << EOF
+prompt
+timeout=50
+relocatable

- # make a copy in the root of the image
- cp $MBD_BOOTTREE_TMP/EFI/boot/* $MBD_BOOTTREE_TMP
+image=efi/boot/vmlinuz
+ label=linux
+ read-only
+ initrd=efi/boot/initrd.img
+EOF
}

makeBootImages() {
--
1.6.6

_______________________________________________
Anaconda-devel-list mailing list
Anaconda-devel-list@redhat.com
https://www.redhat.com/mailman/listinfo/anaconda-devel-list
 
Old 02-15-2010, 08:09 PM
Chris Lumens
 
Default Cut the size of the boot.img for ia64 in half (#556976).

> diff --git a/scripts/mk-images.ia64 b/scripts/mk-images.ia64
> index 5843aa8..3e3f52f 100644
> --- a/scripts/mk-images.ia64
> +++ b/scripts/mk-images.ia64
> @@ -102,9 +102,20 @@ image=vmlinuz
> read-only
> initrd=initrd.img
> EOF
> + # now crate a similar copy in the root of the image

You mean "create".

> + # (this is arguably needed for certain EFI implementations)
> + cp -a $BOOTDISKDIR/* $MBD_BOOTTREE_TMP/
> + cp $MBD_BOOTTREE_TMP/elilo.efi $MBD_BOOTTREE_TMP/bootia64.efi
> + cat > $MBD_BOOTTREE_TMP/elilo.conf << EOF
> +prompt
> +timeout=50
> +relocatable
>
> - # make a copy in the root of the image
> - cp $MBD_BOOTTREE_TMP/EFI/boot/* $MBD_BOOTTREE_TMP
> +image=efi/boot/vmlinuz
> + label=linux
> + read-only
> + initrd=efi/boot/initrd.img
> +EOF
> }

The indentation is a little weird here - shouldn't label=, read-only,
and initrd= be brought back out to the far left as well?

Anyway this looks fine to me, and the test results are excellent.
Thanks for putting together the test images.

- Chris

_______________________________________________
Anaconda-devel-list mailing list
Anaconda-devel-list@redhat.com
https://www.redhat.com/mailman/listinfo/anaconda-devel-list
 
Old 02-16-2010, 06:45 AM
Ales Kozumplik
 
Default Cut the size of the boot.img for ia64 in half (#556976).

On 02/15/2010 10:09 PM, Chris Lumens wrote:


- # make a copy in the root of the image
- cp $MBD_BOOTTREE_TMP/EFI/boot/* $MBD_BOOTTREE_TMP
+image=efi/boot/vmlinuz
+ label=linux
+ read-only
+ initrd=efi/boot/initrd.img
+EOF
}


The indentation is a little weird here - shouldn't label=, read-only,
and initrd= be brought back out to the far left as well?



It should probably. I copied the heredoc exactly from the epi/boot:
image= unindented, label= and read-only indented with spaces and
initrd=indented with a tab. It looks horrible but works there OK and I
can't test whether indenting it nicely wouldn't break it.


Ales

_______________________________________________
Anaconda-devel-list mailing list
Anaconda-devel-list@redhat.com
https://www.redhat.com/mailman/listinfo/anaconda-devel-list
 

Thread Tools




All times are GMT. The time now is 11:45 PM.

VBulletin, Copyright ©2000 - 2014, Jelsoft Enterprises Ltd.
Content Relevant URLs by vBSEO ©2007, Crawlability, Inc.
Copyright 2007 - 2008, www.linux-archive.org