Browse Source

Merge pull request #6604 from AvaloniaUI/fix/x11-freeze-segfault-on-close

[X11] Fixed potential freezes/segfaults on window close
stable/0.10.x-codice
Dan Walmsley 4 years ago
committed by Steven Kirk
parent
commit
ac7ca2e37a
  1. 33
      src/Avalonia.Visuals/Rendering/DeferredRenderer.cs
  2. 3
      src/Avalonia.X11/X11Window.cs

33
src/Avalonia.Visuals/Rendering/DeferredRenderer.cs

@ -35,6 +35,8 @@ namespace Avalonia.Rendering
private IRef<IDrawOperation> _currentDraw; private IRef<IDrawOperation> _currentDraw;
private readonly IDeferredRendererLock _lock; private readonly IDeferredRendererLock _lock;
private readonly object _sceneLock = new object(); private readonly object _sceneLock = new object();
private readonly object _startStopLock = new object();
private readonly object _renderLoopIsRenderingLock = new object();
private readonly Action _updateSceneIfNeededDelegate; private readonly Action _updateSceneIfNeededDelegate;
/// <summary> /// <summary>
@ -139,6 +141,8 @@ namespace Avalonia.Rendering
} }
Stop(); Stop();
// Wait for any in-progress rendering to complete
lock(_renderLoopIsRenderingLock){}
DisposeRenderTarget(); DisposeRenderTarget();
} }
@ -233,20 +237,26 @@ namespace Avalonia.Rendering
/// <inheritdoc/> /// <inheritdoc/>
public void Start() public void Start()
{ {
if (!_running && _renderLoop != null) lock (_startStopLock)
{ {
_renderLoop.Add(this); if (!_running && _renderLoop != null)
_running = true; {
_renderLoop.Add(this);
_running = true;
}
} }
} }
/// <inheritdoc/> /// <inheritdoc/>
public void Stop() public void Stop()
{ {
if (_running && _renderLoop != null) lock (_startStopLock)
{ {
_renderLoop.Remove(this); if (_running && _renderLoop != null)
_running = false; {
_renderLoop.Remove(this);
_running = false;
}
} }
} }
@ -255,7 +265,16 @@ namespace Avalonia.Rendering
void IRenderLoopTask.Update(TimeSpan time) => UpdateScene(); void IRenderLoopTask.Update(TimeSpan time) => UpdateScene();
void IRenderLoopTask.Render() => Render(false); void IRenderLoopTask.Render()
{
lock (_renderLoopIsRenderingLock)
{
lock(_startStopLock)
if(!_running)
return;
Render(false);
}
}
/// <inheritdoc/> /// <inheritdoc/>
Size IVisualBrushRenderer.GetRenderTargetSize(IVisualBrush brush) Size IVisualBrushRenderer.GetRenderTargetSize(IVisualBrush brush)

3
src/Avalonia.X11/X11Window.cs

@ -805,13 +805,14 @@ namespace Avalonia.X11
if (_handle != IntPtr.Zero) if (_handle != IntPtr.Zero)
{ {
XDestroyWindow(_x11.Display, _handle);
_platform.Windows.Remove(_handle); _platform.Windows.Remove(_handle);
_platform.XI2?.OnWindowDestroyed(_handle); _platform.XI2?.OnWindowDestroyed(_handle);
var handle = _handle;
_handle = IntPtr.Zero; _handle = IntPtr.Zero;
Closed?.Invoke(); Closed?.Invoke();
_mouse.Dispose(); _mouse.Dispose();
_touch.Dispose(); _touch.Dispose();
XDestroyWindow(_x11.Display, handle);
} }
if (_useRenderWindow && _renderHandle != IntPtr.Zero) if (_useRenderWindow && _renderHandle != IntPtr.Zero)

Loading…
Cancel
Save