Hello, On Mon, Nov 2, 2015 at 8:21 AM, Lu, Wenzhuo <wenzhuo.lu at intel.com> wrote:
> > > @@ -2502,7 +2503,14 @@ static int igbvf_set_vfta(struct e1000_hw *hw, > > uint16_t vid, bool on) > > if (on) > > msgbuf[0] |= E1000_VF_SET_VLAN_ADD; > > > > - return (mbx->ops.write_posted(hw, msgbuf, 2, 0)); > > + mbx->ops.write_posted(hw, msgbuf, 2, 0); > How about if write failed? I think we'd better check it. > Yes, just trying to test this before resubmitting. Thanks. -- David Marchand
