search for: memptr

...00e+00 Attempting IC_opt on: %call = call i32 %5(%class.A* %this.adjusted, i32 1, i32 2), !prof !8 with: !{!"indirect_call_targets", i64 1, !"_ZN1A8test_subEii", i64 1} in function: _Z8testmainM1AFiiiE CS hotness% = 1.000000e+02 Target hotness% = 100 == Basic Block Before == memptr.end: ; preds = %memptr.nonvirtual, %memptr.virtual %5 = phi i32 (%class.A*, i32, i32)* [ %memptr.virtualfn, %memptr.virtual ], [ %memptr.nonvirtualfn, %memptr.nonvirtual ] %call = call i32 %5(%class.A* %this.adjusted, i32 1, i32 2), !prof !8 ret i32 %call...

...> Nonlocal queries have a null instruction. OK, i'll work on it. > > + } else if (StoreInst *S = dyn_cast<StoreInst>(dep.getInst())) { > ... > + uint64_t ptrSize = AA.getTypeStoreSize(pointer->getType()); > ... > + uint64_t memSize = AA.getTypeStoreSize(memPtr->getType()); > ... > + if (AA.alias(pointer, ptrSize, memPtr, memSize) != > AliasAnalysis::MustAlias) > > This is passing in the size of the pointer, not the pointee. > However, since you only care about must alias, you don't care about > the size of the object...

....code64 + +EXCEPTION_HANDLER boot_page_fault do_boot_page_fault error_code=1 diff --git a/arch/x86/boot/compressed/misc.h b/arch/x86/boot/compressed/misc.h index 3a030a878d53..345c90fbc500 100644 --- a/arch/x86/boot/compressed/misc.h +++ b/arch/x86/boot/compressed/misc.h @@ -37,6 +37,9 @@ #define memptr unsigned #endif +/* boot/compressed/vmlinux start and end markers */ +extern char _head[], _end[]; + /* misc.c */ extern memptr free_mem_ptr; extern memptr free_mem_end_ptr; @@ -146,4 +149,7 @@ extern pteval_t __default_kernel_pte_mask; extern gate_desc boot_idt[BOOT_IDT_ENTRIES]; extern st...

...any case, the check for dep.isNonLocal() can be removed. Nonlocal queries have a null instruction. + } else if (StoreInst *S = dyn_cast<StoreInst>(dep.getInst())) { ... + uint64_t ptrSize = AA.getTypeStoreSize(pointer->getType()); ... + uint64_t memSize = AA.getTypeStoreSize(memPtr->getType()); ... + if (AA.alias(pointer, ptrSize, memPtr, memSize) != AliasAnalysis::MustAlias) This is passing in the size of the pointer, not the pointee. However, since you only care about must alias, you don't care about the size of the object, you can just pass in 1 for both...

...quot;page_fault" would give you all then. > diff --git a/arch/x86/boot/compressed/misc.h b/arch/x86/boot/compressed/misc.h > index 3a030a878d53..eff4ed0b1cea 100644 > --- a/arch/x86/boot/compressed/misc.h > +++ b/arch/x86/boot/compressed/misc.h > @@ -37,6 +37,9 @@ > #define memptr unsigned > #endif > > +/* boot/compressed/vmlinux start and end markers */ > +extern char _head[], _end[]; > + > /* misc.c */ > extern memptr free_mem_ptr; > extern memptr free_mem_end_ptr; > @@ -146,4 +149,7 @@ extern pteval_t __default_kernel_pte_mask; > exter...

Hello, I fixed my patch as you asked. Sorry for the delay, I'd been working on my SSU patch (http://lists.cs.uiuc.edu/pipermail/llvmdev/2009-August/025347.html ) I hope that everything is fine now. -Jakub -------------- next part -------------- A non-text attachment was scrubbed... Name: pr2218-3.patch Type: application/octet-stream Size: 7511 bytes Desc: not available URL:

...n the switch -- see below). It seems this should be returning a "def" instead. My thinking is the code should look like this (from line 288 on, +++ marks addition): // See if this instruction (e.g. a call or vaarg) mod/ref's the pointer. switch (AA->getModRefInfo(Inst, MemPtr, MemSize)) { case AliasAnalysis::NoModRef: // If the call has no effect on the queried pointer, just ignore it. continue; case AliasAnalysis::Mod: // If we're in an invariant region, we can ignore calls that ONLY // modify the pointer. if (InvariantT...

...L }; spawn_load(p, 1, argv); } } diff --git a/com32/lib/sys/module/elfutils.c b/com32/lib/sys/module/elfutils.c index 64be077..b7d760b 100644 --- a/com32/lib/sys/module/elfutils.c +++ b/com32/lib/sys/module/elfutils.c @@ -37,7 +37,7 @@ struct memalign_info { }; int elf_malloc(void **memptr, size_t alignment, size_t size) { - void *start_addr = NULL; + char *start_addr = NULL; struct memalign_info *info; if ((alignment & (alignment - 1)) != 0) @@ -63,7 +63,7 @@ int elf_malloc(void **memptr, size_t alignment, size_t size) { return 0; } -void elf_free(void *memptr) { +voi...

Hi Josh et al, I'm compiling on an Ubuntu Edgy x86 machine. Edgy comes standard with gcc-4.1 and I'm getting a number of warnings: memory.c: In function 'FLAC__memory_alloc_aligned': memory.c:52: warning: cast from pointer to integer of different size memory.c:52: warning: cast to pointer from integer of different size metadata_iterators.c: In function

...It seems this should be returning a "def" > instead.  My thinking is the code should look like this (from line 288 on, > +++ marks addition): > >      // See if this instruction (e.g. a call or vaarg) mod/ref's the > pointer. >      switch (AA->getModRefInfo(Inst, MemPtr, MemSize)) { >      case AliasAnalysis::NoModRef: >        // If the call has no effect on the queried pointer, just ignore it. >        continue; >      case AliasAnalysis::Mod: >        // If we're in an invariant region, we can ignore calls that ONLY >        // modify the...

...) { + phdr = &phdrs[i]; + + switch (phdr->p_type) { + case PT_LOAD: + memcpy((void*)phdr->p_paddr, + output + phdr->p_offset, + phdr->p_filesz); + break; + default: /* Ignore other PT_* */ break; + } + } +} + asmlinkage void decompress_kernel(void *rmode, memptr heap, uch *input_data, unsigned long input_len, uch *output) @@ -408,6 +456,7 @@ asmlinkage void decompress_kernel(void *rmode, memptr heap, makecrc(); putstr("\nDecompressing Linux... "); gunzip(); + parse_elf(output); putstr("done.\nBooting the kernel.\n"...

...) { + phdr = &phdrs[i]; + + switch (phdr->p_type) { + case PT_LOAD: + memcpy((void*)phdr->p_paddr, + output + phdr->p_offset, + phdr->p_filesz); + break; + default: /* Ignore other PT_* */ break; + } + } +} + asmlinkage void decompress_kernel(void *rmode, memptr heap, uch *input_data, unsigned long input_len, uch *output) @@ -408,6 +456,7 @@ asmlinkage void decompress_kernel(void *rmode, memptr heap, makecrc(); putstr("\nDecompressing Linux... "); gunzip(); + parse_elf(output); putstr("done.\nBooting the kernel.\n"...

...+= (phdr->p_paddr - LOAD_PHYSICAL_ADDR); +#else + dest = (void*)(phdr->p_paddr); +#endif + memcpy(dest, + output + phdr->p_offset, + phdr->p_filesz); + break; + default: /* Ignore other PT_* */ break; + } + } +} + asmlinkage void decompress_kernel(void *rmode, memptr heap, uch *input_data, unsigned long input_len, uch *output) @@ -408,6 +463,7 @@ asmlinkage void decompress_kernel(void *rmode, memptr heap, makecrc(); putstr("\nDecompressing Linux... "); gunzip(); + parse_elf(output); putstr("done.\nBooting the kernel.\n"...

...+= (phdr->p_paddr - LOAD_PHYSICAL_ADDR); +#else + dest = (void*)(phdr->p_paddr); +#endif + memcpy(dest, + output + phdr->p_offset, + phdr->p_filesz); + break; + default: /* Ignore other PT_* */ break; + } + } +} + asmlinkage void decompress_kernel(void *rmode, memptr heap, uch *input_data, unsigned long input_len, uch *output) @@ -408,6 +463,7 @@ asmlinkage void decompress_kernel(void *rmode, memptr heap, makecrc(); putstr("\nDecompressing Linux... "); gunzip(); + parse_elf(output); putstr("done.\nBooting the kernel.\n"...

..."def" > > instead. My thinking is the code should look like this (from line 288 > on, > > +++ marks addition): > > > > // See if this instruction (e.g. a call or vaarg) mod/ref's the > > pointer. > > switch (AA->getModRefInfo(Inst, MemPtr, MemSize)) { > > case AliasAnalysis::NoModRef: > > // If the call has no effect on the queried pointer, just ignore > it. > > continue; > > case AliasAnalysis::Mod: > > // If we're in an invariant region, we can ignore calls that O...

...stead.  My thinking is the code should look like this (from line 288 >> > on, >> > +++ marks addition): >> > >> >      // See if this instruction (e.g. a call or vaarg) mod/ref's the >> > pointer. >> >      switch (AA->getModRefInfo(Inst, MemPtr, MemSize)) { >> >      case AliasAnalysis::NoModRef: >> >        // If the call has no effect on the queried pointer, just ignore >> > it. >> >        continue; >> >      case AliasAnalysis::Mod: >> >        // If we're in an invariant regio...

...+= (phdr->p_paddr - LOAD_PHYSICAL_ADDR); +#else + dest = (void*)(phdr->p_paddr); +#endif + memcpy(dest, + output + phdr->p_offset, + phdr->p_filesz); + break; + default: /* Ignore other PT_* */ break; + } + } +} + asmlinkage void decompress_kernel(void *rmode, memptr heap, uch *input_data, unsigned long input_len, uch *output) @@ -408,6 +463,7 @@ asmlinkage void decompress_kernel(void *rmode, memptr heap, makecrc(); putstr("\nDecompressing Linux... "); gunzip(); + parse_elf(output); putstr("done.\nBooting the kernel.\n"...

...+= (phdr->p_paddr - LOAD_PHYSICAL_ADDR); +#else + dest = (void*)(phdr->p_paddr); +#endif + memcpy(dest, + output + phdr->p_offset, + phdr->p_filesz); + break; + default: /* Ignore other PT_* */ break; + } + } +} + asmlinkage void decompress_kernel(void *rmode, memptr heap, uch *input_data, unsigned long input_len, uch *output) @@ -408,6 +463,7 @@ asmlinkage void decompress_kernel(void *rmode, memptr heap, makecrc(); putstr("\nDecompressing Linux... "); gunzip(); + parse_elf(output); putstr("done.\nBooting the kernel.\n"...

...age2_vc error_code=1 #endif diff --git a/arch/x86/boot/compressed/misc.c b/arch/x86/boot/compressed/misc.c index 9652d5c2afda..dba49e75095a 100644 --- a/arch/x86/boot/compressed/misc.c +++ b/arch/x86/boot/compressed/misc.c @@ -441,6 +441,13 @@ asmlinkage __visible void *extract_kernel(void *rmode, memptr heap, parse_elf(output); handle_relocations(output, output_len, virt_addr); debug_putstr("done.\nBooting the kernel.\n"); + + /* + * Flush GHCB from cache and map it encrypted again when running as + * SEV-ES guest. + */ + sev_es_shutdown_ghcb(); + return output; } diff --gi...

This patch series introduces the VHD library. It is based on the blktap2 one, with changes coming from the blktap2.5 one. Signed-off-by: Thanos Makatos <thanos.makatos@citrix.com>