wined3d: Use WGL_SWAP_COPY_ARB for swapchains with COPY swap effects, if available.

Avoiding the (slow) GDI present path.

Signed-off-by: Henri Verbeet <hverbeet@codeweavers.com>
Signed-off-by: Alexandre Julliard <julliard@winehq.org>
This commit is contained in:
Henri Verbeet 2021-09-10 21:32:50 +02:00 committed by Alexandre Julliard
parent 6181b0ea27
commit 4261369e5d
4 changed files with 28 additions and 15 deletions

View File

@ -4120,8 +4120,8 @@ static void wined3d_adapter_init_fb_cfgs(struct wined3d_adapter_gl *adapter_gl,
{ {
UINT attrib_count = 0; UINT attrib_count = 0;
GLint cfg_count; GLint cfg_count;
int attribs[11]; int attribs[12];
int values[11]; int values[12];
int attribute; int attribute;
attribute = WGL_NUMBER_PIXEL_FORMATS_ARB; attribute = WGL_NUMBER_PIXEL_FORMATS_ARB;
@ -4139,6 +4139,7 @@ static void wined3d_adapter_init_fb_cfgs(struct wined3d_adapter_gl *adapter_gl,
attribs[attrib_count++] = WGL_PIXEL_TYPE_ARB; attribs[attrib_count++] = WGL_PIXEL_TYPE_ARB;
attribs[attrib_count++] = WGL_DOUBLE_BUFFER_ARB; attribs[attrib_count++] = WGL_DOUBLE_BUFFER_ARB;
attribs[attrib_count++] = WGL_AUX_BUFFERS_ARB; attribs[attrib_count++] = WGL_AUX_BUFFERS_ARB;
attribs[attrib_count++] = WGL_SWAP_METHOD_ARB;
for (i = 0, adapter_gl->pixel_format_count = 0; i < cfg_count; ++i) for (i = 0, adapter_gl->pixel_format_count = 0; i < cfg_count; ++i)
{ {
@ -4160,6 +4161,7 @@ static void wined3d_adapter_init_fb_cfgs(struct wined3d_adapter_gl *adapter_gl,
cfg->iPixelType = values[8]; cfg->iPixelType = values[8];
cfg->doubleBuffer = values[9]; cfg->doubleBuffer = values[9];
cfg->auxBuffers = values[10]; cfg->auxBuffers = values[10];
cfg->swap_method = values[11];
cfg->numSamples = 0; cfg->numSamples = 0;
/* Check multisample support. */ /* Check multisample support. */
@ -4225,6 +4227,7 @@ static void wined3d_adapter_init_fb_cfgs(struct wined3d_adapter_gl *adapter_gl,
cfg->iPixelType = (pfd.iPixelType == PFD_TYPE_RGBA) ? WGL_TYPE_RGBA_ARB : WGL_TYPE_COLORINDEX_ARB; cfg->iPixelType = (pfd.iPixelType == PFD_TYPE_RGBA) ? WGL_TYPE_RGBA_ARB : WGL_TYPE_COLORINDEX_ARB;
cfg->doubleBuffer = (pfd.dwFlags & PFD_DOUBLEBUFFER) ? 1 : 0; cfg->doubleBuffer = (pfd.dwFlags & PFD_DOUBLEBUFFER) ? 1 : 0;
cfg->auxBuffers = pfd.cAuxBuffers; cfg->auxBuffers = pfd.cAuxBuffers;
cfg->swap_method = WGL_SWAP_UNDEFINED_ARB;
cfg->numSamples = 0; cfg->numSamples = 0;
TRACE("iPixelFormat=%d, iPixelType=%#x, doubleBuffer=%d, RGBA=%d/%d/%d/%d, " TRACE("iPixelFormat=%d, iPixelType=%#x, doubleBuffer=%d, RGBA=%d/%d/%d/%d, "

View File

@ -1663,7 +1663,7 @@ static void wined3d_context_gl_enter(struct wined3d_context_gl *context_gl)
/* This function takes care of wined3d pixel format selection. */ /* This function takes care of wined3d pixel format selection. */
static int context_choose_pixel_format(const struct wined3d_device *device, HDC hdc, static int context_choose_pixel_format(const struct wined3d_device *device, HDC hdc,
const struct wined3d_format *color_format, const struct wined3d_format *ds_format, const struct wined3d_format *color_format, const struct wined3d_format *ds_format,
BOOL auxBuffers) bool aux_buffers, bool swap_effect_copy)
{ {
unsigned int cfg_count = wined3d_adapter_gl(device->adapter)->pixel_format_count; unsigned int cfg_count = wined3d_adapter_gl(device->adapter)->pixel_format_count;
unsigned int current_value; unsigned int current_value;
@ -1671,9 +1671,9 @@ static int context_choose_pixel_format(const struct wined3d_device *device, HDC
int iPixelFormat = 0; int iPixelFormat = 0;
unsigned int i; unsigned int i;
TRACE("device %p, dc %p, color_format %s, ds_format %s, aux_buffers %#x.\n", TRACE("device %p, dc %p, color_format %s, ds_format %s, aux_buffers %#x, swap_effect_copy %#x.\n",
device, hdc, debug_d3dformat(color_format->id), debug_d3dformat(ds_format->id), device, hdc, debug_d3dformat(color_format->id), debug_d3dformat(ds_format->id),
auxBuffers); aux_buffers, swap_effect_copy);
current_value = 0; current_value = 0;
for (i = 0; i < cfg_count; ++i) for (i = 0; i < cfg_count; ++i)
@ -1705,21 +1705,23 @@ static int context_choose_pixel_format(const struct wined3d_device *device, HDC
continue; continue;
value = 1; value = 1;
if (swap_effect_copy && cfg->swap_method == WGL_SWAP_COPY_ARB)
value += 1;
/* We try to locate a format which matches our requirements exactly. In case of /* We try to locate a format which matches our requirements exactly. In case of
* depth it is no problem to emulate 16-bit using e.g. 24-bit, so accept that. */ * depth it is no problem to emulate 16-bit using e.g. 24-bit, so accept that. */
if (cfg->depthSize == ds_format->depth_size) if (cfg->depthSize == ds_format->depth_size)
value += 1;
if (cfg->stencilSize == ds_format->stencil_size)
value += 2; value += 2;
if (cfg->alphaSize == color_format->alpha_size) if (cfg->stencilSize == ds_format->stencil_size)
value += 4; value += 4;
/* We like to have aux buffers in backbuffer mode */ if (cfg->alphaSize == color_format->alpha_size)
if (auxBuffers && cfg->auxBuffers)
value += 8; value += 8;
/* We like to have aux buffers in backbuffer mode */
if (aux_buffers && cfg->auxBuffers)
value += 16;
if (cfg->redSize == color_format->red_size if (cfg->redSize == color_format->red_size
&& cfg->greenSize == color_format->green_size && cfg->greenSize == color_format->green_size
&& cfg->blueSize == color_format->blue_size) && cfg->blueSize == color_format->blue_size)
value += 16; value += 32;
if (value > current_value) if (value > current_value)
{ {
@ -1898,6 +1900,7 @@ HGLRC context_create_wgl_attribs(const struct wined3d_gl_info *gl_info, HDC hdc,
static BOOL wined3d_context_gl_create_wgl_ctx(struct wined3d_context_gl *context_gl, static BOOL wined3d_context_gl_create_wgl_ctx(struct wined3d_context_gl *context_gl,
struct wined3d_swapchain_gl *swapchain_gl) struct wined3d_swapchain_gl *swapchain_gl)
{ {
enum wined3d_swap_effect swap_effect = swapchain_gl->s.state.desc.swap_effect;
const struct wined3d_format *colour_format, *ds_format; const struct wined3d_format *colour_format, *ds_format;
struct wined3d_context *context = &context_gl->c; struct wined3d_context *context = &context_gl->c;
const struct wined3d_gl_info *gl_info; const struct wined3d_gl_info *gl_info;
@ -1905,6 +1908,7 @@ static BOOL wined3d_context_gl_create_wgl_ctx(struct wined3d_context_gl *context
struct wined3d_adapter *adapter; struct wined3d_adapter *adapter;
unsigned int target_bind_flags; unsigned int target_bind_flags;
struct wined3d_device *device; struct wined3d_device *device;
bool swap_effect_copy;
HGLRC ctx, share_ctx; HGLRC ctx, share_ctx;
unsigned int i; unsigned int i;
@ -1915,6 +1919,8 @@ static BOOL wined3d_context_gl_create_wgl_ctx(struct wined3d_context_gl *context
target = &context->current_rt.texture->resource; target = &context->current_rt.texture->resource;
target_bind_flags = target->bind_flags; target_bind_flags = target->bind_flags;
swap_effect_copy = swap_effect == WINED3D_SWAP_EFFECT_COPY || swap_effect == WINED3D_SWAP_EFFECT_COPY_VSYNC;
if (wined3d_settings.offscreen_rendering_mode == ORM_BACKBUFFER) if (wined3d_settings.offscreen_rendering_mode == ORM_BACKBUFFER)
{ {
static const enum wined3d_format_id ds_formats[] = static const enum wined3d_format_id ds_formats[] =
@ -1951,7 +1957,7 @@ static BOOL wined3d_context_gl_create_wgl_ctx(struct wined3d_context_gl *context
{ {
ds_format = wined3d_get_format(adapter, ds_formats[i], WINED3D_BIND_DEPTH_STENCIL); ds_format = wined3d_get_format(adapter, ds_formats[i], WINED3D_BIND_DEPTH_STENCIL);
if ((context_gl->pixel_format = context_choose_pixel_format(device, if ((context_gl->pixel_format = context_choose_pixel_format(device,
context_gl->dc, colour_format, ds_format, TRUE))) context_gl->dc, colour_format, ds_format, true, swap_effect_copy)))
{ {
swapchain_gl->s.ds_format = ds_format; swapchain_gl->s.ds_format = ds_format;
break; break;
@ -1964,7 +1970,7 @@ static BOOL wined3d_context_gl_create_wgl_ctx(struct wined3d_context_gl *context
else else
{ {
context_gl->pixel_format = context_choose_pixel_format(device, context_gl->pixel_format = context_choose_pixel_format(device,
context_gl->dc, colour_format, swapchain_gl->s.ds_format, TRUE); context_gl->dc, colour_format, swapchain_gl->s.ds_format, true, swap_effect_copy);
} }
} }
else else
@ -1980,7 +1986,7 @@ static BOOL wined3d_context_gl_create_wgl_ctx(struct wined3d_context_gl *context
colour_format = wined3d_get_format(adapter, WINED3DFMT_B8G8R8A8_UNORM, target_bind_flags); colour_format = wined3d_get_format(adapter, WINED3DFMT_B8G8R8A8_UNORM, target_bind_flags);
ds_format = wined3d_get_format(adapter, WINED3DFMT_UNKNOWN, WINED3D_BIND_DEPTH_STENCIL); ds_format = wined3d_get_format(adapter, WINED3DFMT_UNKNOWN, WINED3D_BIND_DEPTH_STENCIL);
context_gl->pixel_format = context_choose_pixel_format(device, context_gl->pixel_format = context_choose_pixel_format(device,
context_gl->dc, colour_format, ds_format, FALSE); context_gl->dc, colour_format, ds_format, false, swap_effect_copy);
} }
if (!context_gl->pixel_format) if (!context_gl->pixel_format)

View File

@ -594,6 +594,7 @@ static void swapchain_gl_present(struct wined3d_swapchain *swapchain,
{ {
struct wined3d_swapchain_gl *swapchain_gl = wined3d_swapchain_gl(swapchain); struct wined3d_swapchain_gl *swapchain_gl = wined3d_swapchain_gl(swapchain);
struct wined3d_texture *back_buffer = swapchain->back_buffers[0]; struct wined3d_texture *back_buffer = swapchain->back_buffers[0];
const struct wined3d_pixel_format *pixel_format;
const struct wined3d_gl_info *gl_info; const struct wined3d_gl_info *gl_info;
struct wined3d_context_gl *context_gl; struct wined3d_context_gl *context_gl;
struct wined3d_context *context; struct wined3d_context *context;
@ -609,7 +610,9 @@ static void swapchain_gl_present(struct wined3d_swapchain *swapchain,
TRACE("Presenting DC %p.\n", context_gl->dc); TRACE("Presenting DC %p.\n", context_gl->dc);
if (context_gl->dc == swapchain_gl->backup_dc || swapchain_present_is_partial_copy(swapchain, dst_rect)) pixel_format = &wined3d_adapter_gl(swapchain->device->adapter)->pixel_formats[context_gl->pixel_format];
if (context_gl->dc == swapchain_gl->backup_dc || (pixel_format->swap_method != WGL_SWAP_COPY_ARB
&& swapchain_present_is_partial_copy(swapchain, dst_rect)))
{ {
swapchain_blit_gdi(swapchain, context, src_rect, dst_rect); swapchain_blit_gdi(swapchain, context, src_rect, dst_rect);
} }

View File

@ -2847,6 +2847,7 @@ struct wined3d_pixel_format
BOOL doubleBuffer; BOOL doubleBuffer;
int auxBuffers; int auxBuffers;
int numSamples; int numSamples;
int swap_method;
}; };
enum wined3d_pci_vendor enum wined3d_pci_vendor