Re: [PATCH 2/5] d3d8/tests: Test device caps on multiple adapters.

[Henri Verbeet]

On Fri, 13 Mar 2020 at 13:06, Zhiyi Zhang  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.
> - 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.