Re: [dpdk-dev] [PATCH v2] e1000/base: fix multicast setting in VF

2017-02-24 Thread Ferruh Yigit
On 2/24/2017 12:32 AM, Lu, Wenzhuo wrote: > Hi, > >> -Original Message- >> From: Yong Wang [mailto:wang.yon...@zte.com.cn] >> Sent: Tuesday, February 21, 2017 5:33 PM >> To: Lu, Wenzhuo >> Cc: dev@dpdk.org; Yong Wang >> Subject: [PATCH v2] e1000/base: fix multicast setting in VF >> >> In f

Re: [dpdk-dev] [PATCH v2] e1000/base: fix multicast setting in VF

2017-02-24 Thread Ferruh Yigit
On 2/21/2017 9:33 AM, Yong Wang wrote: > 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

Re: [dpdk-dev] [PATCH v2] e1000/base: fix multicast setting in VF

2017-02-23 Thread Lu, Wenzhuo
Hi, > -Original Message- > From: Yong Wang [mailto:wang.yon...@zte.com.cn] > Sent: Tuesday, February 21, 2017 5:33 PM > To: Lu, Wenzhuo > Cc: dev@dpdk.org; Yong Wang > Subject: [PATCH v2] e1000/base: fix multicast setting in VF > > In function e1000_update_mc_addr_list_vf(), "msgbuf[0]" i

[dpdk-dev] [PATCH v2] e1000/base: fix multicast setting in VF

2017-02-21 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. --- v2: *