drivers/net/xen-netfront.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-)
The original comments contained spelling errors and incomplete logical
descriptions, which could easily lead to misunderstandings of the code
logic. The specific modifications are as follows:
Correct the spelling error by changing "inut max" to "but not exceed the
maximum limit";
Add the note "If the user has not specified a value, the default maximum
limit is 8" to clarify the default value logic;
Improve the coherence of the statement to make the queue quantity rules
clearer.
After the modification, the comments can accurately reflect the code
behavior of "taking the smaller value between the number of CPUs and the
default maximum limit of 8 for the number of queues", enhancing code
maintainability.
Signed-off-by: Chu Guangqing <chuguangqing@inspur.com>
---
drivers/net/xen-netfront.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/drivers/net/xen-netfront.c b/drivers/net/xen-netfront.c
index a11a0e949400..7c2220366623 100644
--- a/drivers/net/xen-netfront.c
+++ b/drivers/net/xen-netfront.c
@@ -2696,8 +2696,9 @@ static int __init netif_init(void)
pr_info("Initialising Xen virtual ethernet driver\n");
- /* Allow as many queues as there are CPUs inut max. 8 if user has not
- * specified a value.
+ /* Allow the number of queues to match the number of CPUs, but not exceed
+ * the maximum limit. If the user has not specified a value, the default
+ * maximum limit is 8.
*/
if (xennet_max_queues == 0)
xennet_max_queues = min_t(unsigned int, MAX_QUEUES_DEFAULT,
--
2.43.7
Hello:
This patch was applied to netdev/net-next.git (main)
by Jakub Kicinski <kuba@kernel.org>:
On Mon, 3 Nov 2025 11:22:12 +0800 you wrote:
> The original comments contained spelling errors and incomplete logical
> descriptions, which could easily lead to misunderstandings of the code
> logic. The specific modifications are as follows:
>
> Correct the spelling error by changing "inut max" to "but not exceed the
> maximum limit";
>
> [...]
Here is the summary with links:
- xen/netfront: Comment Correction: Fix Spelling Error and Description of Queue Quantity Rules
https://git.kernel.org/netdev/net-next/c/52665fcc2241
You are awesome, thank you!
--
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html
On 03.11.25 04:22, Chu Guangqing wrote: > The original comments contained spelling errors and incomplete logical > descriptions, which could easily lead to misunderstandings of the code > logic. The specific modifications are as follows: > > Correct the spelling error by changing "inut max" to "but not exceed the > maximum limit"; > > Add the note "If the user has not specified a value, the default maximum > limit is 8" to clarify the default value logic; > > Improve the coherence of the statement to make the queue quantity rules > clearer. > > After the modification, the comments can accurately reflect the code > behavior of "taking the smaller value between the number of CPUs and the > default maximum limit of 8 for the number of queues", enhancing code > maintainability. > > Signed-off-by: Chu Guangqing <chuguangqing@inspur.com> Reviewed-by: Juergen Gross <jgross@suse.com> Juergen
© 2016 - 2025 Red Hat, Inc.