logo       

Re: In kernel PIC support: kernel patch: msg#00677

emulators.kvm.devel

Subject: Re: In kernel PIC support: kernel patch

Rusty Russell wrote:
> On Fri, 2007-06-22 at 13:50 +0300, Avi Kivity wrote:
>
>> Dong, Eddie wrote:
>>
>>> +
>>> +/* set irq level. If an edge is detected, then the IRR is set to 1 */
>>>
>>>
>> Kernel comments like this:
>>
>> /*
>> * blah blah blah
>> */
>>
>> This comment applies to the rest of the patch.
>>
>
> For one-liners? Last I paid attention the main debate was about
> multi-line comments.
>

I'd say for multiline comments and function headers.



-------------------------------------------------------------------------
This SF.net email is sponsored by DB2 Express
Download DB2 Express C - the FREE version of DB2 express and take
control of your XML. No limits. Just data. Click to get it now.
http://sourceforge.net/powerbar/db2/


<Prev in Thread] Current Thread [Next in Thread>
Google Custom Search

News | FAQ | advertise