<div dir="ltr"><div dir="ltr"><br></div><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, 14 Jan 2022 at 18:44, Ferruh Yigit <<a href="mailto:ferruh.yigit@intel.com">ferruh.yigit@intel.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On 1/14/2022 4:24 PM, Stephen Hemminger wrote:<br>
> On Fri, 14 Jan 2022 17:18:19 +0200<br>
> Tudor Cornea <<a href="mailto:tudor.cornea@gmail.com" target="_blank">tudor.cornea@gmail.com</a>> wrote:<br>
> <br>
>> +module_param(min_scheduling_interval, long, 0644);<br>
>> +MODULE_PARM_DESC(min_scheduling_interval,<br>
>> +"\t\tKni thread min scheduling interval (default=100 microseconds):\n"<br>
>> +"\t\t"<br>
>> +);<br>
>> +<br>
>> +module_param(max_scheduling_interval, long, 0644);<br>
>> +MODULE_PARM_DESC(max_scheduling_interval,<br>
>> +"\t\tKni thread max scheduling interval (default=200 microseconds):\n"<br>
>> +"\t\t"<br>
>> +);<br>
> <br>
> Please don't add more bad module parameter strings.<br>
> The KNI author did something no other kernel modules do with tabs<br>
> and double spacing, stop this bogus stuff.<br>
> <br>
<br>
The patch is good, let's not block it for the module parameter string,<br>
all can be fixed with another patch.<br>
<br>
Can you please give a sample what is a common way of it, me or Tudor can<br>
do the patch?<br>
<br></blockquote><div><br></div><div>I agree that the module parameter string is in non-standard format.  <br></div><div><div>I was planning to send a follow-up patch, which would correct the description for all of the KNI parameters (including the two new parameters that the current patch would add) in one shot.</div><div> <br></div></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
> Is there any reason you have to use KNI at all.<br>
> KNI is broken on many levels and is not fixable.<br>
> What about virtio or tap?<br>
<br></blockquote><div><br></div><div>We've run some tests with tap interfaces and found the performance to not be good enough for our use.<br></div><div>We're going to experiment with virtio_user in the future. I'm aware that there is a long term plan to deprecate the KNI.</div><div><br></div><div><br></div><div> </div></div></div>