On Mar 25 12:40, Maxim Levitsky wrote: > On Mon, 2020-03-16 at 07:28 -0700, Klaus Jensen wrote: > > From: Klaus Jensen <k.jen...@samsung.com> > > > > It might seem wierd to implement this feature for an emulated device, > > but it is mandatory to support and the feature is useful for testing > > asynchronous event request support, which will be added in a later > > patch. > > > > Signed-off-by: Klaus Jensen <k.jen...@samsung.com> > > Acked-by: Keith Busch <kbu...@kernel.org> > > --- > > hw/block/nvme.c | 48 ++++++++++++++++++++++++++++++++++++++++++++ > > hw/block/nvme.h | 2 ++ > > include/block/nvme.h | 8 +++++++- > > 3 files changed, 57 insertions(+), 1 deletion(-) > > > > diff --git a/hw/block/nvme.h b/hw/block/nvme.h > > index b7c465560eea..8cda5f02c622 100644 > > --- a/hw/block/nvme.h > > +++ b/hw/block/nvme.h > > @@ -108,6 +108,7 @@ typedef struct NvmeCtrl { > > uint64_t irq_status; > > uint64_t host_timestamp; /* Timestamp sent by the > > host */ > > uint64_t timestamp_set_qemu_clock_ms; /* QEMU clock time */ > > + uint16_t temperature; > You forgot to move this too. >
Fixed! > > With 'temperature' field removed from the header: > > Reviewed-by: Maxim Levitsky <mlevi...@redhat.com> > > Best regards, > Maxim Levitsky >