Fix u16 type handling in jvpp
Change-Id: I6e5ed2562c65dde6c9f6f085c8b9d40f80684894 Signed-off-by: Marek Gradzki <mgradzki@cisco.com>
This commit is contained in:

committed by
Dave Wallace

parent
cb51846ae0
commit
34e7772443
@ -116,7 +116,7 @@ vl_api_ip6_fib_counter_t_array_struct_setter_template = Template("""
|
||||
// vl_api_ip6_fib_counter_t_array_field_setter_template FIXME""")
|
||||
|
||||
struct_setter_templates = {'u8': u8_struct_setter_template,
|
||||
'u16': u32_struct_setter_template,
|
||||
'u16': u16_struct_setter_template,
|
||||
'u32': u32_struct_setter_template,
|
||||
'i32': u32_struct_setter_template,
|
||||
'u64': u64_struct_setter_template,
|
||||
@ -218,6 +218,10 @@ default_dto_field_setter_template = Template("""
|
||||
(*env)->Set${jni_setter}(env, dto, ${java_name}FieldId, mp->${c_name});
|
||||
""")
|
||||
|
||||
u16_dto_field_setter_template = Template("""
|
||||
(*env)->Set${jni_setter}(env, dto, ${java_name}FieldId, clib_net_to_host_u16(mp->${c_name}));
|
||||
""")
|
||||
|
||||
u32_dto_field_setter_template = Template("""
|
||||
(*env)->Set${jni_setter}(env, dto, ${java_name}FieldId, clib_net_to_host_u32(mp->${c_name}));
|
||||
""")
|
||||
@ -247,11 +251,11 @@ u64_array_dto_field_setter_template = Template("""
|
||||
""")
|
||||
|
||||
dto_field_setter_templates = {'u8': default_dto_field_setter_template,
|
||||
'u16': u32_dto_field_setter_template,
|
||||
'u16': u16_dto_field_setter_template,
|
||||
'u32': u32_dto_field_setter_template,
|
||||
'i32': u32_dto_field_setter_template,
|
||||
'u64': u64_dto_field_setter_template,
|
||||
'f64': default_dto_field_setter_template,
|
||||
'f64': default_dto_field_setter_template, #fixme
|
||||
'u64[]': u64_array_dto_field_setter_template,
|
||||
'u8[]': u8_array_dto_field_setter_template
|
||||
}
|
||||
|
@ -0,0 +1,124 @@
|
||||
/*
|
||||
* Copyright (c) 2016 Cisco and/or its affiliates.
|
||||
*
|
||||
* Licensed under the Apache License, Version 2.0 (the "License");
|
||||
* you may not use this file except in compliance with the License.
|
||||
* You may obtain a copy of the License at:
|
||||
*
|
||||
* http://www.apache.org/licenses/LICENSE-2.0
|
||||
*
|
||||
* Unless required by applicable law or agreed to in writing, software
|
||||
* distributed under the License is distributed on an "AS IS" BASIS,
|
||||
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
||||
* See the License for the specific language governing permissions and
|
||||
* limitations under the License.
|
||||
*/
|
||||
|
||||
package org.openvpp.jvpp.test;
|
||||
|
||||
import static java.util.Objects.requireNonNull;
|
||||
|
||||
import org.openvpp.jvpp.JVppImpl;
|
||||
import org.openvpp.jvpp.VppJNIConnection;
|
||||
import org.openvpp.jvpp.dto.CreateSubif;
|
||||
import org.openvpp.jvpp.dto.CreateSubifReply;
|
||||
import org.openvpp.jvpp.dto.SwInterfaceDetailsReplyDump;
|
||||
import org.openvpp.jvpp.dto.SwInterfaceDump;
|
||||
import org.openvpp.jvpp.future.FutureJVppFacade;
|
||||
|
||||
/**
|
||||
* <p>Tests sub-interface creation.<br> Equivalent to:<br>
|
||||
*
|
||||
* <pre>{@code
|
||||
* vppctl create sub GigabitEthernet0/9/0 1 dot1q 100 inner-dot1q any
|
||||
* }
|
||||
* </pre>
|
||||
*
|
||||
* To verify invoke:<br>
|
||||
* <pre>{@code
|
||||
* vpp_api_test json
|
||||
* vat# sw_interface_dump
|
||||
* }
|
||||
*/
|
||||
public class CreateSubInterfaceTest {
|
||||
|
||||
|
||||
private static SwInterfaceDump createSwInterfaceDumpRequest(final String ifaceName) {
|
||||
SwInterfaceDump request = new SwInterfaceDump();
|
||||
request.nameFilter = ifaceName.getBytes();
|
||||
request.nameFilterValid = 1;
|
||||
return request;
|
||||
}
|
||||
|
||||
private static void requireSingleIface(final SwInterfaceDetailsReplyDump response, final String ifaceName) {
|
||||
if (response.swInterfaceDetails.size() != 1) {
|
||||
throw new IllegalStateException(
|
||||
String.format("Expected one interface matching filter %s but was %d", ifaceName,
|
||||
response.swInterfaceDetails.size()));
|
||||
}
|
||||
}
|
||||
|
||||
private static CreateSubif createSubifRequest(final int swIfIndex, final int subId) {
|
||||
CreateSubif request = new CreateSubif();
|
||||
request.swIfIndex = swIfIndex; // super interface id
|
||||
request.subId = subId;
|
||||
request.noTags = 0;
|
||||
request.oneTag = 0;
|
||||
request.twoTags = 1;
|
||||
request.dot1Ad = 0;
|
||||
request.exactMatch = 1;
|
||||
request.defaultSub = 0;
|
||||
request.outerVlanIdAny = 0;
|
||||
request.innerVlanIdAny = 1;
|
||||
request.outerVlanId = 100;
|
||||
request.innerVlanId = 0;
|
||||
return request;
|
||||
}
|
||||
|
||||
private static void print(CreateSubifReply reply) {
|
||||
System.out.printf("CreateSubifReply: context=%d, retval=%d, swIfIndex=%d\n",
|
||||
reply.context,
|
||||
reply.retval,
|
||||
reply.swIfIndex);
|
||||
}
|
||||
|
||||
private static void testCreateSubInterface() throws Exception {
|
||||
System.out.println("Testing sub-interface creation using Java callback API");
|
||||
final JVppImpl jvpp = new JVppImpl(new VppJNIConnection("SubIfaceTest"));
|
||||
final FutureJVppFacade jvppFacade = new FutureJVppFacade(jvpp);
|
||||
|
||||
System.out.println("Successfully connected to VPP");
|
||||
Thread.sleep(1000);
|
||||
|
||||
final String ifaceName = "GigabitEthernet0/9/0";
|
||||
|
||||
final SwInterfaceDetailsReplyDump swInterfaceDetails =
|
||||
jvppFacade.swInterfaceDump(createSwInterfaceDumpRequest(ifaceName)).toCompletableFuture().get();
|
||||
|
||||
requireNonNull(swInterfaceDetails, "swInterfaceDump returned null");
|
||||
requireNonNull(swInterfaceDetails.swInterfaceDetails, "swInterfaceDetails is null");
|
||||
requireSingleIface(swInterfaceDetails, ifaceName);
|
||||
|
||||
final int swIfIndex = swInterfaceDetails.swInterfaceDetails.get(0).swIfIndex;
|
||||
final int subId = 1;
|
||||
|
||||
final CreateSubifReply createSubifReply =
|
||||
jvppFacade.createSubif(createSubifRequest(swIfIndex, subId)).toCompletableFuture().get();
|
||||
print(createSubifReply);
|
||||
|
||||
final String subIfaceName = "GigabitEthernet0/9/0." + subId;
|
||||
final SwInterfaceDetailsReplyDump subIface =
|
||||
jvppFacade.swInterfaceDump(createSwInterfaceDumpRequest(subIfaceName)).toCompletableFuture().get();
|
||||
requireNonNull(swInterfaceDetails, "swInterfaceDump returned null");
|
||||
requireNonNull(subIface.swInterfaceDetails, "swInterfaceDump returned null");
|
||||
requireSingleIface(swInterfaceDetails, ifaceName);
|
||||
|
||||
System.out.println("Disconnecting...");
|
||||
jvpp.close();
|
||||
Thread.sleep(1000);
|
||||
}
|
||||
|
||||
public static void main(String[] args) throws Exception {
|
||||
testCreateSubInterface();
|
||||
}
|
||||
}
|
@ -9,4 +9,5 @@ ControlPingTest - Simple test executing a single control ping using low level JV
|
||||
CallbackApiTest - Similar to ControlPingTest, invokes more complex calls (e.g. interface dump) using low level JVpp APIs
|
||||
FutureApiTest - Execution of more complex calls using Future based JVpp facade
|
||||
CallbackJVppFacadeTest - Execution of more complex calls using Callback based JVpp facade
|
||||
L2AclTest - Tests L2 ACL creation
|
||||
L2AclTest - Tests L2 ACL creation
|
||||
CreateSubInterfaceTest - Tests sub-interface creation
|
Reference in New Issue
Block a user