Re: [PATCH] examples/dma: fix max-frame-size cannot be zero

05/03/2024 13:52, Thomas Monjalon: > 05/03/2024 12:42, fengchengwen: > > Hi Thomas, > > > > This commit fix bug "Bug 1387 - [dpdk24.03] cbdma: Failed to launch > > dpdk-dma app" [1] > > > > Should I send v2 to add the following line in commit log? > > Bugzilla ID: 1387 > > I can add it while me

Re: [PATCH] examples/dma: fix max-frame-size cannot be zero

05/03/2024 12:42, fengchengwen: > Hi Thomas, > > This commit fix bug "Bug 1387 - [dpdk24.03] cbdma: Failed to launch dpdk-dma > app" [1] > > Should I send v2 to add the following line in commit log? > Bugzilla ID: 1387 I can add it while merging.

Re: [PATCH] examples/dma: fix max-frame-size cannot be zero

Original Message- >> From: Chengwen Feng >> Sent: Tuesday, February 20, 2024 10:32 AM >> To: tho...@monjalon.net; dev@dpdk.org; Jiang, YuX ; >> Richardson, Bruce ; Laatz, Kevin >> >> Subject: [PATCH] examples/dma: fix max-frame-size cannot be zero >> >

RE: [PATCH] examples/dma: fix max-frame-size cannot be zero

> -Original Message- > From: Chengwen Feng > Sent: Tuesday, February 20, 2024 10:32 AM > To: tho...@monjalon.net; dev@dpdk.org; Jiang, YuX ; > Richardson, Bruce ; Laatz, Kevin > > Subject: [PATCH] examples/dma: fix max-frame-size cannot be zero > > In the o

[PATCH] examples/dma: fix max-frame-size cannot be zero

In the original implementation, the max_frame_size could be zero, but commit ("examples/dma: replace getopt with argparse") treat zero as an error. This commit fixes it. Also, since unsigned doesn't < 0, adjust "<= 0" judgement to "== 0". Fixes: 8d85afb19af7 ("examples/dma: replace getopt with ar