[PATCH 2/2] app/testpmd: fix stack overflow for EEPROM display

Ferruh Yigit ferruh.yigit at intel.com
Thu Feb 3 14:15:55 CET 2022


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.yang at intel.com>
> 
> Acked-by: Aman Singh <aman.deep.singh at intel.com>
> 

Tested-by: Ferruh Yigit <ferruh.yigit at intel.com>

Applied to dpdk-next-net/main, thanks.


More information about the dev mailing list