From: Marcin Bernatowicz <marcin.bernatowicz@linux.intel.com>
To: Janusz Krzysztofik <janusz.krzysztofik@linux.intel.com>,
igt-dev@lists.freedesktop.org
Cc: intel-gfx@lists.freedesktop.org
Subject: Re: [Intel-gfx] [PATCH i-g-t] tests/core_hotunplug: Reduce debug noise on stdout
Date: Tue, 19 Jan 2021 12:08:10 +0100 [thread overview]
Message-ID: <7ac828ba904e3fc4153f5aa8041bfcf79128407b.camel@linux.intel.com> (raw)
In-Reply-To: <20210119084257.25552-1-janusz.krzysztofik@linux.intel.com>
On Tue, 2021-01-19 at 09:42 +0100, Janusz Krzysztofik wrote:
> Since igt_fixture sections are processed unconditionally regardless
> of
> which subtest has been requested, they can now emit a lot of
> unrelated
> debug messages which can make the picture less clear. Avoid emitting
> those messages from outside igt_subtest sections.
>
> Move device close status prerequisite checks from igt_fixture
> sections
> into subtest execution paths. For simplicity, pass any device close
> errors, including those from health checks, to next sections via a
> single .fd.drm data structure field.
>
> Moreover, postpone initial device health check until first actually
> selected subtest is started. In order to let that subtest skip on
> unsuccessful initial health check, not fail, move the decision
> whether
> to fail or skip on error from the health check helper to its users.
>
> Signed-off-by: Janusz Krzysztofik <janusz.krzysztofik@linux.intel.com
> >
> ---
> tests/core_hotunplug.c | 86 +++++++++++++++++++++++-----------------
> --
> 1 file changed, 48 insertions(+), 38 deletions(-)
>
> diff --git a/tests/core_hotunplug.c b/tests/core_hotunplug.c
> index 91506fa72..e7d2a4472 100644
> --- a/tests/core_hotunplug.c
> +++ b/tests/core_hotunplug.c
> @@ -49,6 +49,7 @@ struct hotunplug {
> } fd; /* >= 0: valid fd, == -1: closed, < -1: close failed
> */
> const char *dev_bus_addr;
> const char *failure;
> + bool need_healthcheck;
> };
>
> /* Helpers */
> @@ -210,6 +211,12 @@ static void cleanup(struct hotunplug *priv)
> priv->fd.drm = close_device(priv->fd.drm, "post ", "exercised
> ");
> priv->fd.drm_hc = close_device(priv->fd.drm_hc, "post ",
> "health checked
> ");
> + /* pass device close errors to next sections via priv->fd.drm
> */
> + if (priv->fd.drm_hc < -1) {
> + priv->fd.drm = priv->fd.drm_hc;
> + priv->fd.drm_hc = -1;
> + }
> +
> priv->fd.sysfs_dev = close_sysfs(priv->fd.sysfs_dev);
> }
>
> @@ -346,9 +353,9 @@ static void node_healthcheck(struct hotunplug
> *priv, unsigned flags)
>
> priv->failure = "Device sysfs healthckeck failure!";
> local_debug("%s\n", "running device sysfs
> healthcheck");
> - igt_assert(igt_sysfs_path(fd_drm, path, sizeof(path)));
> - igt_assert(igt_debugfs_path(fd_drm, path,
> sizeof(path)));
> - priv->failure = NULL;
> + if (igt_sysfs_path(fd_drm, path, sizeof(path)) &&
> + igt_debugfs_path(fd_drm, path, sizeof(path)))
> + priv->failure = NULL;
> }
>
> fd_drm = close_device(fd_drm, "", "health checked ");
> @@ -356,7 +363,7 @@ static void node_healthcheck(struct hotunplug
> *priv, unsigned flags)
> priv->fd.drm_hc = fd_drm;
> }
>
> -static void healthcheck(struct hotunplug *priv, bool recover)
> +static bool healthcheck(struct hotunplug *priv, bool recover)
> {
> /* device name may have changed, rebuild IGT device list */
> igt_devices_scan(true);
> @@ -366,8 +373,19 @@ static void healthcheck(struct hotunplug *priv,
> bool recover)
> node_healthcheck(priv,
> FLAG_RENDER | (recover ? FLAG_RECOVER
> : 0));
>
> - /* not only request igt_abort on failure, also fail the health
> check */
> - igt_fail_on_f(priv->failure, "%s\n", priv->failure);
> + return !priv->failure;
> +}
> +
> +static void pre_check(struct hotunplug *priv)
> +{
> + igt_require(priv->fd.drm == -1);
> +
> + if (priv->need_healthcheck) {
> + igt_require_f(healthcheck(priv, false), "%s\n", priv-
> >failure);
> + priv->need_healthcheck = false;
> +
> + igt_require(priv->fd.drm_hc == -1);
> + }
> }
>
> static void recover(struct hotunplug *priv)
> @@ -386,7 +404,7 @@ static void recover(struct hotunplug *priv)
> driver_bind(priv, 60);
>
> if (priv->failure)
> - healthcheck(priv, true);
> + igt_assert_f(healthcheck(priv, true), "%s\n", priv-
> >failure);
> }
>
> static void post_healthcheck(struct hotunplug *priv)
> @@ -394,8 +412,6 @@ static void post_healthcheck(struct hotunplug
> *priv)
> igt_abort_on_f(priv->failure, "%s\n", priv->failure);
>
> cleanup(priv);
> - igt_require(priv->fd.drm == -1);
> - igt_require(priv->fd.drm_hc == -1);
> }
>
> static void set_filter_from_device(int fd)
> @@ -417,32 +433,30 @@ static void set_filter_from_device(int fd)
>
> static void unbind_rebind(struct hotunplug *priv)
> {
> - igt_assert_eq(priv->fd.drm, -1);
> - igt_assert_eq(priv->fd.drm_hc, -1);
> + pre_check(priv);
>
> driver_unbind(priv, "", 0);
>
> driver_bind(priv, 0);
>
> - healthcheck(priv, false);
> + igt_assert_f(healthcheck(priv, false), "%s\n", priv->failure);
> }
>
> static void unplug_rescan(struct hotunplug *priv)
> {
> - igt_assert_eq(priv->fd.drm, -1);
> - igt_assert_eq(priv->fd.drm_hc, -1);
> + pre_check(priv);
>
> device_unplug(priv, "", 0);
>
> bus_rescan(priv, 0);
>
> - healthcheck(priv, false);
> + igt_assert_f(healthcheck(priv, false), "%s\n", priv->failure);
> }
>
> static void hotunbind_rebind(struct hotunplug *priv)
> {
> - igt_assert_eq(priv->fd.drm, -1);
> - igt_assert_eq(priv->fd.drm_hc, -1);
> + pre_check(priv);
> +
> priv->fd.drm = local_drm_open_driver(false, "", " for hot
> unbind");
>
> driver_unbind(priv, "hot ", 0);
> @@ -452,13 +466,13 @@ static void hotunbind_rebind(struct hotunplug
> *priv)
>
> driver_bind(priv, 0);
>
> - healthcheck(priv, false);
> + igt_assert_f(healthcheck(priv, false), "%s\n", priv->failure);
> }
>
> static void hotunplug_rescan(struct hotunplug *priv)
> {
> - igt_assert_eq(priv->fd.drm, -1);
> - igt_assert_eq(priv->fd.drm_hc, -1);
> + pre_check(priv);
> +
> priv->fd.drm = local_drm_open_driver(false, "", " for hot
> unplug");
>
> device_unplug(priv, "hot ", 0);
> @@ -468,39 +482,39 @@ static void hotunplug_rescan(struct hotunplug
> *priv)
>
> bus_rescan(priv, 0);
>
> - healthcheck(priv, false);
> + igt_assert_f(healthcheck(priv, false), "%s\n", priv->failure);
> }
>
> static void hotrebind(struct hotunplug *priv)
> {
> - igt_assert_eq(priv->fd.drm, -1);
> - igt_assert_eq(priv->fd.drm_hc, -1);
> + pre_check(priv);
> +
> priv->fd.drm = local_drm_open_driver(false, "", " for hot
> rebind");
>
> driver_unbind(priv, "hot ", 60);
>
> driver_bind(priv, 0);
>
> - healthcheck(priv, false);
> + igt_assert_f(healthcheck(priv, false), "%s\n", priv->failure);
> }
>
> static void hotreplug(struct hotunplug *priv)
> {
> - igt_assert_eq(priv->fd.drm, -1);
> - igt_assert_eq(priv->fd.drm_hc, -1);
> + pre_check(priv);
> +
> priv->fd.drm = local_drm_open_driver(false, "", " for hot
> replug");
>
> device_unplug(priv, "hot ", 60);
>
> bus_rescan(priv, 0);
>
> - healthcheck(priv, false);
> + igt_assert_f(healthcheck(priv, false), "%s\n", priv->failure);
> }
>
> static void hotrebind_lateclose(struct hotunplug *priv)
> {
> - igt_assert_eq(priv->fd.drm, -1);
> - igt_assert_eq(priv->fd.drm_hc, -1);
> + pre_check(priv);
> +
> priv->fd.drm = local_drm_open_driver(false, "", " for hot
> rebind");
>
> driver_unbind(priv, "hot ", 60);
> @@ -510,13 +524,13 @@ static void hotrebind_lateclose(struct
> hotunplug *priv)
> priv->fd.drm = close_device(priv->fd.drm, "late ", "unbound ");
> igt_assert_eq(priv->fd.drm, -1);
>
> - healthcheck(priv, false);
> + igt_assert_f(healthcheck(priv, false), "%s\n", priv->failure);
> }
>
> static void hotreplug_lateclose(struct hotunplug *priv)
> {
> - igt_assert_eq(priv->fd.drm, -1);
> - igt_assert_eq(priv->fd.drm_hc, -1);
> + pre_check(priv);
> +
> priv->fd.drm = local_drm_open_driver(false, "", " for hot
> replug");
>
> device_unplug(priv, "hot ", 60);
> @@ -526,7 +540,7 @@ static void hotreplug_lateclose(struct hotunplug
> *priv)
> priv->fd.drm = close_device(priv->fd.drm, "late ", "removed ");
> igt_assert_eq(priv->fd.drm, -1);
>
> - healthcheck(priv, false);
> + igt_assert_f(healthcheck(priv, false), "%s\n", priv->failure);
> }
>
> /* Main */
> @@ -536,6 +550,7 @@ igt_main
> struct hotunplug priv = {
> .fd = { .drm = -1, .drm_hc = -1,
> .sysfs_dev = -1, },
> .failure = NULL,
> + .need_healthcheck = true,
> };
>
> igt_fixture {
> @@ -570,11 +585,6 @@ igt_main
> igt_assert_eq(close_device(fd_drm, "", "selected "),
> -1);
>
> prepare(&priv);
> -
> - node_healthcheck(&priv, 0);
> - if (!priv.failure)
> - node_healthcheck(&priv, FLAG_RENDER);
> - igt_skip_on_f(priv.failure, "%s\n", priv.failure);
> }
>
> igt_subtest_group {
LGTM,
Reviewed-by: Marcin Bernatowicz <marcin.bernatowicz@linux.intel.com>
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
WARNING: multiple messages have this Message-ID (diff)
From: Marcin Bernatowicz <marcin.bernatowicz@linux.intel.com>
To: Janusz Krzysztofik <janusz.krzysztofik@linux.intel.com>,
igt-dev@lists.freedesktop.org
Cc: intel-gfx@lists.freedesktop.org
Subject: Re: [igt-dev] [Intel-gfx] [PATCH i-g-t] tests/core_hotunplug: Reduce debug noise on stdout
Date: Tue, 19 Jan 2021 12:08:10 +0100 [thread overview]
Message-ID: <7ac828ba904e3fc4153f5aa8041bfcf79128407b.camel@linux.intel.com> (raw)
In-Reply-To: <20210119084257.25552-1-janusz.krzysztofik@linux.intel.com>
On Tue, 2021-01-19 at 09:42 +0100, Janusz Krzysztofik wrote:
> Since igt_fixture sections are processed unconditionally regardless
> of
> which subtest has been requested, they can now emit a lot of
> unrelated
> debug messages which can make the picture less clear. Avoid emitting
> those messages from outside igt_subtest sections.
>
> Move device close status prerequisite checks from igt_fixture
> sections
> into subtest execution paths. For simplicity, pass any device close
> errors, including those from health checks, to next sections via a
> single .fd.drm data structure field.
>
> Moreover, postpone initial device health check until first actually
> selected subtest is started. In order to let that subtest skip on
> unsuccessful initial health check, not fail, move the decision
> whether
> to fail or skip on error from the health check helper to its users.
>
> Signed-off-by: Janusz Krzysztofik <janusz.krzysztofik@linux.intel.com
> >
> ---
> tests/core_hotunplug.c | 86 +++++++++++++++++++++++-----------------
> --
> 1 file changed, 48 insertions(+), 38 deletions(-)
>
> diff --git a/tests/core_hotunplug.c b/tests/core_hotunplug.c
> index 91506fa72..e7d2a4472 100644
> --- a/tests/core_hotunplug.c
> +++ b/tests/core_hotunplug.c
> @@ -49,6 +49,7 @@ struct hotunplug {
> } fd; /* >= 0: valid fd, == -1: closed, < -1: close failed
> */
> const char *dev_bus_addr;
> const char *failure;
> + bool need_healthcheck;
> };
>
> /* Helpers */
> @@ -210,6 +211,12 @@ static void cleanup(struct hotunplug *priv)
> priv->fd.drm = close_device(priv->fd.drm, "post ", "exercised
> ");
> priv->fd.drm_hc = close_device(priv->fd.drm_hc, "post ",
> "health checked
> ");
> + /* pass device close errors to next sections via priv->fd.drm
> */
> + if (priv->fd.drm_hc < -1) {
> + priv->fd.drm = priv->fd.drm_hc;
> + priv->fd.drm_hc = -1;
> + }
> +
> priv->fd.sysfs_dev = close_sysfs(priv->fd.sysfs_dev);
> }
>
> @@ -346,9 +353,9 @@ static void node_healthcheck(struct hotunplug
> *priv, unsigned flags)
>
> priv->failure = "Device sysfs healthckeck failure!";
> local_debug("%s\n", "running device sysfs
> healthcheck");
> - igt_assert(igt_sysfs_path(fd_drm, path, sizeof(path)));
> - igt_assert(igt_debugfs_path(fd_drm, path,
> sizeof(path)));
> - priv->failure = NULL;
> + if (igt_sysfs_path(fd_drm, path, sizeof(path)) &&
> + igt_debugfs_path(fd_drm, path, sizeof(path)))
> + priv->failure = NULL;
> }
>
> fd_drm = close_device(fd_drm, "", "health checked ");
> @@ -356,7 +363,7 @@ static void node_healthcheck(struct hotunplug
> *priv, unsigned flags)
> priv->fd.drm_hc = fd_drm;
> }
>
> -static void healthcheck(struct hotunplug *priv, bool recover)
> +static bool healthcheck(struct hotunplug *priv, bool recover)
> {
> /* device name may have changed, rebuild IGT device list */
> igt_devices_scan(true);
> @@ -366,8 +373,19 @@ static void healthcheck(struct hotunplug *priv,
> bool recover)
> node_healthcheck(priv,
> FLAG_RENDER | (recover ? FLAG_RECOVER
> : 0));
>
> - /* not only request igt_abort on failure, also fail the health
> check */
> - igt_fail_on_f(priv->failure, "%s\n", priv->failure);
> + return !priv->failure;
> +}
> +
> +static void pre_check(struct hotunplug *priv)
> +{
> + igt_require(priv->fd.drm == -1);
> +
> + if (priv->need_healthcheck) {
> + igt_require_f(healthcheck(priv, false), "%s\n", priv-
> >failure);
> + priv->need_healthcheck = false;
> +
> + igt_require(priv->fd.drm_hc == -1);
> + }
> }
>
> static void recover(struct hotunplug *priv)
> @@ -386,7 +404,7 @@ static void recover(struct hotunplug *priv)
> driver_bind(priv, 60);
>
> if (priv->failure)
> - healthcheck(priv, true);
> + igt_assert_f(healthcheck(priv, true), "%s\n", priv-
> >failure);
> }
>
> static void post_healthcheck(struct hotunplug *priv)
> @@ -394,8 +412,6 @@ static void post_healthcheck(struct hotunplug
> *priv)
> igt_abort_on_f(priv->failure, "%s\n", priv->failure);
>
> cleanup(priv);
> - igt_require(priv->fd.drm == -1);
> - igt_require(priv->fd.drm_hc == -1);
> }
>
> static void set_filter_from_device(int fd)
> @@ -417,32 +433,30 @@ static void set_filter_from_device(int fd)
>
> static void unbind_rebind(struct hotunplug *priv)
> {
> - igt_assert_eq(priv->fd.drm, -1);
> - igt_assert_eq(priv->fd.drm_hc, -1);
> + pre_check(priv);
>
> driver_unbind(priv, "", 0);
>
> driver_bind(priv, 0);
>
> - healthcheck(priv, false);
> + igt_assert_f(healthcheck(priv, false), "%s\n", priv->failure);
> }
>
> static void unplug_rescan(struct hotunplug *priv)
> {
> - igt_assert_eq(priv->fd.drm, -1);
> - igt_assert_eq(priv->fd.drm_hc, -1);
> + pre_check(priv);
>
> device_unplug(priv, "", 0);
>
> bus_rescan(priv, 0);
>
> - healthcheck(priv, false);
> + igt_assert_f(healthcheck(priv, false), "%s\n", priv->failure);
> }
>
> static void hotunbind_rebind(struct hotunplug *priv)
> {
> - igt_assert_eq(priv->fd.drm, -1);
> - igt_assert_eq(priv->fd.drm_hc, -1);
> + pre_check(priv);
> +
> priv->fd.drm = local_drm_open_driver(false, "", " for hot
> unbind");
>
> driver_unbind(priv, "hot ", 0);
> @@ -452,13 +466,13 @@ static void hotunbind_rebind(struct hotunplug
> *priv)
>
> driver_bind(priv, 0);
>
> - healthcheck(priv, false);
> + igt_assert_f(healthcheck(priv, false), "%s\n", priv->failure);
> }
>
> static void hotunplug_rescan(struct hotunplug *priv)
> {
> - igt_assert_eq(priv->fd.drm, -1);
> - igt_assert_eq(priv->fd.drm_hc, -1);
> + pre_check(priv);
> +
> priv->fd.drm = local_drm_open_driver(false, "", " for hot
> unplug");
>
> device_unplug(priv, "hot ", 0);
> @@ -468,39 +482,39 @@ static void hotunplug_rescan(struct hotunplug
> *priv)
>
> bus_rescan(priv, 0);
>
> - healthcheck(priv, false);
> + igt_assert_f(healthcheck(priv, false), "%s\n", priv->failure);
> }
>
> static void hotrebind(struct hotunplug *priv)
> {
> - igt_assert_eq(priv->fd.drm, -1);
> - igt_assert_eq(priv->fd.drm_hc, -1);
> + pre_check(priv);
> +
> priv->fd.drm = local_drm_open_driver(false, "", " for hot
> rebind");
>
> driver_unbind(priv, "hot ", 60);
>
> driver_bind(priv, 0);
>
> - healthcheck(priv, false);
> + igt_assert_f(healthcheck(priv, false), "%s\n", priv->failure);
> }
>
> static void hotreplug(struct hotunplug *priv)
> {
> - igt_assert_eq(priv->fd.drm, -1);
> - igt_assert_eq(priv->fd.drm_hc, -1);
> + pre_check(priv);
> +
> priv->fd.drm = local_drm_open_driver(false, "", " for hot
> replug");
>
> device_unplug(priv, "hot ", 60);
>
> bus_rescan(priv, 0);
>
> - healthcheck(priv, false);
> + igt_assert_f(healthcheck(priv, false), "%s\n", priv->failure);
> }
>
> static void hotrebind_lateclose(struct hotunplug *priv)
> {
> - igt_assert_eq(priv->fd.drm, -1);
> - igt_assert_eq(priv->fd.drm_hc, -1);
> + pre_check(priv);
> +
> priv->fd.drm = local_drm_open_driver(false, "", " for hot
> rebind");
>
> driver_unbind(priv, "hot ", 60);
> @@ -510,13 +524,13 @@ static void hotrebind_lateclose(struct
> hotunplug *priv)
> priv->fd.drm = close_device(priv->fd.drm, "late ", "unbound ");
> igt_assert_eq(priv->fd.drm, -1);
>
> - healthcheck(priv, false);
> + igt_assert_f(healthcheck(priv, false), "%s\n", priv->failure);
> }
>
> static void hotreplug_lateclose(struct hotunplug *priv)
> {
> - igt_assert_eq(priv->fd.drm, -1);
> - igt_assert_eq(priv->fd.drm_hc, -1);
> + pre_check(priv);
> +
> priv->fd.drm = local_drm_open_driver(false, "", " for hot
> replug");
>
> device_unplug(priv, "hot ", 60);
> @@ -526,7 +540,7 @@ static void hotreplug_lateclose(struct hotunplug
> *priv)
> priv->fd.drm = close_device(priv->fd.drm, "late ", "removed ");
> igt_assert_eq(priv->fd.drm, -1);
>
> - healthcheck(priv, false);
> + igt_assert_f(healthcheck(priv, false), "%s\n", priv->failure);
> }
>
> /* Main */
> @@ -536,6 +550,7 @@ igt_main
> struct hotunplug priv = {
> .fd = { .drm = -1, .drm_hc = -1,
> .sysfs_dev = -1, },
> .failure = NULL,
> + .need_healthcheck = true,
> };
>
> igt_fixture {
> @@ -570,11 +585,6 @@ igt_main
> igt_assert_eq(close_device(fd_drm, "", "selected "),
> -1);
>
> prepare(&priv);
> -
> - node_healthcheck(&priv, 0);
> - if (!priv.failure)
> - node_healthcheck(&priv, FLAG_RENDER);
> - igt_skip_on_f(priv.failure, "%s\n", priv.failure);
> }
>
> igt_subtest_group {
LGTM,
Reviewed-by: Marcin Bernatowicz <marcin.bernatowicz@linux.intel.com>
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
next prev parent reply other threads:[~2021-01-19 12:15 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-01-19 8:42 [Intel-gfx] [PATCH i-g-t] tests/core_hotunplug: Reduce debug noise on stdout Janusz Krzysztofik
2021-01-19 8:42 ` [igt-dev] " Janusz Krzysztofik
2021-01-19 11:08 ` Marcin Bernatowicz [this message]
2021-01-19 11:08 ` [igt-dev] [Intel-gfx] " Marcin Bernatowicz
2021-01-21 8:48 ` Janusz Krzysztofik
2021-01-21 8:48 ` [igt-dev] " Janusz Krzysztofik
2021-01-19 11:59 ` [igt-dev] ✗ Fi.CI.BAT: failure for " Patchwork
2021-01-19 16:39 ` Janusz Krzysztofik
2021-01-19 16:51 ` Vudum, Lakshminarayana
2021-01-19 16:49 ` [igt-dev] ✓ Fi.CI.BAT: success " Patchwork
2021-01-19 18:38 ` [igt-dev] ✗ Fi.CI.IGT: failure " Patchwork
2021-01-20 9:36 ` Janusz Krzysztofik
2021-01-20 15:58 ` Patchwork
2021-01-20 16:10 ` Janusz Krzysztofik
2021-01-20 16:14 ` Vudum, Lakshminarayana
2021-01-20 16:13 ` [igt-dev] ✓ Fi.CI.IGT: success " Patchwork
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
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=7ac828ba904e3fc4153f5aa8041bfcf79128407b.camel@linux.intel.com \
--to=marcin.bernatowicz@linux.intel.com \
--cc=igt-dev@lists.freedesktop.org \
--cc=intel-gfx@lists.freedesktop.org \
--cc=janusz.krzysztofik@linux.intel.com \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.