Differential D23433

loader: rewrite zfs reader zap code to use malloc
ClosedPublic
Actions

Authored by tsoome on Jan 30 2020, 8:33 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Tue, Sep 8, 11:10 PM

	Unknown Object (File)
	Fri, Sep 4, 9:37 PM

	Unknown Object (File)
	Fri, Sep 4, 9:14 PM

	Unknown Object (File)
	Fri, Sep 4, 10:55 AM

	Unknown Object (File)
	Fri, Sep 4, 5:05 AM

	Unknown Object (File)
	Thu, Sep 3, 4:54 PM

	Unknown Object (File)
	Thu, Sep 3, 5:55 AM

	Unknown Object (File)
	Thu, Sep 3, 5:52 AM

Subscribers

Details

Reviewers

kevans
imp
allanjude
delphij

Commits

rS357497: loader: rewrite zfs reader zap code to use malloc

Summary

First step on removing zfs_alloc.

Test Plan

I have used this implementation since 21 Nov 2019.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

tsoome created this revision.Jan 30 2020, 8:33 PM

Herald added a subscriber: delphij. · View Herald TranscriptJan 30 2020, 8:33 PM

Harbormaster completed remote builds in B29054: Diff 67531.Jan 30 2020, 8:33 PM

replace if chain by switch. Print out entire zap_block_type.

Harbormaster completed remote builds in B29066: Diff 67556.Jan 31 2020, 4:12 PM

LGTM. There are a few minor nits but these are optional.

stand/libsa/zfs/zfsimpl.c
2458 ↗	(On Diff #67556)	Nit, optional: 'lp' is passed and from an on-stack variable, we could use it directly, e.g. lp = malloc(sizeof(*lp)) below.
2609 ↗	(On Diff #67556)	Nit, optional: I think it's better to have something like zap_leaf_free() that free's zl->l_phys and zl.

This revision is now accepted and ready to land.Feb 3 2020, 2:49 AM

Suggestions from delphij, thanks!

This revision now requires review to proceed.Feb 3 2020, 5:51 PM

Harbormaster completed remote builds in B29144: Diff 67725.Feb 3 2020, 5:51 PM

This revision was not accepted when it landed; it landed in state Needs Review.Feb 4 2020, 7:38 AM

Closed by commit rS357497: loader: rewrite zfs reader zap code to use malloc (authored by tsoome). · Explain Why

This revision was automatically updated to reflect the committed changes.

tsoome added a commit: rS357497: loader: rewrite zfs reader zap code to use malloc.

Revision Contents
Changeset List

Path

Size

head/

stand/

i386/

zfsboot/

29 lines

libsa/

zfs/

434 lines

sys/

cddl/

boot/

zfs/

13 lines

Diff 67764

head/stand/i386/zfsboot/zfsboot.c

Loading...

head/stand/libsa/zfs/zfsimpl.c

Loading...

head/sys/cddl/boot/zfs/zfsimpl.h

Loading...