[Skiboot] [PATCH] external/mambo: support mambo COW mode for PMEM disk

[Oliver O'Halloran]

On Tue, Feb 25, 2020 at 8:15 AM Michael Neuling <mikey at neuling.org> wrote:
>
> On Tue, 2020-02-18 at 14:16 -0600, Aaron Sawdey wrote:
> > I've added support in mambo's "memory mmap" command to have a "cow" mode
> > which just uses MAP_PRIVATE instead of MAP_SHARED on the file so that writes
> > to the memory region are not sent back to the file. This allows multiple
> > mambo instances to share the same filesystem image.
> >
> > This is implemented by having a PMEM_MODES environment variable. If this
> > is set, it is expected to contain a comma separated list of modes (rw or cow)
> > for the list of files in PMEM_DISK. If there are fewer modes than files, the
> > remaining files default to "rw".
>
> I'm not a big fan of the COW name. It suggests the writes are going somewhere. I
> realise this is the name used in mambo, but I'm not super keen on it. "PRIVATE"
> might be a better name.
>
> Can we call this (singular) PMEM_MODE since we have PMEM_DISK?
>
> Can you also include an example ie
>
>    export PMEM_DISK="mydisk1, mydisk2"
>    export PMEM_MODE="rw, cow"
>
>    Other than that, the code looks good.
>
>    Mikey

Jokes on you. I already merged it.