mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
* PageReserved removal woes: vbetool, suspend-to-ram breakage
@ 2005-11-11 10:38 Dominik Brodowski
  2005-11-22 18:54 ` Matthew Garrett
  0 siblings, 1 reply; 3+ messages in thread
From: Dominik Brodowski @ 2005-11-11 10:38 UTC (permalink / raw)
  To: npiggin, linux-kernel

Hi!

Just wanted to let you know that the warning introduced in
[PATCH] core remove PageReserved
triggers when using vbetool, which is needed on several systems for proper
suspend-to-mem support:

[4301282.746000] program vbetool is using MAP_PRIVATE, PROT_WRITE mmap of
		 VM_RESERVED memory, which is deprecated. Please report
		 this to linux-kernel@vger.kernel.org

Reports of this haven't generated many results yet, that's why I'm
addressing this with a subject line which startles more people ;)
http://lkml.org/lkml/2005/11/6/72
http://lkml.org/lkml/2005/11/6/76
http://lkml.org/lkml/2005/11/10/280

	Dominik

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: PageReserved removal woes: vbetool, suspend-to-ram breakage
  2005-11-11 10:38 PageReserved removal woes: vbetool, suspend-to-ram breakage Dominik Brodowski
@ 2005-11-22 18:54 ` Matthew Garrett
  2005-11-22 19:51   ` Hugh Dickins
  0 siblings, 1 reply; 3+ messages in thread
From: Matthew Garrett @ 2005-11-22 18:54 UTC (permalink / raw)
  To: Dominik Brodowski; +Cc: linux-kernel, npiggin

Dominik Brodowski <linux@dominikbrodowski.net> wrote:

> Just wanted to let you know that the warning introduced in
> [PATCH] core remove PageReserved

It's not much of a warning, it seems to stop vbetool from working (which
may explain a few complaints about ACPI suspend being broken in 2.6.15 -
it's not, but vbetool is...)

vbetool (well, strictly it's LRMI, but...) needs to access real-mode
memory. On the other hand, it's probably a bad idea to let it actually
scribble over RAM that the kernel may be using. So
MAP_PRIVATE|PROT_WRITE seems to be the correct thing to do. This
certainly /seemed/ to work in older kernels, but doesn't any more. What
should I be doing instead?

(I'm also not quite sure why the error claims that it's deprecated,
whereas in fact it doesn't actually work at all. Breaking userspace
without warning isn't terribly nice)
-- 
Matthew Garrett | mjg59-chiark.mail.linux-rutgers.kernel@srcf.ucam.org

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: PageReserved removal woes: vbetool, suspend-to-ram breakage
  2005-11-22 18:54 ` Matthew Garrett
@ 2005-11-22 19:51   ` Hugh Dickins
  0 siblings, 0 replies; 3+ messages in thread
From: Hugh Dickins @ 2005-11-22 19:51 UTC (permalink / raw)
  To: Matthew Garrett; +Cc: Dominik Brodowski, linux-kernel, npiggin

On Tue, 22 Nov 2005, Matthew Garrett wrote:
> Dominik Brodowski <linux@dominikbrodowski.net> wrote:
> 
> > Just wanted to let you know that the warning introduced in
> > [PATCH] core remove PageReserved
> 
> It's not much of a warning, it seems to stop vbetool from working (which
> may explain a few complaints about ACPI suspend being broken in 2.6.15 -
> it's not, but vbetool is...)
> 
> vbetool (well, strictly it's LRMI, but...) needs to access real-mode
> memory. On the other hand, it's probably a bad idea to let it actually
> scribble over RAM that the kernel may be using. So
> MAP_PRIVATE|PROT_WRITE seems to be the correct thing to do. This
> certainly /seemed/ to work in older kernels, but doesn't any more. What
> should I be doing instead?
> 
> (I'm also not quite sure why the error claims that it's deprecated,
> whereas in fact it doesn't actually work at all. Breaking userspace
> without warning isn't terribly nice)

Sorry about that, we really didn't expect that it was being used.
And sorry I didn't get the fixes ready in time for 2.6.15-rc2.
But they've gone into Linus' git tree a couple of hours ago.
Please let me know if tomorrow's 2.6.15-rc2-git3 does not work for you.

Hugh

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2005-11-22 19:51 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2005-11-11 10:38 PageReserved removal woes: vbetool, suspend-to-ram breakage Dominik Brodowski
2005-11-22 18:54 ` Matthew Garrett
2005-11-22 19:51   ` Hugh Dickins

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®