All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] Documentation: arm64 supports cma
@ 2020-09-18  3:15 Tian Tao
  2020-09-18  3:28 ` Song Bao Hua (Barry Song)
  0 siblings, 1 reply; 4+ messages in thread
From: Tian Tao @ 2020-09-18  3:15 UTC (permalink / raw)
  To: corbet, keescook, anton, ccross, tony.luck, paulmck, tglx, akpm,
	bp, mchehab+huawei, pawan.kumar.gupta, rdunlap, mike.kravetz,
	oneukum, gpiccoli, linux-doc
  Cc: linuxarm

arm64 also supports cma, updated documentation.

Signed-off-by: Tian Tao <tiantao6@hisilicon.com>
---
 Documentation/admin-guide/kernel-parameters.txt | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Documentation/admin-guide/kernel-parameters.txt b/Documentation/admin-guide/kernel-parameters.txt
index 7ebe3be..38a1602 100644
--- a/Documentation/admin-guide/kernel-parameters.txt
+++ b/Documentation/admin-guide/kernel-parameters.txt
@@ -591,7 +591,7 @@
 			some critical bits.
 
 	cma=nn[MG]@[start[MG][-end[MG]]]
-			[ARM,X86,KNL]
+			[ARM,ARM64,X86,KNL]
 			Sets the size of kernel global memory area for
 			contiguous memory allocations and optionally the
 			placement constraint by the physical address range of
-- 
2.7.4


^ permalink raw reply related	[flat|nested] 4+ messages in thread

* RE: [PATCH] Documentation: arm64 supports cma
  2020-09-18  3:15 [PATCH] Documentation: arm64 supports cma Tian Tao
@ 2020-09-18  3:28 ` Song Bao Hua (Barry Song)
  2020-09-18  4:00   ` Randy Dunlap
  0 siblings, 1 reply; 4+ messages in thread
From: Song Bao Hua (Barry Song) @ 2020-09-18  3:28 UTC (permalink / raw)
  To: tiantao (H),
	corbet, keescook, anton, ccross, tony.luck, paulmck, tglx, akpm,
	bp, mchehab+huawei, pawan.kumar.gupta, rdunlap, mike.kravetz,
	oneukum, gpiccoli, linux-doc
  Cc: Linuxarm



> -----Original Message-----
> From: Tian Tao [mailto:tiantao6@hisilicon.com]
> Sent: Friday, September 18, 2020 3:16 PM
> To: corbet@lwn.net; keescook@chromium.org; anton@enomsg.org;
> ccross@android.com; tony.luck@intel.com; paulmck@kernel.org;
> tglx@linutronix.de; akpm@linux-foundation.org; bp@suse.de;
> mchehab+huawei@kernel.org; pawan.kumar.gupta@linux.intel.com;
> rdunlap@infradead.org; mike.kravetz@oracle.com; oneukum@suse.com;
> gpiccoli@canonical.com; linux-doc@vger.kernel.org
> Cc: Linuxarm <linuxarm@huawei.com>
> Subject: [PATCH] Documentation: arm64 supports cma
> 
> arm64 also supports cma, updated documentation.
> 
> Signed-off-by: Tian Tao <tiantao6@hisilicon.com>
> ---
>  Documentation/admin-guide/kernel-parameters.txt | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/Documentation/admin-guide/kernel-parameters.txt
> b/Documentation/admin-guide/kernel-parameters.txt
> index 7ebe3be..38a1602 100644
> --- a/Documentation/admin-guide/kernel-parameters.txt
> +++ b/Documentation/admin-guide/kernel-parameters.txt
> @@ -591,7 +591,7 @@
>  			some critical bits.
> 
>  	cma=nn[MG]@[start[MG][-end[MG]]]
> -			[ARM,X86,KNL]
> +			[ARM,ARM64,X86,KNL]

Yes. It supports ARM64. On the other hand, it also supports other architectures like
MIPS
S390
MICROBLAZE
...
So I am not sure if it is a completely correct fix.

CMA only depends on MMU. It probably doesn't depend on arch too much.

Is it better to just move to [KNL]?

>  			Sets the size of kernel global memory area for
>  			contiguous memory allocations and optionally the
>  			placement constraint by the physical address range of
> --
> 2.7.4

Thanks
Barry


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] Documentation: arm64 supports cma
  2020-09-18  3:28 ` Song Bao Hua (Barry Song)
@ 2020-09-18  4:00   ` Randy Dunlap
  2020-09-18  4:22     ` Song Bao Hua (Barry Song)
  0 siblings, 1 reply; 4+ messages in thread
From: Randy Dunlap @ 2020-09-18  4:00 UTC (permalink / raw)
  To: Song Bao Hua (Barry Song), tiantao (H),
	corbet, keescook, anton, ccross, tony.luck, paulmck, tglx, akpm,
	bp, mchehab+huawei, pawan.kumar.gupta, mike.kravetz, oneukum,
	gpiccoli, linux-doc
  Cc: Linuxarm

On 9/17/20 8:28 PM, Song Bao Hua (Barry Song) wrote:
> 
> 
>> -----Original Message-----
>> From: Tian Tao [mailto:tiantao6@hisilicon.com]
>> Sent: Friday, September 18, 2020 3:16 PM
>> To: corbet@lwn.net; keescook@chromium.org; anton@enomsg.org;
>> ccross@android.com; tony.luck@intel.com; paulmck@kernel.org;
>> tglx@linutronix.de; akpm@linux-foundation.org; bp@suse.de;
>> mchehab+huawei@kernel.org; pawan.kumar.gupta@linux.intel.com;
>> rdunlap@infradead.org; mike.kravetz@oracle.com; oneukum@suse.com;
>> gpiccoli@canonical.com; linux-doc@vger.kernel.org
>> Cc: Linuxarm <linuxarm@huawei.com>
>> Subject: [PATCH] Documentation: arm64 supports cma
>>
>> arm64 also supports cma, updated documentation.
>>
>> Signed-off-by: Tian Tao <tiantao6@hisilicon.com>
>> ---
>>  Documentation/admin-guide/kernel-parameters.txt | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/Documentation/admin-guide/kernel-parameters.txt
>> b/Documentation/admin-guide/kernel-parameters.txt
>> index 7ebe3be..38a1602 100644
>> --- a/Documentation/admin-guide/kernel-parameters.txt
>> +++ b/Documentation/admin-guide/kernel-parameters.txt
>> @@ -591,7 +591,7 @@
>>  			some critical bits.
>>
>>  	cma=nn[MG]@[start[MG][-end[MG]]]
>> -			[ARM,X86,KNL]
>> +			[ARM,ARM64,X86,KNL]
> 
> Yes. It supports ARM64. On the other hand, it also supports other architectures like
> MIPS
> S390
> MICROBLAZE
> ...
> So I am not sure if it is a completely correct fix.
> 
> CMA only depends on MMU. It probably doesn't depend on arch too much.
> 
> Is it better to just move to [KNL]?

How about [KNL,CMA]?  CMA is already listed as a build option/restriction
in Documentation/admin-guide/kernel-parameters.rst.

> 
>>  			Sets the size of kernel global memory area for
>>  			contiguous memory allocations and optionally the
>>  			placement constraint by the physical address range of
>> --

thanks.
-- 
~Randy


^ permalink raw reply	[flat|nested] 4+ messages in thread

* RE: [PATCH] Documentation: arm64 supports cma
  2020-09-18  4:00   ` Randy Dunlap
@ 2020-09-18  4:22     ` Song Bao Hua (Barry Song)
  0 siblings, 0 replies; 4+ messages in thread
From: Song Bao Hua (Barry Song) @ 2020-09-18  4:22 UTC (permalink / raw)
  To: Randy Dunlap, tiantao (H),
	corbet, keescook, anton, ccross, tony.luck, paulmck, tglx, akpm,
	bp, mchehab+huawei, pawan.kumar.gupta, mike.kravetz, oneukum,
	gpiccoli, linux-doc
  Cc: Linuxarm



> -----Original Message-----
> From: Randy Dunlap [mailto:rdunlap@infradead.org]
> Sent: Friday, September 18, 2020 4:00 PM
> To: Song Bao Hua (Barry Song) <song.bao.hua@hisilicon.com>; tiantao (H)
> <tiantao6@hisilicon.com>; corbet@lwn.net; keescook@chromium.org;
> anton@enomsg.org; ccross@android.com; tony.luck@intel.com;
> paulmck@kernel.org; tglx@linutronix.de; akpm@linux-foundation.org;
> bp@suse.de; mchehab+huawei@kernel.org;
> pawan.kumar.gupta@linux.intel.com; mike.kravetz@oracle.com;
> oneukum@suse.com; gpiccoli@canonical.com; linux-doc@vger.kernel.org
> Cc: Linuxarm <linuxarm@huawei.com>
> Subject: Re: [PATCH] Documentation: arm64 supports cma
> 
> On 9/17/20 8:28 PM, Song Bao Hua (Barry Song) wrote:
> >
> >
> >> -----Original Message-----
> >> From: Tian Tao [mailto:tiantao6@hisilicon.com]
> >> Sent: Friday, September 18, 2020 3:16 PM
> >> To: corbet@lwn.net; keescook@chromium.org; anton@enomsg.org;
> >> ccross@android.com; tony.luck@intel.com; paulmck@kernel.org;
> >> tglx@linutronix.de; akpm@linux-foundation.org; bp@suse.de;
> >> mchehab+huawei@kernel.org; pawan.kumar.gupta@linux.intel.com;
> >> rdunlap@infradead.org; mike.kravetz@oracle.com; oneukum@suse.com;
> >> gpiccoli@canonical.com; linux-doc@vger.kernel.org
> >> Cc: Linuxarm <linuxarm@huawei.com>
> >> Subject: [PATCH] Documentation: arm64 supports cma
> >>
> >> arm64 also supports cma, updated documentation.
> >>
> >> Signed-off-by: Tian Tao <tiantao6@hisilicon.com>
> >> ---
> >>  Documentation/admin-guide/kernel-parameters.txt | 2 +-
> >>  1 file changed, 1 insertion(+), 1 deletion(-)
> >>
> >> diff --git a/Documentation/admin-guide/kernel-parameters.txt
> >> b/Documentation/admin-guide/kernel-parameters.txt
> >> index 7ebe3be..38a1602 100644
> >> --- a/Documentation/admin-guide/kernel-parameters.txt
> >> +++ b/Documentation/admin-guide/kernel-parameters.txt
> >> @@ -591,7 +591,7 @@
> >>  			some critical bits.
> >>
> >>  	cma=nn[MG]@[start[MG][-end[MG]]]
> >> -			[ARM,X86,KNL]
> >> +			[ARM,ARM64,X86,KNL]
> >
> > Yes. It supports ARM64. On the other hand, it also supports other
> architectures like
> > MIPS
> > S390
> > MICROBLAZE
> > ...
> > So I am not sure if it is a completely correct fix.
> >
> > CMA only depends on MMU. It probably doesn't depend on arch too much.
> >
> > Is it better to just move to [KNL]?
> 
> How about [KNL,CMA]?  CMA is already listed as a build option/restriction
> in Documentation/admin-guide/kernel-parameters.rst.

Sounds good to me.

> 
> >
> >>  			Sets the size of kernel global memory area for
> >>  			contiguous memory allocations and optionally the
> >>  			placement constraint by the physical address range of
> >> --
> 

Thanks.
Barry

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2020-09-18  4:22 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-09-18  3:15 [PATCH] Documentation: arm64 supports cma Tian Tao
2020-09-18  3:28 ` Song Bao Hua (Barry Song)
2020-09-18  4:00   ` Randy Dunlap
2020-09-18  4:22     ` Song Bao Hua (Barry Song)

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.