diff options
| author | Dmitry Vyukov <dvyukov@google.com> | 2020-04-26 14:14:14 +0200 |
|---|---|---|
| committer | Dmitry Vyukov <dvyukov@google.com> | 2020-05-01 13:31:17 +0200 |
| commit | e54e9781a4e043b3140b0c908ba4f4e469fd317e (patch) | |
| tree | 16e6387d78a8577c5f3d9fb8d05a51752da6338e /sys/linux/init_alg.go | |
| parent | 3f4dbb2f6fff9479d6c250e224bc3cb7f5cd66ed (diff) | |
prog: remove Dir from Type
Having Dir is Type is handy, but forces us to duplicate lots of types.
E.g. if a struct is referenced as both in and out, then we need to
have 2 copies and 2 copies of structs/types it includes.
If also prevents us from having the struct type as struct identity
(because we can have up to 3 of them).
Revert to the old way we used to do it: propagate Dir as we walk
syscall arguments. This moves lots of dir passing from pkg/compiler
to prog package.
Now Arg contains the dir, so once we build the tree, we can use dirs
as before.
Reduces size of sys/linux/gen/amd64.go from 6058336 to 5661150 (-6.6%).
Update #1580
Diffstat (limited to 'sys/linux/init_alg.go')
| -rw-r--r-- | sys/linux/init_alg.go | 41 |
1 files changed, 21 insertions, 20 deletions
diff --git a/sys/linux/init_alg.go b/sys/linux/init_alg.go index 74734907a..0e6e13fac 100644 --- a/sys/linux/init_alg.go +++ b/sys/linux/init_alg.go @@ -9,58 +9,59 @@ import ( "github.com/google/syzkaller/prog" ) -func (arch *arch) generateSockaddrAlg(g *prog.Gen, typ0 prog.Type, old prog.Arg) ( +func (arch *arch) generateSockaddrAlg(g *prog.Gen, typ0 prog.Type, dir prog.Dir, old prog.Arg) ( arg prog.Arg, calls []*prog.Call) { typ := typ0.(*prog.StructType) - family := g.GenerateArg(typ.Fields[0], &calls) + family := g.GenerateArg(typ.Fields[0], dir, &calls) // There is very little point in generating feat/mask, // because that can only fail otherwise correct bind. - feat := prog.MakeConstArg(typ.Fields[2], 0) - mask := prog.MakeConstArg(typ.Fields[3], 0) + feat := prog.MakeConstArg(typ.Fields[2], dir, 0) + mask := prog.MakeConstArg(typ.Fields[3], dir, 0) if g.NOutOf(1, 1000) { - feat = g.GenerateArg(typ.Fields[2], &calls).(*prog.ConstArg) - mask = g.GenerateArg(typ.Fields[3], &calls).(*prog.ConstArg) + feat = g.GenerateArg(typ.Fields[2], dir, &calls).(*prog.ConstArg) + mask = g.GenerateArg(typ.Fields[3], dir, &calls).(*prog.ConstArg) } algType, algName := generateAlgName(g.Rand()) // Extend/truncate type/name to their fixed sizes. algTypeData := fixedSizeData(algType, typ.Fields[1].Size()) algNameData := fixedSizeData(algName, typ.Fields[4].Size()) - arg = prog.MakeGroupArg(typ, []prog.Arg{ + arg = prog.MakeGroupArg(typ, dir, []prog.Arg{ family, - prog.MakeDataArg(typ.Fields[1], algTypeData), + prog.MakeDataArg(typ.Fields[1], dir, algTypeData), feat, mask, - prog.MakeDataArg(typ.Fields[4], algNameData), + prog.MakeDataArg(typ.Fields[4], dir, algNameData), }) return } -func (arch *arch) generateAlgName(g *prog.Gen, typ prog.Type, old prog.Arg) ( +func (arch *arch) generateAlgName(g *prog.Gen, typ prog.Type, dir prog.Dir, old prog.Arg) ( arg prog.Arg, calls []*prog.Call) { - return generateAlgNameStruct(g, typ, allTypes[g.Rand().Intn(len(allTypes))].typ) + return generateAlgNameStruct(g, typ, dir, allTypes[g.Rand().Intn(len(allTypes))].typ) } -func (arch *arch) generateAlgAeadName(g *prog.Gen, typ prog.Type, old prog.Arg) ( +func (arch *arch) generateAlgAeadName(g *prog.Gen, typ prog.Type, dir prog.Dir, old prog.Arg) ( arg prog.Arg, calls []*prog.Call) { - return generateAlgNameStruct(g, typ, ALG_AEAD) + return generateAlgNameStruct(g, typ, dir, ALG_AEAD) } -func (arch *arch) generateAlgHashName(g *prog.Gen, typ prog.Type, old prog.Arg) ( +func (arch *arch) generateAlgHashName(g *prog.Gen, typ prog.Type, dir prog.Dir, old prog.Arg) ( arg prog.Arg, calls []*prog.Call) { - return generateAlgNameStruct(g, typ, ALG_HASH) + return generateAlgNameStruct(g, typ, dir, ALG_HASH) } -func (arch *arch) generateAlgSkcipherhName(g *prog.Gen, typ prog.Type, old prog.Arg) ( +func (arch *arch) generateAlgSkcipherhName(g *prog.Gen, typ prog.Type, dir prog.Dir, old prog.Arg) ( arg prog.Arg, calls []*prog.Call) { - return generateAlgNameStruct(g, typ, ALG_SKCIPHER) + return generateAlgNameStruct(g, typ, dir, ALG_SKCIPHER) } -func generateAlgNameStruct(g *prog.Gen, typ0 prog.Type, algTyp int) (arg prog.Arg, calls []*prog.Call) { +func generateAlgNameStruct(g *prog.Gen, typ0 prog.Type, dir prog.Dir, algTyp int) ( + arg prog.Arg, calls []*prog.Call) { typ := typ0.(*prog.StructType) algName := generateAlg(g.Rand(), algTyp) algNameData := fixedSizeData(algName, typ.Fields[0].Size()) - arg = prog.MakeGroupArg(typ, []prog.Arg{ - prog.MakeDataArg(typ.Fields[0], algNameData), + arg = prog.MakeGroupArg(typ, dir, []prog.Arg{ + prog.MakeDataArg(typ.Fields[0], dir, algNameData), }) return } |
