lkml.org 
[lkml]   [2012]   [Apr]   [13]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH v2 3/4] fat: mark superblock as dirty less often
    Date
    From: Artem Bityutskiy <artem.bityutskiy@linux.intel.com>

    This patch is a preparation for further changes. It touches few functions
    in fatent.c and prevents them from marking the superblock as dirty
    unnecessarily often. Namely, instead of marking it as dirty in the internal
    tight loops - do it only once at the end of the functions. And instead of
    marking it as dirty while holding the FAT table lock, do it outside the lock.

    The reason for this patch is that marking the superblock as dirty will soon
    become a little bit heavier operation, so it is cleaner to do this only when it
    is necessary.

    Signed-off-by: Artem Bityutskiy <artem.bityutskiy@linux.intel.com>
    ---
    fs/fat/fatent.c | 12 ++++++------
    1 files changed, 6 insertions(+), 6 deletions(-)

    diff --git a/fs/fat/fatent.c b/fs/fat/fatent.c
    index e49d274..85a1ec4 100644
    --- a/fs/fat/fatent.c
    +++ b/fs/fat/fatent.c
    @@ -503,7 +503,6 @@ int fat_alloc_clusters(struct inode *inode, int *cluster, int nr_cluster)
    sbi->prev_free = entry;
    if (sbi->free_clusters != -1)
    sbi->free_clusters--;
    - mark_fsinfo_dirty(sb);

    cluster[idx_clus] = entry;
    idx_clus++;
    @@ -525,11 +524,11 @@ int fat_alloc_clusters(struct inode *inode, int *cluster, int nr_cluster)
    /* Couldn't allocate the free entries */
    sbi->free_clusters = 0;
    sbi->free_clus_valid = 1;
    - mark_fsinfo_dirty(sb);
    err = -ENOSPC;

    out:
    unlock_fat(sbi);
    + mark_fsinfo_dirty(sb);
    fatent_brelse(&fatent);
    if (!err) {
    if (inode_needs_sync(inode))
    @@ -590,10 +589,8 @@ int fat_free_clusters(struct inode *inode, int cluster)
    }

    ops->ent_put(&fatent, FAT_ENT_FREE);
    - if (sbi->free_clusters != -1) {
    + if (sbi->free_clusters != -1)
    sbi->free_clusters++;
    - mark_fsinfo_dirty(sb);
    - }

    if (nr_bhs + fatent.nr_bhs > MAX_BUF_PER_PAGE) {
    if (sb->s_flags & MS_SYNCHRONOUS) {
    @@ -622,6 +619,8 @@ error:
    for (i = 0; i < nr_bhs; i++)
    brelse(bhs[i]);
    unlock_fat(sbi);
    + if (!err)
    + mark_fsinfo_dirty(sb);

    return err;
    }
    @@ -682,9 +681,10 @@ int fat_count_free_clusters(struct super_block *sb)
    }
    sbi->free_clusters = free;
    sbi->free_clus_valid = 1;
    - mark_fsinfo_dirty(sb);
    fatent_brelse(&fatent);
    out:
    unlock_fat(sbi);
    + if (!err)
    + mark_fsinfo_dirty(sb);
    return err;
    }
    --
    1.7.7.6


    \
     
     \ /
      Last update: 2012-04-13 16:21    [W:0.025 / U:31.320 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site