On 2/2/2022 10:00 AM, Singh, Aman Deep wrote:
Hi Steve
The patch looks ok to me.
On 1/20/2022 8:29 AM, Steve Yang wrote:
When the size of EEPROM exceeds the default thread stack size(8MB),
e.g.: 10Mb size, it will be cashed with stack overflow.
Just spelling: 10Mb/10MB, cashed/crashed
Can even rephrase, like: it will crash due to stack overflow
fixed while merging
Allocate the data of EPPROM information on the heap.
Fixes: 6b67721dee2a ("app/testpmd: add EEPROM command")
Signed-off-by: Steve Yang<stevex.y...@intel.com>
Acked-by: Aman Singh <aman.deep.si...@intel.com>
Tested-by: Ferruh Yigit <ferruh.yi...@intel.com>
Applied to dpdk-next-net/main, thanks.
