Reviewed, very thanks for fixing this bug. On Tue, 2013-10-22 at 12:02 +0800, Ruiling Song wrote: > scratchSize was missed in the binary, which will cause a random value > when kernel is loaded from binary. add it in the binary format. > > Signed-off-by: Ruiling Song <[email protected]> > --- > backend/src/backend/program.cpp | 3 +++ > backend/src/backend/program.hpp | 2 ++ > 2 files changed, 5 insertions(+) > > diff --git a/backend/src/backend/program.cpp b/backend/src/backend/program.cpp > index b2879a4..b9925d6 100644 > --- a/backend/src/backend/program.cpp > +++ b/backend/src/backend/program.cpp > @@ -244,6 +244,7 @@ namespace gbe { > OUT_UPDATE_SZ(curbeSize); > OUT_UPDATE_SZ(simdWidth); > OUT_UPDATE_SZ(stackSize); > + OUT_UPDATE_SZ(scratchSize); > OUT_UPDATE_SZ(useSLM); > OUT_UPDATE_SZ(slmSize); > > @@ -332,6 +333,7 @@ namespace gbe { > IN_UPDATE_SZ(curbeSize); > IN_UPDATE_SZ(simdWidth); > IN_UPDATE_SZ(stackSize); > + IN_UPDATE_SZ(scratchSize); > IN_UPDATE_SZ(useSLM); > IN_UPDATE_SZ(slmSize); > > @@ -408,6 +410,7 @@ namespace gbe { > outs << spaces_nl << " curbeSize: " << curbeSize << "\n"; > outs << spaces_nl << " simdWidth: " << simdWidth << "\n"; > outs << spaces_nl << " stackSize: " << stackSize << "\n"; > + outs << spaces_nl << " scratchSize: " << scratchSize << "\n"; > outs << spaces_nl << " useSLM: " << useSLM << "\n"; > outs << spaces_nl << " slmSize: " << slmSize << "\n"; > > diff --git a/backend/src/backend/program.hpp b/backend/src/backend/program.hpp > index 895cd01..9b33b7c 100644 > --- a/backend/src/backend/program.hpp > +++ b/backend/src/backend/program.hpp > @@ -147,7 +147,9 @@ namespace gbe { > curbeSize | > simdWidth | > stackSize | > + scratchSize | > useSLM | > + slmSize | > samplers | > images | > code_size |
_______________________________________________ Beignet mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/beignet
