* [PATCH] [PV Xen] Mouse stuck after save/restore of guest.
@ 2011-04-14 15:45 Igor Mammedov
2011-04-14 15:59 ` Konrad Rzeszutek Wilk
0 siblings, 1 reply; 15+ messages in thread
From: Igor Mammedov @ 2011-04-14 15:45 UTC (permalink / raw)
To: xen-devel
Mouse stuck after restore of PV guest but buttons are
in working condition.
If driver has been configured for ABS coordinates at
start it will get XENKBD_TYPE_POS events and then
suddenly after restore it'll start getting
XENKBD_TYPE_MOTION events, that will be dropped later
and they won't get into user-space.
Regression was introduced by hunk 5 and 6 of 5ea5254
in upstream.
Driver on restore should ask xen for request-abs-pointer
again if it's available. So restore parts that did it
before 5ea5254.
---
drivers/input/xen-kbdfront.c | 14 +++++++++++++-
1 files changed, 13 insertions(+), 1 deletions(-)
diff --git a/drivers/input/xen-kbdfront.c b/drivers/input/xen-kbdfront.c
index 53e6273..71e2fc5 100644
--- a/drivers/input/xen-kbdfront.c
+++ b/drivers/input/xen-kbdfront.c
@@ -286,7 +286,7 @@ static void xenkbd_backend_changed(struct xenbus_device *dev,
enum xenbus_state backend_state)
{
struct xenkbd_info *info = dev_get_drvdata(&dev->dev);
- int val;
+ int ret, val;
switch (backend_state) {
case XenbusStateInitialising:
@@ -299,6 +299,18 @@ static void xenkbd_backend_changed(struct xenbus_device *dev,
case XenbusStateInitWait:
InitWait:
+ ret = xenbus_scanf(XBT_NIL, info->xbdev->otherend,
+ "feature-abs-pointer", "%d", &val);
+ if (ret < 0)
+ val = 0;
+ if (val) {
+ ret = xenbus_printf(XBT_NIL, info->xbdev->nodename,
+ "request-abs-pointer", "1");
+ if (ret)
+ printk(KERN_WARNING
+ "xenkbd: can't request abs-pointer");
+ }
+
xenbus_switch_state(dev, XenbusStateConnected);
break;
--
1.7.1
^ permalink raw reply related [flat|nested] 15+ messages in thread
* Re: [PATCH] [PV Xen] Mouse stuck after save/restore of guest.
2011-04-14 15:45 [PATCH] [PV Xen] Mouse stuck after save/restore of guest Igor Mammedov
@ 2011-04-14 15:59 ` Konrad Rzeszutek Wilk
2011-04-14 17:26 ` Olaf Hering
0 siblings, 1 reply; 15+ messages in thread
From: Konrad Rzeszutek Wilk @ 2011-04-14 15:59 UTC (permalink / raw)
To: Igor Mammedov, olaf; +Cc: xen-devel
On Thu, Apr 14, 2011 at 05:45:07PM +0200, Igor Mammedov wrote:
> Mouse stuck after restore of PV guest but buttons are
> in working condition.
> If driver has been configured for ABS coordinates at
> start it will get XENKBD_TYPE_POS events and then
> suddenly after restore it'll start getting
> XENKBD_TYPE_MOTION events, that will be dropped later
> and they won't get into user-space.
>
> Regression was introduced by hunk 5 and 6 of 5ea5254
> in upstream.
>
> Driver on restore should ask xen for request-abs-pointer
> again if it's available. So restore parts that did it
> before 5ea5254.
Olaf?
> ---
> drivers/input/xen-kbdfront.c | 14 +++++++++++++-
> 1 files changed, 13 insertions(+), 1 deletions(-)
>
> diff --git a/drivers/input/xen-kbdfront.c b/drivers/input/xen-kbdfront.c
> index 53e6273..71e2fc5 100644
> --- a/drivers/input/xen-kbdfront.c
> +++ b/drivers/input/xen-kbdfront.c
> @@ -286,7 +286,7 @@ static void xenkbd_backend_changed(struct xenbus_device *dev,
> enum xenbus_state backend_state)
> {
> struct xenkbd_info *info = dev_get_drvdata(&dev->dev);
> - int val;
> + int ret, val;
>
> switch (backend_state) {
> case XenbusStateInitialising:
> @@ -299,6 +299,18 @@ static void xenkbd_backend_changed(struct xenbus_device *dev,
>
> case XenbusStateInitWait:
> InitWait:
> + ret = xenbus_scanf(XBT_NIL, info->xbdev->otherend,
> + "feature-abs-pointer", "%d", &val);
> + if (ret < 0)
> + val = 0;
> + if (val) {
> + ret = xenbus_printf(XBT_NIL, info->xbdev->nodename,
> + "request-abs-pointer", "1");
> + if (ret)
> + printk(KERN_WARNING
> + "xenkbd: can't request abs-pointer");
Any reason for not using the pr_warning as it was before?
> + }
> +
> xenbus_switch_state(dev, XenbusStateConnected);
> break;
>
> --
> 1.7.1
>
>
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@lists.xensource.com
> http://lists.xensource.com/xen-devel
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH] [PV Xen] Mouse stuck after save/restore of guest.
2011-04-14 15:59 ` Konrad Rzeszutek Wilk
@ 2011-04-14 17:26 ` Olaf Hering
2011-04-14 18:38 ` Konrad Rzeszutek Wilk
0 siblings, 1 reply; 15+ messages in thread
From: Olaf Hering @ 2011-04-14 17:26 UTC (permalink / raw)
To: Konrad Rzeszutek Wilk; +Cc: Igor Mammedov, xen-devel
On Thu, Apr 14, Konrad Rzeszutek Wilk wrote:
> On Thu, Apr 14, 2011 at 05:45:07PM +0200, Igor Mammedov wrote:
> > Mouse stuck after restore of PV guest but buttons are
> > in working condition.
> > If driver has been configured for ABS coordinates at
> > start it will get XENKBD_TYPE_POS events and then
> > suddenly after restore it'll start getting
> > XENKBD_TYPE_MOTION events, that will be dropped later
> > and they won't get into user-space.
> >
> > Regression was introduced by hunk 5 and 6 of 5ea5254
> > in upstream.
> >
> > Driver on restore should ask xen for request-abs-pointer
> > again if it's available. So restore parts that did it
> > before 5ea5254.
>
> Olaf?
This change is correct. Thanks for spotting, Igor.
I did not test suspend/resume, migration etc. Sorry for that. During
resume the guest has to renegotiate with the new qemu-dm process.
Olaf
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [Xen-devel] [PATCH] [PV Xen] Mouse stuck after save/restore of guest.
2011-04-14 17:26 ` Olaf Hering
@ 2011-04-14 18:38 ` Konrad Rzeszutek Wilk
0 siblings, 0 replies; 15+ messages in thread
From: Konrad Rzeszutek Wilk @ 2011-04-14 18:38 UTC (permalink / raw)
To: Olaf Hering, dtor, linux-kernel; +Cc: Igor Mammedov, xen-devel
On Thu, Apr 14, 2011 at 07:26:50PM +0200, Olaf Hering wrote:
> On Thu, Apr 14, Konrad Rzeszutek Wilk wrote:
>
> > On Thu, Apr 14, 2011 at 05:45:07PM +0200, Igor Mammedov wrote:
> > > Mouse stuck after restore of PV guest but buttons are
> > > in working condition.
> > > If driver has been configured for ABS coordinates at
> > > start it will get XENKBD_TYPE_POS events and then
> > > suddenly after restore it'll start getting
> > > XENKBD_TYPE_MOTION events, that will be dropped later
> > > and they won't get into user-space.
> > >
> > > Regression was introduced by hunk 5 and 6 of 5ea5254
> > > in upstream.
> > >
> > > Driver on restore should ask xen for request-abs-pointer
> > > again if it's available. So restore parts that did it
> > > before 5ea5254.
> >
> > Olaf?
>
> This change is correct. Thanks for spotting, Igor.
Dmitry,
Was wondering if you are OK pushing this for 2.6.39-rc3 or whether you are OK
with me doing. It fixes a regression introduced by the last hunk of
5ea5254aa0ad269cfbd2875c973ef25ab5b5e9db
The patch rebase on top of 2.6.39-rc3 looks as so:
(and the patch is in git://git.kernel.org/pub/scm/linux/kernel/git/konrad/xen.git stable/bug-fixes-rc3)
commit 876f162fe6831273fc5d678fa066f26f58be4a2c
Author: Igor Mammedov <imammedo@redhat.com>
Date: Thu Apr 14 17:45:07 2011 +0200
Input: xen-kbdfront: Mouse stuck after save/restore of guest.
Mouse stuck after restore of PV guest but buttons are
in working condition.
If driver has been configured for ABS coordinates at
start it will get XENKBD_TYPE_POS events and then
suddenly after restore it'll start getting
XENKBD_TYPE_MOTION events, that will be dropped later
and they won't get into user-space.
Regression was introduced by hunk 5 and 6 of
5ea5254aa0ad269cfbd2875c973ef25ab5b5e9db
("Input: xen-kbdfront - advertise either absolute or relative coordinates"
Driver on restore should ask xen for request-abs-pointer
again if it's available. So restore parts that did it
before 5ea5254.
CC: Dmitry Torokhov <dtor@mail.ru>
Acked-by: Olaf Hering <olaf@aepfle.de>
Signed-off-by: Igor Mammedov <imammedo@redhat.com>
[v1: Expanded the commit description]
Signed-off-by: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
diff --git a/drivers/input/misc/xen-kbdfront.c b/drivers/input/misc/xen-kbdfront.c
index 7077f9b..6d119d5 100644
--- a/drivers/input/misc/xen-kbdfront.c
+++ b/drivers/input/misc/xen-kbdfront.c
@@ -303,7 +303,7 @@ static void xenkbd_backend_changed(struct xenbus_device *dev,
enum xenbus_state backend_state)
{
struct xenkbd_info *info = dev_get_drvdata(&dev->dev);
- int val;
+ int ret, val;
switch (backend_state) {
case XenbusStateInitialising:
@@ -316,6 +316,17 @@ static void xenkbd_backend_changed(struct xenbus_device *dev,
case XenbusStateInitWait:
InitWait:
+ ret = xenbus_scanf(XBT_NIL, info->xbdev->otherend,
+ "feature-abs-pointer", "%d", &val);
+ if (ret < 0)
+ val = 0;
+ if (val) {
+ ret = xenbus_printf(XBT_NIL, info->xbdev->nodename,
+ "request-abs-pointer", "1");
+ if (ret)
+ pr_warning("xenkbd: can't request abs-pointer");
+ }
+
xenbus_switch_state(dev, XenbusStateConnected);
break;
^ permalink raw reply related [flat|nested] 15+ messages in thread
* Re: [PATCH] [PV Xen] Mouse stuck after save/restore of guest.
@ 2011-04-14 18:38 ` Konrad Rzeszutek Wilk
0 siblings, 0 replies; 15+ messages in thread
From: Konrad Rzeszutek Wilk @ 2011-04-14 18:38 UTC (permalink / raw)
To: Olaf Hering, dtor, linux-kernel; +Cc: Igor Mammedov, xen-devel
On Thu, Apr 14, 2011 at 07:26:50PM +0200, Olaf Hering wrote:
> On Thu, Apr 14, Konrad Rzeszutek Wilk wrote:
>
> > On Thu, Apr 14, 2011 at 05:45:07PM +0200, Igor Mammedov wrote:
> > > Mouse stuck after restore of PV guest but buttons are
> > > in working condition.
> > > If driver has been configured for ABS coordinates at
> > > start it will get XENKBD_TYPE_POS events and then
> > > suddenly after restore it'll start getting
> > > XENKBD_TYPE_MOTION events, that will be dropped later
> > > and they won't get into user-space.
> > >
> > > Regression was introduced by hunk 5 and 6 of 5ea5254
> > > in upstream.
> > >
> > > Driver on restore should ask xen for request-abs-pointer
> > > again if it's available. So restore parts that did it
> > > before 5ea5254.
> >
> > Olaf?
>
> This change is correct. Thanks for spotting, Igor.
Dmitry,
Was wondering if you are OK pushing this for 2.6.39-rc3 or whether you are OK
with me doing. It fixes a regression introduced by the last hunk of
5ea5254aa0ad269cfbd2875c973ef25ab5b5e9db
The patch rebase on top of 2.6.39-rc3 looks as so:
(and the patch is in git://git.kernel.org/pub/scm/linux/kernel/git/konrad/xen.git stable/bug-fixes-rc3)
commit 876f162fe6831273fc5d678fa066f26f58be4a2c
Author: Igor Mammedov <imammedo@redhat.com>
Date: Thu Apr 14 17:45:07 2011 +0200
Input: xen-kbdfront: Mouse stuck after save/restore of guest.
Mouse stuck after restore of PV guest but buttons are
in working condition.
If driver has been configured for ABS coordinates at
start it will get XENKBD_TYPE_POS events and then
suddenly after restore it'll start getting
XENKBD_TYPE_MOTION events, that will be dropped later
and they won't get into user-space.
Regression was introduced by hunk 5 and 6 of
5ea5254aa0ad269cfbd2875c973ef25ab5b5e9db
("Input: xen-kbdfront - advertise either absolute or relative coordinates"
Driver on restore should ask xen for request-abs-pointer
again if it's available. So restore parts that did it
before 5ea5254.
CC: Dmitry Torokhov <dtor@mail.ru>
Acked-by: Olaf Hering <olaf@aepfle.de>
Signed-off-by: Igor Mammedov <imammedo@redhat.com>
[v1: Expanded the commit description]
Signed-off-by: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
diff --git a/drivers/input/misc/xen-kbdfront.c b/drivers/input/misc/xen-kbdfront.c
index 7077f9b..6d119d5 100644
--- a/drivers/input/misc/xen-kbdfront.c
+++ b/drivers/input/misc/xen-kbdfront.c
@@ -303,7 +303,7 @@ static void xenkbd_backend_changed(struct xenbus_device *dev,
enum xenbus_state backend_state)
{
struct xenkbd_info *info = dev_get_drvdata(&dev->dev);
- int val;
+ int ret, val;
switch (backend_state) {
case XenbusStateInitialising:
@@ -316,6 +316,17 @@ static void xenkbd_backend_changed(struct xenbus_device *dev,
case XenbusStateInitWait:
InitWait:
+ ret = xenbus_scanf(XBT_NIL, info->xbdev->otherend,
+ "feature-abs-pointer", "%d", &val);
+ if (ret < 0)
+ val = 0;
+ if (val) {
+ ret = xenbus_printf(XBT_NIL, info->xbdev->nodename,
+ "request-abs-pointer", "1");
+ if (ret)
+ pr_warning("xenkbd: can't request abs-pointer");
+ }
+
xenbus_switch_state(dev, XenbusStateConnected);
break;
^ permalink raw reply related [flat|nested] 15+ messages in thread
* Re: [Xen-devel] [PATCH] [PV Xen] Mouse stuck after save/restore of guest.
2011-04-14 18:38 ` Konrad Rzeszutek Wilk
(?)
@ 2011-04-15 9:37 ` Olaf Hering
2011-04-19 13:46 ` Konrad Rzeszutek Wilk
2011-04-20 18:29 ` Konrad Rzeszutek Wilk
-1 siblings, 2 replies; 15+ messages in thread
From: Olaf Hering @ 2011-04-15 9:37 UTC (permalink / raw)
To: Konrad Rzeszutek Wilk; +Cc: dtor, linux-kernel, Igor Mammedov, xen-devel
On Thu, Apr 14, Konrad Rzeszutek Wilk wrote:
> On Thu, Apr 14, 2011 at 07:26:50PM +0200, Olaf Hering wrote:
> > On Thu, Apr 14, Konrad Rzeszutek Wilk wrote:
> >
> > > On Thu, Apr 14, 2011 at 05:45:07PM +0200, Igor Mammedov wrote:
> > > > Mouse stuck after restore of PV guest but buttons are
> > > > in working condition.
> > > > If driver has been configured for ABS coordinates at
> > > > start it will get XENKBD_TYPE_POS events and then
> > > > suddenly after restore it'll start getting
> > > > XENKBD_TYPE_MOTION events, that will be dropped later
> > > > and they won't get into user-space.
> > > >
> > > > Regression was introduced by hunk 5 and 6 of 5ea5254
> > > > in upstream.
> > > >
> > > > Driver on restore should ask xen for request-abs-pointer
> > > > again if it's available. So restore parts that did it
> > > > before 5ea5254.
> > >
> > > Olaf?
> >
> > This change is correct. Thanks for spotting, Igor.
>
> Dmitry,
>
> Was wondering if you are OK pushing this for 2.6.39-rc3 or whether you are OK
> with me doing. It fixes a regression introduced by the last hunk of
> 5ea5254aa0ad269cfbd2875c973ef25ab5b5e9db
Looking through my backlog, I noticed that the patch went also to
stable@kernel.org and is now in various trees already.
So please forward this revert also to stable@suse.de
Olaf
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [Xen-devel] [PATCH] [PV Xen] Mouse stuck after save/restore of guest.
2011-04-15 9:37 ` [Xen-devel] " Olaf Hering
@ 2011-04-19 13:46 ` Konrad Rzeszutek Wilk
2011-04-19 14:55 ` Olaf Hering
2011-04-20 18:29 ` Konrad Rzeszutek Wilk
1 sibling, 1 reply; 15+ messages in thread
From: Konrad Rzeszutek Wilk @ 2011-04-19 13:46 UTC (permalink / raw)
To: Olaf Hering; +Cc: dtor, xen-devel, linux-kernel, Igor Mammedov
> > Was wondering if you are OK pushing this for 2.6.39-rc3 or whether you are OK
> > with me doing. It fixes a regression introduced by the last hunk of
> > 5ea5254aa0ad269cfbd2875c973ef25ab5b5e9db
>
> Looking through my backlog, I noticed that the patch went also to
> stable@kernel.org and is now in various trees already.
> So please forward this revert also to stable@suse.de
OK. The upstream git commit is c36b58e8a9112017c2bcc322cc98e71241814303.
Which stable tree did it go through? 2.6.38 only I presume?
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [Xen-devel] [PATCH] [PV Xen] Mouse stuck after save/restore of guest.
2011-04-19 13:46 ` Konrad Rzeszutek Wilk
@ 2011-04-19 14:55 ` Olaf Hering
2011-04-20 9:19 ` Igor Mammedov
0 siblings, 1 reply; 15+ messages in thread
From: Olaf Hering @ 2011-04-19 14:55 UTC (permalink / raw)
To: Konrad Rzeszutek Wilk; +Cc: dtor, xen-devel, linux-kernel, Igor Mammedov
On Tue, Apr 19, Konrad Rzeszutek Wilk wrote:
> > > Was wondering if you are OK pushing this for 2.6.39-rc3 or whether you are OK
> > > with me doing. It fixes a regression introduced by the last hunk of
> > > 5ea5254aa0ad269cfbd2875c973ef25ab5b5e9db
> >
> > Looking through my backlog, I noticed that the patch went also to
> > stable@kernel.org and is now in various trees already.
> > So please forward this revert also to stable@suse.de
>
> OK. The upstream git commit is c36b58e8a9112017c2bcc322cc98e71241814303.
>
> Which stable tree did it go through? 2.6.38 only I presume?
All longterm versions, 2.6.32 and up.
Olaf
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [Xen-devel] [PATCH] [PV Xen] Mouse stuck after save/restore of guest.
2011-04-19 14:55 ` Olaf Hering
@ 2011-04-20 9:19 ` Igor Mammedov
0 siblings, 0 replies; 15+ messages in thread
From: Igor Mammedov @ 2011-04-20 9:19 UTC (permalink / raw)
To: Olaf Hering; +Cc: Konrad Rzeszutek Wilk, dtor, xen-devel, linux-kernel
On 04/19/2011 04:55 PM, Olaf Hering wrote:
> On Tue, Apr 19, Konrad Rzeszutek Wilk wrote:
>
>>>> Was wondering if you are OK pushing this for 2.6.39-rc3 or whether you are OK
>>>> with me doing. It fixes a regression introduced by the last hunk of
>>>> 5ea5254aa0ad269cfbd2875c973ef25ab5b5e9db
>>> Looking through my backlog, I noticed that the patch went also to
>>> stable@kernel.org and is now in various trees already.
>>> So please forward this revert also to stable@suse.de
>> OK. The upstream git commit is c36b58e8a9112017c2bcc322cc98e71241814303.
>>
>> Which stable tree did it go through? 2.6.38 only I presume?
> All longterm versions, 2.6.32 and up.
In Fedora 13 it still works (2.6.34.*)
> Olaf
>
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@lists.xensource.com
> http://lists.xensource.com/xen-devel
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH] [PV Xen] Mouse stuck after save/restore of guest.
@ 2011-04-20 9:19 ` Igor Mammedov
0 siblings, 0 replies; 15+ messages in thread
From: Igor Mammedov @ 2011-04-20 9:19 UTC (permalink / raw)
To: Olaf Hering; +Cc: dtor, xen-devel, linux-kernel, Konrad Rzeszutek Wilk
On 04/19/2011 04:55 PM, Olaf Hering wrote:
> On Tue, Apr 19, Konrad Rzeszutek Wilk wrote:
>
>>>> Was wondering if you are OK pushing this for 2.6.39-rc3 or whether you are OK
>>>> with me doing. It fixes a regression introduced by the last hunk of
>>>> 5ea5254aa0ad269cfbd2875c973ef25ab5b5e9db
>>> Looking through my backlog, I noticed that the patch went also to
>>> stable@kernel.org and is now in various trees already.
>>> So please forward this revert also to stable@suse.de
>> OK. The upstream git commit is c36b58e8a9112017c2bcc322cc98e71241814303.
>>
>> Which stable tree did it go through? 2.6.38 only I presume?
> All longterm versions, 2.6.32 and up.
In Fedora 13 it still works (2.6.34.*)
> Olaf
>
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@lists.xensource.com
> http://lists.xensource.com/xen-devel
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [Xen-devel] [PATCH] [PV Xen] Mouse stuck after save/restore of guest. - stable tree candidate.
2011-04-15 9:37 ` [Xen-devel] " Olaf Hering
@ 2011-04-20 18:29 ` Konrad Rzeszutek Wilk
2011-04-20 18:29 ` Konrad Rzeszutek Wilk
1 sibling, 0 replies; 15+ messages in thread
From: Konrad Rzeszutek Wilk @ 2011-04-20 18:29 UTC (permalink / raw)
To: stable, stable; +Cc: dtor, xen-devel, linux-kernel, Igor Mammedov, olaf
[-- Attachment #1: Type: text/plain, Size: 3645 bytes --]
> > > > On Thu, Apr 14, 2011 at 05:45:07PM +0200, Igor Mammedov wrote:
> > > > > Mouse stuck after restore of PV guest but buttons are
> > > > > in working condition.
> > > > > If driver has been configured for ABS coordinates at
> > > > > start it will get XENKBD_TYPE_POS events and then
> > > > > suddenly after restore it'll start getting
> > > > > XENKBD_TYPE_MOTION events, that will be dropped later
> > > > > and they won't get into user-space.
> > > > >
> > > > > Regression was introduced by hunk 5 and 6 of 5ea5254
> > > > > in upstream.
> > > > >
> > > > > Driver on restore should ask xen for request-abs-pointer
> > > > > again if it's available. So restore parts that did it
> > > > > before 5ea5254.
> > > >
> > > > Olaf?
> > >
> > > This change is correct. Thanks for spotting, Igor.
> >
> > Dmitry,
> >
> > Was wondering if you are OK pushing this for 2.6.39-rc3 or whether you are OK
> > with me doing. It fixes a regression introduced by the last hunk of
> > 5ea5254aa0ad269cfbd2875c973ef25ab5b5e9db
Hello stable-tree maintainers.
Please apply the upstream git commit c36b58e8a9112017c2bcc322cc98e71241814303
(Input: xen-kbdfront - fix mouse getting stuck after save/restore)
which fixes a regression introduced by 8c3c283e6bf463ab498d6e7823aff6c4762314b6
(Input: xen-kbdfront - advertise either absolute or relative coordinates)
The 2.6.38 and 2.6.37 tree both contain the regression.
For convience, attached and inline is the patch for stable tree submission.
commit c36b58e8a9112017c2bcc322cc98e71241814303
Author: Igor Mammedov <imammedo@redhat.com>
Date: Mon Apr 18 10:17:17 2011 -0700
Input: xen-kbdfront - fix mouse getting stuck after save/restore
Mouse gets "stuck" after restore of PV guest but buttons are in working
condition.
If driver has been configured for ABS coordinates at start it will get
XENKBD_TYPE_POS events and then suddenly after restore it'll start getting
XENKBD_TYPE_MOTION events, that will be dropped later and they won't get
into user-space.
Regression was introduced by hunk 5 and 6 of
5ea5254aa0ad269cfbd2875c973ef25ab5b5e9db
("Input: xen-kbdfront - advertise either absolute or relative
coordinates").
Driver on restore should ask xen for request-abs-pointer again if it is
available. So restore parts that did it before 5ea5254.
Acked-by: Olaf Hering <olaf@aepfle.de>
Signed-off-by: Igor Mammedov <imammedo@redhat.com>
[v1: Expanded the commit description]
Signed-off-by: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
Signed-off-by: Dmitry Torokhov <dtor@mail.ru>
diff --git a/drivers/input/misc/xen-kbdfront.c b/drivers/input/misc/xen-kbdfront.c
index 7077f9b..62bae99 100644
--- a/drivers/input/misc/xen-kbdfront.c
+++ b/drivers/input/misc/xen-kbdfront.c
@@ -303,7 +303,7 @@ static void xenkbd_backend_changed(struct xenbus_device *dev,
enum xenbus_state backend_state)
{
struct xenkbd_info *info = dev_get_drvdata(&dev->dev);
- int val;
+ int ret, val;
switch (backend_state) {
case XenbusStateInitialising:
@@ -316,6 +316,17 @@ static void xenkbd_backend_changed(struct xenbus_device *dev,
case XenbusStateInitWait:
InitWait:
+ ret = xenbus_scanf(XBT_NIL, info->xbdev->otherend,
+ "feature-abs-pointer", "%d", &val);
+ if (ret < 0)
+ val = 0;
+ if (val) {
+ ret = xenbus_printf(XBT_NIL, info->xbdev->nodename,
+ "request-abs-pointer", "1");
+ if (ret)
+ pr_warning("xenkbd: can't request abs-pointer");
+ }
+
xenbus_switch_state(dev, XenbusStateConnected);
break;
[-- Attachment #2: c36b58e8a9112017c2bcc322cc98e71241814303.patch --]
[-- Type: text/x-diff, Size: 2102 bytes --]
commit c36b58e8a9112017c2bcc322cc98e71241814303
Author: Igor Mammedov <imammedo@redhat.com>
Date: Mon Apr 18 10:17:17 2011 -0700
Input: xen-kbdfront - fix mouse getting stuck after save/restore
Mouse gets "stuck" after restore of PV guest but buttons are in working
condition.
If driver has been configured for ABS coordinates at start it will get
XENKBD_TYPE_POS events and then suddenly after restore it'll start getting
XENKBD_TYPE_MOTION events, that will be dropped later and they won't get
into user-space.
Regression was introduced by hunk 5 and 6 of
5ea5254aa0ad269cfbd2875c973ef25ab5b5e9db
("Input: xen-kbdfront - advertise either absolute or relative
coordinates").
Driver on restore should ask xen for request-abs-pointer again if it is
available. So restore parts that did it before 5ea5254.
Acked-by: Olaf Hering <olaf@aepfle.de>
Signed-off-by: Igor Mammedov <imammedo@redhat.com>
[v1: Expanded the commit description]
Signed-off-by: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
Signed-off-by: Dmitry Torokhov <dtor@mail.ru>
diff --git a/drivers/input/misc/xen-kbdfront.c b/drivers/input/misc/xen-kbdfront.c
index 7077f9b..62bae99 100644
--- a/drivers/input/misc/xen-kbdfront.c
+++ b/drivers/input/misc/xen-kbdfront.c
@@ -303,7 +303,7 @@ static void xenkbd_backend_changed(struct xenbus_device *dev,
enum xenbus_state backend_state)
{
struct xenkbd_info *info = dev_get_drvdata(&dev->dev);
- int val;
+ int ret, val;
switch (backend_state) {
case XenbusStateInitialising:
@@ -316,6 +316,17 @@ static void xenkbd_backend_changed(struct xenbus_device *dev,
case XenbusStateInitWait:
InitWait:
+ ret = xenbus_scanf(XBT_NIL, info->xbdev->otherend,
+ "feature-abs-pointer", "%d", &val);
+ if (ret < 0)
+ val = 0;
+ if (val) {
+ ret = xenbus_printf(XBT_NIL, info->xbdev->nodename,
+ "request-abs-pointer", "1");
+ if (ret)
+ pr_warning("xenkbd: can't request abs-pointer");
+ }
+
xenbus_switch_state(dev, XenbusStateConnected);
break;
^ permalink raw reply related [flat|nested] 15+ messages in thread
* Re: [PATCH] [PV Xen] Mouse stuck after save/restore of guest. - stable tree candidate.
@ 2011-04-20 18:29 ` Konrad Rzeszutek Wilk
0 siblings, 0 replies; 15+ messages in thread
From: Konrad Rzeszutek Wilk @ 2011-04-20 18:29 UTC (permalink / raw)
To: stable, stable; +Cc: dtor, olaf, xen-devel, linux-kernel, Igor Mammedov
[-- Attachment #1: Type: text/plain, Size: 3645 bytes --]
> > > > On Thu, Apr 14, 2011 at 05:45:07PM +0200, Igor Mammedov wrote:
> > > > > Mouse stuck after restore of PV guest but buttons are
> > > > > in working condition.
> > > > > If driver has been configured for ABS coordinates at
> > > > > start it will get XENKBD_TYPE_POS events and then
> > > > > suddenly after restore it'll start getting
> > > > > XENKBD_TYPE_MOTION events, that will be dropped later
> > > > > and they won't get into user-space.
> > > > >
> > > > > Regression was introduced by hunk 5 and 6 of 5ea5254
> > > > > in upstream.
> > > > >
> > > > > Driver on restore should ask xen for request-abs-pointer
> > > > > again if it's available. So restore parts that did it
> > > > > before 5ea5254.
> > > >
> > > > Olaf?
> > >
> > > This change is correct. Thanks for spotting, Igor.
> >
> > Dmitry,
> >
> > Was wondering if you are OK pushing this for 2.6.39-rc3 or whether you are OK
> > with me doing. It fixes a regression introduced by the last hunk of
> > 5ea5254aa0ad269cfbd2875c973ef25ab5b5e9db
Hello stable-tree maintainers.
Please apply the upstream git commit c36b58e8a9112017c2bcc322cc98e71241814303
(Input: xen-kbdfront - fix mouse getting stuck after save/restore)
which fixes a regression introduced by 8c3c283e6bf463ab498d6e7823aff6c4762314b6
(Input: xen-kbdfront - advertise either absolute or relative coordinates)
The 2.6.38 and 2.6.37 tree both contain the regression.
For convience, attached and inline is the patch for stable tree submission.
commit c36b58e8a9112017c2bcc322cc98e71241814303
Author: Igor Mammedov <imammedo@redhat.com>
Date: Mon Apr 18 10:17:17 2011 -0700
Input: xen-kbdfront - fix mouse getting stuck after save/restore
Mouse gets "stuck" after restore of PV guest but buttons are in working
condition.
If driver has been configured for ABS coordinates at start it will get
XENKBD_TYPE_POS events and then suddenly after restore it'll start getting
XENKBD_TYPE_MOTION events, that will be dropped later and they won't get
into user-space.
Regression was introduced by hunk 5 and 6 of
5ea5254aa0ad269cfbd2875c973ef25ab5b5e9db
("Input: xen-kbdfront - advertise either absolute or relative
coordinates").
Driver on restore should ask xen for request-abs-pointer again if it is
available. So restore parts that did it before 5ea5254.
Acked-by: Olaf Hering <olaf@aepfle.de>
Signed-off-by: Igor Mammedov <imammedo@redhat.com>
[v1: Expanded the commit description]
Signed-off-by: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
Signed-off-by: Dmitry Torokhov <dtor@mail.ru>
diff --git a/drivers/input/misc/xen-kbdfront.c b/drivers/input/misc/xen-kbdfront.c
index 7077f9b..62bae99 100644
--- a/drivers/input/misc/xen-kbdfront.c
+++ b/drivers/input/misc/xen-kbdfront.c
@@ -303,7 +303,7 @@ static void xenkbd_backend_changed(struct xenbus_device *dev,
enum xenbus_state backend_state)
{
struct xenkbd_info *info = dev_get_drvdata(&dev->dev);
- int val;
+ int ret, val;
switch (backend_state) {
case XenbusStateInitialising:
@@ -316,6 +316,17 @@ static void xenkbd_backend_changed(struct xenbus_device *dev,
case XenbusStateInitWait:
InitWait:
+ ret = xenbus_scanf(XBT_NIL, info->xbdev->otherend,
+ "feature-abs-pointer", "%d", &val);
+ if (ret < 0)
+ val = 0;
+ if (val) {
+ ret = xenbus_printf(XBT_NIL, info->xbdev->nodename,
+ "request-abs-pointer", "1");
+ if (ret)
+ pr_warning("xenkbd: can't request abs-pointer");
+ }
+
xenbus_switch_state(dev, XenbusStateConnected);
break;
[-- Attachment #2: c36b58e8a9112017c2bcc322cc98e71241814303.patch --]
[-- Type: text/x-diff, Size: 2102 bytes --]
commit c36b58e8a9112017c2bcc322cc98e71241814303
Author: Igor Mammedov <imammedo@redhat.com>
Date: Mon Apr 18 10:17:17 2011 -0700
Input: xen-kbdfront - fix mouse getting stuck after save/restore
Mouse gets "stuck" after restore of PV guest but buttons are in working
condition.
If driver has been configured for ABS coordinates at start it will get
XENKBD_TYPE_POS events and then suddenly after restore it'll start getting
XENKBD_TYPE_MOTION events, that will be dropped later and they won't get
into user-space.
Regression was introduced by hunk 5 and 6 of
5ea5254aa0ad269cfbd2875c973ef25ab5b5e9db
("Input: xen-kbdfront - advertise either absolute or relative
coordinates").
Driver on restore should ask xen for request-abs-pointer again if it is
available. So restore parts that did it before 5ea5254.
Acked-by: Olaf Hering <olaf@aepfle.de>
Signed-off-by: Igor Mammedov <imammedo@redhat.com>
[v1: Expanded the commit description]
Signed-off-by: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
Signed-off-by: Dmitry Torokhov <dtor@mail.ru>
diff --git a/drivers/input/misc/xen-kbdfront.c b/drivers/input/misc/xen-kbdfront.c
index 7077f9b..62bae99 100644
--- a/drivers/input/misc/xen-kbdfront.c
+++ b/drivers/input/misc/xen-kbdfront.c
@@ -303,7 +303,7 @@ static void xenkbd_backend_changed(struct xenbus_device *dev,
enum xenbus_state backend_state)
{
struct xenkbd_info *info = dev_get_drvdata(&dev->dev);
- int val;
+ int ret, val;
switch (backend_state) {
case XenbusStateInitialising:
@@ -316,6 +316,17 @@ static void xenkbd_backend_changed(struct xenbus_device *dev,
case XenbusStateInitWait:
InitWait:
+ ret = xenbus_scanf(XBT_NIL, info->xbdev->otherend,
+ "feature-abs-pointer", "%d", &val);
+ if (ret < 0)
+ val = 0;
+ if (val) {
+ ret = xenbus_printf(XBT_NIL, info->xbdev->nodename,
+ "request-abs-pointer", "1");
+ if (ret)
+ pr_warning("xenkbd: can't request abs-pointer");
+ }
+
xenbus_switch_state(dev, XenbusStateConnected);
break;
[-- Attachment #3: Type: text/plain, Size: 138 bytes --]
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xensource.com
http://lists.xensource.com/xen-devel
^ permalink raw reply related [flat|nested] 15+ messages in thread
* Re: [Xen-devel] [PATCH] [PV Xen] Mouse stuck after save/restore of guest. - stable tree candidate.
2011-04-20 18:29 ` Konrad Rzeszutek Wilk
(?)
@ 2011-04-27 16:31 ` Olaf Hering
2011-04-27 23:32 ` [stable] " Greg KH
-1 siblings, 1 reply; 15+ messages in thread
From: Olaf Hering @ 2011-04-27 16:31 UTC (permalink / raw)
To: stable
Cc: Konrad Rzeszutek Wilk, dtor, xen-devel, linux-kernel, Igor Mammedov
On Wed, Apr 20, Konrad Rzeszutek Wilk wrote:
> Hello stable-tree maintainers.
>
> Please apply the upstream git commit c36b58e8a9112017c2bcc322cc98e71241814303
> (Input: xen-kbdfront - fix mouse getting stuck after save/restore)
> which fixes a regression introduced by 8c3c283e6bf463ab498d6e7823aff6c4762314b6
> (Input: xen-kbdfront - advertise either absolute or relative coordinates)
>
> The 2.6.38 and 2.6.37 tree both contain the regression.
I checked which trees contain the broken commit from mainline:
broken stable trees:
2.6.38
http://git.kernel.org/?p=linux/kernel/git/stable/linux-2.6.38.y.git;a=commit;h=5ea5254aa0ad269cfbd2875c973ef25ab5b5e9db
2.6.37
http://git.kernel.org/?p=linux/kernel/git/stable/linux-2.6.37.y.git;a=commit;h=723517a796adf59f9ae628629fcb606c4cec8715
2.6.35
http://git.kernel.org/?p=linux/kernel/git/longterm/linux-2.6.35.y.git;a=commit;h=b88505e58ce6d36732cd15841cc821f97efca9bb
2.6.33
http://git.kernel.org/?p=linux/kernel/git/longterm/linux-2.6.33.y.git;a=commit;h=6997348a861d3540085bf9adf4166b86ac7a96e5
2.6.32
http://git.kernel.org/?p=linux/kernel/git/longterm/linux-2.6.32.y.git;a=commit;h=95e7148d08d0578687ea26cd3a8925aa5f7bc8fa
2.6.27
Does not have a variant of 8c3c283e6bf463ab498d6e7823aff6c4762314b6, yet.
So please apply upstream commit c36b58e8a9112017c2bcc322cc98e71241814303
to 2.6.38, 2.6.37, 2.6.35, 2.6.33 and 2.6.32 to fix the introduced
regression.
Thanks.
Olaf
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [stable] [Xen-devel] [PATCH] [PV Xen] Mouse stuck after save/restore of guest. - stable tree candidate.
2011-04-27 16:31 ` [Xen-devel] " Olaf Hering
@ 2011-04-27 23:32 ` Greg KH
0 siblings, 0 replies; 15+ messages in thread
From: Greg KH @ 2011-04-27 23:32 UTC (permalink / raw)
To: Olaf Hering
Cc: stable, dtor, Igor Mammedov, xen-devel, linux-kernel,
Konrad Rzeszutek Wilk
On Wed, Apr 27, 2011 at 06:31:43PM +0200, Olaf Hering wrote:
> On Wed, Apr 20, Konrad Rzeszutek Wilk wrote:
>
> > Hello stable-tree maintainers.
> >
> > Please apply the upstream git commit c36b58e8a9112017c2bcc322cc98e71241814303
> > (Input: xen-kbdfront - fix mouse getting stuck after save/restore)
> > which fixes a regression introduced by 8c3c283e6bf463ab498d6e7823aff6c4762314b6
> > (Input: xen-kbdfront - advertise either absolute or relative coordinates)
> >
> > The 2.6.38 and 2.6.37 tree both contain the regression.
>
> I checked which trees contain the broken commit from mainline:
>
> broken stable trees:
> 2.6.38
> http://git.kernel.org/?p=linux/kernel/git/stable/linux-2.6.38.y.git;a=commit;h=5ea5254aa0ad269cfbd2875c973ef25ab5b5e9db
> 2.6.37
> http://git.kernel.org/?p=linux/kernel/git/stable/linux-2.6.37.y.git;a=commit;h=723517a796adf59f9ae628629fcb606c4cec8715
> 2.6.35
> http://git.kernel.org/?p=linux/kernel/git/longterm/linux-2.6.35.y.git;a=commit;h=b88505e58ce6d36732cd15841cc821f97efca9bb
> 2.6.33
> http://git.kernel.org/?p=linux/kernel/git/longterm/linux-2.6.33.y.git;a=commit;h=6997348a861d3540085bf9adf4166b86ac7a96e5
> 2.6.32
> http://git.kernel.org/?p=linux/kernel/git/longterm/linux-2.6.32.y.git;a=commit;h=95e7148d08d0578687ea26cd3a8925aa5f7bc8fa
> 2.6.27
> Does not have a variant of 8c3c283e6bf463ab498d6e7823aff6c4762314b6, yet.
>
> So please apply upstream commit c36b58e8a9112017c2bcc322cc98e71241814303
> to 2.6.38, 2.6.37, 2.6.35, 2.6.33 and 2.6.32 to fix the introduced
> regression.
That commit fails to apply on the .38-stable tree, or anything else I
tried. Please provide a backported version if you wish to have it
applied.
Note that the .37-stable tree is now closed, so that doesn't matter
anymore.
thanks,
greg k-h
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [stable] [PATCH] [PV Xen] Mouse stuck after save/restore of guest. - stable tree candidate.
@ 2011-04-27 23:32 ` Greg KH
0 siblings, 0 replies; 15+ messages in thread
From: Greg KH @ 2011-04-27 23:32 UTC (permalink / raw)
To: Olaf Hering
Cc: xen-devel, Konrad Rzeszutek Wilk, dtor, linux-kernel,
Igor Mammedov, stable
On Wed, Apr 27, 2011 at 06:31:43PM +0200, Olaf Hering wrote:
> On Wed, Apr 20, Konrad Rzeszutek Wilk wrote:
>
> > Hello stable-tree maintainers.
> >
> > Please apply the upstream git commit c36b58e8a9112017c2bcc322cc98e71241814303
> > (Input: xen-kbdfront - fix mouse getting stuck after save/restore)
> > which fixes a regression introduced by 8c3c283e6bf463ab498d6e7823aff6c4762314b6
> > (Input: xen-kbdfront - advertise either absolute or relative coordinates)
> >
> > The 2.6.38 and 2.6.37 tree both contain the regression.
>
> I checked which trees contain the broken commit from mainline:
>
> broken stable trees:
> 2.6.38
> http://git.kernel.org/?p=linux/kernel/git/stable/linux-2.6.38.y.git;a=commit;h=5ea5254aa0ad269cfbd2875c973ef25ab5b5e9db
> 2.6.37
> http://git.kernel.org/?p=linux/kernel/git/stable/linux-2.6.37.y.git;a=commit;h=723517a796adf59f9ae628629fcb606c4cec8715
> 2.6.35
> http://git.kernel.org/?p=linux/kernel/git/longterm/linux-2.6.35.y.git;a=commit;h=b88505e58ce6d36732cd15841cc821f97efca9bb
> 2.6.33
> http://git.kernel.org/?p=linux/kernel/git/longterm/linux-2.6.33.y.git;a=commit;h=6997348a861d3540085bf9adf4166b86ac7a96e5
> 2.6.32
> http://git.kernel.org/?p=linux/kernel/git/longterm/linux-2.6.32.y.git;a=commit;h=95e7148d08d0578687ea26cd3a8925aa5f7bc8fa
> 2.6.27
> Does not have a variant of 8c3c283e6bf463ab498d6e7823aff6c4762314b6, yet.
>
> So please apply upstream commit c36b58e8a9112017c2bcc322cc98e71241814303
> to 2.6.38, 2.6.37, 2.6.35, 2.6.33 and 2.6.32 to fix the introduced
> regression.
That commit fails to apply on the .38-stable tree, or anything else I
tried. Please provide a backported version if you wish to have it
applied.
Note that the .37-stable tree is now closed, so that doesn't matter
anymore.
thanks,
greg k-h
^ permalink raw reply [flat|nested] 15+ messages in thread
end of thread, other threads:[~2011-04-27 23:36 UTC | newest]
Thread overview: 15+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-04-14 15:45 [PATCH] [PV Xen] Mouse stuck after save/restore of guest Igor Mammedov
2011-04-14 15:59 ` Konrad Rzeszutek Wilk
2011-04-14 17:26 ` Olaf Hering
2011-04-14 18:38 ` [Xen-devel] " Konrad Rzeszutek Wilk
2011-04-14 18:38 ` Konrad Rzeszutek Wilk
2011-04-15 9:37 ` [Xen-devel] " Olaf Hering
2011-04-19 13:46 ` Konrad Rzeszutek Wilk
2011-04-19 14:55 ` Olaf Hering
2011-04-20 9:19 ` Igor Mammedov
2011-04-20 9:19 ` Igor Mammedov
2011-04-20 18:29 ` [Xen-devel] [PATCH] [PV Xen] Mouse stuck after save/restore of guest. - stable tree candidate Konrad Rzeszutek Wilk
2011-04-20 18:29 ` Konrad Rzeszutek Wilk
2011-04-27 16:31 ` [Xen-devel] " Olaf Hering
2011-04-27 23:32 ` [stable] " Greg KH
2011-04-27 23:32 ` [stable] " Greg KH
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.