dri-devel Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: bugzilla-daemon@freedesktop.org
To: dri-devel@lists.freedesktop.org
Subject: [Bug 61182] r600g causes KWin crashes with kernel 3.8
Date: Sun, 21 Apr 2013 01:55:33 +0000	[thread overview]
Message-ID: <bug-61182-502-m7nld4P1BP@http.bugs.freedesktop.org/> (raw)
In-Reply-To: <bug-61182-502@http.bugs.freedesktop.org/>


[-- Attachment #1.1: Type: text/plain, Size: 1564 bytes --]

https://bugs.freedesktop.org/show_bug.cgi?id=61182

--- Comment #34 from D. Hugh Redelmeier <hugh@mimosa.com> ---
Thanks, Alex, for the clear restatement.

Naively, I think of two C/UNIX conventions. For allocations that can either
succeed or fail, typically the result is a pointer which is NULL for failure --
eg. malloc(3)).  For allocations that can partially succeed, the result is the
amount successfully processed (think write(2) which returns the length actually
transferred).

It seems to me that for allocating address space in the VRAM window
(aperture?), success can partial, and anything that deals with that window
needs to be aware that accessing some object may require piecewise operations,
punctuated by adjustment of the mapping.

In other words, this case doesn't sound pathological; it should be handled as a
normal case.

I'm not saying that comment #29 is wrong.  I'm saying that the existing code
ought to have been written to handle this case.  Clearly one solution is to
replace the code as suggested.  But fixing the code ought to be feasible too. 
Are there other lurking bugs where code assumes addressability?

I reiterate: I'm not knowledgeable about modern video architectures or about X
server architectures.

Footnote: Why do I think that VRAM windows cannot always map the whole VRAM? 
Because video cards now routinely have gigabytes of VRAM and 32-bit address
spaces for x86 machines cannot spare enough to allocate for that much VRAM
(ignoring PAE).

-- 
You are receiving this mail because:
You are the assignee for the bug.

[-- Attachment #1.2: Type: text/html, Size: 2364 bytes --]

[-- Attachment #2: Type: text/plain, Size: 159 bytes --]

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel

  parent reply	other threads:[~2013-04-21  1:55 UTC|newest]

Thread overview: 56+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-02-20 18:58 [Bug 61182] New: r600g causes KWin crashes with kernel 3.8 bugzilla-daemon
2013-02-22 23:33 ` [Bug 61182] " bugzilla-daemon
2013-02-23 19:04 ` bugzilla-daemon
2013-02-24 23:41 ` bugzilla-daemon
2013-03-01 11:27 ` bugzilla-daemon
2013-03-01 11:28 ` bugzilla-daemon
2013-03-03 22:57 ` bugzilla-daemon
2013-03-12 15:03 ` bugzilla-daemon
2013-03-19 11:14 ` bugzilla-daemon
2013-03-19 12:54 ` bugzilla-daemon
2013-03-19 13:14 ` bugzilla-daemon
2013-03-19 16:31 ` bugzilla-daemon
2013-03-19 16:58 ` bugzilla-daemon
2013-03-19 17:03 ` bugzilla-daemon
2013-03-19 17:36 ` bugzilla-daemon
2013-03-20 14:41 ` bugzilla-daemon
2013-03-22 17:12 ` bugzilla-daemon
2013-03-29 18:21 ` bugzilla-daemon
2013-03-29 18:31 ` bugzilla-daemon
2013-03-29 20:00 ` bugzilla-daemon
2013-03-31  4:09 ` bugzilla-daemon
2013-04-02  1:19 ` bugzilla-daemon
2013-04-02  8:21 ` bugzilla-daemon
2013-04-02 10:28 ` bugzilla-daemon
2013-04-02 13:00 ` bugzilla-daemon
2013-04-02 18:31 ` bugzilla-daemon
2013-04-02 23:07 ` bugzilla-daemon
2013-04-05 13:51 ` bugzilla-daemon
2013-04-06  0:18 ` bugzilla-daemon
2013-04-06  1:47 ` bugzilla-daemon
2013-04-08 15:30 ` bugzilla-daemon
2013-04-19  9:15 ` bugzilla-daemon
2013-04-20 20:30 ` bugzilla-daemon
2013-04-20 22:19 ` bugzilla-daemon
2013-04-21  1:55 ` bugzilla-daemon [this message]
2013-04-21  5:14 ` bugzilla-daemon
2013-04-21 13:20 ` bugzilla-daemon
2013-04-21 13:27 ` bugzilla-daemon
2013-04-21 17:46 ` bugzilla-daemon
2013-04-21 18:30 ` bugzilla-daemon
2013-04-21 19:03 ` bugzilla-daemon
2013-04-22  8:05 ` bugzilla-daemon
2013-04-23  2:25 ` bugzilla-daemon
2013-04-23  3:01 ` bugzilla-daemon
2013-04-23  3:08 ` bugzilla-daemon
2013-04-23  8:56 ` bugzilla-daemon
2013-04-23 11:33 ` bugzilla-daemon
2013-04-23 17:48 ` bugzilla-daemon
2013-05-13 12:07 ` [Bug 61182] r600g causes KWin, Gnome Shell and Unity " bugzilla-daemon
2013-05-13 12:19 ` bugzilla-daemon
2013-05-13 14:32 ` bugzilla-daemon
2013-05-13 14:32 ` bugzilla-daemon
2013-05-13 14:52 ` bugzilla-daemon
2013-07-24 15:39 ` bugzilla-daemon
2013-07-24 15:51 ` bugzilla-daemon
2013-07-24 17:35 ` bugzilla-daemon

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=bug-61182-502-m7nld4P1BP@http.bugs.freedesktop.org/ \
    --to=bugzilla-daemon@freedesktop.org \
    --cc=dri-devel@lists.freedesktop.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox