diff options
| author |   Jaegeuk Kim <jaegeuk.kim@samsung.com> | 2013-01-21 17:34:21 +0900 |
|---|---|---|
| committer | Jaegeuk Kim <jaegeuk.kim@samsung.com> | 2013-01-22 10:48:59 +0900 |
| commit | 9af45ef5ab8ce4a13c553200dc15509441fbd68f (patch) | |
| tree | 7a1d76bb6d1d87ab0a3eace1311446666ec01c18 /fs/f2fs | |
| parent | f2fs: avoid issuing small bios due to several dirty node pages (diff) | |
| download | linux-dev-9af45ef5ab8ce4a13c553200dc15509441fbd68f.tar.xz linux-dev-9af45ef5ab8ce4a13c553200dc15509441fbd68f.zip | |
f2fs: add comments of start_bidx_of_node
The caller of start_bidx_of_node() should give proper node offsets which
point only direct node blocks. Otherwise, it is a caller's bug.
This patch adds comments to make it clear.
Reported-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Jaegeuk Kim <jaegeuk.kim@samsung.com>
Diffstat (limited to 'fs/f2fs')
| -rw-r--r-- | fs/f2fs/gc.c | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/fs/f2fs/gc.c b/fs/f2fs/gc.c index 809cfec6683c..c386910dacc5 100644 --- a/fs/f2fs/gc.c +++ b/fs/f2fs/gc.c @@ -424,7 +424,11 @@ next_step: } /* - * Calculate start block index that this node page contains + * Calculate start block index indicating the given node offset. + * Be careful, caller should give this node offset only indicating direct node + * blocks. If any node offsets, which point the other types of node blocks such + * as indirect or double indirect node blocks, are given, it must be a caller's + * bug. */ block_t start_bidx_of_node(unsigned int node_ofs) { |