August 2021 - Wine-Devel - list.winehq.org

[PATCH 2/2] ws2_32: Add tests for exclusive flag for IOCTL_AFD_POLL.

by Guillaume Charifi

Signed-off-by: Guillaume Charifi <guillaume.charifi(a)sfr.fr> --- dlls/ws2_32/tests/afd.c | 73 ++++++++++++++++++++++++++++++++++++++++- 1 file changed, 72 insertions(+), 1 deletion(-) diff --git a/dlls/ws2_32/tests/afd.c b/dlls/ws2_32/tests/afd.c index eb7c8ee50de..2568f251b6f 100644 --- a/dlls/ws2_32/tests/afd.c +++ b/dlls/ws2_32/tests/afd.c @@ -148,8 +148,10 @@ static void test_poll(void) const struct sockaddr_in bind_addr = {.sin_family = AF_INET, .sin_addr.s_addr = htonl(INADDR_LOOPBACK)}; char in_buffer[offsetof(struct afd_poll_params, sockets[3])]; char out_buffer[offsetof(struct afd_poll_params, sockets[3])]; + char out_buffer1[offsetof(struct afd_poll_params, sockets[3])]; struct afd_poll_params *in_params = (struct afd_poll_params *)in_buffer; struct afd_poll_params *out_params = (struct afd_poll_params *)out_buffer; + struct afd_poll_params *out_params1 = (struct afd_poll_params *)out_buffer1; int large_buffer_size = 1024 * 1024; SOCKET client, server, listener; struct sockaddr_in addr; @@ -157,13 +159,14 @@ static void test_poll(void) IO_STATUS_BLOCK io; LARGE_INTEGER now; ULONG params_size; - HANDLE event; + HANDLE event, event1; int ret, len; large_buffer = malloc(large_buffer_size); memset(in_buffer, 0, sizeof(in_buffer)); memset(out_buffer, 0, sizeof(out_buffer)); event = CreateEventW(NULL, TRUE, FALSE, NULL); + event1 = CreateEventW(NULL, TRUE, FALSE, NULL); listener = socket(AF_INET, SOCK_STREAM, IPPROTO_TCP); ret = bind(listener, (const struct sockaddr *)&bind_addr, sizeof(bind_addr)); @@ -208,10 +211,77 @@ static void test_poll(void) IOCTL_AFD_POLL, in_params, params_size, out_params, params_size); ok(ret == STATUS_INVALID_PARAMETER, "got %#x\n", ret); + /* Test exclusive flag */ + + client = socket(AF_INET, SOCK_STREAM, IPPROTO_TCP); + + in_params->timeout = -1000 * 10000; + in_params->count = 1; + in_params->exclusive = FALSE; + in_params->sockets[0].socket = client; + in_params->sockets[0].flags = ~0; + in_params->sockets[0].status = 0; + + out_params->sockets[0].socket = 0; + out_params->sockets[0].flags = 0; + out_params->sockets[0].status = 0xdeadbeef; + + ret = NtDeviceIoControlFile((HANDLE)listener, event, NULL, NULL, &io, + IOCTL_AFD_POLL, in_params, params_size, out_params, params_size); + ok(ret == STATUS_PENDING, "got %#x\n", ret); + + in_params->timeout = 0x7fffffffffffffff; /* TIMEOUT_INFINITE */ + in_params->exclusive = TRUE; + + ret = NtDeviceIoControlFile((HANDLE)listener, event1, NULL, NULL, &io, + IOCTL_AFD_POLL, in_params, params_size, out_params1, params_size); + ok(ret == STATUS_PENDING, "got %#x\n", ret); + + ret = WaitForSingleObject(event, 100); + ok(ret == STATUS_TIMEOUT, "got %#x\n", ret); + ok(!out_params->sockets[0].socket, "got socket %#Ix\n", out_params->sockets[0].socket); + ok(!out_params->sockets[0].flags, "got flags %#x\n", out_params->sockets[0].flags); + ok(out_params->sockets[0].status == 0xdeadbeef, "got status %#x\n", out_params->sockets[0].status); + + closesocket(client); + + ret = WaitForSingleObject(event, 100); + ok(!ret, "got %#x\n", ret); + + client = socket(AF_INET, SOCK_STREAM, IPPROTO_TCP); + + out_params->sockets[0].socket = 0; + out_params->sockets[0].flags = 0; + out_params->sockets[0].status = 0xdeadbeef; + + ret = NtDeviceIoControlFile((HANDLE)listener, event, NULL, NULL, &io, + IOCTL_AFD_POLL, in_params, params_size, out_params, params_size); + ok(ret == STATUS_PENDING, "got %#x\n", ret); + + ret = NtDeviceIoControlFile((HANDLE)listener, event1, NULL, NULL, &io, + IOCTL_AFD_POLL, in_params, params_size, out_params1, params_size); + ok(ret == STATUS_PENDING, "got %#x\n", ret); + + ret = WaitForSingleObject(event, 100); + ok(!ret, "got %#x\n", ret); + ok(io.Status == STATUS_SUCCESS, "got %#x\n", io.Status); + ok(io.Information == offsetof(struct afd_poll_params, sockets[0]), "got %#Ix\n", io.Information); + ok(out_params->timeout == 0x7fffffffffffffff, "got timeout %#I64x\n", out_params->timeout); + ok(!out_params->count, "got count %u\n", out_params->count); + ok(!out_params->sockets[0].socket, "got socket %#Ix\n", out_params->sockets[0].socket); + ok(!out_params->sockets[0].flags, "got flags %#x\n", out_params->sockets[0].flags); + ok(out_params->sockets[0].status == 0xdeadbeef, "got status %#x\n", out_params->sockets[0].status); + + closesocket(client); + + ret = WaitForSingleObject(event1, 100); + ok(!ret, "got %#x\n", ret); + /* Basic semantics of the ioctl. */ in_params->timeout = 0; in_params->count = 1; + in_params->exclusive = FALSE; in_params->sockets[0].socket = listener; in_params->sockets[0].flags = ~0; in_params->sockets[0].status = 0xdeadbeef; @@ -743,6 +813,7 @@ static void test_poll(void) closesocket(server); CloseHandle(event); + CloseHandle(event1); free(large_buffer); } -- 2.33.0

Aug. 31, 2021

[PATCH 1/2 v2] ntdll/socket: Implement exclusive flag for IOCTL_AFD_POLL.

by Guillaume Charifi

Signed-off-by: Guillaume Charifi <guillaume.charifi(a)sfr.fr> --- dlls/ntdll/unix/socket.c | 6 ++--- include/wine/afd.h | 2 +- include/wine/server_protocol.h | 5 ++-- server/protocol.def | 1 + server/request.h | 1 + server/sock.c | 46 +++++++++++++++++++++++++++++++--- server/trace.c | 3 ++- 7 files changed, 54 insertions(+), 10 deletions(-) diff --git a/dlls/ntdll/unix/socket.c b/dlls/ntdll/unix/socket.c index 8469def786a..b1b90b363ee 100644 --- a/dlls/ntdll/unix/socket.c +++ b/dlls/ntdll/unix/socket.c @@ -747,12 +747,11 @@ static NTSTATUS sock_poll( HANDLE handle, HANDLE event, PIO_APC_ROUTINE apc, voi || in_size < offsetof( struct afd_poll_params, sockets[params->count] )) return STATUS_INVALID_PARAMETER; - TRACE( "timeout %s, count %u, unknown %#x, padding (%#x, %#x, %#x), sockets[0] {%04lx, %#x}\n", - wine_dbgstr_longlong(params->timeout), params->count, params->unknown, + TRACE( "timeout %s, count %u, exclusive %u, padding (%#x, %#x, %#x), sockets[0] {%04lx, %#x}\n", + wine_dbgstr_longlong(params->timeout), params->count, params->exclusive, params->padding[0], params->padding[1], params->padding[2], params->sockets[0].socket, params->sockets[0].flags ); - if (params->unknown) FIXME( "unknown boolean is %#x\n", params->unknown ); if (params->padding[0]) FIXME( "padding[0] is %#x\n", params->padding[0] ); if (params->padding[1]) FIXME( "padding[1] is %#x\n", params->padding[1] ); if (params->padding[2]) FIXME( "padding[2] is %#x\n", params->padding[2] ); @@ -793,6 +792,7 @@ static NTSTATUS sock_poll( HANDLE handle, HANDLE event, PIO_APC_ROUTINE apc, voi SERVER_START_REQ( poll_socket ) { req->async = server_async( handle, &async->io, event, apc, apc_user, iosb_client_ptr(io) ); + req->exclusive = !!params->exclusive; req->timeout = params->timeout; wine_server_add_data( req, input, params->count * sizeof(*input) ); wine_server_set_reply( req, async->sockets, params->count * sizeof(async->sockets[0]) ); diff --git a/include/wine/afd.h b/include/wine/afd.h index e67ecae25a9..f4682f464e8 100644 --- a/include/wine/afd.h +++ b/include/wine/afd.h @@ -104,7 +104,7 @@ struct afd_poll_params { LONGLONG timeout; unsigned int count; - BOOLEAN unknown; + BOOLEAN exclusive; BOOLEAN padding[3]; struct { diff --git a/include/wine/server_protocol.h b/include/wine/server_protocol.h index bb4862c9669..14518f0bf6f 100644 --- a/include/wine/server_protocol.h +++ b/include/wine/server_protocol.h @@ -1761,7 +1761,8 @@ struct poll_socket_output struct poll_socket_request { struct request_header __header; - char __pad_12[4]; + char exclusive; + char __pad_13[3]; async_data_t async; timeout_t timeout; /* VARARG(sockets,poll_socket_input); */ @@ -6252,7 +6253,7 @@ union generic_reply /* ### protocol_version begin ### */ -#define SERVER_PROTOCOL_VERSION 726 +#define SERVER_PROTOCOL_VERSION 727 /* ### protocol_version end ### */ diff --git a/server/protocol.def b/server/protocol.def index 133d6ad0552..8eefdaae17f 100644 --- a/server/protocol.def +++ b/server/protocol.def @@ -1450,6 +1450,7 @@ struct poll_socket_output /* Perform an async poll on a socket */ @REQ(poll_socket) + char exclusive; async_data_t async; /* async I/O parameters */ timeout_t timeout; /* timeout */ VARARG(sockets,poll_socket_input); /* list of sockets to poll */ diff --git a/server/request.h b/server/request.h index 18a6a2df3c7..f1c8f1c5432 100644 --- a/server/request.h +++ b/server/request.h @@ -1045,6 +1045,7 @@ C_ASSERT( sizeof(struct recv_socket_request) == 64 ); C_ASSERT( FIELD_OFFSET(struct recv_socket_reply, wait) == 8 ); C_ASSERT( FIELD_OFFSET(struct recv_socket_reply, options) == 12 ); C_ASSERT( sizeof(struct recv_socket_reply) == 16 ); +C_ASSERT( FIELD_OFFSET(struct poll_socket_request, exclusive) == 12 ); C_ASSERT( FIELD_OFFSET(struct poll_socket_request, async) == 16 ); C_ASSERT( FIELD_OFFSET(struct poll_socket_request, timeout) == 56 ); C_ASSERT( sizeof(struct poll_socket_request) == 64 ); diff --git a/server/sock.c b/server/sock.c index 50bfc08e145..cf4e1ac34b2 100644 --- a/server/sock.c +++ b/server/sock.c @@ -128,6 +128,7 @@ struct poll_req struct async *async; struct iosb *iosb; struct timeout_user *timeout; + char exclusive; unsigned int count; struct poll_socket_output *output; struct @@ -2853,8 +2854,8 @@ static int poll_single_socket( struct sock *sock, int mask ) return get_poll_flags( sock, pollfd.revents ) & mask; } -static int poll_socket( struct sock *poll_sock, struct async *async, timeout_t timeout, - unsigned int count, const struct poll_socket_input *input ) +static int poll_socket( struct sock *poll_sock, struct async *async, char exclusive, + timeout_t timeout, unsigned int count, const struct poll_socket_input *input ) { struct poll_socket_output *output; struct poll_req *req; @@ -2893,11 +2894,50 @@ static int poll_socket( struct sock *poll_sock, struct async *async, timeout_t t req->sockets[i].flags = input[i].flags; } + req->exclusive = exclusive; req->count = count; req->async = (struct async *)grab_object( async ); req->iosb = async_get_iosb( async ); req->output = output; + if (exclusive) + { + int has_shared = FALSE; + struct poll_req *areq; + + LIST_FOR_EACH_ENTRY( areq, &poll_list, struct poll_req, entry ) + { + if (!areq->exclusive) + { + has_shared = TRUE; + break; + } + } + + LIST_FOR_EACH_ENTRY( areq, &poll_list, struct poll_req, entry ) + { + if (has_shared) + break; + + for (i = 0; i < areq->count; ++i) + { + struct sock *asock = areq->sockets[i].sock; + + for (j = 0; j < req->count; ++j) + { + if (asock == req->sockets[j].sock) + { + areq->iosb->status = STATUS_SUCCESS; + areq->iosb->out_data = areq->output; + areq->iosb->out_size = areq->count * sizeof(*areq->output); + async_terminate( areq->async, STATUS_ALERTED ); + break; + } + } + } + } + } + list_add_tail( &poll_list, &req->entry ); async_set_completion_callback( async, free_poll_req, req ); queue_async( &poll_sock->poll_q, async ); @@ -3312,7 +3352,7 @@ DECL_HANDLER(poll_socket) if ((async = create_request_async( sock->fd, get_fd_comp_flags( sock->fd ), &req->async ))) { - reply->wait = async_handoff( async, poll_socket( sock, async, req->timeout, count, input ), NULL, 0 ); + reply->wait = async_handoff( async, poll_socket( sock, async, req->exclusive, req->timeout, count, input ), NULL, 0 ); reply->options = get_fd_options( sock->fd ); release_object( async ); } diff --git a/server/trace.c b/server/trace.c index 4e91d933a5d..567c54c4df7 100644 --- a/server/trace.c +++ b/server/trace.c @@ -2122,7 +2122,8 @@ static void dump_recv_socket_reply( const struct recv_socket_reply *req ) static void dump_poll_socket_request( const struct poll_socket_request *req ) { - dump_async_data( " async=", &req->async ); + fprintf( stderr, " exclusive=%c", req->exclusive ); + dump_async_data( ", async=", &req->async ); dump_timeout( ", timeout=", &req->timeout ); dump_varargs_poll_socket_input( ", sockets=", cur_size ); } -- 2.33.0

Aug. 31, 2021

Re: [PATCH vkd3d 3/5] vkd3d-shader/hlsl: Write SM4 negation instructions.

by Matteo Bruni

On Mon, Aug 30, 2021 at 7:21 PM Zebediah Figura <zfigura(a)codeweavers.com> wrote: > > Signed-off-by: Zebediah Figura <zfigura(a)codeweavers.com> > --- > Makefile.am | 5 +---- > libs/vkd3d-shader/hlsl_sm4.c | 32 ++++++++++++++++++++++++++++++++ > 2 files changed, 33 insertions(+), 4 deletions(-) > diff --git a/libs/vkd3d-shader/hlsl_sm4.c b/libs/vkd3d-shader/hlsl_sm4.c > index ace2eb66..160da47f 100644 > --- a/libs/vkd3d-shader/hlsl_sm4.c > +++ b/libs/vkd3d-shader/hlsl_sm4.c > @@ -561,6 +561,7 @@ struct sm4_register > unsigned int idx_count; > enum vkd3d_sm4_dimension dim; > uint32_t immconst_uint[4]; > + unsigned int mod; > }; > > struct sm4_instruction > @@ -710,6 +711,8 @@ static uint32_t sm4_register_order(const struct sm4_register *reg) > if (reg->type == VKD3D_SM4_RT_IMMCONST) > order += reg->dim == VKD3D_SM4_DIMENSION_VEC4 ? 4 : 1; > order += reg->idx_count; > + if (reg->mod) > + ++order; > return order; > } Something that came to my mind at least twice while looking at that function... Do you know e.g. by random looks to DXC's sources what the size of a register in the bytecode is supposed to be called? "Order" feels like a curious choice for a name. No need to go investigate, just wondering if you happened to see anything.

Aug. 31, 2021

Re: [PATCH vkd3d 5/5] vkd3d-shader/hlsl: Use the modifiers location when validating modifiers on casts.

by Matteo Bruni

Signed-off-by: Matteo Bruni <mbruni(a)codeweavers.com>

Aug. 31, 2021

Re: [PATCH vkd3d 4/5] vkd3d-shader/hlsl: Merge the "type" and "base_type" rules.

by Matteo Bruni

Signed-off-by: Matteo Bruni <mbruni(a)codeweavers.com> --- I guess the original choice was mostly a "defensive" one: it's generally better to start with a restrictive grammar and then remove constraints as necessary rather than the other way around.

Aug. 31, 2021

Re: [PATCH vkd3d 3/5] vkd3d-shader/hlsl: Write SM4 negation instructions.

by Matteo Bruni

Signed-off-by: Matteo Bruni <mbruni(a)codeweavers.com>

Aug. 31, 2021

Re: [PATCH vkd3d 2/5] vkd3d-shader/hlsl: Write SM4 division instructions.

by Matteo Bruni

Signed-off-by: Matteo Bruni <mbruni(a)codeweavers.com>

Aug. 31, 2021

Re: [PATCH vkd3d 1/5] vkd3d-shader/hlsl: Write SM4 multiplication instructions.

by Matteo Bruni

Signed-off-by: Matteo Bruni <mbruni(a)codeweavers.com>

Aug. 31, 2021

Re: [PATCH 0/5] winemac.drv: Layer-backed views

by Chip Davis

Ugh. Don't know how the patchset summary was lost. It's supposed to say something to the effect of: This patchset enables layer-backed views for the macOS graphics driver, and sets the drawn bits directly into the layer. We are able to do this now, because the minimum version is now 10.8. It is my hope that this will ease the implementation of full Retina support as well as drawing to other processes' windows. August 31, 2021 6:03 PM, "Chip Davis" <cdavis(a)codeweavers.com> wrote: > Chip

Aug. 31, 2021

Re: [PATCH 4/5] d3d10/effect: Support stream output declaration when creating geometry shaders.

by Matteo Bruni

On Tue, Aug 31, 2021 at 3:57 PM Nikolay Sivov <nsivov(a)codeweavers.com> wrote: > > > > On 8/31/21 4:32 PM, Matteo Bruni wrote: > > On Mon, Aug 30, 2021 at 7:07 AM Nikolay Sivov <nsivov(a)codeweavers.com> wrote: > >> Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> > >> --- > >> dlls/d3d10/effect.c | 172 ++++++++++++++++++++++++++++++++++++++++++-- > >> 1 file changed, 166 insertions(+), 6 deletions(-) > > Patch is okay, a few notes below. > > > >> diff --git a/dlls/d3d10/effect.c b/dlls/d3d10/effect.c > >> index 113125e9cbd..c3c98a0da31 100644 > >> --- a/dlls/d3d10/effect.c > >> +++ b/dlls/d3d10/effect.c > >> @@ -566,6 +592,127 @@ static HRESULT get_fx10_shader_resources(struct d3d10_effect_variable *v, const > >> return S_OK; > >> } > >> > >> +struct d3d10_effect_so_decl > >> +{ > >> + D3D10_SO_DECLARATION_ENTRY *entries; > >> + SIZE_T capacity; > >> + SIZE_T count; > >> + unsigned int stride; > >> + char *decl; > >> +}; > >> + > >> +static void d3d10_effect_cleanup_so_decl(struct d3d10_effect_so_decl *so_decl) > >> +{ > >> + heap_free(so_decl->entries); > >> + heap_free(so_decl->decl); > >> + memset(so_decl, 0, sizeof(*so_decl)); > >> +} > >> + > >> +static HRESULT d3d10_effect_parse_stream_output_declaration(const char *decl, > >> + struct d3d10_effect_so_decl *so_decl) > >> +{ > >> + static const char * allmask = "xyzw"; > > Usually in HLSL you can use "rgba" in place of "xyzw" for swizzles / > > writemasks, it might be worth checking if those also work (and are > > passed through to the effect bytecode unchanged). > Yes, "rgba" also works apparently. Compiler preserves string as is, does > not convert to coordinate mask names. I'll add a test for that. > > > >> + char *p, *ptr, *end, *next, *mask, *m, *slot; > >> + unsigned int len = strlen(decl); > >> + D3D10_SO_DECLARATION_ENTRY e; > >> + > >> + memset(so_decl, 0, sizeof(*so_decl)); > >> + > >> + if (!(so_decl->decl = heap_alloc(len + 1))) > >> + return E_OUTOFMEMORY; > >> + memcpy(so_decl->decl, decl, len + 1); > >> + > >> + p = so_decl->decl; > > I think it would be nice to have a small comment showing the expected > > syntax of a stream out declaration (entry) somewhere around here. Just > > something to quickly glance at while reading the following code. > Okay. > > > >> + > >> + while (p && *p) > >> + { > >> + memset(&e, 0, sizeof(e)); > >> + > >> + end = strchr(p, ';'); > >> + next = end ? end + 1 : p + strlen(p); > >> + > >> + len = next - p; > >> + if (end) len--; > >> + > >> + /* Remove leading and trailing spaces. */ > >> + while (len && isspace(*p)) { len--; p++; } > >> + while (len && isspace(p[len - 1])) len--; > >> + > >> + p[len] = 0; > >> + > >> + /* Output slot */ > >> + if ((slot = strchr(p, ':'))) > >> + { > >> + *slot = 0; > >> + > >> + ptr = p; > >> + while (*ptr) > >> + { > >> + if (!isdigit(*ptr)) > >> + { > >> + WARN("Invalid output slot %s.\n", debugstr_a(p)); > >> + goto failed; > >> + } > >> + ptr++; > >> + } > >> + > >> + e.OutputSlot = atoi(p); > >> + p = slot + 1; > >> + } > >> + > >> + /* Mask */ > >> + if ((mask = strchr(p, '.'))) > >> + { > >> + *mask = 0; mask++; > >> + > >> + if (!(m = strstr(allmask, mask))) > >> + { > >> + WARN("Invalid component mask %s.\n", debugstr_a(mask)); > >> + goto failed; > >> + } > >> + > >> + e.StartComponent = m - allmask; > >> + e.ComponentCount = strlen(mask); > > Did you verify that it's illegal to have a wrongly ordered mask (e.g. > > something like .wzyx)? > > I expect that to be invalid but better to be sure. > > Likewise for other assumptions involved here, like repeating the same > > component twice. > Yes, it is disallowed to break the order. API manifestation is that you > specify start and count, which assumes it's always forward looking. > Regarding error, it won't pass compiler check first, complaining about > "invalid mask declaration". As far as I can tell that's the only thing > that compiler checks in this string, you can have non-existent semantics > as much as you like. That will break effect creation later though, > presumably because shader object can't be created with invalid declaration. Excellent. That should already do the right thing then. > > > > In general, some more tests with interesting stream out declarations > > would be nice. I guess you have more planned after the test in patch > > 5/5. > I can think of adding more variations with spaces around, and maybe with > multiple entries. Do you have other ideas? Yes, that sounds good. I guess generally trying to exercise all the edge cases of the parser. I just noticed that there is probably one case that needs special handling. From the MS docs: 'There is one special Semantic, labeled "$SKIP" which indicates an empty semantic, leaving the corresponding memory in the stream out buffer untouched. The $SKIP semantic cannot have a SemanticIndex, but can have a Mask.' It should probably be translated into a declaration entry with NULL semantic name. See test_stream_output() in d3d10core/tests/d3d10core.c for a few examples (both valid and not) if you haven't yet. > >> + } > >> + else > >> + { > >> + e.StartComponent = 0; > >> + e.ComponentCount = 4; > >> + } > >> + > >> + /* Semantic index and name */ > >> + len = strlen(p); > >> + while (isdigit(p[len - 1])) > >> + len--; > >> + > >> + if (p[len]) > >> + { > >> + e.SemanticIndex = atoi(&p[len]); > >> + p[len] = 0; > >> + } > >> + > >> + e.SemanticName = p; > >> + > >> + if (!d3d_array_reserve((void **)&so_decl->entries, &so_decl->capacity, so_decl->count + 1, > >> + sizeof(*so_decl->entries))) > >> + goto failed; > >> + > >> + so_decl->entries[so_decl->count++] = e; > >> + > >> + if (e.OutputSlot == 0) > >> + so_decl->stride += e.ComponentCount * sizeof(float); > > This looks weird, but that's probably because the API for > > ID3D10Device_CreateGeometryShaderWithStreamOutput() is confusing (just > > one stride for up to 4 output buffers? What does it even mean?) and > > neither the documentation nor our tests seem to clarify that. Any clue > > what's supposed to happen with multiple output buffers? > > > > FWIW that function's signature makes a lot more sense in d3d11. > My understanding is that in d3d10 you can specify stride only when you > use one SO buffer. And if you use multiple buffers runtime will use > default strides for all. For d3d11 you can set each explicitly. Somehow I missed the relevant tests in d3d10core.c. It is still weird but I guess not as mysterious now. After playing with this a bit, it looks like it's actually more strict than that: if there is just one buffer you need to pass a non-zero stride; OTOH when there are multiple buffers you need to pass 0 as stride. Please double check that it's the case (as I still don't trust myself a lot on this) and consider sending a fix if necessary.

Aug. 31, 2021