All of lore.kernel.org
 help / color / mirror / Atom feed
From: Tormod Volden <lists.tormod@gmail.com>
To: dri-devel@lists.freedesktop.org
Subject: multiple framebuffer drm maps
Date: Mon, 25 Apr 2011 23:54:37 +0200	[thread overview]
Message-ID: <BANLkTimJ+jBvVawHo67oB44SkZm=ToZURA@mail.gmail.com> (raw)

In commit 41c2e75e60200a860a74b7c84a6375c105e7437f "drm: Make
drm_local_map use a resource_size_t offset" [1] the support for multiple
_DRM_FRAMEBUFFER maps per device was taken away. This change made the
savage drivers upset, since these cards have several apertures (the
layout is different between card families) for which the kernel drm
driver sets up maps. And these maps are now mixed up into one broken one.

The drivers (drm, ddx, mesa) for instance expects a framebuffer map and a
tiled aperture map, and the broken maps show up as rendering corruption
[2] and allocation failures. I have tried to come up with userland
workarounds but it seems impossible since the kernel will only return
the handle to a broken map and there is no way to remap it correctly.

Would it be possible to reintroduce this support? One solution could be
a new flag _DRM_IGNORE_FB_OFFSET that can be used by those drivers that
need it, or the other way around, a _DRM_CHECK_FB_OFFSET to be added
by the savage drivers and others in the same situation. I can of course
try to write a patch if people think this is a good idea.

Best regards,
Tormod

[1] http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=41c2e75e60200a860a74b7c84a6375c105e7437f
[2] https://bugs.freedesktop.org/show_bug.cgi?id=32511

             reply	other threads:[~2011-04-25 21:54 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-04-25 21:54 Tormod Volden [this message]
2011-05-03 21:20 ` multiple framebuffer drm maps Tormod Volden
2011-05-03 21:54   ` Tormod Volden
2011-05-22 19:27     ` [PATCH] drm/savage: Do not add framebuffer and aperture maps Tormod Volden
2011-05-27 18:19       ` [PATCH resend] " Tormod Volden
2011-05-27 20:48         ` Dave Airlie
2011-05-27 23:44           ` Tormod Volden
2011-05-29 23:12             ` [PATCH] drm: Compare only lower 32 bits of framebuffer map offsets Tormod Volden
2011-05-29 23:29               ` Dave Airlie
2011-05-30 19:45                 ` [PATCH v2] " Tormod Volden
2011-05-31 22:04                   ` Tormod Volden
2011-06-10 22:59                   ` Tormod Volden

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='BANLkTimJ+jBvVawHo67oB44SkZm=ToZURA@mail.gmail.com' \
    --to=lists.tormod@gmail.com \
    --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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.