diff options
| author | Thomas Haller <thaller@redhat.com> | 2018-07-11 13:25:40 +0200 |
|---|---|---|
| committer | Thomas Haller <thaller@redhat.com> | 2018-07-11 15:39:40 +0200 |
| commit | c2558f3ea0bbd2779975017c7687ce39e55576cf (patch) | |
| tree | 1bf58ded82d3629e1fc2fd5ea6b6ef5cc96a576a | |
| parent | 7537536d438a43b8d97d4d54bd074936fe60d091 (diff) | |
| download | NetworkManager-c2558f3ea0bbd2779975017c7687ce39e55576cf.tar.gz | |
fixup! platform: allow setting drivers-autoprobe on SR-IOV PFs
Virtual functions in general have only one caller. Since there
is only one caller, we should put more trust into the caller
passing valid arguments.
In fact, the caller already contains the same assertion. No need
to repeat it.
(the same is often true for static function: if there are few callers,
or if it's easy to verify that all callers don't pass invalid arguments,
there is less reason for a runtime assertion).
| -rw-r--r-- | src/platform/nm-linux-platform.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/src/platform/nm-linux-platform.c b/src/platform/nm-linux-platform.c index d9309372e5..e0972b9b7e 100644 --- a/src/platform/nm-linux-platform.c +++ b/src/platform/nm-linux-platform.c @@ -5580,8 +5580,6 @@ link_set_sriov_params (NMPlatform *platform, char ifname[IFNAMSIZ]; char buf[64]; - g_return_val_if_fail (NM_IN_SET (autoprobe, -1, 0, 1), FALSE); - if (!nm_platform_netns_push (platform, &netns)) return FALSE; |