Received: by 2002:a05:6358:3188:b0:123:57c1:9b43 with SMTP id q8csp2647187rwd; Mon, 22 May 2023 02:01:08 -0700 (PDT) X-Google-Smtp-Source: ACHHUZ4g2f0aScDZ2WJKSoW2tMN83aCNW6NHMWCLhgvnVCPCXAtyVKFXReAFiITimLBbSzNQjPyR X-Received: by 2002:a05:6a20:3c92:b0:10b:8bc7:e112 with SMTP id b18-20020a056a203c9200b0010b8bc7e112mr2661052pzj.10.1684746067732; Mon, 22 May 2023 02:01:07 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1684746067; cv=none; d=google.com; s=arc-20160816; b=eMJ1yEJCtH5DNJBE+YJwduTTCw80dr6UvsEhaBoMdFVxqRQgUkDbF/AcFcEfZ4dy/a ZrIiWz0jK1ItXPmLF7yv2wYGNXoCJDgqxYAy0oHx4nke61bbpvr4yY/B1FL1+pc6vzgp bJ9XG+vn9wwt0oQE86Vk0ZAyeGiaWg/zpGArwmzv/diSIq+nuEbpsMheGC1Gtc4Uzydz Joxx2uRB/bjbs4WyLMCiOgXF36NjU9mXMw3pwGnDRxE8af8PX7G8J+2dzHDYUAh3cr/0 Q6FNcmeBQktO6cknF4IQEvKBlKMvBRNKN/U2vJgmACWhMCO/5cj6GfiK+SF4cyyfbhA7 RoXg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:cc:to:subject :message-id:date:from:in-reply-to:references:mime-version :dkim-signature; bh=WQPFzLEwac+E5AoIBX7hUXtHAL/mEVRDL3gEWHhERoM=; b=VYpX8BJrO5Vz1mvqSVeCMeECap+VdYqWu5wL5cIU35O1xyeZfWPHDDBW7JUZ4V8/Mt KAHrJEGZCivPu9B08DSTMw1YqVLOMtY7KDq7y7NeDU6rIUbLMJJcI+7Q8pdQpdCSOA5Y oXwaZJReOIZvNmYxIkeW02ilMOrmd3rUOWk4Zez8NX54Z4wbMCW0sv4kK3DUm9OowUtG xRW/3rK05s0ZcRlTNoz+uPUOPDTjkGqfQIRYpPpGWpMol1345f11IDSTY8g3NjwBbvc/ V69ok8qzkta7pqeNIQo7xuMEgbiaAFiztVG2G/4sYxqBcjFIHOo5IcQyZkTodSJcEcxG 33sw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@google.com header.s=20221208 header.b=P7E1GPik; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=REJECT sp=REJECT dis=NONE) header.from=google.com Return-Path: Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id y186-20020a638ac3000000b0053475d8615csi4289029pgd.32.2023.05.22.02.00.24; Mon, 22 May 2023 02:01:07 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@google.com header.s=20221208 header.b=P7E1GPik; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=REJECT sp=REJECT dis=NONE) header.from=google.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S232316AbjEVIlg (ORCPT + 99 others); Mon, 22 May 2023 04:41:36 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:34542 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230186AbjEVIle (ORCPT ); Mon, 22 May 2023 04:41:34 -0400 Received: from mail-wm1-x330.google.com (mail-wm1-x330.google.com [IPv6:2a00:1450:4864:20::330]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id D8AB7B6 for ; Mon, 22 May 2023 01:41:33 -0700 (PDT) Received: by mail-wm1-x330.google.com with SMTP id 5b1f17b1804b1-3f417ea5252so38652755e9.0 for ; Mon, 22 May 2023 01:41:33 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20221208; t=1684744892; x=1687336892; h=content-transfer-encoding:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:from:to:cc:subject:date :message-id:reply-to; bh=WQPFzLEwac+E5AoIBX7hUXtHAL/mEVRDL3gEWHhERoM=; b=P7E1GPikLVt35N2MgYkkC6MJEueFa+/LwbcxXJ6JhTBmv3oTFGoIVnphyX67MGILnQ KcdrZVwfeaG0F/t8ms/2YSdc4i5dq1nzyD6m7IOiSc/P88Mt1SblJAUIsn5UiySv8G6h JVObu2ucqORbd9tjWDrfZy5mEJFS2UVrrgj2ldUWqmjHCZkf6LNrj7nTHEHFHHox/6W1 Ex4AC7Gao7i19lwq8W0S825VwWAbuyn/rA1ZDqPaYGcuOJ4qUFx3CN/S8cWXRjTrQ6Si 1e4ffhnivTT1fonDJa7ahEc77RkatBQW0lzGwtNT3RvE3K9JHlWEfAb3wYYdCB0F+RTb +BhQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1684744892; x=1687336892; h=content-transfer-encoding:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=WQPFzLEwac+E5AoIBX7hUXtHAL/mEVRDL3gEWHhERoM=; b=KgqsyjVmDBOIPZSsUG06lIYXWJUhGkukr+sU44cAQVDiH31NOJ3Y5vfoaV536prDTE H6MZ7eUCjfj+Lsi4o7r+6DWg9KIo043+PRqFDa+w7lbgrFtAY3ejtbdCTNanFusD2H0q 3PwidgXp60t2vxDXxafcEZiZyvTnLp/4GaE7lu9851AOXGzKxqzEG7jJ/cyEmaCDKPM3 /FXYJbw6HBLe19rvWoaUAVMXV7L922BBVVKbQsFLJacvtAZCZdM4Wt8IAKx2Lu7YTuAD maYK+v5ajq6T8cqNaKxo6z86hdLh1zTGZdXxd2qtwTwAhEY4vD2V6Md3E98r2MhScidY FIYA== X-Gm-Message-State: AC+VfDwy/z5P6mVk3MpBCt+rogndzdxOniZ8STd564+fFZVzwjEdpK+8 7yKb8WPWcYwxJVo7rABUNMWDsR7FiXfpW8zDl4L0Dg== X-Received: by 2002:a1c:7315:0:b0:3f5:9ba:442d with SMTP id d21-20020a1c7315000000b003f509ba442dmr7273496wmb.33.1684744892238; Mon, 22 May 2023 01:41:32 -0700 (PDT) MIME-Version: 1.0 References: <20230516182537.3139-1-osalvador@suse.de> <20230516182537.3139-4-osalvador@suse.de> In-Reply-To: <20230516182537.3139-4-osalvador@suse.de> From: Alexander Potapenko Date: Mon, 22 May 2023 10:40:55 +0200 Message-ID: Subject: Re: [PATCH v5 3/3] mm,page_owner: Filter out stacks by a threshold counter To: Oscar Salvador Cc: Andrew Morton , linux-kernel@vger.kernel.org, linux-mm@kvack.org, Michal Hocko , Vlastimil Babka , Waiman Long , Suren Baghdasaryan , Marco Elver , Andrey Konovalov , Eric Dumazet Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable X-Spam-Status: No, score=-17.6 required=5.0 tests=BAYES_00,DKIMWL_WL_MED, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF, ENV_AND_HDR_SPF_MATCH,RCVD_IN_DNSWL_NONE,SPF_HELO_NONE,SPF_PASS, T_SCC_BODY_TEXT_LINE,USER_IN_DEF_DKIM_WL,USER_IN_DEF_SPF_WL autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, May 16, 2023 at 8:25=E2=80=AFPM Oscar Salvador = wrote: > > We want to be able to filter out the output on a threshold basis, > in this way we can get rid of a lot of noise and focus only on those > stacks which have an allegedly high counter. > > We can control the threshold value by a new file called > 'page_owner_threshold', which is 0 by default. > > Signed-off-by: Oscar Salvador Nack. We must decouple stackdepot from page_owner as much as possible, not add extra dependencies by sharing variables between the two. What you need here is a pair of xxx_iter_start()/xxx_iter_next() functions in stackdepot.c that will hold the iteration state in a struct and return the next stack trace. The threshold value can also be part of that struct.