| Message ID | 20200921090220.14981-1-sathnaga@linux.vnet.ibm.com (mailing list archive) |
|---|---|
| State | Handled Elsewhere |
| Headers | show |
| Series | [V2] Doc: admin-guide: Add entry for kvm_cma_resv_ratio kernel param | expand |
| Context | Check | Description |
|---|---|---|
| snowpatch_ozlabs/apply_patch | success | Successfully applied on branch powerpc/merge (ace1986562a0814f179ecd2f1e648215ebc6625a) |
| snowpatch_ozlabs/build-ppc64le | warning | Upstream build failed, couldn't test patch |
| snowpatch_ozlabs/build-ppc64be | warning | Upstream build failed, couldn't test patch |
| snowpatch_ozlabs/build-ppc64e | warning | Upstream build failed, couldn't test patch |
| snowpatch_ozlabs/build-pmac32 | warning | Upstream build failed, couldn't test patch |
| snowpatch_ozlabs/checkpatch | success | total: 0 errors, 0 warnings, 0 checks, 14 lines checked |
| snowpatch_ozlabs/needsstable | success | Patch has no Fixes tags |
On 9/21/20 2:02 AM, sathnaga@linux.vnet.ibm.com wrote: > From: Satheesh Rajendran <sathnaga@linux.vnet.ibm.com> > > Add document entry for kvm_cma_resv_ratio kernel param which > is used to alter the KVM contiguous memory allocation percentage > for hash pagetable allocation used by hash mode PowerPC KVM guests. > > Cc: linux-kernel@vger.kernel.org > Cc: kvm-ppc@vger.kernel.org > Cc: linuxppc-dev@lists.ozlabs.org > Cc: Paul Mackerras <paulus@samba.org> > Cc: Michael Ellerman <mpe@ellerman.id.au> > Cc: Jonathan Corbet <corbet@lwn.net> > Reviewed-by: Randy Dunlap <rdunlap@infradead.org> > Signed-off-by: Satheesh Rajendran <sathnaga@linux.vnet.ibm.com> Hi, > --- > > V2: > Addressed review comments from Randy. > > V1: https://lkml.org/lkml/2020/9/16/72 In reply to V1, I didn't add a Reviewed-by: tag, but I can now. Reviewed-by: Randy Dunlap <rdunlap@infradead.org> > --- > Documentation/admin-guide/kernel-parameters.txt | 8 ++++++++ > 1 file changed, 8 insertions(+) > > diff --git a/Documentation/admin-guide/kernel-parameters.txt b/Documentation/admin-guide/kernel-parameters.txt > index a1068742a6df..932ed45740c9 100644 > --- a/Documentation/admin-guide/kernel-parameters.txt > +++ b/Documentation/admin-guide/kernel-parameters.txt > @@ -2258,6 +2258,14 @@ > [KVM,ARM] Allow use of GICv4 for direct injection of > LPIs. > > + kvm_cma_resv_ratio=n [PPC] > + Reserves given percentage from system memory area for > + contiguous memory allocation for KVM hash pagetable > + allocation. > + By default it reserves 5% of total system memory. > + Format: <integer> > + Default: 5 > + > kvm-intel.ept= [KVM,Intel] Disable extended page tables > (virtualized MMU) support on capable Intel chips. > Default is 1 (enabled) > thanks.
On Mon, Sep 21, 2020 at 02:32:20PM +0530, sathnaga@linux.vnet.ibm.com wrote: > From: Satheesh Rajendran <sathnaga@linux.vnet.ibm.com> > > Add document entry for kvm_cma_resv_ratio kernel param which > is used to alter the KVM contiguous memory allocation percentage > for hash pagetable allocation used by hash mode PowerPC KVM guests. > > Cc: linux-kernel@vger.kernel.org > Cc: kvm-ppc@vger.kernel.org > Cc: linuxppc-dev@lists.ozlabs.org > Cc: Paul Mackerras <paulus@samba.org> > Cc: Michael Ellerman <mpe@ellerman.id.au> > Cc: Jonathan Corbet <corbet@lwn.net> > Reviewed-by: Randy Dunlap <rdunlap@infradead.org> > Signed-off-by: Satheesh Rajendran <sathnaga@linux.vnet.ibm.com> > --- > > V2: > Addressed review comments from Randy. > > V1: https://lkml.org/lkml/2020/9/16/72 > --- > Documentation/admin-guide/kernel-parameters.txt | 8 ++++++++ > 1 file changed, 8 insertions(+) > > diff --git a/Documentation/admin-guide/kernel-parameters.txt b/Documentation/admin-guide/kernel-parameters.txt > index a1068742a6df..932ed45740c9 100644 > --- a/Documentation/admin-guide/kernel-parameters.txt > +++ b/Documentation/admin-guide/kernel-parameters.txt > @@ -2258,6 +2258,14 @@ > [KVM,ARM] Allow use of GICv4 for direct injection of > LPIs. > > + kvm_cma_resv_ratio=n [PPC] > + Reserves given percentage from system memory area for > + contiguous memory allocation for KVM hash pagetable > + allocation. > + By default it reserves 5% of total system memory. I am concerned that using the term "reserve" here could give the impression that this memory is then not available for any other use. It is in fact available for other uses as long as they are movable allocations. So this memory is available for uses such as process anonymous memory and page cache, just not for things like kmalloc. I'm not sure what would be a better term than "reserve", though. Perhaps we need to add a sentence something like "The reserved memory is available for use as process memory and page cache when it is not being used by KVM." Paul.
On Mon, 21 Sep 2020 14:32:20 +0530 sathnaga@linux.vnet.ibm.com wrote: > From: Satheesh Rajendran <sathnaga@linux.vnet.ibm.com> > > Add document entry for kvm_cma_resv_ratio kernel param which > is used to alter the KVM contiguous memory allocation percentage > for hash pagetable allocation used by hash mode PowerPC KVM guests. > > Cc: linux-kernel@vger.kernel.org > Cc: kvm-ppc@vger.kernel.org > Cc: linuxppc-dev@lists.ozlabs.org > Cc: Paul Mackerras <paulus@samba.org> > Cc: Michael Ellerman <mpe@ellerman.id.au> > Cc: Jonathan Corbet <corbet@lwn.net> > Reviewed-by: Randy Dunlap <rdunlap@infradead.org> > Signed-off-by: Satheesh Rajendran <sathnaga@linux.vnet.ibm.com> > --- > > V2: > Addressed review comments from Randy. > > V1: https://lkml.org/lkml/2020/9/16/72 Applied, thanks. jon
diff --git a/Documentation/admin-guide/kernel-parameters.txt b/Documentation/admin-guide/kernel-parameters.txt index a1068742a6df..932ed45740c9 100644 --- a/Documentation/admin-guide/kernel-parameters.txt +++ b/Documentation/admin-guide/kernel-parameters.txt @@ -2258,6 +2258,14 @@ [KVM,ARM] Allow use of GICv4 for direct injection of LPIs. + kvm_cma_resv_ratio=n [PPC] + Reserves given percentage from system memory area for + contiguous memory allocation for KVM hash pagetable + allocation. + By default it reserves 5% of total system memory. + Format: <integer> + Default: 5 + kvm-intel.ept= [KVM,Intel] Disable extended page tables (virtualized MMU) support on capable Intel chips. Default is 1 (enabled)