]> git.proxmox.com Git - mirror_zfs.git/commit
Fixing gang ABD child removal race condition
authorBrian Atkinson <batkinson@lanl.gov>
Tue, 14 Jul 2020 18:04:35 +0000 (12:04 -0600)
committerGitHub <noreply@github.com>
Tue, 14 Jul 2020 18:04:35 +0000 (11:04 -0700)
commite4d3d776844fd9c53b4bb641e21a7eff62052dca
tree892ee7dbea9fa1774efbb12f1e340c28596aeb0a
parentc15d36c674bcfb10975fc835978d4a49d159bf0b
Fixing gang ABD child removal race condition

On linux the list debug code has been setting off a failure when
checking that the node->next->prev value is pointing back at the node.
At times this check evaluates to 0xdead. When removing a child from a
gang ABD we must acquire the child's abd_mtx to make sure that the
same ABD is not being added to another gang ABD while it is being
removed from a gang ABD. This fixes a race condition when checking
if an ABDs link is already active and part of another gang ABD before
adding it to a gang.

Added additional debug code for the gang ABD in abd_verify() to make
sure each child ABD has active links. Also check to make sure another
gang ABD is not added to a gang ABD.

Reviewed-by: Serapheim Dimitropoulos <serapheim@delphix.com>
Reviewed-by: Brian Behlendorf <behlendorf1@llnl.gov>
Reviewed-by: Matt Ahrens <matt@delphix.com>
Signed-off-by: Brian Atkinson <batkinson@lanl.gov>
Closes #10511
include/os/linux/spl/sys/list.h
lib/libspl/list.c
module/os/freebsd/spl/list.c
module/zfs/abd.c