Package: wnpp
Severity: wishlist
Owner: Robin Jarry
* Package name: golang-github-soniakeys-quant
Version : 1.0.0-1
Upstream Author : Sonia Keys
* URL : https://github.com/soniakeys/quant
* License : Expat
Programming Lang: Go
Description
Package: wnpp
Severity: wishlist
Owner: Robin Jarry
* Package name: golang-github-soniakeys-quant
Version : 1.0.0-1
Upstream Author : Sonia Keys
* URL : https://github.com/soniakeys/quant
* License : Expat
Programming Lang: Go
Description
Package: wnpp
Severity: wishlist
Owner: Robin Jarry
* Package name: golang-github-soniakeys-quant
Version : 1.0.0-1
Upstream Author : Sonia Keys
* URL : https://github.com/soniakeys/quant
* License : Expat
Programming Lang: Go
Description
Package: wnpp
Severity: wishlist
Owner: Robin Jarry
* Package name: golang-github-soniakeys-quant
Version : 1.0.0-1
Upstream Author : Sonia Keys
* URL : https://github.com/soniakeys/quant
* License : Expat
Programming Lang: Go
Description
Package: wnpp
Severity: wishlist
Owner: Robin Jarry
* Package name: golang-github-mattn-go-sixel
Version : 0.0.5-1
Upstream Author : mattn
* URL : https://github.com/mattn/go-sixel
* License : Expat
Programming Lang: Go
Description : DRCS/Sixel Encoder
Package: wnpp
Severity: wishlist
Owner: Robin Jarry
* Package name: golang-github-mattn-go-sixel
Version : 0.0.5-1
Upstream Author : mattn
* URL : https://github.com/mattn/go-sixel
* License : Expat
Programming Lang: Go
Description : DRCS/Sixel Encoder
Package: wnpp
Severity: wishlist
Owner: Robin Jarry
* Package name: golang-github-mattn-go-sixel
Version : 0.0.5-1
Upstream Author : mattn
* URL : https://github.com/mattn/go-sixel
* License : Expat
Programming Lang: Go
Description : DRCS/Sixel Encoder
Package: wnpp
Severity: wishlist
Owner: Robin Jarry
* Package name: golang-github-mattn-go-sixel
Version : 0.0.5-1
Upstream Author : mattn
* URL : https://github.com/mattn/go-sixel
* License : Expat
Programming Lang: Go
Description : DRCS/Sixel Encoder
Package: wnpp
Severity: wishlist
Owner: Robin Jarry
* Package name: golang-sourcehut-rockorager-vaxis
Version : 0.8.2-1
Upstream Author : Tim Culverhouse <>
* URL : https://git.sr.ht/~rockorager/vaxis
* License : Apache-2.0
Programming Lang: Go
Descr
Package: wnpp
Severity: wishlist
Owner: Robin Jarry
* Package name: golang-sourcehut-rockorager-vaxis
Version : 0.8.2-1
Upstream Author : Tim Culverhouse <>
* URL : https://git.sr.ht/~rockorager/vaxis
* License : Apache-2.0
Programming Lang: Go
Descr
Package: wnpp
Severity: wishlist
Owner: Robin Jarry
* Package name: golang-sourcehut-rockorager-vaxis
Version : 0.8.2-1
Upstream Author : Tim Culverhouse <>
* URL : https://git.sr.ht/~rockorager/vaxis
* License : Apache-2.0
Programming Lang: Go
Descr
Package: wnpp
Severity: wishlist
Owner: Robin Jarry
* Package name: golang-sourcehut-rockorager-vaxis
Version : 0.8.2-1
Upstream Author : Tim Culverhouse <>
* URL : https://git.sr.ht/~rockorager/vaxis
* License : Apache-2.0
Programming Lang: Go
Descr
Nilesh Patra, Feb 01, 2024 at 10:07:
From what I gather, filters/html still uses socksify from dante package.
This looks OK as recommends - given that in 2024, people do get a bunch of HTML
mails.
I want to close this bug if you agree.
We *could* drop it and change the default html filter to
/c340fc5a17db85a06a15b345cbf6b9dfe14bb63e
d/patches: force C.UTF-8 locale when building docs
Closes: #1057929
Signed-off-by: Robin Jarry
(this message was generated
Nilesh Patra, Dec 06, 2023 at 16:44:
My understanding is that we can maybe ignore the CI here.
Hey Nilesh,
thanks for looking at this. Unless you see something wrong I think it is
good for another attempt at NEW.
Cheers!
Nilesh Patra, Dec 05, 2023 at 20:44:
You should have the permissions now for this repo. LMK if there's
still an issue.
Thanks I managed to push everything needed. However, the CI fails for
some obscure reason now:
Hi Nilesh,
I finally get time to work on this. I tried updating the repo but
somehow I messed up the upstream branch (updated it with gbp import-orig
--uscan) last time.
https://salsa.debian.org/go-team/packages/golang-sourcehut-rockorager-go-jmap/
I'd like to follow the standard go team
Hi Nilesh,
Nilesh Patra, Nov 22, 2023 at 21:03:
I am pinging here to ask if you have a clue as to what could be done
about it.
As this is blocking aerc update, do you find it possible to use
another golang library in place of rockorager-go-jmap that may offer
similar functionality?
Sorry,
-at: https://bugzilla.redhat.com/1949855
Signed-off-by: Jakob Meng
Hi Jakob, this looks good. Thanks!
Reviewed-by: Robin Jarry
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
--
2.39.2
Perfect, thanks Jakob!
Reviewed-by: Robin-Jarry
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
Jakob Meng, Oct 26, 2023 at 14:17:
On 26.10.23 13:52, Robin Jarry wrote:
> , Oct 26, 2023 at 13:07:
>> From: Jakob Meng
>>
>> Wildcard sections [*] and [**] are unsafe because properties cannot be
>> applied safely to any filetype in general. For example, IDEs
_width = 8
[lib/strsep.c]
+charset = utf-8
+end_of_line = lf
indent_style = tab
indent_size = tab
+insert_final_newline = true
tab_width = 8
--
2.39.2
--
Robin Jarry
Principal Software Engineer
Red Hat, Telco/NFV
___
dev mailing list
d...@openvswitch
Eelco Chaudron, Oct 25, 2023 at 14:56:
On 25 Oct 2023, at 13:52, jm...@redhat.com wrote:
> From: Jakob Meng
>
> A patch created with 'git format-patch' can contain trailing spaces.
> When editing a patch, e.g. to fix a typo in the title, the trailing
> spaces should not be removed. This
accordingly.
Reported-at: https://bugzilla.redhat.com/1949855
Signed-off-by: Jakob Meng
Hi Jakob, this looks good, thanks!
Reviewed-by: Robin Jarry
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
Kevin Traynor, Oct 12, 2023 at 17:34:
ok, 'rss' is documented as default, so maybe we don't need to display if
it is in use by default, selected by user or as fallback.
That would make things a bit easier as 'rx-steering:' is free to use to
indicate the unsupported case.
So maybe status
Nilesh Patra, Oct 05, 2023 at 23:52:
Uploaded to NEW -- I hope the copyright checks would be green!
Thanks \o/
Nilesh Patra, Oct 02, 2023 at 00:08:
When a file has multiple licenses, you can mention the license as
"License: Expat and RFC-Reference" and add a
comment section stating the doctrings are licensed under RFC license.
I took a hint for how it's done in gnupg2 package[1]. You could do something
Hi Mark,
Mark Michelson, Oct 03, 2023 at 23:09:
Hi Robin,
Thanks a bunch for putting these two emails together. I've read through
them and the replies.
I think there's one major issue: a lack of data.
That's my concern as well... The problem is, it is very hard to get
reliable and
Hi Felix,
Felix Huettner, Oct 04, 2023 at 09:24:
> Hi Robin,
>
> i'll try to answer what i can.
>
> On Tue, Oct 03, 2023 at 09:22:53AM +0200, Robin Jarry via discuss wrote:
> > Hi all,
> >
> > Felix Huettner, Oct 02, 2023 at 09:35:
> > > Hi everyone,
Frode Nordahl, Oct 02, 2023 at 10:05:
> > I must admit I don't have enough field experience operating large
> > network fabrics to state what issues multicast can cause with these.
> > This is why I raised this in the cons list :)
> >
> > What specific issues did you have in mind?
>
> It has been
Hi all,
Felix Huettner, Oct 02, 2023 at 09:35:
> Hi everyone,
>
> just want to add my experience below
> On Sun, Oct 01, 2023 at 03:49:31PM -0700, Han Zhou wrote:
> > On Sun, Oct 1, 2023 at 12:34 PM Robin Jarry wrote:
> > >
> > > Hi Han,
> > >
Han Zhou, Oct 01, 2023 at 21:30:
> Please note that tunnels are needed not only between nodes related to same
> logical switches, but also when they are related to different logical
> switches connected by logical routers (even multiple LR+LS hops away).
Yep.
> To clarify a little more,
Nilesh Patra, Oct 01, 2023 at 12:28:
> The jmap.go file header says:
>
> """
> // Package jmap implements JMAP Core protocol as defined in
> // draft-ietf-jmap-core-17 (published March 2019).
> //
> // Documentation strings for most of the protocol objects are taken from (or
> // based on)
Hi Han,
Please see my comments/questions inline.
Han Zhou, Sep 30, 2023 at 21:59:
> > Distributed mac learning
> >
> >
> > Use one OVS bridge per logical switch with mac learning enabled. Only
> > create the bridge if the logical switch has a port bound to the local
> >
Hi Han,
thanks a lot for your detailed answer.
Han Zhou, Sep 30, 2023 at 01:03:
> > I think ovn-controller only consumes the logical flows. The chassis and
> > port bindings tables are used by northd to updated these logical flows.
>
> Felix was right. For example, port-binding is firstly a
Hi Vladislav, Frode,
Thanks for your replies.
Frode Nordahl, Sep 30, 2023 at 10:55:
> On Sat, Sep 30, 2023 at 9:43 AM Vladislav Odintsov via discuss
> wrote:
> > > On 29 Sep 2023, at 18:14, Robin Jarry via discuss
> > > wrote:
> > >
> >
Hi all,
I am preparing for packaging the latest release of aerc:
https://git.sr.ht/~rjarry/aerc/refs/0.16.0
It requires a new dependency which is not available in Debian yet:
git.sr.ht/~rockorager/go-jmaphttps://bugs.debian.org/1053244
Using dh-make-golang, I have prepared the package on
Package: wnpp
Severity: wishlist
Owner: Robin Jarry
* Package name: golang-sourcehut-rockorager-go-jmap
Version : 0.3.0-1
Upstream Author : Tim Culverhouse
* URL : https://git.sr.ht/~rockorager/go-jmap
* License : Expat
Programming Lang: Go
Description
Package: wnpp
Severity: wishlist
Owner: Robin Jarry
* Package name: golang-sourcehut-rockorager-go-jmap
Version : 0.3.0-1
Upstream Author : Tim Culverhouse
* URL : https://git.sr.ht/~rockorager/go-jmap
* License : Expat
Programming Lang: Go
Description
Package: wnpp
Severity: wishlist
Owner: Robin Jarry
* Package name: golang-sourcehut-rockorager-go-jmap
Version : 0.3.0-1
Upstream Author : Tim Culverhouse
* URL : https://git.sr.ht/~rockorager/go-jmap
* License : Expat
Programming Lang: Go
Description
Package: wnpp
Severity: wishlist
Owner: Robin Jarry
* Package name: golang-sourcehut-rockorager-go-jmap
Version : 0.3.0-1
Upstream Author : Tim Culverhouse
* URL : https://git.sr.ht/~rockorager/go-jmap
* License : Expat
Programming Lang: Go
Description
Felix Huettner, Sep 29, 2023 at 15:23:
> > Distributed mac learning
> >
[snip]
> >
> > Cons:
> >
> > - How to manage seamless upgrades?
> > - Requires ovn-controller to move/plug ports in the correct bridge.
> > - Multiple openflow connections (one per managed bridge).
> >
Hi Felix,
Thanks a lot for your message.
Felix Huettner, Sep 29, 2023 at 14:35:
> I can get that when running 10k ovn-controllers the benefits of
> optimizing cpu and memory load are quite significant. However i am
> unsure about reducing the footprint of ovn-northd.
> When running so many nodes
ut I think these
propositions could benefit Kubernetes as well.
I hope I didn't bore everyone to death. Let me know what you think.
Cheers!
--
Robin Jarry
Red Hat, Telco/NFV
___
discuss mailing list
disc...@openvswitch.org
https://mail.openvswitch.org/mailman/li
there are other issues that would prevent scaling to
thousands of nodes. My point was mainly focused on L2 network scaling.
I would love to get other opinions on these statements.
Cheers!
--
Robin Jarry
Red Hat, Telco/NFV
___
discuss mailing list
di
Ilya Maximets, Sep 19, 2023 at 13:47:
> With flexibility of appctl comes absolutely no guarantee for API
> stability. But as soon as we have structured output, someone will
> expect it. If we can agree that users cannot rely on the structure
> of that structured output, then it's fine.
Adrian Moreno, Sep 19, 2023 at 09:18:
> > Both OVSDB and appctl are literally JSON-RPC protocols. There is no
> > need to re-invent anything.
>
> Right. Isn't appctl simpler in this case? IIUC, it would still satisfy
> the requirements: client decides update interval, flexible schema, etc
> with
Eelco Chaudron, Sep 12, 2023 at 09:17:
> I feel like if we do need another way of getting (real time)
> statistics out of OVS, we should use the same communication channel as
> the other ovs-xxx utilities are using. But rather than returning
> text-based responses, we might be able to make it JSON
Hey Kevin,
Kevin Traynor, Sep 07, 2023 at 15:37:
> This came up in conversation with other maintainers as I mentioned I was
> reviewing and the question raised was - Why add this ? if you want these
> values exposed, wouldn't it be better to to add to ovsdb ?
That's a good point. I had
Lucas Nussbaum, Aug 14, 2023 at 03:54:
> If you fail to reproduce this, please provide a build log and diff it with
> mine
> so that we can identify if something relevant changed in the meantime.
Hi Lucas,
I didn't manage to reproduce this. Here are both build logs binary and
source. I
Robin Jarry, Aug 31, 2023 at 15:11:
> diff --git a/lib/dpif-netdev.c b/lib/dpif-netdev.c
> index 70b953ae6dd3..ebf43a0f62e4 100644
> --- a/lib/dpif-netdev.c
> +++ b/lib/dpif-netdev.c
> @@ -1427,6 +1427,41 @@ dpif_netdev_pmd_rebalance(struct unixctl_conn *conn,
> int argc,
Link: https://git.dpdk.org/dpdk/commit/?id=9ab1804922ba583b0b16
Cc: David Marchand
Cc: Kevin Traynor
Signed-off-by: Robin Jarry
---
lib/dpdk-stub.c | 5 +++
lib/dpdk.c| 95 ++-
lib/dpdk.h| 5 +++
lib/dpif-netdev.c | 38 +
Eelco Chaudron, Aug 30, 2023 at 11:21:
> > This is very unlikely that any code will trigger a UserWarning with that
> > text content.
>
> Well you never know ;) I would prefer the original one, as it’s more strict.
Fair enough :)
___
dev mailing list
David Marchand, Aug 30, 2023 at 11:11:
> > warnings.filterwarnings("ignore", r"(blowfish|cast5)", UserWarning)
>
> It means waiving more warnings than before, but I don't think it
> really matters for this script.
> No strong opinion against your proposal.
This is very unlikely that any code will
David Marchand, Aug 30, 2023 at 10:16:
> Tests using mfex_fuzzy.py will fail on systems lacking the Python
> cryptography library.
> cryptography is not required, it is only imported in mfex_fuzzy.py to
> silence some warnings when scapy tries to load some libraries.
>
> Fixes: c3ed0bf34b8a
On Sun, 30 Jul 2023 17:48:07 +0200 Pelle wrote:
> I think it would make sense if address book completion was enabled for
> the reply-to header field like it is for other fields for entering
> email addresses (From, To, CC, BCC).
Hi Pelle,
this is not supported at the moment. The list of headers
be replaced by `python3 -m build --sdist`.
Link: https://blog.ganssle.io/articles/2021/10/setup-py-deprecated.html#summary
Signed-off-by: Robin Jarry
---
python/automake.mk | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/python/automake.mk b/python/automake.mk
index d9f6803691d7
/articles/2021/10/setup-py-deprecated.html#summary
Reported-by: Terry Wilson
Signed-off-by: Robin Jarry
---
python/automake.mk | 7 +--
1 file changed, 5 insertions(+), 2 deletions(-)
diff --git a/python/automake.mk b/python/automake.mk
index 6c7ac84b9c40..d9f6803691d7 100644
--- a/python
v2:
* Keep code in the python dir but rename it so that it does not clash
with `python -m build`.
* Remove duplication in makefiles.
Robin Jarry (3):
python: Rename build related code to ovs_build_helpers.
python: Use twine to upload sdist package to pypi.org.
python: Use build
-m build
Which is a way to generate PEP517 compatible source archives and binary
wheel packages.
Rename that folder to ovs_build_helpers which is more explicit. Update
all imports accordingly.
Link: https://peps.python.org/pep-0517/
Link: https://pypi.org/project/build/
Signed-off-by: Rob
of the article:
`python3 setup.py sdist` should be replaced by `python3 -m build
--sdist`
Link: https://blog.ganssle.io/articles/2021/10/setup-py-deprecated.html#summary
Signed-off-by: Robin Jarry
---
python/automake.mk | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git
sle.io/articles/2021/10/setup-py-deprecated.html#summary
Reported-by: Terry Wilson
Signed-off-by: Robin Jarry
---
python/automake.mk | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/python/automake.mk b/python/automake.mk
index 8b6266de214e..8854e656a5ae 100644
--- a/
-m build
Which is a way to generate PEP517 compatible source archives and binary
wheel packages.
Move that folder into build-aux. Update PYTHONPATH accordingly.
Link: https://peps.python.org/pep-0517/
Link: https://pypi.org/project/build/
Signed-off-by: Robin Jarry
---
Mak
Hi all,
This is a small series to fix the python ovs package publication on
pypi.org.
Robin Jarry (3):
python: Move build related code into build-aux.
python: Use twine to upload sdist package to pypi.org.
python: Use build to generate PEP517 compatible archives.
Makefile.am
Hello OVN community,
Following up on my previous email:
Robin Jarry, Jul 26, 2023 at 21:19:
> as discussed yesterday during the community meeting and today on IRC,
> I have created a public form that we can use to aggregate the results.
>
> https://forms.gle/bmZARvrxgfrJvzkn6
&g
Hi all,
as discussed yesterday during the community meeting and today on IRC,
I have created a public form that we can use to aggregate the results.
https://forms.gle/bmZARvrxgfrJvzkn6
No account is required to fill in the form and the results are
anonymous.
Please hold off before providing
Nilesh Patra, Jul 14, 2023 at 20:35:
> On Fri, Jul 14, 2023 at 08:13:16PM +0200, Robin Jarry wrote:
> > Nilesh Patra, Jul 14, 2023 at 19:59:
> > > There you go
> > >
> > > https://pb.envs.net/?816fb8c61575e2ba#4kD2xS4wJMGrsCd8tBhPfqHDJZ7qeM6qERaobgoYj46F
> &
Nilesh Patra, Jul 14, 2023 at 19:59:
> There you go
>
> https://pb.envs.net/?816fb8c61575e2ba#4kD2xS4wJMGrsCd8tBhPfqHDJZ7qeM6qERaobgoYj46F
Thanks a lot that helps. It looks like there is a deadlock due to two
goroutines trying to interact with the terminal:
goroutine 1 [sync.Mutex.Lock]:
If you manage to reproduce this easily, could you try the following:
1) aerc >trace.log 2>&1
2) Compose a message reproduce the freeze.
3) From another terminal, kill aerc with SIGQUIT.
After aerc has died, share the trace.log file contents. Ideally via
paste service, lists.sr.ht bounce emails
Nilesh Patra, Jul 14, 2023 at 19:14:
> Are you using debian though?
No, I am on Fedora 38.
Linux ringo 6.3.11-200.fc38.x86_64 #1 SMP PREEMPT_DYNAMIC Sun Jul 2 13:17:31
UTC 2023 x86_64 GNU/Linux
go version go1.20.5 linux/amd64
sway version 1.8.1
foot version: 1.14.0 -pgo +ime +graphemes
pelle, Jul 14, 2023 at 18:23:
> > Robin and I are both on wayland...I'm guessing with xclip you are on
> > X? I wonder if that comes into play at all?
>
> I'm having the issue in Sway. I've noticed that the freeze happens in
> vim when pasting with `CTRL+V` or `SHIFT+INSERT` while pasting works
>
Nilesh Patra, Jul 13, 2023 at 19:27:
> Hi,
>
> On Wed, 12 Jul 2023 11:09:03 +0200 Pelle wrote:
> > Package: aerc
> > Version: 0.15.2-1
> > Severity: normal
> >
> > I find that aerc almost always freezes when pasting into the compose area.
> >
> > It can easily be reproduced here:
> >* Open
Ilya Maximets, Jul 10, 2023 at 15:48:
> In order to speed up the process a bit, proposing a following incremental
> change:
>
> diff --git a/lib/netdev-dpdk.c b/lib/netdev-dpdk.c
> index 27b2fa5e0..aa87ee546 100644
> --- a/lib/netdev-dpdk.c
> +++ b/lib/netdev-dpdk.c
> @@ -2074,23 +2074,23 @@
all traffic in the isolated queue.
Cc: Anthony Harivel
Cc: Christophe Fontaine
Cc: David Marchand
Cc: Ilya Maximets
Signed-off-by: Robin Jarry
Acked-by: Kevin Traynor
---
Notes:
v13 -> v14:
* rebased on b4c7009c20e7 ("system-offloads-traffic.at: Add vxlan gbp
offload test.
Dumitru Ceara, Jul 04, 2023 at 16:16:
> Given that July 5th is tomorrow I'm assuming there won't be a community
> meeting as it seems quite short notice.
>
> Frode, is Tuesday July 11th 13:30 UTC still an option for you? If I'm
> not wrong this was the slot that worked for everyone that replied.
Fontaine
Cc: David Marchand
Cc: Ilya Maximets
Signed-off-by: Robin Jarry
Acked-by: Kevin Traynor
---
Notes:
v12 -> v13:
* Add "rx_steering" and adjust "requested_n_rxq" to dev->user_n_rxq in
netdev_dpdk_get_config().
Frode Nordahl, Jun 30, 2023 at 09:25:
> Below are some proposals for dates and times, please respond with a
> prioritized list of what works best for you and we'll try to land on a
> single date/time for a first meeting:
> Wednesday July 5th 13:00 UTC
> Tuesday July 11th 13:30 UTC
> Wednesday July
Hi Ilya,
Ilya Maximets, Jun 29, 2023 at 23:57:
> > +if (flags && ovsrcu_get(void *, >hw_info.offload_data)) {
>
> We probbaly shouldn't use the offload_data outside of netdev-offload
> modules. Simply checking netdev_is_flow_api_enabled() should be
> enough. Since both features require
Hi Kevin,
Kevin Traynor, Jun 22, 2023 at 15:06:
> Hi Robin,
>
> As discussed offline, the issue is fixed in v12. In the case where
> offload has failed on device A, and device B triggers set_config the
> set_config for device B will see that device B has the correct number of
> queues
he Fontaine
Cc: David Marchand
Cc: Kevin Traynor
Cc: Ilya Maximets
Signed-off-by: Robin Jarry
---
v11 -> 12:
* Rebased on c91867030234 ("MAINTAINERS: Add Eelco Chaudron.")
* Added dev->user_n_rxq back. It is required to handle reconfiguration
triggered by non-user config
/editorconfig/editorconfig-emacs
Link: https://github.com/editorconfig/editorconfig-vim
Link:
https://github.com/editorconfig/editorconfig/wiki/EditorConfig-Properties#max_line_length
Signed-off-by: Robin Jarry
Cc: Mike Pattrick
Cc: Eelco Chaudron
Cc: Ilya Maximets
---
Notes:
v4:
* Listed
Kevin Traynor, Jun 15, 2023 at 12:53:
> > new_n_rxq = MAX(smap_get_int(args, "n_rxq", NR_QUEUE), 1);
> > +if (dev->requested_rx_steer_flags != 0) {
> > +new_n_rxq += 1;
>
> If rx-steering is set for the port and the flow has previously not been
> able to be offloaded, the
Ilya Maximets, Jun 14, 2023 at 13:21:
> While files are present in tabs-allowed it doesn't mean that
> all of them actually use tabs. I'm not sure about other
> folders, but at least the windows include folder contains files
> with different styles.
>
> Is it possible to just exclude the folder
/editorconfig/editorconfig-emacs
Link: https://github.com/editorconfig/editorconfig-vim
Link:
https://github.com/editorconfig/editorconfig/wiki/EditorConfig-Properties#max_line_length
Signed-off-by: Robin Jarry
Acked-by: Mike Pattrick
Acked-by: Eelco Chaudron
Cc: Ilya Maximets
---
Notes:
v3: added
Ilya Maximets, Jun 12, 2023 at 21:53:
> The build-aux/initial-tab-allowed-files seems to list most of the
> non-conventionally styled files.
I'll have a look there. Thanks.
> Some files with different styles are in the same subtree, so describing
> at the root sounds better.
Ack.
> Well,
Hi Ilya,
Ilya Maximets, Jun 12, 2023 at 17:47:
> I'm a bit concerned that this patch is applying the same config
> to all the .c|.h files in the repo while not all of them have
> the same coding style. The most obvious is that linux headers
> are using tabs, not spaces. Not so obvious - sFlow
Hi Mike, Eelco,
Eelco Chaudron, Jun 12, 2023 at 16:54:
> I also like the idea, but the editoconfig does not seem to bring much
> to the table for C. And it looks like there is not much activity on
> the project. It might be better to look at clang-format, however, this
> is c/c++ specific.
>
>
purpose. This is not meant to be
configurable by users. Some limited configurability could be considered
in the future but it would expose to more potential issues if users are
accidentally redirecting all traffic in the isolated queue.
Cc: Anthony Harivel
Cc: Christophe Fontaine
Cc: David Marchand
purpose. This is not meant to be
configurable by users. Some limited configurability could be considered
in the future but it would expose to more potential issues if users are
accidentally redirecting all traffic in the isolated queue.
Cc: Anthony Harivel
Cc: Christophe Fontaine
Cc: David Marchand
dmp;
| ~^
../lib/netdev-dpdk.c:854:21: note: 'dmp' was declared here
854 | struct dpdk_mp *dmp, *next;
| ^~~
NB: this looks like a false positive, gcc 13 probably fails to see the link
between reuse and dmp in dpdk_mp_get().
Signed-off-by: Robin Jarry
Reviewed-by: Da
/editorconfig/editorconfig-emacs
Link: https://github.com/editorconfig/editorconfig-vim
Link:
https://github.com/editorconfig/editorconfig/wiki/EditorConfig-Properties#max_line_length
Signed-off-by: Robin Jarry
---
Notes:
v2: add .editorconfig to EXTRA_DIST
.editorconfig | 14
dmp;
| ~^
../lib/netdev-dpdk.c:854:21: note: 'dmp' was declared here
854 | struct dpdk_mp *dmp, *next;
| ^~~
Signed-off-by: Robin Jarry
---
lib/netdev-dpdk.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/lib/netdev-dpdk.c b/lib/netdev-dpdk.c
in
/editorconfig/editorconfig-emacs
Link: https://github.com/editorconfig/editorconfig-vim
Link:
https://github.com/editorconfig/editorconfig/wiki/EditorConfig-Properties#max_line_length
Signed-off-by: Robin Jarry
---
.editorconfig | 14 ++
1 file changed, 14 insertions(+)
create mode
Hey Ilya,
Ilya Maximets, May 24, 2023 at 17:05:
> I had a '+' because rss and lacp are two different entities and I looked
> at it as a mode of operation. i.e. RSS plus special handling for LACP.
> RSS looks strange in a comma-separated list, IMO.
For now, there is only LACP but if other
Kevin Traynor, May 24, 2023 at 16:54:
> > The "rss" item should be mandatory anyway. There should be no way to
> > disable it. I assume that it is what Ilya meant with these "+" symbols.
> > That would leave us with these examples:
> >
> > # lacp+bfd on separate queue, rss on other queues
> >
>
Hey Kevin,
Kevin Traynor, May 24, 2023 at 16:13:
> Hi Robin,
>
> I tested combinations of enabling/disabling cp-proto and enabling
> hwol. It is working as expected and hwol feature always has a clear
> priority, regardless of the order they are enabled in.
>
> I didn't test lacp traffic, but I
Kevin Traynor, May 24, 2023 at 16:06:
> Hmm, not sure on this one. I have a feeling that having a 'hash' mode
> for tx-steering that only applies to vhost devices, and 'hash' mode
> for rx-steering that only applies to NICs means people will miss the
> subtlety and try to enable the wrong hash
Ilya Maximets, May 23, 2023 at 22:04:
> 'rxq-isolate' will be confused with 'other_config:pmd-rxq-isolate'.
> Same likely goes for the 'isolated-rxq'.
>
> 'rxq-steernig' may be confused with 'other_config:tx-steering'.
> But this can be argued that it's essentially similar functionality,
> so
Aaron Conole, May 23, 2023 at 15:32:
> I think one issue I have with this is that the name is a bit
> misleading. Control plane, from OVS perspective, would be like OpenFlow
> communications. This is more like a traffic steering mechanism.
>
> Maybe it would help to call it something like
f users are
accidentally redirecting all traffic in the control plane queue.
Cc: Anthony Harivel
Cc: Christophe Fontaine
Cc: David Marchand
Cc: Kevin Traynor
Signed-off-by: Robin Jarry
---
v9 -> v10:
* If hw-offload is enabled, cp-protection will now be forcibly disabled
on all ports
1 - 100 of 275 matches
Mail list logo
