From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pl1-f172.google.com (mail-pl1-f172.google.com [209.85.214.172]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 68A623AFD06 for ; Fri, 17 Jul 2026 11:25:40 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.214.172 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1784287541; cv=none; b=A+AMFb63+igNhJvTJWjqsb7gZonSm3OEK4JtMBw0hcX4IdZ2Sca9kVSezF7VAVtCozzitDFN2rbwn5f6H+n3wQm4ZfeaHltPTWAcgb3LP8gEf93kYzGd7tB5c+KSDZWjAleOMi471CzpAiL1JDSp6Dc8Wo6NZe3FfcJKxVtNEk8= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1784287541; c=relaxed/simple; bh=b13YWFUSi2ABCS9GNvKmbC8JRTyS6lZ2xmdiWv/pWXY=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=VhYDSxb9C6AerKEiIKnP6L06Z81IriqQDZy0d4Ba/ooKFoGQs/7GBvggIjUZ1Et1Y/9z2axB0E2pHyKNqK2V5Tb3Ro/X27jUAwfNsyFcUYjf/2wID53QuP4BytCKvTIa+Tibl1zz/gicKd9EPeEphZ8jQXk8BACpT56HFvlpUrQ= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=mIOcj4Ss; arc=none smtp.client-ip=209.85.214.172 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="mIOcj4Ss" Received: by mail-pl1-f172.google.com with SMTP id d9443c01a7336-2ceae1ed204so52355995ad.0 for ; Fri, 17 Jul 2026 04:25:40 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1784287540; x=1784892340; darn=vger.kernel.org; h=content-transfer-encoding:content-type:in-reply-to:from :content-language:references:cc:to:subject:user-agent:mime-version :date:message-id:from:to:cc:subject:date:message-id:reply-to :content-type; bh=pVuierYlugSu0sUY+XJcj2RzObrDWMBeCLTVAQKDCbA=; b=mIOcj4SsowtR8l7Sy2SghhLQEv4P+821zCh2DDE6VFx1nwQYpFmP8/OvLFU+yeiHWn bADl6Eh03bVk6mBZupCOBp/GAT/jXyazu5PZq3Kyr05GmKkCzLlp9GVBHDPJlciYJSEW M3ZwY3m8oeWosIdOWu7oQIanUXPb64BJU7rSEJmH8l/R1EWaO8xGWR7IzXpvDNRv7qbR nBrkQsoOXpbXC8wB7xkq9eYmuhUKrXrXKuDBJfPas+CCUVRTFpBBJ+tO7TNn51cIrzRe LfOF4kAZT5Qe+BqKreQ72qH5+s6XJtajTRwTCQhwaet6BY06Qxav6Pgf12qEda3sIyax 9FnQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1784287540; x=1784892340; h=content-transfer-encoding:content-type:in-reply-to:from :content-language:references:cc:to:subject:user-agent:mime-version :date:message-id:x-gm-gg:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to:content-type; bh=pVuierYlugSu0sUY+XJcj2RzObrDWMBeCLTVAQKDCbA=; b=rhzAifoq5lErDUm4PwDelFdJAx/Gc7nLTysgo3yRrCitRzuf8lTXWqI/T+/eLDS9Tg eM1+T5LMKRFPDSufa8BuN0wJViQI8plhOqPRn1Mk71bbdps13MfBKNY2fPnCy/FnuCLS 1PtollRUDLAkyPrgrxbMYmrTgP1HfrNGFWpQENud/qtnG9ZkWzxfNLlFOlb8LmR39Gjj vZXtFPUgJ/U6s/Q9kT/H0IrPf44iLMooLqD8uu7FkeL8ukXvIfY5mCvv1iZfYKm7/Rfs 19k3l1u79zl4/AjZjeq6/u1NbLindIzoImbhTbFK3nJrB48Th3BDTpp/YHgGmxXRCQHy 2dqA== X-Forwarded-Encrypted: i=1; AHgh+RobVL3Q99giZZOXa67SavM536J1Cp7HYBG9P6vcOIPcxnYpz96viNSr/YSsA0bGgr6bbdf3JRq1yEM2evw=@vger.kernel.org X-Gm-Message-State: AOJu0Yy7DyOYRIQWuYJq5XMObWkDtf+ZxyMVFJ7D95ime0pW0Pt3+afx NjB42Jw5MLTWBEJmZrx22i0s2nsX/QjT8I5xNAwzLGtSJ8G5uL2eOUKc X-Gm-Gg: AfdE7clt8fbheFATDctcycaslietXDu2aagXOOHoYNwviirBvmjO7BHDkiqXKMW4sf9 6RPAY6bg7tQbomR2thu+D/XkmkzYiDBRSiLYLtluZ+a2lLXmebpf+KOyxhabAnbCKpOG1mJJa6H QOvtXmGK4X5QQD6YH58IrNoqVPSS+VqxPOPPsFGv6r6u2YllQt+WrqfgZ+RlFANzqp+8mLSEuE9 ZjWLTmiLOHgXp6rpsaWX2DBpD9QhHnO57+tosmxCErwze0OEW0GOAchhwWmCJ0F2l2BAVhYrHZF 78YZ4+LvNcCjfzWKhobDYIU+i7UxJwri61DdDGaBBFaTZibQcMbnBrjhm5+ncgSC8EOnIvISwP7 y1tl9Bx8685SkZLQ2uTnEheF4Y8K1DDZ6zRAHM6hp6wPjZTN50rfCggx6/VBRBEGlU67YJxvrRr WJH47KMN3+btngr8kL2M2vSPebLL+zilo9r4/6js5vSYPrKvZA5M1keLQSpz5nYTlAcwPyLNLbd A== X-Received: by 2002:a17:903:1b2e:b0:2ca:281:27f9 with SMTP id d9443c01a7336-2cf349978f5mr22647765ad.27.1784287539501; Fri, 17 Jul 2026 04:25:39 -0700 (PDT) Received: from ?IPV6:2400:79e0:1203:64b0:6cf3:a642:5711:319? ([2400:79e0:1203:64b0:6cf3:a642:5711:319]) by smtp.gmail.com with ESMTPSA id d9443c01a7336-2cf344d80e6sm8978785ad.29.2026.07.17.04.25.36 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Fri, 17 Jul 2026 04:25:38 -0700 (PDT) Message-ID: <3a711d12-5184-45b0-a4d5-2d02a17d8b3b@gmail.com> Date: Fri, 17 Jul 2026 19:25:35 +0800 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH] blk-throttle: remove dead field last_check_time from throtl_grp To: Tao Cui , axboe@kernel.dk, linux-block@vger.kernel.org Cc: tj@kernel.org, josef@toxicpanda.com, yukuai3@huawei.com, cgroups@vger.kernel.org, linux-kernel@vger.kernel.org, Tao Cui References: <20260717054855.2018578-1-cui.tao@linux.dev> Content-Language: en-US From: Tang Yizhou In-Reply-To: <20260717054855.2018578-1-cui.tao@linux.dev> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit On 17/7/26 1:48 pm, Tao Cui wrote: > From: Tao Cui > > The last_check_time field in throtl_grp was used by the > CONFIG_BLK_DEV_THROTTLING_LOW mechanism (in throtl_upgrade_check() and > the downgrade logic) to timestamp the last upgrade/downgrade check. > > Commit bf20ab538c81 ("blk-throttle: remove CONFIG_BLK_DEV_THROTTLING_LOW") > removed all five of its uses in blk-throttle.c and the surrounding LOW > fields (latency_target, last_finish_time, checked_last_finish_time, > avg_idletime, idletime_threshold, bio_cnt, bad_bio_cnt, > bio_cnt_reset_time), but missed the field definition itself. > > It has been a dead field since then: zero references in blk-throttle.c > and the whole tree. Remove it. > > Signed-off-by: Tao Cui > --- > block/blk-throttle.h | 2 -- > 1 file changed, 2 deletions(-) > > diff --git a/block/blk-throttle.h b/block/blk-throttle.h > index 9d7a42c039a1..1b5775771e07 100644 > --- a/block/blk-throttle.h > +++ b/block/blk-throttle.h > @@ -120,8 +120,6 @@ struct throtl_grp { > int64_t bytes_disp[2]; > int io_disp[2]; > > - unsigned long last_check_time; > - > /* When did we start a new slice */ > unsigned long slice_start[2]; > unsigned long slice_end[2]; Reviewed-by: Tang Yizhou -- Best Regards, Yi