xdp icon indicating copy to clipboard operation
xdp copied to clipboard

Create a new release to make sure it's in sync with the latest code

Open atoonk opened this issue 2 years ago • 1 comments

Just a heads-up: A while back (2,5 years), GetDescs(n int) was changed to GetDescs(n int, rx bool). It looks like the release was never updated to include this change, so the latest release still uses the old way GetDescs(n int) I'm assuming this is the recommended way. Also, as a result, folks copying the example from the repo will see an error.

atoonk avatar Feb 24 '24 03:02 atoonk

Just in case someone else has the same problem when following the example

# command-line-arguments
./sendudp.go:101:39: too many arguments in call to xsk.GetDescs
        have (number, bool)
        want (int)
./sendudp.go:130:47: too many arguments in call to xsk.GetDescs
        have (int, bool)
        want (int)

You can add the following to your go.mod , which will make sure you use a more recent version with the fix: github.com/asavie/xdp v0.3.4-0.20220212172814-56d71236a029 example: https://github.com/atoonk/go-pktgen/blob/main/go.mod#L6

Btw, this is not a problem if you clone the repo, in that case you have the latest code.

atoonk avatar Feb 26 '24 16:02 atoonk