Skip to content

Conversation

@opcm
Copy link
Contributor

@opcm opcm commented Feb 2, 2021

No description provided.

Change-Id: I13523637f7e012d5495c0e6de3871348fa8086d7
potential fix for #252

Change-Id: Ie038acaa5d1d8e34edc4bbe72605aea73d38c278
This is to protect against very unlikely occasional overflows when values
reach 2^^48 for extremelely long collections in the own programming logic
(not using perf API).

Change-Id: I6b60a17b1100ea5c0d5d9364a788064c96578efe
Change-Id: I39550c8845a9e22cf1e68f2c7f58cc12a81c1b2e
Change-Id: Ib9e22bcec570782573302f7c0c931b58ac215ac9
Change-Id: Ic5fd31fe1e7d6c7084a9afdcfc1d9b57cc5c0e6f
Change-Id: I29fc0e5f0bc76ee2f30d9449b9f1fef091bf70f2
Change-Id: Ibb6cf541fe5c2e490c1845cda3511bad029aba00
related to #248

Change-Id: I29f475bf3c6561e4a07fb6cc8ac43e14926a5422
Change-Id: I300dc36df036eefcb31cc1b624899bb5b3ff3471
this is slow but no other way found to make it
work with resctrl. rewind does not allow a new read.
Change-Id: I67ca07d1462cc1a770e85347b9dd57319275c728
Change-Id: I16ea25463edab0d47b122d14e8d42be107b91bf3
Change-Id: Ib8290fd7cdfd09112daad017d5aba9627e21b8ce
Change-Id: I6267b360751024d7c12d35e766bc940a681a38b7
Change-Id: I6e7614be7e7962f6eff0514d2e2c4069ec8149f2
Change-Id: If4789fdbe7e30625f14b7b62fb09aa26cca6ea27
Change-Id: If24208e76574fee14e6fb0cf6d1740255289e402
Change-Id: Ief40371294675a94ab8e68a4d36e00bbfb234c22
Change-Id: If19bdd6517e08390aa6f0c4e98bd182a016d88cc
The warning:
Value stored to 'atPos' is never read

Change-Id: I8224eac6e5eff915373057c09771cce1722c028b
rdementi and others added 27 commits January 1, 2021 14:13
Change-Id: I833dbe3030cfc96f0fbf8438315e98f0dd0b21af
Member variable ... is not initialized in the constructor.
Return value of function strerror() is not used
Member variable 'HTTPSServer::sslCTX_' is not initialized in the
constructor.
Change-Id: I2b5fda7f1ec80b6f9036b512f0d7c5db6338c208
Change-Id: I5b87637b83e8201281dc2b36dabcb94773ea5f65
Change-Id: I953afe4063fdbbed83272bf82e64b06ec7fe0f98
Change-Id: Ia5a1948dd1556687817118975e07312981472615
Change-Id: I848095e095af2773524798d94b64e9f4847396ca
Windows ci

See merge request pcm/opcm!114
Change-Id: I290f9cc5ce4c0cb8a2d97c1f50984aee0d897981
Change-Id: Ib91fbb3044e4146913bbb234665ba6e25fa746cc
Change-Id: I022c8e346a7df960f15322f6cbdb7b3b502960bd
Change-Id: Ic5be8f08fca941e2e9f0b7189b5da5a3ad94f4a0
addresses #263

Change-Id: I8837afa632eefe2910c94b05f40f2978ab74a6c0
Change-Id: Ib6db1edf1f4699173cf9f9c60e8a4bf365415622
Change-Id: I74d6dacfee695fc15bc660a3fae7aff7dfe4ded5
…ores

Change-Id: I2f03367d5c2dc0174d946e467a8da6188aaa5c6d
Adding CFLAGS as the base C options, and appending C++ specific
CXXFLAGS values to CFLAGS allows a common set of flags for both
C and C++ without duplication.

Rework the C example binaries to use $(CFLAGS) when building.

Signed-off-by: Harry van Haaren <harry.van.haaren@intel.com>
This commit fixes an issue where Linux encodes the NUMA ID of a
given CPU in bits above 12 of IA32_TSC_AUX[31:0].
The code here was mistakenly including bit 12 and above of the
returned value in the CPU integer value.

When running the c_example on e.g. NUMA 1 of dual-NUMA platform, bit 12
was set causing an OOB value to be passed to e.g. pcm_c_get_cycles(),
resulting in a segfault.

Fix issue by removing the NUMA bits from the CPU calculation.

Signed-off-by: Harry van Haaren <harry.van.haaren@intel.com>
This commit:
- fixes an unused-variable on "c" array
- Makes whitespace consistent to tabs
- Adds [c_example] prints to output to show beginners where what parts
  of the info logs are coming from.
c_example: fix segfault when run on NUMA1 and minor improvements

See merge request pcm/opcm!115
Change-Id: I42568afef265a6db0395358ca1bdf23d4835f05e
Change-Id: I40d17727192e04f6f8f4cfc819fa333d6a3ab581
Change-Id: If9be91c5a10653dc00c92bb841621d01be8d1b8a
@opcm opcm merged commit 0f5076b into master Feb 2, 2021
@opcm opcm deleted the push_202101 branch November 26, 2021 08:08
rdementi added a commit that referenced this pull request Oct 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants