[PATCH v2 0/2] fs: jffs2: remove slab cache substitute with malloc

Holger Assmann h.assmann at pengutronix.de
Mon Nov 29 04:45:43 PST 2021


This series aimes to handle a bug in the Barebox jffs2 driver where the
system crashes once a second unmounting is performed when more than one
jffs2 file system were mounted at the same time at one point.

- The first patch "introduce reference counting at probe" solves the
  initial issue.

- The second patch "remove unnecessary slab cache structure" performs a
  cleanup on the respective code with the opportunity for simplification
  being taken.

Signed-off-by: Holger Assmann <h.assmann at pengutronix.de>

---

This series is a v2 of a former single commit now split up into two for
clarity reasons.

changes v1 -> v2:

   - split up v1 patch into two distinct patches
   - reworded commit messages

   by Ahmad Fatoum <a.fatoum at pengutronix.de>:
   - bugfix: perform jffs2_fill_super() with every probe() again
   - bugfix: removed unfunctional if-branch in probe()
   - use sizeof(*ret) instead of hardcoding the size in malloc calls

   by Sascha Hauer <s.hauer at pengutronix.de>:
   - additional infornation for commit messages added

---

 fs/jffs2/fs.c       |  32 +++++------
 fs/jffs2/malloc.c   | 131 +++++++-------------------------------------
 fs/jffs2/nodelist.h |   2 -
 3 files changed, 35 insertions(+), 130 deletions(-)

-- 
2.30.2




More information about the barebox mailing list