lkml.org 
[lkml]   [2020]   [Sep]   [1]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 10/27] blk-iocost: make ioc_now->now and ioc->period_at 64bit
    Date
    They are in microseconds and wrap in around 1.2 hours with u32. While
    unlikely, confusions from wraparounds are still possible. We aren't saving
    anything meaningful by keeping these u32. Let's make them u64.

    Signed-off-by: Tejun Heo <tj@kernel.org>
    ---
    block/blk-iocost.c | 4 ++--
    1 file changed, 2 insertions(+), 2 deletions(-)

    diff --git a/block/blk-iocost.c b/block/blk-iocost.c
    index 00c5a3ad2b5b..dc72cd965837 100644
    --- a/block/blk-iocost.c
    +++ b/block/blk-iocost.c
    @@ -409,7 +409,7 @@ struct ioc {
    atomic64_t vtime_rate;

    seqcount_spinlock_t period_seqcount;
    - u32 period_at; /* wallclock starttime */
    + u64 period_at; /* wallclock starttime */
    u64 period_at_vtime; /* vtime starttime */

    atomic64_t cur_period; /* inc'd each period */
    @@ -508,7 +508,7 @@ struct ioc_cgrp {

    struct ioc_now {
    u64 now_ns;
    - u32 now;
    + u64 now;
    u64 vnow;
    u64 vrate;
    };
    --
    2.26.2
    \
     
     \ /
      Last update: 2020-09-01 20:56    [W:3.165 / U:0.032 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site