<html>
  <head>
    <meta content="text/html; charset=utf-8" http-equiv="Content-Type">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    <div class="moz-cite-prefix">On 24.09.2015 00:52, Steven Newbury
      wrote:<br>
    </div>
    <blockquote cite="mid:1443048770.1731.33.camel@snewbury.org.uk"
      type="cite">
      <pre wrap="">On Wed, 2015-09-23 at 23:41 +0200, Lukas Wunner wrote:
</pre>
      <blockquote type="cite">
        <pre wrap="">Hi,

On Wed, Sep 23, 2015 at 08:37:48PM +0000, Steven Newbury wrote:
</pre>
        <blockquote type="cite">
          <pre wrap="">I can't figure out how to get a pointer to the radeon_device struct
for a specific card, or the parent drm_device from an external
device
driver.
</pre>
        </blockquote>
        <pre wrap="">
struct device -> struct drm_device: dev_get_drvdata()
struct pci_dev -> struct drm_device: pci_get_drvdata()
struct drm_device -> struct radeon_device: drm_device->dev_private

</pre>
      </blockquote>
      <pre wrap="">Thanks, that's useful.

</pre>
      <blockquote type="cite">
        <pre wrap="">
</pre>
        <blockquote type="cite">
          <pre wrap="">I imagine I somehow need to take a reference to the drm class
kobject
for the card in question, and in so doing presumably I should then
be
able to discover the pointer to device.
</pre>
        </blockquote>
        <pre wrap="">
It sounds like you want to discover the available radeon cards in the
system? If so you could iterate over all pci devices and look for
pci->vendor == PCI_VENDOR_ID_ATI || pci->vendor == PCI_VENDOR_ID_AMD,
then get to the radeon_device as shown above.

</pre>
      </blockquote>
      <pre wrap="">Yes, my plan was to eventually discover all the drm devices on the
system and make available a sysfs entry to allocate a volume from VRAM
for each capable card selecting an appropriate backend for driver.  I
was initially just going to implement it for radoen using a static
allocation from module params.

</pre>
      <blockquote type="cite">
        <pre wrap="">However as Christian König pointed out, memory allocation is driver
dependent. For an initial proof of concept it may be simplest to hack
the radeon driver. Then you'll get an idea which parts are generic
and
which are driver specific and you can move the generic stuff to a
central broker.

</pre>
      </blockquote>
      <pre wrap="">Hacking the radeon driver was very much something I'd considered; I'd
only decided not to because I was thinking too far ahead really.  I
need to keep it simple, as you say, and only add complexity once I have
something working, and hopefully able to demonstrate its utility.

</pre>
      <blockquote type="cite">
        <pre wrap="">Rather than discovering the VRAM it probably makes more sense to have
drm devices register a set of callbacks with the central broker
(e.g. return amount of currently free VRAM, allocate VRAM for use as
block device, deallocate VRAM, read vector of blocks, write vector
of blocks). The broker could then be controlled from user space via
sysfs or ioctls or whatever.
</pre>
      </blockquote>
      <pre wrap="">For now I think I'll just take the approach bcache does with "thinly
provisioned volumes", and have an entry in the radeon sysfs:

/sys/class/drm/card0/device/vram_volume_create

which will attempt to allocate a given sized volume and register a
vrambd[0]. (or some better name?) Unregistering/deallocation can be
triggered from /sys/block/vramd[0]/vrambd/unregister
</pre>
    </blockquote>
    <br>
    Yeah, that approach sounds reasonable to me I would just go into a
    different direction with the sysfs interface. <br>
    <br>
    It doesn't make much sense to have more than one volume for each
    card (doesn't it?). So I would rather say instead of having a
    vram_volume_create sysfs you should rather go with something like
    vram_volume_size.<br>
    <br>
    E.g. setting the size makes the volume available, setting it to zero
    tries to free it again. If anybody is using it while you try to
    change the size return -EBUSY.<br>
    <br>
    Regards,<br>
    Christian.<br>
    <br>
    <blockquote cite="mid:1443048770.1731.33.camel@snewbury.org.uk"
      type="cite"><br>
      <fieldset class="mimeAttachmentHeader"></fieldset>
      <br>
      <pre wrap="">_______________________________________________
dri-devel mailing list
<a class="moz-txt-link-abbreviated" href="mailto:dri-devel@lists.freedesktop.org">dri-devel@lists.freedesktop.org</a>
<a class="moz-txt-link-freetext" href="http://lists.freedesktop.org/mailman/listinfo/dri-devel">http://lists.freedesktop.org/mailman/listinfo/dri-devel</a>
</pre>
    </blockquote>
    <br>
  </body>
</html>