Re: [PATCH v6 1/4] mm: add NR_SECONDARY_PAGETABLE to count secondary page table uses.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

To: Yosry Ahmed <yosryahmed@xxxxxxxxxx>
Subject: Re: [PATCH v6 1/4] mm: add NR_SECONDARY_PAGETABLE to count secondary page table uses.
From: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx>
Date: Mon, 22 Aug 2022 17:08:19 -0700
Cc: Tejun Heo <tj@xxxxxxxxxx>, Johannes Weiner <hannes@xxxxxxxxxxx>, Zefan Li <lizefan.x@xxxxxxxxxxxxx>, Marc Zyngier <maz@xxxxxxxxxx>, James Morse <james.morse@xxxxxxx>, Alexandru Elisei <alexandru.elisei@xxxxxxx>, Suzuki K Poulose <suzuki.poulose@xxxxxxx>, Paolo Bonzini <pbonzini@xxxxxxxxxx>, Sean Christopherson <seanjc@xxxxxxxxxx>, Vitaly Kuznetsov <vkuznets@xxxxxxxxxx>, Wanpeng Li <wanpengli@xxxxxxxxxxx>, Jim Mattson <jmattson@xxxxxxxxxx>, Joerg Roedel <joro@xxxxxxxxxx>, Michal Hocko <mhocko@xxxxxxxxxx>, Roman Gushchin <roman.gushchin@xxxxxxxxx>, Shakeel Butt <shakeelb@xxxxxxxxxx>, Oliver Upton <oupton@xxxxxxxxxx>, Huang@xxxxxxxxxx, Shaoqin <shaoqin.huang@xxxxxxxxx>, Cgroups <cgroups@xxxxxxxxxxxxxxx>, Linux Kernel Mailing List <linux-kernel@xxxxxxxxxxxxxxx>, linux-arm-kernel@xxxxxxxxxxxxxxxxxxx, kvmarm@xxxxxxxxxxxxxxxxxxxxx, kvm@xxxxxxxxxxxxxxx, Linux-MM <linux-mm@xxxxxxxxx>
In-reply-to: <CAJD7tkYiVBsWfwQ6qZ3NVzW=3UPTAjSmR5aYgT2M3gk+5Hq0_Q@mail.gmail.com>
References: <20220628220938.3657876-1-yosryahmed@google.com> <20220628220938.3657876-2-yosryahmed@google.com> <20220817102408.7b048f198a736f053ced2862@linux-foundation.org> <CAJD7tkZQ07dZtcTSirj0qLawaE3Ndyn-385m_kL09=gsfO9QwA@mail.gmail.com> <CAJD7tkYiVBsWfwQ6qZ3NVzW=3UPTAjSmR5aYgT2M3gk+5Hq0_Q@mail.gmail.com>

On Mon, 22 Aug 2022 17:04:57 -0700 Yosry Ahmed <yosryahmed@xxxxxxxxxx> wrote:

> > SecondaryPageTables is too long (unfortunately), it messes up the
> > formatting in node_read_meminfo() and meminfo_proc_show(). I would
> > prefer "secondary" as well, but I don't know if breaking the format in
> > this way is okay.
> 
> Any thoughts here Andrew? Change to SecondaryPageTables anyway? Change
> all to use "sec" instead of "secondary"? Leave as-is?

Leave as-is, I guess.

References:
- [PATCH v6 0/4] KVM: mm: count KVM mmu usage in memory stats
  - From: Yosry Ahmed
- [PATCH v6 1/4] mm: add NR_SECONDARY_PAGETABLE to count secondary page table uses.
  - From: Yosry Ahmed
- Re: [PATCH v6 1/4] mm: add NR_SECONDARY_PAGETABLE to count secondary page table uses.
  - From: Andrew Morton
- Re: [PATCH v6 1/4] mm: add NR_SECONDARY_PAGETABLE to count secondary page table uses.
  - From: Yosry Ahmed
- Re: [PATCH v6 1/4] mm: add NR_SECONDARY_PAGETABLE to count secondary page table uses.
  - From: Yosry Ahmed

Prev by Date: [GIT PULL] hotfixes for 6.0-rc3
Next by Date: Re: [PATCH v2 01/11] mm/ioremap: change the return value of io[re|un]map_allowed and rename
Previous by thread: Re: [PATCH v6 1/4] mm: add NR_SECONDARY_PAGETABLE to count secondary page table uses.
Next by thread: Re: [PATCH v6 0/4] KVM: mm: count KVM mmu usage in memory stats
Index(es):
- Date
- Thread

[Index of Archives] [Linux ARM Kernel] [Linux ARM] [Linux Omap] [Fedora ARM] [IETF Annouce] [Bugtraq] [Linux OMAP] [Linux MIPS] [eCos] [Asterisk Internet PBX] [Linux API]