From mboxrd@z Thu Jan 1 00:00:00 1970 From: Petr Vorel Date: Thu, 13 Dec 2018 17:37:11 +0100 Subject: [LTP] [PATCH v4] syscalls/statx: Add test for sync flags In-Reply-To: <3bc62fa106ebc6031105d2cfca26b7cd.squirrel@pike.zilogic.com> References: <1544172141-1676-1-git-send-email-vaishnavi.d@zilogic.com> <20181207155220.GA14562@dell5510> <3bc62fa106ebc6031105d2cfca26b7cd.squirrel@pike.zilogic.com> Message-ID: <20181213163711.GC18491@dell5510> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: ltp@lists.linux.it Hi Vaishnavi, > Actually, support for sync flags was added only from kernel version 4.16. > Since, I was using 4.16, I didn't face that issue. Thats why minimum kernel > version has been updated to 4.16 in the patch v5. > (https://github.com/torvalds/linux/commit/9ccee940bd5b766b6dab6c1a80908b9490a4850d) Thanks for update. > >> +#include "old_tmpdir.h" > > I think, we don't want to use old_*.h headers (although library itself > > lib/tst_test.c uses it). IMHO previous version with getcwd() was better. > The function get tempdir will be ported to the new library soon. So, I > have not modified that part. Other changes have been made in patch v5. Good, I noticed Cyril's intention to fix the headers. We'll just wait for it and remove import old_tmpdir.h before merging v5. > Regards, > Vaishnavi D Kind regards, Petr