Page MenuHomeFreeBSD

markm (Mark R V Murray)
User

Projects

User Details

User Since
May 14 2014, 8:42 PM (265 w, 4 d)

Recent Activity

Sun, Jun 9

markm accepted D20313: Fortuna: allow increased concurrency.
Sun, Jun 9, 9:18 AM

Fri, Jun 7

markm accepted D20312: random(4): Generalize algorithm-independent APIs.
Fri, Jun 7, 6:22 PM
markm accepted D20392: random(4): Add regression tests for uint128 implementation, Chacha CTR.

None job, thanks!

Fri, Jun 7, 6:21 PM

Mon, Jun 3

markm accepted D20419: virtio_random(4): Fix random(4) integration.
Mon, Jun 3, 5:29 PM

Fri, May 31

markm accepted D20466: random(4): Fix RANDOM_LOADABLE build.
Fri, May 31, 7:44 AM

Thu, May 30

markm accepted D20466: random(4): Fix RANDOM_LOADABLE build.
Thu, May 30, 7:24 AM

Tue, May 28

markm accepted D20419: virtio_random(4): Fix random(4) integration.

Good catch!

Tue, May 28, 7:18 AM

Thu, May 23

markm added a comment to D20358: save-entropy(8), rc.d/random: Set nodump flag.

Good move!

Thu, May 23, 8:54 AM

Sun, May 19

markm accepted D20313: Fortuna: allow increased concurrency.

This is a step in a direction that I've been wanting to doo for quite some time; pin multiple output generators to processors. There is more, but I need to write it in detail first.

Sun, May 19, 10:11 AM
markm added inline comments to D20312: random(4): Generalize algorithm-independent APIs.
Sun, May 19, 10:04 AM
markm requested changes to D20312: random(4): Generalize algorithm-independent APIs.
Sun, May 19, 9:47 AM

May 12 2019

markm accepted D19928: random(4): More thoroughly attempt to ensure seeding during priming.

Looks OK to me!

May 12 2019, 9:10 AM

May 8 2019

markm accepted D20195: random: Don't complain noisily when an entropy source is slow.

I'm OK with this.

May 8 2019, 7:26 AM

Apr 18 2019

markm accepted D19944: random(4): Restore availability tradeoff prior to r346250.

LGTM.

Apr 18 2019, 7:34 AM

Apr 11 2019

markm added a comment to D19744: random(4): Block read_random(9) on initial seeding.

Against my better judgement, I kept the !DEV_RANDOM case as some folks insisted on being able to use preferred tools (ssh) even on insecure embedded hardware. I'm happy to se you fix it, if it means I don't get the flak ]:->

Apr 11 2019, 9:09 AM

Apr 6 2019

markm committed rS345981: Backport fixes from FreeBSD-12 to help the random(4) device thread.
Backport fixes from FreeBSD-12 to help the random(4) device thread
Apr 6 2019, 9:00 AM

Apr 5 2019

markm accepted D19744: random(4): Block read_random(9) on initial seeding.

I'm happy with this, in principle. I defer on the kernel innards, but the general engineering looks sound.

Apr 5 2019, 11:05 AM

Mar 29 2019

markm accepted D19742: random(4): Attempt to persist entropy promptly.

Accept with delphi's changes.

Mar 29 2019, 9:31 AM

Mar 22 2019

markm accepted D19620: Add an option to use TPM as entropy source.

OK. I like these diffs even better.

Mar 22 2019, 4:55 PM

Mar 21 2019

markm added a comment to D19620: Add an option to use TPM as entropy source.

You need the security officer's say-so. I have a personal wavier when I do such commits myself (as I wrote the main random device), but this is non-transferrable, as far as I know.

Mar 21 2019, 3:00 PM
markm accepted D19620: Add an option to use TPM as entropy source.

LGTM.

Mar 21 2019, 1:00 PM

Mar 7 2019

markm accepted D19475: Fortuna: Add Chacha20 as an alternative stream cipher.

Very good!

Mar 7 2019, 12:06 PM

Mar 6 2019

markm added inline comments to D19475: Fortuna: Add Chacha20 as an alternative stream cipher.
Mar 6 2019, 9:14 AM
markm added a comment to D19475: Fortuna: Add Chacha20 as an alternative stream cipher.

Nice work! Out for interest, why make the output generator selectable? Why not switch completely to Chacha20?

Mar 6 2019, 9:08 AM

Mar 1 2019

markm abandoned D10440: Change arc4random(9) over to using 3-BSD licensed Chacha20..

Overcome By Events.

Mar 1 2019, 10:03 AM
markm accepted D19411: Fortuna: push CTR-mode loop down into randomdev hash.h interface.

I like this!

Mar 1 2019, 10:01 AM
markm accepted D19410: Embedded chacha: Add 0-bit iv + 128-bit counter build mode.

OK.

Mar 1 2019, 9:57 AM
markm accepted D19409: fortuna: Deduplicate kernel vs user includes.

Looks good to me.

Mar 1 2019, 9:55 AM

Nov 26 2018

markm committed rS341002: Add dependency to allow if_muge device to be only ethernet device in stripped….
Add dependency to allow if_muge device to be only ethernet device in stripped…
Nov 26 2018, 10:46 PM

Sep 20 2018

markm accepted D17252: random(4): Match enabled sources mask to build options.
Sep 20 2018, 7:31 AM

Sep 14 2018

markm accepted D17047: Fortuna: Add failpoints to simulate initial seeding conditions.
Sep 14 2018, 7:46 AM

Sep 7 2018

markm accepted D17049: random(4): Squash non-error timeout codes from tsleep(9).
Sep 7 2018, 6:08 PM
markm requested changes to D17047: Fortuna: Add failpoints to simulate initial seeding conditions.
Sep 7 2018, 6:07 PM

Sep 6 2018

markm accepted D16986: fortuna: Drop global lock to zero stack variables.
Sep 6 2018, 12:24 PM

Sep 2 2018

markm committed rD52196: Update to the key that I use..
Update to the key that I use.
Sep 2 2018, 10:09 AM

Sep 1 2018

markm added inline comments to D16986: fortuna: Drop global lock to zero stack variables.
Sep 1 2018, 7:19 PM
markm added inline comments to D16986: fortuna: Drop global lock to zero stack variables.
Sep 1 2018, 7:17 PM
markm accepted D16985: Fortuna: fix a correctness issue in reseed (fortuna_pre_read).
Sep 1 2018, 10:13 AM
markm accepted D16984: Fortuna: Fix a race to prevent reseed spamming.
Sep 1 2018, 10:10 AM
markm accepted D16983: random(4): Correct a bare zero to the appropriate enum.
Sep 1 2018, 10:07 AM
markm accepted D16982: random(4): Translate a comment requirement into a compile-time invariant.
Sep 1 2018, 10:05 AM

Aug 29 2018

markm accepted D16934: Fortuna: Clean up reseeding key material to closer match FS&K.
Aug 29 2018, 7:36 PM
markm accepted D16937: Fortuna: trivial static variable cleanup.

If it doesn't crash anything, then fine! I had stack problems at some point.

Aug 29 2018, 7:29 PM
markm accepted D16936: Fortuna: Add trivial assert to match FS&K definition.
Aug 29 2018, 7:28 PM
markm added inline comments to D16934: Fortuna: Clean up reseeding key material to closer match FS&K.
Aug 29 2018, 7:26 PM

Aug 26 2018

markm committed rS338324: Remove the Yarrow PRNG algorithm option in accordance with due notice.
Remove the Yarrow PRNG algorithm option in accordance with due notice
Aug 26 2018, 1:07 PM
markm closed D16898: Remove the Yarrow PRNG algorithm option in accordance with random(4)..

Commit r338324 closes this.

Aug 26 2018, 1:03 PM

Aug 25 2018

markm added inline comments to D16898: Remove the Yarrow PRNG algorithm option in accordance with random(4)..
Aug 25 2018, 11:02 PM
markm added inline comments to D16898: Remove the Yarrow PRNG algorithm option in accordance with random(4)..
Aug 25 2018, 10:42 PM
markm updated the diff for D16898: Remove the Yarrow PRNG algorithm option in accordance with random(4)..

Sigh. I think I have all cem@'s review comments done now.

Aug 25 2018, 10:39 PM
markm updated the diff for D16898: Remove the Yarrow PRNG algorithm option in accordance with random(4)..

Rebase to top of HEAD.

Aug 25 2018, 10:20 PM
markm added a comment to D16898: Remove the Yarrow PRNG algorithm option in accordance with random(4)..

Address cmem@'s review comments.

Aug 25 2018, 10:09 PM
markm updated the diff for D16898: Remove the Yarrow PRNG algorithm option in accordance with random(4)..

Adress some review concerns from cem@, and a few of my own while I'm here.

Aug 25 2018, 10:05 PM
markm added inline comments to D16898: Remove the Yarrow PRNG algorithm option in accordance with random(4)..
Aug 25 2018, 9:56 PM
markm added inline comments to D16898: Remove the Yarrow PRNG algorithm option in accordance with random(4)..
Aug 25 2018, 8:36 PM
markm updated the diff for D16898: Remove the Yarrow PRNG algorithm option in accordance with random(4)..

Fix documentation nit.

Aug 25 2018, 8:35 PM
markm updated the diff for D16898: Remove the Yarrow PRNG algorithm option in accordance with random(4)..

Add the "differential update" URI to the commit message.
Correct sys/conf/NOTES comment.

Aug 25 2018, 11:29 AM
markm added inline comments to rS338239: Add a special note to UPDATING for the devmatch stuff. While tested,.
Aug 25 2018, 8:35 AM
markm added inline comments to D16898: Remove the Yarrow PRNG algorithm option in accordance with random(4)..
Aug 25 2018, 8:33 AM
markm updated the summary of D16898: Remove the Yarrow PRNG algorithm option in accordance with random(4)..
Aug 25 2018, 8:30 AM
markm created D16898: Remove the Yarrow PRNG algorithm option in accordance with random(4)..
Aug 25 2018, 8:23 AM

Aug 24 2018

markm added a comment to D16873: Limit the harvest rate of "fast" entropy for random(4) so as not to overload the system..
In D16873#359829, @jmg wrote:

@delphij this is my comment copied over from https://reviews.freebsd.org/D16866?id=47165 that was unaddressed.

Aug 24 2018, 7:15 PM
markm closed D16873: Limit the harvest rate of "fast" entropy for random(4) so as not to overload the system..
Aug 24 2018, 2:54 PM
markm committed rS338293: Limit the amount of "fast" entropy. We don't need nearly as much.
Limit the amount of "fast" entropy. We don't need nearly as much
Aug 24 2018, 2:54 PM
markm committed rS338292: Fix braino of mine where the reseeds would happen far too often,.
Fix braino of mine where the reseeds would happen far too often,
Aug 24 2018, 2:54 PM
markm closed D16872: Fix excessive reseed rate in random(4)..
Aug 24 2018, 2:54 PM

Aug 23 2018

markm added a comment to D16872: Fix excessive reseed rate in random(4)..

Thanks folks! Tinderbox build going, then I'll commit.

Aug 23 2018, 7:19 PM
markm added a comment to D16873: Limit the harvest rate of "fast" entropy for random(4) so as not to overload the system..

Thanks folks! Tinderbox build going, then I'll commit.

Aug 23 2018, 7:19 PM
markm added inline comments to D16873: Limit the harvest rate of "fast" entropy for random(4) so as not to overload the system..
Aug 23 2018, 5:31 PM
markm updated the diff for D16873: Limit the harvest rate of "fast" entropy for random(4) so as not to overload the system..

Address review commnts from delphij.

Aug 23 2018, 5:20 PM
markm added inline comments to D16873: Limit the harvest rate of "fast" entropy for random(4) so as not to overload the system..
Aug 23 2018, 5:09 PM
markm added a comment to D16872: Fix excessive reseed rate in random(4)..

Apologies to delphij for abandoning the previous identical-looking commit. I was making a pig's ear of the archanist work.

Aug 23 2018, 5:00 PM
markm added a reviewer for D16872: Fix excessive reseed rate in random(4).: delphij.
Aug 23 2018, 4:59 PM
markm added a comment to D16873: Limit the harvest rate of "fast" entropy for random(4) so as not to overload the system..

This improves things drastically from what they were. More detailed work will follow.

Aug 23 2018, 4:56 PM
markm created D16873: Limit the harvest rate of "fast" entropy for random(4) so as not to overload the system..
Aug 23 2018, 4:52 PM
markm created D16872: Fix excessive reseed rate in random(4)..
Aug 23 2018, 4:46 PM
markm abandoned D16866: Fix excessive reseeding in random(4)..
Aug 23 2018, 4:39 PM
markm retitled D16866: Fix excessive reseeding in random(4). from Fix performance-damaging issues in random(4) to Fix excessive reseeding in random(4)..
Aug 23 2018, 4:32 PM
markm updated the diff for D16866: Fix excessive reseeding in random(4)..

Split into two separate commits as per review request.

Aug 23 2018, 4:27 PM
markm updated the diff for D16866: Fix excessive reseeding in random(4)..

Add the differential revision to the commit message.

Aug 23 2018, 1:21 PM
markm created D16866: Fix excessive reseeding in random(4)..
Aug 23 2018, 1:16 PM

Aug 19 2018

markm accepted D16760: Use Chacha20 for userland arc4random() and friends.

There were a couple of changes which are largely cosmetic to do with #if logic. Change if you want, otherwise LGTM.

Aug 19 2018, 11:24 AM
markm added a comment to D16760: Use Chacha20 for userland arc4random() and friends.

My "LGTM" assumes requested changes are taken.

Aug 19 2018, 9:20 AM
markm accepted D16760: Use Chacha20 for userland arc4random() and friends.

LGTM

Aug 19 2018, 9:18 AM

Aug 2 2018

markm accepted D16552: random: Add PowerPC 'darn' instruction entropy source.

Looks good to me!

Aug 2 2018, 11:22 AM

Apr 19 2018

markm committed rS332791: Remove empty dir no longer required..
Remove empty dir no longer required.
Apr 19 2018, 8:32 PM

Nov 17 2017

markm accepted D12132: Avoid spinning in random_harvest_queue.

I'm happy with this.

Nov 17 2017, 5:57 PM

Sep 8 2017

markm added a comment to D10440: Change arc4random(9) over to using 3-BSD licensed Chacha20..

(from hospital bed)

Sep 8 2017, 10:00 AM
markm added a comment to D12132: Avoid spinning in random_harvest_queue.

(From hospital bed)

Sep 8 2017, 9:47 AM

Sep 4 2017

markm added a comment to D12132: Avoid spinning in random_harvest_queue.

I will be able to help in a week or two when I get out of hospital - I'm about to have an operation.

Sep 4 2017, 1:16 PM

Aug 28 2017

markm added a comment to D12132: Avoid spinning in random_harvest_queue.

That might be an idea. The actual PPS rate could be unable, as optimal choices may not be universal. I'm leery of turning off harvesting altogether as that would be a big POLA violation, but lessening the impact makes sense.

Aug 28 2017, 7:30 PM
markm added a comment to D12132: Avoid spinning in random_harvest_queue.

Do you mean not harvesting the packet contents?

Aug 28 2017, 7:26 PM

Aug 27 2017

markm added a comment to D12132: Avoid spinning in random_harvest_queue.

I'm having a bit of a problem parsing this; could I please ask you to rephrase?

Aug 27 2017, 3:48 PM
markm added a comment to D12132: Avoid spinning in random_harvest_queue.

I see an opportunity for documentation improvement here. :-)

Aug 27 2017, 9:43 AM

Jul 29 2017

markm accepted D11769: Remove Redundant Include Directories.
Jul 29 2017, 11:08 AM

Jul 28 2017

markm added a comment to D11769: Remove Redundant Include Directories.

Does Kerberised telnet still work after this? ISTR (many years ago) that a successful compile does not necessarily mean a working binary due to similarly named libraries.

Jul 28 2017, 7:36 AM

Apr 20 2017

markm created D10440: Change arc4random(9) over to using 3-BSD licensed Chacha20..
Apr 20 2017, 7:22 PM

Apr 16 2017

markm committed rS317015: Replace the RC4 algorithm for generating in-kernel secure random.
Replace the RC4 algorithm for generating in-kernel secure random
Apr 16 2017, 9:11 AM
markm closed D10048: Replace the kernel RC4 with Chacha20. by committing rS317015: Replace the RC4 algorithm for generating in-kernel secure random.
Apr 16 2017, 9:11 AM
markm added a comment to D10048: Replace the kernel RC4 with Chacha20..
In D10048#215609, @des wrote:

Please allow me some time to commit my Chacha20 implementation first so we can use that instead of the legally dubious version which is included in this patch. I hit a snag that I haven't had time to debug, but I'm hoping to have it done by Tuesday.

Apr 16 2017, 8:55 AM