[Cluster-devel] [fence-virt PATCH] fix handling of VIR_DOMAIN_EVENT_STARTED

[Lon Hohberger <lhh@redhat.com>]

On 09/19/2011 04:32 PM, Lon Hohberger wrote:
> On 09/13/2011 04:44 AM, Kazunori INOUE wrote:
> 
>> I attached the patch for this problem.
>>
>> * Modified to send a SIGHUP to the primary thread from the secondary
>>     thread detects the start of the VM, and
>>     the primary thread is modified so as to select() the requests from
>>     the VM which started.
> 
> That's certainly one way of doing it.  I might have used
> 'pthread_cond_singal'.

I am sorry for the delay.

I have a fix based on yours that uses a pipe to wake up select() rather
than pthread_kill; I'll attach it here.  This ends up being a bit lower
impact, as well; it doesn't require changing the main program - only the
serial/vmchannel plugin.

Apart from that, the patch here is largely the same.

Let me know what you think at your convenience.

-- Lon
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: 0001-Fix-serial-domain-handling.patch
URL: <http://listman.redhat.com/archives/cluster-devel/attachments/20111025/d7b5bd33/attachment.ksh>