From 076ed09e9eaeeb751ce87bac67ae0becc7b2417b Mon Sep 17 00:00:00 2001
From: Marek Gradzki <mgradzki@cisco.com>
Date: Mon, 11 Jun 2018 16:45:47 +0200
Subject: HC2VPP-354: use hw_interface_set_mtu for Ethernet interfaces

The API was renamed by
https://gerrit.fd.io/r/#/c/12930/

The VPP change spearates setting of hardware interface
and software interface MTU.

More details:
https://git.fd.io/vpp/tree/src/vnet/MTU.md

HC supports only hardware interface MTU configuration
for Ethernet interfaces.

MTU for software interfaces (per protocol MTU) is not
supported (HC2VPP-355).

Change-Id: I7eb1cb035a7b0f428a7bc7a9bb1c73819b52f0fa
Signed-off-by: Marek Gradzki <mgradzki@cisco.com>
---
 .../io/fd/hc2vpp/v3po/interfaces/EthernetCustomizer.java     | 12 +++++++++---
 1 file changed, 9 insertions(+), 3 deletions(-)

(limited to 'v3po/v3po2vpp/src/main/java/io/fd/hc2vpp/v3po/interfaces/EthernetCustomizer.java')

diff --git a/v3po/v3po2vpp/src/main/java/io/fd/hc2vpp/v3po/interfaces/EthernetCustomizer.java b/v3po/v3po2vpp/src/main/java/io/fd/hc2vpp/v3po/interfaces/EthernetCustomizer.java
index 6945f45e7..dfac5b99d 100644
--- a/v3po/v3po2vpp/src/main/java/io/fd/hc2vpp/v3po/interfaces/EthernetCustomizer.java
+++ b/v3po/v3po2vpp/src/main/java/io/fd/hc2vpp/v3po/interfaces/EthernetCustomizer.java
@@ -21,7 +21,7 @@ import io.fd.hc2vpp.common.translate.util.JvppReplyConsumer;
 import io.fd.hc2vpp.common.translate.util.NamingContext;
 import io.fd.honeycomb.translate.write.WriteContext;
 import io.fd.honeycomb.translate.write.WriteFailedException;
-import io.fd.vpp.jvpp.core.dto.SwInterfaceSetMtu;
+import io.fd.vpp.jvpp.core.dto.HwInterfaceSetMtu;
 import io.fd.vpp.jvpp.core.future.FutureJVppCore;
 import javax.annotation.Nonnull;
 import org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.iana._if.type.rev140508.EthernetCsmacd;
@@ -75,10 +75,16 @@ public class EthernetCustomizer extends AbstractInterfaceTypeCustomizer<Ethernet
         final String name = id.firstKeyOf(Interface.class).getName();
         final int index = interfaceContext.getIndex(name, writeContext.getMappingContext());
         LOG.debug("Setting Ethernet attributes for interface: {}, {}. Ethernet: {}", name, index, dataAfter);
-        final SwInterfaceSetMtu request = new SwInterfaceSetMtu();
+
+        // Set the physical payload MTU. I.e. not including L2 overhead.
+        // Setting the hardware MTU will program the NIC.
+        // Setting MTU for software interfaces is currently not supported (TODO: HC2VPP-355).
+        // More details:
+        // https://git.fd.io/vpp/tree/src/vnet/MTU.md
+        final HwInterfaceSetMtu request = new HwInterfaceSetMtu();
         request.swIfIndex = index;
         request.mtu = dataAfter.getMtu().shortValue();
-        getReplyForWrite(getFutureJVpp().swInterfaceSetMtu(request).toCompletableFuture(), id);
+        getReplyForWrite(getFutureJVpp().hwInterfaceSetMtu(request).toCompletableFuture(), id);
         LOG.debug("Ethernet attributes set successfully for: {}, {}. Ethernet: {}", name, index, dataAfter);
     }
 }
-- 
cgit 1.2.3-korg