• Qu Wenruo's avatar
    btrfs: raid56: switch recovery path to a single function · d817ce35
    Qu Wenruo authored
    Currently btrfs uses end_io functions to jump between different stages
    of recovery.
    
    For example, we go the following different functions:
    
    - raid56_bio_end_io()
      This handles the read for all the sectors (except the missing device).
    
    - __raid_recover_end_io()
      This does the real work, it's called inside the delayed work function
      raid_recover_end_io_work().
    
    This one recovery path involves at least 3 different functions, which is
    a big burden for readers.
    
    This patch will change the behavior by:
    
    - Introduce a unified recovery entrance, recover_rbio()
    
    - Use submit-and-wait method
      So the workflow is not interrupted by the endio function jump.
      This doesn't bring performance change, but reduce the burden for
      reviewers.
    
    - Run the main function in the rmw_workers workqueue
      Now raid56_parity_recover() only needs to setup the work, and
      queue the work using start_async_work().
    
    Now readers only need to do one function jump (start_async_work()) to
    find out the main entrance of recovery path.
    
    Furthermore, recover_rbio() function can easily be reused by other paths.
    
    The old recovery path is still utilized by degraded write path.
    It will be cleaned up when we have migrated the write path.
    Signed-off-by: default avatarQu Wenruo <wqu@suse.com>
    Signed-off-by: default avatarDavid Sterba <dsterba@suse.com>
    d817ce35
raid56.c 75.8 KB