Message-ID: <05c0da33-f225-41a4-4310-9cc9b3348275@oracle.com>
Date:   Mon, 13 Dec 2021 08:20:35 -0600
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:91.0)
 Gecko/20100101 Thunderbird/91.4.0
Subject: Re: [PATCH v3 1/5] docs: kernel-parameters: Update to reflect the
 current default size of atomic pool
Content-Language: en-US
To:     Baoquan He <bhe@redhat.com>, linux-kernel@vger.kernel.org
Cc:     linux-mm@kvack.org, akpm@linux-foundation.org, hch@lst.de,
        cl@linux.com, John.p.donnelly@oracle.com, kexec@lists.infradead.org
References: <20211213122712.23805-1-bhe@redhat.com>
 <20211213122712.23805-2-bhe@redhat.com>
From:   john.p.donnelly@oracle.com
In-Reply-To: <20211213122712.23805-2-bhe@redhat.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?Ykk2eVVFK2haNHdNK3Q0ZUlWTmdWMFdrTEZWWElYWi9qbzIzaFNmejB6eWw1?=
 =?utf-8?B?VWxDcXFoTU1tSGMxem9xZ1ZhSGlMb2c5VENWS3lWK3pWMFVPWU85SjFnUHJ3?=
 =?utf-8?B?RkxaZXZhWjA5ZW9IN0svZWNVdUhBdDhiaUJyaUc1MUZTck5ZNkMvYktEcWgy?=
 =?utf-8?B?L3MyK3hwdWl5Q1pTRVM5eDExdDRIVEozd0RjNEkzUCt6RGpSOStrdkcwOXNB?=
 =?utf-8?B?ZUcyM0R0RmxOYzBBQk5jaVpNb3BMUEV0Z2dGWnJRMHRFVjRVdDZZdllYSlow?=
 =?utf-8?B?RGhLdm5CdEw1R3dmeVV2RFpYYkRiQjVCYlVzMEdwZ3RWZkJKeXJsYUxMS3pW?=
 =?utf-8?B?OW5zWGlhZm5DeE9uOCtOc0E0TitYS2VzK0hncmJ5NmNRTzBnSHRyMjVTNTVC?=
 =?utf-8?B?ZVN2NFFZcHBXTG9vd0dlUjZ0T3RqK0hFNVEvYitPcVRJeFAvcUlyRG5XbHd0?=
 =?utf-8?B?MjVZaU13bk9VS2YxYVByL292dnJtOS9TUDVyWXVsUHlqNitKbklMdENFb2RK?=
 =?utf-8?B?eitOQk1zOEpMUWR5UCtVSWtSVDlIbGF2R294cktlZzhINVZmREpoVkRFNDRk?=
 =?utf-8?B?UkdGT3d4SWVhSkx1a1BFdGk0OHdzMXdKRGxpaVJIQVA3aG41K3JOVDIzeXZz?=
 =?utf-8?B?dktrTnJrM2pzUWRPK0NnWDMrVEpNWTNndThqTDhidkFCbnlzVFlwTEdlYnVa?=
 =?utf-8?B?dEVSbXNVRDA4MUpRelAyK0hjeEtGdjZJbHZvVEZwRGh3K0V6ZHpSdVEzcXIv?=
 =?utf-8?B?eTBSOE5uVURmMDE4VjRIb3E3azdZTXZianBCM280bjVMS0VKVUtvK1ZrTmNQ?=
 =?utf-8?B?ai9WcDRjTjFmL2ZtdytVNm1XTGxWMWZ2WWVmaXhvNGRSTEx3MC93eUhIL1RR?=
 =?utf-8?B?M2pHdW91bnJCWjVPRmJWNmpKT2lkNFQ2MHpoOGtDTmFZbnh6VnZHVmtsY0hD?=
 =?utf-8?B?VVRMbjEyblZmSGVMNXNUUEF0RURDK2d0SEdycHJWNTMxMEk1WURXT1YyMWJQ?=
 =?utf-8?B?U3pMSHlSTGVSSHZ5cWxqVUJOTENCMm1YQTg0eWtPeFJidjhvUXZKUjZZaDVB?=
 =?utf-8?B?dkVZdTZyMnU3cXYzK3h1QjNxS0RYcmRLb1libUtPdzZuMlp3NDJ2SmZnK1h5?=
 =?utf-8?B?Qk1CaFBUSzYyc3B3YndGNEFvS0RrTVZYdmFzVHp5MTR1dVQycFdYUUc1NmZR?=
 =?utf-8?B?a3FkOHp5OU5YWnpRY1BicVFFMWxwVzBMQ0RGRU9ERjlaYWJCOUJXREppZGI5?=
 =?utf-8?B?NXlxWlNpR1dVZEN6ZzZ2Vm04R3B2RnRvbFJOOHNRRWxwS1dwNDlYQWJCbzJQ?=
 =?utf-8?B?QUlvbmNWSUh6RHNIMklFYmpBSURvOUFReklWMi9UbG55UExITzl2Q2JZa2tp?=
 =?utf-8?B?a05PNW8waEEySnhxbGRDZWhidUc5VXJhTTZIaEhmRFFpdDVFQkVndWV6R1Z2?=
 =?utf-8?B?UG9waVNrcHhsYlNQSGpRQXFzSjJsYk01eVRhZENBVGs1YXYwNDdMQVEzT0g5?=
 =?utf-8?B?aTArSHNjTnliZ0l1TVFDdU1TdXZzaHVySjQ4SFJuclFKWTNmcytndkFScU5T?=
 =?utf-8?B?bGVZV2ZhRnZkYnVvVVlweHpoTWdSbHdzMmRteExHbXJoRENwa0dET1BsdTVS?=
 =?utf-8?B?MDRiQ01NMXpFWDNVaURKYW4vNzhaODN0QXRsdmFTZGNFNWJGS3p2RnE0K1h6?=
 =?utf-8?B?K2lLUUhSeHVHdmJ2MjgvSEM2Qmt4cW5qdnZZS21SaTBpMXZzYUxlU2dpV3RY?=
 =?utf-8?B?VzFaaVdhU0tlRno3UFdSOW9UL1dqWEF3akRJVjQzL2RHdndVQXpNSE9pUXVP?=
 =?utf-8?B?YllabmRFeWZ1Ujc1NFlWUGY3cnhSRlE4V28zS1c1RWJyRnpFS3dqTWRVQ2Fy?=
 =?utf-8?B?RXM0UmJwVFEwdXhkVEx6cWo4aytNYjZVTXNsa21vTi9rL1FOcmNUS1RRZE5r?=
 =?utf-8?B?SXhZUTRhSXp3Qzh6LzdjV2tNZmt1UHhpdXJsc0FraFpqYSt0SjhxZS9nTXRH?=
 =?utf-8?B?WFU3VFdoMlFMRklINUltQjJVWml0blBXajNPTVdHS2FDVUs2L3JnalRjZjV6?=
 =?utf-8?B?Z0Q5NXl4OWN6QTBRdExJcDFqNzhuZDlnaHFxUDZlY243Mk9xaS9kVndoQ29l?=
 =?utf-8?B?dVBZV0kxKy9mY2dsTCs1VHd0MHpCYmVFWDFIZzh3bjJSeWJOUXRJQ1FoT2lV?=
 =?utf-8?B?NmdVb3VzVVdRVGdKaGVKRlVWbklKTERhVk54VmMwRGlrejA0Q25RUklQRjBU?=
 =?utf-8?B?UTZQYitOZlFnK0ExeWd3RTlCaWZ3PT0=?=
X-OriginatorOrg: oracle.com
X-MS-Exchange-CrossTenant-Network-Message-Id: bd3ea1fa-db83-4cb4-fcaf-08d9be43bddd
X-MS-Exchange-CrossTenant-AuthSource: CO1PR10MB4722.namprd10.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 13 Dec 2021 14:20:40.9662
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 4e2c6054-71cb-48f1-bd6c-3a9705aca71b
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: c4R5nPKpYosTSa/T1ciMXh4j2WCZsQ48eTPuUQ3Vxf4DLnboL5QQtPICgzSSMWXekNRxwRRsG09db24tdZN1Gia9HjVPAb2A4yZmqHFu8I4=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MW5PR10MB5691
X-Proofpoint-Virus-Version: vendor=nai engine=6300 definitions=10196 signatures=668683
X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 phishscore=0 spamscore=0 suspectscore=0
 malwarescore=0 mlxlogscore=999 bulkscore=0 mlxscore=0 adultscore=0
 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2110150000
 definitions=main-2112130091
X-Proofpoint-ORIG-GUID: 9PVRotviwCF39wccsY3-kEByhhv4jGtO
X-Proofpoint-GUID: 9PVRotviwCF39wccsY3-kEByhhv4jGtO
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On 12/13/21 6:27 AM, Baoquan He wrote:
> Since commit 1d659236fb43("dma-pool: scale the default DMA coherent pool
> size with memory capacity"), the default size of atomic pool has been
> changed to take by scaling with system memory capacity. So update the
> document in kerenl-parameter.txt accordingly.
> 
> Signed-off-by: Baoquan He <bhe@redhat.com>
 >
  Acked-by: John Donnelly <john.p.donnelly@oracle.com>
  Tested-by:  John Donnelly <john.p.donnelly@oracle.com>

> ---
>   Documentation/admin-guide/kernel-parameters.txt | 4 +++-
>   1 file changed, 3 insertions(+), 1 deletion(-)
> 
> diff --git a/Documentation/admin-guide/kernel-parameters.txt b/Documentation/admin-guide/kernel-parameters.txt
> index 9725c546a0d4..ec4d25e854a8 100644
> --- a/Documentation/admin-guide/kernel-parameters.txt
> +++ b/Documentation/admin-guide/kernel-parameters.txt
> @@ -664,7 +664,9 @@
>   
>   	coherent_pool=nn[KMG]	[ARM,KNL]
>   			Sets the size of memory pool for coherent, atomic dma
> -			allocations, by default set to 256K.
> +			allocations. Otherwise the default size will be scaled
> +			with memory capacity, while clamped between 128K and
> +			1 << (PAGE_SHIFT + MAX_ORDER-1).
>   
>   	com20020=	[HW,NET] ARCnet - COM20020 chipset
>   			Format: