• Steven Whitehouse's avatar
    [GFS2] Clean up gfs2_trans_add_revoke() · 1ad38c43
    Steven Whitehouse authored
    The following alters gfs2_trans_add_revoke() to take a struct
    gfs2_bufdata as an argument. This eliminates the memory allocation which
    was previously required by making use of the already existing struct
    gfs2_bufdata. It makes some sanity checks to ensure that the
    gfs2_bufdata has been removed from all the lists before its recycled as
    a revoke structure. This saves one memory allocation and one free per
    revoke structure.
    Also as a result, and to simplify the locking, since there is no longer
    any blocking code in gfs2_trans_add_revoke() we must hold the log lock
    whenever this function is called. This reduces the amount of times we
    take and unlock the log lock.
    Signed-off-by: default avatarSteven Whitehouse <swhiteho@redhat.com>
trans.h 1.12 KB