Hi all,
We are shutting down 0day-bot for today for a system update. We will be
back online later by the end of the day EST.
Best
On 10/13/21 4:03 AM, Serena He wrote:
CI jobs are triggered only for repos installed with given GHApp and runners
Cc: sta...@dpdk.org
Signed-off-by: Serena He
---
.github/workflows/build-arm64.yml | 118 ++
1 file changed, 118 insertions(+)
create mode 1006
Hi all,
We are shutting down 0day-bot for scheduled electrical work in the lab. We
will be back online next week the latest
Best
Hi all,
We are planning on doing a system upgrade on our OVSCI automation
system this Friday 9/9/22.
This will affect our upstream robots (i.e. ovs, ovn, dpdk). This
affects patchwork sync and github actions sync, etc.
After the upgrade is completed the robots will be started again and
expected t
il to the test-report
list informing anyone interested in the automated build (including a
result).
Signed-off-by: Aaron Conole
Signed-off-by: Michael Santana
---
.ci/linux-build.sh | 34 +
.ci/linux-setup.sh | 3 +++
.travi
From: Aaron Conole
The vhost_scsi example code is set to build, even if the requisite header
file virtio_scsi.h isn't available. This happens on some Ubuntu systems
when some versions of the libc-dev package aren't available.
Check whether the virtio_scsi.h file exists, and if not, set the buil
This series introduces the ability for any github mirrors of the DPDK
project, including developer mirrors, to kick off builds under the
travis CI infrastructure. For now, this just means compilation - no
other kinds of automated run exists yet. In the future, this can be
expanded to execute and
il to the test-report
list informing anyone interested in the automated build (including a
result).
Signed-off-by: Aaron Conole
Signed-off-by: Michael Santana
---
v2:
- Added aarch64 build
- Added multiple meson options for builds
- Added multiple make/config options for builds
.ci/linux-bui
lready-defined variable
v1 -> v2:
- Added patch 1/2, "examples/vhost_scsi: don't build..."
- Included arm64 builds
- Included multiple meson+ninja builds (full library, minimal library)
- Included multiple 'classic make' builds
Aaron Conole (1):
examples/vhos
From: Aaron Conole
The vhost_scsi example code is set to build, even if the requisite header
file virtio_scsi.h isn't available. This happens on some Ubuntu systems
when some versions of the libc-dev package aren't available.
Check whether the virtio_scsi.h file exists, and if not, set the buil
il to the test-report
list informing anyone interested in the automated build (including a
result).
Signed-off-by: Aaron Conole
Signed-off-by: Michael Santana
---
v3:
- Renamed ambiguous variable names and comments, including the variable
KERNEL to DISABLE_KERNEL_MODULES and comment '
- Included multiple meson+ninja builds (full library, minimal library)
- Included multiple 'classic make' builds
Aaron Conole (1):
examples/vhost_scsi: Don't build without virtio_scsi.h
Michael Santana (1):
ci: Introduce travis builds for github repositories
.ci/linu
From: Aaron Conole
The vhost_scsi example code is set to build, even if the requisite header
file virtio_scsi.h isn't available. This happens on some Ubuntu systems
when some versions of the libc-dev package aren't available.
Check whether the virtio_scsi.h file exists, and if not, set the buil
il to the test-report
list informing anyone interested in the automated build (including a
result).
Signed-off-by: Aaron Conole
Signed-off-by: Michael Santana
Acked-by: Bruce Richardson
---
v4:
- Remove non-existing file form maintainers list:
meson_cross_aarch64_gcc.txt
- Renamed ARCH
From: Aaron Conole
The vhost_scsi example code is set to build, even if the requisite header
file virtio_scsi.h isn't available. This happens on some Ubuntu systems
when some versions of the libc-dev package aren't available.
Check whether the virtio_scsi.h file exists, and if not, set the buil
7;t build..."
- Included arm64 builds
- Included multiple meson+ninja builds (full library, minimal library)
- Included multiple 'classic make' builds
Aaron Conole (1):
examples/vhost_scsi: Don't build without virtio_scsi.h
Michael Santana (1):
ci: Introduce travis bu
il to the test-report
list informing anyone interested in the automated build (including a
result).
Signed-off-by: Aaron Conole
Signed-off-by: Michael Santana
Acked-by: Bruce Richardson
Reviewed-by: Honnappa Nagarahalli
---
v5:
Renamed ARM64 to AARCH64
.ci/linux-build.sh
ariable in
checkpatches.sh. With this change the user can decide to turn
on codespell from a config file rather than directly modifying
checkpatches.sh
Signed-off-by: Michael Santana
---
devtools/checkpatches.sh | 4
1 file changed, 4 insertions(+)
diff --git a/devtools/checkpatches.sh
Fixed a minor bug with variable assignment, as well as added an
option for checkpatches
Michael Santana (2):
Enable codespell from config file.
Fix variable assignment.
devtools/checkpatches.sh | 6 +-
1 file changed, 5 insertions(+), 1 deletions(-)
--
2.20.1
Fix trivial bug. In sh shell, 'foo = 1' is not the same as
'foo=1'. Using 'foo = 1' makes the shell attempt to interpret foo
as a command, rather than a simple variable assignment.
Signed-off-by: Michael Santana
Fixes: dafc04c15174 ("hash: fix out-of
Fix trivial bug. In sh shell, 'foo = 1' is not the same as
'foo=1'. Using 'foo = 1' makes the shell attempt to interpret foo
as a command, rather than a simple variable assignment.
Signed-off-by: Michael Santana
Fixes: dafc04c15174 ("hash: fix out-of-bound wr
enabled by default.
The user can decide to turn off codespell from a one of the config
files read by checkpatches.sh.
Signed-off-by: Michael Santana
Reviewed-by: Rami Rosen
---
devtools/checkpatches.sh | 6 ++
1 file changed, 6 insertions(+)
diff --git a/devtools/checkpatches.sh
Fixed a minor bug with variable assignment, as well as added an
option for checkpatches
v1->v2:
Enable codespell by default. Disable via config file
Michael Santana (2):
Enable codespell by default. Can be disabled from config file.
Fix variable assignment.
devtools/checkpatches.sh
Fixed a minor bug with variable assignment, as well as added an
option for checkpatches
v2->v3:
Also enable codespell by setting a path to a dictionary file.
v1->v2:
Enable codespell by default. Disable via config file
Michael Santana (2):
Enable codespell by default. Can be disable
enabled by default.
The user can decide to turn off codespell from a one of the config
files read by checkpatches.sh.
Signed-off-by: Michael Santana
Reviewed-by: Rami Rosen
---
v2->v3:
Also enable codespell by setting a path to a dictionary file.
devtools/checkpatches.sh | 11 +++
1
Fix trivial bug. In sh shell, 'foo = 1' is not the same as
'foo=1'. Using 'foo = 1' makes the shell attempt to interpret foo
as a command, rather than a simple variable assignment.
Signed-off-by: Michael Santana
Fixes: dafc04c15174 ("devtools: fix
patch 1/2, "examples/vhost_scsi: don't build..."
- Included arm64 builds
- Included multiple meson+ninja builds (full library, minimal library)
- Included multiple 'classic make' builds
Michael Santana (1):
ci: Introduce travis builds for github repositories
il to the test-report
list informing anyone interested in the automated build (including a
result).
Signed-off-by: Aaron Conole
Signed-off-by: Michael Santana
---
v6:
Removed all classic make builds
.ci/linux-build.sh | 21 +
.ci/linux-setup.sh
Fixed a minor bug with variable assignment, as well as added an
option for checkpatches
v3->v4:
Support enabling codespell via environment variable.
v2->v3:
Also enable codespell by setting a path to a dictionary file.
v1->v2:
Enable codespell by default. Disable via config file
enabled by default.
The user can decide to turn off codespell from a one of the config
files read by checkpatches.sh.
Signed-off-by: Michael Santana
Reviewed-by: Rami Rosen
---
v3->v4:
Support enabling codespell via environment variable.
devtools/checkpatches.sh | 11 +++
1 fil
uot;)
Cc: sta...@dpdk.org
Signed-off-by: Michael Santana
---
devtools/checkpatches.sh | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/devtools/checkpatches.sh b/devtools/checkpatches.sh
index 7e862e0a4..521534fce 100755
--- a/devtools/checkpatches.sh
+++ b/devtools/checkpatches.sh
@@
The files added under .ci/ exist so that in the future, other CI
support platforms (such as cirrus, appveyor, etc.) could have a common
place to put their requisite scripts without polluting the main tree.
Signed-off-by: Aaron Conole
Signed-off-by: Michael Santana
Acked-by: Bruce Richardson
Ack
The files added under .ci/ exist so that in the future, other CI
support platforms (such as cirrus, appveyor, etc.) could have a common
place to put their requisite scripts without polluting the main tree.
Signed-off-by: Aaron Conole
Signed-off-by: Michael Santana
Acked-by: Bruce Richardson
Ack
increasing the time out
for this test seems appropriate as sometimes it halts at exactly 30s
because of the time out.
Acked-by: Michael Santana
On 1/28/2020 11:28 AM, Ferruh Yigit wrote:
> Timeout multiplier was 3, which gives 30 seconds for unit test but still
> some unit test was timi
This patch reduces the memory footprint of tables for the unit test.
Lower memory footprint means the test now passes when trying to allocate
the tables.
Signed-off-by: Michael Santana
---
app/test/test_table_tables.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/app
This patch reduces the memory footprint of tables for the unit test.
Lower memory footprint means the test now passes when trying to allocate
the tables. This patch is mainly to make table_autotest pass in CI.
Signed-off-by: Michael Santana
Acked-by: Cristian Dumitrescu
---
app/test
Change email address for Michael Santana to personal email in the
MAINTAINERS file
Signed-off-by: Michael Santana
Signed-off-by: Michael Santana
---
NOTE: My internship at Red Hat is ending soon so I will not be able
to monitor my red hat email. I will not monitor the mailing list
as much, so
g the last character in the buffer. 'sizeof(buffer)' is
enough.
Cc: sta...@dpdk.org
Signed-off-by: Michael Santana
---
drivers/net/qede/base/bcm_osal.c| 4 ++--
drivers/net/qede/qede_filter.c | 2 +-
drivers/net/vdev_
/205045946#L645
https://travis-ci.com/Maickii/dpdk-2/jobs/205045946#L671
SKIP:
https://travis-ci.com/Maickii/dpdk-2/jobs/205047293#L645
https://travis-ci.com/Maickii/dpdk-2/jobs/205047293#L671
Signed-off-by: Michael Santana
---
app/test/test_hash_readwrite.c | 2 +-
app/test/test_rcu_qsbr.c | 4
That's a good suggestion. I will look into it.
~Michael Santana
On Thu, Jan 24, 2019 at 2:27 PM Honnappa Nagarahalli <
honnappa.nagaraha...@arm.com> wrote:
> >
> > GitHub is a service used by developers to store repositories. GitHub
> > provides service inte
On 2/7/19 5:01 PM, Michael Santana wrote:
ping
This series introduces the ability for any github mirrors of the DPDK
project, including developer mirrors, to kick off builds under the
travis CI infrastructure. For now, this just means compilation - no
other kinds of automated run exists yet
On 2/28/19 6:21 AM, Thomas Monjalon wrote:
14/02/2019 20:35, Michael Santana:
Enable codespell by default.
codespell is a feature by checkpatch.pl that
checks for common spelling mistakes in patches.
What is the difference between codespell and spelling.txt included
with checkpatch?
Is it just
On 3/1/19 12:51 PM, Thomas Monjalon wrote:
01/03/2019 18:43, Thomas Monjalon:
01/03/2019 18:08, Michael Santana:
+# Enable codespell by default. This can be overwritten from a config file.
+# You can also enable codespell by setting DPDK_CHECKPATCH_CODESPELL to a
valid path
+# to a
On 3/1/19 4:08 PM, Thomas Monjalon wrote:
01/03/2019 21:24, Michael Santana Francisco:
On 3/1/19 12:51 PM, Thomas Monjalon wrote:
01/03/2019 18:43, Thomas Monjalon:
01/03/2019 18:08, Michael Santana:
+# Enable codespell by default. This can be overwritten from a config file.
+# You can also
On 3/4/19 1:14 PM, Luca Boccassi wrote:
On Mon, 2019-03-04 at 11:12 -0500, Michael Santana wrote:
GitHub is a service used by developers to store repositories. GitHub
provides service integrations that allow 3rd party services to access
developer repositories and perform actions. One of these
Thank you for taking at look at this
On 3/20/19 12:01 PM, Thomas Monjalon wrote:
Hi,
04/03/2019 17:12, Michael Santana:
.ci/linux-build.sh | 21 +
.ci/linux-setup.sh | 3 ++
.travis.yml | 73
On 3/20/19 5:11 PM, Luca Boccassi wrote:
On Wed, 2019-03-20 at 15:28 -0400, Michael Santana Francisco wrote:
+matrix:
+ include:
+ - env: DEF_LIB="static" OPTS="-Denable_kmods=false" AARCH64=1
+compiler: gcc
+addons:
+ apt:
+packages:
+ -
On 7/1/19 2:07 PM, Michael Santana Francisco wrote:
On Mon, Jul 1, 2019 at 6:04 PM Aaron Conole wrote:
- rwlock_autotest and hash_readwrite_lf_autotest are taking a little more
than 10s,
Occasionally the distributor test times out as well. I've moved them as
part of a separate
On Mon, Jul 8, 2019 at 4:18 PM Aaron Conole wrote:
>
> Thomas Monjalon writes:
>
> > Hi please find some comments below:
> >
> > 06/06/2019 13:59, Hari Kumar Vemula:
> >> +++ b/doc/guides/prog_guide/meson_ut.rst
> >> @@ -0,0 +1,151 @@
> >> +.. SPDX-License-Identifier: BSD-3-Clause
> >> +
> >
> >
On 7/26/19 3:09 AM, David Marchand wrote:
Hello Michael,
On Thu, Jul 25, 2019 at 11:31 PM Michael Santana wrote:
This patch reduces the memory footprint of tables for the unit test.
Lower memory footprint means the test now passes when trying to allocate
the tables.
This is mainly for tests
On 7/26/19 10:06 AM, David Marchand wrote:
Sort the experimental symbols per release to make it easier/quicker to
check for how long we have them.
Signed-off-by: David Marchand
---
Changelog since v1:
- rte_service symbols who got promoted to stable got reintroduced in
experimental when reba
habetical order per release
>
> Changelog since v1:
> - rte_service symbols who got promoted to stable got reintroduced in
> experimental when rebasing this patch on master
>
Acked-by: Michael Santana
> ---
> lib/librte_eal/rte_eal_version.map | 115
> +++
On Wed, Jul 31, 2019 at 2:56 PM David Marchand
wrote:
>
> Following removal of -c and -n options, the array should have been
> shrunk to avoid launch_proc to access unitialised strings.
>
> Fixes: b4dbacdb1ad7 ("test/eal: set core mask/list config only in dedicated
> test")
> Fixes: 501fa9a40298
On Wed, Jul 31, 2019 at 10:50 AM Aaron Conole wrote:
>
> When building under Travis (or another linux CI service), enable running the
> fast-tests for selected builds. Only the shared builds are enabled at this
> point, since they are the ones passing. Builds that are statically linked
> still s
On Fri, Aug 2, 2019 at 9:34 AM Aaron Conole wrote:
>
> Michael Santana Francisco writes:
>
> > On Wed, Jul 31, 2019 at 10:50 AM Aaron Conole wrote:
> >>
> >> When building under Travis (or another linux CI service), enable running
> >> the
> >&g
On Thu, Aug 1, 2019 at 3:25 AM David Marchand wrote:
>
> On Wed, Jul 31, 2019 at 9:57 PM Michael Santana Francisco
> wrote:
> >
> > On Wed, Jul 31, 2019 at 2:56 PM David Marchand
> > wrote:
> > >
> > > Following removal of -c and -n options, the
lds which execute the unit tests may be distinguished by the
> > > > "RUN_TESTS" parameter.
> > > >
> > > > Aaron Conole (1):
> > > > ci: enable unit tests under travis-ci
> > > >
> > > > Michael Santana (1):
>
system. An example run is available
> > > at:
> > >
> > > https://travis-ci.org/orgcandman/dpdk/builds/565991679
> > >
> > > The builds which execute the unit tests may be distinguished by the
> > > "RUN_TESTS" parameter.
> &
the dependent library are not enabled.
+#. Dependent driver modules are not installed on the system.
#. Note enough processing cores. Some tests are skipped on machines with
2 or 4 cores
The rest lgtm
Acked-by: Michael Santana mailto:msant...@redhat.com>>
+
+To help find missing libraries, the u
g mode as it can help post-mortem.
>
> 1: https://travis-ci.com/DPDK/dpdk/jobs/223511683
>
> Signed-off-by: David Marchand
Acked-by: Michael Santana
> ---
> .ci/linux-setup.sh | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/.ci/linux-setup.sh b/.c
>
>
>
> On Mon, Jul 1, 2019 at 6:04 PM Aaron Conole wrote:
>>
>> >> - rwlock_autotest and hash_readwrite_lf_autotest are taking a little more
>> >> than 10s,
>>
>> Occasionally the distributor test times out as well. I've moved them as
>> part of a separate patch, that I'll post along with a bi
being deleted correctly.
Is this an anomaly or is anyone else seeing this issue as well?
Michael Santana
Note, if you are running on a system with less than 8 cores please see
patch
https://github.com/Maickii/dpdk-2/commit/7cfad856611e3ded4050f670ec11d1b2e17851d8.patch
not being deleted
correctly.
Is this an anomaly or is anyone else seeing this issue as well?
Michael Santana
Note, if you are running on a system with less than 8 cores please see
patch
https://github.com/Maickii/dpdk-2/commit/7cfad856611e3ded4050f670ec11d1b2e17851d8.patch
On 5/10/19 11:28 AM, Burakov, Anatoly wrote:
On 10-May-19 3:53 PM, Michael Santana wrote:
snprintf guarantees to always correctly place a null terminator in
the buffer
string. So manually placing a null terminator in a buffer right after
a call
to snprintf is redundant code.
Additionally
ype: 'string', value: 'all',
+ description: 'List of compress drivers to build. Defaults to all')
+option('drivers_crypto', type: 'string', value: 'all',
+ description: 'List of crypto drivers to build. Defaults to all')
+option('drivers_event', type: 'string', value: 'all',
+ description: 'List of event drivers to build. Defaults to all')
+option('drivers_mempool', type: 'string', value: 'all',
+ description: 'List of mempool drivers to build. Defaults to all')
+option('drivers_net', type: 'string', value: 'all',
+ description: 'List of net drivers to build. Defaults to all')
+option('drivers_raw', type: 'string', value: 'all',
+ description: 'List of rawdev drivers to build. Defaults to all')
option('drivers_install_subdir', type: 'string', value: 'dpdk/pmds-',
description: 'Subdirectory of libdir where to install PMDs. Defaults to
using a versioned subdirectory.')
option('enable_docs', type: 'boolean', value: false,
@@ -16,6 +36,8 @@ option('include_subdir_arch', type: 'string', value: '',
description: 'subdirectory where to install arch-dependent headers')
option('kernel_dir', type: 'string', value: '',
description: 'path to the kernel for building kernel modules, they will
be installed in $DEST_DIR/$kernel_dir/../extra/dpdk')
+option('libs', type: 'string', value: 'all',
+ description: 'List of libraries to build. Defaults to all')
option('lib_musdk_dir', type: 'string', value: '',
description: 'path to the MUSDK library installation directory')
option('machine', type: 'string', value: 'native',
I think this patch is the right step forward. This patch will speed up
travis builds which is a big plus
Just one small comment, could you document/add some build examples to
doc/build-sdk-meson.txt and other relevant doc files? Thanks
Acked-by: Michael Santana
On 6/4/19 4:59 AM, David Marchand wrote:
This is a joint effort to make the unit tests ready for CI.
The first 8 patches are fixes that I had accumulated.
Then the second part of the series focuses on skipping tests when some
requirements are not fulfilled so that we can start them in a restraine
On 6/5/19 5:36 PM, Honnappa Nagarahalli wrote:
Thomas Monjalon writes:
05/06/2019 21:40, Aaron Conole:
Thomas Monjalon writes:
The compilation of the master branch is failing for aarch64:
https://travis-ci.com/DPDK/dpdk
The log is so much verbose that I am not able to understand wh
on4(input, flows.trans, index_array);
Fixed on travis: https://travis-ci.com/Maickii/dpdk-2/builds/114612090
Acked-by: Michael Santana
On 6/6/19 10:57 AM, Jerin Jacob Kollanukkaran wrote:
-Original Message-
From: dev On Behalf Of Michael Santana
Francisco
Sent: Thursday, June 6, 2019 4:09 AM
To: Honnappa Nagarahalli ; Aaron Conole
; tho...@monjalon.net
Cc: Ruifeng Wang (Arm Technology China) ; Gavin
Hu (Arm Technology
69 matches
Mail list logo