<html xmlns:v="urn:schemas-microsoft-com:vml" xmlns:o="urn:schemas-microsoft-com:office:office" xmlns:w="urn:schemas-microsoft-com:office:word" xmlns:m="http://schemas.microsoft.com/office/2004/12/omml" xmlns="http://www.w3.org/TR/REC-html40"> <head> <meta http-equiv="Content-Type" content="text/html; charset=gb2312"> <meta name="Generator" content="Microsoft Word 14 (filtered medium)"> <style></style> </head> <body bgcolor="white" lang="ZH-CN" link="blue" vlink="purple"> <div class="WordSection1"> Hi,<o:p></o:p> I think you misunderstand my meaning. My solution includes step1 + step2. Step1 is used to implement thread mutex. Step2 is used to <o:p></o:p> handle “initialized” visibility. Without step2, the initialization could be executed several times.<o:p></o:p> <o:p> </o:p> B.R.<o:p></o:p> Benjamin Wang<o:p></o:p> <o:p> </o:p> <div> <div style="border:none;border-top:solid #B5C4DF 1.0pt;padding:3.0pt 0cm 0cm 0cm"> From: Guannan Ren [mailto:gren@redhat.com] Sent: 2012年9月29日 17:22 To: Benjamin Wang (gendwang) Cc: Daniel Veillard; libvir-list@redhat.com; Yang Zhou (yangzho); cbley@av-test.de Subject: Re: [libvirt] Potential race condition problem<o:p></o:p> </div> </div> <o:p> </o:p> <div> On 09/29/2012 03:52 PM, Benjamin Wang (gendwang) wrote:<o:p></o:p> </div> <blockquote style="margin-top:5.0pt;margin-bottom:5.0pt"> Hi,<o:p></o:p> OK. Now I am using JNA to access libvirt. If we add another mutex which used to access “initialized” parameter. This mutex must be pthread_mutex_init firstly and only once.<o:p></o:p> But it seems that there is no way to change libvirt code. I do it as following:<o:p></o:p> <![if !supportLists]>1. <![endif]>Changing libvirt JNA code in Connect.java<o:p></o:p> Old Code:<o:p></o:p> public Connect(String uri) throws LibvirtException {<o:p></o:p> VCP = libvirt.virConnectOpen(uri);<o:p></o:p> // Check for an error<o:p></o:p> processError();<o:p></o:p> ErrorHandler.processError(Libvirt.INSTANCE);<o:p></o:p> }<o:p></o:p> <o:p></o:p> New Code:<o:p></o:p> public Connect(String uri) throws LibvirtException {<o:p></o:p> synchronized(this.getClass()) {<o:p></o:p> VCP = libvirt.virConnectOpen(uri);<o:p></o:p> }<o:p></o:p> // Check for an error<o:p></o:p> processError();<o:p></o:p> ErrorHandler.processError(Libvirt.INSTANCE);<o:p></o:p> }<o:p></o:p> <o:p></o:p> This can make sure only that one thread can execute Connect. For a server application, we only need one time. So the performance is OK<o:p></o:p> <o:p></o:p> <![if !supportLists]>2. <![endif]>Changing libvirt code in libvirt.c<o:p></o:p> Old Code:<o:p></o:p> static int initialized = 0;<o:p></o:p> <o:p></o:p> New Code:<o:p></o:p> static int volatile initialized = 0;<o:p></o:p> <o:p></o:p> This can make sure the initialization will be executed once.<o:p></o:p> <o:p></o:p> Would you give your comments for this solution?<o:p></o:p> <o:p></o:p> B.R.<o:p></o:p> Benjamin Wang<o:p></o:p> <o:p> </o:p> </blockquote> As far as I know the operations on <code>volatile</code> variable is not atomic, the usage of <code>volatile</code> keyword as a portable synchronization mechanism is discouraged by C. But in Java, it is a global ordering on the reads and writes to a volatile variable. So, maybe, your first solution is pretty enough good. Guannan <o:p></o:p> </div> </body> </html>