- User Since
- Jul 9 2015, 9:56 PM (227 w, 5 d)
Mon, Nov 18
Sat, Nov 16
Fri, Nov 15
It seems like ESNs are just AAD in integrity modes and maybe sometimes plaintext or sometimes AAD in other ipsec modes. I’m not sure why they’d be considered an algorithm? I think this will be easier to express in the rework branch so I’m glad to hear John plans to circulate it soon.
Thu, Nov 14
Can you provide more background of ESNs and motivation for including them as an ocf transformation? Thanks!
The majority of ocf drivers aren’t ready for this. Can you give me some context on what ESNs are and why they ought to be part of doing an OCF transform? Have you considered integrating these in @jhb’s OCF rework branch?
Wed, Nov 13
(The changes look good to me, aside from the question.)
Tue, Nov 12
Mon, Nov 11
So this fixes a single live && MROUTING && VIMAGE case, but leaves the !live && MROUTING && VIMAGE case broken, with the same misleading print, as well as any other !live && VIMAGE code. It seems like a better solution might be teaching it about VIMAGE.
Change looks functionally correct to me, my suggestions are all stylistic.
Sat, Nov 9
Thu, Nov 7
Any reason to avoid generating to a temporary file first (possibly in tmpfs) and installing with install -C (i.e., don't re-write or bump mtime if the contents did not change)? Slightly more friendly to low-endurance media /var devices.
With the caveat that I am unfamiliar with this part of the build, looks reasonable to me. (And I am supportive of the objective.)
Wed, Nov 6
Carrying around an openssh patch is far from ideal. Is there any chance it can be upstreamed?
FWIW, tinderbox is clean.
Some initial tentative feedback and CCing kib@. Caveat, I haven't yet consulted any official documentation nor have I looked at the adjacent code to understand what it is doing, so my comments may just be uninformed. But, provisionally, I don't understand the motivation for some of this changeset.
Tue, Nov 5
Sun, Nov 3
Fri, Nov 1
Is "not terrible" the bar now?
This is a weird and not obviously better change for FreeBSD. I assume it makes sense for some particular reason on CHERI, but CHERI isn't even a Tier 4 FreeBSD arch. Most of the changes are just obfuscating formatted prints rather than any actual change to casts.
Wed, Oct 30
Tue, Oct 29
Looks good to me, thanks!
I’m not sure it’s a good idea to default no-strip all tests. It likely fixes my test in particular, but my test could add STRIP= in isolation.
The new patch doesn't have context, so phabricator doesn't know how to show inter-diff and us casual readers cannot expand adjacent code in the web interface. Would you mind uploading a diff with full context (i.e., diff -U99999 ... or using the arc utility)? Thanks!
Mon, Oct 28
Sun, Oct 27
Sat, Oct 26
Thu, Oct 24
I didn't look too closely but this look a lot better, thanks. One question below.
Thanks, this is kinda weird but I like it better than before.
Wed, Oct 23
Fix a number of ENTRY()s without matched END()s.
Mon, Oct 21
Oct 19 2019
Looks good to me, thanks.
Oct 18 2019
Scott, thanks for picking this up! I'm really glad to see this being driven towards commit. I also really appreciate the cf_dev -> dev revert; keeping name changes divorced from functional changes reduces the size of the patch and increases clarity of what has changed.
Oct 17 2019
(Committed, link/revision above.)
Committed in r353685, I'm not sure why Phabricator has not closed this yet.