Thread (12 messages) 12 messages, 3 authors, 2008-06-13

Re: [PATCH] mm: fix comment formatting in asm-generic/pgtable.h:__ptep_modify_prot_

From: Rusty Russell <hidden>
Date: 2008-06-02 23:46:19
Also in: lkml, xen-devel

On Monday 02 June 2008 23:02:34 Jeremy Fitzhardinge wrote:
quoted hunk ↗ jump to hunk
Signed-off-by: Jeremy Fitzhardinge <redacted>
---
 include/asm-generic/pgtable.h |   14 +++++++++-----
 1 file changed, 9 insertions(+), 5 deletions(-)

===================================================================
--- a/include/asm-generic/pgtable.h
+++ b/include/asm-generic/pgtable.h
@@ -201,9 +201,11 @@
 					     unsigned long addr,
 					     pte_t *ptep)
 {
-	/* Get the current pte state, but zero it out to make it
-	   non-present, preventing the hardware from asynchronously
-	   updating it. */
+	/*
+	 * Get the current pte state, but zero it out to make it
+	 * non-present, preventing the hardware from asynchronously
+	 * updating it.
+	 */
Since there is debate over whether winged comments are a feature, I'm not sure 
this can be termed a "fix".
 {
-	/* The pte is non-present, so there's no hardware state to
-	   preserve. */
+	/*
+	 * The pte is non-present, so there's no hardware state to
+	 * preserve.
+	 */
 	set_pte_at(mm, addr, ptep, pte);
 }
This will fit in one line, no? 

Rusty.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help