[meta-virtualization] Issues with 50cdecc sysvinit-inittab_xen.inc: fix hypervisor console

[Manjukumar Harthikote Matha]

Hi Bruce,

> -----Original Message-----
> From: Bruce Ashfield [mailto:bruce.ashfield at gmail.com]
> Sent: Monday, April 8, 2019 4:34 AM
> To: Sathees Balya <Sathees.Balya at arm.com>
> Cc: Filipe Rinaldi <Filipe.Rinaldi at arm.com>; meta-
> virtualization at yoctoproject.org; Manjukumar Harthikote Matha
> <MANJUKUM at xilinx.com>; nd <nd at arm.com>
> Subject: Re: [meta-virtualization] Issues with 50cdecc sysvinit-inittab_xen.inc: fix
> hypervisor console
> 
> On Mon, Apr 8, 2019 at 7:26 AM Sathees Balya <Sathees.Balya at arm.com>
> wrote:
> >
> > Hi Bruce,
> >
> > We reverted the commit "50cdecca7b2a0e1bb5eacbec132f26b802f950fc" and
> the console starts as expected. Thanks.
> >

We should revert this commit, and we can update the patch later after testing it out on different cases. 

Thanks,
Manju

> 
> We had already established that without the commit things start, they were
> starting before we made that change. It was for fixing a different issue and for
> standardizing on the use of the variable.
> 
> I'm trying to find out if we change the startup check, if the console still works.
> 
> Bruce
> 
> >
> > Regards,
> > Sathees
> >
> > On 08/04/2019, 04:12, "Bruce Ashfield" <bruce.ashfield at gmail.com> wrote:
> >
> >     On Thu, Apr 4, 2019 at 4:11 PM Filipe Rinaldi <Filipe.Rinaldi at arm.com>
> wrote:
> >     >
> >     > Hi,
> >     >
> >     >
> >     > With regards commit "50cdecc sysvinit-inittab_xen.inc: fix hypervisor
> console"...
> >     >
> >     >
> >     > We are having issues with this patch. When using start_getty (a wrapper to
> getty) it will first check if the console device is a "serial" device driver [1]. It then
> fails because hvc0 is a "system" device, not a "serial" [2].
> >     >
> >     >
> >     > [1] https://github.com/openembedded/openembedded-
> core/blob/master/meta/recipes-core/sysvinit/sysvinit-inittab/start_getty#L11
> >     >
> >     > [2]
> https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/drivers/tt
> y/hvc/hvc_console.c#n1036
> >     >
> >     >
> >     > Any thoughts?
> >
> >     Can you confirm that if that check is removed that the console still
> >     functions as expected ?
> >
> >     Rather than bouncing back and forth between fixing one issue, and
> >     creating another, I'm tempted to just bbappend and patch the
> >     problematic check out of the startup. But I'm not currently setup to
> >     test the issue.
> >
> >     Bruce
> >
> >     >
> >     >
> >     > Cheers,
> >     >
> >     > Filipe
> >     >
> >     >
> >     > IMPORTANT NOTICE: The contents of this email and any attachments are
> confidential and may also be privileged. If you are not the intended recipient,
> please notify the sender immediately and do not disclose the contents to any
> other person, use it for any purpose, or store or copy the information in any
> medium. Thank you.
> >     > --
> >     > _______________________________________________
> >     > meta-virtualization mailing list
> >     > meta-virtualization at yoctoproject.org
> >     > https://lists.yoctoproject.org/listinfo/meta-virtualization
> >
> >
> >
> >     --
> >     - Thou shalt not follow the NULL pointer, for chaos and madness await
> >     thee at its end
> >     - "Use the force Harry" - Gandalf, Star Trek II
> >
> >
> 
> 
> --
> - Thou shalt not follow the NULL pointer, for chaos and madness await thee at
> its end
> - "Use the force Harry" - Gandalf, Star Trek II