Re: [PATCH v3 2/3] resource: Add release_mem_region_adjustable()

2013-04-24 Thread Toshi Kani
On Wed, 2013-04-24 at 16:42 +0800, Ram Pai wrote: > On Thu, Apr 11, 2013 at 10:30:02AM -0600, Toshi Kani wrote: > > On Wed, 2013-04-10 at 15:24 -0700, Andrew Morton wrote: > > > On Wed, 10 Apr 2013 15:08:29 -0700 (PDT) David Rientjes > > > wrote: > > > > > > > On Wed, 10 Apr 2013, Toshi Kani wro

Re: [PATCH v3 2/3] resource: Add release_mem_region_adjustable()

2013-04-24 Thread Ram Pai
On Thu, Apr 11, 2013 at 10:30:02AM -0600, Toshi Kani wrote: > On Wed, 2013-04-10 at 15:24 -0700, Andrew Morton wrote: > > On Wed, 10 Apr 2013 15:08:29 -0700 (PDT) David Rientjes > > wrote: > > > > > On Wed, 10 Apr 2013, Toshi Kani wrote: > > > > > > > > I'll switch it to GFP_ATOMIC. Which is h

Re: [PATCH v3 2/3] resource: Add release_mem_region_adjustable()

2013-04-11 Thread Toshi Kani
On Wed, 2013-04-10 at 15:24 -0700, Andrew Morton wrote: > On Wed, 10 Apr 2013 15:08:29 -0700 (PDT) David Rientjes > wrote: > > > On Wed, 10 Apr 2013, Toshi Kani wrote: > > > > > > I'll switch it to GFP_ATOMIC. Which is horridly lame but the > > > > allocation is small and alternatives are unob

Re: [PATCH v3 2/3] resource: Add release_mem_region_adjustable()

2013-04-10 Thread Andrew Morton
On Wed, 10 Apr 2013 15:08:29 -0700 (PDT) David Rientjes wrote: > On Wed, 10 Apr 2013, Toshi Kani wrote: > > > > I'll switch it to GFP_ATOMIC. Which is horridly lame but the > > > allocation is small and alternatives are unobvious. > > > > Great! Again, thanks for the update! > > release_mem

Re: [PATCH v3 2/3] resource: Add release_mem_region_adjustable()

2013-04-10 Thread David Rientjes
On Wed, 10 Apr 2013, Toshi Kani wrote: > > I'll switch it to GFP_ATOMIC. Which is horridly lame but the > > allocation is small and alternatives are unobvious. > > Great! Again, thanks for the update! release_mem_region_adjustable() allocates at most one struct resource, so why not do kmalloc

Re: [PATCH v3 2/3] resource: Add release_mem_region_adjustable()

2013-04-10 Thread Toshi Kani
On Wed, 2013-04-10 at 14:44 -0700, Andrew Morton wrote: > On Wed, 10 Apr 2013 11:17:00 -0600 Toshi Kani wrote: > > > Added release_mem_region_adjustable(), which releases a requested > > region from a currently busy memory resource. This interface > > adjusts the matched memory resource accordin

Re: [PATCH v3 2/3] resource: Add release_mem_region_adjustable()

2013-04-10 Thread Andrew Morton
On Wed, 10 Apr 2013 11:17:00 -0600 Toshi Kani wrote: > Added release_mem_region_adjustable(), which releases a requested > region from a currently busy memory resource. This interface > adjusts the matched memory resource accordingly even if the > requested region does not match exactly but stil