* -mm swsusp: copy_page is harmfull
@ 2004-07-28 22:26 Pavel Machek
0 siblings, 0 replies; only message in thread
From: Pavel Machek @ 2004-07-28 22:26 UTC (permalink / raw)
To: Patrick Mochel, Andrew Morton, kernel list
Hi!
This is my fault from long time ago: copy_page can't be used for
copying task struct, therefore we can't use it in swsusp. Please
apply,
Pavel
--- clean-mm/kernel/power/swsusp.c 2004-07-28 23:39:49.000000000 +0200
+++ linux-mm/kernel/power/swsusp.c 2004-07-28 23:30:33.000000000 +0200
@@ -614,12 +614,8 @@
struct page * page;
page = pfn_to_page(zone_pfn + zone->zone_start_pfn);
pbe->orig_address = (long) page_address(page);
- /* Copy page is dangerous: it likes to mess with
- preempt count on specific cpus. Wrong preempt
- count is then copied, oops.
- */
- copy_page((void *)pbe->address,
- (void *)pbe->orig_address);
+ /* copy_page is no usable for copying task structs. */
+ memcpy((void *)pbe->address, (void *)pbe->orig_address, PAGE_SIZE);
pbe++;
}
}
--
People were complaining that M$ turns users into beta-testers...
...jr ghea gurz vagb qrirybcref, naq gurl frrz gb yvxr vg gung jnl!
^ permalink raw reply [flat|nested] only message in thread
only message in thread, other threads:[~2004-07-28 22:30 UTC | newest]
Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2004-07-28 22:26 -mm swsusp: copy_page is harmfull Pavel Machek
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®