From d666b713716af62708cc9fefe3ab136e7f18b5ae Mon Sep 17 00:00:00 2001 From: Birte Kristina Friesel Date: Tue, 13 May 2025 16:00:22 +0200 Subject: TS: Add AspectC++ support --- README.md | 2 +- TS/Makefile | 25 ++++++++++++--- TS/host/app.c | 46 +++++++++++++++++++--------- TS/include/common.h | 47 ++++++++++++++++++++++++++++ TS/include/dfatool_host.ah | 25 +++++++++++++++ TS/include/params.h | 67 ++++++++++++++++++++++++++++++++++++++++ TS/include/timer.h | 5 +++ TS/support/common.h | 45 --------------------------- TS/support/params.h | 67 ---------------------------------------- TS/support/timer.h | 74 --------------------------------------------- include/dfatool_host_dpu.ah | 16 ++++++++++ 11 files changed, 213 insertions(+), 206 deletions(-) create mode 100644 TS/include/common.h create mode 100644 TS/include/dfatool_host.ah create mode 100644 TS/include/params.h create mode 100644 TS/include/timer.h delete mode 100755 TS/support/common.h delete mode 100644 TS/support/params.h delete mode 100755 TS/support/timer.h diff --git a/README.md b/README.md index 542a980..309d296 100644 --- a/README.md +++ b/README.md @@ -84,7 +84,7 @@ CPU and DPU benchmarks in this repository have been adjusted as follows: * SEL: DLN * SpMV: DL * TRNS: ABDLN -* TS: DLN +* TS: ADLN * UNI: DL * VA: ADLN diff --git a/TS/Makefile b/TS/Makefile index ac081bd..8eedaf9 100644 --- a/TS/Makefile +++ b/TS/Makefile @@ -5,14 +5,29 @@ WITH_ALLOC_OVERHEAD ?= 0 WITH_LOAD_OVERHEAD ?= 0 WITH_FREE_OVERHEAD ?= 0 -COMMON_INCLUDES := support HOST_SOURCES := $(wildcard host/*.c) DPU_SOURCES := $(wildcard dpu/*.c) -COMMON_FLAGS := -Wall -Wextra -g -I${COMMON_INCLUDES} -DBL=${BL} -HOST_FLAGS := ${COMMON_FLAGS} -std=c11 -O3 `dpu-pkg-config --cflags --libs dpu` -DNR_TASKLETS=${NR_TASKLETS} -DNR_DPUS=${NR_DPUS} -DWITH_ALLOC_OVERHEAD=${WITH_ALLOC_OVERHEAD} -DWITH_LOAD_OVERHEAD=${WITH_LOAD_OVERHEAD} -DWITH_FREE_OVERHEAD=${WITH_FREE_OVERHEAD} -lm +aspectc ?= 0 +aspectc_timing ?= 0 +dfatool_timing ?= 1 + +COMMON_FLAGS := -Wall -Wextra -g -Iinclude -DBL=${BL} +HOST_FLAGS := ${COMMON_FLAGS} -O3 `dpu-pkg-config --cflags --libs dpu` -DNR_TASKLETS=${NR_TASKLETS} -DNR_DPUS=${NR_DPUS} -DWITH_ALLOC_OVERHEAD=${WITH_ALLOC_OVERHEAD} -DWITH_LOAD_OVERHEAD=${WITH_LOAD_OVERHEAD} -DWITH_FREE_OVERHEAD=${WITH_FREE_OVERHEAD} -DDFATOOL_TIMING=${dfatool_timing} -DASPECTC=${aspectc} -lm DPU_FLAGS := ${COMMON_FLAGS} -O2 -DNR_TASKLETS=${NR_TASKLETS} +ifeq (${aspectc_timing}, 1) + ASPECTC_HOST_FLAGS += -ainclude/dfatool_host_dpu.ah -ainclude/dfatool_host.ah +endif + +ASPECTC_HOST_FLAGS ?= -a0 + +ifeq (${aspectc}, 1) + HOST_CC = ag++ -r repo.acp -v 0 ${ASPECTC_HOST_FLAGS} --c_compiler ${UPMEM_HOME}/bin/clang++ -p . --Xcompiler +else + HOST_FLAGS += -std=c11 +endif + QUIET = @ ifdef verbose @@ -25,7 +40,9 @@ bin: ${QUIET}mkdir -p bin bin/ts_host: ${HOST_SOURCES} ${COMMON_INCLUDES} bin - ${QUIET}${CC} -o $@ ${HOST_SOURCES} ${HOST_FLAGS} + ${QUIET}cp ../include/dfatool_host_dpu.ah include + ${QUIET}${HOST_CC} -o $@ ${HOST_SOURCES} ${HOST_FLAGS} + ${QUIET}rm -f include/dfatool_host_dpu.ah bin/ts_dpu: ${DPU_SOURCES} ${COMMON_INCLUDES} bin ${QUIET}dpu-upmem-dpurte-clang ${DPU_FLAGS} -o $@ ${DPU_SOURCES} diff --git a/TS/host/app.c b/TS/host/app.c index a19232b..a75cfe9 100644 --- a/TS/host/app.c +++ b/TS/host/app.c @@ -7,8 +7,18 @@ #include #include #include + +#if ASPECTC +extern "C" { +#endif + #include #include + +#if ASPECTC +} +#endif + #include #include #include @@ -86,26 +96,26 @@ static void compute_ts_statistics(unsigned int timeSeriesLength, unsigned int ProfileLength, unsigned int queryLength) { - double *ACumSum = malloc(sizeof(double) * timeSeriesLength); + double *ACumSum = (double*)malloc(sizeof(double) * timeSeriesLength); ACumSum[0] = tSeries[0]; for (uint64_t i = 1; i < timeSeriesLength; i++) ACumSum[i] = tSeries[i] + ACumSum[i - 1]; - double *ASqCumSum = malloc(sizeof(double) * timeSeriesLength); + double *ASqCumSum = (double*)malloc(sizeof(double) * timeSeriesLength); ASqCumSum[0] = tSeries[0] * tSeries[0]; for (uint64_t i = 1; i < timeSeriesLength; i++) ASqCumSum[i] = tSeries[i] * tSeries[i] + ASqCumSum[i - 1]; - double *ASum = malloc(sizeof(double) * ProfileLength); + double *ASum = (double*)malloc(sizeof(double) * ProfileLength); ASum[0] = ACumSum[queryLength - 1]; for (uint64_t i = 0; i < timeSeriesLength - queryLength; i++) ASum[i + 1] = ACumSum[queryLength + i] - ACumSum[i]; - double *ASumSq = malloc(sizeof(double) * ProfileLength); + double *ASumSq = (double*)malloc(sizeof(double) * ProfileLength); ASumSq[0] = ASqCumSum[queryLength - 1]; for (uint64_t i = 0; i < timeSeriesLength - queryLength; i++) ASumSq[i + 1] = ASqCumSum[queryLength + i] - ASqCumSum[i]; - double *AMean_tmp = malloc(sizeof(double) * ProfileLength); + double *AMean_tmp = (double*)malloc(sizeof(double) * ProfileLength); for (uint64_t i = 0; i < ProfileLength; i++) AMean_tmp[i] = ASum[i] / queryLength; - double *ASigmaSq = malloc(sizeof(double) * ProfileLength); + double *ASigmaSq = (double*)malloc(sizeof(double) * ProfileLength); for (uint64_t i = 0; i < ProfileLength; i++) ASigmaSq[i] = ASumSq[i] / queryLength - AMean[i] * AMean[i]; for (uint64_t i = 0; i < ProfileLength; i++) { @@ -136,18 +146,24 @@ int main(int argc, char **argv) // Allocate DPUs and load binary #if !WITH_ALLOC_OVERHEAD DPU_ASSERT(dpu_alloc(NR_DPUS, NULL, &dpu_set)); +#if DFATOOL_TIMING timer.time[0] = 0; // alloc #endif +#endif #if !WITH_LOAD_OVERHEAD DPU_ASSERT(dpu_load(dpu_set, DPU_BINARY, NULL)); DPU_ASSERT(dpu_get_nr_dpus(dpu_set, &nr_of_dpus)); DPU_ASSERT(dpu_get_nr_ranks(dpu_set, &nr_of_ranks)); assert(nr_of_dpus == NR_DPUS); +#if DFATOOL_TIMING timer.time[1] = 0; // load #endif +#endif #if !WITH_FREE_OVERHEAD +#if DFATOOL_TIMING timer.time[6] = 0; // free #endif +#endif #if ENERGY struct dpu_probe_t probe; @@ -195,8 +211,8 @@ int main(int argc, char **argv) unsigned int kernel = 0; dpu_arguments_t input_arguments = - { ts_size, query_length, query_mean, query_std, slice_per_dpu, 0, - kernel + { (uint32_t)ts_size, query_length, query_mean, query_std, slice_per_dpu, 0, + (enum kernels) kernel }; uint32_t mem_offset; @@ -400,22 +416,22 @@ int main(int argc, char **argv) printf("[" ANSI_COLOR_GREEN "OK" ANSI_COLOR_RESET "] results are equal\n"); if (rep >= p.n_warmup) { - printf + dfatool_printf ("[::] TS UPMEM | n_dpus=%d n_ranks=%d n_tasklets=%d e_type=%s block_size_B=%d n_elements=%lu", NR_DPUS, nr_of_ranks, NR_TASKLETS, XSTR(DTYPE), BLOCK_SIZE, ts_size); - printf + dfatool_printf (" b_with_alloc_overhead=%d b_with_load_overhead=%d b_with_free_overhead=%d ", WITH_ALLOC_OVERHEAD, WITH_LOAD_OVERHEAD, WITH_FREE_OVERHEAD); - printf("| latency_alloc_us=%f latency_load_us=%f latency_write_us=%f latency_kernel_us=%f latency_read_us=%f latency_free_us=%f latency_cpu_us=%f ", timer.time[0], // alloc + dfatool_printf("| latency_alloc_us=%f latency_load_us=%f latency_write_us=%f latency_kernel_us=%f latency_read_us=%f latency_free_us=%f latency_cpu_us=%f ", timer.time[0], // alloc timer.time[1], // load timer.time[2], // write timer.time[3], // kernel timer.time[4], // read timer.time[5], // free timer.time[6]); // CPU - printf + dfatool_printf (" throughput_cpu_MBps=%f throughput_upmem_kernel_MBps=%f throughput_upmem_total_MBps=%f", ts_size * sizeof(DTYPE) / timer.time[6], ts_size * sizeof(DTYPE) / (timer.time[3]), @@ -425,7 +441,7 @@ int main(int argc, char **argv) timer.time[3] + timer.time[4] + timer.time[5])); - printf + dfatool_printf (" throughput_upmem_wxr_MBps=%f throughput_upmem_lwxr_MBps=%f throughput_upmem_alwxr_MBps=%f", ts_size * sizeof(DTYPE) / (timer.time[2] + timer.time[3] + @@ -439,14 +455,14 @@ int main(int argc, char **argv) timer.time[2] + timer.time[3] + timer.time[4])); - printf + dfatool_printf (" throughput_cpu_MOpps=%f throughput_upmem_kernel_MOpps=%f throughput_upmem_total_MOpps=%f", ts_size / timer.time[6], ts_size / (timer.time[3]), ts_size / (timer.time[0] + timer.time[1] + timer.time[2] + timer.time[3] + timer.time[4] + timer.time[5])); - printf + dfatool_printf (" throughput_upmem_wxr_MOpps=%f throughput_upmem_lwxr_MOpps=%f throughput_upmem_alwxr_MOpps=%f\n", ts_size / (timer.time[2] + timer.time[3] + timer.time[4]), diff --git a/TS/include/common.h b/TS/include/common.h new file mode 100644 index 0000000..6d37bdc --- /dev/null +++ b/TS/include/common.h @@ -0,0 +1,47 @@ +#ifndef _COMMON_H_ +#define _COMMON_H_ + +// Transfer size between MRAM and WRAM +#ifdef BL +#define BLOCK_SIZE_LOG2 BL +#define BLOCK_SIZE (1 << BLOCK_SIZE_LOG2) +#else +#define BLOCK_SIZE_LOG2 8 +#define BLOCK_SIZE (1 << BLOCK_SIZE_LOG2) +#endif + +// Data type +#define DTYPE int32_t +#define DTYPE_MAX INT32_MAX + +enum kernels { + kernel1 = 0, + nr_kernels = 1, +} kernel; + +typedef struct { + uint32_t ts_length; + uint32_t query_length; + DTYPE query_mean; + DTYPE query_std; + uint32_t slice_per_dpu; + int32_t exclusion_zone; + enum kernels kernel; +} dpu_arguments_t; + +typedef struct { + DTYPE minValue; + uint32_t minIndex; + DTYPE maxValue; + uint32_t maxIndex; +} dpu_result_t; + +#ifndef ENERGY +#define ENERGY 0 +#endif +#define PRINT 0 + +#define ANSI_COLOR_RED "\x1b[31m" +#define ANSI_COLOR_GREEN "\x1b[32m" +#define ANSI_COLOR_RESET "\x1b[0m" +#endif diff --git a/TS/include/dfatool_host.ah b/TS/include/dfatool_host.ah new file mode 100644 index 0000000..2a45918 --- /dev/null +++ b/TS/include/dfatool_host.ah @@ -0,0 +1,25 @@ +#pragma once + +#include +#include "dfatool_host_dpu.ah" + +aspect DfatoolHostTiming : public DfatoolHostDPUTiming { + + DfatoolHostTiming() { + element_size = sizeof(DTYPE); + } + + advice call("% input_params(...)") : after() { + Params* p = tjp->result(); + input_size = p->input_size_n; + printf("[>>] TS | n_dpus=%u n_elements=%lu\n", NR_DPUS, input_size); + } + + advice call("% streamp(...)") : before() { + printf("[--] TS | n_dpus=%u n_elements=%lu\n", NR_DPUS, input_size); + } + + advice execution("% main(...)") : after() { + printf("[<<] TS | n_dpus=%u n_elements=%lu\n", NR_DPUS, input_size); + } +}; diff --git a/TS/include/params.h b/TS/include/params.h new file mode 100644 index 0000000..b7d9763 --- /dev/null +++ b/TS/include/params.h @@ -0,0 +1,67 @@ +#ifndef _PARAMS_H_ +#define _PARAMS_H_ + +#include "common.h" + +// Params --------------------------------------------------------------------- +typedef struct Params { + unsigned long input_size_n; + unsigned long input_size_m; + int n_warmup; + int n_reps; +} Params; + +void usage() +{ + fprintf(stderr, + "\nUsage: ./program [options]" + "\n" + "\nGeneral options:" + "\n -h help" + "\n -w # of untimed warmup iterations (default=1)" + "\n -e # of timed repetition iterations (default=3)" + "\n" + "\nBenchmark-specific options:" + "\n -n n (TS length. Default=64K elements)" + "\n -m m (Query length. Default=256 elements)" "\n"); +} + +struct Params input_params(int argc, char **argv) +{ + struct Params p; + p.input_size_n = 1 << 16; + p.input_size_m = 1 << 8; + + p.n_warmup = 1; + p.n_reps = 3; + + int opt; + while ((opt = getopt(argc, argv, "hw:e:n:m:")) >= 0) { + switch (opt) { + case 'h': + usage(); + exit(0); + break; + case 'w': + p.n_warmup = atoi(optarg); + break; + case 'e': + p.n_reps = atoi(optarg); + break; + case 'n': + p.input_size_n = atol(optarg); + break; + case 'm': + p.input_size_m = atol(optarg); + break; + default: + fprintf(stderr, "\nUnrecognized option!\n"); + usage(); + exit(0); + } + } + assert(NR_DPUS > 0 && "Invalid # of dpus!"); + + return p; +} +#endif diff --git a/TS/include/timer.h b/TS/include/timer.h new file mode 100644 index 0000000..7b80823 --- /dev/null +++ b/TS/include/timer.h @@ -0,0 +1,5 @@ +#pragma once + +#define N_TIMERS 7 +#include "../../include/timer_base.h" +#undef N_TIMERS diff --git a/TS/support/common.h b/TS/support/common.h deleted file mode 100755 index 7585b90..0000000 --- a/TS/support/common.h +++ /dev/null @@ -1,45 +0,0 @@ -#ifndef _COMMON_H_ -#define _COMMON_H_ - -// Transfer size between MRAM and WRAM -#ifdef BL -#define BLOCK_SIZE_LOG2 BL -#define BLOCK_SIZE (1 << BLOCK_SIZE_LOG2) -#else -#define BLOCK_SIZE_LOG2 8 -#define BLOCK_SIZE (1 << BLOCK_SIZE_LOG2) -#endif - -// Data type -#define DTYPE int32_t -#define DTYPE_MAX INT32_MAX - -typedef struct { - uint32_t ts_length; - uint32_t query_length; - DTYPE query_mean; - DTYPE query_std; - uint32_t slice_per_dpu; - int32_t exclusion_zone; - enum kernels { - kernel1 = 0, - nr_kernels = 1, - } kernel; -} dpu_arguments_t; - -typedef struct { - DTYPE minValue; - uint32_t minIndex; - DTYPE maxValue; - uint32_t maxIndex; -} dpu_result_t; - -#ifndef ENERGY -#define ENERGY 0 -#endif -#define PRINT 0 - -#define ANSI_COLOR_RED "\x1b[31m" -#define ANSI_COLOR_GREEN "\x1b[32m" -#define ANSI_COLOR_RESET "\x1b[0m" -#endif diff --git a/TS/support/params.h b/TS/support/params.h deleted file mode 100644 index b7d9763..0000000 --- a/TS/support/params.h +++ /dev/null @@ -1,67 +0,0 @@ -#ifndef _PARAMS_H_ -#define _PARAMS_H_ - -#include "common.h" - -// Params --------------------------------------------------------------------- -typedef struct Params { - unsigned long input_size_n; - unsigned long input_size_m; - int n_warmup; - int n_reps; -} Params; - -void usage() -{ - fprintf(stderr, - "\nUsage: ./program [options]" - "\n" - "\nGeneral options:" - "\n -h help" - "\n -w # of untimed warmup iterations (default=1)" - "\n -e # of timed repetition iterations (default=3)" - "\n" - "\nBenchmark-specific options:" - "\n -n n (TS length. Default=64K elements)" - "\n -m m (Query length. Default=256 elements)" "\n"); -} - -struct Params input_params(int argc, char **argv) -{ - struct Params p; - p.input_size_n = 1 << 16; - p.input_size_m = 1 << 8; - - p.n_warmup = 1; - p.n_reps = 3; - - int opt; - while ((opt = getopt(argc, argv, "hw:e:n:m:")) >= 0) { - switch (opt) { - case 'h': - usage(); - exit(0); - break; - case 'w': - p.n_warmup = atoi(optarg); - break; - case 'e': - p.n_reps = atoi(optarg); - break; - case 'n': - p.input_size_n = atol(optarg); - break; - case 'm': - p.input_size_m = atol(optarg); - break; - default: - fprintf(stderr, "\nUnrecognized option!\n"); - usage(); - exit(0); - } - } - assert(NR_DPUS > 0 && "Invalid # of dpus!"); - - return p; -} -#endif diff --git a/TS/support/timer.h b/TS/support/timer.h deleted file mode 100755 index c569de7..0000000 --- a/TS/support/timer.h +++ /dev/null @@ -1,74 +0,0 @@ -/* - * Copyright (c) 2016 University of Cordoba and University of Illinois - * All rights reserved. - * - * Developed by: IMPACT Research Group - * University of Cordoba and University of Illinois - * http://impact.crhc.illinois.edu/ - * - * Permission is hereby granted, free of charge, to any person obtaining a copy - * of this software and associated documentation files (the "Software"), to deal - * with the Software without restriction, including without limitation the - * rights to use, copy, modify, merge, publish, distribute, sublicense, and/or - * sell copies of the Software, and to permit persons to whom the Software is - * furnished to do so, subject to the following conditions: - * - * > Redistributions of source code must retain the above copyright notice, - * this list of conditions and the following disclaimers. - * > Redistributions in binary form must reproduce the above copyright - * notice, this list of conditions and the following disclaimers in the - * documentation and/or other materials provided with the distribution. - * > Neither the names of IMPACT Research Group, University of Cordoba, - * University of Illinois nor the names of its contributors may be used - * to endorse or promote products derived from this Software without - * specific prior written permission. - * - * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR - * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, - * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE - * CONTRIBUTORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER - * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, - * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS WITH - * THE SOFTWARE. - * - */ - -#include - -typedef struct Timer { - - struct timeval startTime[7]; - struct timeval stopTime[7]; - double time[7]; - -} Timer; - -void start(Timer *timer, int i, int rep) -{ - if (rep == 0) { - timer->time[i] = 0.0; - } - gettimeofday(&timer->startTime[i], NULL); -} - -void stop(Timer *timer, int i) -{ - gettimeofday(&timer->stopTime[i], NULL); - timer->time[i] += - (timer->stopTime[i].tv_sec - - timer->startTime[i].tv_sec) * 1000000.0 + - (timer->stopTime[i].tv_usec - timer->startTime[i].tv_usec); -} - -void print(Timer *timer, int i, int REP) -{ - printf("%f\t", timer->time[i] / (1000 * REP)); -} - -void printall(Timer *timer, int maxt) -{ - for (int i = 0; i <= maxt; i++) { - printf(" timer%d_us=%f", i, timer->time[i]); - } - printf("\n"); -} diff --git a/include/dfatool_host_dpu.ah b/include/dfatool_host_dpu.ah index 560e410..0e03d31 100644 --- a/include/dfatool_host_dpu.ah +++ b/include/dfatool_host_dpu.ah @@ -88,6 +88,22 @@ aspect DfatoolHostDPUTiming { ); } + advice call("% dpu_copy_to(...)") : around() { + size_t payload_size = *(tjp->arg<4>()); + gettimeofday(&starttime, NULL); + tjp->proceed(); + gettimeofday(&stoptime, NULL); + double time_us = (stoptime.tv_sec - starttime.tv_sec) * 1000000.0 + (stoptime.tv_usec - starttime.tv_usec); + printf("[::] dpu_copy_to @ %s:%d | n_dpus=%u n_ranks=%u payload_B=%lu | latency_us=%f throughput_MiBps=%f\n", + tjp->filename(), + tjp->line(), + n_dpus, n_ranks, + payload_size, + time_us, + payload_size / (time_us * M_to_Mi) + ); + } + advice call("% dpu_push_xfer(...)") : around() { size_t payload_size = *(tjp->arg<4>()); gettimeofday(&starttime, NULL); -- cgit v1.2.3