[PATCH 2/5] d3d8/tests: Test device caps on multiple adapters.
Zhiyi Zhang
zzhang at codeweavers.com
Mon Mar 16 08:23:14 CDT 2020
On 3/16/20 8:02 PM, Henri Verbeet wrote:
> On Fri, 13 Mar 2020 at 13:06, Zhiyi Zhang <zzhang at codeweavers.com> wrote:
>> -static IDirect3DDevice8 *create_device(IDirect3D8 *d3d8, HWND focus_window, const struct device_desc *desc)
>> +static IDirect3DDevice8 *create_device_on_adapter(IDirect3D8 *d3d8, unsigned int adapter_idx,
>> + HWND focus_window, const struct device_desc *desc)
> Introducing an extra function works, but I think I'd prefer adding a
> field for the adapter to the existing device_desc structure.
Well. That would introduce a lot of diffs because a NULL can not be passed to 'desc' any more.
Adding an extra function makes the diff smaller. And if later we want to create a device
using the default adapter, we can still use create_device directly.
>> + for (adapter_idx = 0; adapter_idx < adapter_count; ++adapter_idx)
>> {
>> - skip("Failed to create a D3D device.\n");
>> - IDirect3D8_Release(d3d);
>> - DestroyWindow(window);
>> - return;
>> + /* Test IDirect3D8_GetDeviceCaps */
>> + hr = IDirect3D8_GetDeviceCaps(d3d, adapter_idx, D3DDEVTYPE_HAL, &caps);
>> + ok(hr == D3D_OK || hr == D3DERR_NOTAVAILABLE, "GetDeviceCaps failed, hr %#x.\n", hr);
>> + if (hr == D3DERR_NOTAVAILABLE)
>> + {
>> + skip("No Direct3D support, skipping test.\n");
>> + break;
>> + }
>> + ok(caps.AdapterOrdinal == adapter_idx, "Expect adapter ordinal %u, got %u.\n", adapter_idx,
>> + caps.AdapterOrdinal);
>> +
>> + /* Test IDirect3DDevice8_GetDeviceCaps */
>> + device = create_device_on_adapter(d3d, adapter_idx, window, NULL);
>> + ok(!!device, "Failed to create a D3D device on adapter %u.\n", adapter_idx);
>> + hr = IDirect3DDevice8_GetDeviceCaps(device, &caps);
>> + ok(SUCCEEDED(hr), "Failed to get caps, hr %#x.\n", hr);
>> +
>> + ok(caps.AdapterOrdinal == adapter_idx, "Expect adapter ordinal %u, got %u.\n", adapter_idx,
>> + caps.AdapterOrdinal);
>> + ok(!(caps.Caps & ~D3DCAPS_READ_SCANLINE), "Caps field has unexpected flags %#x.\n", caps.Caps);
>> + ok(!(caps.Caps2 & ~(D3DCAPS2_CANCALIBRATEGAMMA | D3DCAPS2_CANRENDERWINDOWED
>> + | D3DCAPS2_CANMANAGERESOURCE | D3DCAPS2_DYNAMICTEXTURES | D3DCAPS2_FULLSCREENGAMMA
>> + | D3DCAPS2_NO2DDURING3DSCENE | D3DCAPS2_RESERVED)),
>> + "Caps2 field has unexpected flags %#x.\n", caps.Caps2);
>> + /* Nvidia returns that 0x400 flag, which is probably Vista+
>> + * D3DCAPS3_DXVAHD from d3d9caps.h */
>> + /* AMD doesn't filter all the ddraw / d3d9 caps. Consider that behavior
>> + * broken. */
>> + ok(!(caps.Caps3 & ~(D3DCAPS3_ALPHA_FULLSCREEN_FLIP_OR_DISCARD | D3DCAPS3_RESERVED | 0x400))
>> + || broken(!(caps.Caps3 & ~(D3DCAPS3_ALPHA_FULLSCREEN_FLIP_OR_DISCARD | 0x80))),
>> + "Caps3 field has unexpected flags %#x.\n", caps.Caps3);
>> + ok(!(caps.PrimitiveMiscCaps & ~(D3DPMISCCAPS_MASKZ | D3DPMISCCAPS_LINEPATTERNREP
>> + | D3DPMISCCAPS_CULLNONE | D3DPMISCCAPS_CULLCW | D3DPMISCCAPS_CULLCCW
>> + | D3DPMISCCAPS_COLORWRITEENABLE | D3DPMISCCAPS_CLIPPLANESCALEDPOINTS
>> + | D3DPMISCCAPS_CLIPTLVERTS | D3DPMISCCAPS_TSSARGTEMP | D3DPMISCCAPS_BLENDOP
>> + | D3DPMISCCAPS_NULLREFERENCE))
>> + || broken(!(caps.PrimitiveMiscCaps & ~0x003fdff6)),
>> + "PrimitiveMiscCaps field has unexpected flags %#x.\n", caps.PrimitiveMiscCaps);
>> + /* AMD includes an unknown 0x2 flag. */
>> + ok(!(caps.RasterCaps & ~(D3DPRASTERCAPS_DITHER | D3DPRASTERCAPS_PAT | D3DPRASTERCAPS_ZTEST
>> + | D3DPRASTERCAPS_FOGVERTEX | D3DPRASTERCAPS_FOGTABLE | D3DPRASTERCAPS_ANTIALIASEDGES
>> + | D3DPRASTERCAPS_MIPMAPLODBIAS | D3DPRASTERCAPS_ZBIAS | D3DPRASTERCAPS_ZBUFFERLESSHSR
>> + | D3DPRASTERCAPS_FOGRANGE | D3DPRASTERCAPS_ANISOTROPY | D3DPRASTERCAPS_WBUFFER
>> + | D3DPRASTERCAPS_WFOG | D3DPRASTERCAPS_ZFOG | D3DPRASTERCAPS_COLORPERSPECTIVE
>> + | D3DPRASTERCAPS_STRETCHBLTMULTISAMPLE))
>> + || broken(!(caps.RasterCaps & ~0x0ff7f19b)),
>> + "RasterCaps field has unexpected flags %#x.\n", caps.RasterCaps);
>> + ok(!(caps.SrcBlendCaps & ~(D3DPBLENDCAPS_ZERO | D3DPBLENDCAPS_ONE | D3DPBLENDCAPS_SRCCOLOR
>> + | D3DPBLENDCAPS_INVSRCCOLOR | D3DPBLENDCAPS_SRCALPHA | D3DPBLENDCAPS_INVSRCALPHA
>> + | D3DPBLENDCAPS_DESTALPHA | D3DPBLENDCAPS_INVDESTALPHA | D3DPBLENDCAPS_DESTCOLOR
>> + | D3DPBLENDCAPS_INVDESTCOLOR | D3DPBLENDCAPS_SRCALPHASAT | D3DPBLENDCAPS_BOTHSRCALPHA
>> + | D3DPBLENDCAPS_BOTHINVSRCALPHA)),
>> + "SrcBlendCaps field has unexpected flags %#x.\n", caps.SrcBlendCaps);
>> + ok(!(caps.DestBlendCaps & ~(D3DPBLENDCAPS_ZERO | D3DPBLENDCAPS_ONE | D3DPBLENDCAPS_SRCCOLOR
>> + | D3DPBLENDCAPS_INVSRCCOLOR | D3DPBLENDCAPS_SRCALPHA | D3DPBLENDCAPS_INVSRCALPHA
>> + | D3DPBLENDCAPS_DESTALPHA | D3DPBLENDCAPS_INVDESTALPHA | D3DPBLENDCAPS_DESTCOLOR
>> + | D3DPBLENDCAPS_INVDESTCOLOR | D3DPBLENDCAPS_SRCALPHASAT | D3DPBLENDCAPS_BOTHSRCALPHA
>> + | D3DPBLENDCAPS_BOTHINVSRCALPHA)),
>> + "DestBlendCaps field has unexpected flags %#x.\n", caps.DestBlendCaps);
>> + ok(!(caps.TextureCaps & ~(D3DPTEXTURECAPS_PERSPECTIVE | D3DPTEXTURECAPS_POW2
>> + | D3DPTEXTURECAPS_ALPHA | D3DPTEXTURECAPS_SQUAREONLY
>> + | D3DPTEXTURECAPS_TEXREPEATNOTSCALEDBYSIZE | D3DPTEXTURECAPS_ALPHAPALETTE
>> + | D3DPTEXTURECAPS_NONPOW2CONDITIONAL | D3DPTEXTURECAPS_PROJECTED
>> + | D3DPTEXTURECAPS_CUBEMAP | D3DPTEXTURECAPS_VOLUMEMAP | D3DPTEXTURECAPS_MIPMAP
>> + | D3DPTEXTURECAPS_MIPVOLUMEMAP | D3DPTEXTURECAPS_MIPCUBEMAP
>> + | D3DPTEXTURECAPS_CUBEMAP_POW2 | D3DPTEXTURECAPS_VOLUMEMAP_POW2)),
>> + "TextureCaps field has unexpected flags %#x.\n", caps.TextureCaps);
>> + ok(!(caps.TextureFilterCaps & ~(D3DPTFILTERCAPS_MINFPOINT | D3DPTFILTERCAPS_MINFLINEAR
>> + | D3DPTFILTERCAPS_MINFANISOTROPIC | D3DPTFILTERCAPS_MIPFPOINT
>> + | D3DPTFILTERCAPS_MIPFLINEAR | D3DPTFILTERCAPS_MAGFPOINT | D3DPTFILTERCAPS_MAGFLINEAR
>> + | D3DPTFILTERCAPS_MAGFANISOTROPIC | D3DPTFILTERCAPS_MAGFAFLATCUBIC
>> + | D3DPTFILTERCAPS_MAGFGAUSSIANCUBIC))
>> + || broken(!(caps.TextureFilterCaps & ~0x0703073f)),
>> + "TextureFilterCaps field has unexpected flags %#x.\n", caps.TextureFilterCaps);
>> + ok(!(caps.CubeTextureFilterCaps & ~(D3DPTFILTERCAPS_MINFPOINT | D3DPTFILTERCAPS_MINFLINEAR
>> + | D3DPTFILTERCAPS_MINFANISOTROPIC | D3DPTFILTERCAPS_MIPFPOINT
>> + | D3DPTFILTERCAPS_MIPFLINEAR | D3DPTFILTERCAPS_MAGFPOINT | D3DPTFILTERCAPS_MAGFLINEAR
>> + | D3DPTFILTERCAPS_MAGFANISOTROPIC | D3DPTFILTERCAPS_MAGFAFLATCUBIC
>> + | D3DPTFILTERCAPS_MAGFGAUSSIANCUBIC)),
>> + "CubeTextureFilterCaps field has unexpected flags %#x.\n", caps.CubeTextureFilterCaps);
>> + ok(!(caps.VolumeTextureFilterCaps & ~(D3DPTFILTERCAPS_MINFPOINT | D3DPTFILTERCAPS_MINFLINEAR
>> + | D3DPTFILTERCAPS_MINFANISOTROPIC | D3DPTFILTERCAPS_MIPFPOINT
>> + | D3DPTFILTERCAPS_MIPFLINEAR | D3DPTFILTERCAPS_MAGFPOINT | D3DPTFILTERCAPS_MAGFLINEAR
>> + | D3DPTFILTERCAPS_MAGFANISOTROPIC | D3DPTFILTERCAPS_MAGFAFLATCUBIC
>> + | D3DPTFILTERCAPS_MAGFGAUSSIANCUBIC)),
>> + "VolumeTextureFilterCaps field has unexpected flags %#x.\n", caps.VolumeTextureFilterCaps);
>> + ok(!(caps.LineCaps & ~(D3DLINECAPS_TEXTURE | D3DLINECAPS_ZTEST | D3DLINECAPS_BLEND
>> + | D3DLINECAPS_ALPHACMP | D3DLINECAPS_FOG)),
>> + "LineCaps field has unexpected flags %#x.\n", caps.LineCaps);
>> + ok(!(caps.StencilCaps & ~(D3DSTENCILCAPS_KEEP | D3DSTENCILCAPS_ZERO | D3DSTENCILCAPS_REPLACE
>> + | D3DSTENCILCAPS_INCRSAT | D3DSTENCILCAPS_DECRSAT | D3DSTENCILCAPS_INVERT
>> + | D3DSTENCILCAPS_INCR | D3DSTENCILCAPS_DECR)),
>> + "StencilCaps field has unexpected flags %#x.\n", caps.StencilCaps);
>> + ok(!(caps.VertexProcessingCaps & ~(D3DVTXPCAPS_TEXGEN | D3DVTXPCAPS_MATERIALSOURCE7
>> + | D3DVTXPCAPS_DIRECTIONALLIGHTS | D3DVTXPCAPS_POSITIONALLIGHTS | D3DVTXPCAPS_LOCALVIEWER
>> + | D3DVTXPCAPS_TWEENING | D3DVTXPCAPS_NO_VSDT_UBYTE4)),
>> + "VertexProcessingCaps field has unexpected flags %#x.\n", caps.VertexProcessingCaps);
>> + /* Both Nvidia and AMD give 10 here. */
>> + ok(caps.MaxActiveLights <= 10,
>> + "MaxActiveLights field has unexpected value %u.\n", caps.MaxActiveLights);
>> + /* AMD gives 6, Nvidia returns 8. */
>> + ok(caps.MaxUserClipPlanes <= 8,
>> + "MaxUserClipPlanes field has unexpected value %u.\n", caps.MaxUserClipPlanes);
>> + ok(caps.MaxVertexW == 0.0f || caps.MaxVertexW >= 1e10f,
>> + "MaxVertexW field has unexpected value %.8e.\n", caps.MaxVertexW);
>> +
>> + refcount = IDirect3DDevice8_Release(device);
>> + ok(!refcount, "Device has %u references left.\n", refcount);
>> }
> You're not printing the adapter index in the ok() messages here.
adapter index for every ok() messages in the loop?
More information about the wine-devel
mailing list