Subrata Modak wrote:
> Would you like to incorporate Andi´s review comments inthe next patch.
> On Mon, 2008-06-30 at 14:39 +0200, Andi Kleen wrote:
> > > +void
> > > +verify_pages_on_nodes(int *status, unsigned int num, int *nodes)
> > > +{
> > > + unsigned int i;
> > > +
> > > + for (i = 0; i < num; i++) {
> > > + if (status[i] != nodes[i]) {
> > > + tst_resm(TFAIL, "page %d on node %d, "
> > > + "expected on node %d", i,
> > > + status[i], nodes[i]);
> > > + return;
> > > + }
> >
> > The correct way to verify this would be to use get_mempolicy(...,
> > MPOL_F_NODE|MPOL_F_ADDR, &node) on each page and check if the node
> > is correct.
OK, I will make this change and provide an updated patch.
It would also be great if someone can explain the interaction between
mlock and move_pages. The man page
http://linux.die.net/man/2/move_pages says that the status[] will be
set to -EPERM if the page is mlocked. But that does not seem to
happen, and an mlocked page gets happily moved around. So what is the
actual expected behaviour?
Regards,
Vijay
-------------------------------------------------------------------------
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list
