Re: [PATCH 5/5] Swapless V2: Revise main migration logic

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi,
On Fri, 14 Apr 2006 09:48:25 -0700 (PDT)
Christoph Lameter <[email protected]> wrote:
> > +/* write protected page under migration*/
> > +#define SWP_TYPE_MIGRATION_WP	(MAX_SWAPFILES - 1)
> > +/* write enabled migration type */
> > +#define SWP_TYPE_MIGRATION_WE	(MAX_SWAPFILES)
> 
> Could we call this SWP_TYPE_MIGRATION_READ / WRITE?
> 
ok, it looks better.

> > +	pte = pte_mkold(mk_pte(new, vma->vm_page_prot));
> > +	if (is_migration_entry_we(entry)) {
> is_write_migration_entry?
> 
> > +		pte = pte_mkwrite(pte);
> > +	}
> 
> No {} needed.
> 
> > -			entry = make_migration_entry(page);
> > +			if (pte_write(pteval))
> > +				entry = make_migration_entry(page, 1);
> > +			else
> > +				entry = make_migration_entry(page, 0);
> >  		}
> 
> entry = make_migration_entry(page, pte_write(pteval))
> 
> ?
Ah, O.K.

Thanks,
-Kame


-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]
  Powered by Linux