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 shutting down 0day-bot for today for a system update. We will be
back online later by the end of the day EST.
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
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
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
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 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
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
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.
> &
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):
>
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
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 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 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
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
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 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
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
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
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/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 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
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
on4(input, flows.trans, index_array);
Fixed on travis: https://travis-ci.com/Maickii/dpdk-2/builds/114612090
Acked-by: Michael Santana
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
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
/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
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 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
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_
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
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
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
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:
+ -
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/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
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
@@
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
- 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
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 '
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
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
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
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
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
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
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
69 matches
Mail list logo