[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [PATCH v2] nvme: add Get/Set Feature Timestamp support
From: |
Kevin Wolf |
Subject: |
Re: [Qemu-block] [PATCH v2] nvme: add Get/Set Feature Timestamp support |
Date: |
Mon, 3 Jun 2019 13:14:09 +0200 |
User-agent: |
Mutt/1.11.3 (2019-02-01) |
Am 28.05.2019 um 08:18 hat Klaus Birkelund geschrieben:
> On Mon, May 20, 2019 at 11:40:30AM -0600, Kenneth Heitke wrote:
> > Signed-off-by: Kenneth Heitke <address@hidden>
> > diff --git a/hw/block/nvme.h b/hw/block/nvme.h
> > index 56c9d4b4b1..d7277e72b7 100644
> > --- a/hw/block/nvme.h
> > +++ b/hw/block/nvme.h
> > @@ -69,6 +69,7 @@ typedef struct NvmeCtrl {
> > uint16_t max_prp_ents;
> > uint16_t cqe_size;
> > uint16_t sqe_size;
> > + uint16_t oncs;
>
> Looks like this unused member snuck its way into the patch. But I see no
> harm in it being there.
Good catch. I'll just remove it again from my branch.
> > +static inline void nvme_set_timestamp(NvmeCtrl *n, uint64_t ts)
> > +{
> > + trace_nvme_setfeat_timestamp(ts);
> > +
> > + n->host_timestamp = le64_to_cpu(ts);
> > + n->timestamp_set_qemu_clock_ms =
> > qemu_clock_get_ms(QEMU_CLOCK_REALTIME);
> > +}
> > +
> > +static inline uint64_t nvme_get_timestamp(const NvmeCtrl *n)
> > +{
> > + uint64_t current_time = qemu_clock_get_ms(QEMU_CLOCK_REALTIME);
Here I wonder why we use QEMU_CLOCK_REALTIME in a device emulation.
Wouldn't QEMU_CLOCK_VIRTUAL make more sense?
Kevin
- Re: [Qemu-block] [PATCH v2] nvme: add Get/Set Feature Timestamp support,
Kevin Wolf <=
- Re: [Qemu-block] [PATCH v2] nvme: add Get/Set Feature Timestamp support, Heitke, Kenneth, 2019/06/03
- Re: [Qemu-block] [PATCH v2] nvme: add Get/Set Feature Timestamp support, Klaus Birkelund, 2019/06/04
- Re: [Qemu-block] [PATCH v2] nvme: add Get/Set Feature Timestamp support, Kevin Wolf, 2019/06/04
- Re: [Qemu-block] [PATCH v2] nvme: add Get/Set Feature Timestamp support, Klaus Birkelund, 2019/06/04
- Re: [Qemu-block] [PATCH v2] nvme: add Get/Set Feature Timestamp support, Kevin Wolf, 2019/06/04
- Re: [Qemu-block] [PATCH v2] nvme: add Get/Set Feature Timestamp support, Heitke, Kenneth, 2019/06/04
- Re: [Qemu-block] [PATCH v2] nvme: add Get/Set Feature Timestamp support, Kevin Wolf, 2019/06/05