On Tue, 5 Feb 2013 11:15:54 -0800 Ben Pfaff <b...@cs.stanford.edu> wrote:
> On Tue, Feb 05, 2013 at 06:49:45PM +0800, Cong Wang wrote: > > On Tue, Feb 5, 2013 at 2:08 PM, Ben Pfaff <b...@cs.stanford.edu> wrote: > > > Stephen Hemminger <step...@networkplumber.org> writes: > > > > > >> On Tue, 29 Jan 2013 22:15:18 +0800 > > >> Cong Wang <xiyou.wangc...@gmail.com> wrote: > > >> > > >>> Hello, Rusty, Jesse, > > >>> > > >>> I met an interesting problem when I compile openvswitch module as a > > >>> built-in (actually I compile ALL kernel modules as built-in), there is > > >>> no /sys/module/openvswitch/ directory created by the kernel in this > > >>> case. > > >>> > > >>> What's worse, the user-space init script thinks openvswitch module is > > >>> not loaded by checking the exist of this directory, therefore refuses > > >>> to start. > > >> > > >> Shouldn't the OVS init script be testing for some other API. > > > > > > I agree that's a bug in the OVS init script. I will fix it. > > > > Thanks for taking care of it, Ben! > > I posted a patch to ovs-dev for review: > http://openvswitch.org/pipermail/dev/2013-February/025128.html Good. The ovsctl test is actually more reliable than the /sys/module because it means that there isn't some other part broken in the API. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
