Please don't reply to lustre-devel. Instead, comment in Bugzilla by using the
following link:
https://bugzilla.lustre.org/show_bug.cgi?id=10866
(In reply to comment #16)
> I agree the opinion without holding a reference from the obd_device. But i
> don't
> understand why we need to move lprocfs_obd_cleanup() to precleanup SELF_EXP.
By the time it gets to xxx_cleanup, much of the obd has already been destroyed -
all imports, exports, including the self export, locks, much private obd data
(the lov target list, etc.) Some of the proc files may be trying to read some
of that destroyed data, which could again lead to failures. By moving the proc
cleanup to precleanup, we insure that any such referenced structures are still
around. The SELF_EXP stage of precleanup means the self-export is still around,
and the obd is still "fully set up", but with no exports. (The export-specific
proc files (in b1_5) are already correctly cleaned up when the export is
destroyed, so they're safe.)
_______________________________________________
Lustre-devel mailing list
[email protected]
https://mail.clusterfs.com/mailman/listinfo/lustre-devel
