Martchus
/
mdadm
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

mdadm: do not try to hold dlm lock in free_super1 

Since free_super1 actually doesn't change the sb, it
just free the addr space of sb. Also free_super1 is
called in lots of place within mdadm, so remove dlm
lock code since the func doesn't need the protection
and also reduce latency.

Signed-off-by: Guoqing Jiang <gqjiang@suse.com>
Signed-off-by: NeilBrown <neilb@suse.com>
This commit is contained in:
Guoqing Jiang 2015-12-17 01:54:25 +08:00 committed by NeilBrown
parent 53e76b1def
commit 1b78e47021
1 changed files with 0 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
super1.c
View File

@ -2422,15 +2422,6 @@ static int write_bitmap1(struct supertype *st, int fd, enum bitmap_update update
static void free_super1(struct supertype *st)
{
int rv, lockid;
if (is_clustered(st)) {
rv = cluster_get_dlmlock(st, &lockid);
if (rv) {
pr_err("Cannot get dlmlock in %s return %d\n", __func__, rv);
cluster_release_dlmlock(st, lockid);
return;
}
}
if (st->sb)
free(st->sb);
@ -2442,8 +2433,6 @@ static void free_super1(struct supertype *st)
free(di);
}
st->sb = NULL;
if (is_clustered(st))
cluster_release_dlmlock(st, lockid);
}
#ifndef MDASSEMBLE