Re: [PULL 1/6] linux-user/sparc: Don't use 16-bit UIDs on SPARC V9

[Laurent Vivier]

Le 12/05/2023 à 14:08, John Paul Adrian Glaubitz a écrit :
> Hello Laurent!
>
> On Fri, 2023-05-12 at 13:13 +0200, Laurent Vivier wrote:
> > This patch breaks something with LTP (20230127) test fchown05_16 on sid/sparc64:
> >
> > tst_test.c:1558: TINFO: Timeout per run is 0h 00m 30s
> > fchown05.c:44: TPASS: fchown(3, 700, 701), change owner/group ids passed
> > fchown05.c:44: TPASS: fchown(3, 702, -1), change owner id only passed
> > fchown05.c:49: TFAIL: testfile: incorrect ownership set, expected 702 701
> > fchown05.c:44: TPASS: fchown(3, 703, 701), change owner id only passed
> > fchown05.c:44: TPASS: fchown(3, -1, 704), change group id only passed
> > fchown05.c:49: TFAIL: testfile: incorrect ownership set, expected 703 704
> > fchown05.c:44: TPASS: fchown(3, 703, 705), change group id only passed
> > fchown05.c:44: TPASS: fchown(3, -1, -1), no change passed
> > fchown05.c:49: TFAIL: testfile: incorrect ownership set, expected 703 705
> >
> > expected result;
> >
> > tst_test.c:1558: TINFO: Timeout per run is 0h 00m 30s
> > fchown05.c:44: TPASS: fchown(3, 700, 701), change owner/group ids passed
> > fchown05.c:44: TPASS: fchown(3, 702, -1), change owner id only passed
> > fchown05.c:44: TPASS: fchown(3, 703, 701), change owner id only passed
> > fchown05.c:44: TPASS: fchown(3, -1, 704), change group id only passed
> > fchown05.c:44: TPASS: fchown(3, 703, 705), change group id only passed
> > fchown05.c:44: TPASS: fchown(3, -1, -1), no change passed
>
> Where do these tests reside? I'm not sure I know what LTP is. In any case,
> the patch should be correct as QEMU needs to differentiate between 32-bit
> and 64-bit SPARC.

I agree, it could be a side effect. I didn't check.

https://github.com/linux-test-project/ltp/releases/tag/20230127

Thanks,
Laurent