Re: connector.h
Evgeniy Polyakov <[EMAIL PROTECTED]> wrote: > > On Thu, 2005-03-31 at 17:31 -0800, Andrew Morton wrote: > > > > > > struct cb_id > > > { > > > __u32 idx; > > > __u32 val; > > > }; > > > > It is vital that all data structures be skilfully commented - they are the > > key to understanding the code. Why the struct exists, which actor passes > > it to which other actor(s), whether the data structure is communicated with > > userspace, what other data structures it is aggregated with or linked to, > > locking rules, etc. > > It is described in Documentation/connector/connector.txt. > Should it also be placed here? I think it's better to document these things in the code. Those structs which are communicated to userspace should be described in connector.txt because they are part of the API. But a lot of the structs you have there are purely knerel-internal. > > > struct cn_msg > > > { > > > > Please do > > > > struct cn_msg { > > Neither structure declaration should have opening brace on the new > string? I don't understand your question. We lay out struct definitions thusly: struct foo { int a; int b; }; - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: connector.h
On Thu, 2005-03-31 at 17:31 -0800, Andrew Morton wrote: > > > > struct cb_id > > { > > __u32 idx; > > __u32 val; > > }; > > It is vital that all data structures be skilfully commented - they are the > key to understanding the code. Why the struct exists, which actor passes > it to which other actor(s), whether the data structure is communicated with > userspace, what other data structures it is aggregated with or linked to, > locking rules, etc. It is described in Documentation/connector/connector.txt. Should it also be placed here? > > struct cn_msg > > { > > Please do > > struct cn_msg { Neither structure declaration should have opening brace on the new string? > > > > #define CN_CBQ_NAMELEN 32 > > Commentary? Maximum allowed callback name - name will be truncated if it exceeds that limit. I will place this doc in the code. -- Evgeniy Polyakov Crash is better than data corruption -- Arthur Grabowski signature.asc Description: This is a digitally signed message part
connector.h
> > struct cb_id > { > __u32 idx; > __u32 val; > }; It is vital that all data structures be skilfully commented - they are the key to understanding the code. Why the struct exists, which actor passes it to which other actor(s), whether the data structure is communicated with userspace, what other data structures it is aggregated with or linked to, locking rules, etc. > struct cn_msg > { Please do struct cn_msg { > > #define CN_CBQ_NAMELEN32 Commentary? - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/