LU-6142 tests: Fix style issues under lustre/tests This patch fixes issues reported by checkpatch for all files under folder lustre/tests Signed-off-by: Arshad Hussain <arshad.hussain@aeoncomputing.com> Change-Id: I93f18f737c219222593b9689cd3c1b5eaba7630d Reviewed-on: https://review.whamcloud.com/c/fs/lustre-release/+/54110 Tested-by: jenkins <devops@whamcloud.com> Tested-by: Maloo <maloo@whamcloud.com> Reviewed-by: Timothy Day <timday@amazon.com> Reviewed-by: Andreas Dilger <adilger@whamcloud.com> Reviewed-by: Oleg Drokin <green@whamcloud.com>
LU-8585 llite: add special fid handling for fhandle API Lustre has been moving its FIDs handling to the fhandle API. This works well for normal files but Lustre has special FIDs that don't map to normal files which are used by user land applications. Add special handling to ll_iget_for_nfs() so the fhandle API can work with these special FIDs. These FIDs should also work with filesets. Change-Id: I4b55d96cc9eea0b1fb898f94c071c8b30c7b2bd5 Signed-off-by: James Simmons <jsimmons@infradead.org> Reviewed-on: https://review.whamcloud.com/c/fs/lustre-release/+/51707 Tested-by: jenkins <devops@whamcloud.com> Tested-by: Maloo <maloo@whamcloud.com> Reviewed-by: Andreas Dilger <adilger@whamcloud.com> Reviewed-by: Etienne AUJAMES <eaujames@ddn.com> Reviewed-by: Neil Brown <neilb@suse.de> Reviewed-by: Oleg Drokin <green@whamcloud.com>
LU-14398 llapi: add llapi_fid2path_at() Add llapi_fid2path_at() which works like llapi_fid2path() takes an open FD on the moint point instead of a 'fsname or dirirectory path' and a const struct lu_fid * instead of a const char *. Signed-off-by: John L. Hammond <jhammond@whamcloud.com> Change-Id: I76234bc28de231587b65c5d866954441e0893aac Reviewed-on: https://review.whamcloud.com/41406 Tested-by: jenkins <devops@whamcloud.com> Reviewed-by: Andreas Dilger <adilger@whamcloud.com> Tested-by: Maloo <maloo@whamcloud.com> Reviewed-by: Alex Zhuravlev <bzzz@whamcloud.com> Reviewed-by: James Simmons <jsimmons@infradead.org> Reviewed-by: Oleg Drokin <green@whamcloud.com>
LU-11233 tests: fix gcc8 build warnings this patch covers Lustre tests Signed-off-by: Alex Zhuravlev <bzzz@whamcloud.com> Change-Id: I6345d603772fb32bbc4b38a758a3e97f0361d116 Reviewed-on: https://review.whamcloud.com/34661 Tested-by: Jenkins Reviewed-by: Jian Yu <yujian@whamcloud.com> Reviewed-by: Li Dongyang <dongyangli@ddn.com> Reviewed-by: James Simmons <uja.ornl@yahoo.com> Tested-by: Maloo <maloo@whamcloud.com> Reviewed-by: Oleg Drokin <green@whamcloud.com>
LU-10308 misc: update Intel copyright messages for 2017 Update copyright messages for files updated in 2016, excluding trivial patches. Add trivial patches to updatecw.sh script exclude list. Revert some changes that were incorrectly attributed to the 2016 (d10200a80770f0029d1d665af954187b9ad883df) and 2015 (0754bc8f2623bea184111af216f7567608db35b6) copyright update patches themselves, since they were not in the exclude list when the subsequent script was run. Test-Parameters: trivial Signed-off-by: Andreas Dilger <andreas.dilger@intel.com> Change-Id: I82f21c30c4dac75792bb49fc139bee2ca51f5545 Reviewed-on: https://review.whamcloud.com/30341 Tested-by: Jenkins Tested-by: Maloo <hpdd-maloo@intel.com> Reviewed-by: Jian Yu <jian.yu@intel.com> Reviewed-by: James Nunez <james.a.nunez@intel.com> Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
LU-6142 uapi: Get rid of lustre_fid typedef Replace it with struct lu_fid. Update the userland code and man pages to reflect this change. Linux-commit: d8f6bc9a53f97d1ea4b2b955672904338643308b Test-Parameters: trivial Change-Id: I0b7e0770dd9da9bdac55c02c2ec98aea7cea7100 Signed-off-by: James Simmons <uja.ornl@yahoo.com> Signed-off-by: Oleg Drokin <oleg.drokin@intel.com> Reviewed-on: https://review.whamcloud.com/29849 Tested-by: Jenkins Tested-by: Maloo <hpdd-maloo@intel.com> Reviewed-by: Andreas Dilger <andreas.dilger@intel.com> Reviewed-by: Ben Evans <bevans@cray.com> Reviewed-by: John L. Hammond <john.hammond@intel.com>
LU-6401 uapi: migrate remaining uapi headers to uapi directory Move all the remaining lustre headers shared between user land and kernel space to the uapi directory. OBD_MAX_IOCTL_BUFFER can be removed since the utilites don't even acknowledge it value. Change-Id: I6dcdf3f824dae6bc8a0786f0e61787ca68c10bd8 Signed-off-by: James Simmons <uja.ornl@yahoo.com> Reviewed-on: https://review.whamcloud.com/25246 Tested-by: Jenkins Reviewed-by: Quentin Bouget <quentin.bouget@cea.fr> Tested-by: Maloo <hpdd-maloo@intel.com> Reviewed-by: Ben Evans <bevans@cray.com> Reviewed-by: Dmitry Eremin <dmitry.eremin@intel.com> Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
LU-8847 tests: llapi_fid_test uses incorrect parameters Several function calls in test12 were providing incorrect parameters. The value "0600" was provided, in an attempt to create a file with read/write access for the owner. However, the parameters actually represent the open flags, so the incorrect parameters were instead requesting (O_EXCL | O_NOCTTY). Since llapi_create_volatile_idx by default already uses (O_RDWR | O_CREAT) and a mask of (S_IRUSR | S_IWUSR), there are no further flags required. For llapi_open_by_fid, only the minimum O_RDONLY is required (the call should not attempt to create the file). Test-Parameters: trivial Signed-off-by: Steve Guminski <stephenx.guminski@intel.com> Change-Id: I5a1c9a014ab19a1a6a1901e2a8a091d4d65548f6 Reviewed-on: https://review.whamcloud.com/23939 Tested-by: Jenkins Tested-by: Maloo <hpdd-maloo@intel.com> Reviewed-by: Jian Yu <jian.yu@intel.com> Reviewed-by: Dmitry Eremin <dmitry.eremin@intel.com> Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
LU-8648 all: remove all Sun license and URL references The link to the GPL license at Sun is no longer there. Also Sun Inc is gone so lets remove all references to it in our source code. Test-Parameters: trivial Change-Id: Ib47d07dcef49a1ef624d0d8a4a91b9731f486278 Signed-off-by: James Simmons <uja.ornl@yahoo.com> Reviewed-on: http://review.whamcloud.com/22800 Tested-by: Jenkins Tested-by: Maloo <hpdd-maloo@intel.com> Reviewed-by: Andreas Dilger <andreas.dilger@intel.com> Reviewed-by: John L. Hammond <john.hammond@intel.com> Reviewed-by: Frank Zago <fzago@cray.com>
LU-6245 libcfs: remove userland headers from libcfs.h Currently libcfs.h is used as a master header that contains all the needed headers. Since Lustre user land utilities and applications no longer have a strong dependency on libcfs.h we can remove all the added user land headers contained in libcfs.h. Signed-off-by: James Simmons <uja.ornl@yahoo.com> Change-Id: I6403d109875a1d42d8490a3a1c7635f2dac9fc90 Reviewed-on: http://review.whamcloud.com/16914 Tested-by: Jenkins Tested-by: Maloo <hpdd-maloo@intel.com> Reviewed-by: Dmitry Eremin <dmitry.eremin@intel.com> Reviewed-by: John L. Hammond <john.hammond@intel.com> Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
LU-6406 tests: fix undersized fsname and FID strings A Lustre filesystem name is up to 8 characters, plus 1 for the NUL terminator. If a filesystem name is 8 characters, then the llapi_layout_test, llapi_hsm_test, group_lock_test and llapi_fid_test may have their stack scribbled or overran when they call llapi_search_mounts(). Add one extra byte for the NUL character. Same issue with a FID string, which also needs an extra character for NUL termination. Signed-off-by: frank zago <fzago@cray.com> Change-Id: Ic7757ebce051e12f4723e27b5b00d4db1637b384 Reviewed-on: http://review.whamcloud.com/14213 Tested-by: Jenkins Reviewed-by: Dmitry Eremin <dmitry.eremin@intel.com> Tested-by: Maloo <hpdd-maloo@intel.com> Reviewed-by: John L. Hammond <john.hammond@intel.com> Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
LU-6394 all: fix compilation errors with FORTIFY_SOURCE When Lustre is configured with CFLAGS="-D_FORTIFY_SOURCE=2 -O2" on Centos 6, the compilation will fails with errors such as this one: cacheio.c: In function ‘qword_printhex’: cacheio.c:174: error: ignoring return value of ‘fwrite’, declared with attribute warn_unused_result Signed-off-by: frank zago <fzago@cray.com> Change-Id: Ie06fd5a26b62daf62bfd0133a2d7ebc66ece5be6 Reviewed-on: http://review.whamcloud.com/14126 Tested-by: Jenkins Tested-by: Maloo <hpdd-maloo@intel.com> Reviewed-by: Dmitry Eremin <dmitry.eremin@intel.com> Reviewed-by: James Simmons <uja.ornl@gmail.com> Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
LU-4239 tests: test FID related APIs This adds a few stress tests to the user lustre API, related to FIDs. Change-Id: I34144a8f4c446e55c6630d31cae6a133d61eb304 Signed-off-by: Frank Zago <fzago@cray.com> Test-Parameters: alwaysuploadlogs envdefinitions=ONLY=154g testlist=sanity Reviewed-on: http://review.whamcloud.com/12545 Tested-by: Jenkins Tested-by: Maloo <hpdd-maloo@intel.com> Reviewed-by: John L. Hammond <john.hammond@intel.com> Reviewed-by: Jian Yu <jian.yu@intel.com> Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>