2022-03-22 6:48 GMT+09:00, Paulo Alcantara <pc@xxxxxx>: > Tom Talpey <tom@xxxxxxxxxx> writes: > >> I agree this is the most practical way to address the issue, and >> any potential static checker warnings. >> >> Have you tested on both endian clients I assume? Might be tricky >> to catch all the ops/cases, especially that cancel. > > I did some quick tests on x86_64 and s390x under QEMU against Windows > 2022 server and ksmbd and all seemed to work. Hopefully Steve & Namjae > can also help with more tests. > >> Reviewed-By: Tom Talpey <tom@xxxxxxxxxx> ^ Minor nit, B must be lowercase. This will cause a warning from checkpatch.pl. And looks good to me too, Reviewed-by: Namjae Jeon <linkinjeon@xxxxxxxxxx> Thanks:) > > Thanks! >