Linux Virtualization - Mar 2019 - [RFC PATCH net-next] failover: allow name change on IFF_UP slave interfaces

On 3/6/2019 1:26 PM, si-wei liu wrote:
>
>
>
> On 3/6/2019 4:04 AM, Jiri Pirko wrote:
>>> --- a/net/core/failover.c
>>> +++ b/net/core/failover.c
>>> @@ -16,6 +16,11 @@
>>>
>>> static LIST_HEAD(failover_list);
>>> static DEFINE_SPINLOCK(failover_lock);
>>> +static bool slave_rename_ok = true;
>>> +
>>> +module_param(slave_rename_ok, bool, (S_IRUGO | S_IWUSR));
>>> +MODULE_PARM_DESC(slave_rename_ok,
>>> +		 "If set allow renaming the slave when failover master is
up");
>> No module parameters please. If you need to set something do it using
>> rtnl_link_ops. Thanks.
>>
> I understand what you ask for, but without module parameters userspace 
> don't work. During boot (dracut) the virtio netdev gets enslaved 
> earlier than when userspace comes up, so failover has to determine the 
> setting during initialization/creation. This config is not dynamic, at 
> least for the life cycle of a particular failover link it shouldn't be 
> changed. Without module parameter, how does the userspace specify this 
> value during kernel initialization?
>
Can we enable this by default and not make it configurable via module 
parameter?
Is there any? usecase where someone expects rename to fail with failover 
slaves?
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
<http://lists.linuxfoundation.org/pipermail/virtualization/attachments/20190306/80558e5c/attachment.html>