Re: [Spice-devel] [PATCH] how can i trace monitor change (etc) events

2014-05-08 Thread David Mansfield
On 05/08/2014 07:26 PM, David Airlie wrote: - Original Message - I have an explanation for this, but not a fix. The fix needs to be made by the owner of this code (Alon or Dave according to the header!) The bug lies in qxl_display.c:qxl_crtc_mode_set. In this method, there is a condi

Re: [Spice-devel] how can i trace monitor change (etc) events

2014-05-08 Thread David Airlie
> > - Original Message - > > I have an explanation for this, but not a fix. The fix needs to be made > > by the owner of this code (Alon or Dave according to the header!) > > > > The bug lies in qxl_display.c:qxl_crtc_mode_set. In this method, there > > is a conditional termed "recreat

Re: [Spice-devel] how can i trace monitor change (etc) events

2014-05-08 Thread Marc-André Lureau
Hi - Original Message - > > On 05/08/2014 10:59 AM, Marc-André Lureau wrote: > > Hi > > > > Thanks a lot for your analysis so far! > > > > Adding David Airlie in CC > > > > - Original Message - > >> I have an explanation for this, but not a fix. The fix needs to be made > >> by t

Re: [Spice-devel] how can i trace monitor change (etc) events

2014-05-08 Thread David Mansfield
On 05/08/2014 10:59 AM, Marc-André Lureau wrote: Hi Thanks a lot for your analysis so far! Adding David Airlie in CC - Original Message - I have an explanation for this, but not a fix. The fix needs to be made by the owner of this code (Alon or Dave according to the header!) The bu

Re: [Spice-devel] how can i trace monitor change (etc) events

2014-05-08 Thread Marc-André Lureau
Hi Thanks a lot for your analysis so far! Adding David Airlie in CC - Original Message - > I have an explanation for this, but not a fix. The fix needs to be made > by the owner of this code (Alon or Dave according to the header!) > > The bug lies in qxl_display.c:qxl_crtc_mode_set. I

Re: [Spice-devel] how can i trace monitor change (etc) events

2014-05-08 Thread David Mansfield
On 05/07/2014 05:41 PM, David Mansfield wrote: On 05/02/2014 09:05 AM, Marc-André Lureau wrote: Hi FYI: I have been running with the attached patch (not the inline above) to spice-gtk for one week now, and so has my colleague. Dual monitor works perfectly. There is one other crash scenario