aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorOle Troan <ot@cisco.com>2019-08-28 14:12:02 +0200
committerAndrew Yourtchenko <ayourtch@gmail.com>2019-09-04 13:02:26 +0000
commitdaa4bff164729c5068413a086d52b0490626250c (patch)
treec748fde80b031118e7cd7649c8e9a6e8eabb39c7
parent27eeb87f43d9d9dfab3d807f33fc5a4a68cb55d9 (diff)
api: memclnt api use string type.
Type: fix Signed-off-by: Ole Troan <ot@cisco.com> Change-Id: Idad79286b7730c8e85202c4b8e675ea50f8bbc48 Signed-off-by: Ole Troan <ot@cisco.com>
-rw-r--r--src/vlibmemory/memclnt.api12
-rw-r--r--src/vpp-api/python/vpp_papi/vpp_papi.py6
-rw-r--r--src/vpp-api/python/vpp_papi/vpp_transport_shmem.py4
-rw-r--r--src/vpp-api/python/vpp_papi/vpp_transport_socket.py2
4 files changed, 12 insertions, 12 deletions
diff --git a/src/vlibmemory/memclnt.api b/src/vlibmemory/memclnt.api
index 2f1e2daa4b6..08b829cfe20 100644
--- a/src/vlibmemory/memclnt.api
+++ b/src/vlibmemory/memclnt.api
@@ -34,7 +34,7 @@ define memclnt_create {
u32 context; /* opaque value to be returned in the reply */
i32 ctx_quota; /* requested punt context quota */
u64 input_queue; /* client's queue */
- u8 name[64]; /* for show, find by name, whatever */
+ string name[64]; /* for show, find by name, whatever */
u32 api_versions[8]; /* client-server pairs use as desired */
};
@@ -102,7 +102,7 @@ autoreply define rpc_call {
define get_first_msg_id {
u32 client_index;
u32 context;
- u8 name[64];
+ string name[64];
};
define get_first_msg_id_reply {
@@ -118,7 +118,7 @@ typedef module_version {
u32 major;
u32 minor;
u32 patch;
- u8 name[64];
+ string name[64];
};
define api_versions {
u32 client_index;
@@ -141,7 +141,7 @@ manual_print define trace_plugin_msg_ids
{
u32 client_index;
u32 context;
- u8 plugin_name[128];
+ string plugin_name[128];
u16 first_msg_id;
u16 last_msg_id;
};
@@ -149,7 +149,7 @@ manual_print define trace_plugin_msg_ids
typedef message_table_entry
{
u16 index;
- u8 name[64];
+ string name[64];
};
/*
@@ -157,7 +157,7 @@ typedef message_table_entry
*/
define sockclnt_create {
u32 context; /* opaque value to be returned in the reply */
- u8 name[64]; /* for show, find by name, whatever */
+ string name[64]; /* for show, find by name, whatever */
};
define sockclnt_create_reply {
diff --git a/src/vpp-api/python/vpp_papi/vpp_papi.py b/src/vpp-api/python/vpp_papi/vpp_papi.py
index b3f2a156939..5af58a41766 100644
--- a/src/vpp-api/python/vpp_papi/vpp_papi.py
+++ b/src/vpp-api/python/vpp_papi/vpp_papi.py
@@ -435,7 +435,7 @@ class VPPApiClient(object):
self._api = VppApiDynamicMethodHolder()
for name, msg in vpp_iterator(self.messages):
n = name + '_' + msg.crc[2:]
- i = self.transport.get_msg_index(n.encode('utf-8'))
+ i = self.transport.get_msg_index(n)
if i > 0:
self.id_msgdef[i] = msg
self.id_names[i] = name
@@ -457,7 +457,7 @@ class VPPApiClient(object):
do_async):
pfx = chroot_prefix.encode('utf-8') if chroot_prefix else None
- rv = self.transport.connect(name.encode('utf-8'), pfx,
+ rv = self.transport.connect(name, pfx,
msg_handler, rx_qlen)
if rv != 0:
raise VPPIOError(2, 'Connect failed')
@@ -467,7 +467,7 @@ class VPPApiClient(object):
# Initialise control ping
crc = self.messages['control_ping'].crc
self.control_ping_index = self.transport.get_msg_index(
- ('control_ping' + '_' + crc[2:]).encode('utf-8'))
+ ('control_ping' + '_' + crc[2:]))
self.control_ping_msgdef = self.messages['control_ping']
if self.async_thread:
self.event_thread = threading.Thread(
diff --git a/src/vpp-api/python/vpp_papi/vpp_transport_shmem.py b/src/vpp-api/python/vpp_papi/vpp_transport_shmem.py
index 64c1a59ba74..4e40f2352e4 100644
--- a/src/vpp-api/python/vpp_papi/vpp_transport_shmem.py
+++ b/src/vpp-api/python/vpp_papi/vpp_transport_shmem.py
@@ -83,7 +83,7 @@ class VppTransport(object):
self.connected = True
if not pfx:
pfx = ffi.NULL
- return vpp_api.vac_connect(name, pfx, msg_handler, rx_qlen)
+ return vpp_api.vac_connect(name.encode('ascii'), pfx, msg_handler, rx_qlen)
def disconnect(self):
self.connected = False
@@ -99,7 +99,7 @@ class VppTransport(object):
return vac_callback_sync if not do_async else vac_callback_async
def get_msg_index(self, name):
- return vpp_api.vac_get_msg_index(name)
+ return vpp_api.vac_get_msg_index(name.encode('ascii'))
def msg_table_max_index(self):
return vpp_api.vac_msg_table_max_index()
diff --git a/src/vpp-api/python/vpp_papi/vpp_transport_socket.py b/src/vpp-api/python/vpp_papi/vpp_transport_socket.py
index 6c06791959d..3c3c8e514db 100644
--- a/src/vpp-api/python/vpp_papi/vpp_transport_socket.py
+++ b/src/vpp-api/python/vpp_papi/vpp_transport_socket.py
@@ -125,7 +125,7 @@ class VppTransport(object):
r, length = sockclnt_create_reply.unpack(msg)
self.socket_index = r.index
for m in r.message_table:
- n = m.name.rstrip(b'\x00\x13')
+ n = m.name
self.message_table[n] = m.index
self.message_thread.daemon = True