Re: [netsniff-ng] Re: [PATCH 00/15] trafgen: Support dinc & drnd for proto fields

2016-08-09 Thread Vadim Kochan
On Tue, Aug 9, 2016 at 5:11 PM, Tobias Klauser wrote: > Hi Vadim > > On 2016-07-26 at 21:35:05 +0200, Vadim Kochan wrote: >> Implemented 'dinc' and 'drnd' functions to be used for proto fields, >> and generate values at runtime. Parsing of proto field

[netsniff-ng] Re: [PATCH 00/15] trafgen: Support dinc & drnd for proto fields

2016-08-09 Thread Tobias Klauser
Hi Vadim On 2016-07-26 at 21:35:05 +0200, Vadim Kochan wrote: > Implemented 'dinc' and 'drnd' functions to be used for proto fields, > and generate values at runtime. Parsing of proto field values > for unified to make extending of field functions more easier w/o > copy/paste

Re: [netsniff-ng] Re: [PATCH 00/15] trafgen: Support dinc & drnd for proto fields

2016-07-27 Thread Tobias Klauser
Nice work Vadim! This looks good from a bird's-eye view, but due to the size of the changes it will probably take me some time to review it completely and give you feedback (with the exception of patch 1/15 which is a no-brainer and I already applied it). On 2016-07-27 at 13:04:45 +0200, Vadim

[netsniff-ng] Re: [PATCH 00/15] trafgen: Support dinc & drnd for proto fields

2016-07-27 Thread Vadim Kochan
I really missed one thing related to csum - so if I have proto field with dynamic function then csum will be calculated minimum twice - after packet is build and after all dynamic fields are applied, well I have no particular solution for this, but let me know if you want to have it fixed in this