Re: [dpdk-dev] [PATCH] e1000/base: fix the wrong assignment to msgbuf[0]

2017-02-14 Thread Thomas Monjalon
2017-02-14 04:14, Yong Wang: > In function e1000_update_mc_addr_list_vf(), "msgbuf[0]" is used prior > to initialization at "msgbuf[0] |= E1000_VF_SET_MULTICAST_OVERFLOW". > And "msgbuf[0]" is overwritten at "msgbuf[0] = E1000_VF_SET_MULTICAST". > Fix it by moving the second line prior to the first

Re: [dpdk-dev] [PATCH] e1000/base: fix the wrong assignment to msgbuf[0]

2017-02-14 Thread Lu, Wenzhuo
Hi Yong, > -Original Message- > From: Yong Wang [mailto:wang.yon...@zte.com.cn] > Sent: Tuesday, February 14, 2017 5:14 PM > To: Lu, Wenzhuo > Cc: dev@dpdk.org; Yong Wang > Subject: [PATCH] e1000/base: fix the wrong assignment to msgbuf[0] > > In function e1000_update_mc_addr_list_vf(), "

[dpdk-dev] [PATCH] e1000/base: fix the wrong assignment to msgbuf[0]

2017-02-14 Thread Yong Wang
In function e1000_update_mc_addr_list_vf(), "msgbuf[0]" is used prior to initialization at "msgbuf[0] |= E1000_VF_SET_MULTICAST_OVERFLOW". And "msgbuf[0]" is overwritten at "msgbuf[0] = E1000_VF_SET_MULTICAST". Fix it by moving the second line prior to the first one that mentioned above. Signed-of