Re: [ovs-dev] [PATCH] include/linux/openvswitch.h: Make OVSP_LOCAL 32-bit.

2013-01-07 Thread Justin Pettit
On Jan 7, 2013, at 9:50 AM, Jesse Gross wrote: > On Mon, Jan 7, 2013 at 2:45 AM, Jarno Rajahalme > wrote: >> >> Signed-off-by: Jarno Rajahalme >> --- >> ODP ports are now 32-bit, so OVSP_LOCAL should be too. >> (Internally, kernel module still keeps port numbers 16-bit, though.) > > Justin,

Re: [ovs-dev] [PATCH] include/linux/openvswitch.h: Make OVSP_LOCAL 32-bit.

2013-01-07 Thread Jesse Gross
On Mon, Jan 7, 2013 at 2:45 AM, Jarno Rajahalme wrote: > > Signed-off-by: Jarno Rajahalme > --- > ODP ports are now 32-bit, so OVSP_LOCAL should be too. > (Internally, kernel module still keeps port numbers 16-bit, though.) Applied, thank you. I included your comments about the ODP ports in the

Re: [ovs-dev] [PATCH] include/linux/openvswitch.h: Make OVSP_LOCAL 32-bit.

2013-01-07 Thread Jesse Gross
On Mon, Jan 7, 2013 at 2:45 AM, Jarno Rajahalme wrote: > > Signed-off-by: Jarno Rajahalme > --- > ODP ports are now 32-bit, so OVSP_LOCAL should be too. > (Internally, kernel module still keeps port numbers 16-bit, though.) Justin, I see two direct uses of OVSP_LOCAL in the ofproto directory. I