On Fri, Nov 30, 2012 at 04:20:33PM -0800, Jesse Gross wrote:
> On Fri, Nov 30, 2012 at 4:13 PM, Ben Pfaff <b...@nicira.com> wrote:
> > On Fri, Nov 30, 2012 at 04:07:29PM -0800, Ansis Atteka wrote:
> >> This patch adds logging support for skb_mark and skb_priority.
> >>
> >> Signed-off-by: Ansis Atteka
> >
> > skb_priority has two 16-bit subfields, at least with the qdiscs that
> > I'm familiar with, so it's easier to read when it's printed in
> > hexadecimal.  This means that I'd change "%"PRIu32 to "%#"PRIx32 and
> > MFS_DECIMAL to MFS_HEXADECIMAL for this field.
> 
> If we do that (which sounds fine), we probably should also update
> odp-util.c to print priority in hex.  

Good idea.

> It might not be a bad idea to make mark hex as well.

I don't know anything about the semantics of skb_mark.  I'll take your
word for it.
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to