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.

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

ci: fix broken apt install #31

Merged
merged 1 commit into from
Jan 12, 2024
Merged

ci: fix broken apt install #31

merged 1 commit into from
Jan 12, 2024

Conversation

oreparaz
Copy link
Owner

need an apt-get update before doing anything

@oreparaz oreparaz merged commit 4d7fcc3 into master Jan 12, 2024
10 checks passed
oreparaz added a commit to itzmeanjan/dudect that referenced this pull request Jan 12, 2024
oreparaz added a commit that referenced this pull request Jan 12, 2024
* fix: issue `mfence` instruction before issuing `rdtsc`

This should make all memory load/ store operations happening before RDTSC (in program order) globally visible. Read https://stackoverflow.com/a/12634857.

Signed-off-by: Anjan Roy <hello@itzmeanjan.in>

* udt: update example showing constant-timeness of `memcmp` varies based on different optimization settings

Signed-off-by: Anjan Roy <hello@itzmeanjan.in>

* chg: get rid of redundant assertion

Comparison of unsigned expression in `>= 0` is always **true**.

Signed-off-by: Anjan Roy <hello@itzmeanjan.in>

* ci: fix broken apt install (#31)

* readme: add credits for PR 30 (Anjan Roy)

* add link to github issue

---------

Signed-off-by: Anjan Roy <hello@itzmeanjan.in>
Co-authored-by: Oscar Reparaz <oreparaz@users.noreply.github.com>
@oreparaz oreparaz deleted the oscar/ci branch January 15, 2024 18:43
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.

1 participant