Bug 555400

Summary: starting userspace STP failed
Product: [Fedora] Fedora Reporter: Michael Monreal <michael.monreal>
Component: kernelAssignee: Neil Horman <nhorman>
Status: CLOSED NEXTRELEASE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: low    
Version: 12CC: anton, berrange, clalance, crobinso, dougsland, gansalmon, itamar, jforbes, jonathan, kernel-maint, nhorman, veillard, virt-maint
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
: 743584 (view as bug list) Environment:
Last Closed: 2010-04-09 17:02:45 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On: 570166    
Bug Blocks:    

Description Michael Monreal 2010-01-14 17:02:17 UTC
At the end of the boot process, I always see the following:

Bridge firewalling registered
virbr0: starting userspace STP failed, starting kernel STP

I am not sure if this indicates any real problem (would be nice to know). However, having the line flash up between boot splash and GDM startup is simply not that nice...

Comment 1 Daniel Berrangé 2010-03-02 15:25:29 UTC
THis isn't a libvirt problem it happens even with brctl

# brctl addbr foo
# brctl stp foo on

# dmesg  | tail -1
foo: starting userspace STP failed, starting kernel STP

The kernel appears to print this message everytime, not matter what you do. IMHO this message should be removed from the kernel, unless there's actually a clear explanation of why this is a real error.

Comment 2 Neil Horman 2010-03-02 19:21:19 UTC
It shouldn't be removed from the kernel, its an accurate message in response to a legitimate failure.  The kernel is trying to fork a user space to exec /sbin/bridge-stp, which is built (normally) from these sources:
http://git.kernel.org/?p=linux/kernel/git/shemminger/rstp.git;a=summary

But fedora hasn't packaged it, or any other rstp userspace implementation yet, so the exec fails, and the kernel backs off to the in-kernel implementation

Comment 3 Daniel Berrangé 2010-03-03 09:09:21 UTC
So is there any benefit to the userspace STP helper, and if so, shouldn't it be included by the bridge-utils package so its available ?  If we don't plan to include the userspace helper in Fedora, then I still think this error message should be hidden/downgraded to look less like a problem, since this bug keeps getting re-reported by people who think something is broken with libvirt+STP.

Comment 4 Neil Horman 2010-03-03 12:26:34 UTC
yes, theres an advantage in that STP is really an approrpiate task for userspace, and things that are can be done in user space, generally should be.

As such, yes, we should likely package it, but no one has yet

As for the warning, I don't see how it could be softened any, nor really why it should be (setting aside the odd complaints.  Its an accurate message detailing exactly what happened, and its reported at KERN_INFO level.

The right solution is to either do nothing (understand that the message is informative, but not dangerous), or package the rstp daemon.  I definately think the latter is the wisest choice.  I can try find some time to package it shortly.

Comment 5 Daniel Berrangé 2010-03-03 12:35:38 UTC
Ok, that plan sounds fine to me.

Comment 6 Neil Horman 2010-03-03 14:18:40 UTC
I'lve opened up a review request to package rstp.  I'll close this once its in, as installing the rstp package will make this warning go away.

Comment 7 Neil Horman 2010-04-09 17:02:45 UTC
so, I've pushed an rstp daemon to rawhide and F-13, so we should be good to go here.