libvirt_ffi 0.5.6 → 0.6.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: dc49a5a9986110313860c3dd0390c7485e598b4407a52d35009381c1255ca9c5
-  data.tar.gz: 6ebd4be029bba4e8a50dc0efc996d91e357e8d7616b1f64f66dde66503dcfc21
+  metadata.gz: bf7335e98f7be6c9145bad5d2fdc6322b9dda60f31ddb92afd2ce76ebfd95d34
+  data.tar.gz: 2c8fe7a75a8778709860128b466faa04d511ed1eb6eec0913a15bee9202a1d67
 SHA512:
-  metadata.gz: 0d9716eca3d06aa6d0e3479e50c1bb93142968a9b34ad3df59a35a53dfab5b651f59d49e096a89362275700b2cdfd72f6b9832c6bbd187645fb3d236e6478afe
-  data.tar.gz: cac12813a17c8ef4a6bd41ebbed45437d3c3908254866435bcb3bdeb17b457e65c6107c72b4fdd9b553699af2ee8608d2a6a56b6d4bc10ac0437d932e1acd15d
+  metadata.gz: 756e46ad598baa4996961a191398b924059333ea248ae4df26c950a340a8a683e44a3bef80995f07b1d2968e56be3a27965c882dafc891f879114c703ab1cd4c
+  data.tar.gz: eb82fede907f3c40474a894d428da233058ef502e6f0fe891c135c1733fe3bcb02f54df0951188bd0a8c779e1800b309ecc4b985d6ca8e504adbeff13f8143f3

data/lib/libvirt.rb

@@ -2,7 +2,7 @@
 
 require 'ffi'
 require 'objspace'
-require 'libvirt/domain_callback_storage'
+require 'libvirt/host_callback_storage'
 require 'libvirt/util'
 require 'libvirt/errors'
 require 'libvirt/ffi'

data/lib/libvirt/connection.rb

@@ -3,19 +3,37 @@
 module Libvirt
   class Connection
     DOMAIN_EVENT_IDS = FFI::Domain.enum_type(:event_id).symbols.dup.freeze
+    POOL_EVENT_IDS = FFI::Storage.enum_type(:event_id).symbols.dup.freeze
 
-    STORAGE = DomainCallbackStorage.new
+    DOMAIN_STORAGE = HostCallbackStorage.new
+    POOL_STORAGE = HostCallbackStorage.new
 
     DOMAIN_EVENT_CALLBACKS = DOMAIN_EVENT_IDS.map do |event_id_sym|
       func = FFI::Domain.event_callback_for(event_id_sym) do |conn_ptr, dom_ptr, *args, op_ptr|
+        Util.log(:debug, "DOMAIN_EVENT_CALLBACKS[#{event_id_sym}]") do
+          "inside callback conn_ptr=#{conn_ptr}, pool_ptr=#{dom_ptr}, args=#{args}, op_ptr=#{op_ptr}"
+        end
         connection = Connection.load_ref(conn_ptr)
         domain = Domain.load_ref(dom_ptr)
-        block, opaque = STORAGE.retrieve_from_pointer(op_ptr)
+        block, opaque = DOMAIN_STORAGE.retrieve_from_pointer(op_ptr)
         block.call(connection, domain, *args, opaque)
       end
       [event_id_sym, func]
     end.to_h.freeze
 
+    POOL_EVENT_CALLBACKS = POOL_EVENT_IDS.map do |event_id_sym|
+      func = FFI::Storage.event_callback_for(event_id_sym) do |conn_ptr, pool_ptr, *args, op_ptr|
+        Util.log(:debug, "POOL_EVENT_CALLBACKS[#{event_id_sym}]") do
+          "inside callback conn_ptr=#{conn_ptr}, pool_ptr=#{pool_ptr}, args=#{args}, op_ptr=#{op_ptr}"
+        end
+        connection = Connection.load_ref(conn_ptr)
+        pool = StoragePool.load_ref(pool_ptr)
+        block, opaque = POOL_STORAGE.retrieve_from_pointer(op_ptr)
+        block.call(connection, pool, *args, opaque)
+      end
+      [event_id_sym, func]
+    end.to_h.freeze
+
     def self.load_ref(conn_ptr)
       ref_result = FFI::Host.virConnectRef(conn_ptr)
       raise Errors::LibError, "Couldn't retrieve connection reference" if ref_result.negative?
@@ -163,7 +181,7 @@ module Libvirt
       event_id, event_id_sym = Util.parse_enum(enum, event_id)
       cb = DOMAIN_EVENT_CALLBACKS.fetch(event_id_sym)
 
-      cb_data, cb_data_free_func = STORAGE.allocate_struct
+      cb_data, cb_data_free_func = DOMAIN_STORAGE.allocate_struct
 
       result = FFI::Domain.virConnectDomainEventRegisterAny(
           @conn_ptr,
@@ -178,7 +196,7 @@ module Libvirt
         raise Errors::LibError, "Couldn't register domain event callback"
       end
 
-      STORAGE.store_struct(
+      DOMAIN_STORAGE.store_struct(
           cb_data,
           connection_pointer: @conn_ptr,
           callback_id: result,
@@ -195,10 +213,42 @@ module Libvirt
       raise Errors::LibError, "Couldn't deregister domain event callback" if result.negative?
 
       # virConnectDomainEventDeregisterAny will call free func
-      # So we don't need to remove object from STORAGE here.
+      # So we don't need to remove object from DOMAIN_STORAGE here.
       true
     end
 
+    def register_storage_pool_event_callback(event_id, storage_pool = nil, opaque = nil, &block)
+      dbg { "#register_storage_pool_event_callback event_id=#{event_id}" }
+
+      enum = FFI::Storage.enum_type(:event_id)
+      event_id, event_id_sym = Util.parse_enum(enum, event_id)
+      cb = POOL_EVENT_CALLBACKS.fetch(event_id_sym)
+
+      cb_data, cb_data_free_func = POOL_STORAGE.allocate_struct
+
+      result = FFI::Storage.virConnectStoragePoolEventRegisterAny(
+          @conn_ptr,
+          storage_pool&.to_ptr,
+          event_id,
+          cb,
+          cb_data.pointer,
+          cb_data_free_func
+      )
+      if result.negative?
+        cb_data.pointer.free
+        raise Errors::LibError, "Couldn't register storage pool event callback"
+      end
+
+      POOL_STORAGE.store_struct(
+          cb_data,
+          connection_pointer: @conn_ptr,
+          callback_id: result,
+          cb: block,
+          opaque: opaque
+      )
+      result
+    end
+
     def lib_version
       version_ptr = ::FFI::MemoryPointer.new(:ulong)
       result = FFI::Host.virConnectGetLibVersion(@conn_ptr, version_ptr)

data/lib/libvirt/domain.rb

@@ -38,7 +38,7 @@ module Libvirt
     end
 
     def uuid
-      buff = ::FFI::MemoryPointer.new(:char, FFI::Domain::UUID_STRING_BUFLEN)
+      buff = ::FFI::MemoryPointer.new(:char, Util::UUID_STRING_BUFLEN)
       result = FFI::Domain.virDomainGetUUIDString(@dom_ptr, buff)
       raise Errors::LibError, "Couldn't get domain uuid" if result.negative?
 
@@ -46,7 +46,10 @@ module Libvirt
     end
 
     def name
-      FFI::Domain.virDomainGetName(@dom_ptr)
+      result = FFI::Domain.virDomainGetName(@dom_ptr)
+      raise Errors::LibError, "Couldn't retrieve storage pool name" if result.nil?
+
+      result
     end
 
     def max_vcpus

data/lib/libvirt/ffi/domain.rb

@@ -9,8 +9,6 @@ module Libvirt
       extend Helpers
       ffi_lib Util.library_path
 
-      UUID_STRING_BUFLEN = 0x80 # RFC4122
-
       EVENT_ID_TO_CALLBACK = {
           LIFECYCLE: :virConnectDomainEventCallback,
           REBOOT: :virConnectDomainEventGenericCallback,
@@ -812,6 +810,10 @@ module Libvirt
         detail_enum[detail]
       end
 
+      # Converts state reason of domain from integer to symbol name.
+      # @param state [Symbol] enum :state (virDomainState)
+      # @param reason [Integer]
+      # @return [Symbol]
       def self.state_reason(state, reason)
         reason_enum = enum_type(:"#{state.to_s.downcase}_reason")
         reason_enum[reason]

data/lib/libvirt/ffi/host.rb

@@ -31,6 +31,14 @@ module Libvirt
                :threads, :uint
       end
 
+      # enum virConnectCloseReason
+      enum :close_reason, [
+        :ERROR, 0x0, # Misc I/O error
+        :EOF, 0x1, # End-of-file from server
+        :KEEPALIVE, 0x2, # Keepalive timer triggered
+        :CLIENT, 0x3 # Client requested it
+      ]
+
       # int  virGetVersion  (
       #   unsigned long *libVer,
       #   const char *type,
@@ -49,7 +57,7 @@ module Libvirt
       #   int reason,
       #   void * opaque
       # )
-      callback :virConnectCloseFunc, [:pointer, :int, :pointer], :void
+      callback :virConnectCloseFunc, [:pointer, :close_reason, :pointer], :void
 
       # virConnectPtr  virConnectOpen (const char * name)
       attach_function :virConnectOpen, [:string], :pointer

data/lib/libvirt/ffi/storage.rb

@@ -9,6 +9,11 @@ module Libvirt
       extend Helpers
       ffi_lib Util.library_path
 
+      EVENT_ID_TO_CALLBACK = {
+          LIFECYCLE: :virConnectStoragePoolEventLifecycleCallback,
+          REFRESH: :virConnectStoragePoolEventGenericCallback
+      }.freeze
+
       # enum virStoragePoolState
       enum :pool_state, [
           :INACTIVE, 0x0, # Not running
@@ -63,6 +68,23 @@ module Libvirt
           :INACTIVE, 0x1 # dump inactive pool/volume information
       ]
 
+      # enum virStoragePoolEventID
+      enum :event_id, [
+          :LIFECYCLE, 0x0, # virConnectStoragePoolEventLifecycleCallback
+          :REFRESH, 0x1 # virConnectStoragePoolEventGenericCallback
+      ]
+
+      # enum virStoragePoolEventLifecycleType
+      enum :event_lifecycle_type, [
+          :DEFINED, 0x0,
+          :UNDEFINED, 0x1,
+          :STARTED, 0x2,
+          :STOPPED, 0x3,
+          :CREATED, 0x4,
+          :DELETED, 0x5,
+          :LAST, 0x6
+      ]
+
       # struct virStoragePoolInfo {
       #   int   state #   virStoragePoolState flags
       #   unsigned long long   capacity #   Logical size bytes
@@ -87,6 +109,26 @@ module Libvirt
                :allocation, :ulong_long
       end
 
+      # typedef void (*virConnectStoragePoolEventGenericCallback) (
+      #   virConnectPtr conn,
+      #   virStoragePoolPtr pool,
+      #   void * opaque
+      # )
+      callback :virConnectStoragePoolEventGenericCallback,
+               [:pointer, :pointer, :pointer],
+               :void
+
+      # typedef void (*virConnectStoragePoolEventLifecycleCallback) (
+      #   virConnectPtr conn,
+      #   virStoragePoolPtr pool,
+      #   int event,
+      #   int detail,
+      #   void * opaque
+      # )
+      callback :virConnectStoragePoolEventLifecycleCallback,
+               [:pointer, :pointer, :event_lifecycle_type, :int, :pointer],
+               :void
+
       # int  virConnectListAllStoragePools (
       #   virConnectPtr conn,
       #   virStoragePoolPtr ** pools,
@@ -144,6 +186,49 @@ module Libvirt
       #   unsigned int flags
       # )
       attach_function :virStorageVolGetXMLDesc, [:pointer, :xml_flags], :string
+
+      # int  virConnectStoragePoolEventRegisterAny (
+      #   virConnectPtr conn,
+      #   virStoragePoolPtr pool,
+      #   int eventID,
+      #   virConnectStoragePoolEventGenericCallback cb,
+      #   void * opaque,
+      #   virFreeCallback freecb
+      # )
+      attach_function :virConnectStoragePoolEventRegisterAny,
+                      [:pointer, :pointer, :event_id, :pointer, :pointer, FFI::Common::FREE_CALLBACK],
+                      :int
+
+      # int  virConnectStoragePoolEventDeregisterAny (
+      #   virConnectPtr conn,
+      #   int callbackID
+      # )
+      attach_function :virConnectStoragePoolEventDeregisterAny,
+                      [:pointer, :int],
+                      :int
+
+      # int  virStoragePoolGetUUIDString  (virStoragePoolPtr pool,
+      #            char * buf)
+      attach_function :virStoragePoolGetUUIDString, [:pointer, :pointer], :int
+
+      # const char *  virStoragePoolGetName  (virStoragePoolPtr pool)
+      attach_function :virStoragePoolGetName, [:pointer], :string
+
+      module_function
+
+      # Creates event callback function for provided event_id
+      # @param event_id [Integer,Symbol]
+      # @yield connect_ptr, domain_ptr, *args, opaque_ptr
+      # @return [FFI::Function]
+      def event_callback_for(event_id, &block)
+        event_id_sym = event_id.is_a?(Symbol) ? event_id : enum_type(:event_id)[event_id]
+
+        callback_name = EVENT_ID_TO_CALLBACK.fetch(event_id_sym)
+        callback_function(callback_name) do |*args|
+          Util.log(:debug, name) { ".event_callback_for #{event_id_sym} CALLBACK #{args.map(&:to_s).join(', ')}," }
+          block.call(*args)
+        end
+      end
     end
   end
 end

data/lib/libvirt/{domain_callback_storage.rb → host_callback_storage.rb}

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 module Libvirt
-  class DomainCallbackStorage
+  class HostCallbackStorage
     class CallbackDataStruct < ::FFI::Struct
       layout :connection_pointer, :pointer,
              :callback_id, :int
@@ -12,7 +12,7 @@ module Libvirt
     end
 
     # @return [Array<2>]
-    #   cb_data [Libvirt::DomainCallbackStorage::CallbackDataStruct],
+    #   cb_data [Libvirt::HostCallbackStorage::CallbackDataStruct],
     #   cb_data_free_func [FFI::Function]
     def allocate_struct
       dbg { '#allocate_struct' }
@@ -20,7 +20,7 @@ module Libvirt
       cb_data_ptr = ::FFI::MemoryPointer.new(:char, CallbackDataStruct.size, false)
       cb_data = CallbackDataStruct.new(cb_data_ptr)
       cb_data_free_func = FFI::Common.free_function do |pointer|
-        dbg { 'Libvirt::DomainCallbackStorage cb_data_free_func triggered' }
+        dbg { 'Libvirt::HostCallbackStorage cb_data_free_func triggered' }
         remove_struct(pointer)
       end
       [cb_data, cb_data_free_func]
@@ -63,7 +63,7 @@ module Libvirt
     private
 
     def dbg(&block)
-      Util.log(:debug, 'Libvirt::DomainCallbackStorage', &block)
+      Util.log(:debug, 'Libvirt::HostCallbackStorage', &block)
     end
   end
 end

data/lib/libvirt/storage_pool.rb

@@ -61,6 +61,21 @@ module Libvirt
       ptr.get_array_of_pointer(0, size).map { |stv_ptr| StorageVolume.new(stv_ptr) }
     end
 
+    def uuid
+      buff = ::FFI::MemoryPointer.new(:char, Util::UUID_STRING_BUFLEN)
+      result = FFI::Storage.virStoragePoolGetUUIDString(@ptr, buff)
+      raise Errors::LibError, "Couldn't get storage pool uuid" if result.negative?
+
+      buff.read_string
+    end
+
+    def name
+      result = FFI::Storage.virStoragePoolGetName(@ptr)
+      raise Errors::LibError, "Couldn't retrieve storage pool name" if result.nil?
+
+      result
+    end
+
     private
 
     def dbg(&block)

data/lib/libvirt/util.rb

@@ -19,6 +19,8 @@ module Libvirt
         TiB: 1_099_511_627_776
     }.freeze
 
+    UUID_STRING_BUFLEN = 0x80 # RFC4122
+
     class << self
       attr_writer :logger
 

data/lib/libvirt/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Libvirt
-  VERSION = '0.5.6'
+  VERSION = '0.6.0'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: libvirt_ffi
 version: !ruby/object:Gem::Version
-  version: 0.5.6
+  version: 0.6.0
 platform: ruby
 authors:
 - Denis Talakevich
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2020-04-23 00:00:00.000000000 Z
+date: 2020-05-30 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: ffi
@@ -61,7 +61,6 @@ files:
 - lib/libvirt/base_info.rb
 - lib/libvirt/connection.rb
 - lib/libvirt/domain.rb
-- lib/libvirt/domain_callback_storage.rb
 - lib/libvirt/errors.rb
 - lib/libvirt/event.rb
 - lib/libvirt/ffi.rb
@@ -73,6 +72,7 @@ files:
 - lib/libvirt/ffi/host.rb
 - lib/libvirt/ffi/storage.rb
 - lib/libvirt/ffi/stream.rb
+- lib/libvirt/host_callback_storage.rb
 - lib/libvirt/node_info.rb
 - lib/libvirt/storage_pool.rb
 - lib/libvirt/storage_pool_info.rb