• Wei Wang's avatar
    net: introduce DST_NOGC in dst_release() to destroy dst based on refcnt · 5f56f409
    Wei Wang authored
    The current mechanism of freeing dst is a bit complicated. dst has its
    ref count and when user grabs the reference to the dst, the ref count is
    properly taken in most cases except in IPv4/IPv6/decnet/xfrm routing
    code due to some historic reasons.
    
    If the reference to dst is always taken properly, we should be able to
    simplify the logic in dst_release() to destroy dst when dst->__refcnt
    drops from 1 to 0. And this should be the only condition to determine
    if we can call dst_destroy().
    And as dst is always ref counted, there is no need for a dst garbage
    list to hold the dst entries that already get removed by the routing
    code but are still held by other users. And the task to periodically
    check the list to free dst if ref count become 0 is also not needed
    anymore.
    
    This patch introduces a temporary flag DST_NOGC(no garbage collector).
    If it is set in the dst, dst_release() will call dst_destroy() when
    dst->__refcnt drops to 0. dst_hold_safe() will also check for this flag
    and do atomic_inc_not_zero() similar as DST_NOCACHE to avoid double free
    issue.
    This temporary flag is mainly used so that we can make the transition
    component by component without breaking other parts.
    This flag will be removed after all components are properly transitioned.
    
    This patch also introduces a new function dst_release_immediate() which
    destroys dst without waiting on the rcu when refcnt drops to 0. It will
    be used in later patches.
    
    Follow-up patches will correct all the places to properly take ref count
    on dst and mark DST_NOGC. dst_release() or dst_release_immediate() will
    be used to release the dst instead of dst_free() and its related
    functions.
    And final clean-up patch will remove the DST_NOGC flag.
    Signed-off-by: default avatarWei Wang <weiwan@google.com>
    Acked-by: default avatarMartin KaFai Lau <kafai@fb.com>
    Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
    5f56f409
dst.h 13.4 KB