Received-SPF: pass (google.com: domain of linux-ext4-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <f6834736-3b68-d6e0-ddb2-9d51b8e720b6@amd.com>
Date:   Mon, 18 Jul 2022 13:52:18 -0400
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101
 Thunderbird/91.11.0
Subject: Re: [PATCH v9 02/14] mm: move page zone helpers from mm.h to mmzone.h
Content-Language: en-US
To:     David Hildenbrand <david@redhat.com>,
        Alex Sierra <alex.sierra@amd.com>, jgg@nvidia.com
Cc:     linux-mm@kvack.org, rcampbell@nvidia.com,
        linux-ext4@vger.kernel.org, linux-xfs@vger.kernel.org,
        amd-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org,
        hch@lst.de, jglisse@redhat.com, apopple@nvidia.com,
        willy@infradead.org, akpm@linux-foundation.org
References: <20220715150521.18165-1-alex.sierra@amd.com>
 <20220715150521.18165-3-alex.sierra@amd.com>
 <12b40848-2e38-df0b-8300-0d338315e9b2@redhat.com>
From:   Felix Kuehling <felix.kuehling@amd.com>
Organization: AMD Inc.
In-Reply-To: <12b40848-2e38-df0b-8300-0d338315e9b2@redhat.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 8bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?eDUxNEd4bHFQcy9XTnpNaEJReVpKbVJXeU80aUhlcU4yemExMWRrZXczVjJa?=
 =?utf-8?B?OWw1YjhKSDJUMmdQbkJ3ZHdCVjZHa1g1NkRUVldtMEVnM1NVYUhQeUNpVmxa?=
 =?utf-8?B?MTNwcWQ2TlpwZEtpV3E4ODRSa2I4bGFCVmkvYlZmQWNaQWJ5QkR0M09xeGtO?=
 =?utf-8?B?eU1jMnlGTmV2dlFlcDhMQmRKOG9IbjVCTXV5K2YwbmdwR2FYVkUwdGYrT2Uz?=
 =?utf-8?B?dUhjR05vNWl1d0haSjV1SXk1clVlbjJhcGdRZXUyZ0NmK1M4WDFBSmdhZ2pS?=
 =?utf-8?B?Mzl4WG9UcFFaSTF0Uk5EOTJ1SENaVW1IK1BjbWhkNlRMdHRiTnNlTnB5aW16?=
 =?utf-8?B?SW5pb0pXQ2M4Mm9vd2pBVno3QVhxWUg1NFM2VEhxVWd0bVJrWkMzTXNLMjg2?=
 =?utf-8?B?L0llQnN6bitqSndxZmo0L3EvTG1vcGVxbVN5NGI1cTFQY05IMGFiUTdteExI?=
 =?utf-8?B?azBPejZyYmhRSEFOaTFKeGxsNUxUUStoQU1oay9sVEVURGRmYjlSRVBhVHFp?=
 =?utf-8?B?M0ZXMkpBTGhLR01aU3NPcnVMT0ZwVmpkekhiWFc2SWV3SnF2RjU3aDFlbU1o?=
 =?utf-8?B?cjhXYzhGSVQ1Q01YY3lIUE55aVBpbncwc3A4bTBmUVY0ZVFwY0tteldnVjZU?=
 =?utf-8?B?akJvbVpLcUhVZEVza2xzYXEvRXV3MTY1eERXcXMwQXNZbDM5cEkzUnNTTXdq?=
 =?utf-8?B?WVI0Ty8vaUtxbWpsWHpxRUxnREJRZUdYajRjVWgvKy9HQ1pvRkMybnFXdmQr?=
 =?utf-8?B?UGg3SWN0WTNxQjdIdEVsdjVNQkI2WWg4WWt1SWt0OFNHYkRSLzlnNXp6c1ZH?=
 =?utf-8?B?c2JUd1ZRd0hyMnNLc3lJdlRTbStWUHR2N0twRGJ3a29FdGErNU81aVdVWngr?=
 =?utf-8?B?Y0laYlViMGF0NkdmMlllWHowelFVLzlhSmNML1lhZUVhZktTd2lCTi9zOEVn?=
 =?utf-8?B?dkJtZTZGZFlXdUIzYUNmRmNuNWlnSVVvRHRIRmNZSWJWRTNZZis2Q2xwREpt?=
 =?utf-8?B?WC8xbzZITldtcmdocG5pUCtUZ3lVNFRTQlU3c1ZsbmpJditoTHNqSUNZOGFs?=
 =?utf-8?B?aGNGMVNJcVlYbXhiZW51aWFJaFhBRUVEZGNFek4vMGhKeEIyR1I0LzVsRGRk?=
 =?utf-8?B?VVZqbSt2cW9zU3hLUmk5NUVrZHpTam90Tzd3RUlSdlRrNW9LY2JvcFhwa0c0?=
 =?utf-8?B?YnB3V0VqTEJNMHpqK3Znclg3REFjdXgxWnR5K0dDck01dU9TZnJ4SkljSXpJ?=
 =?utf-8?B?RlNuSEZ1cDlZeFdDSkZTeVJWN2RNVlpYMTlHVVV4L2cxQmtmSlhtQWl6UUFw?=
 =?utf-8?B?eHpRV0JHUUtPMUJhZHhNMHJyUVowWW81d3RMOXczaDBDNVd6UHlUYWtVNlBE?=
 =?utf-8?B?U1hyTlBjVk9BelI0Lzk1eXZkRmVidk8zS1g3R3dnL1dWLzIxa0JEay9GQ0E1?=
 =?utf-8?B?UjN4K1UwSjRFbTVBYXZBeElpcDNtTEFqek5DZUtHY2tyYkpTSys5VG85ZDVu?=
 =?utf-8?B?d3FwT2s2NjFIWDdSU3habnJDc0J6am44ZzVFdlFaQTgxRVZiMmRHaWN0UlF5?=
 =?utf-8?B?UzJaVWJhUmVUelFoMTYxU205MkV1WGpPNHRHWHo5QlpFWnlteUVHTVFqTXZU?=
 =?utf-8?B?b1h0RVBKdU9wYWJNSmZyLzBYRXBtOXFuZ2pxc1hNY3lCQk92VjlIRGJCKzRK?=
 =?utf-8?B?OUlDMWlYSml2NUFpZzR4aVpkWkVLRHdENWttOTRxc2NnRG15V3d2NFdBZFFv?=
 =?utf-8?B?Q3dHbjFtd2RQdXNZRFgxRjgvbVNEVjZzTVd5NUtZQ1lMRjJHa3RsQW9zcUZI?=
 =?utf-8?B?aHpUNWV5R1padjNOWEIvMVlTc3gybU1pV3FOM0pKODFCcHR2QjE4SXdyVWN4?=
 =?utf-8?B?aHpHS1NhVUtNV1ViY0dJeFpjNkZTams4MkJYZzM5QmgrTzBlNkpEZ0Z1d2ZV?=
 =?utf-8?B?MVBrK1N4cVp5RFFDSFNycno3ZDZJNUJJVGt3Y0NwYURkcWU2ZDB4UWg3QXBv?=
 =?utf-8?B?Sk5Pdkc3L05CTkNocXBhV1c3SitLazA5STh4dFk3aC9PMmRCeDJTcmE5c29v?=
 =?utf-8?B?K0lsTENqNkxNOFJUaE4vaFUwK2laNDhQUkZTbmZGUnFhaVZLYjNTcjl4Qk9z?=
 =?utf-8?Q?wq4Wmm+xTf6RIhp34vB7OTsN9?=
X-OriginatorOrg: amd.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 5f887b3b-142d-4e3e-0a75-08da68e64349
X-MS-Exchange-CrossTenant-AuthSource: BN9PR12MB5115.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 18 Jul 2022 17:52:20.8535
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: DQ5HwFzeRPDCPLbZqaFF68hRb8m+VDb49rDhIirF2hK+9ome5bGI5mFztWjzDcSwDIuF+ywQzRUa8uAinmISwQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SJ0PR12MB6967
X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,NICE_REPLY_A,RCVD_IN_DNSWL_NONE,
        RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_PASS autolearn=ham
        autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-ext4.vger.kernel.org>
X-Mailing-List: linux-ext4@vger.kernel.org

On 2022-07-18 06:50, David Hildenbrand wrote:
> On 15.07.22 17:05, Alex Sierra wrote:
>> [WHY]
>> It makes more sense to have these helpers in zone specific header
>> file, rather than the generic mm.h
>>
>> Signed-off-by: Alex Sierra <alex.sierra@amd.com>
> Acked-by: David Hildenbrand <david@redhat.com>

Thank you! I don't think I have the authority to give this a 
Reviewed-by. Who does?

Regards,
   Felix