search for: nameoffset

...> > > > Thanks a lot for the attention and congratulations for the great work!!! > > > > Best regards > > Mello > > While this will make it compile, the code is not valid to begin with. > It is trying to do a raw memory copy of a non standard-layout type. > nameoffset is not guaranteed to directly follow cmdsize. > > Fixing this will require quite a few changes, so for now I've > committed your change. > > - Michael Spencer > -------------- next part -------------- An HTML attachment was scrubbed... URL: <http://lists.llvm.org/pipermail/...

...h index 754c6af42f30..259b25c2ac6b 100644 --- a/drivers/gpu/drm/nouveau/include/nvrm/535.113.01/nvidia/generated/g_os_nvoc.h +++ b/drivers/gpu/drm/nouveau/include/nvrm/535.113.01/nvidia/generated/g_os_nvoc.h @@ -28,17 +28,17 @@ typedef struct PACKED_REGISTRY_ENTRY { - NvU32 nameOffset; - NvU8 type; - NvU32 data; - NvU32 length; + NvU32 nameOffset; + NvU8 type; + NvU32 data; + NvU32 length; } PACKED_REGISTRY_ENTRY; typedef struct PACKED_REGI...

...ame.size()], 0, cmdsize-(12+_name.size())); > > Thanks a lot for the attention and congratulations for the great work!!! > > Best regards > Mello While this will make it compile, the code is not valid to begin with. It is trying to do a raw memory copy of a non standard-layout type. nameoffset is not guaranteed to directly follow cmdsize. Fixing this will require quite a few changes, so for now I've committed your change. - Michael Spencer

On Jun 16, 2012, at 8:23 PM, Cesar Mello wrote: > While this will make it compile, the code is not valid to begin with. > It is trying to do a raw memory copy of a non standard-layout type. > nameoffset is not guaranteed to directly follow cmdsize. Are you saying that in: struct A { int f1; // offset 0 int f2; // offset 4 }; struct B : public A { int f3; // offset 8 ? }; that f3 is not guaranteed to have the same offset it would have had, had it been the third field of A (...

Hi! I'm trying to build lld Microsoft Visual Studio 2012 RC, but it seems the bzero function is not available. Could memset be used instead of bzero? Or maybe define a bzero for msvc using memset. For example: // in-memory matches on-disk, so copy first fields followed by path ::memcpy(to, (uint8_t*)&cmd, 12); ::memcpy(&to[12], _name.data(), _name.size());

...100644 > --- a/drivers/gpu/drm/nouveau/include/nvrm/535.113.01/nvidia/generated/g_os_nvoc.h > +++ b/drivers/gpu/drm/nouveau/include/nvrm/535.113.01/nvidia/generated/g_os_nvoc.h > @@ -28,17 +28,17 @@ > > typedef struct PACKED_REGISTRY_ENTRY > { > - NvU32 nameOffset; > - NvU8 type; > - NvU32 data; > - NvU32 length; > + NvU32 nameOffset; > + NvU8 type; > + NvU32 data; > + NvU32 length; > } PACKED_REGIS...

...t_size(registry, entries, registry->numEntries); + + list_for_each_entry_safe(reg, n, &gsp->registry_list, list) { + registry->entries[i].type = reg->type; + registry->entries[i].data = reg->data; + registry->entries[i].length = reg->length; + registry->entries[i].nameOffset = str_offset; + memcpy((void *)registry + str_offset, reg->name, reg->name_len); + str_offset += reg->name_len; + i++; + + list_del(&reg->list); + kfree(reg); + } + + /* Double-check that we calculated the sizes correctly */ + WARN_ON(gsp->registry_rpc_size != str_offset); +...

...= struct_size(registry, entries, registry->numEntries); + + list_for_each_entry_safe(reg, n, &gsp->registry_list, head) { + registry->entries[i].type = reg->type; + registry->entries[i].length = reg->klen; + + /* Copy the key name to the table */ + registry->entries[i].nameOffset = str_offset; + memcpy((void *)registry + str_offset, reg->key, reg->klen); + str_offset += reg->klen; + + switch (reg->type) { + case REGISTRY_TABLE_ENTRY_TYPE_DWORD: + registry->entries[i].data = reg->v.dword; + break; + case REGISTRY_TABLE_ENTRY_TYPE_BINARY: + case RE...