<html xmlns:v="urn:schemas-microsoft-com:vml" xmlns:o="urn:schemas-microsoft-com:office:office" xmlns:w="urn:schemas-microsoft-com:office:word" xmlns:m="http://schemas.microsoft.com/office/2004/12/omml" xmlns="http://www.w3.org/TR/REC-html40"> <head> <meta http-equiv="Content-Type" content="text/html; charset=utf-8"> <meta name="Generator" content="Microsoft Word 15 (filtered medium)"> <style></style> </head> <body lang="EN-US" link="#0563C1" vlink="#954F72"> <div class="WordSection1"> >Ok, the naming convention used by me is `CamelCase`. One suggested from your end is `snake_case`.<o:p></o:p> >Does DPDK has a constrain it can not use CamelCase.<o:p></o:p> <o:p> </o:p> Please refer to:<o:p></o:p> <a href="https://doc.dpdk.org/guides/contributing/coding_style.html">https://doc.dpdk.org/guides/contributing/coding_style.html</a><o:p></o:p> In particular:<o:p></o:p> 1.5.4. Variable Declarations<o:p></o:p> In declarations, do not put any whitespace between asterisks and adjacent tokens, except for tokens that are identifiers related to types. (These identifiers are the names of basic types, type qualifiers, and typedef-names other than the one being declared.) Separate these identifiers from asterisks using a single space.<o:p></o:p> For example:<o:p></o:p> int *x; /* no space after asterisk */<o:p></o:p> int * const x; /* space after asterisk when using a type qualifier */<o:p></o:p> <![if !supportLists]>· <![endif]>All externally-visible variables should have an rte_ prefix in the name to avoid namespace collisions.<o:p></o:p> <![if !supportLists]>· <![endif]>Do not use uppercase letters - either in the form of ALL_UPPERCASE, or CamelCase - in variable names. Lower-case letters and underscores only.<o:p></o:p> <o:p> </o:p> <o:p> </o:p> <o:p> </o:p> <o:p> </o:p> <div style="border:none;border-left:solid blue 1.5pt;padding:0in 0in 0in 4.0pt"> <div> <div style="border:none;border-top:solid #E1E1E1 1.0pt;padding:3.0pt 0in 0in 0in"> From: Varghese, Vipin <vipin.varghese@amd.com> Sent: Tuesday, March 12, 2024 3:48 AM To: Fengchengwen <fengchengwen@huawei.com>; dev@dpdk.org Cc: ferruh.yigit@amd.com; neerav.parikh@amd.com Subject: Re: [PATCH v2] app/dma-perf: calrify incorrect NUMA config<o:p></o:p> </div> </div> <o:p> </o:p> <snipped> <o:p></o:p> <blockquote style="margin-top:5.0pt;margin-bottom:5.0pt"> <blockquote style="margin-top:5.0pt;margin-bottom:5.0pt"> <pre>diff --git a/app/test-dma-perf/benchmark.c b/app/test-dma-perf/benchmark.c<o:p></o:p></pre> <pre>index 9b1f58c78c..b6d0dbe4c0 100644<o:p></o:p></pre> <pre>--- a/app/test-dma-perf/benchmark.c<o:p></o:p></pre> <pre>+++ b/app/test-dma-perf/benchmark.c<o:p></o:p></pre> <pre>@@ -311,9 +311,14 @@ setup_memory_env(struct test_configure *cfg, struct rte_mbuf ***srcs,<o:p></o:p></pre> <pre> uint32_t nr_buf = cfg->nr_buf;<o:p></o:p></pre> <pre><o:p> </o:p></pre> <pre> nr_sockets = rte_socket_count();<o:p></o:p></pre> <pre>- if (cfg->src_numa_node >= nr_sockets ||<o:p></o:p></pre> <pre>- cfg->dst_numa_node >= nr_sockets) {<o:p></o:p></pre> <pre>- printf("Error: Source or destination numa exceeds the acture numa nodes.\n");<o:p></o:p></pre> <pre>+<o:p></o:p></pre> <pre>+ bool isSrcNumaIncorrect = (cfg->src_numa_node >= nr_sockets);<o:p></o:p></pre> <pre>+ bool isDstNumaIncorrect = (cfg->dst_numa_node >= nr_sockets);<o:p></o:p></pre> </blockquote> <pre><o:p> </o:p></pre> <pre>The naming style needs to be adjusted, how about<o:p></o:p></pre> <pre>bool is_src_numa_exceed, is_dst_numa_exceed;<o:p></o:p></pre> </blockquote> Ok, the naming convention used by me is `CamelCase`. One suggested from your end is `snake_case`.<o:p></o:p> Does DPDK has a constrain it can not use CamelCase.<o:p></o:p> <blockquote style="margin-top:5.0pt;margin-bottom:5.0pt"> <pre><o:p> </o:p></pre> <pre><o:p> </o:p></pre> <pre>And predefine the variable at the beginning of function, sort by length, some like:<o:p></o:p></pre> <pre><o:p> </o:p></pre> <pre> bool is_src_numa_exceed, is_dst_numa_exceed;<o:p></o:p></pre> <pre> unsigned int buf_size = cfg->buf_size.cur;<o:p></o:p></pre> <pre> uint32_t nr_buf = cfg->nr_buf;<o:p></o:p></pre> <pre> unsigned int nr_sockets;<o:p></o:p></pre> <pre><o:p> </o:p></pre> <pre> nr_sockets = rte_socket_count();<o:p></o:p></pre> <pre> is_src_numa_exceed =<o:p></o:p></pre> <pre> is_dst_numa_exceed =<o:p></o:p></pre> <pre> if (xxx)<o:p></o:p></pre> <pre> ...<o:p></o:p></pre> <pre><o:p> </o:p></pre> <blockquote style="margin-top:5.0pt;margin-bottom:5.0pt"> <pre>+<o:p></o:p></pre> <pre>+ if (isSrcNumaIncorrect || isDstNumaIncorrect) {<o:p></o:p></pre> <pre>+ PRINT_ERR("Error: NUMA config exceeds the actual numa nodes for %s.\n",<o:p></o:p></pre> <pre>+ (isSrcNumaIncorrect && isDstNumaIncorrect) ? "Source & Destination" :<o:p></o:p></pre> <pre>+ (isSrcNumaIncorrect) ? "Source" : "Destination");<o:p></o:p></pre> </blockquote> <pre><o:p> </o:p></pre> <pre>Please don't capitalize the first letter of "Source" and "Destination"<o:p></o:p></pre> </blockquote> Can you please explain why?<o:p></o:p> <o:p> </o:p> <blockquote style="margin-top:5.0pt;margin-bottom:5.0pt"> <pre><o:p> </o:p></pre> <pre><o:p> </o:p></pre> <pre>Thanks<o:p></o:p></pre> <pre><o:p> </o:p></pre> <blockquote style="margin-top:5.0pt;margin-bottom:5.0pt"> <pre> return -1;<o:p></o:p></pre> <pre> }<o:p></o:p></pre> <pre><o:p> </o:p></pre> <pre><o:p> </o:p></pre> </blockquote> </blockquote> </div> </div> </body> </html>