Aaron Conole <acon...@redhat.com> 于2022年2月15日周二 01:33写道：
> Peng He <xnhp0...@gmail.com> writes:
>
> > In the commit 1df7f7aac8c976690167261fec9a50d832ef9e7e, the packet
> > metadata's in_port has been changed from ofp port into odp port,
> > however, the odp->flow->in_port is still ofp_port. This patch changes
> > the odp->flow->in_port into odp_port.
> >
> > Fixes: 1df7f7aac8 ("ofproto-dpif: Restore packet metadata when a
> > continuation is resumed.")
>
> This tag is still wrapped, and I don't think the fixes tag is for the
> correct commit.
>
Oh, sorry, I thought this was Ilya's reply.
please check the link:
http://patchwork.ozlabs.org/project/openvswitch/patch/20220213023642.4248-1-hepeng.0...@bytedance.com/
shows the tag is ok? am I missing something?
>
> > Signed-off-by: Peng He <hepeng.0...@bytedance.com>
> > ---
> > ofproto/ofproto-dpif.c | 1 +
>
> Also, seems there is no unit test as requested. Can you please address
> these?
>
> > 1 file changed, 1 insertion(+)
> >
> > diff --git a/ofproto/ofproto-dpif.c b/ofproto/ofproto-dpif.c
> > index cba49a99e..72de67c63 100644
> > --- a/ofproto/ofproto-dpif.c
> > +++ b/ofproto/ofproto-dpif.c
> > @@ -4973,6 +4973,7 @@ packet_execute_prepare(struct ofproto *ofproto_,
> > ofproto_dpif_set_packet_odp_port(ofproto,
> opo->flow->in_port.ofp_port,
> > opo->packet);
> >
> > + opo->flow->in_port = opo->packet->md.in_port;
> > ofproto_dpif_packet_out_delete(aux);
> > opo->aux = execute;
> > }
>
>
--
hepeng
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
