Date: Fri, 20 Sep 2024 01:00:42 -0700 From: Kevin Bowling <kevin.bowling@kev009.com> To: Doug Moore <unkadoug@gmail.com> Cc: Yasuhiro Kimura <yasu@freebsd.org>, vishwin@freebsd.org, kbowling@freebsd.org, src-committers@freebsd.org, dev-commits-src-all@freebsd.org, dev-commits-src-main@freebsd.org Subject: Re: git: 10746040820e - main - ixgbe: rename VF message type macros Message-ID: <CAK7dMtBqBSQAu8S_Vu9t1TErcphB6tDj%2BDsm1fHi2yVTfQ1a6w@mail.gmail.com> In-Reply-To: <db02a074-81dd-264a-a9f6-3669b6aa2c1b@gmail.com> References: <202409192103.48JL3ffi022099@gitrepo.freebsd.org> <084fa85b-7112-4dc5-bdb0-dbf727a28af0@freebsd.org> <20240920.154700.1207780392961840745.yasu@FreeBSD.org> <db02a074-81dd-264a-a9f6-3669b6aa2c1b@gmail.com>
next in thread | previous in thread | raw e-mail | index | archive | help
On Thu, Sep 19, 2024 at 11:49=E2=80=AFPM Doug Moore <unkadoug@gmail.com> wr= ote: > > https://reviews.freebsd.org/D46708 Your fix is logically correct, thank you Doug. I committed it with your authorship as I would otherwise make the same change to fix the tree now. Pondering aloud, I am uncertain why this did not affect a modular build, I see if_sriov.c in the ix makefile. > > fixes compilation, if you want to copy it. > > On 9/20/24 01:47, Yasuhiro Kimura wrote: > > From: Charlie Li <vishwin@freebsd.org> > > Subject: Re: git: 10746040820e - main - ixgbe: rename VF message type m= acros > > Date: Thu, 19 Sep 2024 23:59:47 -0400 > > > >> Build fails: > >> /usr/src/sys/dev/ixgbe/if_sriov.c:103:16: error: member reference base > >> type 'struct ixgbe_mbx_operations[64]' is not a structure or union > >> 103 | sc->hw.mbx.ops.write(&sc->hw, &msg, 1, vf->pool); > >> | ~~~~~~~~~~~~~~^~~~~~ > >> /usr/src/sys/dev/ixgbe/if_sriov.c:377:13: error: member reference base > >> type 'struct ixgbe_mbx_operations[64]' is not a structure or union > >> 377 | hw->mbx.ops.write(hw, resp, IXGBE_VF_PERMADDR_MSG_LEN, > >> vf->pool); > >> | ~~~~~~~~~~~^~~~~~ > >> /usr/src/sys/dev/ixgbe/if_sriov.c:568:13: error: member reference base > >> type 'struct ixgbe_mbx_operations[64]' is not a structure or union > >> 568 | hw->mbx.ops.write(hw, resp, IXGBE_VF_GET_QUEUES_RESP_LEN, > >> vf->pool); > >> | ~~~~~~~~~~~^~~~~~ > >> /usr/src/sys/dev/ixgbe/if_sriov.c:585:21: error: member reference base > >> type 'struct ixgbe_mbx_operations[64]' is not a structure or union > >> 585 | error =3D hw->mbx.ops.read(hw, msg, IXGBE_VFMAILBOX_SIZE, > >> vf->pool); > >> | ~~~~~~~~~~~^~~~~ > >> /usr/src/sys/dev/ixgbe/if_sriov.c:646:19: error: member reference base > >> type 'struct ixgbe_mbx_operations[64]' is not a structure or union > >> 646 | if (hw->mbx.ops.check_for_rst(hw, vf->pool) =3D=3D 0) > >> | ~~~~~~~~~~~^~~~~~~~~~~~~~ > >> /usr/src/sys/dev/ixgbe/if_sriov.c:649:19: error: member reference base > >> type 'struct ixgbe_mbx_operations[64]' is not a structure or union > >> 649 | if (hw->mbx.ops.check_for_msg(hw, vf->pool) =3D=3D 0) > >> | ~~~~~~~~~~~^~~~~~~~~~~~~~ > >> /usr/src/sys/dev/ixgbe/if_sriov.c:652:19: error: member reference base > >> type 'struct ixgbe_mbx_operations[64]' is not a structure or union > >> 652 | if (hw->mbx.ops.check_for_ack(hw, vf->pool) =3D=3D 0) > >> | ~~~~~~~~~~~^~~~~~~~~~~~~~ > >> 7 errors generated. > >> *** [if_sriov.o] Error code 1 > > Same here with: > > > > * Commit 420c9844702 > > * amd64 > > * Clean build from scratch > > > > --- > > Yasuhiro Kimura
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAK7dMtBqBSQAu8S_Vu9t1TErcphB6tDj%2BDsm1fHi2yVTfQ1a6w>