[OE-core] [PATCH] var/volatile: Mount it if doing ro-rfs

[Khem Raj]

> On Mar 11, 2015, at 7:53 AM, Khem Raj <raj.khem at gmail.com> wrote:
> 
> 
> 
> On Wednesday, March 11, 2015, Richard Purdie <richard.purdie at linuxfoundation.org <mailto:richard.purdie at linuxfoundation.org>> wrote:
> On Wed, 2015-03-11 at 08:33 +0000, Khem Raj wrote:
> > - Do not mount /var/volatile to be tmpfs always only when generating
> >   readonly rootfs
> > - Generate var-volatile.mount unit when using system in readonly-rootfs
> >   to mount /var/volatile as tmpfs
> > - for sysvinit, mount /var/volatile to be tmpfs if its not already
> >   mounted so
> > - Use RequiresMountsFor instead of After=, so systemd figures out the
> >   needed dependencies for making /var/volatile accessible for journald
> >   and not us.
> >
> > Change-Id: I0b176b3e1c1e88e84d2c93154aac04cef565d8ea
> > Signed-off-by: Khem Raj <raj.khem at gmail.com <javascript:;>>
> > ---
> >  meta/classes/image.bbclass                                             | 1 +
> >  meta/recipes-core/base-files/base-files/fstab                          | 1 -
> >  meta/recipes-core/initscripts/initscripts-1.0/read-only-rootfs-hook.sh | 2 +-
> >  meta/recipes-core/systemd/systemd/journald-volatile.conf               | 2 +-
> >  4 files changed, 3 insertions(+), 3 deletions(-)
> 
> Which combinations has this been tested with? Given the time the patch
> has existed for, I'm sceptical all the combinations we need have been
> tested, I'd love to be wrong.
> 
> This is a pretty major change of behaviour and given the problems
> previous changes have caused, I'm very very very nervous about what
> issues this is going to cause :(.
> 
> Systemd in both ways but not sysvinit 


Just when you think all is well, I just sent a V2, that seemed to work on x86 emulator, but due diligence is better I don’t have sysvinit setups, its fine if it doesn’t go in,
the fact that patch is out there folks can always back port after release. 

> 
> Cheers,
> 
> Richard
> 

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openembedded.org/pipermail/openembedded-core/attachments/20150311/b0e191b4/attachment-0002.html>