Re: NUMA API for Linux

[Andi Kleen <ak@suse.de>]

On Wed, 07 Apr 2004 14:24:19 -0700
Matthew Dobson <colpatch@us.ibm.com> wrote:

> 	I must be missing something here, but did you not include mempolicy.h
> and policy.c in these patches?  I can't seem to find them anywhere?!? 
> It's really hard to evaluate your patches if the core of them is
> missing!

It was in the core patch and also in the last patch I sent Andrew.
See ftp://ftp.suse.com/pub/people/ak/numa/* for the full patches

> 
> Andrew already mentioned your mistake on the i386 syscalls which needs
> to be fixed.

That's already fixed
 
> Also, this snippet of code is in 2 of your patches (#1 and #6) causing
> rejects:
> 
> @@ -435,6 +445,8 @@
>  
>  struct page *shmem_nopage(struct vm_area_struct * vma,
>                          unsigned long address, int *type);
> +int shmem_set_policy(struct vm_area_struct *vma, struct mempolicy
> *new);
> +struct mempolicy *shmem_get_policy(struct vm_area_struct *vma, unsigned
> long addr);
>  struct file *shmem_file_setup(char * name, loff_t size, unsigned long
> flags);
>  void shmem_lock(struct file * file, int lock);
>  int shmem_zero_setup(struct vm_area_struct *);


It didn't reject for me. 

> Just from the patches you posted, I would really disagree that these are
> ready for merging into -mm.

Why so? 

-Andi