]> Zhao Yanbai Git Server - minix.git/commitdiff
minix libraries: resolve clang warnings 55/2855/1
authorDavid van Moolenbroek <david@minix3.org>
Tue, 30 Sep 2014 21:04:09 +0000 (21:04 +0000)
committerDavid van Moolenbroek <david@minix3.org>
Tue, 30 Sep 2014 21:04:09 +0000 (21:04 +0000)
Change-Id: I0f3425fb8838708c9848a006f1eee5ab1bad71d2

43 files changed:
minix/include/minix/audio_fw.h
minix/include/minix/blockdriver_mt.h
minix/include/minix/clkconf.h
minix/lib/libaudiodriver/Makefile
minix/lib/libaudiodriver/audio_fw.c
minix/lib/libbdev/Makefile
minix/lib/libbdev/minor.c
minix/lib/libblockdriver/Makefile
minix/lib/libblockdriver/driver.c
minix/lib/libblockdriver/driver_mt.c
minix/lib/libblockdriver/trace.c
minix/lib/libblockdriver/trace.h
minix/lib/libchardriver/Makefile
minix/lib/libchardriver/chardriver.c
minix/lib/libclkconf/Makefile
minix/lib/libdevman/Makefile
minix/lib/libdevman/generic.c
minix/lib/libdevman/usb.c
minix/lib/libexec/exec_elf.c
minix/lib/libexec/libexec.h
minix/lib/libfetch/common.h
minix/lib/libfetch/ftp.c
minix/lib/libfetch/http.c
minix/lib/libi2cdriver/Makefile
minix/lib/libi2cdriver/i2cdriver.c
minix/lib/liblwip/Makefile
minix/lib/liblwip/core/netif.c
minix/lib/liblwip/sys_arch.c
minix/lib/libminixfs/Makefile
minix/lib/libminixfs/cache.c
minix/lib/libnetdriver/Makefile
minix/lib/libnetdriver/netdriver.c
minix/lib/libnetsock/Makefile
minix/lib/libnetsock/socket.c
minix/lib/libusb/Makefile
minix/lib/libusb/usb.c
minix/lib/libvassert/Makefile
minix/lib/libvassert/vassert.c
minix/lib/libvassert/vassert.h
minix/lib/libvirtio/Makefile
minix/lib/libvirtio/virtio.c
minix/lib/libvirtio/virtio_ring.h
minix/lib/libvtreefs/Makefile

index 324dbfe81d12be0402070a05bb89b452780edcef..53c3ee57d1ebe61f6a61c82ece41d56bfd3453ef 100644 (file)
@@ -84,4 +84,8 @@ EXTERN special_file_t special_file[];
 #define READ_DMA 1
 #define WRITE_DMA 2
 
+EXTERN int sef_cb_lu_prepare(int state);
+EXTERN int sef_cb_lu_state_isvalid(int state);
+EXTERN void sef_cb_lu_state_dump(int state);
+
 #endif /* AUDIO_FW_H */
index 2b96c505d208371b740fc2da2b013968234d7eb3..3c74515df303f6fd1fb5743be164ac646638d9b6 100644 (file)
@@ -10,7 +10,7 @@ void blockdriver_mt_task(struct blockdriver *driver_tab);
 void blockdriver_mt_sleep(void);
 void blockdriver_mt_wakeup(thread_id_t id);
 void blockdriver_mt_terminate(void);
-void blockdriver_mt_set_workers(device_id_t id, int workers);
+void blockdriver_mt_set_workers(device_id_t id, unsigned int workers);
 thread_id_t blockdriver_mt_get_tid(void);
 
 #endif /* _MINIX_BLOCKDRIVER_MT_H */
index dc66f4aac273467fdf18495a5c6117fb14828cb0..51645221c54ab53820b57e607ceb23c23d481a45 100644 (file)
@@ -11,8 +11,8 @@
 #define CM_PER_I2C1_CLKCTRL 0x048
 #define CM_WKUP_I2C0_CLKCTRL 0x4B8
 
-int clkconf_init();
+int clkconf_init(void);
 int clkconf_set(u32_t clk, u32_t mask, u32_t value);
-int clkconf_release();
+int clkconf_release(void);
 
 #endif
index c4d84ab0d4417814b9249c9c84c1d0ada1c473ad..545fb769248634950cc267e8e04228735ab57956 100644 (file)
@@ -1,6 +1,7 @@
 # Makefile for the common audio framework
+
 NOGCCERROR=yes
-NOCLANGERROR=yes
+
 CPPFLAGS+= -D_MINIX_SYSTEM
 
 LIB=    audiodriver
index ab45e5bb07c13b003e2bd757422d553507144bb8..39410c98532ed76447abc28eb662f894b1d7314d 100644 (file)
@@ -58,9 +58,6 @@ static int irq_hook_set = FALSE;
 static void sef_local_startup(void);
 static int sef_cb_init_fresh(int type, sef_init_info_t *info);
 static void sef_cb_signal_handler(int signo);
-EXTERN int sef_cb_lu_prepare(int state);
-EXTERN int sef_cb_lu_state_isvalid(int state);
-EXTERN void sef_cb_lu_state_dump(int state);
 
 static struct chardriver audio_tab = {
        .cdr_open       = msg_open,     /* open the special file */
@@ -73,9 +70,6 @@ static struct chardriver audio_tab = {
 
 int main(void)
 {
-       int r, caller;
-       message mess, repl_mess;
-       int ipc_status;
 
        /* SEF local startup. */
        sef_local_startup();
@@ -115,7 +109,7 @@ static void sef_local_startup(void)
 static int sef_cb_init_fresh(int UNUSED(type), sef_init_info_t *UNUSED(info))
 {
 /* Initialize the audio driver framework. */
-       u32_t i; char irq;
+       int i; char irq;
        static int executed = 0;
        sub_dev_t* sub_dev_ptr;
 
@@ -499,7 +493,7 @@ static ssize_t msg_read(devminor_t minor, u64_t UNUSED(position),
 
 static void msg_hardware(unsigned int UNUSED(mask))
 {
-       u32_t     i;
+       int i;
 
        /* loop over all sub devices */
        for ( i = 0; i < drv.NrOfSubDevices; i++) {
@@ -579,7 +573,6 @@ static void handle_int_write(int sub_dev_nr)
 static void handle_int_read(int sub_dev_nr) 
 {
        sub_dev_t *sub_dev_ptr;
-       message m;
 
        sub_dev_ptr = &sub_dev[sub_dev_nr];
 
@@ -651,7 +644,6 @@ static int get_started(sub_dev_t *sub_dev_ptr) {
 static void data_from_user(sub_dev_t *subdev)
 {
        int r;
-       message m;
 
        if (subdev->DmaLength == subdev->NrOfDmaFragments &&
                        subdev->BufLength == subdev->NrOfExtraBuffers) return;/* no space */
@@ -710,7 +702,6 @@ static void data_from_user(sub_dev_t *subdev)
 static void data_to_user(sub_dev_t *sub_dev_ptr)
 {
        int r;
-       message m;
 
        if (!sub_dev_ptr->RevivePending) return; /* nobody is wating for data */
        if (sub_dev_ptr->BufLength == 0 && sub_dev_ptr->DmaLength == 0) return; 
@@ -793,7 +784,7 @@ static int init_buffers(sub_dev_t *sub_dev_ptr)
        }
 
        if ((left = dma_bytes_left(sub_dev_ptr->DmaPhys)) < 
-                       sub_dev_ptr->DmaSize) {
+                       (unsigned int)sub_dev_ptr->DmaSize) {
                /* First half of buffer crosses a 64K boundary,
                 * can't DMA into that */
                sub_dev_ptr->DmaPtr += left;
index bf0e1c01c460b50c793f168b91a34e42f25fd520..f91e5cae70ec746a1fba57057a1a5567583ed76c 100644 (file)
@@ -1,10 +1,10 @@
-NOGCCERROR=yes
-NOCLANGERROR=yes
-CPPFLAGS+= -D_MINIX_SYSTEM
-
 # Makefile for libbdev
 .include <bsd.own.mk>
 
+NOGCCERROR=yes
+
+CPPFLAGS+= -D_MINIX_SYSTEM
+
 LIB=   bdev
 
 SRCS=  bdev.c driver.c call.c ipc.c minor.c
index ca827a7a8cc00b55ae6341a0f5e0151bcef3bcbc..ad4f96c43a49f8bceb304b9011e2cce1d499ebe8 100644 (file)
@@ -59,7 +59,7 @@ int bdev_minor_reopen(dev_t dev)
                }
 
                if (m.m_lblockdriver_lbdev_reply.id != NO_ID) {
-                       printf("bdev: driver (%d) sent invalid ID (%ld)\n",
+                       printf("bdev: driver (%d) sent invalid ID (%d)\n",
                                endpt, m.m_lblockdriver_lbdev_reply.id);
                        return EINVAL;
                }
index e61dc69bcd50b16a8ec708113448226b2b1f80ec..42b9992808589d80b340bdfeb256b44026417131 100644 (file)
@@ -1,10 +1,10 @@
-NOGCCERROR=yes
-NOCLANGERROR=yes
-CPPFLAGS+= -D_MINIX_SYSTEM
-
 # Makefile for libblockdriver
 .include <bsd.own.mk>
 
+NOGCCERROR=yes
+
+CPPFLAGS+= -D_MINIX_SYSTEM
+
 LIB=   blockdriver
 
 SRCS=  driver.c drvlib.c driver_st.c driver_mt.c mq.c trace.c
index a67c766698cd6512302e909e83417edba85b79c7..0c16b9aeba7e16a759a245b0a8bc964b4356ab3e 100644 (file)
@@ -98,7 +98,7 @@ void blockdriver_announce(int type)
   int r;
   char key[DS_MAX_KEYLEN];
   char label[DS_MAX_KEYLEN];
-  char *driver_prefix = "drv.blk.";
+  const char *driver_prefix = "drv.blk.";
 
   /* Callers are allowed to use ipc_sendrec to communicate with drivers.
    * For this reason, there may blocked callers when a driver restarts.
@@ -223,9 +223,9 @@ static int do_vrdwt(struct blockdriver *bdp, message *mp, thread_id_t id)
 {
 /* Carry out an device read or write to/from a vector of buffers. */
   iovec_t iovec[NR_IOREQS];
-  unsigned int nr_req;
+  unsigned int i, nr_req;
   u64_t position;
-  int i, do_write;
+  int do_write;
   ssize_t r, size;
 
   /* Copy the vector from the caller to kernel space. */
@@ -260,7 +260,7 @@ static int do_vrdwt(struct blockdriver *bdp, message *mp, thread_id_t id)
 /*===========================================================================*
  *                             do_dioctl                                    *
  *===========================================================================*/
-static int do_dioctl(struct blockdriver *bdp, dev_t minor,
+static int do_dioctl(struct blockdriver *bdp, devminor_t minor,
   unsigned long request, endpoint_t endpt, cp_grant_id_t grant)
 {
 /* Carry out a disk-specific I/O control request. */
@@ -316,7 +316,7 @@ static int do_ioctl(struct blockdriver *bdp, message *mp)
  * to the tracing module, and handle setting/getting partitions when the driver
  * has specified that it is a disk driver.
  */
-  dev_t minor;
+  devminor_t minor;
   unsigned long request;
   cp_grant_id_t grant;
   endpoint_t user_endpt;
index 66d69485426fbaf4f3c55a5586f8c4d016ec526b..8d82876b1d17d14cef323d7858a5549d25617f3c 100644 (file)
@@ -26,7 +26,7 @@
 #define TID_DEVICE(tid)                ((tid) / MAX_WORKERS)
 #define TID_WORKER(tid)                ((tid) % MAX_WORKERS)
 
-typedef int worker_id_t;
+typedef unsigned int worker_id_t;
 
 typedef enum {
   STATE_DEAD,
@@ -150,7 +150,7 @@ static void *worker_thread(void *param)
   device_t *dp;
   thread_id_t tid;
   message m;
-  int ipc_status, r;
+  int ipc_status;
 
   wp = (worker_t *) param;
   assert(wp != NULL);
@@ -284,7 +284,8 @@ static void master_handle_message(message *m_ptr, int ipc_status)
   device_id_t id;
   worker_t *wp;
   device_t *dp;
-  int r, wid;
+  unsigned int wid;
+  int r;
 
   /* If this is not a block driver request, we cannot get the minor device
    * associated with it, and thus we can not tell which thread should process
@@ -488,7 +489,7 @@ void blockdriver_mt_wakeup(thread_id_t id)
 /*===========================================================================*
  *                             blockdriver_mt_set_workers                   *
  *===========================================================================*/
-void blockdriver_mt_set_workers(device_id_t id, int workers)
+void blockdriver_mt_set_workers(device_id_t id, unsigned int workers)
 {
 /* Set the number of worker threads for the given device.
  */
index 63c2cec31935a0a325bbb07f324c28d5544c910f..f1a048b193f59e571ee334fb383dd831b4fd4e36 100644 (file)
 #include "const.h"
 #include "trace.h"
 
-#define NO_TRACEDEV            ((dev_t) -1)
+#define NO_TRACEDEV            ((devminor_t) -1)
 #define NO_TIME                        ((u32_t) -1)
 
 static int trace_enabled       = FALSE;
-static dev_t trace_dev         = NO_TRACEDEV;
+static devminor_t trace_dev    = NO_TRACEDEV;
 static btrace_entry *trace_buf = NULL;
 static size_t trace_size       = 0;
 static size_t trace_pos;
@@ -48,7 +48,7 @@ static u32_t trace_gettime(void)
 /*===========================================================================*
  *                             trace_ctl                                    *
  *===========================================================================*/
-int trace_ctl(dev_t minor, unsigned long request, endpoint_t endpt,
+int trace_ctl(devminor_t minor, unsigned long request, endpoint_t endpt,
        cp_grant_id_t grant)
 {
 /* Process a block trace control request.
index bb15289d922ec5917ada21a6fae78c738b35b0db..d3e175f90f456bc4b141f61b31c712b76a83d0fc 100644 (file)
@@ -1,7 +1,7 @@
 #ifndef _BLOCKDRIVER_TRACE_H
 #define _BLOCKDRIVER_TRACE_H
 
-int trace_ctl(dev_t minor, unsigned long request, endpoint_t endpt,
+int trace_ctl(devminor_t minor, unsigned long request, endpoint_t endpt,
        cp_grant_id_t grant);
 
 void trace_start(thread_id_t thread_id, message *m_ptr);
index b6997d01f0234ebc98a220c101d698503ff982c6..5c3a87a4f7bcbc1f827edc7f1dc1d55d29c451a7 100644 (file)
@@ -1,10 +1,10 @@
-NOGCCERROR=yes
-NOCLANGERROR=yes
-CPPFLAGS+= -D_MINIX_SYSTEM
-
 # Makefile for libchardriver
 .include <bsd.own.mk>
 
+NOGCCERROR=yes
+
+CPPFLAGS+= -D_MINIX_SYSTEM
+
 LIB=   chardriver
 
 SRCS=  chardriver.c
index 407df37d2bd4d4eb9d1cba8302b7e32ae096570d..7a5a3603c71f04b754892c83b5ae82f63b64aae0 100644 (file)
@@ -102,7 +102,7 @@ void chardriver_announce(void)
   int r;
   char key[DS_MAX_KEYLEN];
   char label[DS_MAX_KEYLEN];
-  char *driver_prefix = "drv.chr.";
+  const char *driver_prefix = "drv.chr.";
 
   /* Callers are allowed to use ipc_sendrec to communicate with drivers.
    * For this reason, there may blocked callers when a driver restarts.
@@ -456,7 +456,7 @@ void chardriver_process(struct chardriver *cdp, message *m_ptr, int ipc_status)
 /* Call the appropiate driver function, based on the type of request. Send a
  * reply to the caller if necessary.
  */
-  int r, reply;
+  int r;
 
   /* Check for notifications first. We never reply to notifications. */
   if (is_ipc_notify(ipc_status)) {
index 49cb3c6593cd328651325331062a12b5b21d506b..f9aa883d34b63172b4a3646a6b0a9cd5d4f720f4 100644 (file)
@@ -1,8 +1,8 @@
-NOGCCERROR=yes
-NOCLANGERROR=yes
 #
 # Makefile for libclkconf
 
+NOGCCERROR=yes
+
 CPPFLAGS+= -D_SYSTEM -D_MINIX_SYSTEM
 
 LIB=   clkconf
index e2ec55bed8e43763ad517551e860882d4f28a600..b4ff41d3884944c0fdf6bbdae1699e35c14b8fbd 100644 (file)
@@ -1,5 +1,6 @@
 NOGCCERROR=yes
-NOCLANGERROR=yes
+
+CPPFLAGS+= -D_MINIX_SYSTEM
 
 LIB = devman
 
index 9e404e81499b180717338ec787d9fb3d078b73bf..e900ca139bb74fb99517b81c19d985a863d8ba1d 100644 (file)
@@ -33,7 +33,7 @@ static int save_string(char *buffer, char *src, size_t *offset)
 /****************************************************************************
  *     serialize_dev                                                        *
  ***************************************************************************/
-void *serialize_dev(struct devman_dev *dev, size_t *overall_size)
+static void *serialize_dev(struct devman_dev *dev, size_t *overall_size)
 {
        /* determine size of serialized version of dev */
        char *buffer;
@@ -179,14 +179,6 @@ int devman_del_device(struct devman_dev *dev)
 
 }
 
-/****************************************************************************
- *     devman_get_ep                                                        *
- ***************************************************************************/
-endpoint_t devman_get_ep(void)
-{
-       return devman_ep;
-}
-
 /****************************************************************************
  *     devman_init                                                          *
  ***************************************************************************/
index 65d0ff347d5a8997c05564684373da4dfa8f1ce7..9b984d36c102e67cfcf3f6134b81b44d696ffb11 100644 (file)
@@ -23,7 +23,7 @@ static int (*unbind_cb) (struct devman_usb_bind_cb_data *data, endpoint_t ep);
  ***************************************************************************/
 static void 
 devman_usb_add_attr
-(struct devman_dev *dev, char *name, char *data)
+(struct devman_dev *dev, const char *name, const char *data)
 {
        struct devman_static_attribute *attr = (struct devman_static_attribute *)
            malloc(sizeof(struct devman_static_attribute));
index fca784aeb15fb3c910e631288cecd65587d5a345..17dcf627a98141f76238c73fa6a85632d1679294 100644 (file)
@@ -56,7 +56,7 @@ static int elf_ph_sane(Elf_Phdr *phdr)
 }
 
 static int elf_unpack(char *exec_hdr,
-       int hdr_len, Elf_Ehdr **hdr, Elf_Phdr **phdr)
+       size_t hdr_len, Elf_Ehdr **hdr, Elf_Phdr **phdr)
 {
   if(hdr_len < sizeof(Elf_Ehdr))
        return ENOEXEC;
@@ -98,7 +98,7 @@ static int check_header(Elf_Ehdr *hdr)
  * Return <0 on error.
  */
 int elf_has_interpreter(char *exec_hdr,                /* executable header */
-               int hdr_len, char *interp, int maxsz)
+               size_t hdr_len, char *interp, size_t maxsz)
 {
   Elf_Ehdr *hdr = NULL;
   Elf_Phdr *phdr = NULL;
index f7e218fd15b848a86e8f6c8086c4e2883c95c8ed..eac0a8e8bcc390c25326d7b758c46436920ce75f 100644 (file)
@@ -52,8 +52,9 @@ struct exec_info {
     vir_bytes stack_high;              /* High stack addr */
 };
 
-int elf_has_interpreter(char *exec_hdr, int hdr_len, char *interp, int maxsz);
-int elf_phdr(char *exec_hdr, int hdr_len, vir_bytes *phdr);
+int elf_has_interpreter(char *exec_hdr, size_t hdr_len, char *interp,
+       size_t maxsz);
+int elf_phdr(char *exec_hdr, size_t hdr_len, vir_bytes *phdr);
 
 int libexec_pm_newexec(endpoint_t proc_e, struct exec_info *execi);
 
index 3949253c7b7ea5a1181d5d1f4042e59b6e5671d5..42f2f80c5e0304422912f17dee5f940dc5d38a34 100644 (file)
@@ -92,7 +92,8 @@ struct fetcherr {
 
 void            fetch_seterr(struct fetcherr *, int);
 void            fetch_syserr(void);
-void            fetch_info(const char *, ...);
+void            fetch_info(const char *, ...)
+                       __attribute__((__format__(__printf__, 1, 2)));
 int             fetch_default_port(const char *);
 int             fetch_default_proxy_port(const char *);
 int             fetch_bind(int, int, const char *);
index d3903b4fa1ff1f53f5df2bcb7d460d51d30ac177..8fd4e00104f5231048afc1850af061b3e3a5010a 100644 (file)
@@ -199,6 +199,9 @@ ftp_chkerr(conn_t *conn)
 /*
  * Send a command and check reply
  */
+static int ftp_cmd(conn_t *conn, const char *fmt, ...)
+       __attribute__((__format__(__printf__, 2, 3)));
+
 #ifndef __minix
 static int
 ftp_cmd(conn_t *conn, const char *fmt, ...)
index 5ff7da749b1ebd27c810668c21cd7e5a3162d7aa..a374d2405da27658ac8121db2bd1733a21075f7a 100644 (file)
@@ -406,6 +406,9 @@ static struct {
 /*
  * Send a formatted line; optionally echo to terminal
  */
+static int http_cmd(conn_t *conn, const char *fmt, ...)
+       __attribute__((__format__(__printf__, 2, 3)));
+
 #ifndef __minix
 static int
 http_cmd(conn_t *conn, const char *fmt, ...)
index 066ecc8abfd7367d72e59d79780e159dc3733868..0fb67e17a8532e2e92a84fc643d7af1ced6ed352 100644 (file)
@@ -1,10 +1,9 @@
+# Makefile for libi2cdriver
+
 NOGCCERROR=yes
-NOCLANGERROR=yes
 
 CPPFLAGS+=-D_MINIX_SYSTEM
 
-# Makefile for libi2cdriver
-
 LIB=   i2cdriver
 
 SRCS=  i2cdriver.c
index 3112ebd5f90693f983a63b36bcd2174f6264184b..df631b225169074e6c40afa3adb61f8d055e6e19 100644 (file)
@@ -15,7 +15,7 @@ i2cdriver_announce(uint32_t bus)
        int r;
        char key[DS_MAX_KEYLEN];
        char label[DS_MAX_KEYLEN];
-       char *driver_prefix = "drv.i2c.";
+       const char *driver_prefix = "drv.i2c.";
 
        /* Callers are allowed to use ipc_sendrec to communicate with drivers.
         * For this reason, there may blocked callers when a driver restarts.
@@ -83,7 +83,7 @@ i2cdriver_bus_endpoint(uint32_t bus)
 {
        /* locate the driver for the i2c bus itself */
        int r;
-       char *label_prefix = "i2c.";
+       const char *label_prefix = "i2c.";
        char label[DS_MAX_KEYLEN];
        endpoint_t bus_endpoint;
 
@@ -197,7 +197,8 @@ static int
 __i2creg_read(endpoint_t bus_endpoint, i2c_addr_t address, uint8_t raw,
     uint8_t reg, uint32_t * val, size_t vallen)
 {
-       int r, i;
+       uint32_t i;
+       int r;
        minix_i2c_ioctl_exec_t ioctl_exec;
 
        assert(val != NULL);
index ac20901fd871e9eb20a7cbd824c8d751466beaf6..fd576e565a8bb92f51f8031f5f7baf67b7049f11 100644 (file)
@@ -1,5 +1,4 @@
 NOGCCERROR=yes
-NOCLANGERROR=yes
 
 LIB = lwip
 
index f8133f76fbc513f6630844729af891fc61b321aa..f4c6d5ea54614e731172cf319b8860881c490aa0 100644 (file)
@@ -716,7 +716,8 @@ netif_loop_output(struct netif *netif, struct pbuf *p,
      netif_poll(). */
 
   /* let last point to the last pbuf in chain r */
-  for (last = r; last->next != NULL; last = last->next);
+  for (last = r; last->next != NULL; last = last->next)
+       ;
 
   SYS_ARCH_PROTECT(lev);
   if(netif->loop_first != NULL) {
index 7d4fc49af1826848c113ba3c2b4fa0849d7189b2..c197efc105a619a0523e58c60c0443b49fafb4bf 100644 (file)
@@ -2,6 +2,8 @@
 #include <minix/sysutil.h>
 #include <errno.h>
 
+#include "lwip/sys.h"
+
 u32_t sys_jiffies(void)
 {
        clock_t ticks;
index 0426137c766152d13b98e0851041b63857fd8abe..bd41e9ebfefcc43d744f3643764586f25a0c32a1 100644 (file)
@@ -1,9 +1,10 @@
+# Makefile for libminixfs
+.include <bsd.own.mk>
+
 NOGCCERROR=yes
-NOCLANGERROR=yes
+
 CPPFLAGS+= -D_MINIX_SYSTEM
 
-# Makefile for libminixfs
-.include <bsd.own.mk>
 LIB=           minixfs
 
 SRCS=          cache.c bio.c
index 8c410981ad882999d5d78747caef4f3a9190dfa4..697b360a9b223119c0d4cfde3d5967c369675dae 100644 (file)
@@ -39,7 +39,7 @@ static struct buf **buf_hash;   /* the buffer hash table */
 static unsigned int nr_bufs;
 static int may_use_vmcache;
 
-static int fs_block_size = PAGE_SIZE;  /* raw i/o block size */
+static size_t fs_block_size = PAGE_SIZE;       /* raw i/o block size */
 
 static int rdwt_err;
 
@@ -106,8 +106,8 @@ void lmfs_blockschange(dev_t dev, int delta)
         static int bitdelta = 0;
         bitdelta += delta;
 #define BANDKB (10*1024)       /* recheck cache every 10MB change */
-        if(bitdelta*fs_block_size/1024 > BANDKB ||
-          bitdelta*fs_block_size/1024 < -BANDKB) {
+        if(bitdelta*(int)fs_block_size/1024 > BANDKB ||
+          bitdelta*(int)fs_block_size/1024 < -BANDKB) {
                 lmfs_cache_reevaluate(dev);
                 bitdelta = 0;
         }
@@ -184,7 +184,7 @@ struct buf *lmfs_get_block(register dev_t dev, register block_t block,
        return lmfs_get_block_ino(dev, block, only_search, VMC_NO_INODE, 0);
 }
 
-void munmap_t(void *a, int len)
+static void munmap_t(void *a, int len)
 {
        vir_bytes av = (vir_bytes) a;
        assert(a);
@@ -627,7 +627,7 @@ void lmfs_rw_scattered(
   static iovec_t iovec[NR_IOREQS];
   off_t pos;
   int iov_per_block;
-  int start_in_use = bufs_in_use, start_bufqsize = bufqsize;
+  unsigned int start_in_use = bufs_in_use, start_bufqsize = bufqsize;
 
   assert(bufqsize >= 0);
   if(bufqsize == 0) return;
index 00b142b1f14fe9913c7ff5f839d84175719bb618..2c670a4ca12ffdb7dd266058201573ca25272cb6 100644 (file)
@@ -1,8 +1,8 @@
+# Makefile for libnetdriver
+
 NOGCCERROR=yes
-NOCLANGERROR=yes
-CPPFLAGS+= -D_MINIX_SYSTEM
 
-# Makefile for libnetdriver
+CPPFLAGS+= -D_MINIX_SYSTEM
 
 LIB=   netdriver
 
index 2b6d0271b96f0c8ade3a080a48624df2c01c823d..51adcf2a54cf9cc3fd63c65601e155434921703a 100644 (file)
@@ -25,7 +25,7 @@ void netdriver_announce()
   int r;
   char key[DS_MAX_KEYLEN];
   char label[DS_MAX_KEYLEN];
-  char *driver_prefix = "drv.net.";
+  const char *driver_prefix = "drv.net.";
 
   /* Publish a driver up event. */
   r = ds_retrieve_label_name(label, sef_self());
index e55e699adda028e42da876e071542a1c50ba17ff..bdf907649a9dda8504e8440789e0c4d2ec6c7876 100644 (file)
@@ -1,5 +1,5 @@
 NOGCCERROR=yes
-NOCLANGERROR=yes
+
 CPPFLAGS+= -D_MINIX_SYSTEM
 
 LIB = netsock
index bef626ec76b3f329e7e8e8895096843a3be36c65..eafaadac03466d66b3838ecc5cc01556930a36b1 100644 (file)
@@ -80,7 +80,7 @@ struct mq {
 
 static struct mq * mq_head, *mq_tail;
 
-int mq_enqueue(struct sock_req * req)
+static int mq_enqueue(struct sock_req * req)
 {
        struct mq * mq;
 
@@ -317,7 +317,7 @@ static int netsock_close(devminor_t minor)
 
 static int netsock_request(struct socket *sock, struct sock_req *req)
 {
-       char *o;
+       const char *o;
 
        /*
         * If an operation is pending (blocking operation) or writing is
index dc5a3c17ca50fbe6a2dbc96c382c922aea128bbc..9bc0782d811d99370079c6297a4138bae8378f6e 100644 (file)
@@ -1,7 +1,8 @@
+# Makefile for libdriver
+
 NOGCCERROR=yes
-NOCLANGERROR=yes
 
-# Makefile for libdriver
+CPPFLAGS+= -D_MINIX_SYSTEM
 
 LIB = usb      
 
index 772c587029ef610eb040bd3e9e3f95cd48595ca9..c7ec9b7946e24bbd923c9f9c5a5a3f4c7da7e53b 100644 (file)
@@ -12,7 +12,7 @@
 static struct usb_urb * pending_urbs = NULL;
 static endpoint_t hcd_ep;
 
-static void _usb_urb_complete(struct usb_driver *ud, long urb_id);
+static void _usb_urb_complete(struct usb_driver *ud, unsigned int urb_id);
 
 /*****************************************************************************
  *         usb_send_urb                                                      *
@@ -154,7 +154,7 @@ int usb_init(char *name)
 /*****************************************************************************
  *      _usb_urb_complete                                                    *
  ****************************************************************************/
-static void _usb_urb_complete(struct usb_driver *ud, long urb_id)
+static void _usb_urb_complete(struct usb_driver *ud, unsigned int urb_id)
 {
        /* find the corresponding URB in the urb_pending list. */
        struct usb_urb * urb = NULL;
@@ -186,7 +186,8 @@ static void _usb_urb_complete(struct usb_driver *ud, long urb_id)
 #endif
                ud->urb_completion(urb);
        } else {
-               printf("WARN: _usb_urb_complete: did not find URB with ID %ld", urb_id);
+               printf("WARN: _usb_urb_complete: did not find URB with ID %u",
+                   urb_id);
        }
 }
 
@@ -210,7 +211,7 @@ int usb_handle_msg(struct usb_driver *ud, message *msg)
 
        switch(msg->m_type) {
                case USB_COMPLETE_URB:
-                       _usb_urb_complete(ud, msg->USB_URB_ID);
+                       _usb_urb_complete(ud, (unsigned int)msg->USB_URB_ID);
                        return 0;
                case USB_ANNOUCE_DEV:
                        ud->connect_device(msg->USB_DEV_ID, msg->USB_INTERFACES);
index 1f131df3953d9a9163b7a85afb94896022b30b55..daf03bccdf80a240b3a03c1431c45d410e13f410 100644 (file)
@@ -1,8 +1,7 @@
-NOGCCERROR=yes
-NOCLANGERROR=yes
-
 # Makefile for libvassert library
 
+NOGCCERROR=yes
+
 MKPIC?=        no
 LIB=    vassert
 SRCS=  backdoor.S vassert.c
index d3d05a8b6b615a75554b5bc255a0a1d0987e837f..f0dde57adce66e1d05ea32c515f6ca60d910b46e 100644 (file)
@@ -61,7 +61,7 @@ void libvassert_process_backdoor(uint32, uint32, uint32, reg_t *, reg_t *,
  *---------------------------------------------------------------------
  */
 
-static void sig_segv(int sig_no)
+static void __dead sig_segv(int sig_no)
 {
    /* jumping to error handling in VAssert_IsInVM. */
    siglongjmp(segv_jmp, 1);
@@ -144,7 +144,7 @@ char VAssert_Init(void)
    libvassert_process_backdoor(CMD_SET_ADDRESS, page_address,
        MAGIC_PORT|(1<<16), &eax, &ebx, &ecx, &edx);
 
-   return (eax != -1) ? 0 : -1;
+   return (eax != (uint32)-1) ? 0 : -1;
 }
 
 
@@ -172,7 +172,7 @@ char VAssert_Uninit(void)
       return -1;
    }
    libvassert_process_backdoor(CMD_SET_ADDRESS, 0, MAGIC_PORT|(0<<16), &eax, &ebx, &ecx, &edx);
-   return (eax != -1) ? 0 : 1;
+   return (eax != (unsigned int)-1) ? 0 : 1;
 }
 
 
index a18c564432087805273737dc4ffa28517aa509fe..0eca9533d392d77dfd7eb315883e5d9896287287 100644 (file)
@@ -72,7 +72,8 @@ extern char VAssert_Uninit(void);
 /*
  * These functions should not be called directly; they need to be wrapped.
  */
-extern void VAssert_LogMain(const char *format, ...);
+extern void VAssert_LogMain(const char *format, ...)
+       __attribute__((__format__(__printf__, 1, 2)));
 extern void VAssert_GoLiveMain(void);
 extern void VAssert_ReturnToReplayMain(void);
 extern char VAssert_Trace(size_t max_size);
index 1df3297dc8bf86691d74f23e1b7cb1701e9df5cd..5a63e91732d92873a9a0a2729ca8f005734667c8 100644 (file)
@@ -1,10 +1,10 @@
-NOGCCERROR=yes
-NOCLANGERROR=yes
-CPPFLAGS+= -D_MINIX_SYSTEM
-
 # Makefile for libvirtio
 .include <bsd.own.mk>
 
+NOGCCERROR=yes
+
+CPPFLAGS+= -D_MINIX_SYSTEM
+
 LIB=   virtio
 
 SRCS=  virtio.c
index c7ae655f9a46f86a09b3968efff01289bdab8c08..1ff715a04c0c20d562708367c632d5921c049836 100644 (file)
@@ -549,7 +549,7 @@ set_indirect_descriptors(struct virtio_device *dev, struct virtio_queue *q,
        vd->len = num * sizeof(desc->descs[0]);
 
        /* Initialize the descriptors in the indirect descriptor table */
-       for (i = 0; i < num; i++) {
+       for (i = 0; i < (int)num; i++) {
                ivd = &desc->descs[i];
 
                use_vring_desc(ivd, &bufs[i]);
index 114f98e373a12fc92fd65f0fb96874c659369d94..fc29d445f9c6d315ce231f9a2c49d9cf6f16dda4 100644 (file)
@@ -133,7 +133,7 @@ static inline void vring_init(struct vring *vr, unsigned int num, void *p,
 {
        vr->num = num;
        vr->desc = p;
-       vr->avail = p + num*sizeof(struct vring_desc);
+       vr->avail = (void *)((char *)p + num*sizeof(struct vring_desc));
        vr->used = (void *)(((unsigned long)&vr->avail->ring[num] + sizeof(u16_t)
                + align-1) & ~(align - 1));
 }
index 379d9b3cb664a425b3fdfd341a097b86a6f8c244..f885652fbc47533fd50598085e248e0c75386aa4 100644 (file)
@@ -6,8 +6,6 @@ CPPFLAGS+= -D_MINIX_SYSTEM
 
 LIB=   vtreefs
 
-CPPFLAGS+= -I${NETBSDSRCDIR}/include
-
 SRCS=  \
        inode.c \
        link.c \