On Tue, 2020-03-31 at 07:40 +0200, Klaus Birkelund Jensen wrote: > 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!
Thanks. > > > > With 'temperature' field removed from the header: > > > > Reviewed-by: Maxim Levitsky <mlevi...@redhat.com> > > > > Best regards, > > Maxim Levitsky > > > > Best regards, Maxim Levitsky