diff options
| author | Juergen Gross <jgross@suse.com> | 2017-04-07 11:28:23 -0400 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2017-04-21 03:31:21 -0400 |
| commit | e485875dff38c79479ba42f80f0230d181a77a56 (patch) | |
| tree | 6b4002e3fbefda743a9ed94550ef56e35fa46ff7 /drivers/video | |
| parent | 92f8aa7bb825eafc3686cdda6dbbce44d8b35e75 (diff) | |
xen, fbfront: fix connecting to backend
commit 9121b15b5628b38b4695282dc18c553440e0f79b upstream.
Connecting to the backend isn't working reliably in xen-fbfront: in
case XenbusStateInitWait of the backend has been missed the backend
transition to XenbusStateConnected will trigger the connected state
only without doing the actions required when the backend has
connected.
Signed-off-by: Juergen Gross <jgross@suse.com>
Reviewed-by: Boris Ostrovsky <boris.ostrovsky@oracle.com>
Signed-off-by: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/video')
| -rw-r--r-- | drivers/video/fbdev/xen-fbfront.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/video/fbdev/xen-fbfront.c b/drivers/video/fbdev/xen-fbfront.c index 0567d517eed3..ea2f19f5fbde 100644 --- a/drivers/video/fbdev/xen-fbfront.c +++ b/drivers/video/fbdev/xen-fbfront.c | |||
| @@ -644,7 +644,6 @@ static void xenfb_backend_changed(struct xenbus_device *dev, | |||
| 644 | break; | 644 | break; |
| 645 | 645 | ||
| 646 | case XenbusStateInitWait: | 646 | case XenbusStateInitWait: |
| 647 | InitWait: | ||
| 648 | xenbus_switch_state(dev, XenbusStateConnected); | 647 | xenbus_switch_state(dev, XenbusStateConnected); |
| 649 | break; | 648 | break; |
| 650 | 649 | ||
| @@ -655,7 +654,8 @@ InitWait: | |||
| 655 | * get Connected twice here. | 654 | * get Connected twice here. |
| 656 | */ | 655 | */ |
| 657 | if (dev->state != XenbusStateConnected) | 656 | if (dev->state != XenbusStateConnected) |
| 658 | goto InitWait; /* no InitWait seen yet, fudge it */ | 657 | /* no InitWait seen yet, fudge it */ |
| 658 | xenbus_switch_state(dev, XenbusStateConnected); | ||
| 659 | 659 | ||
| 660 | if (xenbus_scanf(XBT_NIL, info->xbdev->otherend, | 660 | if (xenbus_scanf(XBT_NIL, info->xbdev->otherend, |
| 661 | "request-update", "%d", &val) < 0) | 661 | "request-update", "%d", &val) < 0) |
