From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 5EE6F31ED83 for ; Sat, 31 Jan 2026 02:26:33 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1769826393; cv=none; b=Achnngp3q5qvkTtSeYGSKQOxuVvsyNaTA6ik+Xh2fpB5y18lLXqFi685c/J4q/Gjm6rpVibof5spptzcRsnDkz705DpHBse825AT+9iw3AWvugbKav0uM0IJhXvcpuyf2uFSUnccoaSgaZ33WQQvggzjWC004tH3187UGjcYuhg= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1769826393; c=relaxed/simple; bh=sVoBYWt86eN4DNlfx895Vh0r1E93SiOa1EZHZwaUzgo=; h=From:To:Cc:Subject:Date:Message-Id:In-Reply-To:References: MIME-Version; b=ax0W8VxNmXI7VSoYpzifgXKmp+wU2Y6iS4H9QUfwfROhdbW+caZsBqoiCezk+IlaJq6a94mOlhhLS6tw47FZvW/WVCvsoqEkNDlsTgoAQEpMa94dWVC6l/YtBEqt2KjDPFo8QalGNDIAKTJu6+iRV51kWy1Kk35gOiYR0qPfDeI= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=E7OmvTuP; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="E7OmvTuP" Received: by smtp.kernel.org (Postfix) with ESMTPSA id AE767C116D0; Sat, 31 Jan 2026 02:26:31 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1769826392; bh=sVoBYWt86eN4DNlfx895Vh0r1E93SiOa1EZHZwaUzgo=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=E7OmvTuPI4a9DiB+hdLXzS39PHbq+2fLhbm1C/n7b+XZCBGvAHDtxJeNh5PYiuzZE bzyJ17vHiFXcVSWH/fcOMvzp4J4BjZVm3eeiqI9zqsh8uC5rL0DVaZ2yRoMSkPrrP0 fHvAONmd7kdJFqUgcB9JgevkPCBXatxGHiQV06eHKYZSYT6qAAFJgrgebOE7qcfbfj ihl4NDUvHcJayNXnO8n1joRyJK/X0e0vj7Ja++DrTCv2GhntW15UxesmWfab4LBjRj sQ5IHE2ex5bXs2VvORXEa+1tyW1yJiPTPQoNP5cPf2IVc9ZdwILOAAoLP939b40An9 +ehtfasEYAvKQ== From: Chao Yu To: jaegeuk@kernel.org Cc: linux-f2fs-devel@lists.sourceforge.net, linux-kernel@vger.kernel.org, Chao Yu Subject: [PATCH 2/2] f2fs: introduce trace_f2fs_priority_update Date: Fri, 30 Jan 2026 21:28:09 +0800 Message-Id: <20260130132809.59707-2-chao@kernel.org> X-Mailer: git-send-email 2.40.1 In-Reply-To: <20260130132809.59707-1-chao@kernel.org> References: <20260130132809.59707-1-chao@kernel.org> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit This patch introduces two new tracepoints for debug purpose. Signed-off-by: Chao Yu --- fs/f2fs/checkpoint.c | 17 +++++++---- include/trace/events/f2fs.h | 57 +++++++++++++++++++++++++++++++++++++ 2 files changed, 69 insertions(+), 5 deletions(-) diff --git a/fs/f2fs/checkpoint.c b/fs/f2fs/checkpoint.c index 2f5a03e29d0b..4afa5d9a19fc 100644 --- a/fs/f2fs/checkpoint.c +++ b/fs/f2fs/checkpoint.c @@ -130,9 +130,13 @@ static void uplift_priority(struct f2fs_rwsem *sem, struct f2fs_lock_context *lc return; set_user_nice(current, lc->new_nice); lc->need_restore = true; + + trace_f2fs_priority_uplift(sem->sbi, sem->name, is_write, current, + NICE_TO_PRIO(lc->orig_nice), NICE_TO_PRIO(lc->new_nice)); } -static void restore_priority(struct f2fs_lock_context *lc) +static void restore_priority(struct f2fs_rwsem *sem, struct f2fs_lock_context *lc, + bool is_write) { if (!lc->need_restore) return; @@ -140,6 +144,9 @@ static void restore_priority(struct f2fs_lock_context *lc) if (task_nice(current) != lc->new_nice) return; set_user_nice(current, lc->orig_nice); + + trace_f2fs_priority_restore(sem->sbi, sem->name, is_write, current, + NICE_TO_PRIO(lc->orig_nice), NICE_TO_PRIO(lc->new_nice)); } void f2fs_down_read_trace(struct f2fs_rwsem *sem, struct f2fs_lock_context *lc) @@ -153,7 +160,7 @@ int f2fs_down_read_trylock_trace(struct f2fs_rwsem *sem, struct f2fs_lock_contex { uplift_priority(sem, lc, false); if (!f2fs_down_read_trylock(sem)) { - restore_priority(lc); + restore_priority(sem, lc, false); return 0; } trace_lock_elapsed_time_start(sem, lc); @@ -163,7 +170,7 @@ int f2fs_down_read_trylock_trace(struct f2fs_rwsem *sem, struct f2fs_lock_contex void f2fs_up_read_trace(struct f2fs_rwsem *sem, struct f2fs_lock_context *lc) { f2fs_up_read(sem); - restore_priority(lc); + restore_priority(sem, lc, false); trace_lock_elapsed_time_end(sem, lc, false); } @@ -178,7 +185,7 @@ int f2fs_down_write_trylock_trace(struct f2fs_rwsem *sem, struct f2fs_lock_conte { uplift_priority(sem, lc, true); if (!f2fs_down_write_trylock(sem)) { - restore_priority(lc); + restore_priority(sem, lc, true); return 0; } trace_lock_elapsed_time_start(sem, lc); @@ -188,7 +195,7 @@ int f2fs_down_write_trylock_trace(struct f2fs_rwsem *sem, struct f2fs_lock_conte void f2fs_up_write_trace(struct f2fs_rwsem *sem, struct f2fs_lock_context *lc) { f2fs_up_write(sem); - restore_priority(lc); + restore_priority(sem, lc, true); trace_lock_elapsed_time_end(sem, lc, true); } diff --git a/include/trace/events/f2fs.h b/include/trace/events/f2fs.h index c3b6b509472f..9364e6775562 100644 --- a/include/trace/events/f2fs.h +++ b/include/trace/events/f2fs.h @@ -2525,6 +2525,63 @@ TRACE_EVENT(f2fs_lock_elapsed_time, __entry->other_time) ); +DECLARE_EVENT_CLASS(f2fs_priority_update, + + TP_PROTO(struct f2fs_sb_info *sbi, enum f2fs_lock_name lock_name, + bool is_write, struct task_struct *p, int orig_prio, + int new_prio), + + TP_ARGS(sbi, lock_name, is_write, p, orig_prio, new_prio), + + TP_STRUCT__entry( + __field(dev_t, dev) + __array(char, comm, TASK_COMM_LEN) + __field(pid_t, pid) + __field(unsigned int, lock_name) + __field(bool, is_write) + __field(int, orig_prio) + __field(int, new_prio) + ), + + TP_fast_assign( + __entry->dev = sbi->sb->s_dev; + memcpy(__entry->comm, p->comm, TASK_COMM_LEN); + __entry->pid = p->pid; + __entry->lock_name = lock_name; + __entry->is_write = is_write; + __entry->orig_prio = orig_prio; + __entry->new_prio = new_prio; + ), + + TP_printk("dev = (%d,%d), comm: %s, pid: %d, lock_name: %s, " + "lock_type: %s, orig_prio: %d, new_prio: %d", + show_dev(__entry->dev), + __entry->comm, + __entry->pid, + show_lock_name(__entry->lock_name), + __entry->is_write ? "wlock" : "rlock", + __entry->orig_prio, + __entry->new_prio) +); + +DEFINE_EVENT(f2fs_priority_update, f2fs_priority_uplift, + + TP_PROTO(struct f2fs_sb_info *sbi, enum f2fs_lock_name lock_name, + bool is_write, struct task_struct *p, int orig_prio, + int new_prio), + + TP_ARGS(sbi, lock_name, is_write, p, orig_prio, new_prio) +); + +DEFINE_EVENT(f2fs_priority_update, f2fs_priority_restore, + + TP_PROTO(struct f2fs_sb_info *sbi, enum f2fs_lock_name lock_name, + bool is_write, struct task_struct *p, int orig_prio, + int new_prio), + + TP_ARGS(sbi, lock_name, is_write, p, orig_prio, new_prio) +); + #endif /* _TRACE_F2FS_H */ /* This part must be outside protection */ -- 2.40.1