[src/trunk]: src/sys/dev/pci/ixgbe Other than IXGBE_VF_RESET should wait ACK, ...

[msaitoh <msaitoh%NetBSD.org@localhost>]

details:   https://anonhg.NetBSD.org/src/rev/8c8093e59ee2
branches:  trunk
changeset: 452387:8c8093e59ee2
user:      msaitoh <msaitoh%NetBSD.org@localhost>
date:      Tue Jul 02 04:09:03 2019 +0000

description:
Other than IXGBE_VF_RESET should wait ACK, so use ixgbevf_write_msg_read_ack()
instead of write_posted() in ixgbe_update_mc_addr_list_vf().

diffstat:

 sys/dev/pci/ixgbe/ixgbe_vf.c |  7 +++----
 1 files changed, 3 insertions(+), 4 deletions(-)

diffs (28 lines):

diff -r becc151e7a06 -r 8c8093e59ee2 sys/dev/pci/ixgbe/ixgbe_vf.c
--- a/sys/dev/pci/ixgbe/ixgbe_vf.c      Mon Jul 01 21:06:47 2019 +0000
+++ b/sys/dev/pci/ixgbe/ixgbe_vf.c      Tue Jul 02 04:09:03 2019 +0000
@@ -1,4 +1,4 @@
-/* $NetBSD: ixgbe_vf.c,v 1.17 2018/04/04 08:59:22 msaitoh Exp $ */
+/* $NetBSD: ixgbe_vf.c,v 1.18 2019/07/02 04:09:03 msaitoh Exp $ */
 
 /******************************************************************************
   SPDX-License-Identifier: BSD-3-Clause
@@ -388,7 +388,6 @@
                                 u32 mc_addr_count, ixgbe_mc_addr_itr next,
                                 bool clear)
 {
-       struct ixgbe_mbx_info *mbx = &hw->mbx;
        u32 msgbuf[IXGBE_VFMAILBOX_SIZE];
        u16 *vector_list = (u16 *)&msgbuf[1];
        u32 vector;
@@ -419,8 +418,8 @@
                DEBUGOUT1("Hash value = 0x%03X\n", vector);
                vector_list[i] = (u16)vector;
        }
-
-       return mbx->ops.write_posted(hw, msgbuf, IXGBE_VFMAILBOX_SIZE, 0);
+       return ixgbevf_write_msg_read_ack(hw, msgbuf, msgbuf,
+           IXGBE_VFMAILBOX_SIZE);
 }
 
 /**