[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH] bootfdt: Unify early printing of memory ranges endpoints


  • To: Michal Orzel <michal.orzel@xxxxxxx>
  • From: Luca Fancellu <Luca.Fancellu@xxxxxxx>
  • Date: Tue, 19 Nov 2024 13:32:26 +0000
  • Accept-language: en-GB, en-US
  • Arc-authentication-results: i=2; mx.microsoft.com 1; spf=pass (sender ip is 63.35.35.123) smtp.rcpttodomain=lists.xenproject.org smtp.mailfrom=arm.com; dmarc=pass (p=none sp=none pct=100) action=none header.from=arm.com; dkim=pass (signature was verified) header.d=arm.com; arc=pass (0 oda=1 ltdi=1 spf=[1,1,smtp.mailfrom=arm.com] dkim=[1,1,header.d=arm.com] dmarc=[1,1,header.from=arm.com])
  • Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=arm.com; dmarc=pass action=none header.from=arm.com; dkim=pass header.d=arm.com; arc=none
  • Arc-message-signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector10001; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=y4tAv/7/vTZygQi185bZEXJBpuLCjoq6TJ5xuCsiaN0=; b=oLmBtldbHD4nBUfDDgpjmuf7hq6Na8uTmOfRi8S0DkJyXChnD0HK9BGQRBoPFqaZhj5VOWavSIB0OOFQRgvF2CPjPJ26TCh1uFcXoCNIjkZ4ReRVEHSHtydVC7wpoEoJjMmLoM/MQD4L0tYiuEJV/m3T8nLe4GU0XoLR/wKQ5b8ni5hPTbe1DUAO8NK5GSFnz5RHS+9jtZKN8KqiRrCmNR5ZSOscsiFU6Ny4Q8ytx/VY7TqUV7DxeaRDc2msVM6eTpQfhjQwei3u02sIvWdl1aitmPHBO+kZPoUadaRWnjTZaSV1nD6hoL0H+dw8DV4IW3fkNUe3OQZTR8ozBfX8sA==
  • Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector10001; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=y4tAv/7/vTZygQi185bZEXJBpuLCjoq6TJ5xuCsiaN0=; b=MD5wMDnb2KppRzv6SB+vFatTDY3rGdlPTd1gyrvmwHDHENvVTkCCSnhs3HLM78KzIX51c9lcyMlGV+1l1wTgk6DKSBl+P80EJaltePjcPGtm54sjiE5aA0vYVP52QAOqqfectbdWea7C6TwDxhnsKGYMCuKNAAB30d/2bhmY5tXr2UB8UihgDS9koZDreY+hipsfGFz8lcePxk1EmHPaSzBnO8bIx71xGwukpz7CpjPDto8sYlD3Ewrwas4DVigbgZ5DnM/xRNhjbgXKk/5fW2QcbnqOI0tI7s9TU9bKmxpybwqEGPE+rBeTzdpnTffqsE6sxKZjuaH+PMKBbg+tmw==
  • Arc-seal: i=2; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=pass; b=dfyRdR/5iT+qBQ4+QYorYcfrHzOvVuDEZPCf+2xxzVNozrGRBLQcb6osjicXTj/g2bxkRzMY7Kuly6UrH5C9Q2jJhWQ37YgUZ0lDd/YnYERAFk+g3JADzev1YYoamddpqy6l+jImNvjzerZDInqvY6RL7qAZg52kIww6AIYPx9uyw133NPGxaJWQFogH+LgpqRenmY62ue+S4aRY8CnVXkLcZVrtJQ/Gy0yBlHi5b4KxW2x6SuB9dFl+AijxhmBZy/m519US+OUzroVNtJ/8dBtg7BvNJR5wjVOl5a72aDVilGFrtnXUOMUaG9bnyLoLmglI3pk/q2Y4ZL1N8HbTzA==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=chzRmR0fNQv0+US3UUdFtPHDw6g3YP7tcF5RfIzV40IooGflXZnFW5H8GBp1kVzLkT3nlxBNSDH5fVGedSqivvmxz4r35InWas5wFcNJBJNyIqv/2/MLssh2syCLGvl5kTNcstxNKb2iTHThgo2RgROocwPSmR7+r4mP3hUFnBHMkb5UlfCmnNgBZ0ZuTokI+kMidW1F3fF/leeyY+6HjDPiWtBPMQWNnhJLLoNG3TX5VQmhI6tmFvUYVEq/M3jOW2e0VB4iI6U3DQKczWHZG2gw0+fy4puI8gm9em3vC/8UPxmuk+seVT449CAM7Lssr2xB/xIFyvffohnfa046CA==
  • Authentication-results-original: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=arm.com;
  • Cc: "xen-devel@xxxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxxx>, Stefano Stabellini <sstabellini@xxxxxxxxxx>, Julien Grall <julien@xxxxxxx>, Bertrand Marquis <Bertrand.Marquis@xxxxxxx>
  • Delivery-date: Tue, 19 Nov 2024 13:32:57 +0000
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
  • Nodisclaimer: true
  • Original-authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=arm.com;
  • Thread-index: AQHbOnmDmplEO1fGPki1YPRhtPoGCbK+mdoA
  • Thread-topic: [PATCH] bootfdt: Unify early printing of memory ranges endpoints

Hi Michal,

> On 19 Nov 2024, at 11:51, Michal Orzel <michal.orzel@xxxxxxx> wrote:
> 
> At the moment, when printing memory ranges during early boot, endpoints
> of some ranges are printed as inclusive (RAM, RESVD, SHMEM) and some
> as exclusive (Initrd, MODULE). Make the behavior consistent and print
> all the endpoints as inclusive.
> 
> Signed-off-by: Michal Orzel <michal.orzel@xxxxxxx>
> ---

Looks good to me!

Reviewed-by: Luca Fancellu <luca.fancellu@xxxxxxx>

Outside early boot there are other places where we print exclusive ranges,
do you know if there is any general style we should apply or does it depend on
case by case?

Cheers,
Luca





 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.