As this function is a relatively lean wrapper around setting ins.Offset
and doesn't do any particular sanity checking to see if e.g. the offset
is within bounds, it's not clear what benefit this provides being part
of the exported API.

As its only use is within the test suite where the instruction stream
is known up front, remove the function for now.

Signed-off-by: Timo Beckers <timo@isovalent.com>

As miscTypes cannot be specified by the caller, there's no strong reason
to input check these arguments. Calls to createMiscProbeAttr() will fail
when the miscTyps is not implemented anyway.

Signed-off-by: Timo Beckers <timo@isovalent.com>

Signed-off-by: Timo Beckers <timo@isovalent.com>

The error string was changed when it was exported in cilium#541 and is potentially
ambiguous.

Signed-off-by: Timo Beckers <timo@isovalent.com>

I keep reading the switch in FixupReferences wrong: I always assume that
any instruction with a reference will trigger the unsatisfied reference
error. This isn't the case since only the program reference switch cases
execute a break.

Make the whole thing easier to understand by copying the error message
into two places.