RE: [f2fs-dev] [PATCH 2/4] f2fs: return early when trying to read null nid

2015-12-24 Thread Chao Yu
24, 2015 10:15 AM > > > To: linux-kernel@vger.kernel.org; linux-fsde...@vger.kernel.org; > > > linux-f2fs-de...@lists.sourceforge.net > > > Cc: Jaegeuk Kim > > > Subject: [f2fs-dev] [PATCH 2/4] f2fs: return early when trying to read > > > null nid > &

Re: [f2fs-dev] [PATCH 2/4] f2fs: return early when trying to read null nid

2015-12-24 Thread Jaegeuk Kim
.kernel.org; > > linux-f2fs-de...@lists.sourceforge.net > > Cc: Jaegeuk Kim > > Subject: [f2fs-dev] [PATCH 2/4] f2fs: return early when trying to read null > > nid > > > > If get_node_page() gets zero nid, we can return early without getting a > > wrong > &

RE: [f2fs-dev] [PATCH 2/4] f2fs: return early when trying to read null nid

2015-12-24 Thread Chao Yu
24, 2015 10:15 AM > > > To: linux-kernel@vger.kernel.org; linux-fsde...@vger.kernel.org; > > > linux-f2fs-de...@lists.sourceforge.net > > > Cc: Jaegeuk Kim > > > Subject: [f2fs-dev] [PATCH 2/4] f2fs: return early when trying to read > > > null nid > &

Re: [f2fs-dev] [PATCH 2/4] f2fs: return early when trying to read null nid

2015-12-24 Thread Jaegeuk Kim
.kernel.org; > > linux-f2fs-de...@lists.sourceforge.net > > Cc: Jaegeuk Kim > > Subject: [f2fs-dev] [PATCH 2/4] f2fs: return early when trying to read null > > nid > > > > If get_node_page() gets zero nid, we can return early without getting a > > wrong > &

RE: [f2fs-dev] [PATCH 2/4] f2fs: return early when trying to read null nid

2015-12-23 Thread Chao Yu
[f2fs-dev] [PATCH 2/4] f2fs: return early when trying to read null > nid > > If get_node_page() gets zero nid, we can return early without getting a wrong > page. For example, get_dnode_of_data() can try to do that. Good catch! > > Signed-off-by: Jaegeuk Kim > --- >

RE: [f2fs-dev] [PATCH 2/4] f2fs: return early when trying to read null nid

2015-12-23 Thread Chao Yu
[f2fs-dev] [PATCH 2/4] f2fs: return early when trying to read null > nid > > If get_node_page() gets zero nid, we can return early without getting a wrong > page. For example, get_dnode_of_data() can try to do that. Good catch! > > Signed-off-by: Jaegeuk Kim <jaeg...@kernel.org&