|  |  | 
  
    |  |  | 
 
  |   |  | 
  
    |  |  | 
  
    |  |  | 
  
    |   xen-devel
[Xen-devel] Re: [PATCH V3 1/2] mm: Add SECTION_ALIGN_UP() and	SECTION_AL 
| To: | David Rientjes <rientjes@xxxxxxxxxx> |  
| Subject: | [Xen-devel] Re: [PATCH V3 1/2] mm: Add SECTION_ALIGN_UP() and	SECTION_ALIGN_DOWN() macro |  
| From: | Daniel Kiper <dkiper@xxxxxxxxxxxx> |  
| Date: | Thu, 19 May 2011 21:56:53 +0200 |  
| Cc: | jeremy@xxxxxxxx, xen-devel@xxxxxxxxxxxxxxxxxxx, ian.campbell@xxxxxxxxxx,	haicheng.li@xxxxxxxxxxxxxxx, konrad.wilk@xxxxxxxxxx,	linux-kernel@xxxxxxxxxxxxxxx, dan.magenheimer@xxxxxxxxxx,	v.tolstov@xxxxxxxxx, Dave Hansen <dave@xxxxxxxxxxxxxxxxxx>,	linux-mm@xxxxxxxxx, Andi Kleen <andi.kleen@xxxxxxxxx>,	Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx>,	fengguang.wu@xxxxxxxxx, Daniel Kiper <dkiper@xxxxxxxxxxxx>,	wdauchy@xxxxxxxxx |  
| Delivery-date: | Thu, 19 May 2011 12:59:51 -0700 |  
| Envelope-to: | www-data@xxxxxxxxxxxxxxxxxxx |  
| In-reply-to: | <alpine.DEB.2.00.1105182019170.20651@xxxxxxxxxxxxxxxxxxxxxxxxx> |  
| List-help: | <mailto:xen-devel-request@lists.xensource.com?subject=help> |  
| List-id: | Xen developer discussion <xen-devel.lists.xensource.com> |  
| List-post: | <mailto:xen-devel@lists.xensource.com> |  
| List-subscribe: | <http://lists.xensource.com/mailman/listinfo/xen-devel>,	<mailto:xen-devel-request@lists.xensource.com?subject=subscribe> |  
| List-unsubscribe: | <http://lists.xensource.com/mailman/listinfo/xen-devel>,	<mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe> |  
| References: | <20110517213750.GB30232@xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx>	<alpine.DEB.2.00.1105182019170.20651@xxxxxxxxxxxxxxxxxxxxxxxxx> |  
| Sender: | xen-devel-bounces@xxxxxxxxxxxxxxxxxxx |  
| User-agent: | Mutt/1.3.28i |  
| On Wed, May 18, 2011 at 08:21:23PM -0700, David Rientjes wrote:
> On Tue, 17 May 2011, Daniel Kiper wrote:
>
> > diff --git a/include/linux/mmzone.h b/include/linux/mmzone.h
> > index d715200..217bcf6 100644
> > --- a/include/linux/mmzone.h
> > +++ b/include/linux/mmzone.h
> > @@ -956,6 +956,9 @@ static inline unsigned long early_pfn_to_nid(unsigned 
> > long pfn)
> >  #define pfn_to_section_nr(pfn) ((pfn) >> PFN_SECTION_SHIFT)
> >  #define section_nr_to_pfn(sec) ((sec) << PFN_SECTION_SHIFT)
> >
> > +#define SECTION_ALIGN_UP(pfn)      (((pfn) + PAGES_PER_SECTION - 1) & 
> > PAGE_SECTION_MASK)
> > +#define SECTION_ALIGN_DOWN(pfn)    ((pfn) & PAGE_SECTION_MASK)
> > +
> >  struct page;
> >  struct page_cgroup;
> >  struct mem_section {
>
> These seem useful.  Could you convert the code in drivers/base/node.c,
> mm/page_cgroup.c, mm/page_alloc.c, and mm/sparse.c that already do this to
> use the new macros?
No problem. I do that in next week or two.
Daniel
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel
 | 
 |  | 
  
    |  |  |