[EXTERNAL] Re: [v25,13/13] compress/zsda: add zsda compressdev capabilities

Akhil Goyal gakhil at marvell.com
Thu Jan 23 07:27:01 CET 2025 

> Hi, Akhil:
> 
> There are warning and some errors in the patches.
> 
> 
> The warning is
> 
> >_coding style issues_
> >
> >
> >__rte_packed_begin and __rte_packed_end should always be used in pairs.
> 
> And the context in the patch is：
> 
> > struct __rte_packed_begin zsda_admin_req {
> >     uint16_t msg_type;
> >     uint8_t data[26];
> >@@ -105,10 +114,30 @@  struct zsda_qp_stat {
> >     uint64_t dequeue_err_count;
> > };
> 
> It's the locating information for the modified code.
> So, the warning may be caused by a bug of checkpatches.sh
> 
@Andre Muezerie Can you please look into this checkpatch issue for rte_packet_begin/end?
I think instead of doing grep on patch, it should grep the complete file.


> 
> The errors are:
> 
> > ci/loongarch-unit-testing    fail    Unit Testing FAIL
> > 43/119 DPDK:fast-tests / eventdev_selftest_sw                    TIMEOUT        80.08s
> exit status -15
> 
> and
> 
> > <!doctype html>
> > <html lang="en">
> > <head>
> >   <title>Not Found</title>
> > </head>
> > <body>
> >   <h1>Not Found</h1><p>The requested resource was not found on this
> server.</p>
> > </body>
> > </html>
> 
> For the both errors, I have no idea about the cause of the problem and how to
> solve it.
> I even don't modify the code about eventdev.
> 
> Thanks.
>

More information about the dev mailing list