[dpdk-dev] [PATCH] ethdev: fix wrong memset

Remy Horton remy.horton at intel.com
Tue Jan 24 09:29:59 CET 2017

On 23/01/2017 11:56, Yuanhan Liu wrote:
>>> http://dpdk.org/ml/archives/dev/2017-January/054422.html
>> Yes, it should fix that issue.
> Well, few more thoughts: it may fix the crash issue Remy saw, but it
> looks like more a workaround to me. Basically, if primary and secondary
> shares a same port id, they should point to same device. Otherwise,
> primary process may use eth_dev->data for a device A, while the
> secondary process may use it for another device, as you said, it
> could be a vdev.
> In such case, there is no way we could continue safely. That said,
> the given patch avoids the total reset of eth_dev->data, while it
> continues reset the eth_dev->data->name, which is wrong.

I did wonder whether 7f95f78a8aea ought to be rolled back rather than 
the memset being made process-conditional. You going to be fixing the 
issue in your own patch?

>> One question: do Remy or you regularly
>> run some multiple process test cases (and with vdev both in primary
>> and secondary process)?

Not aware of there being any multiproc-related unit tests.


More information about the dev mailing list