DPDK CI discussions
 help / color / mirror / Atom feed
* Re: [dpdk-ci] [dpdk-dev] [PATCH v6 0/3] eal: add memory pre-allocation from existing files
       [not found] ` <20211011085644.2716490-1-dkozlyuk@nvidia.com>
@ 2021-10-11 18:52   ` Thomas Monjalon
  2021-10-11 21:12     ` Lincoln Lavoie
  0 siblings, 1 reply; 3+ messages in thread
From: Thomas Monjalon @ 2021-10-11 18:52 UTC (permalink / raw)
  To: Dmitry Kozlyuk; +Cc: dev, ci

11/10/2021 10:56, Dmitry Kozlyuk:
> v6: fix copyright line (Thomas), add SPDX header for the new test file
>     (BTW, why didn't the CI complain in previous versions?)

Probably because the CI doesn't run the script devtools/check-spdx-tag.sh
Cc ci@dpdk.org to add this test.




^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [dpdk-ci] [dpdk-dev] [PATCH v6 0/3] eal: add memory pre-allocation from existing files
  2021-10-11 18:52   ` [dpdk-ci] [dpdk-dev] [PATCH v6 0/3] eal: add memory pre-allocation from existing files Thomas Monjalon
@ 2021-10-11 21:12     ` Lincoln Lavoie
  2021-10-12  6:54       ` Thomas Monjalon
  0 siblings, 1 reply; 3+ messages in thread
From: Lincoln Lavoie @ 2021-10-11 21:12 UTC (permalink / raw)
  To: Thomas Monjalon; +Cc: Dmitry Kozlyuk, dev, ci

[-- Attachment #1: Type: text/plain, Size: 734 bytes --]

Hi All,

I'm assuming this should be added to the "checkpatch" job / report?
Correct, or do folks feel this should be a separate run / report?

Cheers,
Lincoln

On Mon, Oct 11, 2021 at 2:52 PM Thomas Monjalon <thomas@monjalon.net> wrote:

> 11/10/2021 10:56, Dmitry Kozlyuk:
> > v6: fix copyright line (Thomas), add SPDX header for the new test file
> >     (BTW, why didn't the CI complain in previous versions?)
>
> Probably because the CI doesn't run the script devtools/check-spdx-tag.sh
> Cc ci@dpdk.org to add this test.
>
>
>
>

-- 
*Lincoln Lavoie*
Principal Engineer, Broadband Technologies
21 Madbury Rd., Ste. 100, Durham, NH 03824
lylavoie@iol.unh.edu
https://www.iol.unh.edu
+1-603-674-2755 (m)
<https://www.iol.unh.edu>

[-- Attachment #2: Type: text/html, Size: 2089 bytes --]

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [dpdk-ci] [dpdk-dev] [PATCH v6 0/3] eal: add memory pre-allocation from existing files
  2021-10-11 21:12     ` Lincoln Lavoie
@ 2021-10-12  6:54       ` Thomas Monjalon
  0 siblings, 0 replies; 3+ messages in thread
From: Thomas Monjalon @ 2021-10-12  6:54 UTC (permalink / raw)
  To: Lincoln Lavoie; +Cc: Dmitry Kozlyuk, dev, ci

11/10/2021 23:12, Lincoln Lavoie:
> Hi All,
> 
> I'm assuming this should be added to the "checkpatch" job / report?

I think yes.

> Correct, or do folks feel this should be a separate run / report?

Please what are the tests already run in this job?
I may suggest more.


> On Mon, Oct 11, 2021 at 2:52 PM Thomas Monjalon <thomas@monjalon.net> wrote:
> > 11/10/2021 10:56, Dmitry Kozlyuk:
> > > v6: fix copyright line (Thomas), add SPDX header for the new test file
> > >     (BTW, why didn't the CI complain in previous versions?)
> >
> > Probably because the CI doesn't run the script devtools/check-spdx-tag.sh
> > Cc ci@dpdk.org to add this test.




^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2021-10-12  6:54 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <20210921081632.858873-1-dkozlyuk@nvidia.com>
     [not found] ` <20211011085644.2716490-1-dkozlyuk@nvidia.com>
2021-10-11 18:52   ` [dpdk-ci] [dpdk-dev] [PATCH v6 0/3] eal: add memory pre-allocation from existing files Thomas Monjalon
2021-10-11 21:12     ` Lincoln Lavoie
2021-10-12  6:54       ` Thomas Monjalon

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).