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

Re: [Xen-devel] [PATCH] x86/boot: Make the "Building Dom0" messages consistent



>>> On 26.01.18 at 17:28, <andrew.cooper3@xxxxxxxxxx> wrote:
> On 26/01/18 16:20, Roger Pau Monné wrote:
>> On Fri, Jan 26, 2018 at 04:00:03PM +0000, Andrew Cooper wrote:
>>> --- a/xen/arch/x86/pv/dom0_build.c
>>> +++ b/xen/arch/x86/pv/dom0_build.c
>>> @@ -328,7 +328,7 @@ int __init dom0_construct_pv(struct domain *d,
>>>      /* Machine address of next candidate page-table page. */
>>>      paddr_t mpt_alloc;
>>>  
>>> -    printk("*** LOADING DOMAIN 0 ***\n");
>>> +    printk(XENLOG_INFO "*** Building a PV Dom0 ***\n");
>> Now that we have the shim this should probably be:
>>
>> "*** Building a PV Dom%c ***\n", pv_shim ? 'U' : '0'
>>
>> But in any case can be fixed in a subsequent commit.
> 
> Good point. I'll fold that in, if there are no objections.

With that (preferably using Dom%d as just mentioned in anyother
reply)
Acked-by: Jan Beulich <jbeulich@xxxxxxxx>

Jan

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel

 


Rackspace

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