From: Lukasz Majewski <lukma@denx.de>
To: Alistair Francis <alistair.francis@wdc.com>
Cc: libc-alpha@sourceware.org, alistair23@gmail.com
Subject: Re: [PATCH v4 8/8] sysv/alpha: Use generic __timeval32 and helpers
Date: Fri, 21 Feb 2020 10:52:12 +0100 [thread overview]
Message-ID: <20200221105212.6245c62c@jawa> (raw)
In-Reply-To: <20200219181057.10297-9-alistair.francis@wdc.com>
[-- Attachment #1: Type: text/plain, Size: 17844 bytes --]
On Wed, 19 Feb 2020 10:10:57 -0800
Alistair Francis <alistair.francis@wdc.com> wrote:
> Now there is a generic __timeval32 and helpers we can use them for
> Alpha instead of the Alpha specific ones.
> ---
> .../unix/sysv/linux/alpha/alpha-tv32-compat.h | 124
> ------------------ sysdeps/unix/sysv/linux/alpha/osf_adjtime.c |
> 15 ++- sysdeps/unix/sysv/linux/alpha/osf_getitimer.c | 9 +-
> sysdeps/unix/sysv/linux/alpha/osf_getrusage.c | 10 +-
> .../unix/sysv/linux/alpha/osf_gettimeofday.c | 6 +-
> sysdeps/unix/sysv/linux/alpha/osf_setitimer.c | 15 ++-
> .../unix/sysv/linux/alpha/osf_settimeofday.c | 6 +-
> sysdeps/unix/sysv/linux/alpha/osf_utimes.c | 8 +-
> sysdeps/unix/sysv/linux/alpha/osf_wait4.c | 10 +-
> sysdeps/unix/sysv/linux/tv32-compat.h | 26 ++++
> 10 files changed, 67 insertions(+), 162 deletions(-)
> delete mode 100644 sysdeps/unix/sysv/linux/alpha/alpha-tv32-compat.h
>
> diff --git a/sysdeps/unix/sysv/linux/alpha/alpha-tv32-compat.h
> b/sysdeps/unix/sysv/linux/alpha/alpha-tv32-compat.h deleted file mode
> 100644 index 3073005c65..0000000000
> --- a/sysdeps/unix/sysv/linux/alpha/alpha-tv32-compat.h
> +++ /dev/null
> @@ -1,124 +0,0 @@
> -/* Compatibility definitions for `struct timeval' with 32-bit time_t.
> - Copyright (C) 2019-2020 Free Software Foundation, Inc.
> - This file is part of the GNU C Library.
> -
> - The GNU C Library is free software; you can redistribute it and/or
> - modify it under the terms of the GNU Lesser General Public
> - License as published by the Free Software Foundation; either
> - version 2.1 of the License, or (at your option) any later version.
> -
> - The GNU C Library is distributed in the hope that it will be
> useful,
> - but WITHOUT ANY WARRANTY; without even the implied warranty of
> - MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
> - Lesser General Public License for more details.
> -
> - You should have received a copy of the GNU Lesser General Public
> - License along with the GNU C Library; if not, see
> - <http://www.gnu.org/licenses/>. */
> -
> -#ifndef _TV32_COMPAT_H
> -#define _TV32_COMPAT_H 1
> -
> -#include <features.h>
> -
> -#include <bits/types.h>
> -#include <bits/types/time_t.h>
> -#include <bits/types/struct_timeval.h>
> -#include <bits/types/struct_timespec.h>
> -#include <bits/types/struct_rusage.h>
> -
> -#include <stdint.h> // for INT32_MAX
> -#include <string.h> // for memset
> -
> -#define TV_USEC_MAX 999999 // 10**6 - 1
> -
> -/* A version of 'struct timeval' with 32-bit time_t. */
> -struct timeval32
> -{
> - int32_t tv_sec;
> - int32_t tv_usec;
> -};
> -
> -/* Structures containing 'struct timeval' with 32-bit time_t. */
> -struct itimerval32
> -{
> - struct timeval32 it_interval;
> - struct timeval32 it_value;
> -};
> -
> -struct rusage32
> -{
> - struct timeval32 ru_utime; /* user time used */
> - struct timeval32 ru_stime; /* system time used */
> - long ru_maxrss; /* maximum resident set size */
> - long ru_ixrss; /* integral shared memory size */
> - long ru_idrss; /* integral unshared data size */
> - long ru_isrss; /* integral unshared stack size */
> - long ru_minflt; /* page reclaims */
> - long ru_majflt; /* page faults */
> - long ru_nswap; /* swaps */
> - long ru_inblock; /* block input operations */
> - long ru_oublock; /* block output operations */
> - long ru_msgsnd; /* messages sent */
> - long ru_msgrcv; /* messages received */
> - long ru_nsignals; /* signals received */
> - long ru_nvcsw; /* voluntary context switches */
> - long ru_nivcsw; /* involuntary " */
> -};
> -
> -/* Conversion functions. If the seconds field of a timeval32 would
> - overflow, they write { INT32_MAX, TV_USEC_MAX } to the output. */
> -
> -static inline struct timeval
> -alpha_valid_timeval32_to_timeval (const struct timeval32 tv)
> -{
> - return (struct timeval) { tv.tv_sec, tv.tv_usec };
> -}
> -
> -static inline struct timeval32
> -alpha_valid_timeval_to_timeval32 (const struct timeval tv64)
> -{
> - if (__glibc_unlikely (tv64.tv_sec > (time_t) INT32_MAX))
> - return (struct timeval32) { INT32_MAX, TV_USEC_MAX};
> - return (struct timeval32) { tv64.tv_sec, tv64.tv_usec };
> -}
> -
> -static inline struct timespec
> -alpha_valid_timeval32_to_timespec (const struct timeval32 tv)
> -{
> - return (struct timespec) { tv.tv_sec, tv.tv_usec * 1000 };
> -}
> -
> -static inline struct timeval32
> -alpha_valid_timespec_to_timeval32 (const struct timespec ts)
> -{
> - return (struct timeval32) { (time_t) ts.tv_sec, ts.tv_nsec / 1000
> }; -}
> -
> -static inline void
> -alpha_rusage64_to_rusage32 (struct rusage32 *restrict r32,
> - const struct rusage *restrict r64)
> -{
> - /* Make sure the entire output structure is cleared, including
> - padding and reserved fields. */
> - memset (r32, 0, sizeof *r32);
> -
> - r32->ru_utime = alpha_valid_timeval_to_timeval32
> (r64->ru_utime);
> - r32->ru_stime = alpha_valid_timeval_to_timeval32
> (r64->ru_stime);
> - r32->ru_maxrss = r64->ru_maxrss;
> - r32->ru_ixrss = r64->ru_ixrss;
> - r32->ru_idrss = r64->ru_idrss;
> - r32->ru_isrss = r64->ru_isrss;
> - r32->ru_minflt = r64->ru_minflt;
> - r32->ru_majflt = r64->ru_majflt;
> - r32->ru_nswap = r64->ru_nswap;
> - r32->ru_inblock = r64->ru_inblock;
> - r32->ru_oublock = r64->ru_oublock;
> - r32->ru_msgsnd = r64->ru_msgsnd;
> - r32->ru_msgrcv = r64->ru_msgrcv;
> - r32->ru_nsignals = r64->ru_nsignals;
> - r32->ru_nvcsw = r64->ru_nvcsw;
> - r32->ru_nivcsw = r64->ru_nivcsw;
> -}
> -
> -#endif /* alpha-tv32-compat.h */
> diff --git a/sysdeps/unix/sysv/linux/alpha/osf_adjtime.c
> b/sysdeps/unix/sysv/linux/alpha/osf_adjtime.c index
> f0a1123639..95d646452e 100644 ---
> a/sysdeps/unix/sysv/linux/alpha/osf_adjtime.c +++
> b/sysdeps/unix/sysv/linux/alpha/osf_adjtime.c @@ -20,9 +20,10 @@
>
> #if SHLIB_COMPAT (libc, GLIBC_2_0, GLIBC_2_1)
>
> +#include <time.h>
> #include <sys/time.h>
> #include <sys/timex.h>
> -#include <alpha-tv32-compat.h>
> +#include <string.h>
>
> struct timex32 {
> unsigned int modes; /* mode selector */
> @@ -36,7 +37,7 @@ struct timex32 {
> long tolerance; /* clock frequency tolerance
> (ppm)
> * (read only)
> */
> - struct timeval32 time; /* (read only) */
> + struct __timeval32 time; /* (read only) */
> long tick; /* (modified) usecs between clock
> ticks */
> long ppsfreq; /* pps frequency (scaled ppm) (ro) */
> @@ -55,15 +56,15 @@ struct timex32 {
>
> int
> attribute_compat_text_section
> -__adjtime_tv32 (const struct timeval32 *itv, struct timeval32 *otv)
> +__adjtime_tv32 (const struct __timeval32 *itv, struct __timeval32
> *otv) {
> - struct timeval itv64 = alpha_valid_timeval32_to_timeval (*itv);
> + struct timeval itv64 = valid_timeval32_to_timeval (*itv);
> struct timeval otv64;
>
> if (__adjtime (&itv64, &otv64) == -1)
> return -1;
>
> - *otv = alpha_valid_timeval_to_timeval32 (otv64);
> + *otv = valid_timeval_to_timeval32 (otv64);
> return 0;
> }
>
> @@ -91,7 +92,7 @@ __adjtimex_tv32 (struct timex32 *tx)
> tx64.calcnt = tx->calcnt;
> tx64.errcnt = tx->errcnt;
> tx64.stbcnt = tx->stbcnt;
> - tx64.time = alpha_valid_timeval32_to_timeval (tx->time);
> + tx64.time = valid_timeval32_to_timeval (tx->time);
>
> int status = __adjtimex (&tx64);
> if (status < 0)
> @@ -116,7 +117,7 @@ __adjtimex_tv32 (struct timex32 *tx)
> tx->calcnt = tx64.calcnt;
> tx->errcnt = tx64.errcnt;
> tx->stbcnt = tx64.stbcnt;
> - tx->time = alpha_valid_timeval_to_timeval32 (tx64.time);
> + tx->time = valid_timeval_to_timeval32 (tx64.time);
>
> return status;
> }
> diff --git a/sysdeps/unix/sysv/linux/alpha/osf_getitimer.c
> b/sysdeps/unix/sysv/linux/alpha/osf_getitimer.c index
> 204d4ba796..4d25060e96 100644 ---
> a/sysdeps/unix/sysv/linux/alpha/osf_getitimer.c +++
> b/sysdeps/unix/sysv/linux/alpha/osf_getitimer.c @@ -20,12 +20,13 @@
>
> #if SHLIB_COMPAT (libc, GLIBC_2_0, GLIBC_2_1)
>
> +#include <time.h>
> #include <sys/time.h>
> -#include <alpha-tv32-compat.h>
> +#include <tv32-compat.h>
>
> int
> attribute_compat_text_section
> -__getitimer_tv32 (int which, struct itimerval32 *curr_value)
> +__getitimer_tv32 (int which, struct __itimerval32 *curr_value)
> {
> struct itimerval curr_value_64;
> if (__getitimer (which, &curr_value_64) == -1)
> @@ -33,9 +34,9 @@ __getitimer_tv32 (int which, struct itimerval32
> *curr_value)
> /* Write all fields of 'curr_value' regardless of overflow. */
> curr_value->it_interval
> - = alpha_valid_timeval_to_timeval32 (curr_value_64.it_interval);
> + = valid_timeval_to_timeval32 (curr_value_64.it_interval);
> curr_value->it_value
> - = alpha_valid_timeval_to_timeval32 (curr_value_64.it_value);
> + = valid_timeval_to_timeval32 (curr_value_64.it_value);
> return 0;
> }
>
> diff --git a/sysdeps/unix/sysv/linux/alpha/osf_getrusage.c
> b/sysdeps/unix/sysv/linux/alpha/osf_getrusage.c index
> be81994654..95cedb0cf6 100644 ---
> a/sysdeps/unix/sysv/linux/alpha/osf_getrusage.c +++
> b/sysdeps/unix/sysv/linux/alpha/osf_getrusage.c @@ -22,16 +22,16 @@
>
> #include <sys/time.h>
> #include <sys/resource.h>
> -#include <alpha-tv32-compat.h>
> +#include <tv32-compat.h>
>
> int
> -__getrusage_tv32 (int who, struct rusage32 *usage32)
> +__getrusage_tv32 (int who, struct __rusage32 *usage32)
> {
> - struct rusage usage64;
> - if (__getrusage (who, &usage64) == -1)
> + struct rusage usage;
> + if (__getrusage (who, &usage) == -1)
> return -1;
>
> - alpha_rusage64_to_rusage32 (usage32, &usage64);
> + rusage64_to_rusage32 (&usage, usage32);
> return 0;
> }
>
> diff --git a/sysdeps/unix/sysv/linux/alpha/osf_gettimeofday.c
> b/sysdeps/unix/sysv/linux/alpha/osf_gettimeofday.c index
> 9ffda2fde3..8cf5d303f9 100644 ---
> a/sysdeps/unix/sysv/linux/alpha/osf_gettimeofday.c +++
> b/sysdeps/unix/sysv/linux/alpha/osf_gettimeofday.c @@ -20,17 +20,17 @@
>
> #if SHLIB_COMPAT (libc, GLIBC_2_0, GLIBC_2_1)
>
> +#include <time.h>
> #include <string.h>
> #include <time.h>
> #include <sys/time.h>
> -#include <alpha-tv32-compat.h>
>
> /* Get the current time of day and timezone information putting it
> into *TV and *TZ. */
>
> int
> attribute_compat_text_section
> -__gettimeofday_tv32 (struct timeval32 *restrict tv32, void *restrict
> tz) +__gettimeofday_tv32 (struct __timeval32 *restrict tv32, void
> *restrict tz) {
> if (__glibc_unlikely (tz != 0))
> memset (tz, 0, sizeof (struct timezone));
> @@ -38,7 +38,7 @@ __gettimeofday_tv32 (struct timeval32 *restrict
> tv32, void *restrict tz) struct timespec ts;
> __clock_gettime (CLOCK_REALTIME, &ts);
>
> - *tv32 = alpha_valid_timespec_to_timeval32 (ts);
> + *tv32 = valid_timespec_to_timeval32 (ts);
> return 0;
> }
>
> diff --git a/sysdeps/unix/sysv/linux/alpha/osf_setitimer.c
> b/sysdeps/unix/sysv/linux/alpha/osf_setitimer.c index
> 726dfc8b0e..842ab5f64e 100644 ---
> a/sysdeps/unix/sysv/linux/alpha/osf_setitimer.c +++
> b/sysdeps/unix/sysv/linux/alpha/osf_setitimer.c @@ -20,19 +20,20 @@
>
> #if SHLIB_COMPAT (libc, GLIBC_2_0, GLIBC_2_1)
>
> +#include <time.h>
> #include <sys/time.h>
> -#include <alpha-tv32-compat.h>
> +#include <tv32-compat.h>
>
> int
> attribute_compat_text_section
> -__setitimer_tv32 (int which, const struct itimerval32 *restrict
> new_value,
> - struct itimerval32 *restrict old_value)
> +__setitimer_tv32 (int which, const struct __itimerval32 *restrict
> new_value,
> + struct __itimerval32 *restrict old_value)
> {
> struct itimerval new_value_64;
> new_value_64.it_interval
> - = alpha_valid_timeval32_to_timeval (new_value->it_interval);
> + = valid_timeval32_to_timeval (new_value->it_interval);
> new_value_64.it_value
> - = alpha_valid_timeval32_to_timeval (new_value->it_value);
> + = valid_timeval32_to_timeval (new_value->it_value);
>
> if (old_value == NULL)
> return __setitimer (which, &new_value_64, NULL);
> @@ -43,9 +44,9 @@ __setitimer_tv32 (int which, const struct
> itimerval32 *restrict new_value,
> /* Write all fields of 'old_value' regardless of overflow. */
> old_value->it_interval
> - = alpha_valid_timeval_to_timeval32 (old_value_64.it_interval);
> + = valid_timeval_to_timeval32 (old_value_64.it_interval);
> old_value->it_value
> - = alpha_valid_timeval_to_timeval32 (old_value_64.it_value);
> + = valid_timeval_to_timeval32 (old_value_64.it_value);
> return 0;
> }
>
> diff --git a/sysdeps/unix/sysv/linux/alpha/osf_settimeofday.c
> b/sysdeps/unix/sysv/linux/alpha/osf_settimeofday.c index
> 044363e079..5ea0a5afb7 100644 ---
> a/sysdeps/unix/sysv/linux/alpha/osf_settimeofday.c +++
> b/sysdeps/unix/sysv/linux/alpha/osf_settimeofday.c @@ -20,16 +20,16 @@
>
> #if SHLIB_COMPAT (libc, GLIBC_2_0, GLIBC_2_1)
>
> +#include <time.h>
> #include <sys/time.h>
> #include <time.h>
> #include <errno.h>
> -#include <alpha-tv32-compat.h>
>
> /* Set the current time of day and timezone information.
> This call is restricted to the super-user. */
> int
> attribute_compat_text_section
> -__settimeofday_tv32 (const struct timeval32 *tv32,
> +__settimeofday_tv32 (const struct __timeval32 *tv32,
> const struct timezone *tz)
> {
> if (__glibc_unlikely (tz != 0))
> @@ -42,7 +42,7 @@ __settimeofday_tv32 (const struct timeval32 *tv32,
> return __settimezone (tz);
> }
>
> - struct timespec ts = alpha_valid_timeval32_to_timespec (*tv32);
> + struct timespec ts = valid_timeval32_to_timespec (*tv32);
> return __clock_settime (CLOCK_REALTIME, &ts);
> }
>
> diff --git a/sysdeps/unix/sysv/linux/alpha/osf_utimes.c
> b/sysdeps/unix/sysv/linux/alpha/osf_utimes.c index
> 8ad9fb567c..8afbf12185 100644 ---
> a/sysdeps/unix/sysv/linux/alpha/osf_utimes.c +++
> b/sysdeps/unix/sysv/linux/alpha/osf_utimes.c @@ -20,16 +20,16 @@
>
> #if SHLIB_COMPAT (libc, GLIBC_2_0, GLIBC_2_1)
>
> +#include <time.h>
> #include <sys/time.h>
> -#include <alpha-tv32-compat.h>
>
> int
> attribute_compat_text_section
> -__utimes_tv32 (const char *filename, const struct timeval32
> times32[2]) +__utimes_tv32 (const char *filename, const struct
> __timeval32 times32[2]) {
> struct timeval times[2];
> - times[0] = alpha_valid_timeval32_to_timeval (times32[0]);
> - times[1] = alpha_valid_timeval32_to_timeval (times32[1]);
> + times[0] = valid_timeval32_to_timeval (times32[0]);
> + times[1] = valid_timeval32_to_timeval (times32[1]);
> return __utimes (filename, times);
> }
>
> diff --git a/sysdeps/unix/sysv/linux/alpha/osf_wait4.c
> b/sysdeps/unix/sysv/linux/alpha/osf_wait4.c index
> c664e8e93f..4242d5227f 100644 ---
> a/sysdeps/unix/sysv/linux/alpha/osf_wait4.c +++
> b/sysdeps/unix/sysv/linux/alpha/osf_wait4.c @@ -23,17 +23,17 @@
> #include <sys/time.h>
> #include <sys/resource.h>
> #include <sys/wait.h>
> -#include <alpha-tv32-compat.h>
> +#include <tv32-compat.h>
>
> pid_t
> attribute_compat_text_section
> -__wait4_tv32 (pid_t pid, int *status, int options, struct rusage32
> *usage32) +__wait4_tv32 (pid_t pid, int *status, int options, struct
> __rusage32 *usage32) {
> - struct rusage usage64;
> - pid_t child = __wait4 (pid, status, options, &usage64);
> + struct rusage usage;
> + pid_t child = __wait4 (pid, status, options, &usage);
>
> if (child >= 0 && usage32 != NULL)
> - alpha_rusage64_to_rusage32 (usage32, &usage64);
> + rusage64_to_rusage32 (&usage, usage32);
> return child;
> }
>
> diff --git a/sysdeps/unix/sysv/linux/tv32-compat.h
> b/sysdeps/unix/sysv/linux/tv32-compat.h index 1ada1f729c..f906f6acf5
> 100644 --- a/sysdeps/unix/sysv/linux/tv32-compat.h
> +++ b/sysdeps/unix/sysv/linux/tv32-compat.h
> @@ -75,4 +75,30 @@ rusage32_to_rusage64 (const struct __rusage32
> *restrict r32, r64->ru_nivcsw = r32->ru_nivcsw;
> }
>
> +static inline void
> +rusage64_to_rusage32 (const struct __rusage64 *restrict r64,
> + struct __rusage32 *restrict r32)
> +{
> + /* Make sure the entire output structure is cleared, including
> + padding and reserved fields. */
> + memset (r32, 0, sizeof *r32);
> +
> + r32->ru_utime = valid_timeval64_to_timeval32 (r64->ru_utime);
> + r32->ru_stime = valid_timeval64_to_timeval32 (r64->ru_stime);
> + r32->ru_maxrss = r64->ru_maxrss;
> + r32->ru_ixrss = r64->ru_ixrss;
> + r32->ru_idrss = r64->ru_idrss;
> + r32->ru_isrss = r64->ru_isrss;
> + r32->ru_minflt = r64->ru_minflt;
> + r32->ru_majflt = r64->ru_majflt;
> + r32->ru_nswap = r64->ru_nswap;
> + r32->ru_inblock = r64->ru_inblock;
> + r32->ru_oublock = r64->ru_oublock;
> + r32->ru_msgsnd = r64->ru_msgsnd;
> + r32->ru_msgrcv = r64->ru_msgrcv;
> + r32->ru_nsignals = r64->ru_nsignals;
> + r32->ru_nvcsw = r64->ru_nvcsw;
> + r32->ru_nivcsw = r64->ru_nivcsw;
> +}
> +
> #endif /* tv32-compat.h */
Reviewed-by: Lukasz Majewski <lukma@denx.de>
Best regards,
Lukasz Majewski
--
DENX Software Engineering GmbH, Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-59 Fax: (+49)-8142-66989-80 Email: lukma@denx.de
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
prev parent reply other threads:[~2020-02-21 9:52 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-02-19 18:10 [PATCH v4 0/8] Always use 32-bit time_t for certain syscalls Alistair Francis
2020-02-19 18:10 ` [PATCH v4 1/8] sysv/linux: Rename alpha functions to be alpha specific Alistair Francis
2020-02-28 17:30 ` Adhemerval Zanella
2020-02-28 17:52 ` Zack Weinberg
2020-02-19 18:10 ` [PATCH v4 2/8] time: Add a timeval with a 32-bit tv_sec and tv_usec Alistair Francis
2020-02-19 18:10 ` [PATCH v4 3/8] time: Add a __itimerval64 struct Alistair Francis
2020-02-19 18:10 ` [PATCH v4 4/8] sysv: Define __KERNEL_OLD_TIMEVAL_MATCHES_TIMEVAL64 Alistair Francis
2020-02-19 18:10 ` [PATCH v4 5/8] linux: Use long time_t __getitimer/__setitimer Alistair Francis
2020-02-21 10:03 ` Lukasz Majewski
2020-02-28 16:56 ` Lukasz Majewski
2020-02-19 18:10 ` [PATCH v4 6/8] resource: Add a __rusage64 struct Alistair Francis
2020-02-19 18:10 ` [PATCH v4 7/8] linux: Use long time_t for wait4/getrusage Alistair Francis
2020-02-21 10:08 ` Lukasz Majewski
2020-02-19 18:10 ` [PATCH v4 8/8] sysv/alpha: Use generic __timeval32 and helpers Alistair Francis
2020-02-21 9:52 ` Lukasz Majewski [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: https://www.gnu.org/software/libc/involved.html
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20200221105212.6245c62c@jawa \
--to=lukma@denx.de \
--cc=alistair.francis@wdc.com \
--cc=alistair23@gmail.com \
--cc=libc-alpha@sourceware.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).