Randy Dunlap
2023-Jun-10 05:43 UTC
[PATCH] Documentation: virt: clean up paravirt_ops doc.
Clarify language. Clean up grammar. Hyphenate some words.
Change "low-ops" to "low-level" since "low-ops"
isn't defined or even
mentioned anywhere else in the kernel source tree.
Signed-off-by: Randy Dunlap <rdunlap at infradead.org>
Cc: Juergen Gross <jgross at suse.com>
Cc: Ajay Kaher <akaher at vmware.com>
Cc: Alexey Makhalov <amakhalov at vmware.com>
Cc: VMware PV-Drivers Reviewers <pv-drivers at vmware.com>
Cc: virtualization at lists.linux-foundation.org
Cc: x86 at kernel.org
Cc: Paolo Bonzini <pbonzini at redhat.com>
Cc: "Luke Nowakowski-Krijger" <lnowakow at eng.ucsd.edu>
Cc: Luis Chamberlain <mcgrof at kernel.org>
---
Documentation/virt/paravirt_ops.rst | 16 ++++++++--------
1 file changed, 8 insertions(+), 8 deletions(-)
diff -- a/Documentation/virt/paravirt_ops.rst
b/Documentation/virt/paravirt_ops.rst
--- a/Documentation/virt/paravirt_ops.rst
+++ b/Documentation/virt/paravirt_ops.rst
@@ -5,31 +5,31 @@ Paravirt_ops
===========
Linux provides support for different hypervisor virtualization technologies.
-Historically different binary kernels would be required in order to support
-different hypervisors, this restriction was removed with pv_ops.
+Historically, different binary kernels would be required in order to support
+different hypervisors; this restriction was removed with pv_ops.
Linux pv_ops is a virtualization API which enables support for different
hypervisors. It allows each hypervisor to override critical operations and
allows a single kernel binary to run on all supported execution environments
including native machine -- without any hypervisors.
pv_ops provides a set of function pointers which represent operations
-corresponding to low level critical instructions and high level
-functionalities in various areas. pv-ops allows for optimizations at run
-time by enabling binary patching of the low-ops critical operations
+corresponding to low-level critical instructions and high-level
+functionalities in various areas. pv_ops allows for optimizations at run
+time by enabling binary patching of the low-level critical operations
at boot time.
pv_ops operations are classified into three categories:
- simple indirect call
- These operations correspond to high level functionality where it is
+ These operations correspond to high-level functionality where it is
known that the overhead of indirect call isn't very important.
- indirect call which allows optimization with binary patch
- Usually these operations correspond to low level critical instructions. They
+ Usually these operations correspond to low-level critical instructions. They
are called frequently and are performance critical. The overhead is
very important.
- a set of macros for hand written assembly code
Hand written assembly codes (.S files) also need paravirtualization
- because they include sensitive instructions or some of code paths in
+ because they include sensitive instructions or some code paths in
them are very performance critical.
Juergen Gross
2023-Jun-12 05:27 UTC
[PATCH] Documentation: virt: clean up paravirt_ops doc.
On 10.06.23 07:43, Randy Dunlap wrote:> Clarify language. Clean up grammar. Hyphenate some words. > > Change "low-ops" to "low-level" since "low-ops" isn't defined or even > mentioned anywhere else in the kernel source tree. > > Signed-off-by: Randy Dunlap <rdunlap at infradead.org> > Cc: Juergen Gross <jgross at suse.com> > Cc: Ajay Kaher <akaher at vmware.com> > Cc: Alexey Makhalov <amakhalov at vmware.com> > Cc: VMware PV-Drivers Reviewers <pv-drivers at vmware.com> > Cc: virtualization at lists.linux-foundation.org > Cc: x86 at kernel.org > Cc: Paolo Bonzini <pbonzini at redhat.com> > Cc: "Luke Nowakowski-Krijger" <lnowakow at eng.ucsd.edu> > Cc: Luis Chamberlain <mcgrof at kernel.org>Acked-by: Juergen Gross <jgross at suse.com> Juergen -------------- next part -------------- A non-text attachment was scrubbed... Name: OpenPGP_0xB0DE9DD628BF132F.asc Type: application/pgp-keys Size: 3098 bytes Desc: OpenPGP public key URL: <http://lists.linuxfoundation.org/pipermail/virtualization/attachments/20230612/0e326ba0/attachment.bin> -------------- next part -------------- A non-text attachment was scrubbed... Name: OpenPGP_signature Type: application/pgp-signature Size: 495 bytes Desc: OpenPGP digital signature URL: <http://lists.linuxfoundation.org/pipermail/virtualization/attachments/20230612/0e326ba0/attachment.sig>