lkml.org 
[lkml]   [2014]   [Aug]   [4]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] Hibernate: check unsafe page should not in e820 reserved region
Hi Pavel, 

Thanks for your review!

On Mon, Aug 04, 2014 at 03:08:11PM +0200, Pavel Machek wrote:
> > > +
> > > + list_for_each_entry(region, &nosave_regions, list) {
> > > + if (pfn >= region->start_pfn && pfn < region->end_pfn) {
> > > + pr_err("PM: %#010llx in e820 nosave regsion: "
>
> And while looking at it, this should probably be spelled "region".
>
> ...plus maybe past part of changelog here into the comment?
>

Oh, sorry for my typo! I will correct it and comment in next version.

> Otherwise looks good to me,
>
> Acked-by: Pavel Machek <pavel@ucw.cz>
> Pavel

Regards
Joey Lee


\
 
 \ /
  Last update: 2014-08-04 17:21    [W:0.050 / U:0.152 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site