[Bug 783417] Add physical memory allocator base class in badallocator

GStreamer (GNOME Bugzilla) bugzilla at gnome.org
Thu Jun 22 06:17:07 UTC 2017


https://bugzilla.gnome.org/show_bug.cgi?id=783417

--- Comment #4 from Haihua Hu <jared.hu at nxp.com> ---
(In reply to Sebastian Dröge (slomo) from comment #3)
> Review of attachment 353161 [details] [review]:
> 
> ::: gst-libs/gst/allocators/gstallocatorphymem.h
> @@ +40,3 @@
> +  guint8 *caddr;
> +  gsize size;
> +  gpointer *user_data;
> 
> This is all very specific to how things work on imx6. As such it should live
> in some imx6 specific library, probably just stay in the gstreamer-imx (
> https://github.com/Freescale/gstreamer-imx/ ) repository and be installed as
> a library from there. The reason why I went with an interface instead of an
> actual allocator is that this is always going to be very platform specific,
> and there's probably not going to be much common functionality to share.

It's not specific usage. This is an abstract class for phymem allocator, we
derived some phymem allocator from this class such VPU allocator and g2d
allocator. These two allocators are specific usage.

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.


More information about the gstreamer-bugs mailing list