Re: [PATCH iproute2 v2] ss: add -P/--process-age option to display process runtime

[Stephen Hemminger <stephen@networkplumber.org>]

On Thu, 17 Sep 2026 03:04:02 +0530
Harshith Hari <harshithhari000111@gmail.com> wrote:

> Add a new option (-P/--process-age) to display process uptime/age
> alongside process info inside socket user context output.
> 
> Signed-off-by: Harshith Hari <harshithhari000111@gmail.com>
> ---
> v2:
>  - Fixed patch whitespace and tab formatting.
> Signed-off-by: Harshith Hari <harshithhari000111@gmail.com>
> ---
>  man/man8/ss.8 |   3 +
>  misc/ss.c     | 662 +++++++++++++++++++++++++++++---------------------
>  2 files changed, 385 insertions(+), 280 deletions(-)
> 
> diff --git a/man/man8/ss.8 b/man/man8/ss.8
> index b0629b82..5f345c21 100644
> --- a/man/man8/ss.8
> +++ b/man/man8/ss.8
> @@ -163,6 +163,9 @@ the number of packets dropped before they are de-multiplexed into the socket
>  .TP
>  \.B \-P, \-\-process\-age
>  Show process age/runtime alongside process socket information.
> +.TP
> +\.B \-P, \-\-process\-age
> +Show process age/runtime alongside process socket information.
>  Show processes using sockets.
>  .TP
>  .B \-T, \-\-threads
> diff --git a/misc/ss.c b/misc/ss.c
> index 6c32455d..6058ca29 100644
> --- a/misc/ss.c
> +++ b/misc/ss.c
> @@ -56,58 +56,58 @@ static int show_process_age;
>  
>  static void print_process_age(pid_t pid)
>  {
> -char path[64];
> -FILE *f;
> -unsigned long long starttime;
> -double uptime = 0.0;
> -long clock_ticks = sysconf(_SC_CLK_TCK);
> -
> -f = fopen("/proc/uptime", "r");
> -if (!f)
> -return;
> -
> -if (fscanf(f, "%lf", &uptime) != 1) {
> -fclose(f);
> -return;
> -}
> -fclose(f);
> +	char path[64];
> +	FILE *f;
> +	unsigned long long starttime;
> +	double uptime = 0.0;
> +	long clock_ticks = sysconf(_SC_CLK_TCK);
> +
> +	f = fopen("/proc/uptime", "r");
> +	if (!f)
> +	

I don't think you understand how to submit patches.
This is repeated version where the diff is against your misformatted previous version.

And you did not listen to my feedback that the ss tool is about sockets
not processes. If you wanted to this kind of thing better off doing
some local tool that either scripts ss and ps output, or uses kernel API's

Don't need more growth in tools for this.