Re: [PATCH v2] docs: scheduler: Convert schedutil.txt to ReST

[Akira Yokosawa <akiyks@gmail.com>]

Hi Jon,

On Wed, 16 Mar 2022 15:07:10 -0600
Jonathan Corbet <corbet@lwn.net> wrote:
> Tang Yizhou <tangyizhou@huawei.com> writes:
> 
>> All other scheduler documents have been converted to *.rst. Let's do
>> the same for schedutil.txt.
>>
>> Also fixed some typos.
>>
>> Signed-off-by: Tang Yizhou <tangyizhou@huawei.com>
>> ---
>> v2:
>> Remove unnecessary space-to-tab conversions.
>>
>>  Documentation/scheduler/index.rst             |  1 +
>>  .../{schedutil.txt => schedutil.rst}          | 30 +++++++++++--------
>>  2 files changed, 18 insertions(+), 13 deletions(-)
>>  rename Documentation/scheduler/{schedutil.txt => schedutil.rst} (92%)
> 
> I have applied this version.

Jon, I think you missed Peter'z (implied) Nak on this conversion.

Quote from Peter's message [1]:

> As I'm sure you're aware, I'd *love* to convert the whole lot to plain
> text again :-)
> 
> People, throughout the ages, have been able to read plain text, I don't
> see why it should be made more difficult than that.

I'd recommend you to drop this from docs-next for now.
I think this should go through -tip.

I don't want to see another unfortunate removal of doc.
Remember what happened to atomc_opt.txt:

  - conversion [2]
  - addition of alternative plaintext docs [3] (See final words in the changelog.)
  - removal [4]

[1]: https://lore.kernel.org/r/20220312114017.GA6235@worktop.programming.kicks-ass.net
[2]: 326bc876fed4 ("Documentation/atomic_ops.txt: convert to ReST markup")
[3]: 706eeb3e9c6f ("Documentation/locking/atomic: Add documents for new atomic_t APIs")
[4]: f0400a77ebdc ("atomic: Delete obsolete documentation")

        Thanks, Akira
 

> 
> Thanks,
> 
> jon