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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Jon,

On Wed, 16 Mar 2022 15:07:10 -0600
Jonathan Corbet <corbet@xxxxxxx> wrote:
> Tang Yizhou <tangyizhou@xxxxxxxxxx> 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@xxxxxxxxxx>
>> ---
>> 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@xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx
[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



[Index of Archives]     [Kernel Newbies]     [Security]     [Netfilter]     [Bugtraq]     [Linux FS]     [Yosemite Forum]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]     [Linux Resources]

  Powered by Linux