Hi Roman,
On Tue, Apr 7, 2026 at 10:20 PM Roman Bolshakov <[email protected]> wrote:
>
> As of now the scanner may consume a lot of time, descending deep into
> irrelevant directories where no build artifacts are known to be stored.
>
> The change introduces --max-depth flag that has semantics similar to
> -maxdepth in GNU find.
>
> Signed-off-by: Roman Bolshakov <[email protected]>
Thanks for the patch, merged.
> ---
> Changes since v1:
> - Added man page blurb
>
> debuginfod/debuginfod.cxx | 20 ++++++++++++++++++++
> doc/debuginfod.8 | 6 ++++++
> 2 files changed, 26 insertions(+)
>
> diff --git a/debuginfod/debuginfod.cxx b/debuginfod/debuginfod.cxx
> index 2cf77bcc..88a69d09 100644
> --- a/debuginfod/debuginfod.cxx
> +++ b/debuginfod/debuginfod.cxx
> @@ -440,6 +440,7 @@ static const struct argp_option options[] =
>
> { NULL, 0, NULL, 0, "Options:", 2 },
> { "logical", 'L', NULL, 0, "Follow symlinks, default=ignore.", 0 },
> + { "max-depth", 'M', "LEVELS", 0, "Depth of directory levels to descend
> into, default=no-limit.", 0 },
> { "rescan-time", 't', "SECONDS", 0, "Number of seconds to wait between
> rescans, 0=disable.", 0 },
> { "groom-time", 'g', "SECONDS", 0, "Number of seconds to wait between
> database grooming, 0=disable.", 0 },
> { "maxigroom", 'G', NULL, 0, "Run a complete database groom/shrink pass
> at startup.", 0 },
> @@ -520,6 +521,7 @@ static unsigned http_port = 8002;
> static struct sockaddr_in6 http_sockaddr;
> static string addr_info = "";
> static bool webapi_cors = false;
> +static int max_depth = -1;
> static unsigned rescan_s = 300;
> static unsigned groom_s = 86400;
> static bool maxigroom = false;
> @@ -660,6 +662,13 @@ parse_opt (int key, char *arg,
> argp_failure(state, 1, EINVAL, "-D option inconsistent with passive
> mode");
> extra_ddl.push_back(string(arg));
> break;
> + case 'M':
> + if (passive_p)
> + argp_failure(state, 1, EINVAL, "-M option inconsistent with passive
> mode");
> + max_depth = atoi(arg);
> + if (max_depth < 0)
> + argp_failure(state, 1, EINVAL, "-M LEVELS needs to be at least 0");
> + break;
> case 't':
> if (passive_p)
> argp_failure(state, 1, EINVAL, "-t option inconsistent with passive
> mode");
> @@ -5183,6 +5192,17 @@ scan_source_paths()
>
> fts_scanned ++;
>
> + if (max_depth >= 0 && (f->fts_info == FTS_D || f->fts_info == FTS_DP) &&
> + f->fts_level > max_depth)
> + {
> + fts_set(fts, f, FTS_SKIP);
> + if (verbose > 2)
> + obatched(clog) << "fts skip " << f->fts_path
> + << (f->fts_info == FTS_D ? " pre-traversal" :
> + " post-traversal") <<
> endl;
> + continue;
> + };
> +
> if (verbose > 2)
> obatched(clog) << "fts traversing " << f->fts_path << endl;
>
> diff --git a/doc/debuginfod.8 b/doc/debuginfod.8
> index ead3d3b1..9dceb9b0 100644
> --- a/doc/debuginfod.8
> +++ b/doc/debuginfod.8
> @@ -252,6 +252,12 @@ device, and ignore symlinks - as in \fIfind\ -P\
> -xdev\fP. Caution: a
> loops in the symbolic directory tree might lead to \fIinfinite
> traversal\fP.
>
> +.TP
> +.B "\-M LEVELS" "\-\-max\-depth=LEVELS"
> +Limit the depth of directory traversal to \fILEVELS\fP levels
> +below the source paths - as in \fIfind\ -maxdepth\fP.
> +The default is unlimited depth; minimum is 0.
> +
> .TP
> .B "\-\-fdcache\-mbs=MB"
> Configure limits on a cache that keeps recently extracted files from
> --
> 2.43.0
>
