Re: [PATCH] DSPBRIDGE: Prevent memory corruption in DRV_ProcFreeDMMRes

2010-02-11 Thread Ameya Palande
Hi all, On Thu, 2010-02-11 at 16:53 +0100, Contreras Felipe (Nokia-D/Helsinki) wrote: > On Wed, Feb 10, 2010 at 06:31:35PM +0100, ext Guzman Lugo, Fernando wrote: > > >-Original Message- > > >From: Felipe Contreras [mailto:felipe.contre...@nokia.com] > > >> Resource cleanup does not suppor

Re: [PATCH] DSPBRIDGE: Prevent memory corruption in DRV_ProcFreeDMMRes

2010-02-11 Thread Felipe Contreras
On Wed, Feb 10, 2010 at 06:31:35PM +0100, ext Guzman Lugo, Fernando wrote: > >-Original Message- > >From: Felipe Contreras [mailto:felipe.contre...@nokia.com] > >> Resource cleanup does not support that even without my proposed changes. > > > >Aha! I suspected it :P > > > Resource cleanup d

RE: [PATCH] DSPBRIDGE: Prevent memory corruption in DRV_ProcFreeDMMRes

2010-02-10 Thread Guzman Lugo, Fernando
t;Subject: Re: [PATCH] DSPBRIDGE: Prevent memory corruption in >DRV_ProcFreeDMMRes > >On Wed, Feb 10, 2010 at 05:06:26AM +0100, ext Guzman Lugo, Fernando wrote: >> >At this point there's an obvious question; what's the point of >> >reserving a memory region and no

Re: [PATCH] DSPBRIDGE: Prevent memory corruption in DRV_ProcFreeDMMRes

2010-02-10 Thread Felipe Contreras
On Wed, Feb 10, 2010 at 05:06:26AM +0100, ext Guzman Lugo, Fernando wrote: > >At this point there's an obvious question; what's the point of > >reserving a memory region and not mapping it? > > > >I remember the answer from Hari was: some clients prefer to reserve a > >big region once, and map part

RE: [PATCH] DSPBRIDGE: Prevent memory corruption in DRV_ProcFreeDMMRes

2010-02-09 Thread Guzman Lugo, Fernando
pak; felipe.contre...@nokia.com; ext- >phil.2.carm...@nokia.com >Subject: Re: [PATCH] DSPBRIDGE: Prevent memory corruption in >DRV_ProcFreeDMMRes > >On Tue, Feb 9, 2010 at 10:22 PM, Guzman Lugo, Fernando >wrote: >> The root problem here is that remove the DMM element should be

Re: [PATCH] DSPBRIDGE: Prevent memory corruption in DRV_ProcFreeDMMRes

2010-02-09 Thread Felipe Contreras
On Tue, Feb 9, 2010 at 10:22 PM, Guzman Lugo, Fernando wrote: > The root problem here is that remove the DMM element should be in PROC_UnMap > but in PROC_UnReserveMemory, because apart of the problem your are seeing > about memory corruption if the application does a PROC_ReserveMemory and then

RE: [PATCH] DSPBRIDGE: Prevent memory corruption in DRV_ProcFreeDMMRes

2010-02-09 Thread Guzman Lugo, Fernando
gt;Chitriki Rudramuni, Deepak; felipe.contre...@nokia.com; ext- >phil.2.carm...@nokia.com >Subject: Re: [PATCH] DSPBRIDGE: Prevent memory corruption in >DRV_ProcFreeDMMRes > >On Mon, Feb 8, 2010 at 10:25 PM, Ameya Palande >wrote: >> Background: bridge_close() has the r

Re: [PATCH] DSPBRIDGE: Prevent memory corruption in DRV_ProcFreeDMMRes

2010-02-09 Thread Felipe Contreras
On Mon, Feb 8, 2010 at 10:25 PM, Ameya Palande wrote: > Background: bridge_close() has the responsibility to cleanup all the > resources allocated by user space process. One of those resources is > DMMRes which is used for tracking DMM resource allocation done in > PROC_Map() and PROC_UnMap(). > >

[PATCH] DSPBRIDGE: Prevent memory corruption in DRV_ProcFreeDMMRes

2010-02-08 Thread Ameya Palande
Background: bridge_close() has the responsibility to cleanup all the resources allocated by user space process. One of those resources is DMMRes which is used for tracking DMM resource allocation done in PROC_Map() and PROC_UnMap(). DRV_ProcFreeDMMRes() function was used for cleaning up DMMRes has