Messages in this thread | | | Date | Wed, 15 Jun 2022 09:15:35 -0700 | Subject | Re: [PATCH v2] kbuild: remove sed command from cmd_ar_builtin | From | Jeff Johnson <> |
| |
On 6/14/2022 8:03 PM, Masahiro Yamada wrote: > On Wed, Jun 15, 2022 at 3:59 AM Nick Desaulniers > <ndesaulniers@google.com> wrote: >> >> On Mon, Jun 13, 2022 at 10:53 PM Masahiro Yamada <masahiroy@kernel.org> wrote: >>> >>> Replace a pipeline of echo and sed with printf to decrease process forks. >> >> If you're trying to minimize process forks, is it possible to remove >> the use of xargs as well and just invoke $(AR) with the parameters >> splatted out? I don't know myself, but maybe you're creative enough? > > > If I remove xargs, we will go back to the situation > before cd968b97c49214e6557381bddddacbd0e0fb696e. > > This patch tries to avoid "too long argument error" > without forking too many processes. > Maybe I am too worried about the potential issue, though...
From my very clouded perspective avoiding "too long argument error" should take priority :)
| |