On 02/27, Yunlei He wrote: > This patch add a function to modify discard command if one segment > reuse before discard. Split this segment from multi-segments discard > range, and discard the left bigger range.
Now MAX_DISCARD_BLOCKS() is set to section size, so there will be no multi-sections discard range. Am I missing some cases? Thanks, > > Signed-off-by: Yunlei He <heyun...@huawei.com> > --- > fs/f2fs/segment.c | 18 +++++++++++++++++- > 1 file changed, 17 insertions(+), 1 deletion(-) > > diff --git a/fs/f2fs/segment.c b/fs/f2fs/segment.c > index 9006d8e..5af736b 100644 > --- a/fs/f2fs/segment.c > +++ b/fs/f2fs/segment.c > @@ -672,6 +672,22 @@ static void __remove_discard_cmd(struct f2fs_sb_info > *sbi, struct discard_cmd *d > kmem_cache_free(discard_cmd_slab, dc); > } > > +static void __modify_discard_cmd(struct f2fs_sb_info *sbi, struct > discard_cmd *dc, block_t blkaddr) > +{ > + block_t end_block = START_BLOCK(sbi, GET_SEGNO(sbi, blkaddr) + 1); > + > + if (dc->lstart + dc->len <= end_block) { > + __remove_discard_cmd(sbi, dc); > + return; > + } > + > + if (dc->lstart - blkaddr < dc->lstart + dc->len - end_block) { > + dc->lstart = end_block; > + dc->len -= (end_block - dc->lstart); > + } else > + dc->len = blkaddr - dc->lstart; > +} > + > /* This should be covered by global mutex, &sit_i->sentry_lock */ > void f2fs_wait_discard_bio(struct f2fs_sb_info *sbi, block_t blkaddr) > { > @@ -699,7 +715,7 @@ void f2fs_wait_discard_bio(struct f2fs_sb_info *sbi, > block_t blkaddr) > if (dc->state == D_SUBMIT) > wait_for_completion_io(&dc->wait); > else > - __remove_discard_cmd(sbi, dc); > + __modify_discard_cmd(sbi, dc, blkaddr); > } > } > blk_finish_plug(&plug); > -- > 2.10.1 ------------------------------------------------------------------------------ Check out the vibrant tech community on one of the world's most engaging tech sites, SlashDot.org! http://sdm.link/slashdot _______________________________________________ Linux-f2fs-devel mailing list Linux-f2fs-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel