From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by smtp.lore.kernel.org (Postfix) with ESMTP id B97B8C5AD49 for ; Thu, 29 May 2025 11:34:14 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id EF9E66B0140; Thu, 29 May 2025 07:34:12 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id ED3EB6B0142; Thu, 29 May 2025 07:34:12 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id D99F76B0143; Thu, 29 May 2025 07:34:12 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0017.hostedemail.com [216.40.44.17]) by kanga.kvack.org (Postfix) with ESMTP id B707F6B0140 for ; Thu, 29 May 2025 07:34:12 -0400 (EDT) Received: from smtpin02.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay01.hostedemail.com (Postfix) with ESMTP id 6D4B21C9385 for ; Thu, 29 May 2025 11:34:12 +0000 (UTC) X-FDA: 83495736744.02.1BD201F Received: from mailout1.samsung.com (mailout1.samsung.com [203.254.224.24]) by imf28.hostedemail.com (Postfix) with ESMTP id 65190C0004 for ; Thu, 29 May 2025 11:34:09 +0000 (UTC) Authentication-Results: imf28.hostedemail.com; dkim=pass header.d=samsung.com header.s=mail20170921 header.b=cKhvTnhy; dmarc=pass (policy=none) header.from=samsung.com; spf=pass (imf28.hostedemail.com: domain of kundan.kumar@samsung.com designates 203.254.224.24 as permitted sender) smtp.mailfrom=kundan.kumar@samsung.com ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1748518450; a=rsa-sha256; cv=none; b=SlsnTTHQLYbm6b5ulSY3JuPStBbL0aLwhgfUk+iDkqh6+gIR+R+TFSk5rapYVR47bv/2hy 9QR3sRirj6VednljW8LyBFsSiZa3YgNWdOMpgz845My//x4kV4FS97BHk64dhQCwYFxsV1 mwOjgXxkqAi6OMlenlxULeUX+fVGdwU= ARC-Authentication-Results: i=1; imf28.hostedemail.com; dkim=pass header.d=samsung.com header.s=mail20170921 header.b=cKhvTnhy; dmarc=pass (policy=none) header.from=samsung.com; spf=pass (imf28.hostedemail.com: domain of kundan.kumar@samsung.com designates 203.254.224.24 as permitted sender) smtp.mailfrom=kundan.kumar@samsung.com ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1748518450; h=from:from:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references:dkim-signature; bh=2pPW61Ci7mZ49EXMnfxOL8Bt7rnm4f37CanAIhRplY8=; b=1AKppQdNb0nzdYGJ64BQXqNUDA6Wsj6BGm/Cxa6KIfUunlaCGRCKNNLlM/DTPvDC4G3ctS bP7LY0vN2I4A2v9K5wpm8BhwsN4HV6LotLbiom/Hy4zWJS35ejXfVkK5FiffupvD+KyknE hrKjsBLmKyLisSlVXv4dEBtSx6KlBLQ= Received: from epcas5p3.samsung.com (unknown [182.195.41.41]) by mailout1.samsung.com (KnoxPortal) with ESMTP id 20250529113406epoutp01b850350e6d00159f0fa6f8e0383502e5~D-EM9WyeI2841628416epoutp01e for ; Thu, 29 May 2025 11:34:06 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 mailout1.samsung.com 20250529113406epoutp01b850350e6d00159f0fa6f8e0383502e5~D-EM9WyeI2841628416epoutp01e DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=samsung.com; s=mail20170921; t=1748518446; bh=2pPW61Ci7mZ49EXMnfxOL8Bt7rnm4f37CanAIhRplY8=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=cKhvTnhyi2aD1jUbX8cC3lmov481UmPjElcsONOBAU9KoVHX4Y5oUtmQYIyyrjamo ZsG/HmjkgT/dM7upoItfFr8gAwFXmwXiCt9T5zAdRFuqlHPERPZpm53hh1D57HV3PN JZVn/Drf3TYwYdU9UGo6wYP4uWiMM3G4/A1k8CMk= Received: from epsnrtp04.localdomain (unknown [182.195.42.156]) by epcas5p1.samsung.com (KnoxPortal) with ESMTPS id 20250529113405epcas5p1f9353231e6995374c32ea1fd5aa6c3fa~D-EMbdi7T0737307373epcas5p16; Thu, 29 May 2025 11:34:05 +0000 (GMT) Received: from epcas5p3.samsung.com (unknown [182.195.38.179]) by epsnrtp04.localdomain (Postfix) with ESMTP id 4b7PQh1x10z6B9m6; Thu, 29 May 2025 11:34:04 +0000 (GMT) Received: from epsmtrp1.samsung.com (unknown [182.195.40.13]) by epcas5p3.samsung.com (KnoxPortal) with ESMTPA id 20250529113249epcas5p38b29d3c6256337eadc2d1644181f9b74~D-DFraPsh0440004400epcas5p3V; Thu, 29 May 2025 11:32:49 +0000 (GMT) Received: from epsmgmcp1.samsung.com (unknown [182.195.42.82]) by epsmtrp1.samsung.com (KnoxPortal) with ESMTP id 20250529113249epsmtrp126243e34c6c4da550db410654938d7fd~D-DFqNfcq2108121081epsmtrp1Q; Thu, 29 May 2025 11:32:49 +0000 (GMT) X-AuditID: b6c32a52-40bff70000004c16-07-683845e186f8 Received: from epsmtip2.samsung.com ( [182.195.34.31]) by epsmgmcp1.samsung.com (Symantec Messaging Gateway) with SMTP id AB.82.19478.1E548386; Thu, 29 May 2025 20:32:49 +0900 (KST) Received: from localhost.localdomain (unknown [107.99.41.245]) by epsmtip2.samsung.com (KnoxPortal) with ESMTPA id 20250529113245epsmtip28923533fe9c5db9468d6af7c110b6cf2~D-DByXQe82208622086epsmtip2v; Thu, 29 May 2025 11:32:45 +0000 (GMT) From: Kundan Kumar To: jaegeuk@kernel.org, chao@kernel.org, viro@zeniv.linux.org.uk, brauner@kernel.org, jack@suse.cz, miklos@szeredi.hu, agruenba@redhat.com, trondmy@kernel.org, anna@kernel.org, akpm@linux-foundation.org, willy@infradead.org, mcgrof@kernel.org, clm@meta.com, david@fromorbit.com, amir73il@gmail.com, axboe@kernel.dk, hch@lst.de, ritesh.list@gmail.com, djwong@kernel.org, dave@stgolabs.net, p.raghav@samsung.com, da.gomez@samsung.com Cc: linux-f2fs-devel@lists.sourceforge.net, linux-fsdevel@vger.kernel.org, gfs2@lists.linux.dev, linux-nfs@vger.kernel.org, linux-mm@kvack.org, gost.dev@samsung.com, Kundan Kumar , Anuj Gupta Subject: [PATCH 08/13] writeback: add support to collect stats for all writeback ctxs Date: Thu, 29 May 2025 16:44:59 +0530 Message-Id: <20250529111504.89912-9-kundan.kumar@samsung.com> X-Mailer: git-send-email 2.25.1 In-Reply-To: <20250529111504.89912-1-kundan.kumar@samsung.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFnrGIsWRmVeSWpSXmKPExsWy7bCSvO5DV4sMg+9PJSy2rdvNbjFn/Ro2 iwvrVjNatO78z2LRNOEvs8Xqu/1sFq8Pf2K0OD31LJPFlkv2Fu8vb2OyWH1zDaPFlmP3GC0u P+Gz2D39H6vFzQM7mSxWrj7KZDF7ejOTxZP1s5gttn75ympxaZG7xZ69J1ks7q35z2px4cBp VosbE54yWjzbvZHZ4vPSFnaLg6c62C0+zQUacv7vcVaL3z/msDnIeZxaJOGxc9Zddo/NK7Q8 Lp8t9di0qpPNY9OnSeweJ2b8ZvF4sXkmo8fuBZ+ZPHbfbGDzOHexwuP9vqtsHn1bVjF6TJ1d 73FmwRF2jxXTLjIFCEVx2aSk5mSWpRbp2yVwZbx4sZW94IJ8xZwJZ9kaGFdJdjFyckgImEjc fbSeuYuRi0NIYDujxIW1M1kgEjISu+/uZIWwhSVW/nvODlH0kVHi7vU3QA4HB5uArsSPplCQ uIjATWaJc2fPgDUwC/xjlNj9SgfEFhYIlZi1qBVsKIuAqsTkO8+ZQGxeAVuJidOXQi2Tl5h5 6Ts7iM0pYCexaMlXRhBbCKhm6c21LBD1ghInZz5hgZgvL9G8dTbzBEaBWUhSs5CkFjAyrWIU TS0ozk3PTS4w1CtOzC0uzUvXS87P3cQITgpaQTsYl63/q3eIkYmD8RCjBAezkghvk71ZhhBv SmJlVWpRfnxRaU5q8SFGaQ4WJXFe5ZzOFCGB9MSS1OzU1ILUIpgsEwenVAMTd4b24eTzz/J/ X/sTeNk1ZMqq3InvPv6afuN/7uG6s+szXC4GFjBHSK2Q/+Di/zv7GnfPBgcfga2Jp/n6Q+R+ OzTJqG2pqvDR0G6QZEr1fHlz8tqnX48ldeXdPb9mqePyVzuK+m8pissWRtZ4vv1+6xvvOvPt a9PXK30We66z4MbrhY1lbvtuTXq2YMrV65r/K5ay/D70+f5V+RNLbBb3+AcmftzG6FLzpFVa XUvd137BtLawFYLnWK4XTOy48pxf1alwUdAdh/oXdUei3Lii65jz2AymSRiV8AssTt9XKF2Z tPxlQ2KFcP75/qSbqr08783UtpUcYTd757Y44s2zQJM/MXzzJnFdvN647d9FJZbijERDLeai 4kQAQW8tcXkDAAA= X-CMS-MailID: 20250529113249epcas5p38b29d3c6256337eadc2d1644181f9b74 X-Msg-Generator: CA Content-Type: text/plain; charset="utf-8" X-Sendblock-Type: REQ_APPROVE CMS-TYPE: 105P cpgsPolicy: CPGSC10-542,Y X-CFilter-Loop: Reflected X-CMS-RootMailID: 20250529113249epcas5p38b29d3c6256337eadc2d1644181f9b74 References: <20250529111504.89912-1-kundan.kumar@samsung.com> X-Rspamd-Server: rspam08 X-Rspamd-Queue-Id: 65190C0004 X-Stat-Signature: 46k81c8rbzdcukkp4qyur8eyee9bnfhj X-Rspam-User: X-HE-Tag: 1748518449-932507 X-HE-Meta: U2FsdGVkX1+Lmu3K0o5iWb7yiSDyUkHrE82h3oyd7aVb5Kf9wq46BZgPMqGxNMy+uXrRFpeTpGkcc/wlq6T3dH2hfBU93bTrag1w0lJbSgEFaFsdttI6mLNBBxQ0Ly85YD8speU5wXIZ5zCIqAtwj5MetIczDXRfGbY1qkxcUrADTeFgnU1y2bSIk5LZfHsOiXBrrm4YDUeRQXZS3DMYQjNL1krwGbGMA1ieKZ5+WKnUsH0pcamX24BG1zMwjq7icQu0vMhRf7x/AScatg1QBKwG4W+TY55AVcJI3SaCYz2G1kRZFVzoPH9xlZskm4kpXSzban2DSvyxwtarzGfwNTZaJnW/P2eDAE8G22OsgkCUjyAB0JnGpP4g+S4oyf47XSyOI4/XUjm2Rl4AWNvej5v4DI786/uw8iqnThRiFd1t9tVaOFxkHCGMl2w55ZvJalGAo1wsw+RaGn6Pl4TB8xLV5S3W4bT6okUHoxRFhuZyzZXa8d1fobAghNVrQ2CRQsh8Tj0DXFhRjPuDs9bp8romdCgBGmY9XVmmhmSBdR8FKx6omBA4xIEq2N25x0bwMCAZqPrYOXxWVLLxdBF/wiDEv1JbiCGnUd7B3k480ulkicm9R2g8rWZIFnCVl17nQqIIfA6fYrwDEryaRXtkTw0Byuzp2yhMceTHzs1Hf2HjBVpR8n9HrRbFlFdBGRRN982b+TItI9neH8C5g93Cbo4gInjkFJT9PHMrdtEewTXBUYduxmmbIY2H09ZyJT79tc8t7mXAcBRwapL6hfSBOGITfOXFqECQ0SMYQ+91dMu05DAWDEz+O1euMkbUZdKWU2ip91LSRmDVS5v2zK1JtEkUMhbESwZOTdIBH5SlXDE6bTQ0m7DYPWYBzUF5rlVHAy+MYJ+n+IUAczOh2ueaOMN9uGdeAtAz5MU3U83jjiZjP4M8kSVYLEnzlq6rKFWmx3TqHAS4zSNKY8NMp9k 9daRkMMm Bnhm4PBfw4wYefuxtXQlfxyDooh/8zL4aY+VcdC3sDAkOWc+My9hmLC9I84J4Ge6yfQ2HT1WNqTzBJsWRDi/NNtnydYOaXsFsb9aAXW2kd7F58i6rO+ZxQGU4aWnf1fnCdC9qdVf1WFa4wrSdwu/vNEksle3ecEXZr4wH X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: List-Subscribe: List-Unsubscribe: Modified stats collection to collect stats for all the writeback contexts within a bdi. Signed-off-by: Kundan Kumar Signed-off-by: Anuj Gupta --- mm/backing-dev.c | 72 ++++++++++++++++++++++++++++-------------------- 1 file changed, 42 insertions(+), 30 deletions(-) diff --git a/mm/backing-dev.c b/mm/backing-dev.c index 5479e2d34160..d416122e2914 100644 --- a/mm/backing-dev.c +++ b/mm/backing-dev.c @@ -50,6 +50,7 @@ struct wb_stats { unsigned long nr_written; unsigned long dirty_thresh; unsigned long wb_thresh; + unsigned long state; }; static struct dentry *bdi_debug_root; @@ -81,6 +82,7 @@ static void collect_wb_stats(struct wb_stats *stats, stats->nr_dirtied += wb_stat(wb, WB_DIRTIED); stats->nr_written += wb_stat(wb, WB_WRITTEN); stats->wb_thresh += wb_calc_thresh(wb, stats->dirty_thresh); + stats->state |= wb->state; } #ifdef CONFIG_CGROUP_WRITEBACK @@ -89,22 +91,27 @@ static void bdi_collect_stats(struct backing_dev_info *bdi, struct wb_stats *stats) { struct bdi_writeback *wb; + struct bdi_writeback_ctx *bdi_wb_ctx; rcu_read_lock(); - list_for_each_entry_rcu(wb, &bdi->wb_ctx_arr[0]->wb_list, bdi_node) { - if (!wb_tryget(wb)) - continue; + for_each_bdi_wb_ctx(bdi, bdi_wb_ctx) + list_for_each_entry_rcu(wb, &bdi_wb_ctx->wb_list, bdi_node) { + if (!wb_tryget(wb)) + continue; - collect_wb_stats(stats, wb); - wb_put(wb); - } + collect_wb_stats(stats, wb); + wb_put(wb); + } rcu_read_unlock(); } #else static void bdi_collect_stats(struct backing_dev_info *bdi, struct wb_stats *stats) { - collect_wb_stats(stats, &bdi->wb_ctx_arr[0]->wb); + struct bdi_writeback_ctx *bdi_wb_ctx; + + for_each_bdi_wb_ctx(bdi, bdi_wb_ctx) + collect_wb_stats(stats, &bdi_wb_ctx->wb); } #endif @@ -150,7 +157,7 @@ static int bdi_debug_stats_show(struct seq_file *m, void *v) stats.nr_io, stats.nr_more_io, stats.nr_dirty_time, - !list_empty(&bdi->bdi_list), bdi->wb_ctx_arr[0]->wb.state); + !list_empty(&bdi->bdi_list), stats.state); return 0; } @@ -195,35 +202,40 @@ static int cgwb_debug_stats_show(struct seq_file *m, void *v) { struct backing_dev_info *bdi = m->private; struct bdi_writeback *wb; + struct bdi_writeback_ctx *bdi_wb_ctx; unsigned long background_thresh; unsigned long dirty_thresh; + struct wb_stats stats; global_dirty_limits(&background_thresh, &dirty_thresh); + stats.dirty_thresh = dirty_thresh; rcu_read_lock(); - list_for_each_entry_rcu(wb, &bdi->wb_ctx_arr[0]->wb_list, bdi_node) { - struct wb_stats stats = { .dirty_thresh = dirty_thresh }; - - if (!wb_tryget(wb)) - continue; - - collect_wb_stats(&stats, wb); - - /* - * Calculate thresh of wb in writeback cgroup which is min of - * thresh in global domain and thresh in cgroup domain. Drop - * rcu lock because cgwb_calc_thresh may sleep in - * cgroup_rstat_flush. We can do so here because we have a ref. - */ - if (mem_cgroup_wb_domain(wb)) { - rcu_read_unlock(); - stats.wb_thresh = min(stats.wb_thresh, cgwb_calc_thresh(wb)); - rcu_read_lock(); + for_each_bdi_wb_ctx(bdi, bdi_wb_ctx) { + list_for_each_entry_rcu(wb, &bdi_wb_ctx->wb_list, bdi_node) { + if (!wb_tryget(wb)) + continue; + + collect_wb_stats(&stats, wb); + + /* + * Calculate thresh of wb in writeback cgroup which is + * min of thresh in global domain and thresh in cgroup + * domain. Drop rcu lock because cgwb_calc_thresh may + * sleep in cgroup_rstat_flush. We can do so here + * because we have a ref. + */ + if (mem_cgroup_wb_domain(wb)) { + rcu_read_unlock(); + stats.wb_thresh = min(stats.wb_thresh, + cgwb_calc_thresh(wb)); + rcu_read_lock(); + } + + wb_stats_show(m, wb, &stats); + + wb_put(wb); } - - wb_stats_show(m, wb, &stats); - - wb_put(wb); } rcu_read_unlock(); -- 2.25.1