qemu-devel
[Top][All Lists]
Advanced

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

Re: [PATCH v2 1/4] hyperv: SControl is optional to enable SynIc


From: Paolo Bonzini
Subject: Re: [PATCH v2 1/4] hyperv: SControl is optional to enable SynIc
Date: Tue, 15 Mar 2022 12:29:50 +0100
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.5.0

On 2/16/22 11:24, Jon Doron wrote:
+    qemu_mutex_lock(&synic->sint_routes_mutex);
+    QLIST_INSERT_HEAD_RCU(&synic->sint_routes, sint_route, link);
+    qemu_mutex_unlock(&synic->sint_routes_mutex);

Hi,

I don't see any access to sint_routes outside hyperv_sint_route_new and hyperv_sint_route_unref. Am I missing something or is this just for debugging? If so, using the _RCU functions is not necessary.

Otherwise everything looks good, thanks!

Paolo



reply via email to

[Prev in Thread] Current Thread [Next in Thread]