Add comment for struct_ops

shun159 · shun159 · commit 49faf10f10fc · 2025-10-15T17:53:44.000+09:00
Signed-off-by: shun159 &lt;dreamdiagnosis@gmail.com&gt;
diff --git a/map.go b/map.go
@@ -590,8 +590,9 @@ func (spec *MapSpec) createMap(inner *sys.FD, c *btf.Cache) (_ *Map, err error)
 				attr.ValueTypeBtfObjFd = int32(module.FD())
 			}
 
-			// TODO: This is required to make the map load, but doesn't make
-			// much sense. BtfFd mustn't be 0 either.
+			// This is because when specifying the value type of
+			// a StructOps for BtfVmlinuxValueTypeId, the key/value type IDs cannot be used together.
+			// they must be explicitly set to 0.
 			attr.BtfKeyTypeId = 0
 			attr.BtfValueTypeId = 0
 		}

Original file line number	Diff line number	Diff line change
`@@ -590,8 +590,9 @@ func (spec MapSpec) createMap(inner sys.FD, c btf.Cache) (_ Map, err error)`
`590`	`590`	`attr.ValueTypeBtfObjFd = int32(module.FD())`
`591`	`591`	`}`
`592`	`592`
`593`		`- // TODO: This is required to make the map load, but doesn't make`
`594`		`- // much sense. BtfFd mustn't be 0 either.`
	`593`	`+ // This is because when specifying the value type of`
	`594`	`+ // a StructOps for BtfVmlinuxValueTypeId, the key/value type IDs cannot be used together.`
	`595`	`+ // they must be explicitly set to 0.`
`595`	`596`	`attr.BtfKeyTypeId = 0`
`596`	`597`	`attr.BtfValueTypeId = 0`
`597`	`598`	`}`