Re: [f2fs-dev] SMR drive test 2; 128GB partition; no obvious corruption, much more sane behaviour,
Brought to you by:
kjgkr
|
From: Jaegeuk K. <ja...@ke...> - 2015-09-24 17:27:58
|
On Thu, Sep 24, 2015 at 01:39:38AM +0200, Marc Lehmann wrote: > On Wed, Sep 23, 2015 at 03:08:23PM -0700, Jaegeuk Kim <ja...@ke...> wrote: > > > root@shag:/sys/fs/f2fs/dm-1# df -H /mnt > > > Filesystem Size Used Avail Use% Mounted on > > > /dev/mapper/vg_test-test 138G 137G 803k 100% /mnt > > > > Could you please share /sys/kernel/debug/f2fs/status? > > Uh, sorry, I planned to, but forgot, probably because I thought the result > was so good it didn't need any checking :) > > > So, I'm convinced that your inital test set "-o1 -s128", which was an unlucky > > trial. :) > > hmm... since the point is to simulate a full 8TB partition, having large > overprovision/reserved space AND large section size might actually have been > a good test, as it would simulate the TB case better, which would also have > larger overprovisioning and the larger section size. > > In the end, I might settle with -s64, and currently do tests with -s90. Got it. But why -s90? :) > I was just scared that overprovisioning might turn out ot be extremely large > with 8TB. > > I have since then dropped -o from all my mkfs.f2fs invocations, seeing > that the resulting filesystem does not actually have 5% overprovisioning. > > > Subject: [PATCH] mkfs.f2fs: fix wrong ovp space calculation on large section > > Hmm, the latest change in > git://git.kernel.org/pub/scm/linux/kernel/git/jaegeuk/f2fs-tools.git is from > august 10 - do I need to select a branch (I am not good with git)? I just pushed the patches to master branch in f2fs-tools.git. Could you pull them and check them? I added one more patch to avoid harmless sit_type fixes previously you reported. And, for the 8TB case, let me check again. It seems that we need to handle under 1% overprovision ratio. (e.g., 0.5%) Thanks, > > -- > The choice of a Deliantra, the free code+content MORPG > -----==- _GNU_ http://www.deliantra.net > ----==-- _ generation > ---==---(_)__ __ ____ __ Marc Lehmann > --==---/ / _ \/ // /\ \/ / sc...@sc... > -=====/_/_//_/\_,_/ /_/\_\ |