On 2023/10/12 14:50, SAITOH Masanobu wrote:
> Module Name: src
> Committed By: msaitoh
> Date: Thu Oct 12 05:50:56 UTC 2023
>
> Modified Files:
> src/sys/dev/pci/ixgbe: ixgbe.c
>
> Log Message:
> ixg(4): Don't print wrong error message about ixgbe_num_queues.
>
> Don't override the ixgbe_num_queues global variable. It's the default
> value of the number of queues and should not override it because it
> will be referenced by later device attach. For example, the number of
> MSI-X vector is 64 on X540 and 18 on 82599. When both cards are inserted
> to a machine that the number of CPU is 24 and X540 is probed earlier,
> ixgbe_num_queues is overridden to 24 and the following error message is
> printed when attaching 82599:
>
> ixg2: autoconfiguration error: ixgbe_num_queues (24) is too large,
> using reduced amount (17).
>
> Note that the number of queues is in sc->num_queuss and referenced
> by hw.ixgN.num_queues sysctl.
The commit message was incorrect.
- s/82599/82598/
- Worse thing can happen if a smaller number of MSI-X vector's device is
attached earlier. The small number is set as the default value and the
number of queues of the next device is unintentionally limited to it.
> To generate a diff of this commit:
> cvs rdiff -u -r1.341 -r1.342 src/sys/dev/pci/ixgbe/ixgbe.c
>
> Please note that diffs are not public domain; they are subject to the
> copyright notices on the relevant files.
>
--
-----------------------------------------------
SAITOH Masanobu (msai...@execsw.org
msai...@netbsd.org)
