Message-ID: <8d2f86cb-5dd1-4291-cbeb-2c9f72f9ed0b@linuxfoundation.org>
Date:   Fri, 27 Jan 2023 16:49:15 -0700
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101
 Thunderbird/102.4.2
Subject: Re: [PATCH] docs: add workload-tracing document to admin-guide
Content-Language: en-US
To:     Jonathan Corbet <corbet@lwn.net>
Cc:     sshefali021@gmail.com, kstewart@linuxfoundation.org,
        linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org,
        Shuah Khan <skhan@linuxfoundation.org>
References: <20230127165440.19158-1-skhan@linuxfoundation.org>
 <87bkmj99yv.fsf@meer.lwn.net>
 <063c51c3-c276-3b68-f130-db0751b35d42@linuxfoundation.org>
From:   Shuah Khan <skhan@linuxfoundation.org>
In-Reply-To: <063c51c3-c276-3b68-f130-db0751b35d42@linuxfoundation.org>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 8bit
Precedence: bulk

On 1/27/23 12:18, Shuah Khan wrote:
> On 1/27/23 10:13, Jonathan Corbet wrote:
>> Shuah Khan <skhan@linuxfoundation.org> writes:
>>
>>> Add a new section to the admin-guide with information of interest to
>>> application developers and system integrators doing analysis of the
>>> Linux kernel for safety critical applications.
>>>
>>> This section will contain documents supporting analysis of kernel
>>> interactions with applications, and key kernel subsystems expectations.
>>>
>>> Add a new workload-tracing document to this new section.
>>>
>>> Signed-off-by: Shefali Sharma <sshefali021@gmail.com>
>>> Signed-off-by: Shuah Khan <skhan@linuxfoundation.org>
>>> ---
>>>   Documentation/admin-guide/index.rst           |  11 +
>>>   .../admin-guide/workload-tracing.rst          | 486 ++++++++++++++++++
>>>   2 files changed, 497 insertions(+)
>>>   create mode 100644 Documentation/admin-guide/workload-tracing.rst
>>
>> This seems like generally good stuff, thanks.  I have a few nits...
> 
> Thanks for the review.
> 

Cleaned it up and sent you v2. Odd. checkpatch.pl didn't catch long
lines in the document. I was so focused on verifying the html doc,
I totally missed the cut & pasted long lines. Anyway it is all fixed
now in v2.

thanks,
-- Shuah