Re: [PATCH 0/2] Use Media Dev Allocator to fix vimc dev lifetime bugs

2019-07-03 Thread Shuah Khan
On 7/3/19 5:25 PM, Laurent Pinchart wrote: Hi Shuah, On Wed, Jul 03, 2019 at 10:52:17AM -0600, Shuah Khan wrote: On 7/3/19 10:17 AM, Niklas Söderlund wrote: On 2019-06-30 14:41:02 +0300, Laurent Pinchart wrote: On Fri, Jun 28, 2019 at 10:41:07AM -0600, Shuah Khan wrote: On 6/16/19 12:45 PM,

Re: [PATCH 0/2] Use Media Dev Allocator to fix vimc dev lifetime bugs

2019-07-03 Thread Laurent Pinchart
Hi Shuah, On Wed, Jul 03, 2019 at 10:52:17AM -0600, Shuah Khan wrote: > On 7/3/19 10:17 AM, Niklas Söderlund wrote: > > On 2019-06-30 14:41:02 +0300, Laurent Pinchart wrote: > >> On Fri, Jun 28, 2019 at 10:41:07AM -0600, Shuah Khan wrote: > >>> On 6/16/19 12:45 PM, Laurent Pinchart wrote: >

Re: [PATCH 0/2] Use Media Dev Allocator to fix vimc dev lifetime bugs

2019-07-03 Thread Shuah Khan
Hi Niklas, Laurent, On 7/3/19 10:17 AM, Niklas Söderlund wrote: Hi Shauah, Laurent, On 2019-06-30 14:41:02 +0300, Laurent Pinchart wrote: Hi Shuah, On Fri, Jun 28, 2019 at 10:41:07AM -0600, Shuah Khan wrote: On 6/16/19 12:45 PM, Laurent Pinchart wrote: On Fri, Jun 14, 2019 at 05:26:46PM

Re: [PATCH 0/2] Use Media Dev Allocator to fix vimc dev lifetime bugs

2019-07-03 Thread Niklas Söderlund
Hi Shauah, Laurent, On 2019-06-30 14:41:02 +0300, Laurent Pinchart wrote: > Hi Shuah, > > On Fri, Jun 28, 2019 at 10:41:07AM -0600, Shuah Khan wrote: > > On 6/16/19 12:45 PM, Laurent Pinchart wrote: > > > On Fri, Jun 14, 2019 at 05:26:46PM -0600, Shuah Khan wrote: > > >> On 6/13/19 7:24 AM,

Re: [PATCH 0/2] Use Media Dev Allocator to fix vimc dev lifetime bugs

2019-06-30 Thread Laurent Pinchart
Hi Shuah, On Fri, Jun 28, 2019 at 10:41:07AM -0600, Shuah Khan wrote: > On 6/16/19 12:45 PM, Laurent Pinchart wrote: > > On Fri, Jun 14, 2019 at 05:26:46PM -0600, Shuah Khan wrote: > >> On 6/13/19 7:24 AM, Helen Koike wrote: > >>> On 6/13/19 2:44 AM, Hans Verkuil wrote: > On 5/24/19 5:31 AM,

Re: [PATCH 0/2] Use Media Dev Allocator to fix vimc dev lifetime bugs

2019-06-28 Thread Shuah Khan
Hi Laurent, On 6/16/19 12:45 PM, Laurent Pinchart wrote: Hi Shuah, On Fri, Jun 14, 2019 at 05:26:46PM -0600, Shuah Khan wrote: On 6/13/19 7:24 AM, Helen Koike wrote: On 6/13/19 2:44 AM, Hans Verkuil wrote: On 5/24/19 5:31 AM, Shuah Khan wrote: media_device is embedded in struct vimc_device

Re: [PATCH 0/2] Use Media Dev Allocator to fix vimc dev lifetime bugs

2019-06-16 Thread Laurent Pinchart
Hi Shuah, On Fri, Jun 14, 2019 at 05:26:46PM -0600, Shuah Khan wrote: > On 6/13/19 7:24 AM, Helen Koike wrote: > > On 6/13/19 2:44 AM, Hans Verkuil wrote: > >> On 5/24/19 5:31 AM, Shuah Khan wrote: > >>> media_device is embedded in struct vimc_device and when vimc is removed > >>> vimc_device and

Re: [PATCH 0/2] Use Media Dev Allocator to fix vimc dev lifetime bugs

2019-06-16 Thread Laurent Pinchart
Hi Hans, On Thu, Jun 13, 2019 at 07:44:15AM +0200, Hans Verkuil wrote: > On 5/24/19 5:31 AM, Shuah Khan wrote: > > media_device is embedded in struct vimc_device and when vimc is removed > > vimc_device and the embedded media_device goes with it, while the active > > stream and vimc_capture

Re: [PATCH 0/2] Use Media Dev Allocator to fix vimc dev lifetime bugs

2019-06-14 Thread Shuah Khan
On 6/13/19 7:24 AM, Helen Koike wrote: On 6/13/19 2:44 AM, Hans Verkuil wrote: On 5/24/19 5:31 AM, Shuah Khan wrote: media_device is embedded in struct vimc_device and when vimc is removed vimc_device and the embedded media_device goes with it, while the active stream and vimc_capture

Re: [PATCH 0/2] Use Media Dev Allocator to fix vimc dev lifetime bugs

2019-06-13 Thread Hans Verkuil
On 5/24/19 5:31 AM, Shuah Khan wrote: > media_device is embedded in struct vimc_device and when vimc is removed > vimc_device and the embedded media_device goes with it, while the active > stream and vimc_capture continue to access it. > > Fix the media_device lifetime problem by changing vimc to

Re: [PATCH 0/2] Use Media Dev Allocator to fix vimc dev lifetime bugs

2019-06-13 Thread Helen Koike
On 6/13/19 2:44 AM, Hans Verkuil wrote: > On 5/24/19 5:31 AM, Shuah Khan wrote: >> media_device is embedded in struct vimc_device and when vimc is removed >> vimc_device and the embedded media_device goes with it, while the active >> stream and vimc_capture continue to access it. >> >> Fix the

[PATCH 0/2] Use Media Dev Allocator to fix vimc dev lifetime bugs

2019-05-23 Thread Shuah Khan
media_device is embedded in struct vimc_device and when vimc is removed vimc_device and the embedded media_device goes with it, while the active stream and vimc_capture continue to access it. Fix the media_device lifetime problem by changing vimc to create shared media_device using Media Device