[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