2006-08-01 14:32:24

by Rolf Eike Beer

[permalink] [raw]
Subject: [PATCH][Doc] Fix kerneldoc comments in kernel/timer.c

Some of the kerneldoc comments in this file are ignored since the lead-in is
malformed, using either "/*" or "/***" instead of "/**".

Signed-off-by: Rolf Eike Beer <[email protected]>

---
commit 4f52325acb61992f368a341938f8838caeacbec2
tree 8269f608e1bc704a146c1de4419bc59e5e4e312c
parent 55728fac5ce213ab144b6837054e03476ab6b6b7
author Rolf Eike Beer <[email protected]> Tue, 01 Aug 2006 16:23:22 +0200
committer Rolf Eike Beer <[email protected]> Tue, 01 Aug 2006 16:23:22 +0200

kernel/timer.c | 25 +++++++++++++++----------
1 files changed, 15 insertions(+), 10 deletions(-)

diff --git a/kernel/timer.c b/kernel/timer.c
index b650f04..f96c4fa 100644
--- a/kernel/timer.c
+++ b/kernel/timer.c
@@ -136,7 +136,7 @@ static void internal_add_timer(tvec_base
list_add_tail(&timer->entry, vec);
}

-/***
+/**
* init_timer - initialize a timer.
* @timer: the timer to be initialized
*
@@ -235,7 +235,7 @@ int __mod_timer(struct timer_list *timer

EXPORT_SYMBOL(__mod_timer);

-/***
+/**
* add_timer_on - start a timer on a particular CPU
* @timer: the timer to be added
* @cpu: the CPU to start it on
@@ -255,9 +255,10 @@ void add_timer_on(struct timer_list *tim
}


-/***
+/**
* mod_timer - modify a timer's timeout
* @timer: the timer to be modified
+ * @expires: new timeout in jiffies
*
* mod_timer is a more efficient way to update the expire field of an
* active timer (if the timer is inactive it will be activated)
@@ -291,7 +292,7 @@ int mod_timer(struct timer_list *timer,

EXPORT_SYMBOL(mod_timer);

-/***
+/**
* del_timer - deactive a timer.
* @timer: the timer to be deactivated
*
@@ -323,7 +324,10 @@ int del_timer(struct timer_list *timer)
EXPORT_SYMBOL(del_timer);

#ifdef CONFIG_SMP
-/*
+/**
+ * try_to_del_timer_sync - Try to deactivate a timer
+ * @timer: timer do del
+ *
* This function tries to deactivate a timer. Upon successful (ret >= 0)
* exit the timer is not queued and the handler is not running on any CPU.
*
@@ -351,7 +355,7 @@ out:
return ret;
}

-/***
+/**
* del_timer_sync - deactivate a timer and wait for the handler to finish.
* @timer: the timer to be deactivated
*
@@ -401,7 +405,7 @@ static int cascade(tvec_base_t *base, tv
return index;
}

-/***
+/**
* __run_timers - run all expired timers (if any) on this CPU.
* @base: the timer vector to be processed.
*
@@ -970,7 +974,7 @@ void __init timekeeping_init(void)


static int timekeeping_suspended;
-/*
+/**
* timekeeping_resume - Resumes the generic timekeeping subsystem.
* @dev: unused
*
@@ -1106,7 +1110,7 @@ static void clocksource_adjust(struct cl
clock->error -= (interval - offset) << (TICK_LENGTH_SHIFT - clock->shift);
}

-/*
+/**
* update_wall_time - Uses the current clocksource to increment the wall time
*
* Called from the timer interrupt, must hold a write on xtime_lock.
@@ -1497,8 +1501,9 @@ asmlinkage long sys_gettid(void)
return current->pid;
}

-/*
+/**
* sys_sysinfo - fill in sysinfo struct
+ * @info: pointer to buffer to fill
*/
asmlinkage long sys_sysinfo(struct sysinfo __user *info)
{


2006-08-01 14:41:56

by Alan

[permalink] [raw]
Subject: Re: [PATCH][Doc] Fix kerneldoc comments in kernel/timer.c

Ar Maw, 2006-08-01 am 16:31 +0200, ysgrifennodd Rolf Eike Beer:
> Some of the kerneldoc comments in this file are ignored since the lead-in is
> malformed, using either "/*" or "/***" instead of "/**".
>
> Signed-off-by: Rolf Eike Beer <[email protected]>

Acked-by: Alan Cox <[email protected]>

2006-08-06 20:34:53

by Randy Dunlap

[permalink] [raw]
Subject: Re: [PATCH][Doc] Fix kerneldoc comments in kernel/timer.c

On Tue, 1 Aug 2006 16:31:39 +0200 Rolf Eike Beer wrote:

> Some of the kerneldoc comments in this file are ignored since the lead-in is
> malformed, using either "/*" or "/***" instead of "/**".
>
> Signed-off-by: Rolf Eike Beer <[email protected]>

> @@ -401,7 +405,7 @@ static int cascade(tvec_base_t *base, tv
> return index;
> }
>
> -/***
> +/**
> * __run_timers - run all expired timers (if any) on this CPU.
> * @base: the timer vector to be processed.
> *

Thanks for the patch, Eike. I noticed that __run_timers needs
some help. Patch below.

---
From: Randy Dunlap <[email protected]>

Fix function description + definition so that a #macro does
not come between them and interfere with the kernel-doc.

Signed-off-by: Randy Dunlap <[email protected]>
---
kernel/timer.c | 4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)

--- linux-2618-rc3g2.orig/kernel/timer.c
+++ linux-2618-rc3g2/kernel/timer.c
@@ -405,6 +405,8 @@ static int cascade(tvec_base_t *base, tv
return index;
}

+#define INDEX(N) ((base->timer_jiffies >> (TVR_BITS + (N) * TVN_BITS)) & TVN_MASK)
+
/**
* __run_timers - run all expired timers (if any) on this CPU.
* @base: the timer vector to be processed.
@@ -412,8 +414,6 @@ static int cascade(tvec_base_t *base, tv
* This function cascades all vectors and executes all expired timer
* vectors.
*/
-#define INDEX(N) ((base->timer_jiffies >> (TVR_BITS + (N) * TVN_BITS)) & TVN_MASK)
-
static inline void __run_timers(tvec_base_t *base)
{
struct timer_list *timer;