grub-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[PATCH] grub-core/disk: Replace transform_sector function


From: Mukesh Kumar Chaurasiya
Subject: [PATCH] grub-core/disk: Replace transform_sector function
Date: Mon, 3 Apr 2023 15:39:18 +0530

transform_sector function is not very clear in what it's doing
and confusing, grub already has a function which is doing the same 
thing in a very self explanatory way. i.e. grub_disk_to_native_sector
So it's much better to use self explanatory one than transfor_sector

Signed-off-by: Mukesh Kumar Chaurasiya <mchauras@linux.vnet.ibm.com>
---
 grub-core/kern/disk.c        | 7 ++++---
 grub-core/kern/disk_common.c | 6 ------
 grub-core/lib/disk.c         | 6 ++++--
 3 files changed, 8 insertions(+), 11 deletions(-)

diff --git a/grub-core/kern/disk.c b/grub-core/kern/disk.c
index 01190085e..e20117d5f 100644
--- a/grub-core/kern/disk.c
+++ b/grub-core/kern/disk.c
@@ -341,7 +341,7 @@ grub_disk_read_small_real (grub_disk_t disk, 
grub_disk_addr_t sector,
       < (disk->total_sectors << (disk->log_sector_size - 
GRUB_DISK_SECTOR_BITS)))
     {
       grub_err_t err;
-      err = (disk->dev->disk_read) (disk, transform_sector (disk, sector),
+      err = (disk->dev->disk_read) (disk, grub_disk_to_native_sector (disk, 
sector),
                                    1U << (GRUB_DISK_CACHE_BITS
                                           + GRUB_DISK_SECTOR_BITS
                                           - disk->log_sector_size), tmp_buf);
@@ -377,7 +377,8 @@ grub_disk_read_small_real (grub_disk_t disk, 
grub_disk_addr_t sector,
     if (!tmp_buf)
       return grub_errno;
 
-    if ((disk->dev->disk_read) (disk, transform_sector (disk, aligned_sector),
+    if ((disk->dev->disk_read) (disk,
+                               grub_disk_to_native_sector (disk, 
aligned_sector),
                                num, tmp_buf))
       {
        grub_error_push ();
@@ -483,7 +484,7 @@ grub_disk_read (grub_disk_t disk, grub_disk_addr_t sector,
        {
          grub_disk_addr_t i;
 
-         err = (disk->dev->disk_read) (disk, transform_sector (disk, sector),
+         err = (disk->dev->disk_read) (disk, grub_disk_to_native_sector (disk, 
sector),
                                        agglomerate << (GRUB_DISK_CACHE_BITS
                                                        + GRUB_DISK_SECTOR_BITS
                                                        - 
disk->log_sector_size),
diff --git a/grub-core/kern/disk_common.c b/grub-core/kern/disk_common.c
index e09fba8db..ab6cd0c2b 100644
--- a/grub-core/kern/disk_common.c
+++ b/grub-core/kern/disk_common.c
@@ -50,12 +50,6 @@ grub_disk_adjust_range (grub_disk_t disk, grub_disk_addr_t 
*sector,
   return GRUB_ERR_NONE;
 }
 
-static inline grub_disk_addr_t
-transform_sector (grub_disk_t disk, grub_disk_addr_t sector)
-{
-  return sector >> (disk->log_sector_size - GRUB_DISK_SECTOR_BITS);
-}
-
 static unsigned
 grub_disk_cache_get_index (unsigned long dev_id, unsigned long disk_id,
                           grub_disk_addr_t sector)
diff --git a/grub-core/lib/disk.c b/grub-core/lib/disk.c
index b4eb064a8..a3321d5f2 100644
--- a/grub-core/lib/disk.c
+++ b/grub-core/lib/disk.c
@@ -101,7 +101,8 @@ grub_disk_write (grub_disk_t disk, grub_disk_addr_t sector,
 
          grub_disk_cache_invalidate (disk->dev->id, disk->id, sector);
 
-         if ((disk->dev->disk_write) (disk, transform_sector (disk, sector),
+         if ((disk->dev->disk_write) (disk,
+                                      grub_disk_to_native_sector (disk, 
sector),
                                       1, tmp_buf) != GRUB_ERR_NONE)
            {
              grub_free (tmp_buf);
@@ -130,7 +131,8 @@ grub_disk_write (grub_disk_t disk, grub_disk_addr_t sector,
                 << (GRUB_DISK_CACHE_BITS + GRUB_DISK_SECTOR_BITS
                     - disk->log_sector_size));
 
-         if ((disk->dev->disk_write) (disk, transform_sector (disk, sector),
+         if ((disk->dev->disk_write) (disk,
+                                      grub_disk_to_native_sector (disk, 
sector),
                                       n, buf) != GRUB_ERR_NONE)
            goto finish;
 
-- 
2.31.1




reply via email to

[Prev in Thread] Current Thread [Next in Thread]