Re: Race found with parallel config processing

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Mar 23, 2022 at 03:56:21PM -0400, Prarit Bhargava wrote:
<snip>
> 
> Heh, after talking with jforbes about it this is probably just as fast
> (sorry for the cut-and-paste)

Yep, this for sure will work too :), but it looks like the first config
is processed twice, which is IMHO not necessary.

> 
> diff --git a/redhat/configs/process_configs.sh
> b/redhat/configs/process_configs.sh
> index 2ee4037ad4fc..27372afc270a 100755
> --- a/redhat/configs/process_configs.sh
> +++ b/redhat/configs/process_configs.sh
> @@ -304,6 +304,10 @@ function process_configs()
>         count=0
>         for cfg in "$SCRIPT_DIR/${PACKAGE_NAME}${KVERREL}${SUBARCH}"*.config
>         do
> +               if [ "$count" -eq 0 ]; then
> +                       # do the first one by itself so that tools are built
> +                       process_config "$cfg" "$count"
> +               fi
>                 process_config "$cfg" "$count" &
>                 waitpids[${count}]=$!
>                 ((count++))
> 
> Test build in progress now.
> 
> P.

<snip>

-- 
Frantisek Hrbata
_______________________________________________
kernel mailing list -- kernel@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to kernel-leave@xxxxxxxxxxxxxxxxxxxxxxx
Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: https://lists.fedoraproject.org/archives/list/kernel@xxxxxxxxxxxxxxxxxxxxxxx
Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure




[Index of Archives]     [Fedora General Discussion]     [Older Fedora Users Archive]     [Fedora Advisory Board]     [Fedora Security]     [Fedora Devel Java]     [Fedora Legacy]     [Fedora Desktop]     [ATA RAID]     [Fedora Marketing]     [Fedora Mentors]     [Fedora Package Announce]     [Fedora Package Review]     [Fedora Music]     [Fedora Packaging]     [Centos]     [Fedora SELinux]     [Coolkey]     [Yum Users]     [Tux]     [Yosemite News]     [KDE Users]     [Fedora Art]     [Fedora Docs]     [USB]     [Asterisk PBX]

  Powered by Linux