From: Mark Salter on gitlab.com
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1598#note_866037018
The "if (!*mem_section")" is causing a panic on aarch64 systems
(CONFIG_SPARSEMEM_EXTREME=y) which have no memory in the first mem_section
root. The original "if (!mem_section)" is the co
From: Prarit Bhargava on gitlab.com
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1635#note_865862896
Good idea. I've added
```
The next line is throwaway code for transition to parallel
processing. Leaving this line in place is harmless, but it can be
removed the next time anyo
From: Prarit Bhargava
redhat/configs/build_configs.sh: Create unique output files
Create unique output files from merge_configs().
Signed-off-by: Prarit Bhargava
diff --git a/redhat/configs/build_configs.sh b/redhat/configs/build_configs.sh
index blahblah..blahblah 100755
--- a/redhat/configs
From: Prarit Bhargava
redhat/configs/build_configs.sh: Provide better messages
Provide better error messages.
Signed-off-by: Prarit Bhargava
diff --git a/redhat/configs/build_configs.sh b/redhat/configs/build_configs.sh
index blahblah..blahblah 100755
--- a/redhat/configs/build_configs.sh
+++
From: Prarit Bhargava
redhat/configs/build_configs.sh: Parallelize execution
Parallelize the execution of each config building.
The '+' adds job processing to 'non-make' commands [1].
[1] https://www.gnu.org/software/make/manual/html_node/Error-Messages.html
Signed-off-by: Prarit Bhargava
d
From: Prarit Bhargava
redhat/configs/process_configs.sh: Fix issue with old error files
Previous to this changeset, running process_configs.sh would result in an
.mismatches file being left after execution. If someone has done
make dist-configs
\# update to latest
make -j10 dist-configs
the '
From: Prarit Bhargava
redhat/configs/process_configs.sh: Parallelize execution
Parallelize the execution of each config processing.
The '+' adds job processing to 'non-make' commands [1].
[1] https://www.gnu.org/software/make/manual/html_node/Error-Messages.html
Signed-off-by: Prarit Bhargava
From: Prarit Bhargava
redhat/configs/build_configs.sh: Add local variables
Add local variables to the merge_configs function. This will make future
changes for parallel processing easier.
Signed-off-by: Prarit Bhargava
diff --git a/redhat/configs/build_configs.sh b/redhat/configs/build_confi
From: Prarit Bhargava
redhat/configs/process_configs.sh: Provide better messages
Provide better error messages.
Signed-off-by: Prarit Bhargava
diff --git a/redhat/configs/process_configs.sh
b/redhat/configs/process_configs.sh
index blahblah..blahblah 100755
--- a/redhat/configs/process_confi
From: Prarit Bhargava
redhat/configs/process_configs.sh: Add processing config function
Move the code to process a config into its own function. This will make
future changes easier.
Signed-off-by: Prarit Bhargava
diff --git a/redhat/configs/process_configs.sh
b/redhat/configs/process_confi
From: Prarit Bhargava
redhat/configs/process_configs.sh: Create unique output files
Create unique output files from process_config().
Signed-off-by: Prarit Bhargava
diff --git a/redhat/configs/process_configs.sh
b/redhat/configs/process_configs.sh
index blahblah..blahblah 100755
--- a/redhat
From: Prarit Bhargava on gitlab.com
Merge Request: https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1635
The dist-configs target takes an annoying long time.
Add parallel processing of configs in dist-configs.
Testing Info
make dist-configs time 1m22.909s
ma
From: Don Zickus on gitlab.com
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1635#note_865827704
Or at least add a fat blurb that those new lines are throwaway code to
transition to the parallel workflow. I just don't want that code lying around
and then a year later someone reads th
From: Don Zickus on gitlab.com
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1635#note_865816812
Ok. That wasn't obvious. So the correct way to fix it, is bring in your last
patch now as a standalone MR. Then this MR will remove/replace with new
removal code, right?
From: Prarit Bhargava on gitlab.com
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1638#note_865814597
Fixed.
___
kernel mailing list -- kernel@lists.fedoraproject.org
To unsubscribe send an email to kernel-le...@lists.fedoraproject.org
Fedor
From: Prarit Bhargava
redhat: Unify genspec.sh and kernel.spec variable names
Use common variable names between genspec.sh and kernel.spec.
Signed-off-by: Prarit Bhargava
diff --git a/redhat/genspec.sh b/redhat/genspec.sh
index blahblah..blahblah 100755
--- a/redhat/genspec.sh
+++ b/redhat/ge
From: Prarit Bhargava on gitlab.com
Merge Request: https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1638
genspec.sh has _22_ parameters that must be specified on the command line
which are generated by the Makefile. Instead of passing these parameters
on the command line, export them di
From: Prarit Bhargava
redhat/genspec.sh: Remove options and use Makefile variables
genspec.sh has 22 parameters that must be specified on the command line
which are generated by the Makefile. Instead of passing these parameters
on the command line, export them directly from the Makefile.
Remov
From: Prarit Bhargava on gitlab.com
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1635#note_865801544
It's hard to explain. The *ONLY* reason for the last commit is to avoid the
issue @jmflinuxtx hit because he executed both the old code and the new code
in the same tree.
If you do
From: Herton R. Krzesinski on gitlab.com
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1638#note_865715465
could you also update the changelog of the first patch in the series?
___
kernel mailing list -- kernel@lists.fedoraproject.org
To uns
From: Don Zickus on gitlab.com
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1635#note_865713661
Hmm. If you go 15 lines down it does the same thing but returns an error
code. Is that code redundant now or is there an error code to be tracked?
___
From: Prarit Bhargava on gitlab.com
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1638#note_865690949
I've updated the MR description. Fixed.
___
kernel mailing list -- kernel@lists.fedoraproject.org
To unsubscribe send an email to kernel-l
From: Herton R. Krzesinski on gitlab.com
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1638#note_865678257
yes looks good to me now. The changelogs still mention though dist-dump-
variables, but you are not using it instead the result of export (the other
open thread), but this one is
From: Prarit Bhargava
redhat: Unify genspec.sh and kernel.spec variable names
Use common variable names between genspec.sh and kernel.spec.
Signed-off-by: Prarit Bhargava
diff --git a/redhat/genspec.sh b/redhat/genspec.sh
index blahblah..blahblah 100755
--- a/redhat/genspec.sh
+++ b/redhat/ge
From: Prarit Bhargava on gitlab.com
Merge Request: https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1638
genspec.sh has _22_ parameters that must be specified on the command line
which are generated by the Makefile. These parameters can be set in the
script by using the output of the di
From: Prarit Bhargava
redhat/genspec.sh: Remove options and use Makefile variables
genspec.sh has _22_ parameters that must be specified on the command line
which are generated by the Makefile. These parameters can be set in the
script by using the output of the dist-dump-variables target which
From: Prarit Bhargava on gitlab.com
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1638#note_865630233
Actually, just a -f "$SOURCES/$SPECFILE" will work here. If something is
wrong we'll know it ;)
___
kernel mailing list -- kernel@lists.fe
From: Prarit Bhargava on gitlab.com
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1638#note_865625524
Yes, but the test itself needs to change to not only check for NULL but also
check if the file exists. I will make that change now.
___
ke
From: Prarit Bhargava
redhat/configs/build_configs.sh: Create unique output files
Create unique output files from merge_configs().
Signed-off-by: Prarit Bhargava
diff --git a/redhat/configs/build_configs.sh b/redhat/configs/build_configs.sh
index blahblah..blahblah 100755
--- a/redhat/configs
From: Prarit Bhargava
redhat/configs/process_configs.sh: Fix issue with old error files
Previous to this changeset, running process_configs.sh would result in an
.mismatches file being left after execution. If someone has done
make dist-configs
\# update to latest
make -j10 dist-configs
the '
From: Prarit Bhargava
redhat/configs/build_configs.sh: Parallelize execution
Parallelize the execution of each config building.
The '+' adds job processing to 'non-make' commands [1].
[1] https://www.gnu.org/software/make/manual/html_node/Error-Messages.html
Signed-off-by: Prarit Bhargava
d
From: Prarit Bhargava
redhat/configs/build_configs.sh: Provide better messages
Provide better error messages.
Signed-off-by: Prarit Bhargava
diff --git a/redhat/configs/build_configs.sh b/redhat/configs/build_configs.sh
index blahblah..blahblah 100755
--- a/redhat/configs/build_configs.sh
+++
From: Prarit Bhargava
redhat/configs/process_configs.sh: Parallelize execution
Parallelize the execution of each config processing.
The '+' adds job processing to 'non-make' commands [1].
[1] https://www.gnu.org/software/make/manual/html_node/Error-Messages.html
Signed-off-by: Prarit Bhargava
From: Prarit Bhargava
redhat/configs/build_configs.sh: Add local variables
Add local variables to the merge_configs function. This will make future
changes for parallel processing easier.
Signed-off-by: Prarit Bhargava
diff --git a/redhat/configs/build_configs.sh b/redhat/configs/build_confi
From: Prarit Bhargava
redhat/configs/process_configs.sh: Provide better messages
Provide better error messages.
Signed-off-by: Prarit Bhargava
diff --git a/redhat/configs/process_configs.sh
b/redhat/configs/process_configs.sh
index blahblah..blahblah 100755
--- a/redhat/configs/process_confi
From: Prarit Bhargava
redhat/configs/process_configs.sh: Create unique output files
Create unique output files from process_config().
Signed-off-by: Prarit Bhargava
diff --git a/redhat/configs/process_configs.sh
b/redhat/configs/process_configs.sh
index blahblah..blahblah 100755
--- a/redhat
From: Prarit Bhargava on gitlab.com
Merge Request: https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1635
The dist-configs target takes an annoying long time.
Add parallel processing of configs in dist-configs.
Testing Info
make dist-configs time 1m22.909s
ma
From: Prarit Bhargava
redhat/configs/process_configs.sh: Add processing config function
Move the code to process a config into its own function. This will make
future changes easier.
Signed-off-by: Prarit Bhargava
diff --git a/redhat/configs/process_configs.sh
b/redhat/configs/process_confi
From: Prarit Bhargava on gitlab.com
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1635#note_865619782
I should add you can test this by doing:
touch .mismatches
make -j20 dist-configs
This will return an error.
In any case, pushed one additional patch to fix this situation.
___
From: Prarit Bhargava on gitlab.com
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1635#note_865610680
>Actually, weirder, 'make dist-configs-check' works fine in ark-latest, but
'make dist-git' fails as above. If I do a 'make dist-srpm', that succeeds,
and afterwards 'make dist-git'
40 matches
Mail list logo
