From cae5da0611a52bd15dd12fab3a6726f49b25e037 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Okan=20T=C3=BCm=C3=BCkl=C3=BC?=
 <117488504+Okan-tumuklu@users.noreply.github.com>
Date: Sun, 12 May 2024 23:18:08 +0300
Subject: [PATCH] Update inode.c

/*
 * This function, adfs_get_block, is responsible for retrieving the block data
 * within an ADFS (Acorn Disk File System) inode. It checks whether the block
 * exists and maps it to a buffer head for further processing.
 *
 * Parameters:
 *   - inode: Pointer to the inode structure representing the file.
 *   - block: The block number to retrieve.
 *   - bh: Pointer to the buffer head structure to map the block data.
 *   - create: Boolean flag indicating whether block allocation is allowed.
 *
 * Return Values:
 *   - Returns 0 on success, indicating that the block data has been successfully mapped.
 *   - Returns -EFBIG if the provided block number exceeds the inode's block count, indicating an invalid block number.
 *   - Returns -EIO if there is an error during the block mapping process, indicating a mapping error.
 *   - Returns -EOPNOTSUPP if block allocation is not supported, indicating that block allocation is not yet implemented.
 */
---
 fs/adfs/inode.c | 33 +++++++++++++++++----------------
 1 file changed, 17 insertions(+), 16 deletions(-)

diff --git a/fs/adfs/inode.c b/fs/adfs/inode.c
index a183e213a4a5..b76bb97edcae 100644
--- a/fs/adfs/inode.c
+++ b/fs/adfs/inode.c
@@ -14,26 +14,27 @@
  * not support creation of new blocks, so we return -EIO for this case.
  */
 static int
-adfs_get_block(struct inode *inode, sector_t block, struct buffer_head *bh,
-	       int create)
-{
-	if (!create) {
-		if (block >= inode->i_blocks)
-			goto abort_toobig;
+adfs_get_block(struct inode *inode, sector_t block, struct buffer_head *bh, int create) {
+    // If create is false, check if the block number is valid
+    if (!create) {
+        if (block >= inode->i_blocks)
+            return -EFBIG; // The block number is too large, invalid.
 
-		block = __adfs_block_map(inode->i_sb, ADFS_I(inode)->indaddr,
-					 block);
-		if (block)
-			map_bh(bh, inode->i_sb, block);
-		return 0;
-	}
-	/* don't support allocation of blocks yet */
-	return -EIO;
+        // Map the block using the ADFS block mapping function
+        block = __adfs_block_map(inode->i_sb, ADFS_I(inode)->indaddr, block);
+        if (!block)
+            return -EIO; // Error in block mapping.
 
-abort_toobig:
-	return 0;
+        // Map the buffer head to the block
+        map_bh(bh, inode->i_sb, block);
+        return 0; // Success.
+    }
+
+    // Block allocation is not yet supported.
+    return -EOPNOTSUPP; 
 }
 
+
 static int adfs_writepages(struct address_space *mapping,
 		struct writeback_control *wbc)
 {