From 371d777a7c8807891452f153caaef0a70dbd6e83 Mon Sep 17 00:00:00 2001
From: kamkow1 <kamkow256@gmail.com>
Date: Wed, 15 Oct 2025 20:10:00 +0200
Subject: [PATCH] vfs_delete() implement deleting files

---
 kernel/fs/portlfs/portlfs.c | 10 ++++++++++
 kernel/fs/portlfs/portlfs.h |  1 +
 kernel/vfs/vfs.c            | 23 +++++++++++++++++++----
 kernel/vfs/vfs.h            |  2 ++
 4 files changed, 32 insertions(+), 4 deletions(-)

diff --git a/kernel/fs/portlfs/portlfs.c b/kernel/fs/portlfs/portlfs.c
index aa87430..ec05a10 100644
--- a/kernel/fs/portlfs/portlfs.c
+++ b/kernel/fs/portlfs/portlfs.c
@@ -117,6 +117,16 @@ int32_t littlefs_mkdir(struct VfsMountPoint *vmp, const char *path) {
   return E_OK;
 }
 
+int32_t littlefs_delete(struct VfsMountPoint *vmp, const char *path) {
+  spinlock_acquire(&vmp->spinlock);
+  int ok = lfs_remove(&vmp->fs.littlefs.instance, path);
+  spinlock_release(&vmp->spinlock);
+  if (ok < 0) {
+    return E_BADIO;
+  }
+  return E_OK;
+}
+
 struct VfsObj *littlefs_open(struct VfsMountPoint *vmp, const char *path, uint32_t flags) {
   VfsObj *vobj = dlmalloc(sizeof(*vobj));
   if (vobj == NULL) {
diff --git a/kernel/fs/portlfs/portlfs.h b/kernel/fs/portlfs/portlfs.h
index 131051b..919b21b 100644
--- a/kernel/fs/portlfs/portlfs.h
+++ b/kernel/fs/portlfs/portlfs.h
@@ -20,6 +20,7 @@ struct VfsObj *littlefs_open(struct VfsMountPoint *vmp, const char *path, uint32
 int32_t littlefs_stat(struct VfsMountPoint *vmp, const char *path, FsStat *statbuf);
 int32_t littlefs_fetchdirent(struct VfsMountPoint *vmp, const char *path, FsDirent *direntbuf, size_t idx);
 int32_t littlefs_mkdir(struct VfsMountPoint *vmp, const char *path);
+int32_t littlefs_delete(struct VfsMountPoint *vmp, const char *path);
 
 int portlfs_read(const struct lfs_config *c, lfs_block_t block, lfs_off_t off, void *buffer, lfs_size_t size);
 int portlfs_prog(const struct lfs_config *c, lfs_block_t block, lfs_off_t off, const void *buffer, lfs_size_t size);
diff --git a/kernel/vfs/vfs.c b/kernel/vfs/vfs.c
index 7e6afb0..2ded261 100644
--- a/kernel/vfs/vfs.c
+++ b/kernel/vfs/vfs.c
@@ -44,11 +44,12 @@ void vfs_init_littlefs(VfsMountPoint *mp, bool format) {
     ERR("vfs", "Little FS mount failed %d\n", err);
   }
 
-  mp->cleanup = &littlefs_cleanup;
-  mp->open = &littlefs_open;
-  mp->stat = &littlefs_stat;
+  mp->cleanup     = &littlefs_cleanup;
+  mp->open        = &littlefs_open;
+  mp->stat        = &littlefs_stat;
   mp->fetchdirent = &littlefs_fetchdirent;
-  mp->mkdir = &littlefs_mkdir;
+  mp->mkdir       = &littlefs_mkdir;
+  mp->delete      = &littlefs_delete;
 }
 
 int32_t vfs_stat(char *mountpoint, const char *path, FsStat *stat) {
@@ -93,6 +94,20 @@ int32_t vfs_fetchdirent(char *mountpoint, const char *path, FsDirent *direntbuf,
   return mp->fetchdirent(mp, path, direntbuf, idx);
 }
 
+int32_t vfs_delete(char *mountpoint, const char *path) {
+  VfsMountPoint *mp = NULL;
+
+  spinlock_acquire(&VFS_TABLE.spinlock);
+  HSHTB_GET(VFS_TABLE.mountpoints, label, mountpoint, mp);
+  spinlock_release(&VFS_TABLE.spinlock);
+
+  if (mp == NULL) {
+    return E_NOENTRY;
+  }
+
+  return mp->delete(mp, path);
+}
+
 int32_t vfs_mount(char *mountpoint, int32_t fstype, StoreDev *backingsd, bool format) {
   VfsMountPoint *mp = NULL;
 
diff --git a/kernel/vfs/vfs.h b/kernel/vfs/vfs.h
index 91a3905..eaf1ee6 100644
--- a/kernel/vfs/vfs.h
+++ b/kernel/vfs/vfs.h
@@ -53,6 +53,7 @@ typedef struct VfsMountPoint {
   int32_t (*stat)(struct VfsMountPoint *vmp, const char *path, FsStat *statbuf);
   int32_t (*fetchdirent)(struct VfsMountPoint *vmp, const char *path, FsDirent *direntbuf, size_t idx);
   int32_t (*mkdir)(struct VfsMountPoint *vmp, const char *path);
+  int32_t (*delete)(struct VfsMountPoint *vmp, const char *path);
 
   union {
     LittleFs littlefs;
@@ -75,5 +76,6 @@ VfsObj *vfs_open(char *mountpoint, const char *path, uint32_t flags);
 int32_t vfs_stat(char *mountpoint, const char *path, FsStat *stat);
 int32_t vfs_fetchdirent(char *mountpoint, const char *path, FsDirent *direntbuf, size_t idx);
 int32_t vfs_mkdir(char *mountpoint, const char *path);
+int32_t vfs_delete(char *mountpoint, const char *path);
 
 #endif // VFS_VFS_H_