[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 2/4] Introduce event-loop-base abstract class
From: |
Nicolas Saenz Julienne |
Subject: |
Re: [PATCH v2 2/4] Introduce event-loop-base abstract class |
Date: |
Mon, 14 Mar 2022 14:34:52 +0100 |
User-agent: |
Evolution 3.42.4 (3.42.4-1.fc35) |
On Mon, 2022-03-14 at 13:33 +0000, Stefan Hajnoczi wrote:
> On Fri, Mar 11, 2022 at 11:17:22AM +0100, Nicolas Saenz Julienne wrote:
> > On Thu, 2022-03-10 at 10:25 +0000, Stefan Hajnoczi wrote:
> > > On Thu, Mar 03, 2022 at 03:58:20PM +0100, Nicolas Saenz Julienne wrote:
> > > > @@ -2935,13 +2947,6 @@ qemu_syms = custom_target('qemu.syms', output:
> > > > 'qemu.syms',
> > > > capture: true,
> > > > command: [undefsym, nm, '@INPUT@'])
> > > >
> > > > -qom_ss = qom_ss.apply(config_host, strict: false)
> > > > -libqom = static_library('qom', qom_ss.sources() + genh,
> > > > - dependencies: [qom_ss.dependencies()],
> > > > - name_suffix: 'fa')
> > > > -
> > > > -qom = declare_dependency(link_whole: libqom)
> > > > -
> > >
> > > Why was it necessary to move qom_ss and subdir('hw') up? Can
> > > event_loop_base be defined down here instead?
> >
> > The way I setup it up, qemuutil now depdens on event_loop_base which in turn
> > depends on qom. IIUC I can't declare dependencies without declaring first
> > the
> > libraries and source sets. All has to happen sequencially. With this in
> > mind,
> > almost all libraries depend on libqemuutil so moving it down isn't possible.
>
> I see now. The qemuutil dependency on event_loop_base is introduced in
> the next patch so the reason wasn't clear at this point in the patch
> series.
I'll mention it in the commit message.
--
Nicolás Sáenz
[PATCH v2 3/4] util/main-loop: Introduce the main loop into QOM, Nicolas Saenz Julienne, 2022/03/03
[PATCH v2 4/4] util/event-loop-base: Introduce options to set the thread pool size, Nicolas Saenz Julienne, 2022/03/03