Re: [PATCH] iscsi: do not fail to stop a stopped pool

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Apr 29, 2015 at 03:08:14PM +0200, Ján Tomko wrote:
Just as we allow stopping filesystem pools when they were unmounted
externally, do not fail to stop an iscsi pool when someone else
closed the session externally.

Resolves:
https://bugzilla.redhat.com/show_bug.cgi?id=1171984
---
src/storage/storage_backend_iscsi.c | 5 +++++
1 file changed, 5 insertions(+)

diff --git a/src/storage/storage_backend_iscsi.c b/src/storage/storage_backend_iscsi.c
index 197d333..bea6758 100644
--- a/src/storage/storage_backend_iscsi.c
+++ b/src/storage/storage_backend_iscsi.c
@@ -449,8 +449,13 @@ virStorageBackendISCSIStopPool(virConnectPtr conn ATTRIBUTE_UNUSED,
                               virStoragePoolObjPtr pool)
{
    char *portal;
+    char *session;
    int ret = -1;

+    if ((session = virStorageBackendISCSISession(pool, false)) == NULL)

Shouldn't this be called with true and not false, so it doesn't report
an error?

+        return 0;
+    VIR_FREE(session);
+

Will this help that much, since it can be stopped right after the
previous command found it?  Wouldn't it be better to handle this in
virISCSIConnection() or virISCSIConnectionLogout() somehow?

I'm just asking because I'm not the proper one to review iscsi stuff,
otherwise I'd probably ACK'd it (if there was that "true"), since it
will fix more than it will break (1 >> 0).

    if ((portal = virStorageBackendISCSIPortal(&pool->def->source)) == NULL)
        return -1;

--
2.0.5

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

Attachment: signature.asc
Description: PGP signature

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]