spcl/rFaaS

Add C++ allocator

Open

#18 opened on Mar 6, 2023

View on GitHub
 (7 comments) (0 reactions) (1 assignee)C++ (19 forks)auto 404
enhancementgood first issue

Repository metrics

Stars
 (59 stars)
PR merge metrics
 (PR metrics pending)

Description

Currently, we expose a very low-level interface for allocating memory - as shown in this documentation:

  rdmalib::Buffer<char> in(opts.input_size, rdmalib::functions::Submission::DATA_HEADER_SIZE), out(opts.input_size);
  in.register_memory(executor._state.pd(), IBV_ACCESS_LOCAL_WRITE);
  out.register_memory(executor._state.pd(), IBV_ACCESS_LOCAL_WRITE | IBV_ACCESS_REMOTE_WRITE);

While rdmalib::Buffer is applicable, we need a higher-level concept based on the CX++ std::allocator to integrate RDMA-managed memory into user applications.

  • Add allocator implementation in rfaaslib.
  • Encapsulate the memory registration in rdmalib - provide generic enums for local/remote read/write ops + atomics.
  • Support in allocator registration and deregistration of memory.
  • Add test demonstrating standard memory allocation.
  • Add test demonstrating allocation with std::vector and our custom allocator.

Contributor guide

Research direction
Study the existing rdmalib::Buffer and IBV ACCESS flags, then implement an allocator that registers memory with ibv reg mr and wraps it as a std::allocator. Follow the checklist in the issue to add tests using std::vector.
Tech stack
cpp
Domain
backend
Issue type
Feature
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
3
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
1-2 days
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
Active
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
Clear
Prerequisites
C++RDMA
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
55