search for: inputoffset

...sym->setFile(FILE_MEMORY_SHARED); > + break; > + case TGSI_MEMORY_TYPE_INPUT: > + assert(prog->getType() == Program::TYPE_COMPUTE); > + assert(idx == -1); > + sym->setFile(FILE_SHADER_INPUT); > + address += info->prop.cp.inputOffset; What's the idea here? i.e. what is the inputOffset, how is it set, and why? -ilia > + break; > + default: > + assert(0); /* TODO: Add support for global and local memory */ > + } > + } > > if (idx >= 0) { > if (sym->reg....

...>>+ if (tgsiFile == TGSI_FILE_MEMORY) { > >>+ switch (code->memoryFiles[fileIdx].mem_type) { > >>+ case TGSI_MEMORY_TYPE_SHARED: > >>+ sym->setFile(FILE_MEMORY_SHARED); You might want to increment the address by at least `info->prop.cp.inputOffset`, and if inputs still end up in shared on Tesla, then increment further by the input size. This input offset of 0x10 (or is it 0x20?) is due to the card sticking the size of a block and of the grid inside `s[0x0..0x10]` (or maybe Nouveau is doing that, but I doubt it.). So even if the user inputs e...

...;> + break; >>> + case TGSI_MEMORY_TYPE_INPUT: >>> + assert(prog->getType() == Program::TYPE_COMPUTE); >>> + assert(idx == -1); >>> + sym->setFile(FILE_SHADER_INPUT); >>> + address += info->prop.cp.inputOffset; >> >> >> What's the idea here? i.e. what is the inputOffset, how is it set, and >> why? > > > I don't get the idea too, btw. > > But prop.cp.inputOffset is only defined for compute on Kepler. It's the > offset of input parameters in the screen-...

...== TGSI_FILE_MEMORY) { >>>> + switch (code->memoryFiles[fileIdx].mem_type) { >>>> + case TGSI_MEMORY_TYPE_SHARED: >>>> + sym->setFile(FILE_MEMORY_SHARED); > > You might want to increment the address by at least > `info->prop.cp.inputOffset`, and if inputs still end up in shared on Tesla, > then increment further by the input size. This input offset of 0x10 (or is it > 0x20?) is due to the card sticking the size of a block and of the grid inside > `s[0x0..0x10]` (or maybe Nouveau is doing that, but I doubt it.). So even if &g...

..._MEMORY_SHARED); >> + break; >> + case TGSI_MEMORY_TYPE_INPUT: >> + assert(prog->getType() == Program::TYPE_COMPUTE); >> + assert(idx == -1); >> + sym->setFile(FILE_SHADER_INPUT); >> + address += info->prop.cp.inputOffset; > > What's the idea here? i.e. what is the inputOffset, how is it set, and why? I don't get the idea too, btw. But prop.cp.inputOffset is only defined for compute on Kepler. It's the offset of input parameters in the screen->parm BO but as you already know, it is going to...

...k; >>>> + case TGSI_MEMORY_TYPE_INPUT: >>>> + assert(prog->getType() == Program::TYPE_COMPUTE); >>>> + assert(idx == -1); >>>> + sym->setFile(FILE_SHADER_INPUT); >>>> + address += info->prop.cp.inputOffset; >>> >>> >>> What's the idea here? i.e. what is the inputOffset, how is it set, and >>> why? >> >> >> I don't get the idea too, btw. >> >> But prop.cp.inputOffset is only defined for compute on Kepler. It's the >> o...

Hi I apologise if the following message is inappropriate for this list. However, I feel this is the only place where I can get some help with my problem. BOOL CALLBACK RecordProc(HRECORD hRecord, void *buffer, DWORD length, DWORD user) { try { long sent = 0; long inputOffset = 0; long outputOffset = 0; long nbBytes = 0; char *outputBuffer = NULL; outputBuffer = (char*)malloc(65535); memset(speexBuffer, 0, 65536); while (inputOffset < length) { (float*)buffer = (float*)buffer + (frameSize); speex...

...GSI_MEMORY_TYPE_SHARED: + sym->setFile(FILE_MEMORY_SHARED); + break; + case TGSI_MEMORY_TYPE_INPUT: + assert(prog->getType() == Program::TYPE_COMPUTE); + assert(idx == -1); + sym->setFile(FILE_SHADER_INPUT); + address += info->prop.cp.inputOffset; + break; + default: + assert(0); /* TODO: Add support for global and local memory */ + } + } if (idx >= 0) { if (sym->reg.file == FILE_SHADER_INPUT) -- 2.7.2

On Thu, Mar 10, 2016 at 11:03 AM, Pierre Moreau <pierre.morrow at free.fr> wrote: > You might want to increment the address by at least > `info->prop.cp.inputOffset`, and if inputs still end up in shared on Tesla, There's a cp.sharedOffset just for that :) However it doesn't appear to get set anywhere...

...GSI_MEMORY_TYPE_SHARED: + sym->setFile(FILE_MEMORY_SHARED); + break; + case TGSI_MEMORY_TYPE_INPUT: + assert(prog->getType() == Program::TYPE_COMPUTE); + assert(idx == -1); + sym->setFile(FILE_SHADER_INPUT); + address += info->prop.cp.inputOffset; + break; + default: + assert(0); /* TODO: Add support for global and private memory */ + } + } if (idx >= 0) { if (sym->reg.file == FILE_SHADER_INPUT) -- 2.7.2

Hi, Here are patches which implement the support for OpenCL kernel input parameters we discussed. They also add the tgsi parsing bits for adding support for global / local mem, but no implementation yet. Regards, Hans

When support for decl.Atomic and .Shared was added, tgsi_build_declaration was not updated to propagate these properly. Signed-off-by: Hans de Goede <hdegoede at redhat.com> Reviewed-by: Ilia Mirkin <imirkin at alum.mit.edu> --- Changes in v2: -Add Reviewed-by: Ilia Mirkin <imirkin at alum.mit.edu> --- src/gallium/auxiliary/tgsi/tgsi_build.c | 6 ++++++ 1 file changed, 6