Size of buffer is over maxAllocSize

[MichalMrozek]

In compute tests, following pattern is used:

clpeak/src/compute_sp.cpp

Line 24 in bcc21dc

globalWIs = roundToMultipleOf(t, devInfo.maxWGSize);

uint64_t globalWIs = (devInfo.numCUs) * (devInfo.computeWgsPerCU) * (devInfo.maxWGSize);
uint64_t t = MIN((globalWIs * sizeof(cl_float)), devInfo.maxAllocSize) / sizeof(cl_float);
globalWIs = roundToMultipleOf(t, devInfo.maxWGSize);

cl::Buffer outputBuf = cl::Buffer(ctx, CL_MEM_WRITE_ONLY, (globalWIs * sizeof(cl_float)));

t is computed as minimal of globalWIs * sizeof(cl_float) and maxAllocSize, let's assume that maxAllocSize is smaller.

in next line globalWIs is rounded to the next multiple of maxWGSize, it means it may be larger then t which is computed using maxAllocSize.

That will cause the buffer creation to fail, as it will be above maxAllocSize.

[krrishnarraj]

roundToMultipleOf() doesn't round to next multiple, it rounds to an exact or previous multiple.
say if t=100 & devInfo.maxWGSize=64, roundToMultipleOf(t, devInfo.maxWGSize) would be 64

[MichalMrozek]

ok thanks for confirmation