context UPDATE store context module hash

Author: michalvasko

src/context.c

@@ -246,6 +246,9 @@ ly_ctx_load_module(struct ly_ctx *ctx, const char *name, const char *revision, c
         lys_unres_glob_erase(&ctx->unres);
     }
 
+    /* new context state */
+    ly_ctx_new_change(ctx);
+
 cleanup:
     if (ret) {
         lys_unres_glob_revert(ctx, &ctx->unres);
@@ -752,13 +755,23 @@ ly_ctx_get_change_count(const struct ly_ctx *ctx)
 
 LIBYANG_API_DEF uint32_t
 ly_ctx_get_modules_hash(const struct ly_ctx *ctx)
+{
+    LY_CHECK_ARG_RET(ctx, ctx, 0);
+
+    return ctx->mod_hash;
+}
+
+void
+ly_ctx_new_change(struct ly_ctx *ctx)
 {
     const struct lys_module *mod;
     uint32_t i = ly_ctx_internal_modules_count(ctx), hash = 0, fi = 0;
     struct lysp_feature *f = NULL;
 
-    LY_CHECK_ARG_RET(ctx, ctx, 0);
+    /* change counter */
+    ctx->change_count++;
 
+    /* module hash */
     while ((mod = ly_ctx_get_module_iter(ctx, &i))) {
         /* name */
         hash = lyht_hash_multi(hash, mod->name, strlen(mod->name));
@@ -779,8 +792,7 @@ ly_ctx_get_modules_hash(const struct ly_ctx *ctx)
         hash = lyht_hash_multi(hash, (char *)&mod->implemented, sizeof mod->implemented);
     }
 
-    hash = lyht_hash_multi(hash, NULL, 0);
-    return hash;
+    ctx->mod_hash = lyht_hash_multi(hash, NULL, 0);
 }
 
 LIBYANG_API_DEF ly_module_imp_clb

src/ly_common.h

@@ -349,6 +349,8 @@ struct ly_ctx {
                                            compilation if ::LY_CTX_EXPLICIT_COMPILE flag is set */
     uint16_t change_count;            /**< count of changes of the context, on some changes it could be incremented
                                            more times */
+    uint32_t mod_hash;                /**< hash of the current context, includes name/revision/enabled features/implement state
+                                           of every loaded module */
     uint16_t flags;                   /**< context settings, see @ref contextoptions */
 
     ly_ext_data_clb ext_clb;          /**< optional callback for providing extension-specific run-time data for extensions */
@@ -360,6 +362,13 @@ struct ly_ctx {
     struct ly_set plugins_extensions; /**< contets specific set of extension plugins */
 };
 
+/**
+ * @brief Record a change of the context, its modules.
+ *
+ * @param[in] ctx Changed context.
+ */
+void ly_ctx_new_change(struct ly_ctx *ctx);
+
 /**
  * @brief Get the (only) implemented YANG module specified by its name.
  *

src/schema_compile.c

@@ -1732,7 +1732,6 @@ lys_compile(struct lys_module *mod, struct lys_depset_unres *unres)
     LYSC_CTX_INIT_PMOD(ctx, sp, NULL);
     ctx.unres = unres;
 
-    ++mod->ctx->change_count;
     mod->compiled = mod_c = calloc(1, sizeof *mod_c);
     LY_CHECK_ERR_RET(!mod_c, LOGMEM(mod->ctx), LY_EMEM);
     mod_c->mod = mod;
@@ -1822,6 +1821,8 @@ lys_compile(struct lys_module *mod, struct lys_depset_unres *unres)
     /* finish compilation for all unresolved module items in the context */
     LY_CHECK_GOTO(ret = lys_compile_unres_mod(&ctx), cleanup);
 
+    ly_ctx_new_change(mod->ctx);
+
 cleanup:
     ly_log_location_revert(0, 0, 1, 0);
     lys_compile_unres_mod_erase(&ctx, ret);

src/tree_schema.c

@@ -846,7 +846,7 @@ _lys_set_implemented(struct lys_module *mod, const char **features, struct lys_g
         }
     }
 
-    /* Try to find module with LYS_MOD_IMPORTED_REV flag. */
+    /* try to find module with LYS_MOD_IMPORTED_REV flag */
     i = 0;
     while ((mod_iter = ly_ctx_get_module_iter(mod->ctx, &i))) {
         if (!strcmp(mod_iter->name, mod->name) && (mod_iter != mod) && (mod_iter->latest_revision & LYS_MOD_IMPORTED_REV)) {
@@ -1198,6 +1198,9 @@ lys_set_implemented(struct lys_module *mod, const char **features)
         lys_unres_glob_erase(unres);
     }
 
+    /* new context state */
+    ly_ctx_new_change(mod->ctx);
+
 cleanup:
     if (ret) {
         lys_unres_glob_revert(mod->ctx, unres);
@@ -2190,7 +2193,6 @@ lys_parse_in(struct ly_ctx *ctx, struct ly_in *in, LYS_INFORMAT format, const st
     /* add into context */
     rc = ly_set_add(&ctx->list, mod, 1, NULL);
     LY_CHECK_GOTO(rc, cleanup);
-    ctx->change_count++;
 
     /* resolve includes and all imports */
     LY_CHECK_GOTO(rc = lysp_resolve_import_include(pctx, mod->parsed, new_mods), cleanup);
@@ -2299,6 +2301,9 @@ lys_parse(struct ly_ctx *ctx, struct ly_in *in, LYS_INFORMAT format, const char
         lys_unres_glob_erase(&ctx->unres);
     }
 
+    /* new context state */
+    ly_ctx_new_change(ctx);
+
 cleanup:
     if (ret) {
         lys_unres_glob_revert(ctx, &ctx->unres);