Re: [lttng-dev] [PATCH 3/3] vmscan: drop isolate_mode (v6.7+)

[Kienan Stewart via lttng-dev]

Hi Bruce,

Others may chime in here as I'm not sure I have the full context off-hand.
On 2023-11-23 14:56, Bruce Ashfield wrote:
Aha,

So the mailing list isn't used for patch submission anymore ?

Patches submitted to the mailing list are reviewed. That being said, I 
think it's fair to say that the principal workflow has shifted away from 
the mailing list to https://review.lttng.org - many patches are reviewed 
there and not posted on the mailing list.

I think some patch series also get submitted to the mailing list in 
addition to https://review.lttng.org in order to solicit a wider 
feedback from the community.

I didn't
notice any updates in the READMEs, etc, so I just fell back to my
existing workflow.

You are correct that the README for lttng-modules doesn't make any 
mention of this. Thanks for bringing this up, it should definitely be 
addressed!

I'll go through the READMEs and CONTRIBUTING.md files to make sure they 
indicate the current workflows.

thanks,
kienan


I just want to know, so I won't waste time in the future.

Bruce

On Thu, Nov 23, 2023 at 2:49 PM Kienan Stewart <kstew...@efficios.com> wrote:

Hi Bruce,

thanks for the patches! There's a similar series waiting for review:
https://review.lttng.org/q/topic:%22fix-linux-6.7.0-rc1%22

thanks,
kienan

On 2023-11-23 14:32, Bruce Ashfield via lttng-dev wrote:
From: Bruce Ashfield <bruce.ashfi...@gmail.com>

commit 3dfbb555c98ac55b9 [mm, vmscan: remove ISOLATE_UNMAPPED]
makes the isolate_mode parameter unused, so it is removed.

This updates the event to match.

Signed-off-by: Bruce Ashfield <bruce.ashfi...@gmail.com>
---

Repeating the comment from patch 1 & 2 of this series:

Someone else may or may not already have this in flight,
but I didn't see anything on the list or in the git
repo.

I ran into this failure when working on the v6.7-rc
kernel as part of yocto project kernel updates.

Tweak as necessary, as I'm no expert in the right way
to version these sort of changes.

Bruce

   include/instrumentation/events/mm_vmscan.h | 10 +++++++++-
   1 file changed, 9 insertions(+), 1 deletion(-)

diff --git a/include/instrumentation/events/mm_vmscan.h 
b/include/instrumentation/events/mm_vmscan.h
index e7e9b613..a3e9ddf2 100644
--- a/include/instrumentation/events/mm_vmscan.h
+++ b/include/instrumentation/events/mm_vmscan.h
@@ -343,12 +343,18 @@ LTTNG_TRACEPOINT_EVENT(mm_vmscan_lru_isolate,
               unsigned long nr_scanned,
               unsigned long nr_skipped,
               unsigned long nr_taken,
+#if (LTTNG_LINUX_VERSION_CODE < LTTNG_KERNEL_VERSION(6,7,0))
               isolate_mode_t isolate_mode,
+#endif
               int lru
       ),

       TP_ARGS(classzone_idx, order, nr_requested, nr_scanned, nr_skipped,
-             nr_taken, isolate_mode, lru
+             nr_taken,
+#if (LTTNG_LINUX_VERSION_CODE < LTTNG_KERNEL_VERSION(6,7,0))
+             isolate_mode,
+#endif
+             lru
       ),

       TP_FIELDS(
@@ -358,7 +364,9 @@ LTTNG_TRACEPOINT_EVENT(mm_vmscan_lru_isolate,
               ctf_integer(unsigned long, nr_scanned, nr_scanned)
               ctf_integer(unsigned long, nr_skipped, nr_skipped)
               ctf_integer(unsigned long, nr_taken, nr_taken)
+#if (LTTNG_LINUX_VERSION_CODE < LTTNG_KERNEL_VERSION(6,7,0))
               ctf_integer(isolate_mode_t, isolate_mode, isolate_mode)
+#endif
               ctf_integer(int, lru, lru)
       )
   )



_______________________________________________
lttng-dev mailing list
lttng-dev@lists.lttng.org
https://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev