Re: [PATCH] omap: resource: Fix race in register_resource()

2009-09-30 Thread Mike Chan
On Wed, Sep 30, 2009 at 4:49 PM, Kevin Hilman wrote: > Mike Chan writes: > >> Checking if the resource is already registered and adding to the list >> must be atomic or bad things can happen. >> >> Signed-off-by: Mike Chan >> --- >>  arch/arm/plat-omap/resource.c |   13 - >>  1 files

Re: [PATCH] omap: resource: Fix race in register_resource()

2009-09-30 Thread Kevin Hilman
Mike Chan writes: > Checking if the resource is already registered and adding to the list > must be atomic or bad things can happen. > > Signed-off-by: Mike Chan > --- > arch/arm/plat-omap/resource.c | 13 - > 1 files changed, 8 insertions(+), 5 deletions(-) > > diff --git a/arch/

Re: [PATCH] omap: resource: Fix race in register_resource()

2009-09-30 Thread Kevin Hilman
Mike Chan writes: > Checking if the resource is already registered and adding to the list > must be atomic or bad things can happen. > > Signed-off-by: Mike Chan Thanks, this one applies. Pushing to PM branch and pm-2.6.29. Kevin > --- > arch/arm/plat-omap/resource.c | 13 - >

Re: [PATCH] omap: resource: Fix race in register_resource()

2009-09-17 Thread Mike Chan
On Thu, Sep 17, 2009 at 4:32 PM, Mike Chan wrote: > Checking if the resource is already registered and adding to the list > must be atomic or bad things can happen. > > Signed-off-by: Mike Chan > --- >  arch/arm/plat-omap/resource.c |   13 - >  1 files changed, 8 insertions(+), 5 dele

[PATCH] omap: resource: Fix race in register_resource()

2009-09-17 Thread Mike Chan
Checking if the resource is already registered and adding to the list must be atomic or bad things can happen. Signed-off-by: Mike Chan --- arch/arm/plat-omap/resource.c | 13 - 1 files changed, 8 insertions(+), 5 deletions(-) diff --git a/arch/arm/plat-omap/resource.c b/arch/arm/

Re: [PATCH] omap: resource: Fix race in register_resource()

2009-09-08 Thread Mike Chan
On Tue, Sep 8, 2009 at 3:39 PM, Kevin Hilman wrote: > Mike Chan writes: > >> Checking if the resource is already registered and adding to the list >> must be atomic or bad things can happen. >> >> Signed-off-by: Mike Chan > > Functionally, this looks fine.  Some procedural details still to work >

Re: [PATCH] omap: resource: Fix race in register_resource()

2009-09-08 Thread Kevin Hilman
Mike Chan writes: > Checking if the resource is already registered and adding to the list > must be atomic or bad things can happen. > > Signed-off-by: Mike Chan Functionally, this looks fine. Some procedural details still to work out: Are you the original author of this? or was it Chunqiu o

[PATCH] omap: resource: Fix race in register_resource()

2009-09-03 Thread Mike Chan
Checking if the resource is already registered and adding to the list must be atomic or bad things can happen. Signed-off-by: Mike Chan --- arch/arm/plat-omap/resource.c | 14 +- 1 files changed, 9 insertions(+), 5 deletions(-) diff --git a/arch/arm/plat-omap/resource.c b/arch/arm