On 07/07/2015 13:48, Frederic Konrad wrote: >>> this eventually ends up doing a tb_lock on the find_slow path which IIRC >>> is when might end up doing the actual code generation. >> >> Up to this point, system emulation is using the BQL for everything. I >> guess things change later. > > Actually we use tb_lock to protect all the tb related structure such as > TBContext etc.. Is it better to use the global lock for this?
No, on the contrary. But using the BQL is the status as of patch 2, so it's okay to keep the #ifdefs. Thanks for confirming that it changes later in the patch. Paolo