findutils-patches
[Top][All Lists]
Advanced

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

Re: [Findutils-patches] added new xargs option -G, --load-limit


From: Eric Blake
Subject: Re: [Findutils-patches] added new xargs option -G, --load-limit
Date: Thu, 24 Aug 2017 13:15:57 -0500
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.2.1

On 08/22/2017 09:56 AM, Sven C. Dack wrote:
> Hello,
> 
> I'm sending you a patch, which adds load limiting to the xargs command.
> The option is '-G <max-load>' or '--load-limit <max-load>'.

Do we want to burn a short option letter before we know how widely the
option will be used? It's more conservative to introduce only the
long-option form, to avoid burning too many short option letters on
stuff that is infrequently useful.  This is particularly true if...

> 
> The implementation of the option relies on /proc/loadavg as found on
> Linux systems. It reads the number of active threads and processes from
> the file (the 4th value) before the executing a new command. Note, this
> is not the same as using the load averages. This value is updated by the
> kernel in real-time and represents the number of currently active,
> running threads and processes. Thus the option is only supported for
> Linux or on systems with a similar /proc/loadavg file.

...you cannot support it everywhere.

I have not looked at the patch itself, but note that it is non-trivial
in size.  Therefore, to be accepted upstream, you will have to sign
copyright assignment papers with the FSF.  If that is something you are
interested in doing, we can give you more details on that process off-list.

-- 
Eric Blake, Principal Software Engineer
Red Hat, Inc.           +1-919-301-3266
Virtualization:  qemu.org | libvirt.org

Attachment: signature.asc
Description: OpenPGP digital signature


reply via email to

[Prev in Thread] Current Thread [Next in Thread]