Re: more on devstat (missing include)

On Wednesday 30 January 2008 06:09:21 pm Pietro Cerutti wrote:
Hi hackers,

I think that i) sys/devicestat.h should include sys/resource.h or ii)
the man page of devstat(9) should mention that the user himself is
supposed to include sys/resource.h.

Without:

cat devstat.c
#include <devstat.h>

int main(void)
{
devstat_getnumdevs(NULL);
return (0);
}
gcc -ldevstat -o devstat devstat.c
In file included from devstat.c:1:
/usr/include/devstat.h:131: error: 'CPUSTATES' undeclared here (not in a
function)
Exit 1


I can provide a patch if someone could tell me which one of the proposed
solutions above is the preferred one.

I think this might be best:

Index: devstat.h
===================================================================
RCS file: /usr/cvs/src/lib/libdevstat/devstat.h,v
retrieving revision 1.11
diff -u -r1.11 devstat.h
--- devstat.h 22 Jul 2005 17:19:00 -0000 1.11
+++ devstat.h 31 Jan 2008 10:52:37 -0000
@@ -32,6 +32,7 @@
#define _DEVSTAT_H
#include <sys/cdefs.h>
#include <sys/devicestat.h>
+#include <sys/resource.h>

#include <kvm.h>


--
John Baldwin
_______________________________________________
freebsd-hackers@xxxxxxxxxxx mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-hackers
To unsubscribe, send any mail to "freebsd-hackers-unsubscribe@xxxxxxxxxxx"

Relevant Pages

  • Re: EHCI problem on -current, and unknown devs
    ... John Baldwin wrote: ... retrieving revision 1.53 ... diff -u -r1.53 subr_rman.c ... With up to date -current sources, this doesn't compile for me: ...
    (freebsd-current)
  • Re: Cleaning up FILE in stdio..
    ... John Baldwin writes: ... xterm cut-and-paste screwage): ... retrieving revision 1.20 ... diff -u -r1.20 sysconf.c ...
    (freebsd-arch)
  • Re: more on devstat (missing include)
    ... On Thursday 31 January 2008 10:20:42 am Pietro Cerutti wrote: ... diff -u -r1.11 devstat.h ... Are you committing that or I better send in a PR? ... John Baldwin ...
    (freebsd-hackers)
  • Re: more on devstat (missing include)
    ... Pietro Cerutti wrote: ... retrieving revision 1.11 ... diff -u -r1.11 devstat.h ... Are you committing that or I better send in a PR? ...
    (freebsd-hackers)
  • Re: more on devstat (missing include)
    ... John Baldwin wrote: ... Are you committing that or I better send in a PR? ... Pietro Cerutti ... PGP Public Key: ...
    (freebsd-hackers)