On Sat, Apr 12, 2025 at 03:05:13PM +0100, Andrew Cooper wrote: > On 11/04/2025 9:32 pm, Marek Marczykowski-Górecki wrote: > > @@ -28,16 +28,15 @@ cd initrd > > find . | cpio --create --format='newc' | gzip > ../initrd.cpio.gz > > cd .. > > > > -# initrd.tar.gz is Dom0 rootfs > > +# Dom0 rootfs - overlay on top of rootfs.cpio.gz > > mkdir -p rootfs > > cd rootfs > > -tar xvzf ../initrd.tar.gz > > mkdir proc > > mkdir run > > mkdir srv > > mkdir sys > > -rm var/run > > I'm not sure why we need to make proc, run, srv, sys in the overlay > image, but this is repeated everywhere. > > srv is just a plain empty dir. It's missing from $PATHS in the rootfs > generation, so that's easy.
srv is indeed likely not needed, but there is enough changes here
already.
> proc, run and sys are more complicated, because we don't want to simply
> copy them from the rootfs generation environment.
As for those mountpoints, they looks to be necessary - in a job that
failed to unpack this overlay part, openrc failed to mount those
filesystems and crashed rather badly (especially without /run...).
I'm not sure if that's something that is missing in the base archive, or
is a problem for another reason, but it seems to be necessary to have it
here, at least without changes elsewhere.
Looking at PATHS in alpine-rootfs.sh they may be simply skipped there.
> dev is also in the same boat, and what we do now seems to cause openrc
> not to seed it normally, which seems unwise.
It mounts devtmpfs there anyway, no?
> I'll see what I can do to clean this all up.
>
> > diff --git a/automation/scripts/xilinx-smoke-dom0-x86_64.sh
> > b/automation/scripts/xilinx-smoke-dom0-x86_64.sh
> > index 4db249530823..52a0e05e3f1b 100755
> > --- a/automation/scripts/xilinx-smoke-dom0-x86_64.sh
> > +++ b/automation/scripts/xilinx-smoke-dom0-x86_64.sh
> > @@ -80,6 +80,7 @@ argo-exec -l -p 28333 -- /bin/echo
> > "
> > copy_dom0_files ()
> > {
> > + mkdir -p root usr/local/lib
> > cp "${WORKDIR}/binaries/xen-argo.ko" "root/"
> > cp -ar "${WORKDIR}/binaries/lib/"* "usr/local/lib/"
> > cp "${WORKDIR}/binaries/argo-exec" "usr/local/bin/"
>
> This would be better based on top of my patch to switch to the new argo,
> at which point it will simply be to append argo.cpio.gz.
>
> ~Andrew
--
Best Regards,
Marek Marczykowski-Górecki
Invisible Things Lab
signature.asc
Description: PGP signature
