Hi Ferruh, I added to documentation the new parameter "--geneve-port=N" (second patch in the series). I also rebased the series and sent v3.
Regards, Ophir > -----Original Message----- > From: Ferruh Yigit <ferruh.yi...@intel.com> > > I don't think that having the flexibility to specify the Geneve protocol in > > the > command line will have an added value. > > The reason is that any Geneve setup has a fixed protocol number for the > duration of the test lifetime (fixed and synched by all the peers in the lab). > The protocol number is not dynamic. > > So even if we could specify the protocol in the command line - it would be > a onetime setting after testpmd startup. Therefore, it is simpler to have it > as > parameter only. > > > > Overall, not specific to this patch, it bothers me not having any guideline > what to add as a parameter, and not able to say what is implemented as > parameter without searching for it ... > > > Regards, > > Ophir > >
