Red Hat Bugzilla – Bug 630087
[6.1 FEAT] support for IEEE 802.1Qbg in lldpad
Last modified: 2011-05-19 07:52:08 EDT
1. Feature Overview:
Feature Id: 
a. Name of Feature: [6.1 FEAT] support for IEEE 802.1Qbg in lldpad
b. Feature Description
support for edge virtual bridging (EVB) and VSI discovery protocol (VDP, under standardization in
IEEE 802.1Qbg) in lldpad.
2. Feature Details:
Architectures: x86, x86_64,
Arch Specificity: both
Affects Kernel Modules: Field does not exist
Delivery Mechanism: Direct from Community
Category: system management
Request Type: Toolchain - Enhancement from Upstream
d. Upstream Acceptance: Field does not exist
Sponsor Priority P2
f. Severity: high
IBM Confidential: No
Code Contribution: IBM code
g. Component Version Target:---
3. Business Case
In datacenter/cloud environments support for IEEE 802.1Qbg is required to register a large number of
guests and allow easier migration of guests with keeping the correct network connectivity.
4. Primary contact at Red Hat:
John Jarvis, firstname.lastname@example.org
5. Primary contacts at Partner:
Project Management Contact:
Stephanie A. Glass, email@example.com
Jens Osterkamp, Jens.Osterkamp@de.ibm.com
------- Comment From firstname.lastname@example.org 2010-10-04 11:22 EDT-------
Code Upstream Status: Submitted
------- Comment From email@example.com 2010-11-24 04:33 EDT-------
The patches necessary to support this feature are:
- implement VDP keepalive
- do not use macv[tap/lan] interfaces as ports
- lldpad support for libvirt netlink message
- use connect instead of bind
- add libnl dependency to configure.ac
- VDP commandline interface
- implementation of IEEE 802.1Qbg in lldpad, part 2
- BUGFIX: check for existence of ifup
- implementation of IEEE 802.1Qbg in lldpad, part 1
- consolidation of MIN and MAX macros in common.h
All patches apply on top of lldpad 0.9.38, available as tarball from http://sourceforge.net/projects/e1000/files/DCB%20Tools/lldpad/0.9.38/.
They have been posted to firstname.lastname@example.org, email@example.com, cc: firstname.lastname@example.org and are available from the archives at http://sourceforge.net/mailarchive/forum.php?forum_name=e1000-eedc or https://lists.linux-foundation.org/pipermail/virtualization/.
I am in contact with John Fastabend from Intel who is the maintainer of lldpad. He confirmed that the patches look good to him and he wants to include them into lldpad.
This enhancement request was evaluated by the full Red Hat Enterprise Linux
team for inclusion in a Red Hat Enterprise Linux minor release. As a
result of this evaluation, Red Hat has tentatively approved inclusion of
this feature in the next Red Hat Enterprise Linux Update minor release.
While it is a goal to include this enhancement in the next minor release
of Red Hat Enterprise Linux, the enhancement is not yet committed for
inclusion in the next minor release pending the next phase of actual
code integration and successful Red Hat and partner testing.
(Adding the primary maintainer for lldpad to the CC list)
John, apologies for communicating via bugzilla, but can you provide a statement that you plan to include the patches from Jens real soon now in the upstream version of lldpad?
Thank you for your help,
Hi, attached is a link to the lldpad git tree including the EVB patches provided by Jens.
The open-lldp.org web page content is out of date I will make a public announcement regarding open-lldp.org going live and upstream git repo after I have had a chance to update the content and our validation team has finished there testing of open-lldp.
I will be doing this after returning from the holidays. Feel free to ping me in the BZ or email me directly.
------- Comment From email@example.com 2011-01-06 09:48 EDT-------
Any update on when this will be upstream?
Patch set supplied by Jens has been applied to upstream open-lldp project and has passed our internal BAT testing. Available at,
*** Bug 634004 has been marked as a duplicate of this bug. ***
Done in CVS, lldpad-0.9.38-7.el6
CVS lldpad-0.9.38-8.el6: BR cleanup
Just reading the patch, there is lot of sprintf(obuf, "...%s...", ...) (in lldp_evb_cmds.c), I wonder if all such concatenations are verified they do not overflow the obuf. The obuf is pointer inside preallocated buffer of length MAX_CLIF_MSGBUF (ctrl_iface.c:ctrl_iface_receive()) and the appended strings are literals of the protocol with fixed size. I'm just curious.
------- Comment From firstname.lastname@example.org 2011-02-03 06:51 EDT-------
(In reply to comment #16)
> Just reading the patch, there is lot of sprintf(obuf, "...%s...", ...) (in
> lldp_evb_cmds.c), I wonder if all such concatenations are verified they do not
> overflow the obuf. The obuf is pointer inside preallocated buffer of length
> MAX_CLIF_MSGBUF (ctrl_iface.c:ctrl_iface_receive()) and the appended strings
> are literals of the protocol with fixed size. I'm just curious.
I don't think this is a problem for lldp_evb_cmds.c. All the get_* commands who write to obuf only use a few very well defined static strings. With 4096 bytes the buffer is much larger than the strings that are ever used.
------- Comment From email@example.com 2011-02-24 07:31 EDT-------
I verified that lldpad-0.9.41-1.el6.src.rpm contains the lldpad code from
Author: John Fastabend <firstname.lastname@example.org>
Date: Mon Jan 10 18:35:01 2011 -0800
lldpad: version bump 0.9.40 -> 0.9.41
Addition of EVB and sanitized headers warrants a version bump.
Signed-off-by: John Fastabend <email@example.com>"
A separate bugzilla has been opened to include the fixes that have been submitted since.
Comments #14, #15, #24 => Conditions for setting SanityOnly have been met here. Setting Verified SanityOnly.
------- Comment From firstname.lastname@example.org 2011-03-22 09:25 EDT-------
I verified that the IEEE 802.1Qbg is included in lldpad in RHEL6.1.
So from my point of view this request can be closed.
~~ Partners and Customers ~~
This bug was included in RHEL 6.1 Beta. Please confirm the status of this request as soon as possible.
If you're having problems accessing 6.1 bits, are delayed in your test execution or find in testing that the request was not addressed adequately, please let us know.
------- Comment From email@example.com 2011-04-06 09:20 EDT-------
This feature has been verified by IBM
New patches have been added to the latest build, please test the next snapshot (lldpad-0.9.41-4.el6) and update us here with test results. Thanks.
------- Comment From firstname.lastname@example.org 2011-04-27 15:12 EDT-------
(In reply to comment #23)
> New patches have been added to the latest build, please test the next snapshot
> (lldpad-0.9.41-4.el6) and update us here with test results. Thanks.
We verified that all upstream patches are in lldpad-0.9.41-4.el6. We have successfully tested it with snapshot 4 in our setup. Thanks !
An advisory has been issued which should help the problem
described in this bug report. This report is therefore being
closed with a resolution of ERRATA. For more information
on therefore solution and/or where to find the updated files,
please follow the link below. You may reopen this bug report
if the solution does not work for you.