On 11/02/2017 01:09 PM, Ben Pfaff wrote:
On Thu, Nov 02, 2017 at 12:55:13PM -0700, Greg Rose wrote:
On 11/02/2017 11:12 AM, Greg Rose wrote:
On 11/02/2017 11:07 AM, Ben Pfaff wrote:
On Thu, Nov 02, 2017 at 10:46:48AM -0700, Greg Rose wrote:
On certain Ubuntu systems running the 4.4 based kernel depmod
may not correctly search for module dependencies when newer
openvswitch kernel modules have been installed in the extra
directory at /lib/modules/$(uname -r)/extra.  A symptom of
this are the following messages in system log that can be
printed with the dmesg command:

Thanks for figuring out the problem and documenting a fix.  I really
appreciate documentation patches.

+        #!/bin/sh
+
+        check_ubuntu_depmod_config() {
+            if [ -e /etc/depmod.d/ubuntu.conf ]; then
+                if ! grep -q extra /etc/depmod.d/ubuntu.conf
+                    then
+                    sed -i s\/search\/"search extra"\/ 
/etc/depmod.d/ubuntu.conf
+                    /sbin/depmod -a
+                fi
+            fi
+        }

I wonder about the details of this particular fix.  Usually, when there
is a configuration file in a .d directory, like depmod.d, it is possible
to add more files to the same directory that also get used for
configuration.  When that works, it is usually better to add files to
the directory rather than changing existing files, because it behaves
better on upgrade.  So, in this case, I wonder whether it is possible to
add a new file that would have something like:

     search extra

This works if put in a separate file such as /etc/depmod.d/ovs-search-path.conf
or something else.  Suggested names are welcome.  It also works if the
instructions already in place to create the /etc/depmod.d/openvswitch.conf
file are amended so that the first line of the file is 'search extra'.

Like this:

search extra
override * * extra
override * * weak-updates

I forgot we already had instructions for that.  It seems easiest to just
add the extra line there, then.


Will do.  Thanks.

_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to