Skip to content

Commit

Permalink
Merge #1624
Browse files Browse the repository at this point in the history
1624: fix(rebuild): reconnect log on own channel when faulting r=tiagolobocastro a=tiagolobocastro

When an IO fails to submit the channel is removed right away. This means if another IO reaches the channel before the retire or the channel traversal then it might not get logged.
This change ensures it is logged by reconnecting the IO log straight away.

Co-authored-by: Tiago Castro <[email protected]>
  • Loading branch information
mayastor-bors and tiagolobocastro committed Apr 5, 2024
2 parents 007ab46 + f9108d0 commit f2f74e7
Showing 1 changed file with 8 additions and 2 deletions.
10 changes: 8 additions & 2 deletions io-engine/src/bdev/nexus/nexus_channel.rs
Original file line number Diff line number Diff line change
Expand Up @@ -373,8 +373,14 @@ impl<'n> NexusChannel<'n> {
child_device: &str,
reason: FaultReason,
) -> Option<IOLogChannel> {
self.nexus_mut()
.retire_child_device(child_device, reason, true)
let Some(io_log) =
self.nexus_mut()
.retire_child_device(child_device, reason, true)
else {
return None;
};
self.reconnect_io_logs();
Some(io_log)
}

/// Returns core on which channel was created.
Expand Down

0 comments on commit f2f74e7

Please sign in to comment.