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

	F134911667: D5629.diff
	Wed, Nov 5, 4:13 AM

	Unknown Object (File)
	Sat, Nov 1, 1:06 AM

	Unknown Object (File)
	Sat, Nov 1, 1:06 AM

	Unknown Object (File)
	Sat, Nov 1, 1:06 AM

	Unknown Object (File)
	Fri, Oct 31, 8:27 PM

	Unknown Object (File)
	Thu, Oct 30, 4:33 AM

	Unknown Object (File)
	Tue, Oct 7, 5:01 PM

	Unknown Object (File)
	Sep 24 2025, 10:19 PM

View All 80 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...