[Linaro-mm-sig] [PATCH 4/7] fence: dma-buf cross-device synchronization (v11)

2013-01-31 Thread Inki Dae
2013/1/31 Daniel Vetter : > On Thu, Jan 31, 2013 at 06:32:15PM +0900, Inki Dae wrote: >> Hi, >> >> below is my opinion. >> >> > +struct fence; >> > +struct fence_ops; >> > +struct fence_cb; >> > + >> > +/** >> > + * struct fence - software synchronization primitive >> > + * @refcount: refcount for

[Linaro-mm-sig] [PATCH 4/7] fence: dma-buf cross-device synchronization (v11)

2013-01-31 Thread Inki Dae
Hi, below is my opinion. > +struct fence; > +struct fence_ops; > +struct fence_cb; > + > +/** > + * struct fence - software synchronization primitive > + * @refcount: refcount for this fence > + * @ops: fence_ops associated with this fence > + * @cb_list: list of all callbacks to call > + * @lock

[Linaro-mm-sig] [PATCH 4/7] fence: dma-buf cross-device synchronization (v11)

2013-01-31 Thread Daniel Vetter
On Thu, Jan 31, 2013 at 3:38 PM, Inki Dae wrote: > I think I understand as your comment but I don't think that I > understand fully the dma-fence mechanism. So I wish you to give me > some advices for it. In our case, I'm applying the dma-fence to > mali(3d gpu) driver as producer and exynos drm(d

[Linaro-mm-sig] [PATCH 4/7] fence: dma-buf cross-device synchronization (v11)

2013-01-31 Thread Daniel Vetter
On Thu, Jan 31, 2013 at 06:32:15PM +0900, Inki Dae wrote: > Hi, > > below is my opinion. > > > +struct fence; > > +struct fence_ops; > > +struct fence_cb; > > + > > +/** > > + * struct fence - software synchronization primitive > > + * @refcount: refcount for this fence > > + * @ops: fence_ops as

[Linaro-mm-sig] [PATCH 4/7] fence: dma-buf cross-device synchronization (v11)

2013-01-31 Thread Maarten Lankhorst
Op 31-01-13 10:32, Inki Dae schreef: > Hi, > > below is my opinion. > >> +struct fence; >> +struct fence_ops; >> +struct fence_cb; >> + >> +/** >> + * struct fence - software synchronization primitive >> + * @refcount: refcount for this fence >> + * @ops: fence_ops associated with this fence >> + *

Re: [Linaro-mm-sig] [PATCH 4/7] fence: dma-buf cross-device synchronization (v11)

2013-01-31 Thread Daniel Vetter
On Thu, Jan 31, 2013 at 3:38 PM, Inki Dae wrote: > I think I understand as your comment but I don't think that I > understand fully the dma-fence mechanism. So I wish you to give me > some advices for it. In our case, I'm applying the dma-fence to > mali(3d gpu) driver as producer and exynos drm(d

Re: [Linaro-mm-sig] [PATCH 4/7] fence: dma-buf cross-device synchronization (v11)

2013-01-31 Thread Inki Dae
2013/1/31 Daniel Vetter : > On Thu, Jan 31, 2013 at 06:32:15PM +0900, Inki Dae wrote: >> Hi, >> >> below is my opinion. >> >> > +struct fence; >> > +struct fence_ops; >> > +struct fence_cb; >> > + >> > +/** >> > + * struct fence - software synchronization primitive >> > + * @refcount: refcount for

Re: [Linaro-mm-sig] [PATCH 4/7] fence: dma-buf cross-device synchronization (v11)

2013-01-31 Thread Daniel Vetter
On Thu, Jan 31, 2013 at 06:32:15PM +0900, Inki Dae wrote: > Hi, > > below is my opinion. > > > +struct fence; > > +struct fence_ops; > > +struct fence_cb; > > + > > +/** > > + * struct fence - software synchronization primitive > > + * @refcount: refcount for this fence > > + * @ops: fence_ops as

Re: [Linaro-mm-sig] [PATCH 4/7] fence: dma-buf cross-device synchronization (v11)

2013-01-31 Thread Maarten Lankhorst
Op 31-01-13 10:32, Inki Dae schreef: > Hi, > > below is my opinion. > >> +struct fence; >> +struct fence_ops; >> +struct fence_cb; >> + >> +/** >> + * struct fence - software synchronization primitive >> + * @refcount: refcount for this fence >> + * @ops: fence_ops associated with this fence >> + *

Re: [Linaro-mm-sig] [PATCH 4/7] fence: dma-buf cross-device synchronization (v11)

2013-01-31 Thread Inki Dae
Hi, below is my opinion. > +struct fence; > +struct fence_ops; > +struct fence_cb; > + > +/** > + * struct fence - software synchronization primitive > + * @refcount: refcount for this fence > + * @ops: fence_ops associated with this fence > + * @cb_list: list of all callbacks to call > + * @lock

Re: [Linaro-mm-sig] [PATCH 4/7] fence: dma-buf cross-device synchronization (v11)

2013-01-23 Thread Francesco Lavra
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; > *

[Linaro-mm-sig] [PATCH 4/7] fence: dma-buf cross-device synchronization (v11)

2013-01-23 Thread Francesco Lavra
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; > *

[Linaro-mm-sig] [PATCH 4/7] fence: dma-buf cross-device synchronization (v11)

2013-01-23 Thread Maarten Lankhorst
Op 22-01-13 16:13, Francesco Lavra schreef: > Hi, > > On 01/15/2013 01:34 PM, Maarten Lankhorst wrote: > [...] >> diff --git a/include/linux/fence.h b/include/linux/fence.h >> new file mode 100644 >> index 000..d9f091d >> --- /dev/null >> +++ b/include/linux/fence.h >> @@ -0,0 +1,347 @@ >> +/*

Re: [Linaro-mm-sig] [PATCH 4/7] fence: dma-buf cross-device synchronization (v11)

2013-01-23 Thread Maarten Lankhorst
Op 22-01-13 16:13, Francesco Lavra schreef: > Hi, > > On 01/15/2013 01:34 PM, Maarten Lankhorst wrote: > [...] >> diff --git a/include/linux/fence.h b/include/linux/fence.h >> new file mode 100644 >> index 000..d9f091d >> --- /dev/null >> +++ b/include/linux/fence.h >> @@ -0,0 +1,347 @@ >> +/*

Re: [Linaro-mm-sig] [PATCH 4/7] fence: dma-buf cross-device synchronization (v11)

2013-01-22 Thread Francesco Lavra
Hi, On 01/15/2013 01:34 PM, Maarten Lankhorst wrote: [...] > diff --git a/include/linux/fence.h b/include/linux/fence.h > new file mode 100644 > index 000..d9f091d > --- /dev/null > +++ b/include/linux/fence.h > @@ -0,0 +1,347 @@ > +/* > + * Fence mechanism for dma-buf to allow for asynchronou

[Linaro-mm-sig] [PATCH 4/7] fence: dma-buf cross-device synchronization (v11)

2013-01-22 Thread Francesco Lavra
Hi, On 01/15/2013 01:34 PM, Maarten Lankhorst wrote: [...] > diff --git a/include/linux/fence.h b/include/linux/fence.h > new file mode 100644 > index 000..d9f091d > --- /dev/null > +++ b/include/linux/fence.h > @@ -0,0 +1,347 @@ > +/* > + * Fence mechanism for dma-buf to allow for asynchronou