From: Francesco Lavra <francescolavra.fl@gmail.com>
To: Maarten Lankhorst <maarten.lankhorst@canonical.com>
Cc: Maarten Lankhorst <m.b.lankhorst@gmail.com>,
dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org,
linux-media@vger.kernel.org, linaro-mm-sig@lists.linaro.org
Subject: Re: [Linaro-mm-sig] [PATCH 4/7] fence: dma-buf cross-device synchronization (v11)
Date: Wed, 23 Jan 2013 18:14:07 +0100 [thread overview]
Message-ID: <51001A5F.1080903@gmail.com> (raw)
In-Reply-To: <50FFFA31.6000101@canonical.com>
On 01/23/2013 03:56 PM, Maarten Lankhorst wrote:
> Thanks for the review, how does this delta look?
>
> diff --git a/include/linux/fence.h b/include/linux/fence.h
> index d9f091d..831ed0a 100644
> --- a/include/linux/fence.h
> +++ b/include/linux/fence.h
> @@ -42,7 +42,10 @@ struct fence_cb;
> * @ops: fence_ops associated with this fence
> * @cb_list: list of all callbacks to call
> * @lock: spin_lock_irqsave used for locking
> - * @priv: fence specific private data
> + * @context: execution context this fence belongs to, returned by
> + * fence_context_alloc()
> + * @seqno: the sequence number of this fence inside the executation context,
s/executation/execution
Otherwise, looks good to me.
--
Francesco
next prev parent reply other threads:[~2013-01-23 17:14 UTC|newest]
Thread overview: 35+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-01-15 12:33 [PATCH 0/7] cross-device reservation for dma-buf support Maarten Lankhorst
2013-01-15 12:33 ` [PATCH 1/7] arch: add __mutex_fastpath_lock_retval_arg to generic/sh/x86/powerpc/ia64 Maarten Lankhorst
2013-01-15 12:33 ` Maarten Lankhorst
2013-01-15 13:49 ` Maarten Lankhorst
2013-01-15 12:33 ` [PATCH 2/7] mutex: add support for reservation style locks Maarten Lankhorst
2013-01-15 13:43 ` Maarten Lankhorst
2013-01-30 1:07 ` Rob Clark
2013-01-30 1:07 ` Rob Clark
2013-01-30 11:08 ` Daniel Vetter
2013-01-30 11:52 ` Rob Clark
2013-01-31 13:38 ` Rob Clark
2013-01-30 11:16 ` Maarten Lankhorst
2013-01-15 12:34 ` [PATCH 3/7] sched: allow try_to_wake_up to be used internally outside of core.c Maarten Lankhorst
2013-01-15 12:34 ` [PATCH 4/7] fence: dma-buf cross-device synchronization (v11) Maarten Lankhorst
2013-01-22 15:13 ` [Linaro-mm-sig] " Francesco Lavra
2013-01-23 14:56 ` Maarten Lankhorst
2013-01-23 17:14 ` Francesco Lavra [this message]
2013-01-31 9:32 ` Inki Dae
2013-01-31 9:53 ` Maarten Lankhorst
2013-01-31 9:57 ` Daniel Vetter
2013-01-31 14:38 ` Inki Dae
2013-01-31 14:49 ` Daniel Vetter
2013-01-15 12:34 ` [PATCH 5/7] seqno-fence: Hardware dma-buf implementation of fencing (v4) Maarten Lankhorst
2013-01-15 12:34 ` Maarten Lankhorst
2013-01-16 6:28 ` [Linaro-mm-sig] " Inki Dae
2013-01-16 10:36 ` Maarten Lankhorst
2013-01-16 12:00 ` Inki Dae
2013-01-24 14:52 ` Inki Dae
2013-01-15 12:34 ` [PATCH 6/7] reservation: cross-device reservation support Maarten Lankhorst
2013-01-22 16:47 ` [Linaro-mm-sig] " Francesco Lavra
2013-01-22 17:04 ` Maarten Lankhorst
2013-02-04 7:06 ` Inki Dae
2013-02-04 9:57 ` Maarten Lankhorst
2013-02-04 14:51 ` Daniel Vetter
2013-01-15 12:34 ` [PATCH 7/7] reservation: Add lockdep annotation and selftests Maarten Lankhorst
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=51001A5F.1080903@gmail.com \
--to=francescolavra.fl@gmail.com \
--cc=dri-devel@lists.freedesktop.org \
--cc=linaro-mm-sig@lists.linaro.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-media@vger.kernel.org \
--cc=m.b.lankhorst@gmail.com \
--cc=maarten.lankhorst@canonical.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.