[PATCH] Add test case for O_TMPFILE handling in open, openat
Carlos O'Donell
carlos@redhat.com
Tue Sep 20 19:26:00 GMT 2016
On 09/20/2016 02:51 PM, Florian Weimer wrote:
> I noticed that we do not seem to have a test case for O_TMPFILE
> handling.
>
> I have tested the attached patch with recent kernels and Red Hat's
> 3.10 kernels (where it prints UNSUPPORTED). I verified that
> probe_path works as intended with a vfat file system (which does not
> support O_TMPFILE even on a supported kernel).
The actual details of the patch look good to me, so OK conditional
on more comments about:
* What is being tested.
* Is it sufficient?
* If not, where might we improve in the future?
* Under what conditions do we mark as UNSUPPORTED?
The test cases should have lots of comments explaining the what and
why of the various tests we conduct. Sorry for sounding like a broken
record about documenting tests, but future reviewers of the test will
thank you for it.
--
Cheers,
Carlos.
More information about the Libc-alpha
mailing list