diff mbox series

[ovs-dev] actions: Remove unused ovnfield_act_header structure.

Message ID 20230508111455.2745657-1-dceara@redhat.com
State Accepted
Headers show
Series [ovs-dev] actions: Remove unused ovnfield_act_header structure. | expand

Checks

Context Check Description
ovsrobot/apply-robot success apply and check: success
ovsrobot/github-robot-_Build_and_Test success github build: passed
ovsrobot/github-robot-_ovn-kubernetes success github build: passed

Commit Message

Dumitru Ceara May 8, 2023, 11:14 a.m. UTC
It's not referenced anywhere and by the look of it it has never been.

Signed-off-by: Dumitru Ceara <dceara@redhat.com>
---
 include/ovn/actions.h | 6 ------
 1 file changed, 6 deletions(-)

Comments

Simon Horman May 16, 2023, 4:25 p.m. UTC | #1
On Mon, May 08, 2023 at 01:14:55PM +0200, Dumitru Ceara wrote:
> It's not referenced anywhere and by the look of it it has never been.
> 
> Signed-off-by: Dumitru Ceara <dceara@redhat.com>

Reviewed-by: Simon Horman <simon.horman@corigine.com>
Dumitru Ceara May 17, 2023, 3:07 p.m. UTC | #2
On 5/16/23 18:25, Simon Horman wrote:
> On Mon, May 08, 2023 at 01:14:55PM +0200, Dumitru Ceara wrote:
>> It's not referenced anywhere and by the look of it it has never been.
>>
>> Signed-off-by: Dumitru Ceara <dceara@redhat.com>
> 
> Reviewed-by: Simon Horman <simon.horman@corigine.com>
> 

Thanks, Simon, for the review.  I applied the patch to the main branch.

Regards,
Dumitru
diff mbox series

Patch

diff --git a/include/ovn/actions.h b/include/ovn/actions.h
index 28479ede15..b421fbdc6e 100644
--- a/include/ovn/actions.h
+++ b/include/ovn/actions.h
@@ -755,12 +755,6 @@  struct action_header {
 };
 BUILD_ASSERT_DECL(sizeof(struct action_header) == 8);
 
-OVS_PACKED(
-struct ovnfield_act_header {
-    ovs_be16 id; /* one of enum ovnfield_id. */
-    ovs_be16 len; /* Length of the ovnfield data. */
-});
-
 struct ovnact_parse_params {
     /* A table of "struct expr_symbol"s to support (as one would provide to
      * expr_parse()). */