On Sun, Jan 19, 2020 at 02:02:52PM -0800, Christoph Hellwig wrote: > On Sat, Jan 18, 2020 at 05:38:51PM +0800, Murphy Zhou wrote: > > > But do those tests require filefrag -B?? > > > > Not really. But filefrag will fall back to use fibmap. > > But that means it requires either FIEMAP or FIBMAP, and your patch > will cause regressions for file systems only implementing FIEMAP. > So the fix isn't to ask for FIBMAP, but to implement FIEMAP in > any file system missing that support. Hmm, you are right :-)