Skip to content

[SYCL][ESIMD][EMU] Merging buffer/image info to surface info #5930

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 4 commits into from
May 2, 2022
Merged

[SYCL][ESIMD][EMU] Merging buffer/image info to surface info #5930

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
9cfabff
[SYCL][ESIMD][EMU] Merging buffer/image info to surface info
dongkyunahn-intel Mar 30, 2022
9ca2466
Camel-type for surface type enum
dongkyunahn-intel Mar 30, 2022
2bc6f54
Fixing constructor/destructor for _pi_object/_pi_mem
dongkyunahn-intel Mar 31, 2022
c1eebfc
Destructor for _pi_mem struct
dongkyunahn-intel Mar 31, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
83 changes: 42 additions & 41 deletions sycl/plugins/esimd_emulator/pi_esimd_emulator.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1000,17 +1000,17 @@ pi_result piMemBufferCreate(pi_context Context, pi_mem_flags Flags, size_t Size,
}

char *MapBasePtr = nullptr;
cm_buffer_ptr_slot CmBuf;
cm_surface_ptr_t CmBuf;
cm_support::SurfaceIndex *CmIndex;
int Status = cm_support::CM_FAILURE;

if (Flags & PI_MEM_FLAGS_HOST_PTR_USE) {
CmBuf.tag = cm_buffer_ptr_slot::type_user_provided;
CmBuf.tag = cm_surface_ptr_t::type_user_provided_buffer;
Status = Context->Device->CmDevicePtr->CreateBufferUP(
static_cast<unsigned int>(Size), HostPtr, CmBuf.UPBufPtr);
CmBuf.UPBufPtr->GetIndex(CmIndex);
} else {
CmBuf.tag = cm_buffer_ptr_slot::type_regular;
CmBuf.tag = cm_surface_ptr_t::type_regular_buffer;
Status = Context->Device->CmDevicePtr->CreateBuffer(
static_cast<unsigned int>(Size), CmBuf.RegularBufPtr);
CmBuf.RegularBufPtr->GetIndex(CmIndex);
Expand Down Expand Up @@ -1068,24 +1068,23 @@ pi_result piMemRelease(pi_mem Mem) {
if (--(Mem->RefCount) == 0) {
int Status = cm_support::CM_FAILURE;

if (Mem->getMemType() == PI_MEM_TYPE_BUFFER) {
if (Mem->SurfacePtr.tag == cm_surface_ptr_t::type_user_provided_buffer) {
_pi_buffer *PiBuf = static_cast<_pi_buffer *>(Mem);
if (PiBuf->BufferPtr.tag == cm_buffer_ptr_slot::type_user_provided) {
Status = Mem->Context->Device->CmDevicePtr->DestroyBufferUP(
PiBuf->BufferPtr.UPBufPtr);
} else {
Status = Mem->Context->Device->CmDevicePtr->DestroySurface(
PiBuf->BufferPtr.RegularBufPtr);
}
} else if (Mem->getMemType() == PI_MEM_TYPE_IMAGE2D) {
Status = Mem->Context->Device->CmDevicePtr->DestroyBufferUP(
PiBuf->SurfacePtr.UPBufPtr);
} else if (Mem->SurfacePtr.tag == cm_surface_ptr_t::type_regular_buffer) {
_pi_buffer *PiBuf = static_cast<_pi_buffer *>(Mem);
Status = Mem->Context->Device->CmDevicePtr->DestroySurface(
PiBuf->SurfacePtr.RegularBufPtr);
} else if (Mem->SurfacePtr.tag ==
cm_surface_ptr_t::type_user_provided_image) {
_pi_image *PiImg = static_cast<_pi_image *>(Mem);
if (PiImg->ImagePtr.tag == cm_image_ptr_slot::type_user_provided) {
Status = Mem->Context->Device->CmDevicePtr->DestroySurface2DUP(
PiImg->ImagePtr.UPImgPtr);
} else {
Status = Mem->Context->Device->CmDevicePtr->DestroySurface(
PiImg->ImagePtr.RegularImgPtr);
}
Status = Mem->Context->Device->CmDevicePtr->DestroySurface2DUP(
PiImg->SurfacePtr.UPImgPtr);
} else if (Mem->SurfacePtr.tag == cm_surface_ptr_t::type_regular_image) {
_pi_image *PiImg = static_cast<_pi_image *>(Mem);
Status = Mem->Context->Device->CmDevicePtr->DestroySurface(
PiImg->SurfacePtr.RegularImgPtr);
}

if (Status != cm_support::CM_SUCCESS) {
Expand Down Expand Up @@ -1208,18 +1207,19 @@ pi_result piMemImageCreate(pi_context Context, pi_mem_flags Flags,
}

char *MapBasePtr = nullptr;
cm_image_ptr_slot CmImg;
cm_surface_ptr_t CmImg;
cm_support::SurfaceIndex *CmIndex;
int Status = cm_support::CM_SUCCESS;

if (Flags & PI_MEM_FLAGS_HOST_PTR_USE) {
CmImg.tag = cm_image_ptr_slot::type_user_provided;
CmImg.tag = cm_surface_ptr_t::type_user_provided_image;
Status = Context->Device->CmDevicePtr->CreateSurface2DUP(
static_cast<unsigned int>(ImageDesc->image_width),
static_cast<unsigned int>(ImageDesc->image_height), CmSurfFormat,
HostPtr, CmImg.UPImgPtr);
CmImg.UPImgPtr->GetIndex(CmIndex);
} else {
CmImg.tag = cm_surface_ptr_t::type_regular_image;
Status = Context->Device->CmDevicePtr->CreateSurface2D(
static_cast<unsigned int>(ImageDesc->image_width),
static_cast<unsigned int>(ImageDesc->image_height), CmSurfFormat,
Expand Down Expand Up @@ -1495,12 +1495,13 @@ pi_result piEnqueueMemBufferRead(pi_queue Queue, pi_mem Src,
RetEv->IsDummyEvent = true;
}

if (buf->BufferPtr.tag == cm_buffer_ptr_slot::type_user_provided) {
if (buf->SurfacePtr.tag == cm_surface_ptr_t::type_user_provided_buffer) {
// CM does not provide 'ReadSurface' call for 'User-Provided'
// Surface. memcpy is used for BufferRead PI_API call.
memcpy(Dst, buf->MapHostPtr, Size);
} else {
int Status = buf->BufferPtr.RegularBufPtr->ReadSurface(
assert(buf->SurfacePtr.tag == cm_surface_ptr_t::type_regular_buffer);
int Status = buf->SurfacePtr.RegularBufPtr->ReadSurface(
reinterpret_cast<unsigned char *>(Dst),
nullptr, // event
static_cast<uint64_t>(Size));
Expand Down Expand Up @@ -1668,12 +1669,13 @@ pi_result piEnqueueMemImageRead(pi_queue CommandQueue, pi_mem Image,
}

size_t Size = RowPitch * (Region->height);
if (PiImg->ImagePtr.tag == cm_image_ptr_slot::type_user_provided) {
if (PiImg->SurfacePtr.tag == cm_surface_ptr_t::type_user_provided_image) {
// CM does not provide 'ReadSurface' call for 'User-Provided'
// Surface. memcpy is used for ImageRead PI_API call.
memcpy(Ptr, PiImg->MapHostPtr, Size);
} else {
int Status = PiImg->ImagePtr.RegularImgPtr->ReadSurface(
assert(PiImg->SurfacePtr.tag == cm_surface_ptr_t::type_regular_image);
int Status = PiImg->SurfacePtr.RegularImgPtr->ReadSurface(
reinterpret_cast<unsigned char *>(Ptr),
nullptr, // event
static_cast<uint64_t>(Size));
Expand Down Expand Up @@ -1937,24 +1939,23 @@ pi_result piTearDown(void *) {
}
int Status = cm_support::CM_FAILURE;

if (Mem->getMemType() == PI_MEM_TYPE_BUFFER) {
if (Mem->SurfacePtr.tag == cm_surface_ptr_t::type_user_provided_buffer) {
_pi_buffer *PiBuf = static_cast<_pi_buffer *>(Mem);
if (PiBuf->BufferPtr.tag == cm_buffer_ptr_slot::type_user_provided) {
Status = Mem->Context->Device->CmDevicePtr->DestroyBufferUP(
PiBuf->BufferPtr.UPBufPtr);
} else {
Status = Mem->Context->Device->CmDevicePtr->DestroySurface(
PiBuf->BufferPtr.RegularBufPtr);
}
} else if (Mem->getMemType() == PI_MEM_TYPE_IMAGE2D) {
Status = Mem->Context->Device->CmDevicePtr->DestroyBufferUP(
PiBuf->SurfacePtr.UPBufPtr);
} else if (Mem->SurfacePtr.tag == cm_surface_ptr_t::type_regular_buffer) {
_pi_buffer *PiBuf = static_cast<_pi_buffer *>(Mem);
Status = Mem->Context->Device->CmDevicePtr->DestroySurface(
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could this functionality be encapsulated in destrutor of _pi_mem ?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done.

PiBuf->SurfacePtr.RegularBufPtr);
} else if (Mem->SurfacePtr.tag ==
cm_surface_ptr_t::type_user_provided_image) {
_pi_image *PiImg = static_cast<_pi_image *>(Mem);
if (PiImg->ImagePtr.tag == cm_image_ptr_slot::type_user_provided) {
Status = Mem->Context->Device->CmDevicePtr->DestroySurface2DUP(
PiImg->ImagePtr.UPImgPtr);
} else {
Status = Mem->Context->Device->CmDevicePtr->DestroySurface(
PiImg->ImagePtr.RegularImgPtr);
}
Status = Mem->Context->Device->CmDevicePtr->DestroySurface2DUP(
PiImg->SurfacePtr.UPImgPtr);
} else if (Mem->SurfacePtr.tag == cm_surface_ptr_t::type_regular_image) {
_pi_image *PiImg = static_cast<_pi_image *>(Mem);
Status = Mem->Context->Device->CmDevicePtr->DestroySurface(
PiImg->SurfacePtr.RegularImgPtr);
}

if (Status != cm_support::CM_SUCCESS) {
Expand Down
74 changes: 30 additions & 44 deletions sycl/plugins/esimd_emulator/pi_esimd_emulator.hpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -105,6 +105,27 @@ struct _pi_queue : _pi_object {
cm_support::CmQueue *CmQueuePtr = nullptr;
};

struct cm_surface_ptr_t {
Copy link
Contributor

@kbobrovs kbobrovs Apr 28, 2022 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add later comments what is this structure for.

// 'UP' means 'User-Provided' in CM Lib - corresponding to
// Buffer/Image created with PI_MEM_FLAGS_HOST_PTR_USE option in
// SYCL
enum SurfaceType {
type_none,
type_regular_buffer,
type_user_provided_buffer,
type_regular_image,
type_user_provided_image
};
SurfaceType tag = type_none;

union {
cm_support::CmBuffer *RegularBufPtr = nullptr;
cm_support::CmBufferUP *UPBufPtr;
cm_support::CmSurface2D *RegularImgPtr;
cm_support::CmSurface2DUP *UPImgPtr;
};
};

struct _pi_mem : _pi_object {
_pi_mem() = default;

Expand Down Expand Up @@ -135,67 +156,32 @@ struct _pi_mem : _pi_object {
// Supporing multi-threaded mapping/unmapping calls
std::mutex MappingsMutex;

_pi_mem_type getMemType() const { return MemType; };
cm_surface_ptr_t SurfacePtr;

protected:
_pi_mem(pi_context ctxt, char *HostPtr, _pi_mem_type MemTypeArg,
_pi_mem(pi_context ctxt, char *HostPtr, cm_surface_ptr_t SurfacePtrArg,
unsigned int SurfaceIdxArg)
: Context{ctxt}, MapHostPtr{HostPtr},
SurfaceIndex{SurfaceIdxArg}, MemType{MemTypeArg} {}

private:
_pi_mem_type MemType;
};

// TODO: Merge cm_buffer_ptr_slot and cm_image_ptr_slot into one
// struct
struct cm_buffer_ptr_slot {
// 'UP' means 'User-Provided' in CM Lib - corresponding to
// Host-created buffer in SYCL

enum type { type_none, type_regular, type_user_provided };
type tag = type_none;

union {
cm_support::CmBuffer *RegularBufPtr = nullptr;
cm_support::CmBufferUP *UPBufPtr;
};
};

struct cm_image_ptr_slot {
// 'UP' means 'User-Provided' in CM Lib - corresponding to
// Host-created image in SYCL

enum type { type_none, type_regular, type_user_provided };
type tag = type_none;

union {
cm_support::CmSurface2D *RegularImgPtr = nullptr;
cm_support::CmSurface2DUP *UPImgPtr;
};
: Context{ctxt}, MapHostPtr{HostPtr}, SurfaceIndex{SurfaceIdxArg},
SurfacePtr{SurfacePtrArg} {}
};

struct _pi_buffer final : _pi_mem {
// Buffer/Sub-buffer constructor
_pi_buffer(pi_context ctxt, char *HostPtr, cm_buffer_ptr_slot BufferPtrArg,
_pi_buffer(pi_context ctxt, char *HostPtr, cm_surface_ptr_t SurfacePtrArg,
unsigned int SurfaceIdxArg, size_t SizeArg)
: _pi_mem(ctxt, HostPtr, PI_MEM_TYPE_BUFFER, SurfaceIdxArg),
BufferPtr{BufferPtrArg}, Size{SizeArg} {}
: _pi_mem(ctxt, HostPtr, SurfacePtrArg, SurfaceIdxArg), Size{SizeArg} {}

cm_buffer_ptr_slot BufferPtr;
size_t Size;
};

struct _pi_image final : _pi_mem {
// Image constructor
_pi_image(pi_context ctxt, char *HostPtr, cm_image_ptr_slot ImagePtrArg,
_pi_image(pi_context ctxt, char *HostPtr, cm_surface_ptr_t SurfacePtrArg,
unsigned int SurfaceIdxArg, size_t WidthArg, size_t HeightArg,
size_t BPPArg)
: _pi_mem(ctxt, HostPtr, PI_MEM_TYPE_IMAGE2D, SurfaceIdxArg),
ImagePtr(ImagePtrArg), Width{WidthArg}, Height{HeightArg},
BytesPerPixel{BPPArg} {}
: _pi_mem(ctxt, HostPtr, SurfacePtrArg, SurfaceIdxArg), Width{WidthArg},
Height{HeightArg}, BytesPerPixel{BPPArg} {}

cm_image_ptr_slot ImagePtr;
size_t Width;
size_t Height;
size_t BytesPerPixel;
Expand Down