Re: [OS-BUILD PATCH 0/2] random: Add hook to override device reads and getrandom(2)

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



From: Vladis Dronov on gitlab.com
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2757#note_1613416986

hi, Herbert, thank you for the patchset. i'm looking at "changes that were
made after the kernel-ark revert". especially, at
https://gitlab.com/redhat/centos-stream/src/kernel/centos-stream-9/-/merge_req
uests/1360/diffs?commit_id=66d567ab6293dc284952e8d145820d257f3987b9. i see the
first hunk is in your patchset, but the second one is not.

i see `i = min_t(size_t, iov_iter_count(iter), sizeof(tmp)); err =
crypto_rng_get_bytes(rng, tmp, i);` statements in your patch, same as in the
older C9S `66d567ab` above. but not a
`crypto_tfm_set_flags(CRYPTO_TFM_REQ_NEED_RESEED)` call, which is under a
comment "The above call to crypto_rng_get_bytes will reset the
CRYPTO_TFM_REQ_NEED_RESEED flag". it is not in your patchset, is this
intentional? can you please have a look?
_______________________________________________
kernel mailing list -- kernel@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to kernel-leave@xxxxxxxxxxxxxxxxxxxxxxx
Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: https://lists.fedoraproject.org/archives/list/kernel@xxxxxxxxxxxxxxxxxxxxxxx
Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue




[Index of Archives]     [Fedora General Discussion]     [Older Fedora Users Archive]     [Fedora Advisory Board]     [Fedora Security]     [Fedora Devel Java]     [Fedora Legacy]     [Fedora Desktop]     [ATA RAID]     [Fedora Marketing]     [Fedora Mentors]     [Fedora Package Announce]     [Fedora Package Review]     [Fedora Music]     [Fedora Packaging]     [Centos]     [Fedora SELinux]     [Coolkey]     [Yum Users]     [Tux]     [Yosemite News]     [KDE Users]     [Fedora Art]     [Fedora Docs]     [USB]     [Asterisk PBX]

  Powered by Linux