procon_bypass_man 0.1.20.1 → 0.1.22

data/lib/procon_bypass_man/procon/macro_builder.rb

@@ -13,34 +13,32 @@ class ProconBypassMan::Procon::MacroBuilder
 
   class Subject
     def initialize(value)
-      @button =
-        if match = value.match(/_(\w+)\z/)
-          match[1]
-        else
-          :unknown
-        end
+      if not /^shake_/ =~ value
+        @button =
+          if match = value.match(/_(\w+)\z/)
+            match[1]
+          else
+            :unknown
+          end
+      end
       @type =
         if value.start_with?("toggle_")
           :toggle
+        elsif value.start_with?("shake_left_stick")
+          :shake_left_stick
         else
           :pressing
         end
     end
 
-    def toggle?
-      @type == :toggle
-    end
-
-    def pressing?
-      not toggle?
-    end
-
     def to_steps
       case @type
       when :toggle
         [@button.to_sym, :none]
       when :pressing
         [@button.to_sym, @button.to_sym]
+      when :shake_left_stick
+        [:tilt_left_stick_completely_to_left, :tilt_left_stick_completely_to_right]
       end
     end
   end
@@ -56,7 +54,7 @@ class ProconBypassMan::Procon::MacroBuilder
 
   # @return [Arary<Symbol>]
   def build
-    steps = @steps.map { |step|
+    steps = @steps.flat_map { |step|
       if is_reserved?(step: step) || v1_format?(step: step)
         step.to_sym
       elsif value = build_if_v2_format?(step: step)
@@ -65,7 +63,8 @@ class ProconBypassMan::Procon::MacroBuilder
         nil
       end
     }
-    steps.compact.flatten
+
+    steps.compact
   end
 
   private
@@ -81,31 +80,40 @@ class ProconBypassMan::Procon::MacroBuilder
   end
 
   def build_if_v2_format?(step: )
-    # 時間指定なし
-    if(match = step.match(%r!\Atoggle_(\w+)\z!)) && (button_candidate = match[1]) && is_button(button_candidate)
-      button = button_candidate
-      return [button.to_sym, :none]
-    end
-
-    # 時間指定あり
-    if %r!^(pressing_|toggle_)! =~ step && (subjects = step.scan(%r!pressing_[^_]+|toggle_[^_]+!)) && (match = step.match(%r!_for_([\d_]+)(sec)?\z!))
-      sec = match[1]
-      return [
-        { continue_for: to_num(sec),
-          steps: SubjectMerger.merge(subjects.map { |x| Subject.new(x) }),
-        }
-      ]
-    end
-
     # no-op command
     if(match = step.match(%r!wait_for_([\d_]+)(sec)?\z!))
       sec = match[1]
       return [
-        { continue_for: to_num(sec),
+        { continue_for: to_f(sec),
           steps: [:none],
         }
       ]
     end
+
+    if %r!^(pressing_|toggle_|shake_left_stick_)! =~ step && (subjects = step.scan(%r!pressing_[^_]+|shake_left_stick|toggle_[^_]+!)) && (match = step.match(%r!_for_([\d_]+)(sec)?\z!))
+      if sec = match[1]
+        return {
+          continue_for: to_f(sec),
+          steps: SubjectMerger.merge(subjects.map { |x| Subject.new(x) }).select { |x|
+            if x.is_a?(Array)
+              x.select { |y| is_button(y) || RESERVED_WORD_NONE == y }
+            else
+              is_button(x) || RESERVED_WORD_NONE == x || :tilt_left_stick_completely_to_left == x || :tilt_left_stick_completely_to_right == x
+            end
+          },
+        }
+      end
+    end
+
+    if %r!^(pressing_|toggle_|shake_left_stick_)! =~ step && (subjects = step.scan(%r!pressing_[^_]+|shake_left_stick|toggle_[^_]+!))
+      return SubjectMerger.merge(subjects.map { |x| Subject.new(x) }).select { |x|
+        if x.is_a?(Array)
+          x.select { |y| is_button(y) || RESERVED_WORD_NONE == y }
+        else
+          is_button(x) || RESERVED_WORD_NONE == x
+        end
+      }
+    end
   end
 
   # @return [Boolean]
@@ -113,11 +121,11 @@ class ProconBypassMan::Procon::MacroBuilder
     !!ProconBypassMan::Procon::ButtonCollection::BUTTONS_MAP[step.to_sym]
   end
 
-  def to_num(value)
+  def to_f(value)
     if value.include?("_")
       value.sub("_", ".").to_f
     else
-      value.to_i
+      value.to_f
     end
   end
 end

data/lib/procon_bypass_man/procon/macro_plugin_map.rb

@@ -0,0 +1,23 @@
+module ProconBypassMan
+  class Procon
+    # macroキーにメタデータを埋め込んでいる. 通常の処理ではメタデータを露出したくないので露出しないためのクラス
+    class MacroPluginMap < ::Hash
+      def [](value)
+        self.fetch([value, :normal], nil)
+      end
+
+      def each
+        transform_keys(&:first).each { |x| yield(x[0], x[1]) }
+      end
+
+      alias_method :original_keys, :keys
+      def keys
+        super.map(&:first)
+      end
+
+      def raw_keys
+        self.original_keys
+      end
+    end
+  end
+end

data/lib/procon_bypass_man/procon/macro_registry.rb

@@ -3,28 +3,44 @@ class ProconBypassMan::Procon::MacroRegistry
     null: [],
   }
 
-  def self.install_plugin(klass, steps: nil)
-    if plugins[klass.to_s.to_sym]
-      raise "#{klass} macro is already registered"
+  def self.install_plugin(klass, steps: nil, macro_type: :normal)
+    if plugins.fetch([klass.to_s.to_sym, macro_type], nil)
+      Kernel.warn "#{klass} macro is already registered"
+      return
     end
 
-    plugins[klass.to_s.to_sym] = ->{
-      ProconBypassMan::Procon::MacroBuilder.new(steps || klass.steps).build
-    }
+    plugins.store(
+      [klass.to_s.to_sym, macro_type], ->{
+        ProconBypassMan::Procon::MacroBuilder.new(steps || klass.steps).build
+      }
+    )
   end
 
-  def self.load(name)
-    steps = PRESETS[name] || plugins[name].call || raise("unknown macro")
-    ProconBypassMan::Procon::Macro.new(name: name, steps: steps.dup)
+  # @return [ProconBypassMan::Procon::Macro]
+  def self.load(name, macro_type: :normal, force_neutral_buttons: [], &after_callback_block)
+    if(steps = PRESETS[name] || plugins.fetch([name.to_s.to_sym, macro_type], nil)&.call)
+      return ProconBypassMan::Procon::Macro.new(name: name, steps: steps.dup, force_neutral_buttons: force_neutral_buttons, &after_callback_block)
+    else
+      warn "installされていないマクロ(#{name})を使うことはできません"
+      return self.load(:null)
+    end
   end
 
   def self.reset!
-    ProconBypassMan::ButtonsSettingConfiguration.instance.macro_plugins = {}
+    ProconBypassMan::ButtonsSettingConfiguration.instance.macro_plugins = ProconBypassMan::Procon::MacroPluginMap.new
   end
 
   def self.plugins
     ProconBypassMan::ButtonsSettingConfiguration.instance.macro_plugins
   end
 
+  def self.cleanup_remote_macros!
+    remote_keys = ProconBypassMan::Procon::MacroRegistry.plugins.original_keys.select { |_, y| y == :remote }
+    remote_keys.each do |remote_key|
+      ProconBypassMan::Procon::MacroRegistry.plugins.delete(remote_key)
+    end
+    ProconBypassMan::Procon::MacroRegistry.plugins
+  end
+
   reset!
 end

data/lib/procon_bypass_man/procon/user_operation.rb

@@ -33,22 +33,22 @@ class ProconBypassMan::Procon::UserOperation
     binary.write_as_press_button(button)
   end
 
-  # @param [Symbol, Array<Symbol>] button
-  def press_button_only(button)
-    if button.is_a?(Array)
-      binary.set_no_action!
-      button.uniq.each do |b|
-        unless ProconBypassMan::Procon::MacroBuilder::RESERVED_WORD_NONE == b
-          binary.write_as_press_button(b)
-        end
-      end
-      return
-    end
+  # @param [Symbol, Array<Symbol>] macro_step
+  def press_button_only_or_tilt_sticks(macro_step)
+    macro_step = [macro_step] if not macro_step.is_a?(Array)
+    # スティック操作の時はボタン入力を通す
+    binary.set_no_action! if is_button?(macro_step)
+
+    macro_step.uniq.each do |ms|
+      next if ProconBypassMan::Procon::MacroBuilder::RESERVED_WORD_NONE == ms
 
-    if ProconBypassMan::Procon::MacroBuilder::RESERVED_WORD_NONE == button
-      binary.set_no_action!
-    else
-      binary.write_as_press_button_only(button)
+      if is_button?(ms)
+        binary.write_as_press_button(ms)
+      elsif is_stick?(ms)
+        binary.write_as_tilt_left_stick(ms)
+      else
+        warn "知らないmacro stepです"
+      end
     end
   end
 
@@ -62,7 +62,7 @@ class ProconBypassMan::Procon::UserOperation
   # @param [Symbol] button
   # @return [Boolean]
   def pressing_button?(button)
-    ProconBypassMan::PressButtonAware.new(@binary.raw).pressing_button?(button)
+    pressing_all_buttons?([button])
   end
 
   # @param [Array<Symbol>] buttons
@@ -71,4 +71,18 @@ class ProconBypassMan::Procon::UserOperation
     aware = ProconBypassMan::PressButtonAware.new(@binary.raw)
     buttons.all? { |b| aware.pressing_button?(b) }
   end
+
+  # @return [Boolean]
+  def is_button?(button)
+    button = [button] if not button.is_a?(Array)
+
+    button.all? do |b|
+      !!ProconBypassMan::Procon::ButtonCollection::BUTTONS_MAP[b.to_sym] || b.to_sym == ProconBypassMan::Procon::MacroBuilder::RESERVED_WORD_NONE
+    end
+  end
+
+  # @return [Boolean]
+  def is_stick?(step)
+    !!(step =~ /\Atilt_/)
+  end
 end

data/lib/procon_bypass_man/procon/value_objects/analog_stick.rb

@@ -1,6 +1,6 @@
 class ProconBypassMan::Procon::AnalogStick
   attr_accessor :neutral_position
-  attr_accessor :bin_x, :bin_y
+  attr_writer :bin_x, :bin_y
 
   def initialize(binary: )
     @neutral_position = ProconBypassMan::ButtonsSettingConfiguration.instance.neutral_position
@@ -13,18 +13,22 @@ class ProconBypassMan::Procon::AnalogStick
   end
 
   def abs_x
-    bin_x.to_i(2)
+    @bin_x.to_i(2)
   end
 
   def abs_y
-    bin_y.to_i(2)
+    @bin_y.to_i(2)
   end
 
   def relative_x
-    bin_x.to_i(2) - neutral_position.x
+    @bin_x.to_i(2) - neutral_position.x
   end
 
   def relative_y
-    bin_y.to_i(2) - neutral_position.y
+    @bin_y.to_i(2) - neutral_position.y
+  end
+
+  def relative_hypotenuse
+    Math.sqrt((relative_x**2) + (relative_y**2)).floor(6)
   end
 end

data/lib/procon_bypass_man/procon.rb

@@ -1,3 +1,5 @@
+require "procon_bypass_man/procon/macro_plugin_map"
+
 class ProconBypassMan::Procon
   require "procon_bypass_man/procon/consts"
   require "procon_bypass_man/procon/mode_registry"
@@ -19,6 +21,7 @@ class ProconBypassMan::Procon
       ongoing_macro: MacroRegistry.load(:null),
       ongoing_mode: ModeRegistry.load(:manual),
     }
+    @@left_stick_tilting_power_scaler = ProconBypassMan::AnalogStickTiltingPowerScaler.new
   end
   reset!
 
@@ -38,6 +41,7 @@ class ProconBypassMan::Procon
     ProconBypassMan::ButtonsSettingConfiguration.instance.layers[current_layer_key]
   end
 
+  # 内部ステータスを書き換えるフェーズ
   def apply!
     layer_changer = ProconBypassMan::Procon::LayerChanger.new(binary: user_operation.binary)
     if layer_changer.change_layer?
@@ -46,14 +50,52 @@ class ProconBypassMan::Procon
       return
     end
 
-    if ongoing_macro.finished?
+    analog_stick = ProconBypassMan::Procon::AnalogStick.new(binary: user_operation.binary.raw)
+    dumped_tilting_power = @@left_stick_tilting_power_scaler.add_sample(analog_stick.relative_hypotenuse)
+
+    enable_all_macro = true
+    enable_macro_map = Hash.new {|h,k| h[k] = true }
+    current_layer.disable_macros.each do |disable_macro|
+      if (disable_macro[:if_pressed] == [true] || user_operation.pressing_all_buttons?(disable_macro[:if_pressed]))
+        if disable_macro[:name] == :all
+          enable_all_macro = false
+        else
+          enable_macro_map[disable_macro[:name]] = false
+        end
+      end
+    end
+
+    if ongoing_macro.finished? && enable_all_macro
       current_layer.macros.each do |macro_name, options|
+        next unless enable_macro_map[macro_name]
+
+        if(if_tilted_left_stick_value = options[:if_tilted_left_stick])
+          threshold = (if_tilted_left_stick_value.is_a?(Hash) && if_tilted_left_stick_value[:threshold]) || ProconBypassMan::AnalogStickTiltingPowerScaler::DEFAULT_THRESHOLD
+          if dumped_tilting_power&.tilting?(threshold: threshold, current_position_x: analog_stick.relative_x, current_position_y: analog_stick.relative_y) && user_operation.pressing_all_buttons?(options[:if_pressed])
+            @@status[:ongoing_macro] = MacroRegistry.load(macro_name)
+            break
+          end
+
+          next
+        end
+
         if user_operation.pressing_all_buttons?(options[:if_pressed])
-          @@status[:ongoing_macro] = MacroRegistry.load(macro_name)
+          @@status[:ongoing_macro] = MacroRegistry.load(macro_name, force_neutral_buttons: options[:force_neutral])
+          break
         end
       end
     end
 
+    # remote macro
+    if task = ProconBypassMan::RemoteMacro::TaskQueueInProcess.non_blocking_shift
+      ProconBypassMan::Procon::MacroRegistry.cleanup_remote_macros!
+      macro_name = task.name || "RemoteMacro-#{task.steps.join}".to_sym
+      ProconBypassMan::Procon::MacroRegistry.install_plugin(macro_name, steps: task.steps, macro_type: :remote)
+      @@status[:ongoing_macro] = MacroRegistry.load(macro_name, macro_type: :remote) do
+        ProconBypassMan::PostCompletedRemoteMacroJob.perform_async(task.uuid)
+      end
+    end
+
     case current_layer.mode
     when :manual
       @@status[:ongoing_mode] = ModeRegistry.load(:manual)
@@ -88,15 +130,17 @@ class ProconBypassMan::Procon
     status
   end
 
-  # @return [ProconBypassMan::Domains::ProcessingProconBinary]
+  # @return [String]
   def to_binary
     if ongoing_mode.name != :manual
       return user_operation.binary.raw
     end
 
-    if ongoing_macro.ongoing?
-      step = ongoing_macro.next_step or return(user_operation.binary.raw)
-      user_operation.press_button_only(step)
+    if ongoing_macro.ongoing? && (step = ongoing_macro.next_step)
+      ongoing_macro.force_neutral_buttons&.each do |force_neutral_button|
+        user_operation.unpress_button(force_neutral_button)
+      end
+      user_operation.press_button_only_or_tilt_sticks(step)
       return user_operation.binary.raw
     end
 
@@ -105,7 +149,7 @@ class ProconBypassMan::Procon
     end
 
     current_layer.left_analog_stick_caps.each do |config|
-      if config[:if_pressed].nil? || user_operation.pressing_all_buttons?(config[:if_pressed])
+      if !config[:if_pressed] || user_operation.pressing_all_buttons?(config[:if_pressed])
         config[:force_neutral]&.each do |force_neutral_button|
           user_operation.unpress_button(force_neutral_button)
         end
@@ -143,14 +187,4 @@ class ProconBypassMan::Procon
 
     user_operation.binary.raw
   end
-
-  private
-
-  def method_missing(name)
-    if name.to_s =~ /\Apressed_[a-z]+\?\z/
-      user_operation.public_send(name)
-    else
-      super
-    end
-  end
 end

data/lib/procon_bypass_man/remote_macro/queue_over_process.rb

@@ -0,0 +1,62 @@
+class ProconBypassMan::QueueOverProcess
+  attr_reader :drb
+
+  @@drb_server = nil
+  @@drb_server_thread = nil
+
+  def self.start!
+    return unless ProconBypassMan.config.enable_remote_macro?
+    require 'drb/drb'
+
+    FileUtils.rm_rf(file_path) if File.exist?(file_path)
+    begin
+      @@drb_server = DRb.start_service(url, Queue.new, safe_level: 1)
+    rescue Errno::EADDRINUSE => e
+      ProconBypassMan.logger.error e
+      raise
+    end
+
+    @@drb_server_thread =
+      Thread.new do
+        DRb.thread.join
+      end
+  end
+
+  def self.shutdown
+    if @@drb_server
+      @@drb_server_thread.kill
+      @@drb_server.stop_service
+    end
+  end
+
+  def self.push(value)
+    return unless ProconBypassMan.config.enable_remote_macro?
+
+    drb.push(value)
+  end
+
+  def self.pop
+    return unless ProconBypassMan.config.enable_remote_macro?
+
+    drb.pop
+  end
+
+  def self.drb
+    return unless ProconBypassMan.config.enable_remote_macro?
+
+    @@drb ||= new.drb
+  end
+
+  PROTOCOL = "drbunix"
+  def self.url
+    "#{PROTOCOL}:/tmp/procon_bypass_man_queue"
+  end
+
+  def self.file_path
+    url.gsub("#{PROTOCOL}:", "")
+  end
+
+  def initialize
+    @drb = DRbObject.new_with_uri(self.class.url)
+  end
+end

data/lib/procon_bypass_man/remote_macro/remote_macro_object.rb

@@ -0,0 +1,30 @@
+module ProconBypassMan
+  class RemoteMacroObject
+    # valueobjectがvalidatorの責務も持っている. 今度分離する
+    class ValidationError < StandardError; end
+    class MustBeNotNilError < ValidationError; end
+    class NonSupportAction < ValidationError; end
+
+    attr_accessor :name, :uuid, :steps
+
+    # @param [String] name
+    # @param [String] uuid
+    # @param [Array] steps
+    def initialize(name: , uuid:, steps: )
+      @name = name
+      @uuid = uuid
+      @steps = steps
+      freeze
+    end
+
+    # @raise [MustBeNotNilError]
+    # @raise [NonSupportAction]
+    # @return [void]
+    def validate!
+      self.uuid or raise MustBeNotNilError, "uuidは値が必須です"
+      unless self.steps.is_a?(Array)
+        raise ValidationError, "stepsは配列です"
+      end
+    end
+  end
+end

data/lib/procon_bypass_man/remote_macro/remote_macro_receiver.rb

@@ -0,0 +1,37 @@
+class ProconBypassMan::RemoteMacroReceiver
+  # forkしたプロセスで動かすクラス。sock経由で命令を受け取ってmacoのキューに積んでいく
+  def self.start_with_foreground!
+    return unless ProconBypassMan.config.enable_remote_macro?
+
+    run
+  end
+
+  def self.start!
+    return unless ProconBypassMan.config.enable_remote_macro?
+
+    Thread.start do
+      start_with_foreground!
+    end
+  end
+
+  def self.run
+    while(task = ProconBypassMan::QueueOverProcess.pop)
+      receive(task)
+    end
+    shutdown
+  rescue Errno::ENOENT, Errno::ECONNRESET, Errno::ECONNREFUSED => e
+    ProconBypassMan.logger.debug(e)
+  end
+
+
+  # @param [ProconBypassMan::RemoteMacro::Task] task
+  def self.receive(task)
+    ProconBypassMan.logger.info "[remote macro][receiver] name: #{task.name}, uuid: #{task.uuid}, steps: #{task.steps}"
+    ProconBypassMan::RemoteMacro::TaskQueueInProcess.push(task)
+    true
+  end
+
+  def self.shutdown
+    ProconBypassMan.logger.info("ProconBypassMan::RemoteMacroReceiverを終了します。")
+  end
+end

data/lib/procon_bypass_man/remote_macro/remote_macro_sender.rb

@@ -0,0 +1,8 @@
+class ProconBypassMan::RemoteMacroSender
+  def self.execute(name: , uuid: , steps: )
+    ProconBypassMan.logger.info "[remote macro][sender] name: #{name}, uuid: #{uuid}, steps: #{steps}"
+    ProconBypassMan::QueueOverProcess.push(
+      ProconBypassMan::RemoteMacro::Task.new(name, uuid, steps)
+    )
+  end
+end

data/lib/procon_bypass_man/remote_macro/task.rb

@@ -0,0 +1,6 @@
+module ProconBypassMan
+  module RemoteMacro
+    class Task < Struct.new(:name, :uuid, :steps)
+    end
+  end
+end

data/lib/procon_bypass_man/remote_macro/task_queue.rb

@@ -0,0 +1,13 @@
+module ProconBypassMan
+  module RemoteMacro
+    class TaskQueue < ::Queue
+      def present?
+        not empty?
+      end
+
+      def non_blocking_shift
+        present? && shift
+      end
+    end
+  end
+end

data/lib/procon_bypass_man/remote_macro.rb

@@ -0,0 +1,14 @@
+module ProconBypassMan
+  module RemoteMacro
+    require "procon_bypass_man/remote_macro/remote_macro_object"
+    require "procon_bypass_man/remote_macro/remote_macro_receiver"
+    require "procon_bypass_man/remote_macro/remote_macro_sender"
+    require "procon_bypass_man/remote_macro/queue_over_process"
+    require "procon_bypass_man/remote_macro/task"
+    require "procon_bypass_man/remote_macro/task_queue"
+
+    ACTION_KEY = "remote_macro"
+
+    TaskQueueInProcess = ProconBypassMan::RemoteMacro::TaskQueue.new
+  end
+end

data/lib/procon_bypass_man/remote_pbm_action/change_pbm_version_action.rb

@@ -6,8 +6,11 @@ module ProconBypassMan
         require "pbmenv"
         ProconBypassMan.logger.info "execute ChangePbmVersionAction!"
         pbm_version = args["pbm_version"] or raise(ProconBypassMan::RemotePbmAction::NeedPbmVersionError, "pbm_versionが必要です, #{args.inspect}")
-        Pbmenv.install(pbm_version)
+        Pbmenv.uninstall(pbm_version) # 途中でシャットダウンしてしまった、とか状態が途中の状態かもしれないので一旦消す
+        Pbmenv.install(pbm_version, enable_pbm_cloud: true)
         Pbmenv.use(pbm_version)
+        ProconBypassMan.logger.info "#{pbm_version}へアップグレードしました"
+        ProconBypassMan::ReportCompletedUpgradePbmJob.perform
         `reboot` # symlinkの参照先が変わるのでrebootする必要がある
       end
 

data/lib/procon_bypass_man/remote_pbm_action/value_objects/remote_pbm_action_object.rb

@@ -11,7 +11,7 @@ module ProconBypassMan
     # @param [String] status
     # @param [String] #uuid
     # @param [Time] created_at
-    # @return [Hash] job_args
+    # @param [Hash] job_args
     def initialize(action: , status:, uuid:, created_at:, job_args: )
       self.action = action
       self.status = status

data/lib/procon_bypass_man/remote_pbm_action.rb

@@ -6,6 +6,7 @@ module ProconBypassMan
     require "procon_bypass_man/remote_pbm_action/stop_pbm_action"
     require "procon_bypass_man/remote_pbm_action/restore_pbm_setting.rb"
     require "procon_bypass_man/remote_pbm_action/commands/update_remote_pbm_action_status_command"
+    require "procon_bypass_man/remote_pbm_action/value_objects/remote_pbm_action_object"
 
     ACTION_CHANGE_PBM_VERSION = "change_pbm_version"
     ACTION_REBOOT_OS = "reboot_os"

data/lib/procon_bypass_man/runner.rb

@@ -27,7 +27,11 @@ class ProconBypassMan::Runner
     loop do
       $will_terminate_token = false
       # NOTE メインプロセスではThreadをいくつか起動しているので念のためパフォーマンスを優先するためにforkしていく
-      child_pid = Kernel.fork { ProconBypassMan::BypassCommand.new(gadget: @gadget, procon: @procon).execute }
+      child_pid = Kernel.fork {
+        DRb.start_service if defined?(DRb)
+        ProconBypassMan::RemoteMacroReceiver.start!
+        ProconBypassMan::BypassCommand.new(gadget: @gadget, procon: @procon).execute
+      }
 
       begin
         # TODO 小プロセスが消滅した時に、メインプロセスは生き続けてしまい、何もできなくなる問題がある