| From ccb30ac73653ba2d65f460a353746b2e5c569d3b Mon Sep 17 00:00:00 2001 |
| From: Juergen Gross <jgross@suse.com> |
| Date: Fri, 7 Apr 2017 17:28:23 +0200 |
| Subject: [PATCH] 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. |
| |
| Cc: stable@vger.kernel.org |
| 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: Paul Gortmaker <paul.gortmaker@windriver.com> |
| |
| 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, |
| break; |
| |
| case XenbusStateInitWait: |
| -InitWait: |
| xenbus_switch_state(dev, XenbusStateConnected); |
| break; |
| |
| @@ -655,7 +654,8 @@ InitWait: |
| * get Connected twice here. |
| */ |
| if (dev->state != XenbusStateConnected) |
| - goto InitWait; /* no InitWait seen yet, fudge it */ |
| + /* no InitWait seen yet, fudge it */ |
| + xenbus_switch_state(dev, XenbusStateConnected); |
| |
| if (xenbus_scanf(XBT_NIL, info->xbdev->otherend, |
| "request-update", "%d", &val) < 0) |
| -- |
| 2.12.0 |
| |