Re: Why is nouveau using a separate swap storage?

2020-09-17 Thread Christian König

Am 17.09.20 um 12:51 schrieb Ben Skeggs:

On Thu, 17 Sep 2020 at 20:06, Christian König
 wrote:

Hi guys,

just another TTM feature which is only used by nouveau.

We have this bo->bo.persistent_swap_storage pointer which is only set by
nouveau to the GEM filp and used when a BO is swapped to a shmem file.

As far as I can see this doesn't make any sense at all? What is the
background here.

No specific reason I can remember, likely cargo-cult from very very
early versions of TTM/GEM stuff.


Ok, going to send something out to drop this then.

Thanks,
Christian.



Ben.


Thanks,
Christian.
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Flists.freedesktop.org%2Fmailman%2Flistinfo%2Fdri-develdata=02%7C01%7Cchristian.koenig%40amd.com%7C70ce7559f13c4bf181fb08d85af7b85b%7C3dd8961fe4884e608e11a82d994e183d%7C0%7C0%7C637359367286020632sdata=ncJ2%2FkCY7DuGb119M%2BR%2BLtSUA1qjEK5nhGVjKjEd6jQ%3Dreserved=0


___
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel


Re: Why is nouveau using a separate swap storage?

2020-09-17 Thread Ben Skeggs
On Thu, 17 Sep 2020 at 20:06, Christian König
 wrote:
>
> Hi guys,
>
> just another TTM feature which is only used by nouveau.
>
> We have this bo->bo.persistent_swap_storage pointer which is only set by
> nouveau to the GEM filp and used when a BO is swapped to a shmem file.
>
> As far as I can see this doesn't make any sense at all? What is the
> background here.
No specific reason I can remember, likely cargo-cult from very very
early versions of TTM/GEM stuff.

Ben.

>
> Thanks,
> Christian.
> ___
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel


Why is nouveau using a separate swap storage?

2020-09-17 Thread Christian König

Hi guys,

just another TTM feature which is only used by nouveau.

We have this bo->bo.persistent_swap_storage pointer which is only set by 
nouveau to the GEM filp and used when a BO is swapped to a shmem file.


As far as I can see this doesn't make any sense at all? What is the 
background here.


Thanks,
Christian.
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel