[PATCH] wined3d: Increase the CS queue size.

[Jan Sikorski]

Final Fantasy XIV routinely runs out of space in the queue, which
affects the frame rates.

Signed-off-by: Jan Sikorski <jsikorski(a)codeweavers.com>
---
On my particular testing set up the change is from ~85 fps to ~100 fps.
---
 dlls/wined3d/wined3d_private.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/dlls/wined3d/wined3d_private.h b/dlls/wined3d/wined3d_private.h
index 5b3eb0136b0..d53eb3985fa 100644
--- a/dlls/wined3d/wined3d_private.h
+++ b/dlls/wined3d/wined3d_private.h
@@ -4892,7 +4892,7 @@ enum wined3d_push_constants
 };
 
 #define WINED3D_CS_QUERY_POLL_INTERVAL  10u
-#define WINED3D_CS_QUEUE_SIZE           0x100000u
+#define WINED3D_CS_QUEUE_SIZE           0x400000u
 #define WINED3D_CS_SPIN_COUNT           10000000u
 #define WINED3D_CS_QUEUE_MASK           (WINED3D_CS_QUEUE_SIZE - 1)
 
-- 
2.32.0

[Matteo Bruni]

On Wed, Mar 30, 2022 at 12:43 PM Jan Sikorski <jsikorski(a)codeweavers.com> wrote:
> Final Fantasy XIV routinely runs out of space in the queue, which
> affects the frame rates.
>
> Signed-off-by: Jan Sikorski <jsikorski(a)codeweavers.com>
> ---
> On my particular testing set up the change is from ~85 fps to ~100 fps.
> ---
>  dlls/wined3d/wined3d_private.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/dlls/wined3d/wined3d_private.h b/dlls/wined3d/wined3d_private.h
> index 5b3eb0136b0..d53eb3985fa 100644
> --- a/dlls/wined3d/wined3d_private.h
> +++ b/dlls/wined3d/wined3d_private.h
> @@ -4892,7 +4892,7 @@ enum wined3d_push_constants
>  };
>
> #define WINED3D_CS_QUERY_POLL_INTERVAL  10u
> -#define WINED3D_CS_QUEUE_SIZE           0x100000u
> +#define WINED3D_CS_QUEUE_SIZE           0x400000u
>  #define WINED3D_CS_SPIN_COUNT           10000000u
>  #define WINED3D_CS_QUEUE_MASK           (WINED3D_CS_QUEUE_SIZE - 1)
>
> --
> 2.32.0

It might be sensible to use different queue sizes for 32 and 64 bits
(i.e. I don't think increasing the queue size for 64-bit only is going
to encounter any resistance).

[Matteo Bruni]

Signed-off-by: Matteo Bruni <mbruni(a)codeweavers.com>
---
Mostly to show that it's also okay to me to increase the size
unconditionally (I wasn't trying to block the current patch).

[Henri Verbeet]

Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com>