Skip to content

Commit f67b90b

Browse files
bwendlingkees
authored andcommitted
x86/paravirt: clean up typos and grammaros
Drive-by clean up of the comment. [ Impact: cleanup] Signed-off-by: Bill Wendling <[email protected]> Signed-off-by: Kees Cook <[email protected]> Link: https://lore.kernel.org/r/[email protected]
1 parent 9f7d69c commit f67b90b

File tree

1 file changed

+8
-8
lines changed

1 file changed

+8
-8
lines changed

arch/x86/include/asm/paravirt_types.h

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -328,7 +328,7 @@ int paravirt_disable_iospace(void);
328328
* Unfortunately, this is a relatively slow operation for modern CPUs,
329329
* because it cannot necessarily determine what the destination
330330
* address is. In this case, the address is a runtime constant, so at
331-
* the very least we can patch the call to e a simple direct call, or
331+
* the very least we can patch the call to a simple direct call, or,
332332
* ideally, patch an inline implementation into the callsite. (Direct
333333
* calls are essentially free, because the call and return addresses
334334
* are completely predictable.)
@@ -339,10 +339,10 @@ int paravirt_disable_iospace(void);
339339
* on the stack. All caller-save registers (eax,edx,ecx) are expected
340340
* to be modified (either clobbered or used for return values).
341341
* X86_64, on the other hand, already specifies a register-based calling
342-
* conventions, returning at %rax, with parameters going on %rdi, %rsi,
342+
* conventions, returning at %rax, with parameters going in %rdi, %rsi,
343343
* %rdx, and %rcx. Note that for this reason, x86_64 does not need any
344344
* special handling for dealing with 4 arguments, unlike i386.
345-
* However, x86_64 also have to clobber all caller saved registers, which
345+
* However, x86_64 also has to clobber all caller saved registers, which
346346
* unfortunately, are quite a bit (r8 - r11)
347347
*
348348
* The call instruction itself is marked by placing its start address
@@ -360,22 +360,22 @@ int paravirt_disable_iospace(void);
360360
* There are 5 sets of PVOP_* macros for dealing with 0-4 arguments.
361361
* It could be extended to more arguments, but there would be little
362362
* to be gained from that. For each number of arguments, there are
363-
* the two VCALL and CALL variants for void and non-void functions.
363+
* two VCALL and CALL variants for void and non-void functions.
364364
*
365365
* When there is a return value, the invoker of the macro must specify
366366
* the return type. The macro then uses sizeof() on that type to
367-
* determine whether its a 32 or 64 bit value, and places the return
367+
* determine whether it's a 32 or 64 bit value and places the return
368368
* in the right register(s) (just %eax for 32-bit, and %edx:%eax for
369-
* 64-bit). For x86_64 machines, it just returns at %rax regardless of
369+
* 64-bit). For x86_64 machines, it just returns in %rax regardless of
370370
* the return value size.
371371
*
372-
* 64-bit arguments are passed as a pair of adjacent 32-bit arguments
372+
* 64-bit arguments are passed as a pair of adjacent 32-bit arguments;
373373
* i386 also passes 64-bit arguments as a pair of adjacent 32-bit arguments
374374
* in low,high order
375375
*
376376
* Small structures are passed and returned in registers. The macro
377377
* calling convention can't directly deal with this, so the wrapper
378-
* functions must do this.
378+
* functions must do it.
379379
*
380380
* These PVOP_* macros are only defined within this header. This
381381
* means that all uses must be wrapped in inline functions. This also

0 commit comments

Comments
 (0)