Both of your comments have been added to my "to do" list before the next time I publish. Thanks for the feedback.
----- Original Message ----- > From: "Stephen Hemminger" <[email protected]> > To: "Andrew Stiegmann (stieg)" <[email protected]> > Cc: [email protected], [email protected], [email protected], > [email protected], > [email protected], [email protected], > [email protected], [email protected] > Sent: Wednesday, May 16, 2012 10:01:21 AM > Subject: Re: [vmw_vmci RFC 01/11] Apply VMCI context code > > On Tue, 15 May 2012 08:06:58 -0700 > "Andrew Stiegmann (stieg)" <[email protected]> wrote: > > > Context code maintains state for vmci and allows the driver > > to communicate with multiple VMs. > > > > Signed-off-by: Andrew Stiegmann (stieg) <[email protected]> > > Running checkpatch reveals the usual noise, and the following that > should be addressed. > > ERROR: do not use C99 // comments > #272: FILE: drivers/misc/vmw_vmci/vmci_context.c:183: > +static bool ctx_exists_locked(uint32_t cid) // IN > > ERROR: "foo * bar" should be "foo *bar" > #304: FILE: drivers/misc/vmw_vmci/vmci_context.c:215: > + uid_t * user, struct vmci_ctx **outContext) > > I don't mind the C99 style comments, but the // IN convention > is pretty useless and should be removed. > _______________________________________________ Virtualization mailing list [email protected] https://lists.linuxfoundation.org/mailman/listinfo/virtualization
