<div dir="ltr"><div dir="ltr"><br></div><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Dec 6, 2019 at 12:32 PM Frediano Ziglio <<a href="mailto:fziglio@redhat.com">fziglio@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">> <br>
> From: Alexander Nezhinsky <<a href="mailto:anezhins@redhat.com" target="_blank">anezhins@redhat.com</a>><br>
> <br>
> Some corner error scenarios may leave dangling not connected shared CD<br>
> devices.<br>
> As the current flow precludes maintenance of such devices, it may be prudent<br>
> to identify and remove them.<br>
> <br>
> Signed-off-by: Alexander Nezhinsky <<a href="mailto:anezhins@redhat.com" target="_blank">anezhins@redhat.com</a>><br>
<br>
This patch and code seems to indicate a workaround for a bug in<br>
the implementation.<br></blockquote><div>It is not about *a bug*. It is a host of corner cases where the CD remained connected and unremoved.</div><div>This is not a concern with the "real" devices, as they can be present on the system and become unconnected.</div><div>It could be the same case for a shared CD if we'd have supported such a state.</div><div>But we decided to disallow it - if the connection is lost, the Cd devices should be gone.</div><div>This creates a potentially problematic state. </div><div>Instead of hunting after all such cases and in some cases finding it difficult to cope with them, it seems to be reasonable to solve it this way. </div><div>We may decide otherwise, but I'd suggest that even then we accept the patch, leave it as a known issue and dedicate a few separate patches to the fix, perhaps immediately.</div><div><br></div></div></div>