X-Git-Url: http://info.iut-bm.univ-fcomte.fr/pub/gitweb/simgrid.git/blobdiff_plain/a92d7b716f51a53dea7f59db8524d4add713b910..0bb08c80b2bff9de24524dc98f04cd145dfc93c0:/src/mc/mc_snapshot.cpp diff --git a/src/mc/mc_snapshot.cpp b/src/mc/mc_snapshot.cpp index 80c0817be5..09e9e17bef 100644 --- a/src/mc/mc_snapshot.cpp +++ b/src/mc/mc_snapshot.cpp @@ -1,4 +1,4 @@ -/* Copyright (c) 2014-2015. The SimGrid Team. +/* Copyright (c) 2014-2018. The SimGrid Team. * All rights reserved. */ /* This program is free software; you can redistribute it and/or modify it @@ -13,12 +13,12 @@ #include "xbt/sysdep.h" #include "src/internal_config.h" -#include "src/smpi/private.h" +#include "src/smpi/include/private.hpp" -#include "src/mc/mc_snapshot.h" -#include "src/mc/mc_private.h" -#include "src/mc/mc_mmu.h" #include "src/mc/PageStore.hpp" +#include "src/mc/mc_mmu.hpp" +#include "src/mc/mc_private.hpp" +#include "src/mc/mc_snapshot.hpp" extern "C" { @@ -121,10 +121,12 @@ int MC_snapshot_region_memcmp( // Using alloca() for large allocations may trigger stack overflow: // use malloc if the buffer is too big. bool stack_alloc = size < 64; - const bool region1_need_buffer = region1==nullptr || region1->storage_type()==simgrid::mc::StorageType::Flat; - const bool region2_need_buffer = region2==nullptr || region2->storage_type()==simgrid::mc::StorageType::Flat; - void* buffer1a = region1_need_buffer ? nullptr : stack_alloc ? alloca(size) : malloc(size); - void* buffer2a = region2_need_buffer ? nullptr : stack_alloc ? alloca(size) : malloc(size); + void* buffer1a = nullptr; + void* buffer2a = nullptr; + if (region1 != nullptr && region1->storage_type() != simgrid::mc::StorageType::Flat) + buffer1a = stack_alloc ? alloca(size) : ::operator new(size); + if (region2 != nullptr && region2->storage_type() != simgrid::mc::StorageType::Flat) + buffer2a = stack_alloc ? alloca(size) : ::operator new(size); const void* buffer1 = MC_region_read(region1, buffer1a, addr1, size); const void* buffer2 = MC_region_read(region2, buffer2a, addr2, size); int res; @@ -133,33 +135,18 @@ int MC_snapshot_region_memcmp( else res = memcmp(buffer1, buffer2, size); if (not stack_alloc) { - free(buffer1a); - free(buffer2a); + ::operator delete(buffer1a); + ::operator delete(buffer2a); } return res; } -/** Compare memory between snapshots - * - * @param addr1 Address in the first snapshot - * @param snapshot1 First snapshot - * @param addr2 Address in the second snapshot - * @param snapshot2 Second snapshot - * @return same as memcmp - * */ -int MC_snapshot_memcmp( - const void* addr1, simgrid::mc::Snapshot* snapshot1, - const void* addr2, simgrid::mc::Snapshot* snapshot2, int process_index, size_t size) -{ - mc_mem_region_t region1 = mc_get_snapshot_region(addr1, snapshot1, process_index); - mc_mem_region_t region2 = mc_get_snapshot_region(addr2, snapshot2, process_index); - return MC_snapshot_region_memcmp(addr1, region1, addr2, region2, size); -} +} // extern "C" namespace simgrid { namespace mc { -Snapshot::Snapshot(Process* process, int _num_state) +Snapshot::Snapshot(RemoteClient* process, int _num_state) : AddressSpace(process) , num_state(_num_state) , heap_bytes_used(0) @@ -192,18 +179,16 @@ const void* Snapshot::read_bytes(void* buffer, std::size_t size, } } -} - #ifdef SIMGRID_TEST -#include -#include +#include +#include #include -#include "src/mc/mc_private.h" -#include "src/mc/mc_snapshot.h" -#include "src/mc/mc_mmu.h" +#include "src/mc/mc_mmu.hpp" +#include "src/mc/mc_private.hpp" +#include "src/mc/mc_snapshot.hpp" XBT_TEST_SUITE("mc_snapshot", "Snapshots"); @@ -234,7 +219,7 @@ static void test_snapshot(bool sparse_checkpoint) { xbt_assert(xbt_pagesize == getpagesize()); xbt_assert(1 << xbt_pagebits == xbt_pagesize); - std::unique_ptr process(new simgrid::mc::Process(getpid(), -1)); + std::unique_ptr process(new simgrid::mc::RemoteClient(getpid(), -1)); process->init(); mc_model_checker = new ::simgrid::mc::ModelChecker(std::move(process));