| Message ID | 20240924-selftests-gitignore-v1-4-9755ac883388@gmail.com (mailing list archive) |
|---|---|
| State | New |
| Headers | show |
| Series | selftsets: gitignore and clean target file additions | expand |
Hi Javier,
kernel test robot noticed the following build warnings:
[auto build test WARNING on 4d0326b60bb753627437fff0f76bf1525bcda422]
url: https://github.com/intel-lab-lkp/linux/commits/Javier-Carrasco/selftests-add-unshare_test-and-msg_oob-to-gitignore/20240924-205133
base: 4d0326b60bb753627437fff0f76bf1525bcda422
patch link: https://lore.kernel.org/r/20240924-selftests-gitignore-v1-4-9755ac883388%40gmail.com
patch subject: [PATCH 4/4] selftests: exec: update gitignore for load_address
config: x86_64-allnoconfig (https://download.01.org/0day-ci/archive/20240925/202409251828.dMx8LnrF-lkp@intel.com/config)
compiler: clang version 18.1.8 (https://github.com/llvm/llvm-project 3b5b5c1ec4a3095ab096dd780e84d7ab81f3d7ff)
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20240925/202409251828.dMx8LnrF-lkp@intel.com/reproduce)
If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202409251828.dMx8LnrF-lkp@intel.com/
All warnings (new ones prefixed by >>):
tools/testing/selftests/arm64/tags/.gitignore: warning: ignored by one of the .gitignore files
tools/testing/selftests/arm64/tags/Makefile: warning: ignored by one of the .gitignore files
tools/testing/selftests/arm64/tags/tags_test.c: warning: ignored by one of the .gitignore files
>> tools/testing/selftests/exec/load_address.c: warning: ignored by one of the .gitignore files
On 9/24/24 06:49, Javier Carrasco wrote: > The name of the "load_address" objects has been modified, but the > corresponding entry in the gitignore file must be updated. > > Update the load_address entry in the gitignore file to account for > the new names. > > Signed-off-by: Javier Carrasco <javier.carrasco.cruz@gmail.com> > --- > tools/testing/selftests/exec/.gitignore | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/tools/testing/selftests/exec/.gitignore b/tools/testing/selftests/exec/.gitignore > index 90c238ba6a4b..4d9fb7b20ea7 100644 > --- a/tools/testing/selftests/exec/.gitignore > +++ b/tools/testing/selftests/exec/.gitignore > @@ -9,7 +9,7 @@ execveat.ephemeral > execveat.denatured > non-regular > null-argv > -/load_address_* > +/load_address.* Hmm. This will include the load_address.c which shouldn't be included in the .gitignore? > /recursion-depth > xxxxxxxx* > pipe > thanks, -- Shuah
On 25/09/2024 17:46, Shuah Khan wrote: > On 9/24/24 06:49, Javier Carrasco wrote: >> The name of the "load_address" objects has been modified, but the >> corresponding entry in the gitignore file must be updated. >> >> Update the load_address entry in the gitignore file to account for >> the new names. >> >> Signed-off-by: Javier Carrasco <javier.carrasco.cruz@gmail.com> >> --- >> tools/testing/selftests/exec/.gitignore | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/tools/testing/selftests/exec/.gitignore b/tools/testing/ >> selftests/exec/.gitignore >> index 90c238ba6a4b..4d9fb7b20ea7 100644 >> --- a/tools/testing/selftests/exec/.gitignore >> +++ b/tools/testing/selftests/exec/.gitignore >> @@ -9,7 +9,7 @@ execveat.ephemeral >> execveat.denatured >> non-regular >> null-argv >> -/load_address_* >> +/load_address.* > > Hmm. This will include the load_address.c which shouldn't > be included in the .gitignore? > >> /recursion-depth >> xxxxxxxx* >> pipe >> > > thanks, > -- Shuah Hi, the kernel test robot already notified me about that issue, and I sent a v2 to fix it shortly after. Please take a look at the newer version where I added the exception for load_address.c. Thanks and best regards, Javier Carrasco
On 9/25/24 10:25, Javier Carrasco wrote: > On 25/09/2024 17:46, Shuah Khan wrote: >> On 9/24/24 06:49, Javier Carrasco wrote: >>> The name of the "load_address" objects has been modified, but the >>> corresponding entry in the gitignore file must be updated. >>> >>> Update the load_address entry in the gitignore file to account for >>> the new names. >>> >>> Signed-off-by: Javier Carrasco <javier.carrasco.cruz@gmail.com> >>> --- >>> tools/testing/selftests/exec/.gitignore | 2 +- >>> 1 file changed, 1 insertion(+), 1 deletion(-) >>> >>> diff --git a/tools/testing/selftests/exec/.gitignore b/tools/testing/ >>> selftests/exec/.gitignore >>> index 90c238ba6a4b..4d9fb7b20ea7 100644 >>> --- a/tools/testing/selftests/exec/.gitignore >>> +++ b/tools/testing/selftests/exec/.gitignore >>> @@ -9,7 +9,7 @@ execveat.ephemeral >>> execveat.denatured >>> non-regular >>> null-argv >>> -/load_address_* >>> +/load_address.* >> >> Hmm. This will include the load_address.c which shouldn't >> be included in the .gitignore? >> >>> /recursion-depth >>> xxxxxxxx* >>> pipe >>> >> >> thanks, >> -- Shuah > > > Hi, the kernel test robot already notified me about that issue, and I > sent a v2 to fix it shortly after. Please take a look at the newer > version where I added the exception for load_address.c. > Thanks. I saw your v2 after sending this email. I have a comment on v2 to split core and net patch thanks, -- Shuah
diff --git a/tools/testing/selftests/exec/.gitignore b/tools/testing/selftests/exec/.gitignore index 90c238ba6a4b..4d9fb7b20ea7 100644 --- a/tools/testing/selftests/exec/.gitignore +++ b/tools/testing/selftests/exec/.gitignore @@ -9,7 +9,7 @@ execveat.ephemeral execveat.denatured non-regular null-argv -/load_address_* +/load_address.* /recursion-depth xxxxxxxx* pipe
The name of the "load_address" objects has been modified, but the corresponding entry in the gitignore file must be updated. Update the load_address entry in the gitignore file to account for the new names. Signed-off-by: Javier Carrasco <javier.carrasco.cruz@gmail.com> --- tools/testing/selftests/exec/.gitignore | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)