[Bug 730559] dmabuf: fix checking mmap flags

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=730559
  GStreamer | gst-plugins-base | git

Nicolas Dufresne <nicolas.dufresne> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |nicolas.dufresne at collabora.
                   |                            |co.uk

--- Comment #1 from Nicolas Dufresne <nicolas.dufresne at collabora.co.uk> 2014-05-22 13:07:54 UTC ---
It's conceptually correct of course, thanks for the patch. Though, now that I
look at that closely I wonder if we didn't got something wrong.

Normally the rules to allow mapping or not a memory are handled in GstMemory
code and we should never end-up rejecting a map accepted by GStreamer. I would
need to review does rules as I realize I don't know them, but is there a risk
that GStreamer rules would allow mapping a buffers, but because of the previous
mapping we have done, only read, or only write, we would fail doing so ?

-- 
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.