On Thu, Mar 03, 2016 at 03:51:25AM +0000, Wang, Xiao W wrote: > Hi, > > > > > + > > > > + err = mbx->ops.write_posted(hw, msgbuf, 2, 0); > > > > + if (err) > > > > + return err; > > > > + > > > > + err = mbx->ops.read_posted(hw, msgbuf, 2, 0); > > > > > > Is it more reasonable to read a message of size 1 than 2? Pf side only > > > write 1 word into mbx. > > Thanks for the comment. But actually PF writes 16 words into the mbx, and 2 > > words have meaning. > > Word0 is used to check ack/nack. PF uses word1 to write the xcast_mode back. > > I don't check the word1 because I don't see the necessary:) > > > > > > > OK, I learned that kernel PF put 2 words into mbx for this message. > > Acked-by: Xiao Wang <xiao.w.wang at intel.com>
Applied to dpdk-next-net/rel_16_04 /Bruce
