On Feb 10 18:15, Bin Meng wrote: > On Wed, Feb 10, 2021 at 5:54 PM Bin Meng <bmeng...@gmail.com> wrote: > > > > From: Bin Meng <bin.m...@windriver.com> > > > > Current QEMU HEAD nvme.c does not compile: > > > > hw/block/nvme.c: In function ‘nvme_process_sq’: > > Not sure why compiler reports this error happens in nvme_process_sq()? >
Yeah that is kinda wierd. Also, this went through the full CI suite.
What compiler is this?
> But it should be in nvme_get_feature(). I will update the commit message in
> v2.
>
> > hw/block/nvme.c:3242:9: error: ‘result’ may be used uninitialized in this
> > function [-Werror=maybe-uninitialized]
> > trace_pci_nvme_getfeat_vwcache(result ? "enabled" : "disabled");
> > ^
> > hw/block/nvme.c:3150:14: note: ‘result’ was declared here
> > uint32_t result;
> > ^
> >
> > Explicitly initialize the result to fix it.
> >
> > Fixes: aa5e55e3b07e ("hw/block/nvme: open code for volatile write cache")
> > Signed-off-by: Bin Meng <bin.m...@windriver.com>
> > ---
> >
> > hw/block/nvme.c | 1 +
> > 1 file changed, 1 insertion(+)
> >
>
> Regards,
> Bin
>
signature.asc
Description: PGP signature
