Re: [Cluster-devel] gfs2: gfs2_dir_get_hash_table(): avoiding deferred vfree() is easy here...

2014-11-20 Thread Steven Whitehouse
Hi, All five patches now in the GFS2 -nmw tree. Thanks, Steve. On 20/11/14 05:19, Al Viro wrote: vfree() is allowed under spinlock these days, but it's cheaper when it doesn't step into deferred case and here it's very easy to avoid. Signed-off-by: Al Viro --- fs/gfs2/dir.c |7 ---

Re: [Cluster-devel] gfs2: gfs2_dir_get_hash_table(): avoiding deferred vfree() is easy here...

2014-11-20 Thread Bob Peterson
- Original Message - > vfree() is allowed under spinlock these days, but it's cheaper when > it doesn't step into deferred case and here it's very easy to avoid. > > Signed-off-by: Al Viro > --- > fs/gfs2/dir.c |7 --- > 1 file changed, 4 insertions(+), 3 deletions(-) > > diff

Re: [Cluster-devel] gfs2: gfs2_dir_get_hash_table(): avoiding deferred vfree() is easy here...

2014-11-20 Thread Bob Peterson
- Original Message - vfree() is allowed under spinlock these days, but it's cheaper when it doesn't step into deferred case and here it's very easy to avoid. Signed-off-by: Al Viro v...@zeniv.linux.org.uk --- fs/gfs2/dir.c |7 --- 1 file changed, 4 insertions(+), 3

Re: [Cluster-devel] gfs2: gfs2_dir_get_hash_table(): avoiding deferred vfree() is easy here...

2014-11-20 Thread Steven Whitehouse
Hi, All five patches now in the GFS2 -nmw tree. Thanks, Steve. On 20/11/14 05:19, Al Viro wrote: vfree() is allowed under spinlock these days, but it's cheaper when it doesn't step into deferred case and here it's very easy to avoid. Signed-off-by: Al Viro v...@zeniv.linux.org.uk ---