Skip to content

drivers: xen frontend devices should handle missed backend CLOSING

David Vrabel linux-fbdev linux-input linux-pci lkml netdev xen-devel 1 version In review

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest5 patches 16 repliesIn review

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/5xen-netfront: handle backend CLOSED without CLOSINGdrivers/net/xen-netfront.c+4 -10 replies
1/5xen-netfront: handle backend CLOSED without CLOSINGdrivers/net/xen-netfront.c+4 -10 replies
2/5xen-blkfront: handle backend CLOSED without CLOSINGdrivers/block/xen-blkfront.c+4 -10 replies
2/5xen-blkfront: handle backend CLOSED without CLOSINGdrivers/block/xen-blkfront.c+4 -10 replies
3/5xen-pcifront: handle backend CLOSED without CLOSINGdrivers/pci/xen-pcifront.c+4 -10 replies
3/5xen-pcifront: handle backend CLOSED without CLOSINGdrivers/pci/xen-pcifront.c+4 -18 replies
4/5xen-fbfront: handle backend CLOSED without CLOSINGdrivers/video/xen-fbfront.c+4 -14 replies
4/5xen-fbfront: handle backend CLOSED without CLOSINGdrivers/video/xen-fbfront.c+4 -10 replies
5/5xen-kbdfront: handle backend CLOSED without CLOSINGdrivers/input/misc/xen-kbdfront.c+4 -14 replies
5/5xen-kbdfront: handle backend CLOSED without CLOSINGdrivers/input/misc/xen-kbdfront.c+4 -10 replies

Reviews trailers per patch

All 10 patches have a Reviewed-by or Acked-by.

PatchKonrad Rzeszutek WilkDmitry TorokhovFlorian Tobias Schandinat
1/5A-b A-b
1/5A-b A-b
2/5A-b A-b
2/5A-b A-b
3/5A-b A-b
3/5A-b A-b
4/5A-b A-b A-b A-b
4/5A-b A-b A-b A-b
5/5A-b A-b A-b A-b
5/5A-b A-b A-b A-b

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes. Scroll the table sideways for all 3 reviewers.

Read the v1 thread

Cover letter [PATCH 0/5] drivers: xen frontend devices should handle missed backend CLOSING

Subsystem maintainers, you can either pick up the relevant driver patch or ack it to go via Konrad's Xen tree.

The series makes all the Xen frontend drivers handle the backend transitioning to CLOSED without the frontend having previously seen the backend in the CLOSING state.

Backends shouldn't do this but some do. e.g., if the host is XenServer and the toolstack decides to do a forced shutdown of a VBD, then the blkfront may miss the CLOSING transition and the /dev/xvdX device will not be destroyed which prevents it being reused.

I have seen systems that ended up in this state but it's not clear if this was the actual cause. However, I think in general it's a good thing to thing to improve the handling of unexpected state transitions.

David