allanjude (Allan Jude)Administrator
User

Projects

User Details

User Since
May 19 2014, 3:05 PM (204 w, 4 d)
Roles
Administrator

Recent Activity

Today

allanjude accepted D7897: bspatch: add integer overflow checks.
Sat, Apr 21, 3:26 AM

Yesterday

allanjude added a comment to D15140: Update Komodo-Edit to 11.0.2.

Yeah, if we can do the native version that is better, and it means the license is standard MPL

Fri, Apr 20, 6:57 PM
allanjude created D15140: Update Komodo-Edit to 11.0.2.
Fri, Apr 20, 2:24 AM

Wed, Apr 18

allanjude committed rS332662: Make `iostat -xC` print CPU information on its own line.
Make `iostat -xC` print CPU information on its own line
Wed, Apr 18, 2:25 AM
allanjude closed D15076: Make `iostat -xC` print CPU information on its own line.
Wed, Apr 18, 2:25 AM

Sun, Apr 15

allanjude updated the summary of D15076: Make `iostat -xC` print CPU information on its own line.
Sun, Apr 15, 5:24 PM
allanjude updated the diff for D15076: Make `iostat -xC` print CPU information on its own line.

Fix alignment of the cpu header

Sun, Apr 15, 5:23 PM
allanjude updated the summary of D15076: Make `iostat -xC` print CPU information on its own line.
Sun, Apr 15, 5:04 PM
allanjude updated the summary of D15076: Make `iostat -xC` print CPU information on its own line.
Sun, Apr 15, 5:04 PM
allanjude created D15076: Make `iostat -xC` print CPU information on its own line.
Sun, Apr 15, 5:03 PM
allanjude closed D14502: Improve the disk growing instructions.
Sun, Apr 15, 2:54 PM
allanjude committed rD51556: Improve the instructions for growing disks.
Improve the instructions for growing disks
Sun, Apr 15, 2:54 PM
allanjude committed rD51555: Update website link for the Prague BSD User Group.
Update website link for the Prague BSD User Group
Sun, Apr 15, 2:52 PM
allanjude added reviewers for D14553: Set geom_eli module version: oshogbo, pjd.
Sun, Apr 15, 2:51 PM

Mon, Apr 9

allanjude accepted D15000: Allow to verify keys in geli..

Thanks for working on this

Mon, Apr 9, 10:23 PM

Sun, Apr 8

allanjude added inline comments to D15000: Allow to verify keys in geli..
Sun, Apr 8, 6:13 PM

Thu, Mar 29

allanjude added a comment to D14289: VMware Virtual Machine Communication Interface (VMCI) kernel module.

Coverity found a number of issues with this commit. It is possible they are false positives:

Thu, Mar 29, 7:09 PM

Tue, Mar 27

allanjude updated the diff for D11124: WIP: ZSTD in ZFS.

Rebase to r331496

Tue, Mar 27, 12:25 AM

Mar 13 2018

allanjude added inline comments to D14681: Dynamically add jail-enabled filesystems.
Mar 13 2018, 6:19 PM

Mar 11 2018

allanjude committed rS330752: Grant Brad Davis (brd) a src bit.
Grant Brad Davis (brd) a src bit
Mar 11 2018, 12:08 AM

Mar 9 2018

allanjude created D14631: bsdinstall: ssh lockout protection.
Mar 9 2018, 1:08 PM

Mar 1 2018

allanjude requested changes to D14553: Set geom_eli module version.
Mar 1 2018, 1:21 PM

Feb 27 2018

allanjude updated the diff for D11124: WIP: ZSTD in ZFS.

Update to r329600

Feb 27 2018, 3:46 AM
allanjude added a comment to D14517: Correct instructions for extracting multiple sets to create a jail.

We should probably put 'doc' back in the list, there's no fundamental reason docs shouldn't be installed in a jail, perhaps a follow-up after further investigation.

Feb 27 2018, 3:00 AM

Feb 25 2018

allanjude updated the diff for D14502: Improve the disk growing instructions.

Fix tense error pointed out by bcr

Feb 25 2018, 6:43 PM
allanjude created D14502: Improve the disk growing instructions.
Feb 25 2018, 6:05 AM

Feb 22 2018

allanjude added reviewers for D14465: Create sysctl hw.model on ARM: andrew, manu, imp.
Feb 22 2018, 12:52 AM · ARM
allanjude accepted D14408: Sandbox wc(1) with fileargs..
Feb 22 2018, 12:46 AM

Feb 17 2018

allanjude accepted D14408: Sandbox wc(1) with fileargs..

This looks pretty clean, I like it.

Feb 17 2018, 5:13 PM

Feb 13 2018

allanjude accepted D14352: Move zstd malloc()/free()/calloc() macros to stdlib.h..
Feb 13 2018, 4:33 PM

Feb 10 2018

allanjude added a comment to D14286: firewall_type is ignored if not set in rc.conf or rc.conf.local.

For this to work right, shouldn't we load /etc/rc.conf.d/pf and maybe ipf as well?

Feb 10 2018, 5:22 AM

Feb 6 2018

allanjude accepted D13778: rc.d/cleanvar: purgedir function has shell expansion vulnerability.

This seems like a much safer approach

Feb 6 2018, 8:25 PM

Feb 4 2018

allanjude committed rS328839: Add the ThinkPad X1 (sandybridge) to the bsdinstall blacklist.
Add the ThinkPad X1 (sandybridge) to the bsdinstall blacklist
Feb 4 2018, 12:17 PM

Feb 2 2018

allanjude accepted D12690: Add GELI partition types.
Feb 2 2018, 10:44 AM

Jan 31 2018

allanjude accepted D12302: Move EFI FAT creation to here to obviate need for the boot1.efifat file.
Jan 31 2018, 12:55 AM

Jan 29 2018

allanjude accepted D14120: Remove duplicate entries from various conf/files*.
Jan 29 2018, 5:27 PM
allanjude updated the diff for D11124: WIP: ZSTD in ZFS.

Solve issues with L2ARC if compressed_arc_enabled=0

Jan 29 2018, 5:33 AM

Jan 28 2018

allanjude added a comment to D14064: Change the installer default to not install ports.

Maybe as a separate change, but is it time to stop shipping lib32 by default as well?

Jan 28 2018, 12:16 AM
allanjude accepted D14064: Change the installer default to not install ports.
Jan 28 2018, 12:16 AM

Jan 27 2018

allanjude updated the diff for D7655: Capsicumize uudecode/b64decode.

Update to latest -current for @bkidney_briankidney.ca

Jan 27 2018, 11:01 PM
allanjude closed D10239: Create a machine/metadata.h for RISC-V.
Jan 27 2018, 10:47 PM
allanjude abandoned D12295: Kill gets().
Jan 27 2018, 10:30 PM
allanjude abandoned D12624: sys/boot: always use freshly build libstand.
Jan 27 2018, 10:29 PM

Jan 22 2018

allanjude added a comment to D12396: Default ABD chunk size.

avg@ added the loader tunable to make this adjustable in https://svnweb.freebsd.org/base?view=revision&revision=323797 however I have every reason to believe (and measurements from production) that 4K is a going to result in more waste than a 1K default. A 1K default may result in fragmentation with 4K slabs but there is less waste for everyone with 1K than 4K.

@mahrens , do you think OpenZFS will go back to a 1K default?

Jan 22 2018, 3:40 AM · ZFS

Jan 14 2018

allanjude accepted D11970: cddl/sbin/zfs build warning fix.
Jan 14 2018, 5:06 PM

Jan 4 2018

allanjude added inline comments to D13757: morse: implement support for decoding morse code.
Jan 4 2018, 4:54 AM

Dec 7 2017

allanjude added a member for committers: chuck.
Dec 7 2017, 4:09 PM

Dec 5 2017

allanjude updated the diff for D11124: WIP: ZSTD in ZFS.

Newer version of ZSTD (1.3.2)

Dec 5 2017, 4:41 AM

Dec 4 2017

allanjude updated the diff for D10407: Modify zstd so it can be built into the kernel as well.

Switch from symlink to include files

Dec 4 2017, 4:51 AM
allanjude committed rS326507: increase maximum size of zfsboot.
increase maximum size of zfsboot
Dec 4 2017, 2:42 AM
allanjude updated the diff for D10407: Modify zstd so it can be built into the kernel as well.

Switch to Warner's fake-files concept

Dec 4 2017, 2:21 AM
allanjude added inline comments to D10407: Modify zstd so it can be built into the kernel as well.
Dec 4 2017, 1:47 AM
allanjude committed rS326506: Use __has_builtin() to ensure clz and clzll builtins are available.
Use __has_builtin() to ensure clz and clzll builtins are available
Dec 4 2017, 1:16 AM

Dec 3 2017

allanjude updated the diff for D10407: Modify zstd so it can be built into the kernel as well.

Update to latest -current and lots of cleanup suggested by imp

Dec 3 2017, 5:15 AM

Nov 30 2017

allanjude accepted D13301: Fix various warnings related to geli support..

Thanks for cleaning up my mess

Nov 30 2017, 4:59 PM

Nov 23 2017

allanjude added inline comments to D12577: Some more basic flavors work..
Nov 23 2017, 5:16 PM

Nov 22 2017

allanjude accepted D13190: Fix indentation in bsdinstall-created wpa_supplicant.conf.

goatforit

Nov 22 2017, 3:39 AM

Nov 21 2017

allanjude added a comment to D13088: zpool: cannot anymore import cache device which doesn't contain a previos ZFS vdev label.

The best fix would have been to check the errno value inside zpool_read_label() just after the libc calls, like all errno checks should be done. But it necessitates to add one more return value to zpool_read_label():

  • 0 == vdev label was read (continue the normal execution)
  • -1 == ENOMEM, do the no_memory() call and fail.
  • -2 == vdev label not found (don't enter the if block and continue the normal execution)
Nov 21 2017, 1:26 AM

Nov 19 2017

allanjude added a comment to D7538: Correct adaptation ZFS ARC memory pressure to FreeBSD.

I'll try to get some time allocated to that - right now the machine that
I use for this effort is busy doing other things but I may be able to
free up some time on it in the next week or two.

Is there any intent to backport/MFC those changes into 11-STABLE?  That
I can probably test and adapt the code to more-quickly.

The needfree change is a good one, it's just that having it confined to
-HEAD causes problems for me right now due to "free computer of
appropriate configuration" constraints.

Thanks in advance.

Nov 19 2017, 6:49 PM · ZFS
allanjude added a comment to D7538: Correct adaptation ZFS ARC memory pressure to FreeBSD.

Well that's a bitch.

It's easily fixed but I'm not going to be doing much with -HEAD until it
stabilizes and gets closer to being -RELEASE.

Nov 19 2017, 4:53 PM · ZFS

Nov 18 2017

allanjude accepted D13149: bsdinstall: Add ntpdate option.

Note: the modern installer has an entire UI dedicated to you setting the date correctly during the install.

Nov 18 2017, 10:08 PM

Nov 17 2017

allanjude added a comment to D13088: zpool: cannot anymore import cache device which doesn't contain a previos ZFS vdev label.

Thanks for your comments Allan and Andriy.
Concerning the errno set to 12 (ENOMEM), it's effectively set by the ioctl ZFS_IOC_POOL_STATS which is first called with a too small nvlist.
I'm attaching the conditional watchpoint where it happens.

Watchpoint 1 hit:
old value: 0
new value: 12
Process 11604 stopped
* thread #1, name = 'zpool', stop reason = watchpoint 1
    frame #0: 0x0000000800e951e9 libzfs.so.2`zcmd_ioctl(fd=3, request=5, zc=0x00007fffffffc640) at libzfs_compat.c:117
   114 	
   115 		if (ret == 0 && oldsize < zc->zc_nvlist_dst_size) {
   116 			ret = -1;
-> 117 			errno = ENOMEM;
   118 		}
   119 	
   120 		return (ret);
(lldb) bt
* thread #1, name = 'zpool', stop reason = watchpoint 1
  * frame #0: 0x0000000800e951e9 libzfs.so.2`zcmd_ioctl(fd=3, request=5, zc=0x00007fffffffc640) at libzfs_compat.c:117
    frame #1: 0x0000000800e87099 libzfs.so.2`zpool_refresh_stats(zhp=0x0000000802c30000, missing=0x00007fffffffd844) at libzfs_config.c:282
    frame #2: 0x0000000800e800e5 libzfs.so.2`zpool_open_canfail(hdl=0x0000000802c20000, pool="bld311") at libzfs_pool.c:995
    frame #3: 0x0000000800e802ef libzfs.so.2`zpool_open(hdl=0x0000000802c20000, pool=<unavailable>) at libzfs_pool.c:1051
    frame #4: 0x000000000040820c zpool`zpool_do_add(argc=3, argv=0x00007fffffffeb70) at zpool_main.c:540
    frame #5: 0x000000000040766f zpool`main(argc=<unavailable>, argv=0x00007fffffffeb60) at zpool_main.c:5642
    frame #6: 0x00000000004043ff zpool`_start + 399

Concerning your suggestion on how to fix it, I would rather suggest that to modify zpool_read_label() to return -1 in the case of a libc with errno == ENOMEM and return -2 in the case of other internal and logical ZFS "failure" ( no vdev label for exemple ).
In my opinion our real bug here is to check the value of errno after a call to an internal ZFS function and not after the libc call. Currently we are reading errno too late and we cannot easily differentiate a libc failure from a normal ZFS failure.

for example:

if (zpool_read_label(fd, &config) != 0 && errno == ENOMEM) {
        (void) no_memory(hdl);
        return (-1);
}

would be:

if (zpool_read_label(fd, &config) == -1) {
        (void) no_memory(hdl);
        return (-1);
}
Nov 17 2017, 5:35 PM

Nov 15 2017

allanjude added a comment to D13088: zpool: cannot anymore import cache device which doesn't contain a previos ZFS vdev label.

I think since we are returning -1 (a failure), it makes more sense to set errno to 0 at the top of the function, so that we only zero it if it is from before this function was run.

Nov 15 2017, 5:03 PM

Nov 13 2017

allanjude closed D11471: Add deprecation notices for all rcmd tools.
Nov 13 2017, 2:18 AM
allanjude removed a reviewer for D11471: Add deprecation notices for all rcmd tools: jlh.
Nov 13 2017, 2:18 AM
allanjude accepted D11471: Add deprecation notices for all rcmd tools.
Nov 13 2017, 2:18 AM

Nov 12 2017

allanjude added a comment to D10407: Modify zstd so it can be built into the kernel as well.

Thanks for the feedback Warner. I'll work on that stuff later today or tomorrow.

Nov 12 2017, 12:08 AM

Nov 11 2017

allanjude updated the diff for D10407: Modify zstd so it can be built into the kernel as well.

Rebase to zstd 1.3.2

Nov 11 2017, 3:15 PM
allanjude closed D12395: vmstat: Fix duplicate key in libxo output.
Nov 11 2017, 2:02 PM
allanjude committed rS325715: vmstat: fix duplicate key in libxo output.
vmstat: fix duplicate key in libxo output
Nov 11 2017, 2:02 PM

Nov 1 2017

allanjude added a comment to D11124: WIP: ZSTD in ZFS.

@yann.collet.73_gmail.com : Is there any tuning we can do to get the size of the compression context down, since we know the maximum size if the input will be 16MB?

Nov 1 2017, 3:05 AM

Oct 31 2017

allanjude added a comment to D12847: loader ptblread() is broken with >512B sectors.

I will test this on my macbookpro NVMe where FreeBSD currently cannot read the partition table.

Oct 31 2017, 11:51 AM
allanjude added a comment to D12848: Remove 'adding_user.7'.

Did you check for other man pages that .Xr this manpage?

Oct 31 2017, 11:50 AM
allanjude added a comment to D9649: Enable VNET operations for ifconfig and route.

I think an interface like mjg described is the best approach.

Oct 31 2017, 11:21 AM · network
allanjude requested changes to D9649: Enable VNET operations for ifconfig and route.

jail_attach() is not the correct approach, because it makes the actions of the administrator on the host visible within the jail.

Oct 31 2017, 11:08 AM · network

Oct 30 2017

allanjude accepted D12831: bluetooth: Default to discoverable off.

bluetooth should not be unconditional.

Oct 30 2017, 4:35 AM

Oct 29 2017

allanjude accepted D12827: New ports, sysutils/smart: Output disk SMART values.

Thank you very much

Oct 29 2017, 9:37 PM
allanjude accepted D12820: Move up the correct to compile the kernel.

Approved

Oct 29 2017, 12:57 AM
allanjude added a reviewer for D12820: Move up the correct to compile the kernel: allanjude.
Oct 29 2017, 12:52 AM
allanjude added a comment to D12820: Move up the correct to compile the kernel.

It might be useful to mention -DNO_CLEAN as a more supported shortcut

Oct 29 2017, 12:51 AM

Oct 17 2017

allanjude accepted D12678: Improve svn copy documentation.

Approved

Oct 17 2017, 5:22 PM

Oct 16 2017

allanjude added a reviewer for D11124: WIP: ZSTD in ZFS: tsoome.
Oct 16 2017, 4:47 PM
allanjude accepted D12679: Document missing libgeom functions.

Approved By: allanjude

Oct 16 2017, 3:33 PM

Oct 14 2017

allanjude accepted D11793: security/openssh-portable: Remove groff dependency.
Oct 14 2017, 2:28 AM

Oct 12 2017

allanjude accepted D12640: libc: allow posix_fallocate in capability mode.
Oct 12 2017, 2:04 AM

Oct 10 2017

allanjude added inline comments to D11124: WIP: ZSTD in ZFS.
Oct 10 2017, 4:56 AM
allanjude added a dependency for D11124: WIP: ZSTD in ZFS: D10407: Modify zstd so it can be built into the kernel as well.
Oct 10 2017, 3:01 AM
allanjude added a dependent revision for D10407: Modify zstd so it can be built into the kernel as well: D11124: WIP: ZSTD in ZFS.
Oct 10 2017, 3:01 AM
allanjude updated the summary of D11124: WIP: ZSTD in ZFS.
Oct 10 2017, 3:01 AM
allanjude updated the diff for D10407: Modify zstd so it can be built into the kernel as well.

Cleanup debugging cruft

Oct 10 2017, 2:58 AM
allanjude updated the diff for D10407: Modify zstd so it can be built into the kernel as well.

Sparc fixes

Oct 10 2017, 2:55 AM
allanjude updated the diff for D11124: WIP: ZSTD in ZFS.

Update to support ZSTD 1.3.x

Oct 10 2017, 2:53 AM

Oct 8 2017

allanjude added a comment to D12624: sys/boot: always use freshly build libstand.

@imp I know you are working on a sys/boot cleanup, so I figured you might want to integrate this

Oct 8 2017, 7:58 PM
allanjude created D12624: sys/boot: always use freshly build libstand.
Oct 8 2017, 7:58 PM

Oct 7 2017

allanjude updated the diff for D10407: Modify zstd so it can be built into the kernel as well.

Address Conrad's feedback re: M_NOWAIT

Oct 7 2017, 3:18 AM
allanjude updated the summary of D10407: Modify zstd so it can be built into the kernel as well.
Oct 7 2017, 3:18 AM

Oct 6 2017

allanjude added a comment to D10447: Unify boot1 with loader.

This now cleanly applies with 'arc patch' for me

Oct 6 2017, 3:37 PM
allanjude updated the diff for D10407: Modify zstd so it can be built into the kernel as well.

Updated for newer version of zstd

Oct 6 2017, 4:57 AM
allanjude accepted D12298: libc: remove gets.
Oct 6 2017, 3:57 AM

Oct 3 2017

allanjude accepted D12578: Display TRIM/UNMAP support in "diskinfo -v".

Thank you

Oct 3 2017, 8:59 PM