mbox series

[next,0/2] Documentation: KVM: KVM_CAP_VM_DISABLE_NX_HUGE_PAGES documentation fixes

Message ID 20220627095151.19339-1-bagasdotme@gmail.com (mailing list archive)
Headers show
Series Documentation: KVM: KVM_CAP_VM_DISABLE_NX_HUGE_PAGES documentation fixes | expand

Message

Bagas Sanjaya June 27, 2022, 9:51 a.m. UTC
After merging kvm tree for linux-next, Stephen Rothwell reported
htmldocs warnings on KVM_CAP_VM_DISABLE_NX_HUGE_PAGES capability
documentation:

Documentation/virt/kvm/api.rst:8210: WARNING: Title underline too short.

8.38 KVM_CAP_VM_DISABLE_NX_HUGE_PAGES
---------------------------
Documentation/virt/kvm/api.rst:8217: WARNING: Unexpected indentation.

Fix these warnings by:

  [1/2]: extend the heading underline
  [2/2]: properly format the capability table

Cc: Paolo Bonzini <pbonzini@redhat.com>
Cc: Jonathan Corbet <corbet@lwn.net>
Cc: David Matlack <dmatlack@google.com>
Cc: Ben Gardon <bgardon@google.com>
Cc: Peter Xu <peterx@redhat.com>
Cc: kvm@vger.kernel.org
Cc: linux-next@vger.kernel.org
Cc: linux-kernel@vger.kernel.org

Bagas Sanjaya (2):
  Documentation: KVM: extend KVM_CAP_VM_DISABLE_NX_HUGE_PAGES heading
    underline
  KVM: x86/MMU: properly format KVM_CAP_VM_DISABLE_NX_HUGE_PAGES
    capability table

 Documentation/virt/kvm/api.rst | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

Comments

Paolo Bonzini Aug. 11, 2022, 6:36 a.m. UTC | #1
Queued, thanks.

Paolo
Bagas Sanjaya Aug. 11, 2022, 1:30 p.m. UTC | #2
On 8/11/22 13:36, Paolo Bonzini wrote:
> Queued, thanks.
> 
> Paolo
> 
> 

Thanks for picking this up. However, Stephen noted that the issue is
already showed up on mainline [1]. Maybe this series should be queued
for 6.0 release (as -rc fixes), right?

[1]: https://lore.kernel.org/linux-next/20220809164147.131f87d0@canb.auug.org.au/
Paolo Bonzini Aug. 12, 2022, 7:40 a.m. UTC | #3
On 8/11/22 15:30, Bagas Sanjaya wrote:
> On 8/11/22 13:36, Paolo Bonzini wrote:
>> Queued, thanks.
>>
>> Paolo
>>
>>
> 
> Thanks for picking this up. However, Stephen noted that the issue is
> already showed up on mainline [1]. Maybe this series should be queued
> for 6.0 release (as -rc fixes), right?

Yes, it's in Linus's tree already.  Thanks for replying to Stephen.

Paolo
Bagas Sanjaya Aug. 13, 2022, 7:31 a.m. UTC | #4
On 8/12/22 14:40, Paolo Bonzini wrote:
>>
>> Thanks for picking this up. However, Stephen noted that the issue is
>> already showed up on mainline [1]. Maybe this series should be queued
>> for 6.0 release (as -rc fixes), right?
> 
> Yes, it's in Linus's tree already.  Thanks for replying to Stephen.
> 

Thanks.