From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wr1-f50.google.com (mail-wr1-f50.google.com [209.85.221.50]) (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 50D762FCC1E for ; Mon, 27 Oct 2025 08:47:13 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.221.50 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1761554835; cv=none; b=Er2zdwAx8PeLTJtw+9yoT9N7bR9F2D4pvmK2Nj0oaZWvRFiisWWPDbI3lJ9iidZA7b9PtNkK5EbL7aha/iN27bK4hC3sfDnYQiqeH8GVKsq5trWhNEN1VD6zSwL73De9mGuFfZqkVw7fG6xzt+8FUE5HlYBPHZ6PTwYObSyJXus= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1761554835; c=relaxed/simple; bh=L6yB8zi8REP/swCl7Av17BttU5r/j+BG5EnDaugEi4w=; h=From:Date:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=KUaD1jFhGRUC1CryyxRupPamshNEMtuG9fnwxL/TJE5qdDVH5BvK+c1ybSy0HyoqLnaR8diExWfoZXTzyOwF1d+eyTnST15xphDs2subr6Q1Gmi8r28X1HkQpScR/AKu3YGL+407j9SleBVo1EmHZC6y1t430hTi2tgYJEEOUT0= 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=HZYzh/uv; arc=none smtp.client-ip=209.85.221.50 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="HZYzh/uv" Received: by mail-wr1-f50.google.com with SMTP id ffacd0b85a97d-42966ce6dbdso3116359f8f.0 for ; Mon, 27 Oct 2025 01:47:13 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1761554831; x=1762159631; darn=vger.kernel.org; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:date:from:from:to:cc:subject:date:message-id:reply-to; bh=llxyMYDYDt1QJiOwvJpk6RQ31U8QgJmQ3C8aeyRJ80o=; b=HZYzh/uvd8ptLT3acA7V+gfirFgNdQwK+XGqfAVEz+7C3TJgKhNumHg0JnbUaYyWYo Ufq0GwI8eM3HX0yJh/JEzkhQQSAd8jvlQ28gLVJiyy5iB9ur+Y+dxWI6ZJ3bLeSmq0L0 kf7mHfCU0ybDQJRFD3Zyexq+7UuGoTfXPAHf58fxWLSiwMjGYfSOs0dlcErrGkryds4z pCZ5klopXEDF1rff+41+66dSXPPZa+dIZsRlGsfBaqdPCc9cmXkjqzF3hf0fQ73HaLj5 lPT7U0i1PFbBOPVFkbiP0ISxnCGhPPUTizjsR6JSYR7IysF+OKPwkFnR/vZmzmc3avBJ RUag== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1761554831; x=1762159631; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:date:from:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=llxyMYDYDt1QJiOwvJpk6RQ31U8QgJmQ3C8aeyRJ80o=; b=vOxmI3m4vGazQ9uPXVEsp5Z46rCLAFjIUvdUjHJhvwmy4AuFyHknEwNLz3Wypf8CM6 W5l5oB5uebjve+YeWcZXtUcy+Jcvrnjg4+/STjZVjNuL4VxgjAtBIY9+TryCHi3nb5/6 dfqRcOQdMzd7QIM+/iheFXTzK53bOuGiDMW6UAxzhATfywb0Ug8c6GX9tKE7ccNxVsC/ 3NeNg4rEJFTN75WJnYXNirL8auxDwg2sD07GbohXVMA9qOjyoO8vr90nstDMZjR2EtMh U4EuDL9ToE9stIWglPMUQlOH9GVRtRRJ/TqSTRs8Tg++sApO/m5OA0oC2eAnc7+smoqt AWTw== X-Forwarded-Encrypted: i=1; AJvYcCXro6ZwoIMbMXRd1XFhU4fFcNeq5thJeTnHT6EAR3zUo/bTGYmILomr5w37/Q8laCP1spid+lRsJpUvF9M+T62eJ5g=@vger.kernel.org X-Gm-Message-State: AOJu0YzqnsYyfxKbhNwBSAdUu5waPyg4/Z5qBputcXCZZ/qcjEpWfMAS G5yOcJP34/aPNtfXCAEeQo9zOHfbRFvkL/gfbD4C73Bqvjk6Ek5RjLFS X-Gm-Gg: ASbGncu3vo0pbSUGtCr2uV7lWXaYWpoy8G5kPbWfz8UBjy2OgJV7ekrR5C3uFFbCMuU YSmeo/8wdZToM46qDb4aAuZ4Y/mpvoHxBHIEJ2cbEAWPu7SdvK7+HkS9yYdFNMkZqjVhBwJ9kn6 e6oE4IVX6rNNhTyTgOUYfbR3yPlIeJYB3wI0/ZaQFARevWUlQRqGbLWeVU+opIQpBhQ+OklSqLR kKTanveRZtgm2QKXZ+9gZKX6Hnn26jwcsaxZk87HnHBfCKtrx7EhfrJm+6fKBoepvDhFhRrKodg 4XBwCh9XglovREC8qgAgv+Ursd2CTmnzUZp6RABPO5NtiESqOYd4YsipAGoQK3wCGkMyG0UvXgK Nh6OWb3QWE4LXiOrRCUfwk/y93/8mAd0Ncoi3d0PE2/qhZX+s92fe+jDogA9n X-Google-Smtp-Source: AGHT+IGmPorRwgMh/ycuOVlsCFCdyzLjqoKlJVftW4lXoTOXrfSlOItutS9zUJ6ECFW5IHrsQ38MIg== X-Received: by 2002:a05:6000:1884:b0:407:7a7:1cb6 with SMTP id ffacd0b85a97d-42704dc41bbmr25937090f8f.55.1761554831396; Mon, 27 Oct 2025 01:47:11 -0700 (PDT) Received: from krava ([2a02:8308:a00c:e200::b44f]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-475dd489e6dsm123106415e9.6.2025.10.27.01.47.10 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 27 Oct 2025 01:47:11 -0700 (PDT) From: Jiri Olsa X-Google-Original-From: Jiri Olsa Date: Mon, 27 Oct 2025 09:47:09 +0100 To: Song Liu Cc: bpf@vger.kernel.org, linux-trace-kernel@vger.kernel.org, live-patching@vger.kernel.org, ast@kernel.org, daniel@iogearbox.net, andrii@kernel.org, rostedt@goodmis.org, andrey.grodzovsky@crowdstrike.com, mhiramat@kernel.org, kernel-team@meta.com, olsajiri@gmail.com Subject: Re: [PATCH v3 bpf 0/3] Fix ftrace for livepatch + BPF fexit programs Message-ID: References: <20251026205445.1639632-1-song@kernel.org> Precedence: bulk X-Mailing-List: linux-trace-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20251026205445.1639632-1-song@kernel.org> On Sun, Oct 26, 2025 at 01:54:42PM -0700, Song Liu wrote: > livepatch and BPF trampoline are two special users of ftrace. livepatch > uses ftrace with IPMODIFY flag and BPF trampoline uses ftrace direct > functions. When livepatch and BPF trampoline with fexit programs attach to > the same kernel function, BPF trampoline needs to call into the patched > version of the kernel function. > > 1/3 and 2/3 of this patchset fix two issues with livepatch + fexit cases, > one in the register_ftrace_direct path, the other in the > modify_ftrace_direct path. > > 3/3 adds selftests for both cases. > > --- > > Changes v2 => v3: > 1. Incorporate feedback by AI, which also fixes build error reported by > Steven and kernel test robot. Reviewed-by: Jiri Olsa thanks, jirka > > v2: https://lore.kernel.org/bpf/20251024182901.3247573-1-song@kernel.org/ > > Changes v1 => v2: > 1. Target bpf tree. (Alexei) > 2. Bring back the FTRACE_WARN_ON in __ftrace_hash_update_ipmodify > for valid code paths. (Steven) > 3. Update selftests with cleaner way to find livepatch-sample.ko. > (offlline discussion with Ihor) > > v1: https://lore.kernel.org/bpf/20251024071257.3956031-1-song@kernel.org/ > > Song Liu (3): > ftrace: Fix BPF fexit with livepatch > ftrace: bpf: Fix IPMODIFY + DIRECT in modify_ftrace_direct() > selftests/bpf: Add tests for livepatch + bpf trampoline > > kernel/bpf/trampoline.c | 5 - > kernel/trace/ftrace.c | 46 ++++++-- > tools/testing/selftests/bpf/config | 3 + > .../bpf/prog_tests/livepatch_trampoline.c | 107 ++++++++++++++++++ > .../bpf/progs/livepatch_trampoline.c | 30 +++++ > 5 files changed, 177 insertions(+), 14 deletions(-) > create mode 100644 tools/testing/selftests/bpf/prog_tests/livepatch_trampoline.c > create mode 100644 tools/testing/selftests/bpf/progs/livepatch_trampoline.c > > -- > 2.47.3