Skip to contentSkip to conversation

Some generall questions...

DISCUSSION Janosch Machowinski 2 participants 2 messages Jul 15, 2005 1 message not in the archive ACPI

lore
Share & tools
lore
#0 lore raw

Hey,
I got 2 questions :
First, are we supporting some architecture, that does not support the hlt instruction ??

And the second one, is it diswanted to write comments in the kernel source, or are most programmers just to lazy to do it ?

Thx
Janosch

-------------------------------------------------------
SF.Net email is sponsored by: Discover Easy Linux Migration Strategies from IBM. Find simple to follow Roadmaps, straightforward articles, informative Webcasts and more! Get everything you need to get up to speed, fast. http://ads.osdn.com/?ad_id=7477&alloc_id=16492&op=click

A message that is not in the archive (sent off-list or not yet received): [email protected]

#2 lore raw
Quoted 4 lines · On Fri, 15 Jul 2005 22:36:15 +0200 Janosch Machowinski wrote:

Hey,
I got 2 questions :
First, are we supporting some architecture, that does not support the hlt instruction ??

No idea.

Quoted 2 lines

And the second one, is it diswanted to write comments in the kernel source, or are most programmers just to lazy to do it ?

a. we don't want comments for obvious language constructs, e.g., don't do this:

err_count++; /* increment the error count */

b. it seems to be a challenge sometimes to get good comments and to keep comments and code synchronized. :(
But comments are generally a good thing, so that someone can understand the intent of some code or how it's accomplished, if it's tricky. And when adding new globally-visible functions, please use the kerneldoc format for them (see linux/Documentation/kernel-doc-nano-HOWTO.txt).

---
~Randy

-------------------------------------------------------
SF.Net email is sponsored by: Discover Easy Linux Migration Strategies from IBM. Find simple to follow Roadmaps, straightforward articles, informative Webcasts and more! Get everything you need to get up to speed, fast. http://ads.osdn.com/?ad_id=7477&alloc_id=16492&op=click