<div dir="ltr">Thank you so much for your time.<div>Let me try changing JFFS2 to UBIFS as another available option to use for my SOC.</div><div><br></div><div>I would keep you guys posted.</div><div><br></div><div>Regards,</div><div>Karnik Jain</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, May 21, 2019 at 8:18 PM Lennart Poettering <<a href="mailto:lennart@poettering.net">lennart@poettering.net</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Di, 21.05.19 17:37, Kay One (<a href="mailto:kayone007@gmail.com" target="_blank">kayone007@gmail.com</a>) wrote:<br>
<br>
> Thanks for the reply Lennart.<br>
><br>
> 1) mmap2(NULL, 4096, PROT_READ|PROT_WRITE, MAP_SHARED, 28, 0) = -1 EINVAL<br>
> > (Invalid argument) is the culprit for this error???<br>
<br>
Yes.<br>
<br>
> 2) If YES. Is this JFFS2 MTD file system bug or<br>
<br>
It's a missing feature of the fs implementation, that the journal<br>
relies on. Memory mapping a file is not a particularly exotic feature,<br>
and yes, file systems really should implement that.<br>
<br>
So my recommendation would be: work with the jffs2 folks to add this<br>
to the kernel. They already have it for PROT_READ afaik, doing it for<br>
PROT_WRITE is extra work, and not trivial but it will fix jffs2 for<br>
every program that uses mmap for writing, and journald is just one of<br>
them.<br>
<br>
> 3) Is this not considered as systemd bug to support compatibility for such<br>
> file system? Check first FS support writable mappings or not and if not<br>
> then use other method to enable Persistence logging for systemd?<br>
<br>
I think it's better to fix jffs2 in this regard than to add a<br>
userspace workaround to every single consumer of it.<br>
<br>
Doing this without mmap() is not trivial, and I don't see anyone<br>
stepping up to do the work. It wouldn't precisely make our codebase<br>
easier to read either...<br>
<br>
Lennart<br>
<br>
--<br>
Lennart Poettering, Berlin<br>
</blockquote></div>