ligang/vpp
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

lacp: cleanup unnecessary pointer cast

Browse Source 
Change-Id: Idc3a7588dc73e7180a15b6ace3684d3c12986b9d
Signed-off-by: Zhiyong Yang <zhiyong.yang@intel.com>
This commit is contained in:
Zhiyong Yang
2019-06-10 22:23:11 -04:00
committed by Damjan Marion
parent 16ad61735e
commit 91c5129136
5 changed files with 38 additions and 46 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/plugins/lacp/mux_machine.c
+9 -10
View File
@@ -96,8 +96,8 @@ lacp_attach_mux_to_aggregator (vlib_main_t * vm, slave_if_t * sif)
int
lacp_mux_action_detached (void *p1, void *p2)
{
vlib_main_t *vm = (vlib_main_t *) p1;
slave_if_t *sif = (slave_if_t *) p2;
vlib_main_t *vm = p1;
slave_if_t *sif = p2;
lacp_main_t *lm = &lacp_main;
lacp_detach_mux_from_aggregator (vm, sif);
@@ -121,8 +121,8 @@ lacp_mux_action_detached (void *p1, void *p2)
int
lacp_mux_action_attached (void *p1, void *p2)
{
vlib_main_t *vm = (vlib_main_t *) p1;
slave_if_t *sif = (slave_if_t *) p2;
vlib_main_t *vm = p1;
slave_if_t *sif = p2;
lacp_main_t *lm = &lacp_main;
lacp_attach_mux_to_aggregator (vm, sif);
@@ -147,8 +147,8 @@ lacp_mux_action_attached (void *p1, void *p2)
int
lacp_mux_action_waiting (void *p1, void *p2)
{
vlib_main_t *vm = (vlib_main_t *) p1;
slave_if_t *sif = (slave_if_t *) p2;
vlib_main_t *vm = p1;
slave_if_t *sif = p2;
lacp_main_t *lm = &lacp_main;
if (!lacp_timer_is_running (sif->wait_while_timer))
@@ -169,8 +169,8 @@ lacp_mux_action_waiting (void *p1, void *p2)
int
lacp_mux_action_collecting_distributing (void *p1, void *p2)
{
vlib_main_t *vm = (vlib_main_t *) p1;
slave_if_t *sif = (slave_if_t *) p2;
vlib_main_t *vm = p1;
slave_if_t *sif = p2;
lacp_main_t *lm = &lacp_main;
sif->actor.state |= LACP_STATE_SYNCHRONIZATION | LACP_STATE_COLLECTING |
@@ -198,8 +198,7 @@ format_mux_event (u8 * s, va_list * args)
{.str = NULL}
};
int e = va_arg (*args, int);
lacp_event_struct *event_entry =
(lacp_event_struct *) & lacp_mux_event_array;
lacp_event_struct *event_entry = lacp_mux_event_array;
if (e >= (sizeof (lacp_mux_event_array) / sizeof (*event_entry)))
s = format (s, "Bad event %d", e);
src/plugins/lacp/node.h
+4 -8
View File
@@ -186,8 +186,7 @@ format_rx_sm_state (u8 * s, va_list * args)
{.str = NULL}
};
int state = va_arg (*args, int);
lacp_state_struct *state_entry =
(lacp_state_struct *) & lacp_rx_sm_state_array;
lacp_state_struct *state_entry = lacp_rx_sm_state_array;
if (state >= (sizeof (lacp_rx_sm_state_array) / sizeof (*state_entry)))
s = format (s, "Bad state %d", state);
@@ -207,8 +206,7 @@ format_tx_sm_state (u8 * s, va_list * args)
{.str = NULL}
};
int state = va_arg (*args, int);
lacp_state_struct *state_entry =
(lacp_state_struct *) & lacp_tx_sm_state_array;
lacp_state_struct *state_entry = lacp_tx_sm_state_array;
if (state >= (sizeof (lacp_tx_sm_state_array) / sizeof (*state_entry)))
s = format (s, "Bad state %d", state);
@@ -228,8 +226,7 @@ format_mux_sm_state (u8 * s, va_list * args)
{.str = NULL}
};
int state = va_arg (*args, int);
lacp_state_struct *state_entry =
(lacp_state_struct *) & lacp_mux_sm_state_array;
lacp_state_struct *state_entry = lacp_mux_sm_state_array;
if (state >= (sizeof (lacp_mux_sm_state_array) / sizeof (*state_entry)))
s = format (s, "Bad state %d", state);
@@ -249,8 +246,7 @@ format_ptx_sm_state (u8 * s, va_list * args)
{.str = NULL}
};
int state = va_arg (*args, int);
lacp_state_struct *state_entry =
(lacp_state_struct *) & lacp_ptx_sm_state_array;
lacp_state_struct *state_entry = lacp_ptx_sm_state_array;
if (state >= (sizeof (lacp_ptx_sm_state_array) / sizeof (*state_entry)))
s = format (s, "Bad state %d", state);
src/plugins/lacp/ptx_machine.c
+9 -10
View File
@@ -74,8 +74,8 @@ lacp_machine_t lacp_ptx_machine = {
int
lacp_ptx_action_no_periodic (void *p1, void *p2)
{
vlib_main_t *vm = (vlib_main_t *) p1;
slave_if_t *sif = (slave_if_t *) p2;
vlib_main_t *vm = p1;
slave_if_t *sif = p2;
lacp_stop_timer (&sif->periodic_timer);
lacp_ptx_post_short_timeout_event (vm, sif);
@@ -85,8 +85,8 @@ lacp_ptx_action_no_periodic (void *p1, void *p2)
int
lacp_ptx_action_slow_periodic (void *p1, void *p2)
{
vlib_main_t *vm = (vlib_main_t *) p1;
slave_if_t *sif = (slave_if_t *) p2;
vlib_main_t *vm = p1;
slave_if_t *sif = p2;
u8 timer_expired;
lacp_main_t *lm = &lacp_main;
@@ -115,8 +115,8 @@ lacp_ptx_action_slow_periodic (void *p1, void *p2)
int
lacp_ptx_action_fast_periodic (void *p1, void *p2)
{
vlib_main_t *vm = (vlib_main_t *) p1;
slave_if_t *sif = (slave_if_t *) p2;
vlib_main_t *vm = p1;
slave_if_t *sif = p2;
u8 timer_expired;
lacp_main_t *lm = &lacp_main;
@@ -150,8 +150,8 @@ lacp_ptx_action_fast_periodic (void *p1, void *p2)
int
lacp_ptx_action_timer_expired (void *p1, void *p2)
{
vlib_main_t *vm = (vlib_main_t *) p1;
slave_if_t *sif = (slave_if_t *) p2;
vlib_main_t *vm = p1;
slave_if_t *sif = p2;
if (!(sif->partner.state & LACP_STATE_LACP_ACTIVITY) &&
!(sif->actor.state & LACP_STATE_LACP_ACTIVITY))
@@ -183,8 +183,7 @@ format_ptx_event (u8 * s, va_list * args)
{.str = NULL}
};
int e = va_arg (*args, int);
lacp_event_struct *event_entry =
(lacp_event_struct *) & lacp_ptx_event_array;
lacp_event_struct *event_entry = lacp_ptx_event_array;
if (e >= (sizeof (lacp_ptx_event_array) / sizeof (*event_entry)))
s = format (s, "Bad event %d", e);
src/plugins/lacp/rx_machine.c
+13 -14
View File
@@ -240,8 +240,8 @@ lacp_set_port_moved (vlib_main_t * vm, slave_if_t * sif, u8 val)
int
lacp_rx_action_initialize (void *p1, void *p2)
{
vlib_main_t *vm = (vlib_main_t *) p1;
slave_if_t *sif = (slave_if_t *) p2;
vlib_main_t *vm = p1;
slave_if_t *sif = p2;
lacp_set_port_unselected (vm, sif);
lacp_record_default (sif);
@@ -257,8 +257,8 @@ lacp_rx_action_initialize (void *p1, void *p2)
int
lacp_rx_action_port_disabled (void *p1, void *p2)
{
vlib_main_t *vm = (vlib_main_t *) p1;
slave_if_t *sif = (slave_if_t *) p2;
vlib_main_t *vm = p1;
slave_if_t *sif = p2;
sif->partner.state &= ~LACP_STATE_SYNCHRONIZATION;
if (sif->port_moved)
@@ -282,8 +282,8 @@ lacp_rx_action_port_disabled (void *p1, void *p2)
int
lacp_rx_action_expired (void *p1, void *p2)
{
vlib_main_t *vm = (vlib_main_t *) p1;
slave_if_t *sif = (slave_if_t *) p2;
vlib_main_t *vm = p1;
slave_if_t *sif = p2;
u8 timer_expired;
lacp_main_t *lm = &lacp_main;
@@ -310,8 +310,8 @@ lacp_rx_action_expired (void *p1, void *p2)
int
lacp_rx_action_lacp_disabled (void *p1, void *p2)
{
vlib_main_t *vm = (vlib_main_t *) p1;
slave_if_t *sif = (slave_if_t *) p2;
vlib_main_t *vm = p1;
slave_if_t *sif = p2;
lacp_set_port_unselected (vm, sif);
lacp_record_default (sif);
@@ -324,8 +324,8 @@ lacp_rx_action_lacp_disabled (void *p1, void *p2)
int
lacp_rx_action_defaulted (void *p1, void *p2)
{
vlib_main_t *vm = (vlib_main_t *) p1;
slave_if_t *sif = (slave_if_t *) p2;
vlib_main_t *vm = p1;
slave_if_t *sif = p2;
lacp_stop_timer (&sif->current_while_timer);
lacp_update_default_selected (vm, sif);
@@ -363,8 +363,8 @@ lacp_port_is_moved (vlib_main_t * vm, slave_if_t * sif)
int
lacp_rx_action_current (void *p1, void *p2)
{
vlib_main_t *vm = (vlib_main_t *) p1;
slave_if_t *sif = (slave_if_t *) p2;
vlib_main_t *vm = p1;
slave_if_t *sif = p2;
lacp_main_t *lm = &lacp_main;
lacp_update_selected (vm, sif);
@@ -389,8 +389,7 @@ format_rx_event (u8 * s, va_list * args)
{.str = NULL}
};
int e = va_arg (*args, int);
lacp_event_struct *event_entry =
(lacp_event_struct *) & lacp_rx_event_array;
lacp_event_struct *event_entry = lacp_rx_event_array;
if (e >= (sizeof (lacp_rx_event_array) / sizeof (*event_entry)))
s = format (s, "Bad event %d", e);
src/plugins/lacp/tx_machine.c
+3 -4
View File
@@ -38,8 +38,8 @@ lacp_machine_t lacp_tx_machine = {
int
lacp_tx_action_transmit (void *p1, void *p2)
{
vlib_main_t *vm = (vlib_main_t *) p1;
slave_if_t *sif = (slave_if_t *) p2;
vlib_main_t *vm = p1;
slave_if_t *sif = p2;
lacp_main_t *lm = &lacp_main;
f64 now = vlib_time_now (lm->vlib_main);
@@ -70,8 +70,7 @@ format_tx_event (u8 * s, va_list * args)
{.str = NULL}
};
int e = va_arg (*args, int);
lacp_event_struct *event_entry =
(lacp_event_struct *) & lacp_tx_event_array;
lacp_event_struct *event_entry = lacp_tx_event_array;
if (e >= (sizeof (lacp_tx_event_array) / sizeof (*event_entry)))
s = format (s, "Bad event %d", e);