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

Introduce NV_KZALLOC macro #330

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

trixirt
Copy link

@trixirt trixirt commented Jul 22, 2022

The pattern of
p = kmalloc(...);
memset(p, 0, ...);

Can be reduced to
p = kzalloc(...)

So introduce and use NV_KZALLOC, a macro that wraps kzalloc.

Signed-off-by: Tom Rix trix@redhat

The pattern of
  p = kmalloc(...);
  memset(p, 0, ...);

Can be reduced to
  p = kzalloc(...)

So introduce and use NV_KZALLOC, a macro that wraps kzalloc.

Signed-off-by: Tom Rix <trix@redhat>
@mtijanic mtijanic self-assigned this Aug 2, 2022
@mtijanic
Copy link
Collaborator

mtijanic commented Aug 3, 2022

Thanks Tom, I've merged this into our internal development branch. Due to complex release schedules, it's easiest to just let it naturally bubble up in one of the next major releases (52x.xx)

This means that if I merge the PR now into main on github, the changes would be clobbered with the next 515.xx point release comes. So I'm tagging as implemented, and we'll merge once we know the very next release will pick it up.

Please let me know if there's a particular need to backport into 515. And thanks again for the contribution!

@mtijanic mtijanic added the Implemented Fixed, in test prior to release integration label Aug 3, 2022
@PAR2020 PAR2020 added the NV-Triaged An NVBug has been created for dev to investigate label Aug 19, 2022
@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


Tom Rix seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Implemented Fixed, in test prior to release integration NV-Triaged An NVBug has been created for dev to investigate
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants