Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Message-ID: <d4784df8-ca54-5a9a-6422-386e508947c5@oracle.com>
Date:   Fri, 5 Nov 2021 08:18:44 -0400
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:91.0)
 Gecko/20100101 Thunderbird/91.2.1
Subject: Re: [PATCH v2] xen/balloon: rename alloc/free_xenballooned_pages
Content-Language: en-US
To:     Juergen Gross <jgross@suse.com>, xen-devel@lists.xenproject.org,
        linux-kernel@vger.kernel.org
Cc:     Stefano Stabellini <sstabellini@kernel.org>
References: <20211102092234.17852-1-jgross@suse.com>
From:   Boris Ostrovsky <boris.ostrovsky@oracle.com>
In-Reply-To: <20211102092234.17852-1-jgross@suse.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?emdldjZHT29iTU9paE1lQUw2Z2dsSm1USW8xUU9ObkdhbTBuQitUT25uWHRR?=
 =?utf-8?B?b21Ga3lRMkV3U1hXQ1ljdTRjU1paYzNPUVFkQXBOSTlEMzcrcmNReE1VTElT?=
 =?utf-8?B?R2VmYVU2V0N1U3BzMmM3NjAwMnF3YTVodmhKRlZtN2tNeFUwQnViWUZoWDFz?=
 =?utf-8?B?SzJtZlc1RE45UjhudTMrZnJsL0Ruc3BlZGI5VGtvYnlQYVBET0c5WmhNUllv?=
 =?utf-8?B?Zk93TVRHSE11NXgyNGsyak9Nc3M2RzNaMXlHOTN2NUd3TzVzTW9VQ3k0SUJ6?=
 =?utf-8?B?WHVSeG94TVZQRjc3MnozWndXd2VjVjlGcnFkeXVIS1JXNzFWV1hBdmJsVU9W?=
 =?utf-8?B?dGZhVm5MVUhET1FEYzZrNWgzSXFrUStUM1kxeGhDQlQ1YmlzdUFZeWw3VGlE?=
 =?utf-8?B?dnNoSW1iN1RaWDBxMVZuVGwrMjNSYXVvSjlaN3Q2aFhpYVdHUXhIZVhsMUdJ?=
 =?utf-8?B?Q2F0VHplUGtqWU9EMTJUT3pIcWJlendvUWhXSUl2NnNObkt3MFlRMmpNaEg1?=
 =?utf-8?B?NnBkdzROZEtsN2hqMkJTR3RtUGtUNWJTZHhLcnRuc0tkekNZa1g4QThraU4r?=
 =?utf-8?B?VDcrRmVoTlBtMFMxTTF6bS83SFB6ZzBoL3FENjM4WDZMaDlocXo5Um9kMU5p?=
 =?utf-8?B?S2pjbitCY2xNVllFS3p1VSsxZ3N5eDVXb2lCaFdnOXhBZGJ3bVFHZWxWRXVW?=
 =?utf-8?B?ZVdtU0hNQ2JTRy9QN3NodUI5MHlCU0ViZnpTTjc0blRPM3lDVmNZVmFLU0Fn?=
 =?utf-8?B?VnF0YThLc0lJNXlFODNMam9Pb2M0RTBucmtadjhETHhuTDNQc1VTK3ZVRjJz?=
 =?utf-8?B?aE9DRDViU2dkZnpsQTJOZ0JCOEN2cEFHQ0d2YlVHS0JpdG1vbVdYK1lleURN?=
 =?utf-8?B?RVFTbHROSlU2UHZTK2JNOTlzNjJrZkt1UjdvQTlpeXA1WFhLNVZRaHk2eEgy?=
 =?utf-8?B?L3N5MlErc3ZNeUhleU1VeHdrMGpDVnlpS3BQK1hMSjVZUXlVQkJwNVhaNklt?=
 =?utf-8?B?TC9OeGJCbGx1UU5CUFY5bXE5YkFINUxsVVJoQS9QWkYxeFdON0p1bDV5MVJV?=
 =?utf-8?B?eGE4dWR6MUlCc3c2RU9GbnNKRmNJdXVRTGkvRldjMUZINlFoWWVnTGxhTWtI?=
 =?utf-8?B?cEFvdUhvWndOcnlvNld4RmEwWDF2bnJIUjZuR0F2RGMveVZMZytpeTc1K0hK?=
 =?utf-8?B?UFFPQWl0dmZha2tmSmFXRlpLdkVmMGc3Vk9rYUk4NEJ2em9od1JHVjN2ZVlK?=
 =?utf-8?B?VVBpL2h2eGVibE5ZaStmUUFRenU2SzlGcnRFMitpRHUxSUpla0xNdERJRm5M?=
 =?utf-8?B?eFFJa1ZjeExtQVNKK3F5T0kwYWV1OFBINVZjcmIzODQrZERtSk1MT3MwTjR3?=
 =?utf-8?B?L1JZd3ppajlVR0xlOXExNFo2QWhjQndaS2J2NFRYdEpzMVBiS1lxd2Y2ZDdv?=
 =?utf-8?B?K3RwUzRwTXhKbExDNG5leXlYeXd1MkxMSXFUNXRaWk1UWWc4WFV5eGpDQUhv?=
 =?utf-8?B?ZTVwMHdHQkhhYngxOTZKT0JqKys2NGNvbnJWbmtHTjRYVWxnbGZlQ2lpbm5V?=
 =?utf-8?B?d0VnSExRVmkxMk9UNEtsbzBRbG1WRlV5MDNtM091YUpJWkVncHNBYjBMRy9q?=
 =?utf-8?B?aUc2UDRMQS9CUmxMZVZ4Nmc1OUx6NU5oN3ZkaSswMUVUd3dDaGowQzhraC8v?=
 =?utf-8?B?d0F1RHpRc0ZzSmo5a0tGdm1nZVg4bE93Mlc2Y05FcWdrMmpRcEJQTk10VkpY?=
 =?utf-8?B?VitneTZ6cGlLVjROS2pVcVd5Q1EwT2J3YjViTkkyelRhcFVGS2RCYWhkWlBn?=
 =?utf-8?B?aXpOejMramErSnRmNlBGWkd1NGd0My8rRmFFdjZBR2U2cDR6V1U4Q0twdURK?=
 =?utf-8?B?aUFIM3NEd1djbHllNnVzYU1ET3JOOEgzU0FVOXFOSmI0bkR4L0lGUE5ybklr?=
 =?utf-8?B?dUhHYUM1UlYvYmxGVjdWNW1JV2Y3T2JvcURVQzZKSW1KOWJ0bUlRbkdiSU1q?=
 =?utf-8?B?S2pGQis0UEJJL3poWURUeE5GRWZraElRUndtcytFWmVXN3hheDFYK0dJL0ky?=
 =?utf-8?B?OGcrTUUrR3N6STlaTi90dmJkSUlZK1FyWDZpNkM2TndTZTBpZW1UdWJsNFlC?=
 =?utf-8?B?MlJvVzJBQzJWTkx5ZGkrSVFWdVVmMjQzNFZrTFpwQmphSFpDYUl3Sko5Q0Rv?=
 =?utf-8?B?Y1JxR052bS9MK2g5STZxaFRldGVKTHl1aWw3Mjlwb1hPdEJMUVp6UUpJY1cx?=
 =?utf-8?Q?QbYZrbu3JhrNX5ieB0IBFN9XHq3njP2AaLzmjT/hU0=3D?=
X-OriginatorOrg: oracle.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 15ab1160-79d9-4ca2-9cc3-08d9a0566b90
X-MS-Exchange-CrossTenant-AuthSource: BLAPR10MB5009.namprd10.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 05 Nov 2021 12:18:48.5212
 (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: IhoFlncXnMDd/b/rKZAtckdGrA4x53axN0C6LI1ovuIGV5Mb6nU6Zw4afQb90nT9e9WdKiOCan05xoeuLd/R8uNBPwWgufEFBW8VoQkWjFI=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BLAPR10MB4884
X-Proofpoint-Virus-Version: vendor=nai engine=6300 definitions=10158 signatures=668683
X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 mlxlogscore=999 spamscore=0
 adultscore=0 mlxscore=0 malwarescore=0 phishscore=0 bulkscore=0
 suspectscore=0 classifier=spam adjust=0 reason=mlx scancount=1
 engine=8.12.0-2110150000 definitions=main-2111050072
X-Proofpoint-ORIG-GUID: sdyWVEPLMy0y0LWnBC2Fn0GVmVBO8pu9
X-Proofpoint-GUID: sdyWVEPLMy0y0LWnBC2Fn0GVmVBO8pu9
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org


On 11/2/21 5:22 AM, Juergen Gross wrote:
> alloc_xenballooned_pages() and free_xenballooned_pages() are used as
> direct replacements of xen_alloc_unpopulated_pages() and
> xen_free_unpopulated_pages() in case CONFIG_XEN_UNPOPULATED_ALLOC isn't
> defined.
>
> Guard both functions with !CONFIG_XEN_UNPOPULATED_ALLOC and rename them
> to the xen_*() variants they are replacing. This allows to remove some
> ifdeffery from the xen.h header file. Adapt the prototype of the
> functions to match.
>
> Signed-off-by: Juergen Gross <jgross@suse.com>


Applied to for-linus-5-16b.


-boris