Project

General

Profile

Actions

Bug #4800

closed

copyin(9F) doesn't mention sys/systm.h

Added by Rich Lowe over 8 years ago. Updated over 8 years ago.

Status:
Rejected
Priority:
Normal
Assignee:
-
Category:
manpage - manual pages
Start date:
2014-04-24
Due date:
% Done:

0%

Estimated time:
Difficulty:
Bite-size
Tags:
needs-triage
Gerrit CR:

Description

the prototype for copyin(9F) seems to be in sys/systm.h. Now, perhaps I'm an idiot, but I did what the manual page told me, and that was insufficient.

It seems like the manual page should be correct, and list sys/systm.h.

Actions #1

Updated by Garrett D'Amore over 8 years ago

  • Status changed from New to Rejected

I just tested. You almost CERTAINLY forgot to add -D_KERNEL.

All kernel APIs (DDI) require the -D_KERNEL define to be added. Just adding systm.h to the documentation is WRONG, because that header is not part of the DDI.

I'm actually REJECTING this change, on the basis that I've demonstrated it works for me:

cc -c /tmp/t.c

"/tmp/t.c", line 7: warning: implicit function declaration: copyin
"/tmp/t.c", line 8: warning: implicit function declaration: cmn_err

cc -D_KERNEL -c /tmp/t.c

Actions

Also available in: Atom PDF