Hi On Fri, Oct 7, 2022 at 1:04 AM Richard Henderson < richard.hender...@linaro.org> wrote:
> On 10/6/22 05:51, Marc-André Lureau wrote: > > Hi Richard > > > > On Mon, Oct 3, 2022 at 11:39 AM Marc-André Lureau < > marcandre.lur...@redhat.com > > <mailto:marcandre.lur...@redhat.com>> wrote: > > > > Hi > > > > On Fri, Sep 30, 2022 at 6:10 PM Richard Henderson > > <richard.hender...@linaro.org <mailto:richard.hender...@linaro.org>> > wrote: > > > > > > On 9/30/22 07:03, marcandre.lur...@redhat.com <mailto: > marcandre.lur...@redhat.com> > > wrote: > > > > +static bool > > > > +set_thread_description(HANDLE h, const char *name) > > > > +{ > > > > + HRESULT hr; > > > > + g_autofree wchar_t *namew = NULL; > > > > + > > > > + if (!load_set_thread_description()) { > > > > + return false; > > > > + } > > > > > > I don't understand why you're retaining this. > > > What is your logic? > > > > > > > Also, if we change the "static bool name_threads" to be true by > > default, then set_thread_description() might be called without > calling > > qemu_thread_naming() > > > > It really shouldn't hurt to keep it that way. > > > > > > > > Let me know if the current form is ok for you, thanks > > I guess it's ok, sure. > Should I take that for an Ack? :) thanks -- Marc-André Lureau