Page MenuHomeFreeBSD
Differential D35173

kerneldump: remove physical argument from d_dumper
ClosedPublic
Actions

Authored by mhorne on Wed, May 11, 5:29 PM.

Details

Reviewers
markj
imp
jhb
Group Reviewers
cam
Commits
R10:489ba2223676: kerneldump: remove physical argument from d_dumper
Summary

The physical address argument is essentially ignored by every dumper
method. In addition, the dump routines don't actually pass a real
address; every call to dump_append() passes a value of zero for
physical.

Diff Detail

Repository
R10 FreeBSD src repository
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

mhorne created this revision.Wed, May 11, 5:29 PM
Herald added a reviewer: cam. · View Herald TranscriptWed, May 11, 5:29 PM
Herald added subscribers: glebius, ehem_freebsd_m5p.com, dab, melifaro. · View Herald Transcript
mhorne requested review of this revision.Wed, May 11, 5:29 PM
Harbormaster completed remote builds in B45530: Diff 105861.Wed, May 11, 5:29 PM
mhorne added a child revision: D35174: kerneldump: remove physical from dump routines.Wed, May 11, 5:29 PM
mhorne mentioned this in D35174: kerneldump: remove physical from dump routines.Wed, May 11, 5:30 PM
markj accepted this revision.Thu, May 12, 1:03 PM

Thanks! Yeah this parameter was never used as far as I could tell, and stopped making sense when support for compression was added.

This revision is now accepted and ready to land.Thu, May 12, 1:03 PM
ehem_freebsd_m5p.com removed a subscriber: ehem_freebsd_m5p.com.Fri, May 13, 3:07 AM
Closed by commit R10:489ba2223676: kerneldump: remove physical argument from d_dumper (authored by mhorne). · Explain WhyFri, May 13, 1:45 PM
This revision was automatically updated to reflect the committed changes.
mhorne added a commit: R10:489ba2223676: kerneldump: remove physical argument from d_dumper.

Revision Contents
Changeset List

PathSize
sys/
cam/
ata/
4 lines
mmc/
3 lines
nvme/
4 lines
scsi/
4 lines
dev/
aac/
6 lines
ida/
2 lines
ips/
6 lines
mfi/
2 lines
3 lines
mmc/
6 lines
nvd/
2 lines
virtio/
block/
5 lines
xen/
blkfront/
3 lines
geom/
nop/
3 lines
raid/
4 lines
14 lines
1 line
6 lines
6 lines
7 lines
8 lines
4 lines
union/
5 lines
kern/
2 lines
4 lines
netinet/
netdump/
6 lines
sys/
1 line

Diff 105942

View Options

sys/cam/ata/ata_da.c

Loading...
View Options

sys/cam/mmc/mmc_da.c

Loading...
View Options

sys/cam/nvme/nvme_da.c

Loading...
View Options

sys/cam/scsi/scsi_da.c

Loading...
View Options

sys/dev/aac/aac_disk.c

Loading...
View Options

sys/dev/ida/ida_disk.c

Loading...
View Options

sys/dev/ips/ips_disk.c

Loading...
View Options

sys/dev/mfi/mfi_disk.c

Loading...
View Options

sys/dev/mfi/mfi_syspd.c

Loading...
View Options

sys/dev/mmc/mmcsd.c

Loading...
View Options

sys/dev/nvd/nvd.c

Loading...
View Options

sys/dev/virtio/block/virtio_blk.c

Loading...
View Options

sys/dev/xen/blkfront/blkfront.c

Loading...
View Options

sys/geom/nop/g_nop.c

Loading...
View Options

sys/geom/raid/g_raid.h

Loading...
View Options

sys/geom/raid/g_raid.c

Loading...
View Options

sys/geom/raid/g_raid_tr_if.m

Loading...
View Options

sys/geom/raid/tr_concat.c

Loading...
View Options

sys/geom/raid/tr_raid0.c

Loading...
View Options

sys/geom/raid/tr_raid1.c

Loading...
View Options

sys/geom/raid/tr_raid1e.c

Loading...
View Options

sys/geom/raid/tr_raid5.c

Loading...
View Options

sys/geom/union/g_union.c

Loading...
View Options

sys/kern/kern_shutdown.c

Loading...
View Options

sys/kern/kern_vnodedumper.c

Loading...
View Options

sys/netinet/netdump/netdump_client.c

Loading...
View Options

sys/sys/conf.h

Loading...