On Fri, Sep 04, 2020 at 04:19:47PM +0200, Klaus Jensen wrote: > diff --git a/hw/block/nvme.c b/hw/block/nvme.c > index bfac3385cb64..3e32f39c7c1d 100644 > --- a/hw/block/nvme.c > +++ b/hw/block/nvme.c > @@ -110,6 +110,7 @@ static const uint32_t nvme_feature_cap[NVME_FID_MAX] = { > }; > > static void nvme_process_sq(void *opaque); > +static void nvme_aio_cb(void *opaque, int ret);
You don't need the forward declaration here. Just move the implementation above where it's used. It looks safe: nvme_aio_cb() doesn't have any circular dependencies.