[ovs-dev] [PATCH] Documentation: Document module install issues
Greg Rose
gvrose8192 at gmail.com
Thu Nov 2 20:13:22 UTC 2017
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.
More information about the dev
mailing list