[LTP] [PATCH v3 1/4] Hugetlb: Add new tst_test options for hugeltb test support

Cyril Hrubis chrubis@suse.cz
Mon Oct 31 15:49:24 CET 2022


Hi!
> > @@ -357,6 +369,12 @@ unsigned int tst_remaining_runtime(void);
> >   */
> >  void tst_set_max_runtime(int max_runtime);
> >
> > +/*
> > + * Create and open a random file inside the given dir path.
> > + * It unlinks the file after opening and return file descriptor.
> > + */
> > +int tst_create_unlinked_file(const char *path);
> >
> 
> what about renaming this function to tst_'get|create'_unlinked_file_fd?
> I guess the 'fd' part should be emphasized here.

It has create in name and in UNIX creat() creates file and returns a
file descriptor so I think it's fine. Maybe we can be even shorted with
something as tst_creat_unlinked(const char *path).

-- 
Cyril Hrubis
chrubis@suse.cz


More information about the ltp mailing list