Hi Sailus/Mauro,
On 01/26/2017 02:10 AM, Mauro Carvalho Chehab wrote:
> Em Wed, 25 Jan 2017 13:02:31 +0200
> Sakari Ailus escreveu:
>
>> Hi Mauro,
>>
>> On Tue, Jan 24, 2017 at 08:49:02AM -0200, Mauro Carvalho Chehab wrote:
>>> Hi Sakari,
>>>
>>> Just returned this week
Em Wed, 25 Jan 2017 13:02:31 +0200
Sakari Ailus escreveu:
> Hi Mauro,
>
> On Tue, Jan 24, 2017 at 08:49:02AM -0200, Mauro Carvalho Chehab wrote:
> > Hi Sakari,
> >
> > Just returned this week from vacations. I'm reading my long e-mail backlog,
> > starting from my main
Hi Mauro,
On Tue, Jan 24, 2017 at 08:49:02AM -0200, Mauro Carvalho Chehab wrote:
> Hi Sakari,
>
> Just returned this week from vacations. I'm reading my long e-mail backlog,
> starting from my main inbox...
>
> Em Mon, 2 Jan 2017 09:53:49 +0200
> Sakari Ailus escreveu:
>
Hi Sakari,
Just returned this week from vacations. I'm reading my long e-mail backlog,
starting from my main inbox...
Em Mon, 2 Jan 2017 09:53:49 +0200
Sakari Ailus escreveu:
> Hi Mauro,
>
> On Mon, Dec 19, 2016 at 07:46:55AM -0200, Mauro Carvalho Chehab wrote:
> > Em
Hi Mauro,
On Mon, Dec 19, 2016 at 07:46:55AM -0200, Mauro Carvalho Chehab wrote:
> Em Fri, 16 Dec 2016 17:07:23 +0200
> Sakari Ailus escreveu:
>
> > Hi Hans,
>
> > > chrdev_open in fs/char_dev.c increases the refcount on open() and
> > > decreases it
> > > on release().
Hi Shuah,
On Thursday 15 Dec 2016 09:06:41 Shuah Khan wrote:
> On 12/15/2016 08:26 AM, Hans Verkuil wrote:
> > On 15/12/16 15:45, Shuah Khan wrote:
> >> On 12/15/2016 07:03 AM, Hans Verkuil wrote:
> >>> On 15/12/16 13:56, Laurent Pinchart wrote:
> On Thursday 15 Dec 2016 13:30:41 Sakari
Hi Mauro,
On Thursday 15 Dec 2016 15:08:26 Mauro Carvalho Chehab wrote:
> Em Thu, 15 Dec 2016 16:26:19 +0100 Hans Verkuil escreveu:
> >> Should all the entities stick around until all references to media
> >> device are gone? If an application has /dev/media open, does that
> >> mean all entities
Hi Hans,
On Thursday 15 Dec 2016 16:26:19 Hans Verkuil wrote:
> On 15/12/16 15:45, Shuah Khan wrote:
> > On 12/15/2016 07:03 AM, Hans Verkuil wrote:
[snip]
> >> In my view the main problem is that the media core is bound to a struct
> >> device set by the driver that creates the MC. But since
Hi Shuah,
On Thursday 15 Dec 2016 07:45:29 Shuah Khan wrote:
> On 12/15/2016 07:03 AM, Hans Verkuil wrote:
> > On 15/12/16 13:56, Laurent Pinchart wrote:
> >> On Thursday 15 Dec 2016 13:30:41 Sakari Ailus wrote:
> >>> On Tue, Dec 13, 2016 at 10:24:47AM -0200, Mauro Carvalho Chehab wrote:
>
Em Thu, 22 Dec 2016 19:47:15 +0200
Laurent Pinchart escreveu:
> Hi Mauro,
>
> On Tuesday 20 Dec 2016 23:31:42 Mauro Carvalho Chehab wrote:
> > Em Mon, 19 Dec 2016 07:28:29 -0200 Mauro Carvalho Chehab escreveu:
> > > Em Fri, 16 Dec 2016 15:45:10 +0100 Hans
Hi Mauro,
On Tuesday 20 Dec 2016 23:31:42 Mauro Carvalho Chehab wrote:
> Em Mon, 19 Dec 2016 07:28:29 -0200 Mauro Carvalho Chehab escreveu:
> > Em Fri, 16 Dec 2016 15:45:10 +0100 Hans Verkuil escreveu:
> >> We really need a whiteboard for this :-(
> >
> > Well, we could schedule a media summit
On Tue, Dec 20, 2016 at 6:31 PM, Mauro Carvalho Chehab
wrote:
> Em Mon, 19 Dec 2016 07:28:29 -0200
> Mauro Carvalho Chehab escreveu:
>
>> Em Fri, 16 Dec 2016 15:45:10 +0100
>> Hans Verkuil escreveu:
>>
>> > We really need a
Em Mon, 19 Dec 2016 07:28:29 -0200
Mauro Carvalho Chehab escreveu:
> Em Fri, 16 Dec 2016 15:45:10 +0100
> Hans Verkuil escreveu:
>
> > We really need a whiteboard for this :-(
>
> Well, we could schedule a media summit together with ELC NA.
>
>
Em Fri, 16 Dec 2016 17:07:23 +0200
Sakari Ailus escreveu:
> Hi Hans,
> > chrdev_open in fs/char_dev.c increases the refcount on open() and decreases
> > it
> > on release(). Thus ensuring that the cdev can never be removed while in an
> > ioctl.
>
> It does, but it does
Em Fri, 16 Dec 2016 15:45:10 +0100
Hans Verkuil escreveu:
> We really need a whiteboard for this :-(
Well, we could schedule a media summit together with ELC NA.
ELC will be in Feb, 21-23 in Portland.
Comments?
Mauro
--
To unsubscribe from this list: send the line
Hi Shuah,
On Thursday 15 Dec 2016 07:56:55 Shuah Khan wrote:
> On 12/15/2016 03:39 AM, Laurent Pinchart wrote:
> > On Tuesday 13 Dec 2016 15:23:53 Shuah Khan wrote:
[snip]
> >> Please don't pursue this RFC series that makes mc-core changes until
> >> ompa3 driver problems are addressed. There
Hi Mauro,
On Thursday 15 Dec 2016 12:32:07 Mauro Carvalho Chehab wrote:
> Em Thu, 15 Dec 2016 15:03:36 +0100 Hans Verkuil escreveu:
> > On 15/12/16 13:56, Laurent Pinchart wrote:
> >> On Thursday 15 Dec 2016 13:30:41 Sakari Ailus wrote:
> >>> On Tue, Dec 13, 2016 at 10:24:47AM -0200, Mauro
On Thursday 15 Dec 2016 13:45:52 Mauro Carvalho Chehab wrote:
> Em Thu, 15 Dec 2016 15:45:22 +0100
>
> Hans Verkuil escreveu:
> > On 15/12/16 15:32, Mauro Carvalho Chehab wrote:
> > > Em Thu, 15 Dec 2016 15:03:36 +0100
> > >
> > > Hans Verkuil escreveu:
Hi Hans,
On Friday 16 Dec 2016 17:07:23 Sakari Ailus wrote:
> On Thu, Dec 15, 2016 at 03:03:36PM +0100, Hans Verkuil wrote:
> > On 15/12/16 13:56, Laurent Pinchart wrote:
> >> On Thursday 15 Dec 2016 13:30:41 Sakari Ailus wrote:
> >>> On Tue, Dec 13, 2016 at 10:24:47AM -0200, Mauro Carvalho
Hi Hans,
On Thu, Dec 15, 2016 at 03:03:36PM +0100, Hans Verkuil wrote:
> On 15/12/16 13:56, Laurent Pinchart wrote:
> >Hi Sakari,
> >
> >On Thursday 15 Dec 2016 13:30:41 Sakari Ailus wrote:
> >>On Tue, Dec 13, 2016 at 10:24:47AM -0200, Mauro Carvalho Chehab wrote:
> >>>Em Tue, 13 Dec 2016
On 16/12/16 13:00, Mauro Carvalho Chehab wrote:
Em
escreveu:
On 16/12/16 11:57, Mauro Carvalho Chehab wrote:
Em Fri, 16 Dec 2016 11:11:25 +0100
Hans Verkuil escreveu:
Would it make sense to enforce that dependency. Can we tie /dev/media usecount
to /dev/video etc.
Em
escreveu:
> On 16/12/16 11:57, Mauro Carvalho Chehab wrote:
> > Em Fri, 16 Dec 2016 11:11:25 +0100
> > Hans Verkuil escreveu:
> >
> >>> Would it make sense to enforce that dependency. Can we tie /dev/media
> >>> usecount
> >>> to /dev/video etc. usecount? In other
On 16/12/16 11:57, Mauro Carvalho Chehab wrote:
Em Fri, 16 Dec 2016 11:11:25 +0100
Hans Verkuil escreveu:
Would it make sense to enforce that dependency. Can we tie /dev/media usecount
to /dev/video etc. usecount? In other words:
/dev/video is opened, then open
Em Fri, 16 Dec 2016 11:11:25 +0100
Hans Verkuil escreveu:
> > Would it make sense to enforce that dependency. Can we tie /dev/media
> > usecount
> > to /dev/video etc. usecount? In other words:
> >
> > /dev/video is opened, then open /dev/media.
>
> When a device node is
Em Fri, 16 Dec 2016 11:03:09 +0100
Hans Verkuil escreveu:
> So:
>
> 1) subdev drivers should disallow unbind
> 2) interface entities should call media_device_unregister_entity() when they
> are unregistered (if that doesn't already happen)
Sounds like a plan to me.
On 15/12/16 18:51, Shuah Khan wrote:
On 12/15/2016 10:25 AM, Mauro Carvalho Chehab wrote:
Em Thu, 15 Dec 2016 10:09:53 -0700
Shuah Khan escreveu:
On 12/15/2016 09:28 AM, Hans Verkuil wrote:
On 15/12/16 17:06, Shuah Khan wrote:
I think this will work for
On 15/12/16 18:09, Shuah Khan wrote:
On 12/15/2016 09:28 AM, Hans Verkuil wrote:
On 15/12/16 17:06, Shuah Khan wrote:
On 12/15/2016 08:26 AM, Hans Verkuil wrote:
On 15/12/16 15:45, Shuah Khan wrote:
On 12/15/2016 07:03 AM, Hans Verkuil wrote:
On 15/12/16 13:56, Laurent Pinchart wrote:
Hi
On 12/15/2016 10:25 AM, Mauro Carvalho Chehab wrote:
> Em Thu, 15 Dec 2016 10:09:53 -0700
> Shuah Khan escreveu:
>
>> On 12/15/2016 09:28 AM, Hans Verkuil wrote:
>>> On 15/12/16 17:06, Shuah Khan wrote:
>
>>>
>>> I think this will work for interface entities, but for
Em Thu, 15 Dec 2016 10:09:53 -0700
Shuah Khan escreveu:
> On 12/15/2016 09:28 AM, Hans Verkuil wrote:
> > On 15/12/16 17:06, Shuah Khan wrote:
> >
> > I think this will work for interface entities, but for subdev entities this
> > certainly won't work. Unbinding
On 12/15/2016 09:28 AM, Hans Verkuil wrote:
> On 15/12/16 17:06, Shuah Khan wrote:
>> On 12/15/2016 08:26 AM, Hans Verkuil wrote:
>>> On 15/12/16 15:45, Shuah Khan wrote:
On 12/15/2016 07:03 AM, Hans Verkuil wrote:
> On 15/12/16 13:56, Laurent Pinchart wrote:
>> Hi Sakari,
>>
Em Thu, 15 Dec 2016 16:26:19 +0100
Hans Verkuil escreveu:
> > Should all the entities stick around until all references to media
> > device are gone? If an application has /dev/media open, does that
> > mean all entities should not be free'd until this app. exits? What
> >
On 15/12/16 17:06, Shuah Khan wrote:
On 12/15/2016 08:26 AM, Hans Verkuil wrote:
On 15/12/16 15:45, Shuah Khan wrote:
On 12/15/2016 07:03 AM, Hans Verkuil wrote:
On 15/12/16 13:56, Laurent Pinchart wrote:
Hi Sakari,
On Thursday 15 Dec 2016 13:30:41 Sakari Ailus wrote:
On Tue, Dec 13, 2016
On 15/12/16 16:45, Mauro Carvalho Chehab wrote:
Em Thu, 15 Dec 2016 15:45:22 +0100
Hans Verkuil escreveu:
On 15/12/16 15:32, Mauro Carvalho Chehab wrote:
Em Thu, 15 Dec 2016 15:03:36 +0100
Hans Verkuil escreveu:
On 15/12/16 13:56, Laurent Pinchart
On 12/15/2016 08:26 AM, Hans Verkuil wrote:
> On 15/12/16 15:45, Shuah Khan wrote:
>> On 12/15/2016 07:03 AM, Hans Verkuil wrote:
>>> On 15/12/16 13:56, Laurent Pinchart wrote:
Hi Sakari,
On Thursday 15 Dec 2016 13:30:41 Sakari Ailus wrote:
> On Tue, Dec 13, 2016 at 10:24:47AM
Em Thu, 15 Dec 2016 15:45:22 +0100
Hans Verkuil escreveu:
> On 15/12/16 15:32, Mauro Carvalho Chehab wrote:
> > Em Thu, 15 Dec 2016 15:03:36 +0100
> > Hans Verkuil escreveu:
> >
> >> On 15/12/16 13:56, Laurent Pinchart wrote:
> >>> Hi Sakari,
> >>>
>
On 15/12/16 15:45, Shuah Khan wrote:
On 12/15/2016 07:03 AM, Hans Verkuil wrote:
On 15/12/16 13:56, Laurent Pinchart wrote:
Hi Sakari,
On Thursday 15 Dec 2016 13:30:41 Sakari Ailus wrote:
On Tue, Dec 13, 2016 at 10:24:47AM -0200, Mauro Carvalho Chehab wrote:
Em Tue, 13 Dec 2016 12:53:05
On 12/15/2016 03:39 AM, Laurent Pinchart wrote:
> Hello,
>
> On Tuesday 13 Dec 2016 15:23:53 Shuah Khan wrote:
>> On 12/13/2016 05:24 AM, Mauro Carvalho Chehab wrote:
>>> Em Tue, 13 Dec 2016 12:53:05 +0200 Sakari Ailus escreveu:
On Tue, Nov 29, 2016 at 09:13:05AM -0200, Mauro Carvalho Chehab
On 12/15/2016 07:03 AM, Hans Verkuil wrote:
> On 15/12/16 13:56, Laurent Pinchart wrote:
>> Hi Sakari,
>>
>> On Thursday 15 Dec 2016 13:30:41 Sakari Ailus wrote:
>>> On Tue, Dec 13, 2016 at 10:24:47AM -0200, Mauro Carvalho Chehab wrote:
Em Tue, 13 Dec 2016 12:53:05 +0200 Sakari Ailus
On 15/12/16 15:32, Mauro Carvalho Chehab wrote:
Em Thu, 15 Dec 2016 15:03:36 +0100
Hans Verkuil escreveu:
On 15/12/16 13:56, Laurent Pinchart wrote:
Hi Sakari,
On Thursday 15 Dec 2016 13:30:41 Sakari Ailus wrote:
On Tue, Dec 13, 2016 at 10:24:47AM -0200, Mauro Carvalho
Em Thu, 15 Dec 2016 15:03:36 +0100
Hans Verkuil escreveu:
> On 15/12/16 13:56, Laurent Pinchart wrote:
> > Hi Sakari,
> >
> > On Thursday 15 Dec 2016 13:30:41 Sakari Ailus wrote:
> >> On Tue, Dec 13, 2016 at 10:24:47AM -0200, Mauro Carvalho Chehab wrote:
> >>> Em Tue, 13
On 15/12/16 13:56, Laurent Pinchart wrote:
Hi Sakari,
On Thursday 15 Dec 2016 13:30:41 Sakari Ailus wrote:
On Tue, Dec 13, 2016 at 10:24:47AM -0200, Mauro Carvalho Chehab wrote:
Em Tue, 13 Dec 2016 12:53:05 +0200 Sakari Ailus escreveu:
On Tue, Nov 29, 2016 at 09:13:05AM -0200, Mauro Carvalho
Hi Sakari,
On Thursday 15 Dec 2016 13:30:41 Sakari Ailus wrote:
> On Tue, Dec 13, 2016 at 10:24:47AM -0200, Mauro Carvalho Chehab wrote:
> > Em Tue, 13 Dec 2016 12:53:05 +0200 Sakari Ailus escreveu:
> >> On Tue, Nov 29, 2016 at 09:13:05AM -0200, Mauro Carvalho Chehab wrote:
> >>> Hi Sakari,
> >>>
Hi Mauro,
On Tue, Dec 13, 2016 at 10:24:47AM -0200, Mauro Carvalho Chehab wrote:
> Em Tue, 13 Dec 2016 12:53:05 +0200
> Sakari Ailus escreveu:
>
> > Hi Mauro,
> >
> > On Tue, Nov 29, 2016 at 09:13:05AM -0200, Mauro Carvalho Chehab wrote:
> > > Hi Sakari,
> > >
> > > I
Hello,
On Tuesday 13 Dec 2016 15:23:53 Shuah Khan wrote:
> On 12/13/2016 05:24 AM, Mauro Carvalho Chehab wrote:
> > Em Tue, 13 Dec 2016 12:53:05 +0200 Sakari Ailus escreveu:
> >> On Tue, Nov 29, 2016 at 09:13:05AM -0200, Mauro Carvalho Chehab wrote:
> >>> Hi Sakari,
> >>>
> >>> I answered you
Hi Sakari and Mauro,
On 12/13/2016 05:24 AM, Mauro Carvalho Chehab wrote:
> Em Tue, 13 Dec 2016 12:53:05 +0200
> Sakari Ailus escreveu:
>
>> Hi Mauro,
>>
>> On Tue, Nov 29, 2016 at 09:13:05AM -0200, Mauro Carvalho Chehab wrote:
>>> Hi Sakari,
>>>
>>> I answered you point
Em Tue, 13 Dec 2016 12:53:05 +0200
Sakari Ailus escreveu:
> Hi Mauro,
>
> On Tue, Nov 29, 2016 at 09:13:05AM -0200, Mauro Carvalho Chehab wrote:
> > Hi Sakari,
> >
> > I answered you point to point below, but I suspect that you missed how the
> > current approach works.
Hi Mauro,
On Tue, Nov 29, 2016 at 09:13:05AM -0200, Mauro Carvalho Chehab wrote:
> Hi Sakari,
>
> I answered you point to point below, but I suspect that you missed how the
> current approach works. So, I decided to write a quick summary here.
>
> The character devices /dev/media? are created
Hi Sakari,
I answered you point to point below, but I suspect that you missed how the
current approach works. So, I decided to write a quick summary here.
The character devices /dev/media? are created via cdev, with relies on a
kobject per device, with has an embedded struct kref inside.
Hi Mauro,
On Tue, Nov 22, 2016 at 03:44:29PM -0200, Mauro Carvalho Chehab wrote:
> Em Mon, 14 Nov 2016 15:27:22 +0200
> Sakari Ailus escreveu:
>
> > Hi Mauro,
> >
> > I'm replying below but let me first summarise the remaining problem area
> > that this patchset addresses.
On 11/22/2016 10:44 AM, Mauro Carvalho Chehab wrote:
> Em Mon, 14 Nov 2016 15:27:22 +0200
> Sakari Ailus escreveu:
>
>> Hi Mauro,
>>
>> I'm replying below but let me first summarise the remaining problem area
>> that this patchset addresses.
>
> Sorry for answering too
On 11/22/2016 11:13 AM, Hans Verkuil wrote:
> On 22/11/16 18:44, Mauro Carvalho Chehab wrote:
>>> * media: fix use-after-free in cdev_put() when app exits after driver unbind
>>> 5b28dde51d0c
>>>
>>> The patch avoids the problem of deleting a character device (cdev_del())
>>> after its memory
On 22/11/16 18:44, Mauro Carvalho Chehab wrote:
* media: fix use-after-free in cdev_put() when app exits after driver unbind
5b28dde51d0c
The patch avoids the problem of deleting a character device (cdev_del())
after its memory has been released. The change is sound as such but the
problem is
Em Mon, 14 Nov 2016 15:27:22 +0200
Sakari Ailus escreveu:
> Hi Mauro,
>
> I'm replying below but let me first summarise the remaining problem area
> that this patchset addresses.
Sorry for answering too late. Somehow, I missed this email in the cloud.
> The problems you
Hi Mauro,
I'm replying below but let me first summarise the remaining problem area
that this patchset addresses.
The problems you and Shuah have seen and partially addressed are related to
a larger picture which is the lifetime of (mostly) memory resources related
to various objects used by as
Em Wed, 9 Nov 2016 10:00:58 -0700
Shuah Khan escreveu:
> > Maybe we can get the Media Device Allocator API work in and then we can
> > get your RFC series in after that. Here is what I propose:
> >
> > - Keep the fixes in 4.9
Fixes should always be kept. Reverting a
On 11/09/2016 09:49 AM, Shuah Khan wrote:
> On 11/08/2016 01:19 AM, Sakari Ailus wrote:
>> Hi Shuah,
>>
>> On Mon, Nov 07, 2016 at 01:16:45PM -0700, Shuah Khan wrote:
>>> Hi Sakari,
>>>
>>> On 08/26/2016 05:43 PM, Sakari Ailus wrote:
Hi folks,
This is the third version of the RFC
On 11/08/2016 01:19 AM, Sakari Ailus wrote:
> Hi Shuah,
>
> On Mon, Nov 07, 2016 at 01:16:45PM -0700, Shuah Khan wrote:
>> Hi Sakari,
>>
>> On 08/26/2016 05:43 PM, Sakari Ailus wrote:
>>> Hi folks,
>>>
>>> This is the third version of the RFC set to fix referencing in media
>>> devices.
>>>
>>>
Hi Shuah,
On Mon, Nov 07, 2016 at 01:16:45PM -0700, Shuah Khan wrote:
> Hi Sakari,
>
> On 08/26/2016 05:43 PM, Sakari Ailus wrote:
> > Hi folks,
> >
> > This is the third version of the RFC set to fix referencing in media
> > devices.
> >
> > The lifetime of the media device (and media
Hi Sakari,
On 08/26/2016 05:43 PM, Sakari Ailus wrote:
> Hi folks,
>
> This is the third version of the RFC set to fix referencing in media
> devices.
>
> The lifetime of the media device (and media devnode) is now bound to that
> of struct device embedded in it and its memory is only released
59 matches
Mail list logo