Bug 684870 - [Cisco 6.1 Bug Fix] libvirt 802.1Qbh: Add support for PREASSOCIATE_RR
Summary: [Cisco 6.1 Bug Fix] libvirt 802.1Qbh: Add support for PREASSOCIATE_RR
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Red Hat Enterprise Linux 6
Classification: Red Hat
Component: libvirt
Version: 6.1
Hardware: x86_64
OS: Linux
unspecified
high
Target Milestone: rc
: ---
Assignee: Jiri Denemark
QA Contact: Virtualization Bugs
URL:
Whiteboard:
Depends On: 662245
Blocks: 665944 678826 679545
TreeView+ depends on / blocked
 
Reported: 2011-03-14 17:01 UTC by roprabhu
Modified: 2011-05-19 13:29 UTC (History)
14 users (show)

Fixed In Version: libvirt-0.8.7-15.el6
Doc Type: Bug Fix
Doc Text:
Clone Of: 662245
Environment:
Last Closed: 2011-05-19 13:29:00 UTC
Target Upstream Version:


Attachments (Terms of Use)
Proposed patch for this bug (2.50 KB, patch)
2011-03-22 18:15 UTC, roprabhu
no flags Details | Diff


Links
System ID Private Priority Status Summary Last Updated
Red Hat Product Errata RHBA-2011:0596 0 normal SHIPPED_LIVE libvirt bug fix and enhancement update 2011-05-18 17:56:36 UTC

Description roprabhu 2011-03-14 17:01:22 UTC
Having a preassociate step to align with KVM migration prepare step helps 802.1qbh too. This helps with restoring network connectivity on source host if migration fails during the prepare step on destination host with our new 802.1qbh switches (Which is a bug we came across recently).

With new changes in ieee 802.1qbh/bg spec, 802.1qbh will need to move to VDP eventually. 
To fix the migration issue Enic driver for Cisco VIC adapter(802.1qbh) will have support for PREASSOCIATE_RR soon. This bug fix will add PREASSOCIATE_RR request support during 802.1qbh port profile association

Comment 3 Gary Case 2011-03-15 15:08:02 UTC
From their email, Cisco believes that they'll be able to make the code available by March 21.

Comment 4 roprabhu 2011-03-22 18:15:01 UTC
Created attachment 486866 [details]
Proposed patch for this bug

Attached is the libvirt patch for this bug.  The patch has not been posted upstream yet. Its waiting for enic (8021qbh) patches at https://bugzilla.redhat.com/show_bug.cgi?id=684865

Comment 5 Dave Allan 2011-03-22 18:23:32 UTC
Please don't wait to post upstream, the feedback there will be extremely useful even if it's not committed.

Comment 6 roprabhu 2011-03-22 19:07:15 UTC
ok thanks.
we were holding onto this patch because the corresponding enic patch for 802.1qbh is not out yet.

I went ahead and posted the patch upstream just now:
https://www.redhat.com/archives/libvir-list/2011-March/msg01034.html


Thanks dave.

Comment 7 roprabhu 2011-03-22 20:19:13 UTC
Stefan has pushed the patch upstream. 
http://libvirt.org/git/?p=libvirt.git;a=commitdiff;h=7708da38c7b58034bc792a387527f06a85ef83c7

Note that this patch has a dependency on bug  https://bugzilla.redhat.com/show_bug.cgi?id=684865
for rhel 6.1

Comment 9 Jiri Denemark 2011-03-25 13:32:00 UTC
This is not a real FutureFeature, it fixes a migration bug, though by making use of a new request.

Comment 11 Vivian Bian 2011-03-30 05:02:56 UTC
Hi CISCO , 
Would you please update your pkgs to the version under http://veillard.com/libvirt/6.1/x86_64/ . And try if the bug is fixed already . 

Thanks
Vivian Bian

Comment 12 Chris Ward 2011-04-06 11:05:54 UTC
~~ 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.

Thanks!

Comment 13 dyuan 2011-04-28 09:06:07 UTC
Download libvirt-0.8.7-18.el6.src.rpm, and libvirt-8021Qbh-use-preassociate-rr-during-the-migration-prepare-stage.patch can be found in /root/rpmbuild/SOURCES. and no critical regression and blocker was introduced in the latest build for migration.

Comment 16 roprabhu 2011-05-11 07:22:39 UTC
Checked the libvirt src rpm on rhel 6.1 snap5. Pls consider this bug verified. Thanks

Comment 17 errata-xmlrpc 2011-05-19 13:29:00 UTC
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.

http://rhn.redhat.com/errata/RHBA-2011-0596.html


Note You need to log in before you can comment on or make changes to this bug.