Re: [PATCH block/for-4.2-writeback] v9fs: fix error handling in v9fs_session_init()

2015-06-08 Thread Jens Axboe
On 06/07/2015 11:57 PM, Tejun Heo wrote: On failure, v9fs_session_init() returns with the v9fs_session_info struct partially initialized and expects the caller to invoke v9fs_session_close() to clean it up; however, it doesn't track whether the bdi is initialized or not and curiously invokes

Re: [PATCH block/for-4.2-writeback] v9fs: fix error handling in v9fs_session_init()

2015-06-08 Thread Jens Axboe
On 06/07/2015 11:57 PM, Tejun Heo wrote: On failure, v9fs_session_init() returns with the v9fs_session_info struct partially initialized and expects the caller to invoke v9fs_session_close() to clean it up; however, it doesn't track whether the bdi is initialized or not and curiously invokes

[PATCH block/for-4.2-writeback] v9fs: fix error handling in v9fs_session_init()

2015-06-07 Thread Tejun Heo
On failure, v9fs_session_init() returns with the v9fs_session_info struct partially initialized and expects the caller to invoke v9fs_session_close() to clean it up; however, it doesn't track whether the bdi is initialized or not and curiously invokes bdi_destroy() in both vfs_session_init()

[PATCH block/for-4.2-writeback] v9fs: fix error handling in v9fs_session_init()

2015-06-07 Thread Tejun Heo
On failure, v9fs_session_init() returns with the v9fs_session_info struct partially initialized and expects the caller to invoke v9fs_session_close() to clean it up; however, it doesn't track whether the bdi is initialized or not and curiously invokes bdi_destroy() in both vfs_session_init()