From patchwork Fri Jun 2 18:57:57 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Marcelo Tosatti X-Patchwork-Id: 10242 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a59:994d:0:b0:3d9:f83d:47d9 with SMTP id k13csp1242132vqr; Fri, 2 Jun 2023 12:12:04 -0700 (PDT) X-Google-Smtp-Source: ACHHUZ6TZFcOzN2d24Ixq4rspNajWxZG3OZpC4yLjcEGp9h/OSp5s26NKmIOp/88yRvzTvkbJlWJ X-Received: by 2002:a05:6a00:22d2:b0:64d:1185:2414 with SMTP id f18-20020a056a0022d200b0064d11852414mr11104563pfj.4.1685733124201; Fri, 02 Jun 2023 12:12:04 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1685733124; cv=none; d=google.com; s=arc-20160816; b=DRJOBjn4lc/CznNImu9Wb9YrIbKreyKeoVFO8gJFnUVPTFpwzNlBcLIm+nNNS+ASpE Z3UmA/soIVBv66lRUbnQcyN+QyBtEp+pFViZnNG4dv/LS2GZb1C5M1DPYhc66JzHu+sj 7jQeCOaez1fyJIh0pwEwyeOuyf6svAKoGhuiUOSgf3116s28OOy50E1dy/n5ypYE7BIr 2VPoWim4BKsHj+P2ITYl/+mJTzDLb+SzsW2qbnaq/FspbJJr2zADRA84gUm7ExD7jUaM L7AcMS7JNlKYzqMEtYNx1BERE7YNQfjp6NK7djAKS9w+eCZwAKFVWJOhrQjpeMmW+36T mrAA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:subject:cc:to:from:date:user-agent:message-id :dkim-signature; bh=OYTwSNTZkAWeI3H38rCG1HDYpkOkm5vdgy89JxH7eHI=; b=UASx3Vzntnw+/1A9FhmkWg3PZ6BDXR4nJCZ/ZSvgTK8PtSgCQQgzSkZAwkLYfs5fhp zzCT+syVUahR+FR2Bwxq0R9nRCPHrx7tDH2ch8jZnC8+dXZrmZr4rRnNn/iJykC8ml+f tK6vBi4GhmyWtHki+d0vSKTf/6tBF+HR344CaVSFz9QtyW4Bolsexe0oXkYcaZ1U+vhv hbhDoBDEVEUvLAP0QEuF+69IrEb0382V9AS2bZAxrJ+ICMRpY0WnkUDpqeY/1g24incg w96kvQfYn/gzxbqaRQ63NSC6rkaHLMcy7sR7qN4Qx/0sjyt7RSxohkd7tAq0FHRZXJfK HwFg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@redhat.com header.s=mimecast20190719 header.b=Ca3pvrjz; 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=NONE sp=NONE dis=NONE) header.from=redhat.com Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id k26-20020a637b5a000000b00508eeafc9ecsi1445956pgn.176.2023.06.02.12.11.51; Fri, 02 Jun 2023 12:12:04 -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=@redhat.com header.s=mimecast20190719 header.b=Ca3pvrjz; 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=NONE sp=NONE dis=NONE) header.from=redhat.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S236544AbjFBTEw (ORCPT + 99 others); Fri, 2 Jun 2023 15:04:52 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:55744 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S232978AbjFBTEu (ORCPT ); Fri, 2 Jun 2023 15:04:50 -0400 Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.129.124]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 662F61B1 for ; Fri, 2 Jun 2023 12:04:10 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1685732649; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc; bh=OYTwSNTZkAWeI3H38rCG1HDYpkOkm5vdgy89JxH7eHI=; b=Ca3pvrjzebFFNllhEoSfVc+RssKMUl27uLu1mPbXaJ1ZlSbeLgmvdcsn1kjrauytn0rjrL DMUvCO0OZnUavi9LgXdheIC+E//4QgJMUmWBQC6tMPXHYaplAwLq/VHkxoRtuDGmmy1Yu2 n1921IYZrloxuZrKTpzOBOXLQJScdxE= Received: from mimecast-mx02.redhat.com (mimecast-mx02.redhat.com [66.187.233.88]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id us-mta-65-A37PyVDiPPqBJzAx3nR0Jw-1; Fri, 02 Jun 2023 15:04:08 -0400 X-MC-Unique: A37PyVDiPPqBJzAx3nR0Jw-1 Received: from smtp.corp.redhat.com (int-mx04.intmail.prod.int.rdu2.redhat.com [10.11.54.4]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx02.redhat.com (Postfix) with ESMTPS id 6E451185A7AE; Fri, 2 Jun 2023 19:04:06 +0000 (UTC) Received: from tpad.localdomain (ovpn-112-2.gru2.redhat.com [10.97.112.2]) by smtp.corp.redhat.com (Postfix) with ESMTPS id 37F072026D49; Fri, 2 Jun 2023 19:04:06 +0000 (UTC) Received: by tpad.localdomain (Postfix, from userid 1000) id 077AA401E14C0; Fri, 2 Jun 2023 16:03:42 -0300 (-03) Message-ID: <20230602185757.110910188@redhat.com> User-Agent: quilt/0.67 Date: Fri, 02 Jun 2023 15:57:57 -0300 From: Marcelo Tosatti To: Christoph Lameter Cc: Aaron Tomlin , Frederic Weisbecker , Andrew Morton , linux-kernel@vger.kernel.org, linux-mm@kvack.org, Vlastimil Babka , Michal Hocko Subject: [PATCH v2 0/3] vmstat bug fixes for nohz_full and isolated CPUs X-Scanned-By: MIMEDefang 3.1 on 10.11.54.4 X-Spam-Status: No, score=-2.3 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_NONE, SPF_HELO_NONE,SPF_NONE,T_SCC_BODY_TEXT_LINE 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 X-getmail-retrieved-from-mailbox: =?utf-8?q?INBOX?= X-GMAIL-THRID: =?utf-8?q?1767619296752008353?= X-GMAIL-MSGID: =?utf-8?q?1767619296752008353?= This patch series addresses the following two problems: 1. A customer provided evidence indicating that a process was stalled in direct reclaim: - The process was trapped in throttle_direct_reclaim(). The function wait_event_killable() was called to wait condition allow_direct_reclaim(pgdat) for current node to be true. The allow_direct_reclaim(pgdat) examined the number of free pages on the node by zone_page_state() which just returns value in zone->vm_stat[NR_FREE_PAGES]. - On node #1, zone->vm_stat[NR_FREE_PAGES] was 0. However, the freelist on this node was not empty. - This inconsistent of vmstat value was caused by percpu vmstat on nohz_full cpus. Every increment/decrement of vmstat is performed on percpu vmstat counter at first, then pooled diffs are cumulated to the zone's vmstat counter in timely manner. However, on nohz_full cpus (in case of this customer's system, 48 of 52 cpus) these pooled diffs were not cumulated once the cpu had no event on it so that the cpu started sleeping infinitely. I checked percpu vmstat and found there were total 69 counts not cumulated to the zone's vmstat counter yet. - In this situation, kswapd did not help the trapped process. In pgdat_balanced(), zone_wakermark_ok_safe() examined the number of free pages on the node by zone_page_state_snapshot() which checks pending counts on percpu vmstat. Therefore kswapd could know there were 69 free pages correctly. Since zone->_watermark = {8, 20, 32}, kswapd did not work because 69 was greater than 32 as high watermark. 2. With a task that busy loops on a given CPU, the kworker interruption to execute vmstat_update is undesired and may exceed latency thresholds for certain applications. First issue is solved by using _snapshot version of the counters on allow_direct_reclaim. Second issue is fixed by disabling periodic vmstat updates for nohz_full CPUs. Thanks to Aaron Tomlin for diagnosing issue 1 and writing the initial patch series. Performance details for the kworker interruption: oslat 1094.456862: sys_mlock(start: 7f7ed0000b60, len: 1000) oslat 1094.456971: workqueue_queue_work: ... function=vmstat_update ... oslat 1094.456974: sched_switch: prev_comm=oslat ... ==> next_comm=kworker/5:1 ... kworker 1094.456978: sched_switch: prev_comm=kworker/5:1 ==> next_comm=oslat ... The example above shows an additional 7us for the oslat -> kworker -> oslat switches. In the case of a virtualized CPU, and the vmstat_update interruption in the host (of a qemu-kvm vcpu), the latency penalty observed in the guest is higher than 50us, violating the acceptable latency threshold for certain applications. v2: use cpu_is_isolated (Michal Hocko) opencode schedule_on_each_cpu (Michal Hocko) mm/vmscan.c | 2 +- mm/vmstat.c | 39 +++++++++++++++++++++++++++++++++++---- 2 files changed, 36 insertions(+), 5 deletions(-)