Date: Fri, 28 Feb 2025 08:16:03 +0000 From: "kevlo (Kevin Lo)" <phabric-noreply@FreeBSD.org> To: Phabricator <phabric-noreply@FreeBSD.org> Cc: freebsd-net@freebsd.org Subject: [Differential] D49156: Fix a logic error in ixgbe_read_mailbox_vf() Message-ID: <4e79c3212deba281b6be72a02be3393a@localhost.localdomain> In-Reply-To: <differential-rev-PHID-DREV-ja6gkbasd6v4f4bpf67k-req@reviews.freebsd.org> References: <differential-rev-PHID-DREV-ja6gkbasd6v4f4bpf67k-req@reviews.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
[-- Attachment #1 --] This revision was not accepted when it landed; it landed in state "Needs Review". This revision was automatically updated to reflect the committed changes. Closed by commit rG5c7087c349fc: ixgbe: Fix a logic error in ixgbe_read_mailbox_vf() (authored by kevlo). REPOSITORY rG FreeBSD src repository CHANGES SINCE LAST UPDATE https://reviews.freebsd.org/D49156?vs=151619&id=151664 CHANGES SINCE LAST ACTION https://reviews.freebsd.org/D49156/new/ REVISION DETAIL https://reviews.freebsd.org/D49156 AFFECTED FILES sys/dev/ixgbe/ixgbe_mbx.c CHANGE DETAILS diff --git a/sys/dev/ixgbe/ixgbe_mbx.c b/sys/dev/ixgbe/ixgbe_mbx.c --- a/sys/dev/ixgbe/ixgbe_mbx.c +++ b/sys/dev/ixgbe/ixgbe_mbx.c @@ -297,7 +297,7 @@ u32 vf_mailbox = IXGBE_READ_REG(hw, IXGBE_VFMAILBOX); vf_mailbox |= hw->mbx.vf_mailbox; - hw->mbx.vf_mailbox |= vf_mailbox % IXGBE_VFMAILBOX_R2C_BITS; + hw->mbx.vf_mailbox |= vf_mailbox & IXGBE_VFMAILBOX_R2C_BITS; return vf_mailbox; } EMAIL PREFERENCES https://reviews.freebsd.org/settings/panel/emailpreferences/ To: kevlo, kbowling, O11 Intel Networking group Cc: imp, ae, freebsd-net-list, erj, pstef, krzysztof.galazka_intel.com [-- Attachment #2 --] diff --git a/sys/dev/ixgbe/ixgbe_mbx.c b/sys/dev/ixgbe/ixgbe_mbx.c --- a/sys/dev/ixgbe/ixgbe_mbx.c +++ b/sys/dev/ixgbe/ixgbe_mbx.c @@ -297,7 +297,7 @@ u32 vf_mailbox = IXGBE_READ_REG(hw, IXGBE_VFMAILBOX); vf_mailbox |= hw->mbx.vf_mailbox; - hw->mbx.vf_mailbox |= vf_mailbox % IXGBE_VFMAILBOX_R2C_BITS; + hw->mbx.vf_mailbox |= vf_mailbox & IXGBE_VFMAILBOX_R2C_BITS; return vf_mailbox; }
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?4e79c3212deba281b6be72a02be3393a>