On 19/11/2024 14:32, Luca Fancellu wrote:
>
>
> Hi Michal,
>
>> On 19 Nov 2024, at 11:51, Michal Orzel <[email protected]> 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 <[email protected]>
>> ---
>
> Looks good to me!
>
> Reviewed-by: Luca Fancellu <[email protected]>
>
> 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?
Yes, there are other places where we print ranges with endpoints exclusive or
inclusive
but at least they are grouped together. Here it was quite confusing to find
lines next to
each other with different styles. I don't think there is any specific style we
should adhere to.
~Michal
