On Tue, 11 Sep 2018 15:34:44 -0700 Omar Sandoval <osandov@xxxxxxxxxxx> wrote: > From: Omar Sandoval <osandov@xxxxxx> > > The SWP_FILE flag serves two purposes: to make swap_{read,write}page() > go through the filesystem, and to make swapoff() call > ->swap_deactivate(). For Btrfs, we want the latter but not the former, > so split this flag into two. This makes us always call > ->swap_deactivate() if ->swap_activate() succeeded, not just if it > didn't add any swap extents itself. > > This also resolves the issue of the very misleading name of SWP_FILE, > which is only used for swap files over NFS. > Acked-by: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx>