Messages in this thread | | | Date | Fri, 26 Mar 2010 14:43:56 +0530 | From | Sachin Sant <> | Subject | Re: [PATCH 4/7] hvc_console: Fix race between hvc_close and hvc_remove |
| |
Amit Shah wrote: > On (Sun) Mar 21 2010 [10:07:25], Amit Shah wrote: > >> On (Sun) Mar 21 2010 [08:04:39], Benjamin Herrenschmidt wrote: >> >>> On Fri, 2010-03-19 at 08:18 -0700, Greg Kroah-Hartman wrote: >>> >>>> From: Amit Shah <amit.shah@redhat.com> >>>> >>>> Alan pointed out a race in the code where hvc_remove is invoked. The >>>> recent virtio_console work is the first user of hvc_remove(). >>>> >>> This causes hangs during boot on pseries machines. Haven't had a chance >>> to track that down yet, but please revert >>> e74d098c66543d0731de62eb747ccd5b636a6f4c for now. >>> <SNIP>
> > > And this suggests that hvc_kick() is called before hvc_task is > initialised, ie, before hvc_init() is called. > > Does this help? > Hi Amit,
I too ran into this issue as reported by Anton. Unfortunately in my case the following suggested patch does not help boot the powerpc box successfully. The boot still hangs with following messages
/dev/sda5: clean, 229739/2097152 files, 2485598/8377888 blocks (check in 4 mounts) fsck succeeded. Mounting root device read-write. Mounting root /dev/sda5 mount -o rw,acl,user_xattr -t ext3 /dev/sda5 /root kjournald starting. Commit interval 5 seconds EXT3-fs (sda5): using internal journal EXT3-fs (sda5): mounted filesystem with writeback data mode mount: can't find /root/proc in /etc/fstab or /etc/mtab /sbin/smart_agetty: can not determine 'console' speed
Just to confirm i too am facing same issue as reported by Anton, reverted commit e74d098c66543d0731de62eb747ccd5b636a6f4c and that allowed by box to boot successfully.
Thanks -Sachin
> diff --git a/drivers/char/hvc_console.c b/drivers/char/hvc_console.c > index ba55bba..50ac983 100644 > --- a/drivers/char/hvc_console.c > +++ b/drivers/char/hvc_console.c > @@ -285,6 +285,9 @@ EXPORT_SYMBOL_GPL(hvc_instantiate); > /* Wake the sleeping khvcd */ > void hvc_kick(void) > { > + if (!hvc_task) > + return; > + > hvc_kicked = 1; > wake_up_process(hvc_task); > } > > Amit > _______________________________________________ > Linuxppc-dev mailing list > Linuxppc-dev@lists.ozlabs.org > https://lists.ozlabs.org/listinfo/linuxppc-dev > >
--
--------------------------------- Sachin Sant IBM Linux Technology Center India Systems and Technology Labs Bangalore, India ---------------------------------
| |