qemu-trivial
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH] hw/block/nvme: Simplify timestamp sum


From: Laurent Vivier
Subject: Re: [PATCH] hw/block/nvme: Simplify timestamp sum
Date: Mon, 12 Oct 2020 14:24:48 +0200
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.11.0

Le 02/10/2020 à 09:57, Philippe Mathieu-Daudé a écrit :
> As the 'timestamp' variable is declared as a 48-bit bitfield,
> we do not need to wrap the sum result.
> 
> Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
> ---
>  hw/block/nvme.c | 7 +------
>  1 file changed, 1 insertion(+), 6 deletions(-)
> 
> diff --git a/hw/block/nvme.c b/hw/block/nvme.c
> index 63078f6009..44fa5b9076 100644
> --- a/hw/block/nvme.c
> +++ b/hw/block/nvme.c
> @@ -1280,12 +1280,7 @@ static inline uint64_t nvme_get_timestamp(const 
> NvmeCtrl *n)
>  
>      union nvme_timestamp ts;
>      ts.all = 0;
> -
> -    /*
> -     * If the sum of the Timestamp value set by the host and the elapsed
> -     * time exceeds 2^48, the value returned should be reduced modulo 2^48.
> -     */
> -    ts.timestamp = (n->host_timestamp + elapsed_time) & 0xffffffffffff;
> +    ts.timestamp = n->host_timestamp + elapsed_time;
>  
>      /* If the host timestamp is non-zero, set the timestamp origin */
>      ts.origin = n->host_timestamp ? 0x01 : 0x00;
> 

Applied to my trivial-patches branch.

Thanks,
Laurent



reply via email to

[Prev in Thread] Current Thread [Next in Thread]