diff options
author | Nicholas Bellinger <nab@linux-iscsi.org> | 2011-02-09 23:34:53 (GMT) |
---|---|---|
committer | James Bottomley <James.Bottomley@suse.de> | 2011-02-12 17:39:14 (GMT) |
commit | 7c2bf6e925c38b8e3358f5046971b0d6086ddcf8 (patch) | |
tree | 824200b53c4a95cab8e337d6c776aaacb1ef782a /arch/mips | |
parent | 85dc98d93f3dc41cce54118a7abab9e6aa616dd2 (diff) | |
download | linux-fsl-qoriq-7c2bf6e925c38b8e3358f5046971b0d6086ddcf8.tar.xz |
[SCSI] target: Fix top-level configfs_subsystem default_group shutdown breakage
This patch fixes two bugs uncovered during testing with
slub_debug=FPUZ during module_exit() -> target_core_exit_configfs()
with release of configfs subsystem consumer default groups, namely how
this should be working with
fs/configfs/dir.c:configfs_unregister_subsystem() release logic for
struct config_group->default_group.
The first issue involves configfs_unregister_subsystem() expecting to
walk+drain the top-level subsys->su_group.default_groups directly in
unlink_group(), and not directly from the configfs subsystem consumer
for the top level struct config_group->default_groups. This patch
drops the walk+drain of subsys->su_group.default_groups from TCM
configfs subsystem consumer code, and moves the top-level
->default_groups kfree() after configfs_unregister_subsystem() has
been called.
The second issue involves calling
core_alua_free_lu_gp(se_global->default_lu_gp) to release the
default_lu_gp->lu_gp_group before configfs_unregister_subsystem() has
been called. This patches also moves the core_alua_free_lu_gp() call
to release default_lu_group->lu_gp_group after the subsys has been
unregistered.
Finally, this patch explictly clears the
[lu_gp,alua,hba]_cg->default_groups pointers after kfree() to ensure
that no stale memory is picked up from child struct
config_group->default_group[] while configfs_unregister_subsystem() is
called.
Reported-by: Fubo Chen <fubo.chen@gmail.com>
Signed-off-by: Nicholas A. Bellinger <nab@linux-iscsi.org>
Signed-off-by: James Bottomley <James.Bottomley@suse.de>
Diffstat (limited to 'arch/mips')
0 files changed, 0 insertions, 0 deletions