Re: [PATCH] jbd2: remove the conditional test

2018-05-20 Thread Theodore Y. Ts'o
On Mon, Feb 05, 2018 at 10:24:39PM +0800, Wang Long wrote: > kmem_cache_destroy already handles null pointers, so we can remove the > conditional test entirely. > > This patch also set NULL after the kmem_cache_destroy in function > jbd2_journal_destroy_handle_cache. > > Signed-off-by: Wang Long

Re: [PATCH] jbd2: remove the conditional test

2018-02-06 Thread Jan Kara
On Mon 05-02-18 22:24:39, Wang Long wrote: > kmem_cache_destroy already handles null pointers, so we can remove the > conditional test entirely. > > This patch also set NULL after the kmem_cache_destroy in function > jbd2_journal_destroy_handle_cache. > > Signed-off-by: Wang Long Looks good. Yo

[PATCH] jbd2: remove the conditional test

2018-02-05 Thread Wang Long
kmem_cache_destroy already handles null pointers, so we can remove the conditional test entirely. This patch also set NULL after the kmem_cache_destroy in function jbd2_journal_destroy_handle_cache. Signed-off-by: Wang Long --- fs/jbd2/journal.c | 18 +++--- fs/jbd2/revoke.c