Re: [PATCH v2 3/3] replay: do not build if TCG is not available

[Paolo Bonzini]

On 12/10/20 23:45, Claudio Fontana wrote:
> +    ctx = blk_get_aio_context(blk);
> +    if (!replay_bh_schedule_oneshot_event(ctx, error_callback_bh, acb)) {
> +        /* regular case without replay */
> +        aio_bh_schedule_oneshot(ctx, error_callback_bh, acb);
> +    }

Why can't the stub just call aio_bh_schedule_oneshot?  This makes the
API even more complicated.

I think you are doing this in order to avoid link errors in tools, but
it's not necessary. you can have more than one stub file:

- replay/replay-stub.c for functions needed by emulators (added with
"if_false:", it also includes the monitor commands);

- stubs/replay.c for functions needed by tools (including
replay_bh_schedule_oneshot_event which is currently in
stubs/replay-user.c for some reason).

Paolo