cvs commit: src/sys/geom/raid3 g_raid3.c g_raid3.h g_raid3_ctl.c
Pawel Jakub Dawidek
pjd at FreeBSD.org
Wed May 10 07:15:38 UTC 2006
pjd 2006-05-10 07:15:37 UTC
FreeBSD src repository
Modified files: (Branch: RELENG_6)
sys/geom/raid3 g_raid3.c g_raid3.h g_raid3_ctl.c
MFC: sys/geom/raid3/g_raid3.c 1.60,1.62,1.63,1.66
Sync graid3 with HEAD.
- Introduce and use delayed-destruction functionality from a pre-sync hook,
which means that devices will be destroyed on last close.
This fixes destruction order problems when, eg. RAID3 array is build on
top of RAID1 arrays.
Requested, reviewed and tested by: ru
- Be sure to not destroy device twice. This is only possible in theory, but
with this change there is even no theoretical race.
- Remove dead code.
- Comment possible event miss, which isn't critical, but probably can be
fixed by replacing the event lock usage with the queue lock.
- Use G_RAID3_FOREACH_SAFE_BIO() macro instead of G_RAID3_FOREACH_BIO() in
two places where g_io_request() is called. g_io_request() can free bio
structure so we can't reference it after and G_RAID3_FOREACH_BIO() macro
was doing this.
Found by: Coverity Prevent analysis tool (with my new models)
Revision Changes Path
188.8.131.52 +94 -74 src/sys/geom/raid3/g_raid3.c
184.108.40.206 +5 -1 src/sys/geom/raid3/g_raid3.h
220.127.116.11 +7 -1 src/sys/geom/raid3/g_raid3_ctl.c
More information about the cvs-src