2012-02-14 00:04:54

by Bjorn Helgaas

Subject: [PATCH] PM / Hibernate: print physical addresses consistently with other parts of kernel

Print physical address info in a style consistent with the %pR style used
elsewhere in the kernel.

Signed-off-by: Bjorn Helgaas <[email protected]>
---
kernel/power/snapshot.c | 7 ++++---
1 files changed, 4 insertions(+), 3 deletions(-)

diff --git a/kernel/power/snapshot.c b/kernel/power/snapshot.c
index 6a768e5..8e2e746 100644
--- a/kernel/power/snapshot.c
+++ b/kernel/power/snapshot.c
@@ -711,9 +711,10 @@ static void mark_nosave_pages(struct memory_bitmap *bm)
list_for_each_entry(region, &nosave_regions, list) {
unsigned long pfn;

- pr_debug("PM: Marking nosave pages: %016lx - %016lx\n",
- region->start_pfn << PAGE_SHIFT,
- region->end_pfn << PAGE_SHIFT);
+ pr_debug("PM: Marking nosave pages: [mem %#010llx-%#010llx]\n",
+ (unsigned long long) region->start_pfn << PAGE_SHIFT,
+ ((unsigned long long) region->end_pfn << PAGE_SHIFT)
+ - 1);

for (pfn = region->start_pfn; pfn < region->end_pfn; pfn++)
if (pfn_valid(pfn)) {

2012-02-14 21:32:33

by Rafael J. Wysocki

Subject: Re: [PATCH] PM / Hibernate: print physical addresses consistently with other parts of kernel

On Tuesday, February 14, 2012, Bjorn Helgaas wrote:
> Print physical address info in a style consistent with the %pR style used
> elsewhere in the kernel.
>
> Signed-off-by: Bjorn Helgaas <[email protected]>

Applied.

Thanks,
Rafael


> ---
> kernel/power/snapshot.c | 7 ++++---
> 1 files changed, 4 insertions(+), 3 deletions(-)
>
> diff --git a/kernel/power/snapshot.c b/kernel/power/snapshot.c
> index 6a768e5..8e2e746 100644
> --- a/kernel/power/snapshot.c
> +++ b/kernel/power/snapshot.c
> @@ -711,9 +711,10 @@ static void mark_nosave_pages(struct memory_bitmap *bm)
> list_for_each_entry(region, &nosave_regions, list) {
> unsigned long pfn;
>
> - pr_debug("PM: Marking nosave pages: %016lx - %016lx\n",
> - region->start_pfn << PAGE_SHIFT,
> - region->end_pfn << PAGE_SHIFT);
> + pr_debug("PM: Marking nosave pages: [mem %#010llx-%#010llx]\n",
> + (unsigned long long) region->start_pfn << PAGE_SHIFT,
> + ((unsigned long long) region->end_pfn << PAGE_SHIFT)
> + - 1);
>
> for (pfn = region->start_pfn; pfn < region->end_pfn; pfn++)
> if (pfn_valid(pfn)) {
>
>
>