remove provider handle refcount

include/dpusm/provider.h

@@ -13,7 +13,6 @@ typedef struct dpusm_provider_handle {
     struct module *module;
     dpusm_pc_t capabilities; /* constant set of capabilities */
     const dpusm_pf_t *funcs; /* reference to a struct */
-    atomic_t refs;           /* how many users are holding this provider */
     struct list_head list;
     struct dpusm_provider_handle *self;
 } dpusm_ph_t;

src/provider.c

@@ -358,11 +358,10 @@ dpusm_provider_get(dpusm_t *dpusm, const char *name) {
             return NULL;
         }
 
-        atomic_inc(&(*provider)->refs);
         atomic_inc(&dpusm->active);
 
         printk("%s: User has been given a handle to \"%s\" (%p) (now %d users).\n",
-               __func__, name, *provider, atomic_read(&(*provider)->refs));
+               __func__, name, *provider, module_refcount(&(*provider)->module));
 
         if ((*provider)->funcs->at_connect) {
             (*provider)->funcs->at_connect();
@@ -385,14 +384,15 @@ dpusm_provider_put(dpusm_t *dpusm, void *handle) {
         return DPUSM_ERROR;
     }
 
-    if (!atomic_read(&(*provider)->refs)) {
+    struct module *module = (*provider)->module;
+
+    if (!module_refcount(module) {
         printk("%s Error: Cannot decrement provider \"%s\" user count already at 0.\n",
-               __func__, module_name((*provider)->module));
+               __func__, module_name(module));
         return DPUSM_ERROR;
     }
 
-    module_put((*provider)->module);
-    atomic_dec(&(*provider)->refs);
+    module_put(module);
     atomic_dec(&dpusm->active);
 
     if ((*provider)->funcs) { /* provider might have been invalidated */
@@ -402,7 +402,7 @@ dpusm_provider_put(dpusm_t *dpusm, void *handle) {
     }
 
     printk("%s: User has returned a handle to \"%s\" (%p) (now %d users).\n",
-           __func__, module_name((*provider)->module), *provider, atomic_read(&(*provider)->refs));
+           __func__, module_name(module), *provider, module_refcount(module));
     return DPUSM_OK;
 }