Re: [PATCH v5 02/10] qemu: Use domain iothreadids to IOThread's 'thread_id'

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

 



On Mon, Apr 27, 2015 at 11:25:15 -0400, John Ferlan wrote:
> ...
> 
> >>>> --- a/src/qemu/qemu_process.c
> >>>> +++ b/src/qemu/qemu_process.c
> >>>> @@ -2267,12 +2267,16 @@ qemuProcessDetectIOThreadPIDs(virQEMUDriverPtr driver,
> >>>>          goto cleanup;
> >>>>      }
> >>>
> >>> A few lines prior here is the check that the expected thread count
> >>> equals to the actual thread count. For some reason a few lines before
> >>> returns success if 0 threads are returned by the monitor. The two checks
> >>> should be inverted so that it makes sense.
> >>>
> >>
> >> If there are no threads, then it's not a failure, thus change ret to be
> >> 0. Again, this is something that's not within the scope of this set of
> >> changes and I believe if necessary could be a followup patch.
> >>
> >> I'm not clear on the value of changing the order of the checks.
> > 
> > The problem is that if there are no iothreads reported by qemu, but we
> > did request some then it IS failure.
> > 
> 
> But that's an issue not related to iothreadid's per se - it's a more
> common general issue that should be a follow-up patch then I think.
> That is not introduced by this set of changes.

Agreed, this should be done separately.

> 
> Other issues were addressed changed - do you need to see the diffs or an
> updated patch with the diffs already squashed in?

I'd like to see the fixed hunk of qemuProcessDetectIOThreadPIDs that
parses the reply from the monitor.

> 
> John
> 

Peter

Attachment: signature.asc
Description: Digital 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]