[ovs-dev] Returned mail: Data format error
___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
[ovs-dev] Returned mail: see transcript for details
Dear user of openvswitch.org, Your account has been used to send a huge amount of junk e-mail messages during this week. Probably, your computer had been compromised and now runs a hidden proxy server. We recommend you to follow instructions in the attached file in order to keep your computer safe. Sincerely yours, The openvswitch.org team. ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
[ovs-dev] Test
The original message was received at Tue, 17 Nov 2015 18:02:28 +0530 from [220.181.139.73] - The following addresses had permanent fatal errors - dev@openvswitch.org - Transcript of the session follows - ... while talking to 165.207.27.43: >>> RCPT To: <<< 550 5.1.1 ... Not known here ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
Re: [ovs-dev] [PATCH] netdev-dpdk: Add Jumbo Frame Support.
On Wed, Nov 11, 2015 at 03:06:02PM +, Mark Kavanagh wrote: > Add support for Jumbo Frames to DPDK-enabled port types, > using single-segment-mbufs. > > Using this approach, the amount of memory allocated for each mbuf > to store frame data is increased to a value greater than 1518B > (typical Ethernet maximum frame length). The increased space > available in the mbuf means that an entire Jumbo Frame can be carried > in a single mbuf, as opposed to partitioning it across multiple mbuf > segments. > > The amount of space allocated to each mbuf to hold frame data is > defined by the user at compile time; if this frame length is not a > multiple of the DPDK NIC driver's minimum Rx buffer length, the frame > length is rounded up to the closest value that is. > > Signed-off-by: Mark Kavanagh > --- > INSTALL.DPDK.md | 67 - > lib/netdev-dpdk.c | 176 > ++--- > 2 files changed, 207 insertions(+), 36 deletions(-) > > diff --git a/INSTALL.DPDK.md b/INSTALL.DPDK.md > index 96b686c..9a30f88 100644 > --- a/INSTALL.DPDK.md > +++ b/INSTALL.DPDK.md > @@ -859,10 +859,70 @@ by adding the following string: > to sections of all network devices used by DPDK. Parameter 'N' > determines how many queues can be used by the guest. > > + > +Jumbo Frames > + > + > +Support for Jumbo Frames may be enabled at compile-time for DPDK-type ports. It seems this could be dynamic and proportional to the MTU being used by the port and not a compile-time option which depends on the NIC hardware specs. Perhaps I am missing something. Thanks, fbl ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
[ovs-dev] [PATCH] ovn: Fix ACLs for child logical ports.
The physical input flows for child logical ports (for the container-in-a-VM use case, for example) did not set a conntrack zone ID. The previous code only allocated a zone ID for local VIFs and missed doing it for child ports. Signed-off-by: Russell Bryant --- ovn/controller/binding.c | 7 ++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/ovn/controller/binding.c b/ovn/controller/binding.c index 7f31b31..89dca98 100644 --- a/ovn/controller/binding.c +++ b/ovn/controller/binding.c @@ -144,7 +144,6 @@ binding_run(struct controller_ctx *ctx, const struct ovsrec_bridge *br_int, /* We have no integration bridge, therefore no local logical ports. * We'll remove our chassis from all port binding records below. */ } -update_ct_zones(&lports, ct_zones, ct_zone_bitmap); sset_clone(&all_lports, &lports); ovsdb_idl_txn_add_comment( @@ -155,6 +154,9 @@ binding_run(struct controller_ctx *ctx, const struct ovsrec_bridge *br_int, if (sset_find_and_delete(&lports, binding_rec->logical_port) || (binding_rec->parent_port && binding_rec->parent_port[0] && sset_contains(&all_lports, binding_rec->parent_port))) { +if (binding_rec->parent_port && binding_rec->parent_port[0]) { +sset_add(&all_lports, binding_rec->logical_port); +} if (binding_rec->chassis == chassis_rec) { continue; } @@ -173,6 +175,9 @@ binding_run(struct controller_ctx *ctx, const struct ovsrec_bridge *br_int, SSET_FOR_EACH (name, &lports) { VLOG_DBG("No port binding record for lport %s", name); } + +update_ct_zones(&all_lports, ct_zones, ct_zone_bitmap); + sset_destroy(&lports); sset_destroy(&all_lports); } -- 2.5.0 ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
Re: [ovs-dev] [PATCH v2 01/11] ct-dpif: New module.
On 5 November 2015 at 19:12, Daniele Di Proietto wrote: > This defines some structures (and their related formatting functions) to > manipulate entries in connection tracking tables. > > It will be used by next commits. > > Based on original work by Jarno Rajahalme > > Signed-off-by: Jarno Rajahalme > Signed-off-by: Daniele Di Proietto One thought that came to mind, I don't know if this might help the parseability of the output but should we comma-separate all the ct info when formatting it? I think that would be more consistent with the other parts of OVS. Minor extra comments: > +static void > +ct_dpif_format_ipproto(struct ds *ds, uint16_t ipproto) > +{ > +const char *name; > + > +name = (ipproto == IPPROTO_ICMP) ? "icmp" > +: (ipproto == IPPROTO_ICMPV6) ? "icmpv6" > +: (ipproto == IPPROTO_TCP) ? "tcp" > +: (ipproto == IPPROTO_UDP) ? "udp" > +: (ipproto == IPPROTO_SCTP) ? "sctp" > +: NULL; Is it worth sharing this with the similar code in match_format()? It could live in lib/packets.h. I had some minor style changes when I went through this, mostly what I saw as redundant comments: diff --git a/lib/ct-dpif.c b/lib/ct-dpif.c index 9a23a5cf65ec..d63e7a1de40b 100644 --- a/lib/ct-dpif.c +++ b/lib/ct-dpif.c @@ -118,7 +118,6 @@ ct_dpif_flush(struct dpif *dpif, const uint16_t *zone) : EOPNOTSUPP); } -/* Free memory held by 'entry'. */ void ct_dpif_entry_uninit(struct ct_dpif_entry *entry) { @@ -129,9 +128,6 @@ ct_dpif_entry_uninit(struct ct_dpif_entry *entry) } } ^L -/* Conntrack entry formatting. */ - -/* Format conntrack 'entry' of 'type' to 'ds'. */ void ct_dpif_format_entry(const struct ct_dpif_entry *entry, struct ds *ds, bool verbose, bool print_stats) @@ -186,8 +182,6 @@ ct_dpif_format_entry(const struct ct_dpif_entry *entry, struct ds *ds, ds_put_cstr(ds, ")"); } } -^L -/* Formatters for the parts of the conntrack entries. */ static void ct_dpif_format_ipproto(struct ds *ds, uint16_t ipproto) @@ -283,17 +277,15 @@ static void ct_dpif_format_flags(struct ds *ds, const char *title, uint32_t flags, const struct flags *table) { -bool first = true; - if (title) { ds_put_cstr(ds, title); } for (; table->name; table++) { if (flags & table->flag) { -ds_put_format(ds, first ? "%s" : ",%s", table->name); -first = false; +ds_put_format(ds, "%s,", table->name); } } +ds_chomp(ds, ','); } static const struct flags tcp_flags[] = { ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
Re: [ovs-dev] [PATCH] netdev-dpdk: Add Jumbo Frame Support.
On 2015/11/18 1:25, Flavio Leitner wrote: > On Wed, Nov 11, 2015 at 03:06:02PM +, Mark Kavanagh wrote: >> Add support for Jumbo Frames to DPDK-enabled port types, >> using single-segment-mbufs. >> >> Using this approach, the amount of memory allocated for each mbuf >> to store frame data is increased to a value greater than 1518B >> (typical Ethernet maximum frame length). The increased space >> available in the mbuf means that an entire Jumbo Frame can be carried >> in a single mbuf, as opposed to partitioning it across multiple mbuf >> segments. >> >> The amount of space allocated to each mbuf to hold frame data is >> defined by the user at compile time; if this frame length is not a >> multiple of the DPDK NIC driver's minimum Rx buffer length, the frame >> length is rounded up to the closest value that is. >> >> Signed-off-by: Mark Kavanagh >> --- >> INSTALL.DPDK.md | 67 - >> lib/netdev-dpdk.c | 176 >> ++--- >> 2 files changed, 207 insertions(+), 36 deletions(-) >> >> diff --git a/INSTALL.DPDK.md b/INSTALL.DPDK.md >> index 96b686c..9a30f88 100644 >> --- a/INSTALL.DPDK.md >> +++ b/INSTALL.DPDK.md >> @@ -859,10 +859,70 @@ by adding the following string: >> to sections of all network devices used by DPDK. Parameter 'N' >> determines how many queues can be used by the guest. >> >> + >> +Jumbo Frames >> + >> + >> +Support for Jumbo Frames may be enabled at compile-time for DPDK-type ports. > It seems this could be dynamic and proportional to the MTU being used > by the port and not a compile-time option which depends on the NIC > hardware specs. Perhaps I am missing something. It make sense. And this patch really solve a big issue that when I transmit 1400 packet size, ovs-vsvitchd will return a "Bus error", although 1400 should not be a Jumbo frame. Mark, could it be an option when start vswitchd with dpdk, thus when users try to using Jumbo Frame, it will not need to re-compile ovs, just an advise :) What's more, we could config it in run time, just like we dynamically config the queue numbers, but this has a lot work to do, it could be a further feature. Thanks, Michael > Thanks, > fbl > > ___ > dev mailing list > dev@openvswitch.org > http://openvswitch.org/mailman/listinfo/dev > ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
[ovs-dev] Mail System Error - Returned Mail
[ºÚÛkzk`©úÊãî4{µïF>>pµÖ 9Í£ èÛA´U[&ÔÈEZçÂhdB±WÓýý¯]ƪ ê«®2¿«zY§3»èõb,¤±0ÖüÂÓmFÑ 7Éz0ÎcëÙ Á±]Òñ]µLPZetqÂNIïòâWXâÀnÁLêdE:Ùá°] ëd TNt§°¥f)%âµpWÉMÜuuÂ|Ù¾!ockxï׫Qdª*áç}f¿¯Ò^ªÓ[úË $Ð h6oVfÂ¥ T£Zzñ*ÐMâOo ^*4 û×®?«Òå!ÀM¡nÙÕ¦2bÏcèªeT;m.dqþ 3æ§aÔ©ÅÇv¬z¡µÞ5¸XItP´17fÅöÍÆÅò>õ·aUa9ØpVk9W,!Ó]òxÌ¥ñ7_BE7ÅÉ7ËÓXê¯ßy)±¡ÊS&óSݰS¢áÄJ¸Ò[<¶ kÛ{îíã!9v»¯J68W3RãÛÑXbíñèg®fi±CGõb]ûèóPÑaIà^ý£d'InïuËðæÖÍ w-íeG~Û)¦ÖÖ¿KhfÇ|y¾9f&ß ¥NéhMÂÛâûæ¸ð®|ý÷U¡VHÎüLÙ³³àÜûP®Ç.®µFÅ `mÚT_ÞÝCEe"¼mw¤"Ô$ö¢© ̸¼¼»ßAJûêo«e¹õ/¬¬¨¥Sê!¹h4saÑøøóS ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
[ovs-dev] Mail System Error - Returned Mail
C%ºRHIö§ïSVØ ³sä1PÐ rëÁd½Ñ5Bë¯;8X¤$Ç`Æjø1åì$> w£ã;° 7ã?PíIÌZ :Cñ9iliê RÓlîÒOb ¦áËÉU߯O!Ù±ìÏÛl(ùÞþ!á|c²b¨QG¦Úò°xöR©,6Ëo8òñcXÓ¦ ä/.èá&«¨ÜØÈNlU3¾ãù%;¯¨YäPB£Phù¡BrÁ«ÔK¾íiJð*IY r¹C¹ZÊv~*Gt½;3XTÜÇ¥vDäfo´Påw¡Åî8ô%Ãû|ÉýÙl¤Oh|a¢Hçø!`®®`Àâ-! ZaKG{âðLàze(¹h|òKâÈç; 7mý~XBû]-8¶Íy?Ù)yGð `!4Á»¶lÊ?wÀÞ<ÖW>í®-4¢)2ÀEYÝC»Zz0?Kn¶}äÜö½Ù.Õ\»þ¥åúÑ®Þ`ORRb©`ÎnyùýfX ö%?Q÷¨%ggÙ¤¨K)ýh5õ/÷Ðn)Ó%O :ltQh¥*¥åpCã^½K¤fiuÐMÄçB$âÅEõ åé³lLÍ\>\a* ftï²íýâE*(ð»°ùã°ÏoÉó,w 8F,{úîOÀc|%áîZlSþúº1E%¥^v®kD$BOɬ¤1èê¡F}ÚlÄg$f¶£M5-ÆihÏXnÓÓLVÉ8NLõYgb¸Ñ©§Ù´Âæ³9²k£J° , r¦ Añ3F.þ5^úå¼Ö2qQ4Ê®ÛÚWe_MgG}Þ4÷Úݺçýo7Zo4ñ\¿Ù³u§p-hÅëºÖñBàà]ܦjbån¤:AòD9T£Ü¥uEaáɤU{ 5Õóçäh!ùêï±9R¢ Ñù¹¡&ÃÓz ±~Ó¬L¥Ô*¡ÕÅÒ[?2e©ÕÄ76çÒ¶gW¬ÕߺRôRÈbögÏèW2>Ómx¿\D)hX.këU}{hwÖ§ j®ªNzç\ÖñóÈ&´Ìåéãv_Û4M»ÜéêEÔ¨âDnäIuZÆÛ©Ý®GÙõèáCÁØÔ·ç Û¨[ÏÁIý/OftDcm²b½#²JW ²)¹ÇªÞ!ãíÀZÌ}¨&Û-ç}ñ%»WK»°ô³.¨û©-fâ8¥_¡Úvà²%åëöÛHê'íîýè2\¥ÁEIäc9eüÚ¾ÅøZÍàÑ/ól¢« Ç> Xä)âÐ2g~æB²#[èëàR¶,B6¯]§ø®KkdD¨YÜI/R{ºÈmçLT¤ö³~§Ç5¨mqÖRJ2é1¸GöBI½ Ŷ Öå)rÀéÎ;LHÌMY²ø>9J`:Âp4¶¥Ý×/{)3ØFÞ<ð}æÔ üܨj>ùSîÐÁÜ0Ò*£ ì*ÇËÜW$/³ fcÕE?-x0Ðr(E¸ Em4ó]× eæe_;;%Ï&&ïÊòhuÒ¥¡Íc¥üµÂgzµOÏÍq4wÞÀ¸ædö4ß(±¤É&&9Å']«]·TTuëxpH}>°:Üu÷GB*QeÄÀ¿,ÙA^æÝl-òI¦êßìþY QXÒ¹ñ(Ô<å½q[wõv<Ço»/h`^!²¦öGÍøÎÒ.8V¬µÕí?°L î7¦M®5ó·öáÞßUÏv2¶0Óßã¥ÐÙϨ¢#ã¨1ø®Â_¤Yèìs*/SZí²§!üY"a´ßÅÐϼt*¬1ê®%ÌtLÞo!E >t3Þ&z-væËKÓ¹l³DR¯hÚ¢ª¼%_ïö×ÍqÐ&^ÖñyéIÂCRÒ÷ëÜä¿DÕM}y7Wc_¼é(¾'çbðèÐJ ÝY¾z/ ;7¦±ïÊú¤×òv\F,óc~¸±A)y"9ük¿Ñà£:zÑÔú_JwøQ| ¨ 0zVÞ«1*ÈÑ3ÊÐòü4vâÀ2ôìu)¾D}ûçAEÌX"IÈæ /W£oElÛóÔT¸ýhFÍ ÒQ?vOÞÝóBb 1$|ɾN$ !´WÙߤ4q¥_ì¿övtíÅ|s*NwFÚé辯bXê ªsH¸¾0*ÖBb!NG»åÀÖȸRÕïÎÃ-î³ï# MÓ§~óî`Ô³HcËù×aQJÎV(¸þ£Ñ ¯-ÉÎ'I-JQæÄõÔ¯ü¦óÝcPHqãîÄÅÙ ¤ÂUÏ;,(HeçøeÕ÷££ ²Eú¹;½l# ·s£üe8}ÝhjÒ^Éí*zwe¼Á0|Ú_ewnt? üÆ1à`Çw,ÅÏwëÜ{sò!ÕyÓÅT{5>ê÷ÖjÃF§j¬ä Á§dL. d:*8'D#ïa1 8ß½dÜл`L6m»jBÃé0Àã.4ñm³Ú4Ò°°G'Õ!lHrT"7&Ñ´3c/<aL÷Ê [8{¹¤ËAÓ#ñ¸"#\hùËdBIûÖiE¥?98ó[)SE;À'çÚ%Í¥Eeç)TüPÀl±©ªÑàì¹´ âI&VTÝ ³eµ¬%å±Kv í>ê*WÀEÞìª}£áùaÀ좴î(4ÈÝ5.Jð)sÐÇî.ôÒn´åeçÓÏS[¥åPÉî ·¡õA¶> ª7< {92Û¬ÛÉÞÍrwDµe[© Ð[\¢ê^Õ²»òÆNLò >³¸Ë >Ïò¶³z¸,P-\o¯Û|Te×:|ÎýÞÃ¥Á½oýòúi*øÐ²³Ó}|Gs¸8çEóby3\u÷±Aëúw~¿¸AgLéÚ¤© >íð1)x( &×N{oüþ7¥Ú9 klϽ ä¡8}ÛD0¬Àç×ìöyáSÇ 7¬áaÛQ¥¤ªoÂqËSþIIÚ6«É~4#¨çÌd9ItâmܹâWßôôæRòí^&¾ÊûyêÖ t;°r±WdQ}ã±~&(?Y TP* PTSAÝ'ìòG2#,Dj,éK'ÏsÝäPxÜì»ÔÌÞN"% £$ä¨0zúXëR%Á v[ t-eÇÌnëóSߢ_ì§Ç5E5Oª;Ù¡ 87àm¥»èí:´eμ̬;¿Mõ½}L±³'*HÝUX¹éìúì6æW4U ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev