Differential D25400

Remove some redundant assignments and computations.
ClosedPublic
Actions

Authored by markj on Jun 22 2020, 12:52 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Tue, Nov 19, 8:50 PM

	Unknown Object (File)
	Mon, Nov 11, 6:05 AM

	Unknown Object (File)
	Oct 21 2024, 1:22 PM

	Unknown Object (File)
	Sep 25 2024, 3:25 AM

	Unknown Object (File)
	Sep 18 2024, 5:05 AM

	Unknown Object (File)
	Sep 16 2024, 5:21 AM

	Unknown Object (File)
	Sep 8 2024, 12:41 PM

	Unknown Object (File)
	Sep 8 2024, 12:41 PM

View All 51 Files

Subscribers

Details

Reviewers

alc
kib
dougm

Commits

rS362739: Remove some redundant assignments and computations.

Summary

Reported by: alc

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

markj created this revision.Jun 22 2020, 12:52 PM

Herald added a subscriber: emaste. · View Herald TranscriptJun 22 2020, 12:52 PM

markj requested review of this revision.Jun 22 2020, 12:52 PM

Harbormaster completed remote builds in B31872: Diff 73465.Jun 22 2020, 12:52 PM

markj added reviewers: alc, kib, dougm.Jun 22 2020, 12:52 PM

markj mentioned this in D25329: Call swap_pager_freespace() from vm_object_page_remove()..

kib accepted this revision.Jun 22 2020, 2:31 PM

kib added inline comments.

sys/dev/md/md.c
1561 ↗	(On Diff #73465)	I do not see much sense in asserting this, but do not object strongly.

This revision is now accepted and ready to land.Jun 22 2020, 2:31 PM

alc added inline comments.Jun 22 2020, 6:36 PM

sys/compat/linuxkpi/common/src/linux_page.c
223 ↗	(On Diff #73465)	AFAIK, the mips and powerpc versions of ptoa() have a single implementation that is used on both 32- and 64-bit machines, and on 64-bit machine there is no guarantee that the argument is converted to a 64-bit quantity.

Address feedback.

This revision now requires review to proceed.Jun 25 2020, 3:04 PM

Harbormaster completed remote builds in B31956: Diff 73636.Jun 25 2020, 3:04 PM

kib added inline comments.Jun 25 2020, 8:04 PM

sys/compat/linuxkpi/common/src/linux_page.c
223 ↗	(On Diff #73636)	I think size_t or vm_offset_t would be better there.

Use vm_offset_t instead.

Harbormaster completed remote builds in B31964: Diff 73662.Jun 25 2020, 8:24 PM

kib accepted this revision.Jun 25 2020, 8:58 PM

This revision is now accepted and ready to land.Jun 25 2020, 8:58 PM

alc accepted this revision.Jun 26 2020, 6:23 PM

alc added inline comments.

sys/compat/linuxkpi/common/src/linux_page.c
202 ↗	(On Diff #73662)	Here is another possible ptoa() conversion that is identical to the first.

markj marked 2 inline comments as done.Jun 28 2020, 9:23 PM

Closed by commit rS362739: Remove some redundant assignments and computations. (authored by markj). · Explain WhyJun 28 2020, 9:34 PM

This revision was automatically updated to reflect the committed changes.

markj added a commit: rS362739: Remove some redundant assignments and computations..

Herald added a subscriber: imp. · View Herald TranscriptJun 28 2020, 9:34 PM

Revision Contents
Changeset List

Path

Size

head/

sys/

compat/

linuxkpi/

common/

src/

5 lines

dev/

md/

4 lines

Diff 73819

head/sys/compat/linuxkpi/common/src/linux_page.c

Loading...

head/sys/dev/md/md.c

Loading...