wine-commits January 2017

wine-commits@winehq.org

2 participants
492 discussions

Jacek Caban : inetcomm: Added COM aggregation support to MimeHtmlProtocol object.
by Alexandre Julliard 26 Jan '17

26 Jan '17

Module: wine Branch: master Commit: a1d0c6d721196ff5df3a10bab241b5a835b9bb40 URL: http://source.winehq.org/git/wine.git/?a=commit;h=a1d0c6d721196ff5df3a10bab… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Thu Jan 26 15:23:04 2017 +0100 inetcomm: Added COM aggregation support to MimeHtmlProtocol object. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/inetcomm/inetcomm_main.c | 9 +++++- dlls/inetcomm/protocol.c | 75 ++++++++++++++++++++++++++++++------------- dlls/inetcomm/tests/mimeole.c | 42 +++++++++++++++++++++--- 3 files changed, 99 insertions(+), 27 deletions(-) diff --git a/dlls/inetcomm/inetcomm_main.c b/dlls/inetcomm/inetcomm_main.c index 4cb5522..7543628 100644 --- a/dlls/inetcomm/inetcomm_main.c +++ b/dlls/inetcomm/inetcomm_main.c @@ -117,13 +117,20 @@ static HRESULT WINAPI cf_CreateInstance( IClassFactory *iface, LPUNKNOWN pOuter, *ppobj = NULL; + if (pOuter && !IsEqualGUID(&IID_IUnknown, riid)) + return CLASS_E_NOAGGREGATION; + r = This->create_object( pOuter, (LPVOID*) &punk ); if (FAILED(r)) return r; + if (IsEqualGUID(&IID_IUnknown, riid)) { + *ppobj = punk; + return S_OK; + } + r = IUnknown_QueryInterface( punk, riid, ppobj ); IUnknown_Release( punk ); - return r; } diff --git a/dlls/inetcomm/protocol.c b/dlls/inetcomm/protocol.c index d560b90..f6600b9 100644 --- a/dlls/inetcomm/protocol.c +++ b/dlls/inetcomm/protocol.c @@ -28,31 +28,34 @@ WINE_DEFAULT_DEBUG_CHANNEL(inetcomm); typedef struct { + IUnknown IUnknown_inner; IInternetProtocol IInternetProtocol_iface; IInternetProtocolInfo IInternetProtocolInfo_iface; + LONG ref; + IUnknown *outer_unk; } MimeHtmlProtocol; -static inline MimeHtmlProtocol *impl_from_IInternetProtocol(IInternetProtocol *iface) +static inline MimeHtmlProtocol *impl_from_IUnknown(IUnknown *iface) { - return CONTAINING_RECORD(iface, MimeHtmlProtocol, IInternetProtocol_iface); + return CONTAINING_RECORD(iface, MimeHtmlProtocol, IUnknown_inner); } -static HRESULT WINAPI MimeHtmlProtocol_QueryInterface(IInternetProtocol *iface, REFIID riid, void **ppv) +static HRESULT WINAPI MimeHtmlProtocol_QueryInterface(IUnknown *iface, REFIID riid, void **ppv) { - MimeHtmlProtocol *This = impl_from_IInternetProtocol(iface); + MimeHtmlProtocol *This = impl_from_IUnknown(iface); if(IsEqualGUID(&IID_IUnknown, riid)) { - TRACE("(%p)->(IID_IUnknown %p)\n", iface, ppv); + TRACE("(%p)->(IID_IUnknown %p)\n", This, ppv); *ppv = &This->IInternetProtocol_iface; }else if(IsEqualGUID(&IID_IInternetProtocolRoot, riid)) { - TRACE("(%p)->(IID_IInternetProtocolRoot %p)\n", iface, ppv); + TRACE("(%p)->(IID_IInternetProtocolRoot %p)\n", This, ppv); *ppv = &This->IInternetProtocol_iface; }else if(IsEqualGUID(&IID_IInternetProtocol, riid)) { - TRACE("(%p)->(IID_IInternetProtocol %p)\n", iface, ppv); + TRACE("(%p)->(IID_IInternetProtocol %p)\n", This, ppv); *ppv = &This->IInternetProtocol_iface; }else if(IsEqualGUID(&IID_IInternetProtocolInfo, riid)) { - TRACE("(%p)->(IID_IInternetProtocolInfo %p)\n", iface, ppv); + TRACE("(%p)->(IID_IInternetProtocolInfo %p)\n", This, ppv); *ppv = &This->IInternetProtocolInfo_iface; }else { FIXME("unknown interface %s\n", debugstr_guid(riid)); @@ -64,9 +67,9 @@ static HRESULT WINAPI MimeHtmlProtocol_QueryInterface(IInternetProtocol *iface, return S_OK; } -static ULONG WINAPI MimeHtmlProtocol_AddRef(IInternetProtocol *iface) +static ULONG WINAPI MimeHtmlProtocol_AddRef(IUnknown *iface) { - MimeHtmlProtocol *This = impl_from_IInternetProtocol(iface); + MimeHtmlProtocol *This = impl_from_IUnknown(iface); ULONG ref = InterlockedIncrement(&This->ref); TRACE("(%p) ref=%d\n", This, ref); @@ -74,9 +77,9 @@ static ULONG WINAPI MimeHtmlProtocol_AddRef(IInternetProtocol *iface) return ref; } -static ULONG WINAPI MimeHtmlProtocol_Release(IInternetProtocol *iface) +static ULONG WINAPI MimeHtmlProtocol_Release(IUnknown *iface) { - MimeHtmlProtocol *This = impl_from_IInternetProtocol(iface); + MimeHtmlProtocol *This = impl_from_IUnknown(iface); ULONG ref = InterlockedDecrement(&This->ref); TRACE("(%p) ref=%x\n", This, ref); @@ -87,6 +90,35 @@ static ULONG WINAPI MimeHtmlProtocol_Release(IInternetProtocol *iface) return ref; } +static const IUnknownVtbl MimeHtmlProtocolInnerVtbl = { + MimeHtmlProtocol_QueryInterface, + MimeHtmlProtocol_AddRef, + MimeHtmlProtocol_Release +}; + +static inline MimeHtmlProtocol *impl_from_IInternetProtocol(IInternetProtocol *iface) +{ + return CONTAINING_RECORD(iface, MimeHtmlProtocol, IInternetProtocol_iface); +} + +static HRESULT WINAPI InternetProtocol_QueryInterface(IInternetProtocol *iface, REFIID riid, void **ppv) +{ + MimeHtmlProtocol *This = impl_from_IInternetProtocol(iface); + return IUnknown_QueryInterface(This->outer_unk, riid, ppv); +} + +static ULONG WINAPI InternetProtocol_AddRef(IInternetProtocol *iface) +{ + MimeHtmlProtocol *This = impl_from_IInternetProtocol(iface); + return IUnknown_AddRef(This->outer_unk); +} + +static ULONG WINAPI InternetProtocol_Release(IInternetProtocol *iface) +{ + MimeHtmlProtocol *This = impl_from_IInternetProtocol(iface); + return IUnknown_Release(This->outer_unk); +} + static HRESULT WINAPI MimeHtmlProtocol_Start(IInternetProtocol *iface, const WCHAR *szUrl, IInternetProtocolSink* pOIProtSink, IInternetBindInfo* pOIBindInfo, DWORD grfPI, HANDLE_PTR dwReserved) @@ -162,9 +194,9 @@ static HRESULT WINAPI MimeHtmlProtocol_UnlockRequest(IInternetProtocol *iface) } static const IInternetProtocolVtbl MimeHtmlProtocolVtbl = { - MimeHtmlProtocol_QueryInterface, - MimeHtmlProtocol_AddRef, - MimeHtmlProtocol_Release, + InternetProtocol_QueryInterface, + InternetProtocol_AddRef, + InternetProtocol_Release, MimeHtmlProtocol_Start, MimeHtmlProtocol_Continue, MimeHtmlProtocol_Abort, @@ -185,19 +217,19 @@ static inline MimeHtmlProtocol *impl_from_IInternetProtocolInfo(IInternetProtoco static HRESULT WINAPI MimeHtmlProtocolInfo_QueryInterface(IInternetProtocolInfo *iface, REFIID riid, void **ppv) { MimeHtmlProtocol *This = impl_from_IInternetProtocolInfo(iface); - return IInternetProtocol_QueryInterface(&This->IInternetProtocol_iface, riid, ppv); + return IUnknown_QueryInterface(This->outer_unk, riid, ppv); } static ULONG WINAPI MimeHtmlProtocolInfo_AddRef(IInternetProtocolInfo *iface) { MimeHtmlProtocol *This = impl_from_IInternetProtocolInfo(iface); - return IInternetProtocol_AddRef(&This->IInternetProtocol_iface); + return IUnknown_AddRef(This->outer_unk); } static ULONG WINAPI MimeHtmlProtocolInfo_Release(IInternetProtocolInfo *iface) { MimeHtmlProtocol *This = impl_from_IInternetProtocolInfo(iface); - return IInternetProtocol_Release(&This->IInternetProtocol_iface); + return IUnknown_Release(This->outer_unk); } static HRESULT WINAPI MimeHtmlProtocolInfo_ParseUrl(IInternetProtocolInfo *iface, LPCWSTR pwzUrl, @@ -253,17 +285,16 @@ HRESULT MimeHtmlProtocol_create(IUnknown *outer, void **obj) { MimeHtmlProtocol *protocol; - if(outer) - FIXME("outer not supported\n"); - protocol = heap_alloc(sizeof(*protocol)); if(!protocol) return E_OUTOFMEMORY; + protocol->IUnknown_inner.lpVtbl = &MimeHtmlProtocolInnerVtbl; protocol->IInternetProtocol_iface.lpVtbl = &MimeHtmlProtocolVtbl; protocol->IInternetProtocolInfo_iface.lpVtbl = &MimeHtmlProtocolInfoVtbl; protocol->ref = 1; + protocol->outer_unk = outer ? outer : &protocol->IUnknown_inner; - *obj = &protocol->IInternetProtocol_iface; + *obj = &protocol->IUnknown_inner; return S_OK; } diff --git a/dlls/inetcomm/tests/mimeole.c b/dlls/inetcomm/tests/mimeole.c index 7d68a25..924e0e3 100644 --- a/dlls/inetcomm/tests/mimeole.c +++ b/dlls/inetcomm/tests/mimeole.c @@ -772,8 +772,32 @@ static void test_mhtml_protocol_info(void) IInternetProtocolInfo_Release(protocol_info); } +static HRESULT WINAPI outer_QueryInterface(IUnknown *iface, REFIID riid, void **ppv) +{ + ok(0, "unexpected call\n"); + return E_NOINTERFACE; +} + +static ULONG WINAPI outer_AddRef(IUnknown *iface) +{ + return 2; +} + +static ULONG WINAPI outer_Release(IUnknown *iface) +{ + return 1; +} + +static const IUnknownVtbl outer_vtbl = { + outer_QueryInterface, + outer_AddRef, + outer_Release +}; + static void test_mhtml_protocol(void) { + IUnknown outer = { &outer_vtbl }; + IClassFactory *class_factory; IUnknown *unk, *unk2; HRESULT hres; @@ -781,15 +805,25 @@ static void test_mhtml_protocol(void) hres = CoGetClassObject(&CLSID_IMimeHtmlProtocol, CLSCTX_INPROC_SERVER, NULL, &IID_IUnknown, (void**)&unk); ok(hres == S_OK, "CoGetClassObject failed: %08x\n", hres); - hres = IUnknown_QueryInterface(unk, &IID_IClassFactory, (void**)&unk2); - ok(hres == S_OK, "Could not get IClassFactory iface: %08x\n", hres); - IUnknown_Release(unk2); - hres = IUnknown_QueryInterface(unk, &IID_IInternetProtocolInfo, (void**)&unk2); ok(hres == E_NOINTERFACE, "IInternetProtocolInfo supported\n"); + hres = IUnknown_QueryInterface(unk, &IID_IClassFactory, (void**)&class_factory); + ok(hres == S_OK, "Could not get IClassFactory iface: %08x\n", hres); IUnknown_Release(unk); + hres = IClassFactory_CreateInstance(class_factory, &outer, &IID_IUnknown, (void**)&unk); + ok(hres == S_OK, "CreateInstance returned: %08x\n", hres); + hres = IUnknown_QueryInterface(unk, &IID_IInternetProtocol, (void**)&unk2); + ok(hres == S_OK, "Coult not get IInternetProtocol iface: %08x\n", hres); + IUnknown_Release(unk2); + IUnknown_Release(unk); + + hres = IClassFactory_CreateInstance(class_factory, (IUnknown*)0xdeadbeef, &IID_IInternetProtocol, (void**)&unk2); + ok(hres == CLASS_E_NOAGGREGATION, "CreateInstance returned: %08x\n", hres); + + IClassFactory_Release(class_factory); + test_mhtml_protocol_info(); }

1 0

Jacek Caban : inetcomm: Added IInternetProtocolInfo stub implementation.
by Alexandre Julliard 26 Jan '17

26 Jan '17

Module: wine Branch: master Commit: fe3e5b9c31abdcfe1a3850800d07a9d9974b38a8 URL: http://source.winehq.org/git/wine.git/?a=commit;h=fe3e5b9c31abdcfe1a3850800… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Thu Jan 26 15:22:12 2017 +0100 inetcomm: Added IInternetProtocolInfo stub implementation. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/inetcomm/protocol.c | 77 +++++++++++++++++++++++++++++++++++++++++++ dlls/inetcomm/tests/mimeole.c | 14 ++++++++ 2 files changed, 91 insertions(+) diff --git a/dlls/inetcomm/protocol.c b/dlls/inetcomm/protocol.c index a722550..d560b90 100644 --- a/dlls/inetcomm/protocol.c +++ b/dlls/inetcomm/protocol.c @@ -29,6 +29,7 @@ WINE_DEFAULT_DEBUG_CHANNEL(inetcomm); typedef struct { IInternetProtocol IInternetProtocol_iface; + IInternetProtocolInfo IInternetProtocolInfo_iface; LONG ref; } MimeHtmlProtocol; @@ -50,6 +51,9 @@ static HRESULT WINAPI MimeHtmlProtocol_QueryInterface(IInternetProtocol *iface, }else if(IsEqualGUID(&IID_IInternetProtocol, riid)) { TRACE("(%p)->(IID_IInternetProtocol %p)\n", iface, ppv); *ppv = &This->IInternetProtocol_iface; + }else if(IsEqualGUID(&IID_IInternetProtocolInfo, riid)) { + TRACE("(%p)->(IID_IInternetProtocolInfo %p)\n", iface, ppv); + *ppv = &This->IInternetProtocolInfo_iface; }else { FIXME("unknown interface %s\n", debugstr_guid(riid)); *ppv = NULL; @@ -173,6 +177,78 @@ static const IInternetProtocolVtbl MimeHtmlProtocolVtbl = { MimeHtmlProtocol_UnlockRequest }; +static inline MimeHtmlProtocol *impl_from_IInternetProtocolInfo(IInternetProtocolInfo *iface) +{ + return CONTAINING_RECORD(iface, MimeHtmlProtocol, IInternetProtocolInfo_iface); +} + +static HRESULT WINAPI MimeHtmlProtocolInfo_QueryInterface(IInternetProtocolInfo *iface, REFIID riid, void **ppv) +{ + MimeHtmlProtocol *This = impl_from_IInternetProtocolInfo(iface); + return IInternetProtocol_QueryInterface(&This->IInternetProtocol_iface, riid, ppv); +} + +static ULONG WINAPI MimeHtmlProtocolInfo_AddRef(IInternetProtocolInfo *iface) +{ + MimeHtmlProtocol *This = impl_from_IInternetProtocolInfo(iface); + return IInternetProtocol_AddRef(&This->IInternetProtocol_iface); +} + +static ULONG WINAPI MimeHtmlProtocolInfo_Release(IInternetProtocolInfo *iface) +{ + MimeHtmlProtocol *This = impl_from_IInternetProtocolInfo(iface); + return IInternetProtocol_Release(&This->IInternetProtocol_iface); +} + +static HRESULT WINAPI MimeHtmlProtocolInfo_ParseUrl(IInternetProtocolInfo *iface, LPCWSTR pwzUrl, + PARSEACTION ParseAction, DWORD dwParseFlags, LPWSTR pwzResult, DWORD cchResult, + DWORD* pcchResult, DWORD dwReserved) +{ + MimeHtmlProtocol *This = impl_from_IInternetProtocolInfo(iface); + FIXME("(%p)->(%s %d %x %p %d %p %d)\n", This, debugstr_w(pwzUrl), ParseAction, + dwParseFlags, pwzResult, cchResult, pcchResult, dwReserved); + return INET_E_DEFAULT_ACTION; +} + +static HRESULT WINAPI MimeHtmlProtocolInfo_CombineUrl(IInternetProtocolInfo *iface, + LPCWSTR pwzBaseUrl, LPCWSTR pwzRelativeUrl, DWORD dwCombineFlags, LPWSTR pwzResult, + DWORD cchResult, DWORD* pcchResult, DWORD dwReserved) +{ + MimeHtmlProtocol *This = impl_from_IInternetProtocolInfo(iface); + FIXME("(%p)->(%s %s %08x %p %d %p %d)\n", This, debugstr_w(pwzBaseUrl), + debugstr_w(pwzRelativeUrl), dwCombineFlags, pwzResult, cchResult, + pcchResult, dwReserved); + return E_NOTIMPL; +} + +static HRESULT WINAPI MimeHtmlProtocolInfo_CompareUrl(IInternetProtocolInfo *iface, LPCWSTR pwzUrl1, + LPCWSTR pwzUrl2, DWORD dwCompareFlags) +{ + MimeHtmlProtocol *This = impl_from_IInternetProtocolInfo(iface); + FIXME("(%p)->(%s %s %08x)\n", This, debugstr_w(pwzUrl1), debugstr_w(pwzUrl2), dwCompareFlags); + return E_NOTIMPL; +} + +static HRESULT WINAPI MimeHtmlProtocolInfo_QueryInfo(IInternetProtocolInfo *iface, LPCWSTR pwzUrl, + QUERYOPTION QueryOption, DWORD dwQueryFlags, LPVOID pBuffer, DWORD cbBuffer, DWORD* pcbBuf, + DWORD dwReserved) +{ + MimeHtmlProtocol *This = impl_from_IInternetProtocolInfo(iface); + FIXME("(%p)->(%s %08x %08x %p %d %p %d)\n", This, debugstr_w(pwzUrl), QueryOption, dwQueryFlags, pBuffer, + cbBuffer, pcbBuf, dwReserved); + return INET_E_USE_DEFAULT_PROTOCOLHANDLER; +} + +static const IInternetProtocolInfoVtbl MimeHtmlProtocolInfoVtbl = { + MimeHtmlProtocolInfo_QueryInterface, + MimeHtmlProtocolInfo_AddRef, + MimeHtmlProtocolInfo_Release, + MimeHtmlProtocolInfo_ParseUrl, + MimeHtmlProtocolInfo_CombineUrl, + MimeHtmlProtocolInfo_CompareUrl, + MimeHtmlProtocolInfo_QueryInfo +}; + HRESULT MimeHtmlProtocol_create(IUnknown *outer, void **obj) { MimeHtmlProtocol *protocol; @@ -185,6 +261,7 @@ HRESULT MimeHtmlProtocol_create(IUnknown *outer, void **obj) return E_OUTOFMEMORY; protocol->IInternetProtocol_iface.lpVtbl = &MimeHtmlProtocolVtbl; + protocol->IInternetProtocolInfo_iface.lpVtbl = &MimeHtmlProtocolInfoVtbl; protocol->ref = 1; *obj = &protocol->IInternetProtocol_iface; diff --git a/dlls/inetcomm/tests/mimeole.c b/dlls/inetcomm/tests/mimeole.c index fed5fbd..7d68a25 100644 --- a/dlls/inetcomm/tests/mimeole.c +++ b/dlls/inetcomm/tests/mimeole.c @@ -760,6 +760,18 @@ static void test_MimeOleGetPropertySchema(void) IMimePropertySchema_Release(schema); } +static void test_mhtml_protocol_info(void) +{ + IInternetProtocolInfo *protocol_info; + HRESULT hres; + + hres = CoCreateInstance(&CLSID_IMimeHtmlProtocol, NULL, CLSCTX_INPROC_SERVER, + &IID_IInternetProtocolInfo, (void**)&protocol_info); + ok(hres == S_OK, "Could not create protocol info: %08x\n", hres); + + IInternetProtocolInfo_Release(protocol_info); +} + static void test_mhtml_protocol(void) { IUnknown *unk, *unk2; @@ -777,6 +789,8 @@ static void test_mhtml_protocol(void) ok(hres == E_NOINTERFACE, "IInternetProtocolInfo supported\n"); IUnknown_Release(unk); + + test_mhtml_protocol_info(); } START_TEST(mimeole)

1 0

Sebastian Lackner : ntdll: Catch page faults in NtQueryPerformanceCounter.
by Alexandre Julliard 26 Jan '17

26 Jan '17

Module: wine Branch: master Commit: b60b50ba35c762d59b6d1f228a38c5fdf803eec7 URL: http://source.winehq.org/git/wine.git/?a=commit;h=b60b50ba35c762d59b6d1f228… Author: Sebastian Lackner <sebastian(a)fds-team.de> Date: Thu Jan 26 17:05:10 2017 +0100 ntdll: Catch page faults in NtQueryPerformanceCounter. Signed-off-by: Sebastian Lackner <sebastian(a)fds-team.de> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/ntdll/tests/time.c | 23 +++++++++++++++++++++++ dlls/ntdll/time.c | 14 +++++++++++--- 2 files changed, 34 insertions(+), 3 deletions(-) diff --git a/dlls/ntdll/tests/time.c b/dlls/ntdll/tests/time.c index 48f4640..b684bc1 100644 --- a/dlls/ntdll/tests/time.c +++ b/dlls/ntdll/tests/time.c @@ -26,6 +26,7 @@ static VOID (WINAPI *pRtlTimeToTimeFields)( const LARGE_INTEGER *liTime, PTIME_FIELDS TimeFields) ; static VOID (WINAPI *pRtlTimeFieldsToTime)( PTIME_FIELDS TimeFields, PLARGE_INTEGER Time) ; +static NTSTATUS (WINAPI *pNtQueryPerformanceCounter)( LARGE_INTEGER *counter, LARGE_INTEGER *frequency ); static const int MonthLengths[2][12] = { @@ -94,13 +95,35 @@ static void test_pRtlTimeToTimeFields(void) } } +static void test_NtQueryPerformanceCounter(void) +{ + LARGE_INTEGER counter, frequency; + NTSTATUS status; + + status = pNtQueryPerformanceCounter(NULL, NULL); + ok(status == STATUS_ACCESS_VIOLATION, "expected STATUS_ACCESS_VIOLATION, got %08x\n", status); + status = pNtQueryPerformanceCounter(NULL, &frequency); + ok(status == STATUS_ACCESS_VIOLATION, "expected STATUS_ACCESS_VIOLATION, got %08x\n", status); + status = pNtQueryPerformanceCounter(&counter, (void *)0xdeadbee0); + ok(status == STATUS_ACCESS_VIOLATION, "expected STATUS_ACCESS_VIOLATION, got %08x\n", status); + status = pNtQueryPerformanceCounter((void *)0xdeadbee0, &frequency); + ok(status == STATUS_ACCESS_VIOLATION, "expected STATUS_ACCESS_VIOLATION, got %08x\n", status); + + status = pNtQueryPerformanceCounter(&counter, NULL); + ok(status == STATUS_SUCCESS, "expected STATUS_SUCCESS, got %08x\n", status); + status = pNtQueryPerformanceCounter(&counter, &frequency); + ok(status == STATUS_SUCCESS, "expected STATUS_SUCCESS, got %08x\n", status); +} + START_TEST(time) { HMODULE mod = GetModuleHandleA("ntdll.dll"); pRtlTimeToTimeFields = (void *)GetProcAddress(mod,"RtlTimeToTimeFields"); pRtlTimeFieldsToTime = (void *)GetProcAddress(mod,"RtlTimeFieldsToTime"); + pNtQueryPerformanceCounter = (void *)GetProcAddress(mod, "NtQueryPerformanceCounter"); if (pRtlTimeToTimeFields && pRtlTimeFieldsToTime) test_pRtlTimeToTimeFields(); else win_skip("Required time conversion functions are not available\n"); + test_NtQueryPerformanceCounter(); } diff --git a/dlls/ntdll/time.c b/dlls/ntdll/time.c index d232c4c..3ba3c68 100644 --- a/dlls/ntdll/time.c +++ b/dlls/ntdll/time.c @@ -46,6 +46,7 @@ #define WIN32_NO_STATUS #include "windef.h" #include "winternl.h" +#include "wine/exception.h" #include "wine/unicode.h" #include "wine/debug.h" #include "ntdll_misc.h" @@ -472,10 +473,17 @@ NTSTATUS WINAPI NtQuerySystemTime( PLARGE_INTEGER Time ) */ NTSTATUS WINAPI NtQueryPerformanceCounter( LARGE_INTEGER *counter, LARGE_INTEGER *frequency ) { - if (!counter) return STATUS_ACCESS_VIOLATION; + __TRY + { + counter->QuadPart = monotonic_counter(); + if (frequency) frequency->QuadPart = TICKSPERSEC; + } + __EXCEPT_PAGE_FAULT + { + return STATUS_ACCESS_VIOLATION; + } + __ENDTRY - counter->QuadPart = monotonic_counter(); - if (frequency) frequency->QuadPart = TICKSPERSEC; return STATUS_SUCCESS; }

1 0

Piotr Caban : msvcr100: Add reader_writer_lock tests.
by Alexandre Julliard 26 Jan '17

26 Jan '17

Module: wine Branch: master Commit: 00475e1dfd56caa47e5f2b5670e77301f07d89aa URL: http://source.winehq.org/git/wine.git/?a=commit;h=00475e1dfd56caa47e5f2b567… Author: Piotr Caban <piotr(a)codeweavers.com> Date: Thu Jan 26 17:30:45 2017 +0100 msvcr100: Add reader_writer_lock tests. Signed-off-by: Piotr Caban <piotr(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/msvcr100/tests/msvcr100.c | 84 ++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 84 insertions(+) diff --git a/dlls/msvcr100/tests/msvcr100.c b/dlls/msvcr100/tests/msvcr100.c index 4c66ed5..045d698 100644 --- a/dlls/msvcr100/tests/msvcr100.c +++ b/dlls/msvcr100/tests/msvcr100.c @@ -155,6 +155,14 @@ static void (__thiscall *pSpinWait__SetSpinCount)(SpinWait*, unsigned int); static MSVCRT_bool (__thiscall *pSpinWait__ShouldSpinAgain)(SpinWait*); static MSVCRT_bool (__thiscall *pSpinWait__SpinOnce)(SpinWait*); +static void* (__thiscall *preader_writer_lock_ctor)(void*); +static void (__thiscall *preader_writer_lock_dtor)(void*); +static void (__thiscall *preader_writer_lock_lock)(void*); +static void (__thiscall *preader_writer_lock_lock_read)(void*); +static void (__thiscall *preader_writer_lock_unlock)(void*); +static MSVCRT_bool (__thiscall *preader_writer_lock_try_lock)(void*); +static MSVCRT_bool (__thiscall *preader_writer_lock_try_lock_read)(void*); + /* make sure we use the correct errno */ #undef errno #define errno (*p_errno()) @@ -195,6 +203,14 @@ static BOOL init(void) SET(pSpinWait__SetSpinCount, "?_SetSpinCount@?$_SpinWait@$00@details@Concurrency@@QEAAXI@Z"); SET(pSpinWait__ShouldSpinAgain, "?_ShouldSpinAgain@?$_SpinWait@$00@details@Concurrency@@IEAA_NXZ"); SET(pSpinWait__SpinOnce, "?_SpinOnce@?$_SpinWait@$00@details@Concurrency@@QEAA_NXZ"); + + SET(preader_writer_lock_ctor, "??0reader_writer_lock@Concurrency@@QEAA@XZ"); + SET(preader_writer_lock_dtor, "??1reader_writer_lock@Concurrency@@QEAA@XZ"); + SET(preader_writer_lock_lock, "?lock@reader_writer_lock@Concurrency@@QEAAXXZ"); + SET(preader_writer_lock_lock_read, "?lock_read@reader_writer_lock@Concurrency@@QEAAXXZ"); + SET(preader_writer_lock_unlock, "?unlock@reader_writer_lock@Concurrency@@QEAAXXZ"); + SET(preader_writer_lock_try_lock, "?try_lock@reader_writer_lock@Concurrency@@QEAA_NXZ"); + SET(preader_writer_lock_try_lock_read, "?try_lock_read@reader_writer_lock@Concurrency@@QEAA_NXZ"); } else { SET(pSpinWait_ctor_yield, "??0?$_SpinWait@$00@details@Concurrency@@QAE@P6AXXZ@Z"); SET(pSpinWait_dtor, "??_F?$_SpinWait@$00@details@Concurrency@@QAEXXZ"); @@ -203,6 +219,14 @@ static BOOL init(void) SET(pSpinWait__SetSpinCount, "?_SetSpinCount@?$_SpinWait@$00@details@Concurrency@@QAEXI@Z"); SET(pSpinWait__ShouldSpinAgain, "?_ShouldSpinAgain@?$_SpinWait@$00@details@Concurrency@@IAE_NXZ"); SET(pSpinWait__SpinOnce, "?_SpinOnce@?$_SpinWait@$00@details@Concurrency@@QAE_NXZ"); + + SET(preader_writer_lock_ctor, "??0reader_writer_lock@Concurrency@@QAE@XZ"); + SET(preader_writer_lock_dtor, "??1reader_writer_lock@Concurrency@@QAE@XZ"); + SET(preader_writer_lock_lock, "?lock@reader_writer_lock@Concurrency@@QAEXXZ"); + SET(preader_writer_lock_lock_read, "?lock_read@reader_writer_lock@Concurrency@@QAEXXZ"); + SET(preader_writer_lock_unlock, "?unlock@reader_writer_lock@Concurrency@@QAEXXZ"); + SET(preader_writer_lock_try_lock, "?try_lock@reader_writer_lock@Concurrency@@QAE_NXZ"); + SET(preader_writer_lock_try_lock_read, "?try_lock_read@reader_writer_lock@Concurrency@@QAE_NXZ"); } init_thiscall_thunk(); @@ -557,6 +581,65 @@ static void test__SpinWait(void) call_func1(pSpinWait_dtor, &sp); } +static DWORD WINAPI lock_read_thread(void *rw_lock) +{ + return (MSVCRT_bool)call_func1(preader_writer_lock_try_lock_read, rw_lock); +} + +static void test_reader_writer_lock(void) +{ + /* define reader_writer_lock data big enough to hold every version of structure */ + char rw_lock[100]; + MSVCRT_bool ret; + HANDLE thread; + DWORD d; + + call_func1(preader_writer_lock_ctor, rw_lock); + + ret = call_func1(preader_writer_lock_try_lock, rw_lock); + ok(ret, "reader_writer_lock:try_lock returned %x\n", ret); + ret = call_func1(preader_writer_lock_try_lock, rw_lock); + ok(!ret, "reader_writer_lock:try_lock returned %x\n", ret); + ret = call_func1(preader_writer_lock_try_lock_read, rw_lock); + ok(!ret, "reader_writer_lock:try_lock_read returned %x\n", ret); + call_func1(preader_writer_lock_unlock, rw_lock); + + /* test lock for read on already locked section */ + ret = call_func1(preader_writer_lock_try_lock_read, rw_lock); + ok(ret, "reader_writer_lock:try_lock_read returned %x\n", ret); + ret = call_func1(preader_writer_lock_try_lock_read, rw_lock); + ok(ret, "reader_writer_lock:try_lock_read returned %x\n", ret); + ret = call_func1(preader_writer_lock_try_lock, rw_lock); + ok(!ret, "reader_writer_lock:try_lock returned %x\n", ret); + call_func1(preader_writer_lock_unlock, rw_lock); + ret = call_func1(preader_writer_lock_try_lock, rw_lock); + ok(!ret, "reader_writer_lock:try_lock returned %x\n", ret); + call_func1(preader_writer_lock_unlock, rw_lock); + ret = call_func1(preader_writer_lock_try_lock, rw_lock); + ok(ret, "reader_writer_lock:try_lock returned %x\n", ret); + call_func1(preader_writer_lock_unlock, rw_lock); + + call_func1(preader_writer_lock_lock, rw_lock); + thread = CreateThread(NULL, 0, lock_read_thread, rw_lock, 0, NULL); + ok(thread != NULL, "CreateThread failed: %d\n", GetLastError()); + WaitForSingleObject(thread, INFINITE); + ok(GetExitCodeThread(thread, &d), "GetExitCodeThread failed\n"); + ok(!d, "reader_writer_lock::try_lock_read succeeded on already locked object\n"); + CloseHandle(thread); + call_func1(preader_writer_lock_unlock, rw_lock); + + call_func1(preader_writer_lock_lock_read, rw_lock); + thread = CreateThread(NULL, 0, lock_read_thread, rw_lock, 0, NULL); + ok(thread != NULL, "CreateThread failed: %d\n", GetLastError()); + WaitForSingleObject(thread, INFINITE); + ok(GetExitCodeThread(thread, &d), "GetExitCodeThread failed\n"); + ok(d, "reader_writer_lock::try_lock_read failed on object locked for reading\n"); + CloseHandle(thread); + call_func1(preader_writer_lock_unlock, rw_lock); + + call_func1(preader_writer_lock_dtor, rw_lock); +} + START_TEST(msvcr100) { if (!init()) @@ -568,4 +651,5 @@ START_TEST(msvcr100) test__aligned_msize(); test_atoi(); test__SpinWait(); + test_reader_writer_lock(); }

1 0

Piotr Caban : msvcr100: Add reader_writer_lock implementation.
by Alexandre Julliard 26 Jan '17

26 Jan '17

Module: wine Branch: master Commit: a9c215cee3fee0df604c993e5ced26d4d525c7c2 URL: http://source.winehq.org/git/wine.git/?a=commit;h=a9c215cee3fee0df604c993e5… Author: Piotr Caban <piotr(a)codeweavers.com> Date: Thu Jan 26 17:30:30 2017 +0100 msvcr100: Add reader_writer_lock implementation. Signed-off-by: Piotr Caban <piotr(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/msvcrt/lock.c | 177 ++++++++++++++++++++++++++++++++++++++++++++++++++--- 1 file changed, 170 insertions(+), 7 deletions(-) diff --git a/dlls/msvcrt/lock.c b/dlls/msvcrt/lock.c index 0aa1f1f..8c1061a 100644 --- a/dlls/msvcrt/lock.c +++ b/dlls/msvcrt/lock.c @@ -746,7 +746,17 @@ typedef struct DEFINE_THISCALL_WRAPPER(reader_writer_lock_ctor, 4) reader_writer_lock* __thiscall reader_writer_lock_ctor(reader_writer_lock *this) { - FIXME("(%p) stub\n", this); + TRACE("(%p)\n", this); + + if (!keyed_event) { + HANDLE event; + + NtCreateKeyedEvent(&event, GENERIC_READ|GENERIC_WRITE, NULL, 0); + if (InterlockedCompareExchangePointer(&keyed_event, event, NULL) != NULL) + NtClose(event); + } + + memset(this, 0, sizeof(*this)); return this; } @@ -755,7 +765,42 @@ reader_writer_lock* __thiscall reader_writer_lock_ctor(reader_writer_lock *this) DEFINE_THISCALL_WRAPPER(reader_writer_lock_dtor, 4) void __thiscall reader_writer_lock_dtor(reader_writer_lock *this) { - FIXME("(%p) stub\n", this); + TRACE("(%p)\n", this); + + if (this->thread_id != 0 || this->count) + WARN("destroying locked reader_writer_lock\n"); +} + +static inline void spin_wait_for_next_rwl(rwl_queue *q) +{ + SpinWait sw; + + if(q->next) return; + + SpinWait_ctor(&sw, &spin_wait_yield); + SpinWait__Reset(&sw); + while(!q->next) + SpinWait__SpinOnce(&sw); + SpinWait_dtor(&sw); +} + +/* Remove when proper InterlockedOr implementation is added to wine */ +static LONG InterlockedOr(LONG *d, LONG v) +{ + LONG l; + while (~(l = *d) & v) + if (InterlockedCompareExchange(d, l|v, l) == l) break; + return l; +} + +static LONG InterlockedAnd(LONG *d, LONG v) +{ + LONG l = *d, old; + while ((l & v) != l) { + if((old = InterlockedCompareExchange(d, l&v, l)) == l) break; + l = old; + } + return l; } /* ?lock@reader_writer_lock@Concurrency@@QAEXXZ */ @@ -763,7 +808,30 @@ void __thiscall reader_writer_lock_dtor(reader_writer_lock *this) DEFINE_THISCALL_WRAPPER(reader_writer_lock_lock, 4) void __thiscall reader_writer_lock_lock(reader_writer_lock *this) { - FIXME("(%p) stub\n", this); + rwl_queue q = { NULL }, *last; + + TRACE("(%p)\n", this); + + if (this->thread_id == GetCurrentThreadId()) + FIXME("throw improper_lock exception\n"); + + last = InterlockedExchangePointer((void**)&this->writer_tail, &q); + if (last) { + last->next = &q; + NtWaitForKeyedEvent(keyed_event, &q, 0, NULL); + } else { + this->writer_head = &q; + if (InterlockedOr(&this->count, WRITER_WAITING)) + NtWaitForKeyedEvent(keyed_event, &q, 0, NULL); + } + + this->thread_id = GetCurrentThreadId(); + this->writer_head = &this->active; + this->active.next = NULL; + if (InterlockedCompareExchangePointer((void**)&this->writer_tail, &this->active, &q) != &q) { + spin_wait_for_next_rwl(&q); + this->active.next = q.next; + } } /* ?lock_read@reader_writer_lock@Concurrency@@QAEXXZ */ @@ -771,7 +839,37 @@ void __thiscall reader_writer_lock_lock(reader_writer_lock *this) DEFINE_THISCALL_WRAPPER(reader_writer_lock_lock_read, 4) void __thiscall reader_writer_lock_lock_read(reader_writer_lock *this) { - FIXME("(%p) stub\n", this); + rwl_queue q; + + TRACE("(%p)\n", this); + + if (this->thread_id == GetCurrentThreadId()) + FIXME("throw improper_lock exception\n"); + + do { + q.next = this->reader_head; + } while(InterlockedCompareExchangePointer((void**)&this->reader_head, &q, q.next) != q.next); + + if (!q.next) { + rwl_queue *head; + LONG count; + + while (!((count = this->count) & WRITER_WAITING)) + if (InterlockedCompareExchange(&this->count, count+1, count) == count) break; + + if (count & WRITER_WAITING) + NtWaitForKeyedEvent(keyed_event, &q, 0, NULL); + + head = InterlockedExchangePointer((void**)&this->reader_head, NULL); + while(head && head != &q) { + rwl_queue *next = head->next; + InterlockedIncrement(&this->count); + NtReleaseKeyedEvent(keyed_event, head, 0, NULL); + head = next; + } + } else { + NtWaitForKeyedEvent(keyed_event, &q, 0, NULL); + } } /* ?try_lock@reader_writer_lock@Concurrency@@QAE_NXZ */ @@ -779,7 +877,37 @@ void __thiscall reader_writer_lock_lock_read(reader_writer_lock *this) DEFINE_THISCALL_WRAPPER(reader_writer_lock_try_lock, 4) MSVCRT_bool __thiscall reader_writer_lock_try_lock(reader_writer_lock *this) { - FIXME("(%p) stub\n", this); + rwl_queue q = { NULL }; + + TRACE("(%p)\n", this); + + if (this->thread_id == GetCurrentThreadId()) + FIXME("throw improper_lock exception\n"); + + if (InterlockedCompareExchangePointer((void**)&this->writer_tail, &q, NULL)) + return FALSE; + this->writer_head = &q; + if (!InterlockedCompareExchange(&this->count, WRITER_WAITING, 0)) { + this->thread_id = GetCurrentThreadId(); + this->writer_head = &this->active; + this->active.next = NULL; + if (InterlockedCompareExchangePointer((void**)&this->writer_tail, &this->active, &q) != &q) { + spin_wait_for_next_rwl(&q); + this->active.next = q.next; + } + return TRUE; + } + + if (InterlockedCompareExchangePointer((void**)&this->writer_tail, NULL, &q) == &q) + return FALSE; + spin_wait_for_next_rwl(&q); + this->writer_head = q.next; + if (!InterlockedOr(&this->count, WRITER_WAITING)) { + this->thread_id = GetCurrentThreadId(); + this->writer_head = &this->active; + this->active.next = q.next; + return TRUE; + } return FALSE; } @@ -788,7 +916,12 @@ MSVCRT_bool __thiscall reader_writer_lock_try_lock(reader_writer_lock *this) DEFINE_THISCALL_WRAPPER(reader_writer_lock_try_lock_read, 4) MSVCRT_bool __thiscall reader_writer_lock_try_lock_read(reader_writer_lock *this) { - FIXME("(%p) stub\n", this); + LONG count; + + TRACE("(%p)\n", this); + + while (!((count = this->count) & WRITER_WAITING)) + if (InterlockedCompareExchange(&this->count, count+1, count) == count) return TRUE; return FALSE; } @@ -797,7 +930,37 @@ MSVCRT_bool __thiscall reader_writer_lock_try_lock_read(reader_writer_lock *this DEFINE_THISCALL_WRAPPER(reader_writer_lock_unlock, 4) void __thiscall reader_writer_lock_unlock(reader_writer_lock *this) { - FIXME("(%p) stub\n", this); + LONG count; + rwl_queue *head, *next; + + TRACE("(%p)\n", this); + + if ((count = this->count) & ~WRITER_WAITING) { + count = InterlockedDecrement(&this->count); + if (count != WRITER_WAITING) + return; + NtReleaseKeyedEvent(keyed_event, this->writer_head, 0, NULL); + return; + } + + this->thread_id = 0; + next = this->writer_head->next; + if (next) { + NtReleaseKeyedEvent(keyed_event, next, 0, NULL); + return; + } + InterlockedAnd(&this->count, ~WRITER_WAITING); + head = InterlockedExchangePointer((void**)&this->reader_head, NULL); + while (head) { + next = head->next; + InterlockedIncrement(&this->count); + NtReleaseKeyedEvent(keyed_event, head, 0, NULL); + head = next; + } + + if (InterlockedCompareExchangePointer((void**)&this->writer_tail, NULL, this->writer_head) == this->writer_head) + return; + InterlockedOr(&this->count, WRITER_WAITING); } #endif

1 0

Piotr Caban : msvcr100: Add reader_writer_lock class stub.
by Alexandre Julliard 26 Jan '17

26 Jan '17

Module: wine Branch: master Commit: 245426c77e3ca31dc3eca00a79c734b71a6bd754 URL: http://source.winehq.org/git/wine.git/?a=commit;h=245426c77e3ca31dc3eca00a7… Author: Piotr Caban <piotr(a)codeweavers.com> Date: Thu Jan 26 17:30:20 2017 +0100 msvcr100: Add reader_writer_lock class stub. Signed-off-by: Piotr Caban <piotr(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/concrt140/concrt140.spec | 28 ++++++------- dlls/msvcr100/msvcr100.spec | 28 ++++++------- dlls/msvcr110/msvcr110.spec | 42 ++++++++++---------- dlls/msvcr120/msvcr120.spec | 42 ++++++++++---------- dlls/msvcr120_app/msvcr120_app.spec | 42 ++++++++++---------- dlls/msvcrt/lock.c | 79 +++++++++++++++++++++++++++++++++++++ 6 files changed, 170 insertions(+), 91 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=245426c77e3ca31dc3eca…

1 0

Aric Stewart : hidclass.sys: When processing reads fill all the buffers.
by Alexandre Julliard 26 Jan '17

26 Jan '17

Module: wine Branch: master Commit: 68ecd3b76a7f3ef901cf2b648d25773b8ab9db91 URL: http://source.winehq.org/git/wine.git/?a=commit;h=68ecd3b76a7f3ef901cf2b648… Author: Aric Stewart <aric(a)codeweavers.com> Date: Thu Jan 26 08:59:22 2017 -0600 hidclass.sys: When processing reads fill all the buffers. Signed-off-by: Aric Stewart <aric(a)codeweavers.com> Signed-off-by: Sebastian Lackner <sebastian(a)fds-team.de> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/hidclass.sys/buffer.c | 36 +++++++++++++++++++++++++++++++++++- dlls/hidclass.sys/device.c | 2 +- dlls/hidclass.sys/hid.h | 3 ++- 3 files changed, 38 insertions(+), 3 deletions(-) diff --git a/dlls/hidclass.sys/buffer.c b/dlls/hidclass.sys/buffer.c index 0b29f97..731f1e0 100644 --- a/dlls/hidclass.sys/buffer.c +++ b/dlls/hidclass.sys/buffer.c @@ -127,7 +127,7 @@ NTSTATUS RingBuffer_SetSize(struct ReportRingBuffer *ring, UINT size) return STATUS_SUCCESS; } -void RingBuffer_Read(struct ReportRingBuffer *ring, UINT index, void *output, UINT *size) +void RingBuffer_ReadNew(struct ReportRingBuffer *ring, UINT index, void *output, UINT *size) { void *ret = NULL; @@ -155,6 +155,40 @@ void RingBuffer_Read(struct ReportRingBuffer *ring, UINT index, void *output, UI } } +void RingBuffer_Read(struct ReportRingBuffer *ring, UINT index, void *output, UINT *size) +{ + int pointer; + void *ret = NULL; + + EnterCriticalSection(&ring->lock); + if (index >= ring->pointer_alloc || ring->pointers[index] == POINTER_UNUSED + || ring->end == ring->start) + { + LeaveCriticalSection(&ring->lock); + *size = 0; + return; + } + + pointer = ring->pointers[index]; + + if (pointer == ring->end) + pointer--; + + if (pointer < 0) + pointer = ring->size - 1; + + ret = &ring->buffer[pointer * ring->buffer_size]; + memcpy(output, ret, ring->buffer_size); + if (pointer == ring->pointers[index]) + { + ring->pointers[index]++; + if (ring->pointers[index] == ring->size) + ring->pointers[index] = 0; + } + LeaveCriticalSection(&ring->lock); + *size = ring->buffer_size; +} + UINT RingBuffer_AddPointer(struct ReportRingBuffer *ring) { UINT idx; diff --git a/dlls/hidclass.sys/device.c b/dlls/hidclass.sys/device.c index fe79212..ff9655e 100644 --- a/dlls/hidclass.sys/device.c +++ b/dlls/hidclass.sys/device.c @@ -663,7 +663,7 @@ NTSTATUS WINAPI HID_Device_read(DEVICE_OBJECT *device, IRP *irp) ptr = PtrToUlong( irp->Tail.Overlay.OriginalFileObject->FsContext ); irp->IoStatus.Information = 0; - RingBuffer_Read(ext->ring_buffer, ptr, packet, &buffer_size); + RingBuffer_ReadNew(ext->ring_buffer, ptr, packet, &buffer_size); if (buffer_size) { diff --git a/dlls/hidclass.sys/hid.h b/dlls/hidclass.sys/hid.h index adc547c..1829319 100644 --- a/dlls/hidclass.sys/hid.h +++ b/dlls/hidclass.sys/hid.h @@ -62,7 +62,8 @@ typedef struct _BASE_DEVICE_EXTENSION { void RingBuffer_Write(struct ReportRingBuffer *buffer, void *data) DECLSPEC_HIDDEN; UINT RingBuffer_AddPointer(struct ReportRingBuffer *buffer) DECLSPEC_HIDDEN; void RingBuffer_RemovePointer(struct ReportRingBuffer *ring, UINT index) DECLSPEC_HIDDEN; -void RingBuffer_Read(struct ReportRingBuffer *buffer, UINT index, void *output, UINT *size) DECLSPEC_HIDDEN; +void RingBuffer_Read(struct ReportRingBuffer *ring, UINT index, void *output, UINT *size) DECLSPEC_HIDDEN; +void RingBuffer_ReadNew(struct ReportRingBuffer *buffer, UINT index, void *output, UINT *size) DECLSPEC_HIDDEN; UINT RingBuffer_GetBufferSize(struct ReportRingBuffer *buffer) DECLSPEC_HIDDEN; UINT RingBuffer_GetSize(struct ReportRingBuffer *buffer) DECLSPEC_HIDDEN; void RingBuffer_Destroy(struct ReportRingBuffer *buffer) DECLSPEC_HIDDEN;

1 0

Józef Kucia : wined3d: Do not pass LOD to texelFetch( ) for buffer resources.
by Alexandre Julliard 26 Jan '17

26 Jan '17

Module: wine Branch: master Commit: 65089c395dbb44eb4f2bf8a7722092254d277549 URL: http://source.winehq.org/git/wine.git/?a=commit;h=65089c395dbb44eb4f2bf8a77… Author: Józef Kucia <jkucia(a)codeweavers.com> Date: Thu Jan 26 13:07:32 2017 +0100 wined3d: Do not pass LOD to texelFetch() for buffer resources. 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/glsl_shader.c | 14 ++++++++++++-- 1 file changed, 12 insertions(+), 2 deletions(-) diff --git a/dlls/wined3d/glsl_shader.c b/dlls/wined3d/glsl_shader.c index 3cdc957..c6a51c5 100644 --- a/dlls/wined3d/glsl_shader.c +++ b/dlls/wined3d/glsl_shader.c @@ -4956,10 +4956,12 @@ static void shader_glsl_resinfo(const struct wined3d_shader_instruction *ins) /* FIXME: The current implementation does not handle multisample textures correctly. */ static void shader_glsl_ld(const struct wined3d_shader_instruction *ins) { + const struct wined3d_shader_reg_maps *reg_maps = ins->ctx->reg_maps; unsigned int resource_idx, sampler_idx, sampler_bind_idx; struct glsl_src_param coord_param, lod_param; struct glsl_sample_function sample_function; DWORD flags = WINED3D_GLSL_SAMPLE_LOAD; + BOOL has_lod_param; if (wined3d_shader_instruction_has_texel_offset(ins)) flags |= WINED3D_GLSL_SAMPLE_OFFSET; @@ -4967,12 +4969,20 @@ static void shader_glsl_ld(const struct wined3d_shader_instruction *ins) resource_idx = ins->src[1].reg.idx[0].offset; sampler_idx = WINED3D_SAMPLER_DEFAULT; + if (resource_idx >= ARRAY_SIZE(reg_maps->resource_info)) + { + ERR("Invalid resource index %u.\n", resource_idx); + return; + } + has_lod_param = reg_maps->resource_info[resource_idx].type != WINED3D_SHADER_RESOURCE_BUFFER; + shader_glsl_get_sample_function(ins->ctx, resource_idx, sampler_idx, flags, &sample_function); shader_glsl_add_src_param(ins, &ins->src[0], sample_function.coord_mask, &coord_param); shader_glsl_add_src_param(ins, &ins->src[0], WINED3DSP_WRITEMASK_3, &lod_param); - sampler_bind_idx = shader_glsl_find_sampler(&ins->ctx->reg_maps->sampler_map, resource_idx, sampler_idx); + sampler_bind_idx = shader_glsl_find_sampler(&reg_maps->sampler_map, resource_idx, sampler_idx); shader_glsl_gen_sample_code(ins, sampler_bind_idx, &sample_function, ins->src[1].swizzle, - NULL, NULL, lod_param.param_str, &ins->texel_offset, "%s", coord_param.param_str); + NULL, NULL, has_lod_param ? lod_param.param_str : NULL, &ins->texel_offset, + "%s", coord_param.param_str); shader_glsl_release_sample_function(ins->ctx, &sample_function); }

1 0

Józef Kucia : wined3d: Add initial support for shader buffer views.
by Alexandre Julliard 26 Jan '17

26 Jan '17

Module: wine Branch: master Commit: 81878178303ed52a18330fdb668c7d1dc1c02ade URL: http://source.winehq.org/git/wine.git/?a=commit;h=81878178303ed52a18330fdb6… Author: Józef Kucia <jkucia(a)codeweavers.com> Date: Thu Jan 26 13:07:31 2017 +0100 wined3d: Add initial support for shader buffer views. 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/buffer.c | 2 +- dlls/wined3d/view.c | 51 +++++++++++++++++++++++++++++++++++++++++- dlls/wined3d/wined3d_private.h | 2 ++ 3 files changed, 53 insertions(+), 2 deletions(-) diff --git a/dlls/wined3d/buffer.c b/dlls/wined3d/buffer.c index 6883b40..7d0552b 100644 --- a/dlls/wined3d/buffer.c +++ b/dlls/wined3d/buffer.c @@ -567,7 +567,7 @@ static BOOL wined3d_buffer_prepare_location(struct wined3d_buffer *buffer, } } -static BOOL wined3d_buffer_load_location(struct wined3d_buffer *buffer, +BOOL wined3d_buffer_load_location(struct wined3d_buffer *buffer, struct wined3d_context *context, DWORD location) { const struct wined3d_gl_info *gl_info = context->gl_info; diff --git a/dlls/wined3d/view.c b/dlls/wined3d/view.c index 18b7180..c63d913 100644 --- a/dlls/wined3d/view.c +++ b/dlls/wined3d/view.c @@ -121,6 +121,35 @@ static void create_texture_view(struct wined3d_gl_view *view, GLenum view_target context_release(context); } +static void create_buffer_texture(struct wined3d_gl_view *view, + struct wined3d_buffer *buffer, const struct wined3d_format *view_format) +{ + const struct wined3d_gl_info *gl_info; + struct wined3d_context *context; + + context = context_acquire(buffer->resource.device, NULL); + gl_info = context->gl_info; + if (!gl_info->supported[ARB_TEXTURE_BUFFER_OBJECT]) + { + FIXME("OpenGL implementation does not support buffer textures.\n"); + context_release(context); + return; + } + + wined3d_buffer_load_location(buffer, context, WINED3D_LOCATION_BUFFER); + + view->target = GL_TEXTURE_BUFFER; + gl_info->gl_ops.gl.p_glGenTextures(1, &view->name); + + context_bind_texture(context, GL_TEXTURE_BUFFER, view->name); + GL_EXTCALL(glTexBuffer(GL_TEXTURE_BUFFER, view_format->glInternal, buffer->buffer_object)); + checkGLcall("Create buffer texture"); + + context_invalidate_state(context, STATE_SHADER_RESOURCE_BINDING); + + context_release(context); +} + ULONG CDECL wined3d_rendertarget_view_incref(struct wined3d_rendertarget_view *view) { ULONG refcount = InterlockedIncrement(&view->refcount); @@ -415,7 +444,27 @@ static HRESULT wined3d_shader_resource_view_init(struct wined3d_shader_resource_ if (resource->type == WINED3D_RTYPE_BUFFER) { - FIXME("Buffer shader resource views not supported.\n"); + struct wined3d_buffer *buffer = buffer_from_resource(resource); + + if (desc->format_id == WINED3DFMT_UNKNOWN) + { + FIXME("Structured buffer views not supported.\n"); + } + else if (desc->flags & WINED3D_VIEW_BUFFER_RAW) + { + FIXME("Raw buffer views not supported.\n"); + } + else + { + /* FIXME: Support for buffer offsets can be implemented using ARB_texture_buffer_range. */ + if (desc->u.buffer.start_idx + || desc->u.buffer.count * view_format->byte_count != buffer->resource.size) + { + FIXME("Ignoring buffer range %u-%u.\n", desc->u.buffer.start_idx, desc->u.buffer.count); + } + + create_buffer_texture(&view->gl_view, buffer, view_format); + } } else { diff --git a/dlls/wined3d/wined3d_private.h b/dlls/wined3d/wined3d_private.h index 5c94803..051836c 100644 --- a/dlls/wined3d/wined3d_private.h +++ b/dlls/wined3d/wined3d_private.h @@ -3256,6 +3256,8 @@ DWORD wined3d_buffer_get_memory(struct wined3d_buffer *buffer, void wined3d_buffer_invalidate_location(struct wined3d_buffer *buffer, DWORD location) DECLSPEC_HIDDEN; void wined3d_buffer_load(struct wined3d_buffer *buffer, struct wined3d_context *context, const struct wined3d_state *state) DECLSPEC_HIDDEN; +BOOL wined3d_buffer_load_location(struct wined3d_buffer *buffer, + struct wined3d_context *context, DWORD location) DECLSPEC_HIDDEN; BYTE *wined3d_buffer_load_sysmem(struct wined3d_buffer *buffer, struct wined3d_context *context) DECLSPEC_HIDDEN; HRESULT wined3d_buffer_copy(struct wined3d_buffer *dst_buffer, unsigned int dst_offset, struct wined3d_buffer *src_buffer, unsigned int src_offset, unsigned int size) DECLSPEC_HIDDEN;

1 0

Józef Kucia : wined3d: Create dummy buffer textures.
by Alexandre Julliard 26 Jan '17

26 Jan '17

Module: wine Branch: master Commit: e800093706e7b56ae1df73182ec52f906295a975 URL: http://source.winehq.org/git/wine.git/?a=commit;h=e800093706e7b56ae1df73182… Author: Józef Kucia <jkucia(a)codeweavers.com> Date: Thu Jan 26 13:07:30 2017 +0100 wined3d: Create dummy buffer textures. 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/context.c | 7 +++++++ dlls/wined3d/device.c | 26 ++++++++++++++++++++++++++ dlls/wined3d/wined3d_private.h | 1 + 3 files changed, 34 insertions(+) diff --git a/dlls/wined3d/context.c b/dlls/wined3d/context.c index b4e40fd..cd0057e 100644 --- a/dlls/wined3d/context.c +++ b/dlls/wined3d/context.c @@ -1545,6 +1545,9 @@ void context_bind_dummy_textures(const struct wined3d_device *device, const stru if (gl_info->supported[EXT_TEXTURE_ARRAY]) gl_info->gl_ops.gl.p_glBindTexture(GL_TEXTURE_2D_ARRAY, device->dummy_textures.tex_2d_array); + if (gl_info->supported[ARB_TEXTURE_BUFFER_OBJECT]) + gl_info->gl_ops.gl.p_glBindTexture(GL_TEXTURE_BUFFER, device->dummy_textures.tex_buffer); + checkGLcall("Bind dummy textures"); } } @@ -2443,6 +2446,10 @@ void context_bind_texture(struct wined3d_context *context, GLenum target, GLuint gl_info->gl_ops.gl.p_glBindTexture(GL_TEXTURE_3D, device->dummy_textures.tex_3d); checkGLcall("glBindTexture"); break; + case GL_TEXTURE_BUFFER: + gl_info->gl_ops.gl.p_glBindTexture(GL_TEXTURE_BUFFER, device->dummy_textures.tex_buffer); + checkGLcall("glBindTexture"); + break; default: ERR("Unexpected texture target %#x.\n", old_texture_type); } diff --git a/dlls/wined3d/device.c b/dlls/wined3d/device.c index 75a939e..7898527 100644 --- a/dlls/wined3d/device.c +++ b/dlls/wined3d/device.c @@ -694,6 +694,29 @@ static void create_dummy_textures(struct wined3d_device *device, struct wined3d_ checkGLcall("glTexImage3D"); } + if (gl_info->supported[ARB_TEXTURE_BUFFER_OBJECT]) + { + GLuint buffer; + + GL_EXTCALL(glGenBuffers(1, &buffer)); + GL_EXTCALL(glBindBuffer(GL_TEXTURE_BUFFER, buffer)); + GL_EXTCALL(glBufferData(GL_TEXTURE_BUFFER, sizeof(color), &color, GL_STATIC_DRAW)); + GL_EXTCALL(glBindBuffer(GL_TEXTURE_BUFFER, 0)); + checkGLcall("Create buffer object"); + + gl_info->gl_ops.gl.p_glGenTextures(1, &device->dummy_textures.tex_buffer); + checkGLcall("glGenTextures"); + TRACE("Dummy buffer texture given name %u.\n", device->dummy_textures.tex_buffer); + + gl_info->gl_ops.gl.p_glBindTexture(GL_TEXTURE_BUFFER, device->dummy_textures.tex_buffer); + checkGLcall("glBindTexture"); + GL_EXTCALL(glTexBuffer(GL_TEXTURE_BUFFER, GL_RGBA8, buffer)); + checkGLcall("glTexBuffer"); + + GL_EXTCALL(glDeleteBuffers(1, &buffer)); + checkGLcall("glDeleteBuffers"); + } + context_bind_dummy_textures(device, context); } @@ -702,6 +725,9 @@ static void destroy_dummy_textures(struct wined3d_device *device, struct wined3d { const struct wined3d_gl_info *gl_info = context->gl_info; + if (gl_info->supported[ARB_TEXTURE_BUFFER_OBJECT]) + gl_info->gl_ops.gl.p_glDeleteTextures(1, &device->dummy_textures.tex_buffer); + if (gl_info->supported[EXT_TEXTURE_ARRAY]) gl_info->gl_ops.gl.p_glDeleteTextures(1, &device->dummy_textures.tex_2d_array); diff --git a/dlls/wined3d/wined3d_private.h b/dlls/wined3d/wined3d_private.h index 4ee9d5c..5c94803 100644 --- a/dlls/wined3d/wined3d_private.h +++ b/dlls/wined3d/wined3d_private.h @@ -2587,6 +2587,7 @@ struct wined3d_device GLuint tex_3d; GLuint tex_cube; GLuint tex_2d_array; + GLuint tex_buffer; } dummy_textures; /* Default sampler used to emulate the direct resource access without using wined3d_sampler */

1 0

← Newer
1
...
17
18
19
20
21
22
23
...
50
Older →

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

2003

2002

2001

wine-commits January 2017