From: Henri Verbeet Subject: Re: [PATCH v3 2/7] wined3d: Add functions to support device local and non-local memory info management. Message-Id: Date: Mon, 2 Dec 2019 14:38:26 +0330 In-Reply-To: <858ad184094324ac1416f82e8fb4b44f@codeweavers.com> References: <20191125141101.85701-1-cmccarthy@codeweavers.com> <20191125141101.85701-2-cmccarthy@codeweavers.com> <858ad184094324ac1416f82e8fb4b44f@codeweavers.com> On Mon, 2 Dec 2019 at 12:14, Conor McCarthy wrote: > November 27, 2019 6:01 AM, "Henri Verbeet" wrote: > > On Mon, 25 Nov 2019 at 17:43, Conor McCarthy wrote: > > Is it ever necessary to override the total amount of video memory? > If the totals come from adapter_vk they will be correct, but adapter_gl won't have the non-local total. Should a zero total be overridden by a non-zero one? Also should totals be passed in a separate call? That could be done using a more generic messaging callback function. > The GL backend won't be able to query the amount of non-local video memory (unless we add an extension, perhaps), but ultimately that's an implementation detail, and I'm not sure how much it really matters. Chances are we'll ultimately get that information from winex11. > > If we have wined3d_get_adapter_memory_info(), the memory information > > in struct wined3d_adapter_identifier becomes redundant and could be > > removed. > > Should vram_bytes and sysmem_bytes stay in wined3d_driver_info though? > Well, they need to be stored somewhere, but that doesn't necessarily need to be the wined3d_driver_info structure. If you think a separate structure for adapter memory info makes sense, that's fine.