From: Alexandre Courbot <acourbot@nvidia.com>
To: Ben Skeggs <skeggsb@gmail.com>
Cc: Ben Skeggs <bskeggs@redhat.com>,
"nouveau@lists.freedesktop.org" <nouveau@lists.freedesktop.org>,
"dri-devel@lists.freedesktop.org"
<dri-devel@lists.freedesktop.org>,
Alexandre Courbot <gnurou@gmail.com>,
Eric Brower <ebrower@nvidia.com>,
Stephen Warren <swarren@wwwdotorg.org>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
"linux-tegra@vger.kernel.org" <linux-tegra@vger.kernel.org>,
Terje Bergstrom <tbergstrom@nvidia.com>,
Ken Adams <KAdams@nvidia.com>
Subject: Re: [RFC 07/16] drm/nouveau/bar/nvc0: support chips without BAR3
Date: Tue, 4 Feb 2014 17:31:48 +0900 [thread overview]
Message-ID: <52F0A574.1070601@nvidia.com> (raw)
In-Reply-To: <CACAvsv7BDDEOJ-C88PRWy7z21c1LMorb5r=f6atDJAWtTN=Luw@mail.gmail.com>
On 02/04/2014 12:54 PM, Ben Skeggs wrote:
> On Sat, Feb 1, 2014 at 1:16 PM, Alexandre Courbot <acourbot@nvidia.com> wrote:
>> Adapt the NVC0 BAR driver to make it able to support chips that do not
>> expose a BAR3. When this happens, BAR1 is then used for USERD mapping
>> and the BAR alloc() functions is disabled, making GPU objects unable
>> to rely on BAR for data access and falling back to PRAMIN.
>>
>> Signed-off-by: Alexandre Courbot <acourbot@nvidia.com>
>> ---
>> drivers/gpu/drm/nouveau/core/subdev/bar/nvc0.c | 115 +++++++++++++------------
>> 1 file changed, 61 insertions(+), 54 deletions(-)
>>
>> diff --git a/drivers/gpu/drm/nouveau/core/subdev/bar/nvc0.c b/drivers/gpu/drm/nouveau/core/subdev/bar/nvc0.c
>> index 3f30db6..c2bb0e5 100644
>> --- a/drivers/gpu/drm/nouveau/core/subdev/bar/nvc0.c
>> +++ b/drivers/gpu/drm/nouveau/core/subdev/bar/nvc0.c
>> @@ -79,87 +79,88 @@ nvc0_bar_unmap(struct nouveau_bar *bar, struct nouveau_vma *vma)
>> }
>>
>> static int
>> -nvc0_bar_ctor(struct nouveau_object *parent, struct nouveau_object *engine,
>> - struct nouveau_oclass *oclass, void *data, u32 size,
>> - struct nouveau_object **pobject)
>> +nvc0_bar_init_vm(struct nvc0_bar_priv *priv, int nr, int bar)
>> {
>> - struct nouveau_device *device = nv_device(parent);
>> - struct nvc0_bar_priv *priv;
>> + struct nouveau_device *device = nv_device(&priv->base);
>> struct nouveau_gpuobj *mem;
>> struct nouveau_vm *vm;
>> + resource_size_t bar_len;
>> int ret;
>>
>> - ret = nouveau_bar_create(parent, engine, oclass, &priv);
>> - *pobject = nv_object(priv);
>> - if (ret)
>> - return ret;
>> -
>> - /* BAR3 */
>> ret = nouveau_gpuobj_new(nv_object(priv), NULL, 0x1000, 0, 0,
>> - &priv->bar[0].mem);
>> - mem = priv->bar[0].mem;
>> + &priv->bar[nr].mem);
>> + mem = priv->bar[nr].mem;
>> if (ret)
>> return ret;
>>
>> ret = nouveau_gpuobj_new(nv_object(priv), NULL, 0x8000, 0, 0,
>> - &priv->bar[0].pgd);
>> + &priv->bar[nr].pgd);
>> if (ret)
>> return ret;
>>
>> - ret = nouveau_vm_new(device, 0, nv_device_resource_len(device, 3), 0, &vm);
>> + bar_len = nv_device_resource_len(device, bar);
>> +
>> + ret = nouveau_vm_new(device, 0, bar_len, 0, &vm);
>> if (ret)
>> return ret;
>>
>> atomic_inc(&vm->engref[NVDEV_SUBDEV_BAR]);
>>
>> - ret = nouveau_gpuobj_new(nv_object(priv), NULL,
>> - (nv_device_resource_len(device, 3) >> 12) * 8,
>> - 0x1000, NVOBJ_FLAG_ZERO_ALLOC,
>> - &vm->pgt[0].obj[0]);
>> - vm->pgt[0].refcount[0] = 1;
>> - if (ret)
>> - return ret;
>> + /*
>> + * Bootstrap page table lookup.
>> + */
>> + if (bar == 3) {
>> + ret = nouveau_gpuobj_new(nv_object(priv), NULL,
>> + (bar_len >> 12) * 8, 0x1000,
>> + NVOBJ_FLAG_ZERO_ALLOC,
>> + &vm->pgt[0].obj[0]);
>> + vm->pgt[0].refcount[0] = 1;
>> + if (ret)
>> + return ret;
>> + }
>>
>> - ret = nouveau_vm_ref(vm, &priv->bar[0].vm, priv->bar[0].pgd);
>> + ret = nouveau_vm_ref(vm, &priv->bar[nr].vm, priv->bar[nr].pgd);
>> nouveau_vm_ref(NULL, &vm, NULL);
>> if (ret)
>> return ret;
>>
>> - nv_wo32(mem, 0x0200, lower_32_bits(priv->bar[0].pgd->addr));
>> - nv_wo32(mem, 0x0204, upper_32_bits(priv->bar[0].pgd->addr));
>> - nv_wo32(mem, 0x0208, lower_32_bits(nv_device_resource_len(device, 3) - 1));
>> - nv_wo32(mem, 0x020c, upper_32_bits(nv_device_resource_len(device, 3) - 1));
>> + nv_wo32(mem, 0x0200, lower_32_bits(priv->bar[nr].pgd->addr));
>> + nv_wo32(mem, 0x0204, upper_32_bits(priv->bar[nr].pgd->addr));
>> + nv_wo32(mem, 0x0208, lower_32_bits(bar_len - 1));
>> + nv_wo32(mem, 0x020c, upper_32_bits(bar_len - 1));
>>
>> - /* BAR1 */
>> - ret = nouveau_gpuobj_new(nv_object(priv), NULL, 0x1000, 0, 0,
>> - &priv->bar[1].mem);
>> - mem = priv->bar[1].mem;
>> - if (ret)
>> - return ret;
>> + return 0;
>> +}
>>
>> - ret = nouveau_gpuobj_new(nv_object(priv), NULL, 0x8000, 0, 0,
>> - &priv->bar[1].pgd);
>> - if (ret)
>> - return ret;
>> +static int
>> +nvc0_bar_ctor(struct nouveau_object *parent, struct nouveau_object *engine,
>> + struct nouveau_oclass *oclass, void *data, u32 size,
>> + struct nouveau_object **pobject)
>> +{
>> + struct nouveau_device *device = nv_device(parent);
>> + struct nvc0_bar_priv *priv;
>> + bool has_bar3 = nv_device_resource_len(device, 3) != 0;
>> + int ret;
>>
>> - ret = nouveau_vm_new(device, 0, nv_device_resource_len(device, 1), 0, &vm);
>> + ret = nouveau_bar_create(parent, engine, oclass, &priv);
>> + *pobject = nv_object(priv);
>> if (ret)
>> return ret;
>>
>> - atomic_inc(&vm->engref[NVDEV_SUBDEV_BAR]);
>> + /* BAR3 */
>> + if (has_bar3) {
>> + ret = nvc0_bar_init_vm(priv, 0, 3);
>> + if (ret)
>> + return ret;
>> + priv->base.alloc = nouveau_bar_alloc;
>> + priv->base.kmap = nvc0_bar_kmap;
>> + }
>>
>> - ret = nouveau_vm_ref(vm, &priv->bar[1].vm, priv->bar[1].pgd);
>> - nouveau_vm_ref(NULL, &vm, NULL);
>> + /* BAR1 */
>> + ret = nvc0_bar_init_vm(priv, 1, 1);
>> if (ret)
>> return ret;
>>
>> - nv_wo32(mem, 0x0200, lower_32_bits(priv->bar[1].pgd->addr));
>> - nv_wo32(mem, 0x0204, upper_32_bits(priv->bar[1].pgd->addr));
>> - nv_wo32(mem, 0x0208, lower_32_bits(nv_device_resource_len(device, 1) - 1));
>> - nv_wo32(mem, 0x020c, upper_32_bits(nv_device_resource_len(device, 1) - 1));
>> -
>> - priv->base.alloc = nouveau_bar_alloc;
>> - priv->base.kmap = nvc0_bar_kmap;
>> priv->base.umap = nvc0_bar_umap;
>> priv->base.unmap = nvc0_bar_unmap;
>> priv->base.flush = nv84_bar_flush;
>> @@ -176,12 +177,16 @@ nvc0_bar_dtor(struct nouveau_object *object)
>> nouveau_gpuobj_ref(NULL, &priv->bar[1].pgd);
>> nouveau_gpuobj_ref(NULL, &priv->bar[1].mem);
>>
>> - if (priv->bar[0].vm) {
>> - nouveau_gpuobj_ref(NULL, &priv->bar[0].vm->pgt[0].obj[0]);
>> - nouveau_vm_ref(NULL, &priv->bar[0].vm, priv->bar[0].pgd);
>> + if (priv->bar[0].mem) {
>> + if (priv->bar[0].vm) {
>> + nouveau_gpuobj_ref(NULL,
>> + &priv->bar[0].vm->pgt[0].obj[0]);
>> + nouveau_vm_ref(NULL, &priv->bar[0].vm,
>> + priv->bar[0].pgd);
>> + }
>> + nouveau_gpuobj_ref(NULL, &priv->bar[0].pgd);
>> + nouveau_gpuobj_ref(NULL, &priv->bar[0].mem);
>> }
>> - nouveau_gpuobj_ref(NULL, &priv->bar[0].pgd);
>> - nouveau_gpuobj_ref(NULL, &priv->bar[0].mem);
> Did the conditional on priv->bar[0].mem fix anything here? The ref()
> functions called are designed to handle the NULL pointers already.
You're right, this test is not needed at all. Thanks.
next prev parent reply other threads:[~2014-02-04 8:32 UTC|newest]
Thread overview: 40+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-02-01 3:16 [RFC 00/16] drm/nouveau: initial support for GK20A (Tegra K1) Alexandre Courbot
2014-02-01 3:16 ` [RFC 01/16] drm/nouveau: handle -EACCES runtime PM return code Alexandre Courbot
2014-02-01 3:16 ` [RFC 02/16] drm/nouveau: basic support for platform devices Alexandre Courbot
2014-02-01 3:16 ` [RFC 03/16] drm/nouveau: add platform device probing function Alexandre Courbot
2014-02-01 3:16 ` [RFC 04/16] drm/nouveau/fifo: support platform devices Alexandre Courbot
2014-02-01 3:16 ` [RFC 05/16] drm/nouveau/bar: " Alexandre Courbot
2014-02-01 3:16 ` [RFC 06/16] drm/nouveau/bar: only ioremap BAR3 if it exists Alexandre Courbot
2014-02-01 3:16 ` [RFC 07/16] drm/nouveau/bar/nvc0: support chips without BAR3 Alexandre Courbot
2014-02-04 3:54 ` Ben Skeggs
2014-02-04 8:31 ` Alexandre Courbot [this message]
2014-02-01 3:16 ` [RFC 08/16] drm/nouveau/mc: support platform devices Alexandre Courbot
2014-02-01 3:16 ` [RFC 09/16] drm/nouveau/fb: " Alexandre Courbot
2014-02-01 3:16 ` [RFC 10/16] drm/nouveau/timer: skip calibration on GK20A Alexandre Courbot
2014-02-04 3:55 ` Ben Skeggs
2014-02-04 8:39 ` Alexandre Courbot
2014-02-05 20:27 ` Stephen Warren
2014-02-01 3:16 ` [RFC 11/16] drm/nouveau/fifo: allocate usermem as needed Alexandre Courbot
2014-02-01 3:16 ` [RFC 12/16] drm/nouveau/fifo: add GK20A support Alexandre Courbot
2014-02-04 9:15 ` Daniel Vetter
2014-02-05 1:21 ` Alexandre Courbot
2014-02-01 3:16 ` [RFC 13/16] drm/nouveau/ibus: " Alexandre Courbot
2014-02-02 6:35 ` Ilia Mirkin
2014-02-02 9:38 ` Alexandre Courbot
2014-02-01 3:16 ` [RFC 14/16] drm/nouveau/fb: " Alexandre Courbot
2014-02-01 13:40 ` Lucas Stach
2014-02-01 23:28 ` Ilia Mirkin
2014-02-01 23:58 ` Lucas Stach
2014-02-02 13:43 ` Alexandre Courbot
2014-02-07 14:19 ` Alexandre Courbot
2014-02-01 3:16 ` [RFC 15/16] drm/nouveau: support GK20A in nouveau_accel_init() Alexandre Courbot
2014-02-01 3:16 ` [RFC 16/16] drm/nouveau: support for probing GK20A Alexandre Courbot
2014-02-02 19:10 ` [RFC 00/16] drm/nouveau: initial support for GK20A (Tegra K1) Ilia Mirkin
2014-02-03 2:44 ` Alexandre Courbot
2014-02-03 3:14 ` Ilia Mirkin
2014-02-03 3:41 ` Ben Skeggs
2014-02-03 11:25 ` David Herrmann
2014-02-04 2:47 ` Alexandre Courbot
2014-02-03 17:33 ` Daniel Vetter
2014-02-04 3:53 ` Ben Skeggs
2014-02-04 8:44 ` Alexandre Courbot
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=52F0A574.1070601@nvidia.com \
--to=acourbot@nvidia.com \
--cc=KAdams@nvidia.com \
--cc=bskeggs@redhat.com \
--cc=dri-devel@lists.freedesktop.org \
--cc=ebrower@nvidia.com \
--cc=gnurou@gmail.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-tegra@vger.kernel.org \
--cc=nouveau@lists.freedesktop.org \
--cc=skeggsb@gmail.com \
--cc=swarren@wwwdotorg.org \
--cc=tbergstrom@nvidia.com \
/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;
as well as URLs for NNTP newsgroup(s).