In D46269#1055082, @jrm wrote:

Thanks! I don't see anything in the commit log of https://git.kernel.org/pub/scm/linux/kernel/git/firmware/linux-firmware.git/commit/?id=091bd5adf19c7ab01214c64689952acb483 that the update addresses the vulnerabilities. Do you have a reference for that point?

In D40377#922083, @crest_freebsd_rlwinm.de wrote:

The jail_name variable must be initialised to NULL. This should be done through an explicit char * jail_name = NULL; in line 101 of route.c.

• remove usejail variable

@melifaro Thank you so much for reviewing and the suggestion! I have moved the jail attach bit out and added the missed Makefile changes. (oops)

• move jail attach away from arg parse

@dfr A patch for route is available at D40377

@jamie Here is the review for the sysctl

• assign value to prison_fallback_brand varibale right before it is used

In D40262#917097, @jamie wrote:

Rather than add a separate value different from the kern.fallback_elf_brand sysctl, it would make sense for the jail parameter to be tied to the sysctl itself, such as the securelevel parameter is. It's complicated somewhat by the fact that there are three similar sysctls.

• Use pr_spare for pr_elf_fallback_brand

• update date

• use .Pp instead of empty lines in manpage

• use .Xr in manpage

Man page update

• fix sysctl parametetr description

• remove explicit initialization

including the correct range

cleanup

Move jail_attach away from args_parse

• print error and exit if -j specified but ifconfig isn't built with jail support
• bump manpage date

• modify man page to refactor the new -j flag

• improve readability

• fix rescue build

@erj @krzysztof.galazka_intel.com Thank you so much for reviewing. Do you mind to commit this as well? Thank you so much.

@jhb Thank you so much for helping! I have submitted a few patches under "Yan Ka Chiu" lately so I guess I will use that instead for consistency.

Fix typo

• change to use type as the option
• simplify code

• Rephrase error message and formatting
• Removed blank line

@des updated

• Rephrase error message and formatting

Fix the submitted diff

also check rc

• change to use type as the option
• yes I am an idiot and totally forgot bhyve-config(5)

In D29359#657456, @thierry wrote:

Thanks for resurrecting julia!

I have two remarks:

• any reason to work on 1.0.3? The latest release is now 1.5.4, and 1.6.0 is now at -rc3;
• when using the external BLAS, you could set USES+= blaslapack:openblas

remove lang/julia and lang/juila10 from MOVED

• Manually expand the VTOI macro to further avoid dependency of

• Merge branch 'master' of https://github.com/freebsd/freebsd-ports
• revert changes to other files

Revert more files

Revert accidental changes to other ports

add port revision

Related bugzillla ticket: https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=253487

@koobs Please review and commit this if it's alright?

Add link to upstream pr

Use arc4random instead of random

In D28135#628982, @philip wrote:

If the iflib changes have been tested and are known to work, they're probably okay.

Is the extra patch to aq_hw.c intentional? Using random() to generate a MAC address isn't great. Not seeding it makes it even worse.

Use PORTREVISION instead of PORTEPOCH

In D20447#442373, @wulf wrote:

One more question:

Why ver and hor sysctls are added separately?

I think single "natural_scroll" is enough.

In D20447#441492, @wulf wrote:

Hi Nyan,
Could you upload the diff as fullcontext diff next time. See https://wiki.freebsd.org/Phabricator

git diff -U999999 other-branch > change.diff
git show -U999999 <commit-hash> > change.diff
svn diff --diff-cmd=diff -x -U999999 > change.diff`

It is not necessary to reupload current diff