Paths

Table of Contentst

Differential D5629

fix ppc build without COMPAT_FREEBSD6
AbandonedPublic
Actions

Authored by emaste on Mar 14 2016, 1:38 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Tue, Dec 10, 1:18 AM

	Unknown Object (File)
	Oct 28 2024, 8:15 AM

	Unknown Object (File)
	Oct 19 2024, 6:37 PM

	Unknown Object (File)
	Oct 9 2024, 3:12 PM

	Unknown Object (File)
	Oct 2 2024, 7:23 PM

	Unknown Object (File)
	Oct 1 2024, 5:10 AM

	Unknown Object (File)
	Sep 21 2024, 4:51 AM

	Unknown Object (File)
	Sep 18 2024, 12:49 AM

View All 46 Files

Subscribers

Details

Reviewers

jhibbits
jhb

Summary

Use SYS_freebsd6_lseek only if COMPAT_FREEBSD6 is enabled.

Tinderbox had failed with

/scratch/tmp/emaste/freebsd/sys/powerpc/powerpc/exec_machdep.c: In function 'cpu_set_syscall_retval':
/scratch/tmp/emaste/freebsd/sys/powerpc/powerpc/exec_machdep.c:882: error: 'SYS_freebsd6_lseek' undeclared (first use in this function)
/scratch/tmp/emaste/freebsd/sys/powerpc/powerpc/exec_machdep.c:882: error: (Each undeclared identifier is reported only once
/scratch/tmp/emaste/freebsd/sys/powerpc/powerpc/exec_machdep.c:882: error: for each function it appears in.)
*** [exec_machdep.o] Error code 1

Diff Detail

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

emaste updated this revision to Diff 14302.Mar 14 2016, 1:38 AM

emaste retitled this revision from to fix ppc build without COMPAT_FREEBSD6.

emaste updated this object.

emaste edited the test plan for this revision. (Show Details)

emaste added a reviewer: jhibbits.

Herald edited edge metadata. · View Herald TranscriptMar 14 2016, 1:38 AM

jhibbits added inline comments.Mar 14 2016, 2:47 AM

sys/powerpc/powerpc/exec_machdep.c
882–889	It looks like you're reversing the check.

Correct sense of test

Herald edited edge metadata. · View Herald TranscriptMar 14 2016, 2:53 AM

jhibbits accepted this revision.Mar 14 2016, 2:57 AM

jhibbits edited edge metadata.

This revision is now accepted and ready to land.Mar 14 2016, 2:57 AM

Adding @jhb as it seems this is fallout from D5597

@bz committed a similar change in rS296861

Revision Contents
Changeset List

Path

Size

sys/

powerpc/

powerpc/

10 lines

Diff 14303

sys/powerpc/powerpc/exec_machdep.c

Loading...