ofproto: Enqueue improperly resets priority.
authorEthan Jackson <ethan@nicira.com>
Tue, 31 May 2011 23:44:09 +0000 (16:44 -0700)
committerEthan Jackson <ethan@nicira.com>
Wed, 1 Jun 2011 21:18:14 +0000 (14:18 -0700)
commitafabef2b7de84e628b1f2f1ba2e92ec734463eda
treedde127eebff34bd2e38b0d44d774931ac0ad1096
parent80e5eed9c2128f04a1d7da134120d96e961dbe10
ofproto: Enqueue improperly resets priority.

Before this patch, enqueue would reset the priority of a flow to
its original value instead of the value it had immediately before
the enqueue action.

Thus, these openflow actions:
set_queue:2,enqueue:1:1,output:1

Would get translated into these incorrect datapath actions:
set_priority(0x10003),set_priority(0x10002),1,pop_priority,1

Instead of these correct datapath actions:
set_priority(0x10003),set_priority(0x10002),1,set_priority(0x10003),1

A future patch will remove the redundant "set_priority(0x10003)".

Found by inspection.
ofproto/ofproto-dpif.c