From: Greg KH <gregkh@linuxfoundation.org>
To: Maarten Lankhorst <maarten.lankhorst@canonical.com>
Cc: linux-arch@vger.kernel.org, thellstrom@vmware.com,
linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org,
linaro-mm-sig@lists.linaro.org, ccross@google.com,
linux-media@vger.kernel.org
Subject: Re: [PATCH v2 0/9] Updated fence patch series
Date: Tue, 1 Jul 2014 22:37:58 -0700 [thread overview]
Message-ID: <20140702053758.GA7578@kroah.com> (raw)
In-Reply-To: <20140701103432.12718.82795.stgit@patser>
On Tue, Jul 01, 2014 at 12:57:02PM +0200, Maarten Lankhorst wrote:
> So after some more hacking I've moved dma-buf to its own subdirectory,
> drivers/dma-buf and applied the fence patches to its new place. I believe that the
> first patch should be applied regardless, and the rest should be ready now.
> :-)
>
> Changes to the fence api:
> - release_fence -> fence_release etc.
> - __fence_init -> fence_init
> - __fence_signal -> fence_signal_locked
> - __fence_is_signaled -> fence_is_signaled_locked
> - Changing BUG_ON to WARN_ON in fence_later, and return NULL if it triggers.
>
> Android can expose fences to userspace. It's possible to make the new fence
> mechanism expose the same fences to userspace by changing sync_fence_create
> to take a struct fence instead of a struct sync_pt. No other change is needed,
> because only the fence parts of struct sync_pt are used. But because the
> userspace fences are a separate problem and I haven't really looked at it yet
> I feel it should stay in staging, for now.
Ok, that's reasonable.
At first glance, this all looks "sane" to me, any objection from anyone
if I merge this through my driver-core tree for 3.17?
thanks,
greg k-h
WARNING: multiple messages have this Message-ID (diff)
From: Greg KH <gregkh@linuxfoundation.org>
To: Maarten Lankhorst <maarten.lankhorst@canonical.com>
Cc: linux-arch@vger.kernel.org, thellstrom@vmware.com,
linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org,
linaro-mm-sig@lists.linaro.org, robdclark@gmail.com,
thierry.reding@gmail.com, ccross@google.com, daniel@ffwll.ch,
sumit.semwal@linaro.org, linux-media@vger.kernel.org
Subject: Re: [PATCH v2 0/9] Updated fence patch series
Date: Tue, 1 Jul 2014 22:37:58 -0700 [thread overview]
Message-ID: <20140702053758.GA7578@kroah.com> (raw)
Message-ID: <20140702053758.RrrvTZZVd-ZcsGwi83EgFINX1KWk5X_N08vGUTzPEe8@z> (raw)
In-Reply-To: <20140701103432.12718.82795.stgit@patser>
On Tue, Jul 01, 2014 at 12:57:02PM +0200, Maarten Lankhorst wrote:
> So after some more hacking I've moved dma-buf to its own subdirectory,
> drivers/dma-buf and applied the fence patches to its new place. I believe that the
> first patch should be applied regardless, and the rest should be ready now.
> :-)
>
> Changes to the fence api:
> - release_fence -> fence_release etc.
> - __fence_init -> fence_init
> - __fence_signal -> fence_signal_locked
> - __fence_is_signaled -> fence_is_signaled_locked
> - Changing BUG_ON to WARN_ON in fence_later, and return NULL if it triggers.
>
> Android can expose fences to userspace. It's possible to make the new fence
> mechanism expose the same fences to userspace by changing sync_fence_create
> to take a struct fence instead of a struct sync_pt. No other change is needed,
> because only the fence parts of struct sync_pt are used. But because the
> userspace fences are a separate problem and I haven't really looked at it yet
> I feel it should stay in staging, for now.
Ok, that's reasonable.
At first glance, this all looks "sane" to me, any objection from anyone
if I merge this through my driver-core tree for 3.17?
thanks,
greg k-h
next prev parent reply other threads:[~2014-07-02 5:37 UTC|newest]
Thread overview: 42+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-07-01 10:57 [PATCH v2 0/9] Updated fence patch series Maarten Lankhorst
2014-07-01 10:57 ` Maarten Lankhorst
2014-07-01 10:57 ` [PATCH v2 1/9] dma-buf: move to drivers/dma-buf Maarten Lankhorst
2014-07-01 10:57 ` Maarten Lankhorst
2014-07-01 11:06 ` Arend van Spriel
2014-07-01 11:06 ` Arend van Spriel
2014-07-01 11:09 ` Maarten Lankhorst
2014-07-01 11:09 ` Maarten Lankhorst
2014-07-01 10:57 ` [PATCH v2 2/9] fence: dma-buf cross-device synchronization (v18) Maarten Lankhorst
2014-07-01 10:57 ` Maarten Lankhorst
2014-07-01 10:57 ` [PATCH v2 3/9] seqno-fence: Hardware dma-buf implementation of fencing (v6) Maarten Lankhorst
2014-07-01 10:57 ` Maarten Lankhorst
2014-07-01 10:57 ` [PATCH v2 4/9] dma-buf: use reservation objects Maarten Lankhorst
2014-07-01 10:57 ` Maarten Lankhorst
2014-07-01 10:57 ` [PATCH v2 5/9] android: convert sync to fence api, v6 Maarten Lankhorst
2014-07-01 10:57 ` Maarten Lankhorst
2014-07-01 10:57 ` [PATCH v2 6/9] reservation: add support for fences to enable cross-device synchronisation Maarten Lankhorst
2014-07-01 10:57 ` Maarten Lankhorst
2014-07-01 10:57 ` [PATCH v2 7/9] dma-buf: add poll support, v3 Maarten Lankhorst
2014-07-01 10:57 ` Maarten Lankhorst
2014-07-01 10:57 ` [PATCH v2 8/9] reservation: update api and add some helpers Maarten Lankhorst
2014-07-01 10:57 ` Maarten Lankhorst
2014-07-01 10:58 ` [PATCH v2 9/9] reservation: add suppport for read-only access using rcu Maarten Lankhorst
2014-07-01 10:58 ` Maarten Lankhorst
2014-07-02 5:37 ` Greg KH [this message]
2014-07-02 5:37 ` [PATCH v2 0/9] Updated fence patch series Greg KH
2014-07-02 9:12 ` Maarten Lankhorst
2014-07-02 9:12 ` Maarten Lankhorst
2014-07-08 20:46 ` Greg KH
2014-07-08 20:46 ` Greg KH
2014-07-07 13:23 ` Daniel Vetter
2014-07-07 13:23 ` Daniel Vetter
2014-07-07 17:30 ` Greg KH
2014-07-07 17:30 ` Greg KH
2014-07-08 13:44 ` Daniel Vetter
2014-07-08 13:44 ` Daniel Vetter
2014-07-08 14:39 ` Greg KH
2014-07-08 14:52 ` Sumit Semwal
2014-07-08 17:52 ` Greg KH
2014-07-08 17:52 ` Greg KH
2014-07-07 17:27 ` Sumit Semwal
2014-07-07 17:27 ` Sumit Semwal
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20140702053758.GA7578@kroah.com \
--to=gregkh@linuxfoundation.org \
--cc=ccross@google.com \
--cc=dri-devel@lists.freedesktop.org \
--cc=linaro-mm-sig@lists.linaro.org \
--cc=linux-arch@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-media@vger.kernel.org \
--cc=maarten.lankhorst@canonical.com \
--cc=thellstrom@vmware.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.