July 2016 - Wine-commits - list.winehq.org

Józef Kucia : wined3d: Recognize SM5 rcp opcode.
by Alexandre Julliard 25 Jul '16

25 Jul '16

Module: wine Branch: master Commit: 80c8c23573782bf6465381a1df1cfa39c06caf7f URL: http://source.winehq.org/git/wine.git/?a=commit;h=80c8c23573782bf6465381a1d… Author: Józef Kucia <jkucia(a)codeweavers.com> Date: Fri Jul 22 12:28:22 2016 +0200 wined3d: Recognize SM5 rcp opcode. Signed-off-by: Józef Kucia <jkucia(a)codeweavers.com> Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/wined3d/shader_sm4.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/dlls/wined3d/shader_sm4.c b/dlls/wined3d/shader_sm4.c index 256bcc7..520e3c6 100644 --- a/dlls/wined3d/shader_sm4.c +++ b/dlls/wined3d/shader_sm4.c @@ -219,6 +219,7 @@ enum wined3d_sm4_opcode WINED3D_SM5_OP_DERIV_RTY_COARSE = 0x7c, WINED3D_SM5_OP_DERIV_RTY_FINE = 0x7d, WINED3D_SM5_OP_GATHER4_C = 0x7e, + WINED3D_SM5_OP_RCP = 0x81, WINED3D_SM5_OP_UBFE = 0x8a, WINED3D_SM5_OP_BFI = 0x8c, WINED3D_SM5_OP_BFREV = 0x8d, @@ -821,6 +822,7 @@ static const struct wined3d_sm4_opcode_info opcode_table[] = {WINED3D_SM5_OP_DERIV_RTY_COARSE, WINED3DSIH_DSY_COARSE, "f", "f"}, {WINED3D_SM5_OP_DERIV_RTY_FINE, WINED3DSIH_DSY_FINE, "f", "f"}, {WINED3D_SM5_OP_GATHER4_C, WINED3DSIH_GATHER4_C, "f", "fRSf"}, + {WINED3D_SM5_OP_RCP, WINED3DSIH_RCP, "f", "f"}, {WINED3D_SM5_OP_UBFE, WINED3DSIH_UBFE, "u", "iiu"}, {WINED3D_SM5_OP_BFI, WINED3DSIH_BFI, "u", "iiuu"}, {WINED3D_SM5_OP_BFREV, WINED3DSIH_BFREV, "u", "u"},

1 0

Nikolay Sivov : msscript: Implement AddObject().
by Alexandre Julliard 25 Jul '16

25 Jul '16

Module: wine Branch: master Commit: 82c5d08acc3e1b07ab0d0f7ea425bf14e28c70da URL: http://source.winehq.org/git/wine.git/?a=commit;h=82c5d08acc3e1b07ab0d0f7ea… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Thu Jul 21 20:34:11 2016 +0300 msscript: Implement AddObject(). Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/msscript.ocx/msscript.c | 97 ++++++++++++++++++++++-- dlls/msscript.ocx/tests/msscript.c | 150 ++++++++++++++++++++++++++++++++++++- 2 files changed, 236 insertions(+), 11 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=82c5d08acc3e1b07ab0d0…

1 0

Łukasz Wojniłowicz : Polish translation for release 1.9.15
by Jeremy Newman 25 Jul '16

25 Jul '16

Module: website Branch: master Commit: cc99c98f438f2d1f744f90568fee257206652b1e URL: http://source.winehq.org/git/website.git/?a=commit;h=cc99c98f438f2d1f744f90… Author: Łukasz Wojniłowicz <lukasz.wojnilowicz(a)gmail.com> Date: Sat Jul 23 07:19:30 2016 +0200 Polish translation for release 1.9.15 Signed-off-by: Jeremy Newman <jnewman(a)codeweavers.com> --- news/pl/2016072201.xml | 17 +++++++++++++++++ 1 file changed, 17 insertions(+) diff --git a/news/pl/2016072201.xml b/news/pl/2016072201.xml new file mode 100644 index 0000000..b5f89e6 --- /dev/null +++ b/news/pl/2016072201.xml @@ -0,0 +1,17 @@ +<news> +<date>Lipiec 22, 2016</date> +<title>Wydano Wine 1.9.15</title> +<body> + Wydanie rozwojowe Wine 1.9.15 jest już dostępne. + <a href="{$root}/announce/1.9.15">Co nowego</a> w tym wydaniu: +<ul> + <li>Więcej instrukcji cieniowania w Direct3D.</li> + <li>Poprawa wydajności w GDI.</li> + <li>Lepsza obsługa multi-joystick na macOS.</li> + <li>Ulepszenia w aktywnych skryptach.</li> + <li>Obsługa dodatkowego strumienia w bibliotece uruchomieniowej C++.</li> + <li>Rozmaite poprawki błędów.</li> +</ul> +Źródło jest <a href="//dl.winehq.org/wine/source/1.9/wine-1.9.15.tar.bz2">już dostępne</a>. +Paczki binarne są w trakcie budowy i ukażą się wkrótce w przeznaczonych dla nich <a href="{$root}/download">pobieralniach</a>. +</body></news>

1 0

Alexandre Julliard : Wine release 1.9.15
by Alexandre Julliard 22 Jul '16

22 Jul '16

Module: website Branch: master Commit: b46062c82b18e2d0177a98d726987d4288f995a0 URL: http://source.winehq.org/git/website.git/?a=commit;h=b46062c82b18e2d0177a98… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Fri Jul 22 23:18:07 2016 +0900 Wine release 1.9.15 Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- news/en/2016072201.xml | 17 +++++++++++++++++ 1 file changed, 17 insertions(+) diff --git a/news/en/2016072201.xml b/news/en/2016072201.xml new file mode 100644 index 0000000..03accb3 --- /dev/null +++ b/news/en/2016072201.xml @@ -0,0 +1,17 @@ +<news> +<date>July 22, 2016</date> +<title>Wine 1.9.15 Released</title> +<body> + The Wine development release 1.9.15 is now available. + <a href="{$root}/announce/1.9.15">What's new</a> in this release: +<ul> + <li>More shader instructions in Direct3D.</li> + <li>Performance improvements in GDI.</li> + <li>Better multi-joystick support on macOS.</li> + <li>Active Scripting improvements.</li> + <li>Additional stream support in the C++ runtime.</li> + <li>Various bug fixes.</li> +</ul> +The source is <a href="//dl.winehq.org/wine/source/1.9/wine-1.9.15.tar.bz2">available now</a>. +Binary packages are in the process of being built, and will appear soon at their respective <a href="{$root}/download">download locations</a>. +</body></news>

1 0

Alexandre Julliard : server: Avoid unaligned access when printing x86 floating point registers.
by Alexandre Julliard 22 Jul '16

22 Jul '16

Module: wine Branch: master Commit: 0b933f1f4839e5e1a2dd2125547b5ecdf2b91e0f URL: http://source.winehq.org/git/wine.git/?a=commit;h=0b933f1f4839e5e1a2dd21255… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Fri Jul 22 11:49:37 2016 +0900 server: Avoid unaligned access when printing x86 floating point registers. Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- server/trace.c | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/server/trace.c b/server/trace.c index d48c205..9286ebd 100644 --- a/server/trace.c +++ b/server/trace.c @@ -567,7 +567,11 @@ static void dump_varargs_context( const char *prefix, data_size_t size ) fprintf( stderr, ",fp.data_off=%08x,fp.data_sel=%08x,fp.cr0npx=%08x", ctx.fp.i386_regs.data_off, ctx.fp.i386_regs.data_sel, ctx.fp.i386_regs.cr0npx ); for (i = 0; i < 8; i++) - fprintf( stderr, ",fp.reg%u=%Lg", i, *(long double *)&ctx.fp.i386_regs.regs[10*i] ); + { + long double reg = 0; + memcpy( &reg, &ctx.fp.i386_regs.regs[10 * i], 10 ); + fprintf( stderr, ",fp.reg%u=%Lg", i, reg ); + } } if (ctx.flags & SERVER_CTX_EXTENDED_REGISTERS) {

1 0

Henri Verbeet : wined3d: Get rid of redundant ARB_timer_query checks.
by Alexandre Julliard 22 Jul '16

22 Jul '16

Module: wine Branch: master Commit: be2908ed071dd11de5dc2fae270aad15e5df7d17 URL: http://source.winehq.org/git/wine.git/?a=commit;h=be2908ed071dd11de5dc2fae2… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Fri Jul 22 00:43:47 2016 +0200 wined3d: Get rid of redundant ARB_timer_query checks. Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/wined3d/query.c | 35 +++++++++++++---------------------- 1 file changed, 13 insertions(+), 22 deletions(-) diff --git a/dlls/wined3d/query.c b/dlls/wined3d/query.c index 369951d..6b3a990 100644 --- a/dlls/wined3d/query.c +++ b/dlls/wined3d/query.c @@ -588,38 +588,29 @@ static HRESULT wined3d_timestamp_query_ops_get_data(struct wined3d_query *query, static HRESULT wined3d_timestamp_query_ops_issue(struct wined3d_query *query, DWORD flags) { + struct wined3d_timestamp_query *tq = query->extendedData; struct wined3d_device *device = query->device; const struct wined3d_gl_info *gl_info = &device->adapter->gl_info; + struct wined3d_context *context; TRACE("query %p, flags %#x.\n", query, flags); - if (gl_info->supported[ARB_TIMER_QUERY]) + if (flags & WINED3DISSUE_BEGIN) { - struct wined3d_timestamp_query *tq = query->extendedData; - struct wined3d_context *context; - - if (flags & WINED3DISSUE_BEGIN) - { - WARN("Ignoring WINED3DISSUE_BEGIN with a TIMESTAMP query.\n"); - } - if (flags & WINED3DISSUE_END) - { - if (tq->context) - context_free_timestamp_query(tq); - context = context_acquire(query->device, NULL); - context_alloc_timestamp_query(context, tq); - GL_EXTCALL(glQueryCounter(tq->id, GL_TIMESTAMP)); - checkGLcall("glQueryCounter()"); - context_release(context); - } + WARN("Ignoring WINED3DISSUE_BEGIN with a TIMESTAMP query.\n"); } - else + if (flags & WINED3DISSUE_END) { - ERR("Timestamp queries not supported.\n"); - } + if (tq->context) + context_free_timestamp_query(tq); + context = context_acquire(query->device, NULL); + context_alloc_timestamp_query(context, tq); + GL_EXTCALL(glQueryCounter(tq->id, GL_TIMESTAMP)); + checkGLcall("glQueryCounter()"); + context_release(context); - if (flags & WINED3DISSUE_END) query->state = QUERY_SIGNALLED; + } return WINED3D_OK; }

1 0

Henri Verbeet : wined3d: Use a single allocation for event queries.
by Alexandre Julliard 22 Jul '16

22 Jul '16

Module: wine Branch: master Commit: fac012f2e114870c9628e5ea5b5c8312284c0f3e URL: http://source.winehq.org/git/wine.git/?a=commit;h=fac012f2e114870c9628e5ea5… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Fri Jul 22 00:43:48 2016 +0200 wined3d: Use a single allocation for event queries. Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/wined3d/query.c | 81 +++++++++++++++++++++++++++++------------- dlls/wined3d/wined3d_private.h | 61 ++++++++++++++++--------------- 2 files changed, 89 insertions(+), 53 deletions(-) diff --git a/dlls/wined3d/query.c b/dlls/wined3d/query.c index 6b3a990..54e8bb1 100644 --- a/dlls/wined3d/query.c +++ b/dlls/wined3d/query.c @@ -25,6 +25,23 @@ WINE_DEFAULT_DEBUG_CHANNEL(d3d); +static void wined3d_query_init(struct wined3d_query *query, struct wined3d_device *device, + enum wined3d_query_type type, DWORD data_size, const struct wined3d_query_ops *query_ops, void *parent) +{ + query->ref = 1; + query->parent = parent; + query->device = device; + query->state = QUERY_CREATED; + query->type = type; + query->data_size = data_size; + query->query_ops = query_ops; +} + +static struct wined3d_event_query *wined3d_event_query_from_query(struct wined3d_query *query) +{ + return CONTAINING_RECORD(query, struct wined3d_event_query, query); +} + BOOL wined3d_event_query_supported(const struct wined3d_gl_info *gl_info) { return gl_info->supported[ARB_SYNC] || gl_info->supported[NV_FENCE] || gl_info->supported[APPLE_FENCE]; @@ -243,8 +260,8 @@ static void wined3d_query_destroy_object(void *object) * context, and (still) leaking the actual query. */ if (query->type == WINED3D_QUERY_TYPE_EVENT) { - struct wined3d_event_query *event_query = query->extendedData; - wined3d_event_query_destroy(event_query); + wined3d_event_query_destroy(wined3d_event_query_from_query(query)); + return; } else if (query->type == WINED3D_QUERY_TYPE_OCCLUSION) { @@ -375,9 +392,9 @@ static HRESULT wined3d_occlusion_query_ops_get_data(struct wined3d_query *query, static HRESULT wined3d_event_query_ops_get_data(struct wined3d_query *query, void *data, DWORD size, DWORD flags) { - struct wined3d_event_query *event_query = query->extendedData; - BOOL signaled; + struct wined3d_event_query *event_query = wined3d_event_query_from_query(query); enum wined3d_event_query_result ret; + BOOL signaled; TRACE("query %p, data %p, size %#x, flags %#x.\n", query, data, size, flags); @@ -430,16 +447,15 @@ static HRESULT wined3d_event_query_ops_issue(struct wined3d_query *query, DWORD { TRACE("query %p, flags %#x.\n", query, flags); - TRACE("(%p) : flags %#x, type D3DQUERY_EVENT\n", query, flags); if (flags & WINED3DISSUE_END) { - struct wined3d_event_query *event_query = query->extendedData; + struct wined3d_event_query *event_query = wined3d_event_query_from_query(query); wined3d_event_query_issue(event_query, query->device); } else if (flags & WINED3DISSUE_BEGIN) { - /* Started implicitly at device creation */ + /* Started implicitly at query creation. */ ERR("Event query issued with START flag - what to do?\n"); } @@ -659,6 +675,31 @@ static const struct wined3d_query_ops event_query_ops = wined3d_event_query_ops_issue, }; +static HRESULT wined3d_event_query_create(struct wined3d_device *device, + enum wined3d_query_type type, void *parent, struct wined3d_query **query) +{ + const struct wined3d_gl_info *gl_info = &device->adapter->gl_info; + struct wined3d_event_query *object; + + TRACE("device %p, type %#x, parent %p, query %p.\n", device, type, parent, query); + + if (!wined3d_event_query_supported(gl_info)) + { + WARN("Event queries not supported.\n"); + return WINED3DERR_NOTAVAILABLE; + } + + if (!(object = HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, sizeof(*object)))) + return E_OUTOFMEMORY; + + wined3d_query_init(&object->query, device, type, sizeof(BOOL), &event_query_ops, parent); + + TRACE("Created query %p.\n", object); + *query = &object->query; + + return WINED3D_OK; +} + static const struct wined3d_query_ops occlusion_query_ops = { wined3d_occlusion_query_ops_get_data, @@ -704,23 +745,6 @@ static HRESULT query_init(struct wined3d_query *query, struct wined3d_device *de ((struct wined3d_occlusion_query *)query->extendedData)->context = NULL; break; - case WINED3D_QUERY_TYPE_EVENT: - TRACE("Event query.\n"); - if (!wined3d_event_query_supported(gl_info)) - { - WARN("Event queries not supported.\n"); - return WINED3DERR_NOTAVAILABLE; - } - query->query_ops = &event_query_ops; - query->data_size = sizeof(BOOL); - query->extendedData = HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, sizeof(struct wined3d_event_query)); - if (!query->extendedData) - { - ERR("Failed to allocate event query memory.\n"); - return E_OUTOFMEMORY; - } - break; - case WINED3D_QUERY_TYPE_TIMESTAMP: TRACE("Timestamp query.\n"); if (!gl_info->supported[ARB_TIMER_QUERY]) @@ -783,6 +807,15 @@ HRESULT CDECL wined3d_query_create(struct wined3d_device *device, TRACE("device %p, type %#x, parent %p, query %p.\n", device, type, parent, query); + switch (type) + { + case WINED3D_QUERY_TYPE_EVENT: + return wined3d_event_query_create(device, type, parent, query); + + default: + break; + } + object = HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, sizeof(*object)); if (!object) return E_OUTOFMEMORY; diff --git a/dlls/wined3d/wined3d_private.h b/dlls/wined3d/wined3d_private.h index 3c6c869..6f49b1f 100644 --- a/dlls/wined3d/wined3d_private.h +++ b/dlls/wined3d/wined3d_private.h @@ -1367,6 +1367,36 @@ enum fogsource { FOGSOURCE_COORD, }; +/* Direct3D terminology with little modifications. We do not have an issued + * state because only the driver knows about it, but we have a created state + * because D3D allows GetData() on a created query, but OpenGL doesn't. */ +enum wined3d_query_state +{ + QUERY_CREATED, + QUERY_SIGNALLED, + QUERY_BUILDING +}; + +struct wined3d_query_ops +{ + HRESULT (*query_get_data)(struct wined3d_query *query, void *data, DWORD data_size, DWORD flags); + HRESULT (*query_issue)(struct wined3d_query *query, DWORD flags); +}; + +struct wined3d_query +{ + LONG ref; + + void *parent; + struct wined3d_device *device; + enum wined3d_query_state state; + enum wined3d_query_type type; + DWORD data_size; + const struct wined3d_query_ops *query_ops; + + void *extendedData; +}; + struct wined3d_occlusion_query { struct list entry; @@ -1382,6 +1412,8 @@ union wined3d_gl_query_object struct wined3d_event_query { + struct wined3d_query query; + struct list entry; union wined3d_gl_query_object object; struct wined3d_context *context; @@ -3051,35 +3083,6 @@ static inline void wined3d_cs_push_constants(struct wined3d_cs *cs, enum wined3d cs->ops->push_constants(cs, p, start_idx, count, constants); } -/* Direct3D terminology with little modifications. We do not have an issued state - * because only the driver knows about it, but we have a created state because d3d - * allows GetData on a created issue, but opengl doesn't - */ -enum query_state { - QUERY_CREATED, - QUERY_SIGNALLED, - QUERY_BUILDING -}; - -struct wined3d_query_ops -{ - HRESULT (*query_get_data)(struct wined3d_query *query, void *data, DWORD data_size, DWORD flags); - HRESULT (*query_issue)(struct wined3d_query *query, DWORD flags); -}; - -struct wined3d_query -{ - LONG ref; - - void *parent; - const struct wined3d_query_ops *query_ops; - struct wined3d_device *device; - enum query_state state; - enum wined3d_query_type type; - DWORD data_size; - void *extendedData; -}; - /* TODO: Add tests and support for FLOAT16_4 POSITIONT, D3DCOLOR position, other * fixed function semantics as D3DCOLOR or FLOAT16 */ enum wined3d_buffer_conversion_type

1 0

Henri Verbeet : wined3d: Use a single allocation for occlusion queries.
by Alexandre Julliard 22 Jul '16

22 Jul '16

Module: wine Branch: master Commit: 51c7df4f376fa7924752dc32a9715a4475e038df URL: http://source.winehq.org/git/wine.git/?a=commit;h=51c7df4f376fa7924752dc32a… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Fri Jul 22 00:43:49 2016 +0200 wined3d: Use a single allocation for occlusion queries. Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/wined3d/query.c | 63 +++++++++++++++++++++++++++--------------- dlls/wined3d/wined3d_private.h | 16 ++++++----- 2 files changed, 49 insertions(+), 30 deletions(-) diff --git a/dlls/wined3d/query.c b/dlls/wined3d/query.c index 54e8bb1..3168c30 100644 --- a/dlls/wined3d/query.c +++ b/dlls/wined3d/query.c @@ -42,6 +42,11 @@ static struct wined3d_event_query *wined3d_event_query_from_query(struct wined3d return CONTAINING_RECORD(query, struct wined3d_event_query, query); } +static struct wined3d_occlusion_query *wined3d_occlusion_query_from_query(struct wined3d_query *query) +{ + return CONTAINING_RECORD(query, struct wined3d_occlusion_query, query); +} + BOOL wined3d_event_query_supported(const struct wined3d_gl_info *gl_info) { return gl_info->supported[ARB_SYNC] || gl_info->supported[NV_FENCE] || gl_info->supported[APPLE_FENCE]; @@ -265,10 +270,12 @@ static void wined3d_query_destroy_object(void *object) } else if (query->type == WINED3D_QUERY_TYPE_OCCLUSION) { - struct wined3d_occlusion_query *oq = query->extendedData; + struct wined3d_occlusion_query *oq = wined3d_occlusion_query_from_query(query); - if (oq->context) context_free_occlusion_query(oq); - HeapFree(GetProcessHeap(), 0, query->extendedData); + if (oq->context) + context_free_occlusion_query(oq); + HeapFree(GetProcessHeap(), 0, oq); + return; } else if (query->type == WINED3D_QUERY_TYPE_TIMESTAMP) { @@ -325,7 +332,7 @@ static void fill_query_data(void *out, unsigned int out_size, const void *result static HRESULT wined3d_occlusion_query_ops_get_data(struct wined3d_query *query, void *data, DWORD size, DWORD flags) { - struct wined3d_occlusion_query *oq = query->extendedData; + struct wined3d_occlusion_query *oq = wined3d_occlusion_query_from_query(query); struct wined3d_device *device = query->device; const struct wined3d_gl_info *gl_info = &device->adapter->gl_info; struct wined3d_context *context; @@ -469,7 +476,7 @@ static HRESULT wined3d_event_query_ops_issue(struct wined3d_query *query, DWORD static HRESULT wined3d_occlusion_query_ops_issue(struct wined3d_query *query, DWORD flags) { - struct wined3d_occlusion_query *oq = query->extendedData; + struct wined3d_occlusion_query *oq = wined3d_occlusion_query_from_query(query); struct wined3d_device *device = query->device; const struct wined3d_gl_info *gl_info = &device->adapter->gl_info; struct wined3d_context *context; @@ -706,6 +713,31 @@ static const struct wined3d_query_ops occlusion_query_ops = wined3d_occlusion_query_ops_issue, }; +static HRESULT wined3d_occlusion_query_create(struct wined3d_device *device, + enum wined3d_query_type type, void *parent, struct wined3d_query **query) +{ + const struct wined3d_gl_info *gl_info = &device->adapter->gl_info; + struct wined3d_occlusion_query *object; + + TRACE("device %p, type %#x, parent %p, query %p.\n", device, type, parent, query); + + if (!gl_info->supported[ARB_OCCLUSION_QUERY]) + { + WARN("Unsupported in local OpenGL implementation: ARB_OCCLUSION_QUERY.\n"); + return WINED3DERR_NOTAVAILABLE; + } + + if (!(object = HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, sizeof(*object)))) + return E_OUTOFMEMORY; + + wined3d_query_init(&object->query, device, type, sizeof(DWORD), &occlusion_query_ops, parent); + + TRACE("Created query %p.\n", object); + *query = &object->query; + + return WINED3D_OK; +} + static const struct wined3d_query_ops timestamp_query_ops = { wined3d_timestamp_query_ops_get_data, @@ -727,24 +759,6 @@ static HRESULT query_init(struct wined3d_query *query, struct wined3d_device *de switch (type) { - case WINED3D_QUERY_TYPE_OCCLUSION: - TRACE("Occlusion query.\n"); - if (!gl_info->supported[ARB_OCCLUSION_QUERY]) - { - WARN("Unsupported in local OpenGL implementation: ARB_OCCLUSION_QUERY.\n"); - return WINED3DERR_NOTAVAILABLE; - } - query->query_ops = &occlusion_query_ops; - query->data_size = sizeof(DWORD); - query->extendedData = HeapAlloc(GetProcessHeap(), 0, sizeof(struct wined3d_occlusion_query)); - if (!query->extendedData) - { - ERR("Failed to allocate occlusion query extended data.\n"); - return E_OUTOFMEMORY; - } - ((struct wined3d_occlusion_query *)query->extendedData)->context = NULL; - break; - case WINED3D_QUERY_TYPE_TIMESTAMP: TRACE("Timestamp query.\n"); if (!gl_info->supported[ARB_TIMER_QUERY]) @@ -812,6 +826,9 @@ HRESULT CDECL wined3d_query_create(struct wined3d_device *device, case WINED3D_QUERY_TYPE_EVENT: return wined3d_event_query_create(device, type, parent, query); + case WINED3D_QUERY_TYPE_OCCLUSION: + return wined3d_occlusion_query_create(device, type, parent, query); + default: break; } diff --git a/dlls/wined3d/wined3d_private.h b/dlls/wined3d/wined3d_private.h index 6f49b1f..f7d1948 100644 --- a/dlls/wined3d/wined3d_private.h +++ b/dlls/wined3d/wined3d_private.h @@ -1397,13 +1397,6 @@ struct wined3d_query void *extendedData; }; -struct wined3d_occlusion_query -{ - struct list entry; - GLuint id; - struct wined3d_context *context; -}; - union wined3d_gl_query_object { GLuint id; @@ -1434,6 +1427,15 @@ enum wined3d_event_query_result wined3d_event_query_finish(const struct wined3d_ void wined3d_event_query_issue(struct wined3d_event_query *query, const struct wined3d_device *device) DECLSPEC_HIDDEN; BOOL wined3d_event_query_supported(const struct wined3d_gl_info *gl_info) DECLSPEC_HIDDEN; +struct wined3d_occlusion_query +{ + struct wined3d_query query; + + struct list entry; + GLuint id; + struct wined3d_context *context; +}; + struct wined3d_timestamp_query { struct list entry;

1 0

Henri Verbeet : wined3d: Use a single allocation for timestamp queries.
by Alexandre Julliard 22 Jul '16

22 Jul '16

Module: wine Branch: master Commit: 654e960e340cc37a9282c52ba8aca5f779a13dbb URL: http://source.winehq.org/git/wine.git/?a=commit;h=654e960e340cc37a9282c52ba… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Fri Jul 22 00:43:50 2016 +0200 wined3d: Use a single allocation for timestamp queries. Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/wined3d/query.c | 61 ++++++++++++++++++++++++++---------------- dlls/wined3d/wined3d_private.h | 4 +-- 2 files changed, 40 insertions(+), 25 deletions(-) diff --git a/dlls/wined3d/query.c b/dlls/wined3d/query.c index 3168c30..976cdd9 100644 --- a/dlls/wined3d/query.c +++ b/dlls/wined3d/query.c @@ -47,6 +47,11 @@ static struct wined3d_occlusion_query *wined3d_occlusion_query_from_query(struct return CONTAINING_RECORD(query, struct wined3d_occlusion_query, query); } +static struct wined3d_timestamp_query *wined3d_timestamp_query_from_query(struct wined3d_query *query) +{ + return CONTAINING_RECORD(query, struct wined3d_timestamp_query, query); +} + BOOL wined3d_event_query_supported(const struct wined3d_gl_info *gl_info) { return gl_info->supported[ARB_SYNC] || gl_info->supported[NV_FENCE] || gl_info->supported[APPLE_FENCE]; @@ -279,11 +284,12 @@ static void wined3d_query_destroy_object(void *object) } else if (query->type == WINED3D_QUERY_TYPE_TIMESTAMP) { - struct wined3d_timestamp_query *tq = query->extendedData; + struct wined3d_timestamp_query *tq = wined3d_timestamp_query_from_query(query); if (tq->context) context_free_timestamp_query(tq); - HeapFree(GetProcessHeap(), 0, query->extendedData); + HeapFree(GetProcessHeap(), 0, tq); + return; } HeapFree(GetProcessHeap(), 0, query); @@ -552,7 +558,7 @@ static HRESULT wined3d_occlusion_query_ops_issue(struct wined3d_query *query, DW static HRESULT wined3d_timestamp_query_ops_get_data(struct wined3d_query *query, void *data, DWORD size, DWORD flags) { - struct wined3d_timestamp_query *tq = query->extendedData; + struct wined3d_timestamp_query *tq = wined3d_timestamp_query_from_query(query); struct wined3d_device *device = query->device; const struct wined3d_gl_info *gl_info = &device->adapter->gl_info; struct wined3d_context *context; @@ -611,7 +617,7 @@ static HRESULT wined3d_timestamp_query_ops_get_data(struct wined3d_query *query, static HRESULT wined3d_timestamp_query_ops_issue(struct wined3d_query *query, DWORD flags) { - struct wined3d_timestamp_query *tq = query->extendedData; + struct wined3d_timestamp_query *tq = wined3d_timestamp_query_from_query(query); struct wined3d_device *device = query->device; const struct wined3d_gl_info *gl_info = &device->adapter->gl_info; struct wined3d_context *context; @@ -744,6 +750,31 @@ static const struct wined3d_query_ops timestamp_query_ops = wined3d_timestamp_query_ops_issue, }; +static HRESULT wined3d_timestamp_query_create(struct wined3d_device *device, + enum wined3d_query_type type, void *parent, struct wined3d_query **query) +{ + const struct wined3d_gl_info *gl_info = &device->adapter->gl_info; + struct wined3d_timestamp_query *object; + + TRACE("device %p, type %#x, parent %p, query %p.\n", device, type, parent, query); + + if (!gl_info->supported[ARB_TIMER_QUERY]) + { + WARN("Unsupported in local OpenGL implementation: ARB_TIMER_QUERY.\n"); + return WINED3DERR_NOTAVAILABLE; + } + + if (!(object = HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, sizeof(*object)))) + return E_OUTOFMEMORY; + + wined3d_query_init(&object->query, device, type, sizeof(UINT64), &timestamp_query_ops, parent); + + TRACE("Created query %p.\n", object); + *query = &object->query; + + return WINED3D_OK; +} + static const struct wined3d_query_ops timestamp_disjoint_query_ops = { wined3d_timestamp_disjoint_query_ops_get_data, @@ -759,24 +790,6 @@ static HRESULT query_init(struct wined3d_query *query, struct wined3d_device *de switch (type) { - case WINED3D_QUERY_TYPE_TIMESTAMP: - TRACE("Timestamp query.\n"); - if (!gl_info->supported[ARB_TIMER_QUERY]) - { - WARN("Unsupported in local OpenGL implementation: ARB_TIMER_QUERY.\n"); - return WINED3DERR_NOTAVAILABLE; - } - query->query_ops = &timestamp_query_ops; - query->data_size = sizeof(UINT64); - query->extendedData = HeapAlloc(GetProcessHeap(), 0, sizeof(struct wined3d_timestamp_query)); - if (!query->extendedData) - { - ERR("Failed to allocate timestamp query extended data.\n"); - return E_OUTOFMEMORY; - } - ((struct wined3d_timestamp_query *)query->extendedData)->context = NULL; - break; - case WINED3D_QUERY_TYPE_TIMESTAMP_DISJOINT: case WINED3D_QUERY_TYPE_TIMESTAMP_FREQ: TRACE("TIMESTAMP_DISJOINT query.\n"); @@ -788,7 +801,6 @@ static HRESULT query_init(struct wined3d_query *query, struct wined3d_device *de query->query_ops = &timestamp_disjoint_query_ops; query->data_size = type == WINED3D_QUERY_TYPE_TIMESTAMP_DISJOINT ? sizeof(struct wined3d_query_data_timestamp_disjoint) : sizeof(UINT64); - query->extendedData = NULL; break; case WINED3D_QUERY_TYPE_VCACHE: @@ -829,6 +841,9 @@ HRESULT CDECL wined3d_query_create(struct wined3d_device *device, case WINED3D_QUERY_TYPE_OCCLUSION: return wined3d_occlusion_query_create(device, type, parent, query); + case WINED3D_QUERY_TYPE_TIMESTAMP: + return wined3d_timestamp_query_create(device, type, parent, query); + default: break; } diff --git a/dlls/wined3d/wined3d_private.h b/dlls/wined3d/wined3d_private.h index f7d1948..71a8e2b 100644 --- a/dlls/wined3d/wined3d_private.h +++ b/dlls/wined3d/wined3d_private.h @@ -1393,8 +1393,6 @@ struct wined3d_query enum wined3d_query_type type; DWORD data_size; const struct wined3d_query_ops *query_ops; - - void *extendedData; }; union wined3d_gl_query_object @@ -1438,6 +1436,8 @@ struct wined3d_occlusion_query struct wined3d_timestamp_query { + struct wined3d_query query; + struct list entry; GLuint id; struct wined3d_context *context;

1 0

Henri Verbeet : wined3d: Introduce a separate function for creating timestamp disjoint queries.
by Alexandre Julliard 22 Jul '16

22 Jul '16

Module: wine Branch: master Commit: b1861cb86c38114ab15330b75555ae4c1b78c66d URL: http://source.winehq.org/git/wine.git/?a=commit;h=b1861cb86c38114ab15330b75… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Fri Jul 22 00:43:51 2016 +0200 wined3d: Introduce a separate function for creating timestamp disjoint queries. Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/wined3d/query.c | 91 +++++++++++++++++++--------------------------------- 1 file changed, 33 insertions(+), 58 deletions(-) diff --git a/dlls/wined3d/query.c b/dlls/wined3d/query.c index 976cdd9..591d6c5 100644 --- a/dlls/wined3d/query.c +++ b/dlls/wined3d/query.c @@ -271,7 +271,6 @@ static void wined3d_query_destroy_object(void *object) if (query->type == WINED3D_QUERY_TYPE_EVENT) { wined3d_event_query_destroy(wined3d_event_query_from_query(query)); - return; } else if (query->type == WINED3D_QUERY_TYPE_OCCLUSION) { @@ -280,7 +279,6 @@ static void wined3d_query_destroy_object(void *object) if (oq->context) context_free_occlusion_query(oq); HeapFree(GetProcessHeap(), 0, oq); - return; } else if (query->type == WINED3D_QUERY_TYPE_TIMESTAMP) { @@ -289,10 +287,16 @@ static void wined3d_query_destroy_object(void *object) if (tq->context) context_free_timestamp_query(tq); HeapFree(GetProcessHeap(), 0, tq); - return; } - - HeapFree(GetProcessHeap(), 0, query); + else if (query->type == WINED3D_QUERY_TYPE_TIMESTAMP_DISJOINT + || query->type == WINED3D_QUERY_TYPE_TIMESTAMP_FREQ) + { + HeapFree(GetProcessHeap(), 0, query); + } + else + { + ERR("Query %p has invalid type %#x.\n", query, query->type); + } } ULONG CDECL wined3d_query_decref(struct wined3d_query *query) @@ -781,46 +785,31 @@ static const struct wined3d_query_ops timestamp_disjoint_query_ops = wined3d_timestamp_disjoint_query_ops_issue, }; -static HRESULT query_init(struct wined3d_query *query, struct wined3d_device *device, - enum wined3d_query_type type, void *parent) +static HRESULT wined3d_timestamp_disjoint_query_create(struct wined3d_device *device, + enum wined3d_query_type type, void *parent, struct wined3d_query **query) { const struct wined3d_gl_info *gl_info = &device->adapter->gl_info; + struct wined3d_query *object; - query->parent = parent; + TRACE("device %p, type %#x, parent %p, query %p.\n", device, type, parent, query); - switch (type) + if (!gl_info->supported[ARB_TIMER_QUERY]) { - case WINED3D_QUERY_TYPE_TIMESTAMP_DISJOINT: - case WINED3D_QUERY_TYPE_TIMESTAMP_FREQ: - TRACE("TIMESTAMP_DISJOINT query.\n"); - if (!gl_info->supported[ARB_TIMER_QUERY]) - { - WARN("Unsupported in local OpenGL implementation: ARB_TIMER_QUERY.\n"); - return WINED3DERR_NOTAVAILABLE; - } - query->query_ops = &timestamp_disjoint_query_ops; - query->data_size = type == WINED3D_QUERY_TYPE_TIMESTAMP_DISJOINT - ? sizeof(struct wined3d_query_data_timestamp_disjoint) : sizeof(UINT64); - break; - - case WINED3D_QUERY_TYPE_VCACHE: - case WINED3D_QUERY_TYPE_RESOURCE_MANAGER: - case WINED3D_QUERY_TYPE_VERTEX_STATS: - case WINED3D_QUERY_TYPE_PIPELINE_TIMINGS: - case WINED3D_QUERY_TYPE_INTERFACE_TIMINGS: - case WINED3D_QUERY_TYPE_VERTEX_TIMINGS: - case WINED3D_QUERY_TYPE_PIXEL_TIMINGS: - case WINED3D_QUERY_TYPE_BANDWIDTH_TIMINGS: - case WINED3D_QUERY_TYPE_CACHE_UTILIZATION: - default: - FIXME("Unhandled query type %#x.\n", type); - return WINED3DERR_NOTAVAILABLE; + WARN("Unsupported in local OpenGL implementation: ARB_TIMER_QUERY.\n"); + return WINED3DERR_NOTAVAILABLE; } - query->type = type; - query->state = QUERY_CREATED; - query->device = device; - query->ref = 1; + if (!(object = HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, sizeof(*object)))) + return E_OUTOFMEMORY; + + if (type == WINED3D_QUERY_TYPE_TIMESTAMP_DISJOINT) + wined3d_query_init(object, device, type, + sizeof(struct wined3d_query_data_timestamp_disjoint), &timestamp_disjoint_query_ops, parent); + else + wined3d_query_init(object, device, type, sizeof(UINT64), &timestamp_disjoint_query_ops, parent); + + TRACE("Created query %p.\n", object); + *query = object; return WINED3D_OK; } @@ -828,9 +817,6 @@ static HRESULT query_init(struct wined3d_query *query, struct wined3d_device *de HRESULT CDECL wined3d_query_create(struct wined3d_device *device, enum wined3d_query_type type, void *parent, struct wined3d_query **query) { - struct wined3d_query *object; - HRESULT hr; - TRACE("device %p, type %#x, parent %p, query %p.\n", device, type, parent, query); switch (type) @@ -844,23 +830,12 @@ HRESULT CDECL wined3d_query_create(struct wined3d_device *device, case WINED3D_QUERY_TYPE_TIMESTAMP: return wined3d_timestamp_query_create(device, type, parent, query); - default: - break; - } - - object = HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, sizeof(*object)); - if (!object) - return E_OUTOFMEMORY; + case WINED3D_QUERY_TYPE_TIMESTAMP_DISJOINT: + case WINED3D_QUERY_TYPE_TIMESTAMP_FREQ: + return wined3d_timestamp_disjoint_query_create(device, type, parent, query); - if (FAILED(hr = query_init(object, device, type, parent))) - { - WARN("Failed to initialize query, hr %#x.\n", hr); - HeapFree(GetProcessHeap(), 0, object); - return hr; + default: + FIXME("Unhandled query type %#x.\n", type); + return WINED3DERR_NOTAVAILABLE; } - - TRACE("Created query %p.\n", object); - *query = object; - - return WINED3D_OK; }

1 0