* Re: [f2fs-dev][PATCH] resize.f2fs: add option for large_nat_bitmap feature [not found] <1578482416650.67283@xiaomi.com> @ 2020-01-09 7:14 ` Chao Yu 2020-01-10 9:58 ` [f2fs-dev] [PATCH] " xiong ping 0 siblings, 1 reply; 5+ messages in thread From: Chao Yu @ 2020-01-09 7:14 UTC (permalink / raw) To: Ping1 Xiong 熊平, linux-f2fs-devel, jaegeuk; +Cc: linux-kernel On 2020/1/8 19:20, Ping1 Xiong 熊平 wrote: > From d5b8411dbae37180c37d96bf164fab16138fc21a Mon Sep 17 00:00:00 2001 > > From: xiongping1 <xiongping1@xiaomi.com> > Date: Wed, 8 Jan 2020 17:20:55 +0800 > Subject: [PATCH] resize.f2fs: add option for large_nat_bitmap feature Thanks for your contribution. The patch format is incorrect, I guess it was changed by email client or when you paste patch's content, could you check it? > > resize.f2fs has already supported large_nat_bitmap feature, but has no > option to turn on it. > > This change add a new '-i' option to control turning on/off it. We only add a option to turn on this feature, right? rather than turning on or off it? Thanks, > > Signed-off-by: xiongping1 <xiongping1@xiaomi.com> > --- > fsck/main.c | 6 +++++- > fsck/resize.c | 5 +++++ > 2 files changed, 10 insertions(+), 1 deletion(-) > > diff --git a/fsck/main.c b/fsck/main.c > index 9a7d499..e7e3dfc 100644 > --- a/fsck/main.c > +++ b/fsck/main.c > @@ -104,6 +104,7 @@ void resize_usage() > MSG(0, "\nUsage: resize.f2fs [options] device\n"); > MSG(0, "[options]:\n"); > MSG(0, " -d debug level [default:0]\n"); > + MSG(0, " -i extended node bitmap, node ratio is 20%% by default\n"); > MSG(0, " -s safe resize (Does not resize metadata)"); > MSG(0, " -t target sectors [default: device size]\n"); > MSG(0, " -V print the version number and exit\n"); > @@ -449,7 +450,7 @@ void f2fs_parse_options(int argc, char *argv[]) > break; > } > } else if (!strcmp("resize.f2fs", prog)) { > - const char *option_string = "d:st:V"; > + const char *option_string = "d:st:iV"; > > c.func = RESIZE; > while ((option = getopt(argc, argv, option_string)) != EOF) { > @@ -476,6 +477,9 @@ void f2fs_parse_options(int argc, char *argv[]) > ret = sscanf(optarg, "%"PRIx64"", > &c.target_sectors); > break; > + case 'i': > + c.large_nat_bitmap = 1; > + break; > case 'V': > show_version(prog); > exit(0); > diff --git a/fsck/resize.c b/fsck/resize.c > index fc563f2..88e063e 100644 > --- a/fsck/resize.c > +++ b/fsck/resize.c > @@ -519,6 +519,11 @@ static void rebuild_checkpoint(struct f2fs_sb_info *sbi, > else > set_cp(checksum_offset, CP_CHKSUM_OFFSET); > > + if (c.large_nat_bitmap) { > + set_cp(checksum_offset, CP_MIN_CHKSUM_OFFSET); > + flags |= CP_LARGE_NAT_BITMAP_FLAG; > + } > + > set_cp(ckpt_flags, flags); > > memcpy(new_cp, cp, (unsigned char *)cp->sit_nat_version_bitmap - > -- > 2.7.4 > > > > > #/******本邮件及其附件含有小米公司的保密信息，仅限于发送给上面地址中列出的个人或群组。禁止任何其他人以任何形式使用（包括但不限于全部或部分地泄露、复制、或散发）本邮件中的信息。如果您错收了本邮件，请您立即电话或邮件通知发件人并删除本邮件！ This e-mail and its attachments contain confidential information from XIAOMI, which is intended only for the person or entity whose address is listed above. Any use of the information contained herein in any way (including, but not limited to, total or partial disclosure, reproduction, or dissemination) by persons other than the intended recipient(s) is prohibited. If you receive this e-mail in error, please notify the sender by phone or email immediately and delete it!******/# ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [f2fs-dev] [PATCH] resize.f2fs: add option for large_nat_bitmap feature 2020-01-09 7:14 ` [f2fs-dev][PATCH] resize.f2fs: add option for large_nat_bitmap feature Chao Yu @ 2020-01-10 9:58 ` xiong ping 2020-01-11 9:43 ` Chao Yu 0 siblings, 1 reply; 5+ messages in thread From: xiong ping @ 2020-01-10 9:58 UTC (permalink / raw) To: Chao Yu Cc: Ping1 Xiong 熊平, linux-f2fs-devel, jaegeuk, linux-kernel Chao Yu <yuchao0@huawei.com> 于2020年1月9日周四 下午3:16写道： > > On 2020/1/8 19:20, Ping1 Xiong 熊平 wrote: > > From d5b8411dbae37180c37d96bf164fab16138fc21a Mon Sep 17 00:00:00 2001 > > > > From: xiongping1 <xiongping1@xiaomi.com> > > Date: Wed, 8 Jan 2020 17:20:55 +0800 > > Subject: [PATCH] resize.f2fs: add option for large_nat_bitmap feature > > Thanks for your contribution. > > The patch format is incorrect, I guess it was changed by email client or when > you paste patch's content, could you check it? > I have resend the patch with this email account yesterday, can you check it? > > > > resize.f2fs has already supported large_nat_bitmap feature, but has no > > option to turn on it. > > > > This change add a new '-i' option to control turning on/off it. > > We only add a option to turn on this feature, right? rather than turning > on or off it? > yes, the feature is off by default, so we need an option to enable it. > Thanks, > > > > > Signed-off-by: xiongping1 <xiongping1@xiaomi.com> > > --- > > fsck/main.c | 6 +++++- > > fsck/resize.c | 5 +++++ > > 2 files changed, 10 insertions(+), 1 deletion(-) > > > > diff --git a/fsck/main.c b/fsck/main.c > > index 9a7d499..e7e3dfc 100644 > > --- a/fsck/main.c > > +++ b/fsck/main.c > > @@ -104,6 +104,7 @@ void resize_usage() > > MSG(0, "\nUsage: resize.f2fs [options] device\n"); > > MSG(0, "[options]:\n"); > > MSG(0, " -d debug level [default:0]\n"); > > + MSG(0, " -i extended node bitmap, node ratio is 20%% by default\n"); > > MSG(0, " -s safe resize (Does not resize metadata)"); > > MSG(0, " -t target sectors [default: device size]\n"); > > MSG(0, " -V print the version number and exit\n"); > > @@ -449,7 +450,7 @@ void f2fs_parse_options(int argc, char *argv[]) > > break; > > } > > } else if (!strcmp("resize.f2fs", prog)) { > > - const char *option_string = "d:st:V"; > > + const char *option_string = "d:st:iV"; > > > > c.func = RESIZE; > > while ((option = getopt(argc, argv, option_string)) != EOF) { > > @@ -476,6 +477,9 @@ void f2fs_parse_options(int argc, char *argv[]) > > ret = sscanf(optarg, "%"PRIx64"", > > &c.target_sectors); > > break; > > + case 'i': > > + c.large_nat_bitmap = 1; > > + break; > > case 'V': > > show_version(prog); > > exit(0); > > diff --git a/fsck/resize.c b/fsck/resize.c > > index fc563f2..88e063e 100644 > > --- a/fsck/resize.c > > +++ b/fsck/resize.c > > @@ -519,6 +519,11 @@ static void rebuild_checkpoint(struct f2fs_sb_info *sbi, > > else > > set_cp(checksum_offset, CP_CHKSUM_OFFSET); > > > > + if (c.large_nat_bitmap) { > > + set_cp(checksum_offset, CP_MIN_CHKSUM_OFFSET); > > + flags |= CP_LARGE_NAT_BITMAP_FLAG; > > + } > > + > > set_cp(ckpt_flags, flags); > > > > memcpy(new_cp, cp, (unsigned char *)cp->sit_nat_version_bitmap - > > -- > > 2.7.4 > > > > > > > > > > #/******本邮件及其附件含有小米公司的保密信息，仅限于发送给上面地址中列出的个人或群组。禁止任何其他人以任何形式使用（包括但不限于全部或部分地泄露、复制、或散发）本邮件中的信息。如果您错收了本邮件，请您立即电话或邮件通知发件人并删除本邮件！ This e-mail and its attachments contain confidential information from XIAOMI, which is intended only for the person or entity whose address is listed above. Any use of the information contained herein in any way (including, but not limited to, total or partial disclosure, reproduction, or dissemination) by persons other than the intended recipient(s) is prohibited. If you receive this e-mail in error, please notify the sender by phone or email immediately and delete it!******/# > > > _______________________________________________ > Linux-f2fs-devel mailing list > Linux-f2fs-devel@lists.sourceforge.net > https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [f2fs-dev] [PATCH] resize.f2fs: add option for large_nat_bitmap feature 2020-01-10 9:58 ` [f2fs-dev] [PATCH] " xiong ping @ 2020-01-11 9:43 ` Chao Yu 2020-01-13 6:50 ` xiong ping 0 siblings, 1 reply; 5+ messages in thread From: Chao Yu @ 2020-01-11 9:43 UTC (permalink / raw) To: xiong ping Cc: Ping1 Xiong 熊平, linux-f2fs-devel, jaegeuk, linux-kernel On 2020/1/10 17:58, xiong ping wrote: > Chao Yu <yuchao0@huawei.com> 于2020年1月9日周四 下午3:16写道： >> >> On 2020/1/8 19:20, Ping1 Xiong 熊平 wrote: >>> From d5b8411dbae37180c37d96bf164fab16138fc21a Mon Sep 17 00:00:00 2001 >>> >>> From: xiongping1 <xiongping1@xiaomi.com> >>> Date: Wed, 8 Jan 2020 17:20:55 +0800 >>> Subject: [PATCH] resize.f2fs: add option for large_nat_bitmap feature >> >> Thanks for your contribution. >> >> The patch format is incorrect, I guess it was changed by email client or when >> you paste patch's content, could you check it? >> > I have resend the patch with this email account yesterday, can you check it? >>> >>> resize.f2fs has already supported large_nat_bitmap feature, but has no >>> option to turn on it. >>> >>> This change add a new '-i' option to control turning on/off it. >> >> We only add a option to turn on this feature, right? rather than turning >> on or off it? >> > yes, the feature is off by default, so we need an option to enable it. So, I meant it needs to adjust commit message from "turning on/off it" to "turning on it". >> Thanks, >> >>> >>> Signed-off-by: xiongping1 <xiongping1@xiaomi.com> >>> --- >>> fsck/main.c | 6 +++++- >>> fsck/resize.c | 5 +++++ >>> 2 files changed, 10 insertions(+), 1 deletion(-) >>> >>> diff --git a/fsck/main.c b/fsck/main.c >>> index 9a7d499..e7e3dfc 100644 >>> --- a/fsck/main.c >>> +++ b/fsck/main.c >>> @@ -104,6 +104,7 @@ void resize_usage() >>> MSG(0, "\nUsage: resize.f2fs [options] device\n"); >>> MSG(0, "[options]:\n"); >>> MSG(0, " -d debug level [default:0]\n"); >>> + MSG(0, " -i extended node bitmap, node ratio is 20%% by default\n"); >>> MSG(0, " -s safe resize (Does not resize metadata)"); >>> MSG(0, " -t target sectors [default: device size]\n"); >>> MSG(0, " -V print the version number and exit\n"); >>> @@ -449,7 +450,7 @@ void f2fs_parse_options(int argc, char *argv[]) >>> break; >>> } >>> } else if (!strcmp("resize.f2fs", prog)) { >>> - const char *option_string = "d:st:V"; >>> + const char *option_string = "d:st:iV"; >>> >>> c.func = RESIZE; >>> while ((option = getopt(argc, argv, option_string)) != EOF) { >>> @@ -476,6 +477,9 @@ void f2fs_parse_options(int argc, char *argv[]) >>> ret = sscanf(optarg, "%"PRIx64"", >>> &c.target_sectors); >>> break; >>> + case 'i': >>> + c.large_nat_bitmap = 1; >>> + break; >>> case 'V': >>> show_version(prog); >>> exit(0); >>> diff --git a/fsck/resize.c b/fsck/resize.c >>> index fc563f2..88e063e 100644 >>> --- a/fsck/resize.c >>> +++ b/fsck/resize.c >>> @@ -519,6 +519,11 @@ static void rebuild_checkpoint(struct f2fs_sb_info *sbi, >>> else >>> set_cp(checksum_offset, CP_CHKSUM_OFFSET); >>> >>> + if (c.large_nat_bitmap) { >>> + set_cp(checksum_offset, CP_MIN_CHKSUM_OFFSET); >>> + flags |= CP_LARGE_NAT_BITMAP_FLAG; >>> + } How about relocating above codes to below position: flags = update_nat_bits_flags(new_sb, cp, get_cp(ckpt_flags)); + if (c.large_nat_bitmap) + flags |= CP_LARGE_NAT_BITMAP_FLAG; if (flags & CP_COMPACT_SUM_FLAG) flags &= ~CP_COMPACT_SUM_FLAG; Thanks, >>> + >>> set_cp(ckpt_flags, flags); >>> >>> memcpy(new_cp, cp, (unsigned char *)cp->sit_nat_version_bitmap - >>> -- >>> 2.7.4 >>> >>> >>> >>> >>> #/******本邮件及其附件含有小米公司的保密信息，仅限于发送给上面地址中列出的个人或群组。禁止任何其他人以任何形式使用（包括但不限于全部或部分地泄露、复制、或散发）本邮件中的信息。如果您错收了本邮件，请您立即电话或邮件通知发件人并删除本邮件！ This e-mail and its attachments contain confidential information from XIAOMI, which is intended only for the person or entity whose address is listed above. Any use of the information contained herein in any way (including, but not limited to, total or partial disclosure, reproduction, or dissemination) by persons other than the intended recipient(s) is prohibited. If you receive this e-mail in error, please notify the sender by phone or email immediately and delete it!******/# >> >> >> _______________________________________________ >> Linux-f2fs-devel mailing list >> Linux-f2fs-devel@lists.sourceforge.net >> https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel > . > ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [f2fs-dev] [PATCH] resize.f2fs: add option for large_nat_bitmap feature 2020-01-11 9:43 ` Chao Yu @ 2020-01-13 6:50 ` xiong ping 0 siblings, 0 replies; 5+ messages in thread From: xiong ping @ 2020-01-13 6:50 UTC (permalink / raw) To: Chao Yu Cc: Ping1 Xiong 熊平, linux-f2fs-devel, jaegeuk, linux-kernel Chao Yu <yuchao0@huawei.com> 于2020年1月11日周六 下午5:43写道： > > On 2020/1/10 17:58, xiong ping wrote: > > Chao Yu <yuchao0@huawei.com> 于2020年1月9日周四 下午3:16写道： > >> > >> On 2020/1/8 19:20, Ping1 Xiong 熊平 wrote: > >>> From d5b8411dbae37180c37d96bf164fab16138fc21a Mon Sep 17 00:00:00 2001 > >>> > >>> From: xiongping1 <xiongping1@xiaomi.com> > >>> Date: Wed, 8 Jan 2020 17:20:55 +0800 > >>> Subject: [PATCH] resize.f2fs: add option for large_nat_bitmap feature > >> > >> Thanks for your contribution. > >> > >> The patch format is incorrect, I guess it was changed by email client or when > >> you paste patch's content, could you check it? > >> > > I have resend the patch with this email account yesterday, can you check it? > >>> > >>> resize.f2fs has already supported large_nat_bitmap feature, but has no > >>> option to turn on it. > >>> > >>> This change add a new '-i' option to control turning on/off it. > >> > >> We only add a option to turn on this feature, right? rather than turning > >> on or off it? > >> > > yes, the feature is off by default, so we need an option to enable it. > > So, I meant it needs to adjust commit message from "turning on/off it" to > "turning on it". > ok, will fix it in patch v2. > >> Thanks, > >> > >>> > >>> Signed-off-by: xiongping1 <xiongping1@xiaomi.com> > >>> --- > >>> fsck/main.c | 6 +++++- > >>> fsck/resize.c | 5 +++++ > >>> 2 files changed, 10 insertions(+), 1 deletion(-) > >>> > >>> diff --git a/fsck/main.c b/fsck/main.c > >>> index 9a7d499..e7e3dfc 100644 > >>> --- a/fsck/main.c > >>> +++ b/fsck/main.c > >>> @@ -104,6 +104,7 @@ void resize_usage() > >>> MSG(0, "\nUsage: resize.f2fs [options] device\n"); > >>> MSG(0, "[options]:\n"); > >>> MSG(0, " -d debug level [default:0]\n"); > >>> + MSG(0, " -i extended node bitmap, node ratio is 20%% by default\n"); > >>> MSG(0, " -s safe resize (Does not resize metadata)"); > >>> MSG(0, " -t target sectors [default: device size]\n"); > >>> MSG(0, " -V print the version number and exit\n"); > >>> @@ -449,7 +450,7 @@ void f2fs_parse_options(int argc, char *argv[]) > >>> break; > >>> } > >>> } else if (!strcmp("resize.f2fs", prog)) { > >>> - const char *option_string = "d:st:V"; > >>> + const char *option_string = "d:st:iV"; > >>> > >>> c.func = RESIZE; > >>> while ((option = getopt(argc, argv, option_string)) != EOF) { > >>> @@ -476,6 +477,9 @@ void f2fs_parse_options(int argc, char *argv[]) > >>> ret = sscanf(optarg, "%"PRIx64"", > >>> &c.target_sectors); > >>> break; > >>> + case 'i': > >>> + c.large_nat_bitmap = 1; > >>> + break; > >>> case 'V': > >>> show_version(prog); > >>> exit(0); > >>> diff --git a/fsck/resize.c b/fsck/resize.c > >>> index fc563f2..88e063e 100644 > >>> --- a/fsck/resize.c > >>> +++ b/fsck/resize.c > >>> @@ -519,6 +519,11 @@ static void rebuild_checkpoint(struct f2fs_sb_info *sbi, > >>> else > >>> set_cp(checksum_offset, CP_CHKSUM_OFFSET); > >>> > >>> + if (c.large_nat_bitmap) { > >>> + set_cp(checksum_offset, CP_MIN_CHKSUM_OFFSET); > >>> + flags |= CP_LARGE_NAT_BITMAP_FLAG; > >>> + } > > How about relocating above codes to below position: > > flags = update_nat_bits_flags(new_sb, cp, get_cp(ckpt_flags)); > > + if (c.large_nat_bitmap) > + flags |= CP_LARGE_NAT_BITMAP_FLAG; > ok, will fix it in patch v2. > if (flags & CP_COMPACT_SUM_FLAG) > flags &= ~CP_COMPACT_SUM_FLAG; > > Thanks, > > >>> + > >>> set_cp(ckpt_flags, flags); > >>> > >>> memcpy(new_cp, cp, (unsigned char *)cp->sit_nat_version_bitmap - > >>> -- > >>> 2.7.4 > >>> > >>> > >>> > >>> > >>> #/******本邮件及其附件含有小米公司的保密信息，仅限于发送给上面地址中列出的个人或群组。禁止任何其他人以任何形式使用（包括但不限于全部或部分地泄露、复制、或散发）本邮件中的信息。如果您错收了本邮件，请您立即电话或邮件通知发件人并删除本邮件！ This e-mail and its attachments contain confidential information from XIAOMI, which is intended only for the person or entity whose address is listed above. Any use of the information contained herein in any way (including, but not limited to, total or partial disclosure, reproduction, or dissemination) by persons other than the intended recipient(s) is prohibited. If you receive this e-mail in error, please notify the sender by phone or email immediately and delete it!******/# > >> > >> > >> _______________________________________________ > >> Linux-f2fs-devel mailing list > >> Linux-f2fs-devel@lists.sourceforge.net > >> https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel > > . > > ^ permalink raw reply [flat|nested] 5+ messages in thread
* [f2fs-dev][PATCH] resize.f2fs: add option for large_nat_bitmap feature @ 2020-01-09 9:32 ping xiong 0 siblings, 0 replies; 5+ messages in thread From: ping xiong @ 2020-01-09 9:32 UTC (permalink / raw) To: yuchao0; +Cc: jaegeuk, linux-f2fs-devel, linux-kernel, xiongping1 From: xiongping1 <xiongping1@xiaomi.com> resize.f2fs has already supported large_nat_bitmap feature, but has no option to turn on it. This change add a new '-i' option to control turning on/off it. Signed-off-by: xiongping1 <xiongping1@xiaomi.com> --- fsck/main.c | 6 +++++- fsck/resize.c | 5 +++++ 2 files changed, 10 insertions(+), 1 deletion(-) diff --git a/fsck/main.c b/fsck/main.c index 9a7d499..e7e3dfc 100644 --- a/fsck/main.c +++ b/fsck/main.c @@ -104,6 +104,7 @@ void resize_usage() MSG(0, "\nUsage: resize.f2fs [options] device\n"); MSG(0, "[options]:\n"); MSG(0, " -d debug level [default:0]\n"); + MSG(0, " -i extended node bitmap, node ratio is 20%% by default\n"); MSG(0, " -s safe resize (Does not resize metadata)"); MSG(0, " -t target sectors [default: device size]\n"); MSG(0, " -V print the version number and exit\n"); @@ -449,7 +450,7 @@ void f2fs_parse_options(int argc, char *argv[]) break; } } else if (!strcmp("resize.f2fs", prog)) { - const char *option_string = "d:st:V"; + const char *option_string = "d:st:iV"; c.func = RESIZE; while ((option = getopt(argc, argv, option_string)) != EOF) { @@ -476,6 +477,9 @@ void f2fs_parse_options(int argc, char *argv[]) ret = sscanf(optarg, "%"PRIx64"", &c.target_sectors); break; + case 'i': + c.large_nat_bitmap = 1; + break; case 'V': show_version(prog); exit(0); diff --git a/fsck/resize.c b/fsck/resize.c index fc563f2..88e063e 100644 --- a/fsck/resize.c +++ b/fsck/resize.c @@ -519,6 +519,11 @@ static void rebuild_checkpoint(struct f2fs_sb_info *sbi, else set_cp(checksum_offset, CP_CHKSUM_OFFSET); + if (c.large_nat_bitmap) { + set_cp(checksum_offset, CP_MIN_CHKSUM_OFFSET); + flags |= CP_LARGE_NAT_BITMAP_FLAG; + } + set_cp(ckpt_flags, flags); memcpy(new_cp, cp, (unsigned char *)cp->sit_nat_version_bitmap - -- 2.7.4 ^ permalink raw reply related [flat|nested] 5+ messages in thread
end of thread, other threads:[~2020-01-13 6:51 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <1578482416650.67283@xiaomi.com>
2020-01-09 7:14 ` [f2fs-dev][PATCH] resize.f2fs: add option for large_nat_bitmap feature Chao Yu
2020-01-10 9:58 ` [f2fs-dev] [PATCH] " xiong ping
2020-01-11 9:43 ` Chao Yu
2020-01-13 6:50 ` xiong ping
2020-01-09 9:32 [f2fs-dev][PATCH] " ping xiong
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).