hould we consider employing sync.Once to ensure that Init and Shutdown are executed only once?

[LambdaHJ]

Like this

var initError Return = SUCCESS
var intOnce sync.Once

// nvml.Init()
func Init() Return {
	intOnce.Do(func() {
		lib := dl.New(nvmlLibraryName, nvmlLibraryLoadFlags)
		err := lib.Open()
		if err != nil {
			initError = ERROR_LIBRARY_NOT_FOUND
		}
		nvml = lib
		updateVersionedSymbols()
		nvmlInit()
	})

	return initError
}

[elezar]

@LambdaHJ this changes the behaviour between calling nvmlInit from C directly or using the go bindings. I think with #69 ensuring that multiple calls to dl.New() and Open() return the same underlying handle, we don't need to wrap the entire call in sync.Once.

The call to Init() should always forward to nvmlInit(), but we could consider wrapping the other logic in sync.Once. Feel free to propose a PR for review.

[klueska]

I don't think we want to prevent someone from shutting down and reinitializing NVML without restarting their program. This would be required, for example, if someone were to reload the nvidia kernel module.

[elezar]

@LambdaHJ I believe that with the latest iterations of the bindings and specifically #105, this should no longer be required.

Note that there was a bug in how we were handling the underlying dynamic library and you may have been seeing issues due to this.

I'm closing this issue as resolved, but please reopen it if you're still having problems.