<meta http-equiv="Content-Type" content="text/html; charset=GB18030"><div><div style="  ; ; ;; ">> nit: Commit message could be better: shorter subject (first line)nit: Commit message could be better: shorter subject (first line)<br>> and more descriptive description after single blank line.</div><div style="  ; ; ;; ">> <span style="font-family: Verdana;">That would help me avoiding asking question bellow.</span><br style="font-family: Verdana;">> <br style="font-family: Verdana;">> <span style="font-family: Verdana;">> From: Jun Huang <huangjuniter@foxmail.com></span><br style="font-family: Verdana;">> <br style="font-family: Verdana;">> <span style="font-family: Verdana;">nit: there should have been your Signed-off-by:</span><br style="font-family: Verdana;">> <br style="font-family: Verdana;">> <span style="font-family: Verdana;">Signed-off-by: Jun Huang <huangjuniter@foxmail.com></span></div><div style="  ; ; ;; "><br></div><div style="  ; ; ;; position: relative; "><div>Thanks for your suggestion!</div><div><br></div><div>> I was going to merge it, but is it really needed? Unsorted input to unique will<br>> can lead to higher number of count, but because anything higher than 1 results<br>> in echo 0 and 0 will result in 1 due this code:<br>><br>> [ $no_of_sockets -gt 1 ] ; echo $?<br>><br>> i.e. 2 or more different files will result in count higher than 1 regardless<br>> using sorted uniq (sort -u) or just uniq. Or am I missing something?</div><div><br></div><div>The main purpose of this patch is to change 'cpu?' to 'cpu*', otherwise an error</div><div>will occur when the number of cores exceeds 10.</div><div>As you said 'sort -u' has the same effect as 'uniq'. I suggest using 'sort -u'</div><div>to make the command result more in line with the naming of 'no_of_sockets'.</div></div></div><div style="position: relative;"><br></div><div style="position: relative;">Thanks,</div><div style="position: relative;">Jun.</div><div style="position: relative;"><br></div>