On Mon, Oct 27, 2014 at 6:33 PM, Hugo Mills <h...@carfax.org.uk> wrote: > On Mon, Oct 27, 2014 at 11:21:13AM -0700, Christian Kujau wrote: >> On Mon, 27 Oct 2014 at 16:35, David Sterba wrote: >> > Yeah sorry, I sent the v2 too late, here's an incremental that applies >> > on top of current 3.18-rc >> > >> > https://patchwork.kernel.org/patch/5160651/ >> >> Yup, that fixes it. Thank you! If it's needed: >> >> Tested-by: Christian Kujau <li...@nerdbynature.de> >> >> @Filipe: and thanks for warning me about 3.17 - I used 3.17.0 since it >> came out and compiled kernels on the btrfs partition and haven't had any >> issues. But it wasn't used very often, so whatever the serious issues >> were, I haven't experienced any. > > If you make read-only snapshots, there's a good chance of metadata > corruption. It's fixed in 3.17.2.
Well not only because of that issue, but also because of file extent map corruption, which manifests itself in several different ways, such as EEXIST errors on file write [1] or fiemap [2] calls for example, and likely file metadata corruption after fsync + log recovery. This is fixed in 3.17.2 too. [1] http://www.spinics.net/lists/linux-btrfs/msg37567.html [2] http://www.spinics.net/lists/linux-btrfs/msg38045.html > > Hugo. > > -- > === Hugo Mills: hugo@... carfax.org.uk | darksatanic.net | lug.org.uk === > PGP key: 65E74AC0 from wwwkeys.eu.pgp.net or http://www.carfax.org.uk > --- Our so-called leaders speak/with words they try to jail ya/ --- > They subjugate the meek/but it's the rhetoric of failure. > -- Filipe David Manana, "Reasonable men adapt themselves to the world. Unreasonable men adapt the world to themselves. That's why all progress depends on unreasonable men." -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
