[src/trunk]: src/sys/uvm Comments

[ad <ad%NetBSD.org@localhost>]

details:   https://anonhg.NetBSD.org/src/rev/ec0bc3b54e5d
branches:  trunk
changeset: 930755:ec0bc3b54e5d
user:      ad <ad%NetBSD.org@localhost>
date:      Mon Apr 13 15:16:14 2020 +0000

description:
Comments

diffstat:

 sys/uvm/uvm_pglist.h |  13 +++++--------
 1 files changed, 5 insertions(+), 8 deletions(-)

diffs (37 lines):

diff -r 8bd74fe85473 -r ec0bc3b54e5d sys/uvm/uvm_pglist.h
--- a/sys/uvm/uvm_pglist.h      Mon Apr 13 14:04:27 2020 +0000
+++ b/sys/uvm/uvm_pglist.h      Mon Apr 13 15:16:14 2020 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: uvm_pglist.h,v 1.10 2019/12/28 08:33:35 martin Exp $   */
+/*     $NetBSD: uvm_pglist.h,v 1.11 2020/04/13 15:16:14 ad Exp $       */
 
 /*-
  * Copyright (c) 2000, 2001, 2008, 2019 The NetBSD Foundation, Inc.
@@ -49,7 +49,9 @@
  * pglist = &uvm.page_free[freelist].pgfl_buckets[bucket].pgb_color[color];
  *
  * Freelists provide a priority ordering of pages for allocation, based upon
- * how valuable they are for special uses (e.g. device driver DMA).
+ * how valuable they are for special uses (e.g.  device driver DMA).  MD
+ * code decides the number and structure of these.  They are always arranged
+ * in descending order of allocation priority.
  *
  * Pages are then grouped in buckets according to some common factor, for
  * example L2/L3 cache locality.  Each bucket has its own lock, and the
@@ -66,15 +68,10 @@
 };
 
 /*
- * At the root, the freelists.  MD code decides the number and structure of
- * these.  They are always arranged in descending order of allocation
- * priority.
- *
  * 8 buckets should be enough to cover most all current x86 systems (2019),
  * given the way package/core/smt IDs are structured on x86.  For systems
  * that report high package counts despite having a single physical CPU
- * package (e.g. Ampere eMAG) a little bit of sharing isn't going to hurt
- * in the least.
+ * package (e.g. Ampere eMAG) a little bit of sharing isn't going to hurt.
  */
 #define        PGFL_MAX_BUCKETS        8
 struct pgfreelist {