Received-SPF: softfail (google.com: domain of transitioning linux-kernel-owner@vger.kernel.org does not designate 23.128.96.19 as permitted sender) client-ip=23.128.96.19;
Message-ID: <c2d5f0c5-803d-7ca9-3f57-3d230874e310@oracle.com>
Date:   Thu, 7 Apr 2022 16:04:21 -0400
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:91.0)
 Gecko/20100101 Thunderbird/91.7.0
Subject: Re: [PATCH] xen/balloon: don't use PV mode extra memory for zone
 device allocations
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: <20220407093857.1485-1-jgross@suse.com>
From:   Boris Ostrovsky <boris.ostrovsky@oracle.com>
In-Reply-To: <20220407093857.1485-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?MXFIaXZwQVdjdWdzelV0eUhaSmlIeWhIdm9CMXh1OGszSW1WU05mL3FVYnpP?=
 =?utf-8?B?cDZaYStKdVU0TTFGc1BRVFB3b2V1ZFRiZUE0Tm5iSkxNRGRqWGFHSXVEdDh0?=
 =?utf-8?B?UEhyRi9TcEJ5N0hsOWdJdFo2N2QwS1d5SVBHOGJ5dUpSN2FibUw5dWV4cWtX?=
 =?utf-8?B?WDdrWi9kemMxV2RkUlEvSHVkRjRFQVU3QzhLK0N4R0NTbHRSWTNzVUhWZ0xw?=
 =?utf-8?B?YldtaVNpZDVJbi9VbkZNQTNHNHFHNytjSUNZanRBT28zWXpJS2R5N2xWd1ZE?=
 =?utf-8?B?Skt3NWpORmZpT21vQVV2V1dDSU9icUFMc1lTN2tkUFgxYUYvZG1uZFZYM3Jv?=
 =?utf-8?B?WWdld204Z3B0Wm9yUFZrQmxzSU43ZGZiVFJHWG5YdEYwWnB6a1I4WUduQTNN?=
 =?utf-8?B?ZzJKTmxnc2xxZGZVMmpUYTVkRDRpSlhNUG1IN3pvaVhFUFUxdzNTdjgyTHlF?=
 =?utf-8?B?UUtNbTFjRktVMFpVV2J2bStXN1B6QjZFclhyeFJpTlhqdmRJQnlSTnNGTURD?=
 =?utf-8?B?R0JweXFaYmNBTmVubEtJZVRXdWphdXA0WG5KV2tlaFNXL0lhOXJFc2dpUDZy?=
 =?utf-8?B?SVI2SkFJeHdjdUV3N0VzdEdwYTNYcjB6VG93WTcxbXFlcW85b0ExbWI0L3NK?=
 =?utf-8?B?M3BoK2RrNVlzS2o3S00zb3Y5cmN6eS9ndnlNUnNXaEJPb2VXM2ViOTE4aVpJ?=
 =?utf-8?B?V0RyaUNaVnYyZnExYURNUjNqcHNOYUtFeTU1SXA2UGtDMlg0Tzd1WVFzYTZi?=
 =?utf-8?B?OFVlOWdhV0RtL2dSWEdTeFdQSldKc0JhajJ5UER6YUt5OXJTMkxSa2NSNS93?=
 =?utf-8?B?SmdxdUY2OEc1V0pmVXRja2tSMy8wNCtaakk0STZEeVQ2bHNKY2VGeEZqVVBs?=
 =?utf-8?B?TXZnS2c0R3M1VWk4dGxTbnhIRENpY3lvcjU0emlMR2hzNzJpRUhIeFJ0UndE?=
 =?utf-8?B?NFdqc3NpdmJPN3RVUTNJZk1TWWdaN09lM0M3b3RhQk4wV2xSZVNZdmdIbUQ5?=
 =?utf-8?B?Q3R5NmoxSjVIQUxyVGVWQlU4WGkvSXk5TWdkbnJOaGtKTWxweEFnVUNldDBn?=
 =?utf-8?B?RFBJNTlDdGFpTHJWdzBQSUViSFdRSEE1dGFEVUJySVdpWGUwTWtyVlY2RSsz?=
 =?utf-8?B?R0p2NndHMXZ6YlN6MFFFcktIdkhLWjZ5OUhRbTZpc2VOcEFjUk1OSFNDQ0lo?=
 =?utf-8?B?MDIrbjM5WDg4ek5WSXlaKzJ0NURPYXZOd1UrdnJGRVRpT1hXQ3BSUWI5ajhS?=
 =?utf-8?B?ZDQrL2g4YWdVOGVpTFdndzBpbHpDTmkzNmROY2hKRUFJWWYyWjF3MWZaNk9l?=
 =?utf-8?B?WXlVOURWcFlZdExYRzZDVkFWV2pHYXY0MHZnenhjUFQxTlJYdThwZ2ljMGI5?=
 =?utf-8?B?Y1VvSEtnamJDUlFQUzdZdjZkZVRKTEY2NG9mOGtUenIySEdENDk3UDlNVWpn?=
 =?utf-8?B?MWFjUkgyZ3ljaGR5M2ZGOE9wU0FJSThrZmczRXpJVms5UlhNQTErQTdzQmxP?=
 =?utf-8?B?RldEekVTNUpNU3V1RXhkYjNEakRSMmpReS9OT2NOU1J6RC9yaGVjWjdkY1lo?=
 =?utf-8?B?cG1LS0FRNVZvamd6aWFmaGJLd0VhWDcrOXZNck5RekI0cnZkT0FHWlFzOGVP?=
 =?utf-8?B?SG13b1dteWtaUDBWcGlOY2ptV1ZUL3ZDczlEWXlCazlibWtjVjBuYmlCWlUy?=
 =?utf-8?B?cGRXMHQ1Q0pUTC9ucnN1K3dmQWtLNFNWWW1pbm9zbk9ZR3NVU1JteTZXU0to?=
 =?utf-8?B?RXM1elU3cFVZSG84KzloZVlxWUZkQWo0YUpKOHNBSVIwejdRcGhJTjlCd3pL?=
 =?utf-8?B?VmpwenVHeTdxT2xJZXlkdWpRWGgySzM0dGpjdHY2K0JCbjR2QUQxTTlBLzAw?=
 =?utf-8?B?Y256WDYvUzBEWDhYWWxQTmJIWUY0MlZvU3dxZWpDalBOamFTQTVWa2o2UHg2?=
 =?utf-8?B?QjRkMzE0TEdRaVBxWWhVZXlRTUQ4UHZEK2dPRy9wY0ZqN1g0Z3lnL3prelRS?=
 =?utf-8?B?MXppUW5pSXArd0RjQkpJcEYrdS80VXBSU2RJSTAxeFlrUUFnSE9JU3VOalMv?=
 =?utf-8?B?SFNzeVFCMDRHOFlOTUY4N0RTS01HSDlkTmF3YWE1ZHNUL3R6eUtPdGg0YlFy?=
 =?utf-8?B?eThqZHNiRnpSSVNNVm9xUExYNmtpQVBFQVQwRm01azFOc0FzeXFrZk9ybkhu?=
 =?utf-8?B?d05BaC9rWkgyTWNvV1FpWXYyYVQ5RTJhc0VMMU90VDFaVVdQR1pKczlTRVV3?=
 =?utf-8?B?VWxYUFhsR3I5bnlOVExhVG5WRXlRbytwQWtEUUZ1RWhWL0Zrd3NpVVhBSlNY?=
 =?utf-8?B?MmpwWnordGdyaGhFaWNpbTlsYWhPQ1BrSEpFdEFtOWMrOHlHZE9MVzlGRUdE?=
 =?utf-8?Q?ahXLLHU0dBCxssz0=3D?=
X-OriginatorOrg: oracle.com
X-MS-Exchange-CrossTenant-Network-Message-Id: d5b83252-a378-4918-4fe4-08da18d1d0fb
X-MS-Exchange-CrossTenant-AuthSource: BLAPR10MB5009.namprd10.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 07 Apr 2022 20:04:26.2534
 (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: /gdLGAd+j1xDqALvMWQE5Tc7SRuJh/+R9mMsXmQ36H0XlKa6zsVaMZKmTv4oejkNeezk5WHvNjNKsdyeoZne9n/QwRqNS/ZjDEw2eJmTd9A=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MWHPR10MB1472
X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10434:6.0.425,18.0.850
 definitions=2022-04-07_01:2022-04-07,2022-04-07 signatures=0
X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 bulkscore=0 mlxlogscore=999 spamscore=0
 suspectscore=0 malwarescore=0 phishscore=0 adultscore=0 mlxscore=0
 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2202240000
 definitions=main-2204070064
X-Proofpoint-ORIG-GUID: OdMpIMOh9LFK_hvTvb4Zpv18Cayr15SY
X-Proofpoint-GUID: OdMpIMOh9LFK_hvTvb4Zpv18Cayr15SY
X-Spam-Status: No, score=-4.8 required=5.0 tests=BAYES_00,DKIM_SIGNED,
	DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,
	MAILING_LIST_MULTI,NICE_REPLY_A,RDNS_NONE,SPF_HELO_NONE,
	T_SCC_BODY_TEXT_LINE autolearn=unavailable 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-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org


On 4/7/22 5:38 AM, Juergen Gross wrote:
> When running as a Xen PV guest use the extra memory (memory which isn't
> allocated for the guest at boot time) only for ballooning purposes and
> not for zone device allocations. This will remove some code without any
> lack of functionality.
>
> While at it move some code to get rid of another #ifdef.
>
> Remove a comment which is stale since some time now.
>
> Signed-off-by: Juergen Gross <jgross@suse.com>


Reviewed-by: Boris Ostrovsky <boris.ostrovsky@oracle.com>