Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

gmmlib build QA warnings #62

Closed
CaptainBloodz opened this issue Dec 9, 2019 · 2 comments
Closed

gmmlib build QA warnings #62

CaptainBloodz opened this issue Dec 9, 2019 · 2 comments

Comments

@CaptainBloodz
Copy link

Git head on gentoo gcc 9.2

  • QA Notice: Package triggers severe warnings which indicate that it
  •        may exhibit random runtime failures.
  • /var/tmp/portage/media-libs/gmmlib-9999/work/gmmlib-9999/Source/GmmLib/TranslationTable/GmmPageTableMgr.cpp:415:50: warning: converting to non-pointer type 'long unsigned int' from NULL [-Wconversion-null]
  • /var/tmp/portage/media-libs/gmmlib-9999/work/gmmlib-9999/Source/GmmLib/TranslationTable/GmmPageTableMgr.cpp:474:31: warning: NULL used in arithmetic [-Wpointer-arith]
@jbeich jbeich mentioned this issue Jan 1, 2020
Open
@edwarddavidbaker edwarddavidbaker mentioned this issue Feb 24, 2021
Closed
@edwarddavidbaker
Copy link

Duplicate of issue #61.

@johnmach
Copy link
Contributor

Fixed in 5647e87

VPG-SWE-Github pushed a commit that referenced this issue Dec 3, 2021
@johnmach
GMM Global Context removal (#62)
ffd91d0 
* Phase 1: As part of global and static code removal- GMM policy, plaformInfo and texture class updated to have pGmmGlobal contextas part of it

* Updating the GMM #define to make sure it uses parameterized GMMlib Context

* Replace pGmmGlobalContext with member variable with pGmmLibContext in all GMM cache policy files

* Replace pGmmGlobalContext with pGmmLibContext in all Platform Class related API

* Remove any remaining use of pGmmGlobalContext

* Phase 2: Update Macros to use pGmmLibContext (local) instead of global pGmmGlobalContext

* Replace pGmmGlobalContext with pGmmUMDLibContext/pGmmKMDLibContext in GMM_RESOURCE_INFO class and its member functions

* All KMD level pGmmGlobalContext global variable is removed and replaced with pGmmLibContext

* Phase 3: Add support for Multi-Adapter in GMM UMD Library and replace pGmmGloContext to pGmmLibContext

* Make GMM UMD DLL Initialize and Destroy APIs backward compatible for SingleAdapter use case

* Phase 4: Enable and add support for Multi-Adapter ULT for GMM UMD DLL in GMM ULT

* Update the Entry-Level Init and destroy APIs

* Phase 5: Remove the pGmmGlobalContext Global and its dependencies

* Initial cleanup of KMD code

* Updating variable data type and License Info
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@CaptainBloodz @edwarddavidbaker @johnmach