Skip to content
This repository was archived by the owner on Aug 5, 2022. It is now read-only.

Conversation

@jlaako
Copy link
Contributor

@jlaako jlaako commented Feb 20, 2017

In order to properly copy rmc.db and other EFI partition contents,
remove explicit EFI subfolder copy and instead copy all boot/ contents
to the EFI partition. Also make the EFI partitions larger to properly
fit all the contents and the total image size 64 MB under 4 GB.

Signed-off-by: Jussi Laako jussi.laako@linux.intel.com

@mythi
Copy link
Contributor

mythi commented Feb 20, 2017

@jlaako we dont't build the dsk images anymore so the rmc db copying problem was not seen. Is the wic image not suitable for you?

@jlaako
Copy link
Contributor Author

jlaako commented Feb 21, 2017

@mythi I am building dsk images because I don't know why I would want wic images and how to use those. But anyway your patch for rmc was changing this dsk image recipe so I thought it was supposed to work. And IMO as long as there is possibility to build dsk/tar/whatever type of images, the correct things should be copied to correct places. If CI doesn't cover all the image types, then it's a CI deficit.

@mythi
Copy link
Contributor

mythi commented Feb 21, 2017

@jlaako The wic images are used just like dsk. It's recommended to move to the wic images as dsk is going away soon once I find the final home for do_uefiapp.

This fix looks OK until then. However, I wonder how it used to work with, e.g, emmc_partitions_data...

About the partition sizes. Can you split that into another commit and align the settings with refkit-directdisk.wks.in too

@jlaako
Copy link
Contributor Author

jlaako commented Feb 21, 2017

@mythi I'll split the partition size part out and sync.

In order to properly copy rmc.db and other EFI partition contents,
remove explicit EFI subfolder copy and instead copy all boot/ contents
to the EFI partition.

Signed-off-by: Jussi Laako <jussi.laako@linux.intel.com>
@jlaako
Copy link
Contributor Author

jlaako commented Feb 21, 2017

This should be preferably merged together with #36 otherwise dsk image creation will fail due to insufficient space.

@pohly
Copy link
Contributor

pohly commented Feb 21, 2017 via email

@pohly
Copy link
Contributor

pohly commented Feb 21, 2017 via email

@mythi
Copy link
Contributor

mythi commented Feb 21, 2017

@pohly ecae571 looks OK but let's change the sizes based on @jlaako's feedback. Will you submit a PR for it?

@mythi mythi merged commit cc4fd32 into intel:master Feb 24, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants