Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Wed, 22 Mar 2023 16:29:23 +0700
From:   Bagas Sanjaya <bagasdotme@gmail.com>
To:     Shuah Khan <skhan@linuxfoundation.org>, corbet@lwn.net
Cc:     kstewart@linuxfoundation.org, linux-doc@vger.kernel.org,
        linux-kernel@vger.kernel.org
Subject: Re: [PATCH] docs: add system-state document to admin-guide
Message-ID: <ZBrKcwoFMAhmIn1f@debian.me>
References: <20230321231816.31131-1-skhan@linuxfoundation.org>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha512;
        protocol="application/pgp-signature"; boundary="0vnTosCR76VYIEXj"
Content-Disposition: inline
In-Reply-To: <20230321231816.31131-1-skhan@linuxfoundation.org>
Precedence: bulk


--0vnTosCR76VYIEXj
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Tue, Mar 21, 2023 at 05:18:16PM -0600, Shuah Khan wrote:
> +scripts/checksyscalls.sh can be used to check if current architecture is
> +missing any function calls compared to i386.

i386 is the reference architecture for comparing syscalls, right?

> +This scripts parses Documentation/features to find the support status
> +information. It can be used to validate the contents of the files under
> +Documentation/features or simply list.::

"... to validate contents of Documentation/features files or simply list
them::"

> +Let's do one last sanity check on the system to see if the following two
> +command outputs match. If they don't examine your system closely. kernel
> +rootkits install their own ps, find, etc. utilities to mask their activi=
ty.
> +The outputs match on my system. Do they on yours?
> +
> +ps ax | wc -l
> +ls -d /proc/* | grep [0-9]|wc -l

The shell snippet above should be in a code block for consistency
with other snippets.

> +Linux kernel supports several hardening options to make system secure.
> +kconfig-hardened-check tool sanity checks kernel configuration for
> +security. You can checkout the latest::

"... If you don't have the tool handy, you can clone the repository::"

> + * https://docs.kernel.org/trace/index.html

Shouldn't the last ref be internal link (Documentation/trace/index.rst)?

Thanks.

--=20
An old man doll... just what I always wanted! - Clara

--0vnTosCR76VYIEXj
Content-Type: application/pgp-signature; name="signature.asc"

-----BEGIN PGP SIGNATURE-----

iHUEABYKAB0WIQSSYQ6Cy7oyFNCHrUH2uYlJVVFOowUCZBrKbgAKCRD2uYlJVVFO
o3VaAP415FImweEF2ePXNyzWcQP35sUVq4OG8VI0Ht71e73DxwD+OSu5cCXAeH48
YKh9xLW+JHubcBCX8gn0+uJykH5FRQU=
=GHde
-----END PGP SIGNATURE-----

--0vnTosCR76VYIEXj--