• Bob Peterson's avatar
    gfs2: don't lock sd_ail_lock in gfs2_releasepage · 249ffe18
    Bob Peterson authored
    Patch 380f7c65 changed gfs2_releasepage
    so that it held the sd_ail_lock spin_lock for most of its processing.
    It did this for some mysterious undocumented bug somewhere in the
    evict code path. But in the nine years since, evict has been reworked
    and fixed many times, and so have the transactions and ail list.
    I can't see a reason to hold the sd_ail_lock unless it's protecting
    the actual ail lists hung off the transactions. Therefore, this patch
    removes the locking to increase speed and efficiency, and to further help
    us rework the log flush code to be more concurrent with transactions.
    Signed-off-by: default avatarBob Peterson <rpeterso@redhat.com>
    Signed-off-by: default avatarAndreas Gruenbacher <agruenba@redhat.com>
    249ffe18
aops.c 20.5 KB