1. 25 4月, 2013 10 次提交
  2. 24 4月, 2013 19 次提交
  3. 23 4月, 2013 8 次提交
    • D
      Switch to a more extensible annotation system for RPC protocols · a98541bf
      Daniel P. Berrange 提交于
      Currently the RPC protocol files can contain annotations after
      the protocol enum eg
      
         REMOTE_PROC_DOMAIN_SNAPSHOT_LIST_CHILDREN_NAMES = 247, /* autogen autogen priority:high */
      
      This is not very extensible as the number of annotations grows.
      Change it to use
      
          /**
           * @generate: both
           * @priority: high
           */
         REMOTE_PROC_DOMAIN_SNAPSHOT_LIST_CHILDREN_NAMES = 247,
      Signed-off-by: NDaniel P. Berrange <berrange@redhat.com>
      a98541bf
    • P
      qemu: Fix setting of memory tunables · fa006c4f
      Peter Krempa 提交于
      Refactoring done in 19c6ad9a didn't
      correctly take into account the order cgroup limit modification needs to
      be done in. This resulted into errors when decreasing the limits.
      
      The operations need to take place in this order:
      
      decrease hard limit
      change swap hard limit
      
      or
      
      change swap hard limit
      increase hard limit
      
      This patch also fixes the check if the hard_limit is less than
      swap_hard_limit to print better error messages. For this purpose I
      introduced a helper function virCompareLimitUlong to compare limit
      values where value of 0 is equal to unlimited. Additionally the check is
      now applied also when the user does not provide all of the tunables
      through the API and in that case the currently set values are used.
      
      This patch resolves:
      https://bugzilla.redhat.com/show_bug.cgi?id=950478
      fa006c4f
    • J
      fd2e5530
    • J
      qemu: Ignore libvirt logs when reading QEMU error output · 6d1b3edc
      Jiri Denemark 提交于
      When QEMU fails to start, libvirt read its error output and reports it
      back in an error message. However, when libvirtd is configured to log
      debug messages, one would get the following unhelpful garbage:
      
          virsh # start cd
          error: Failed to start domain cd
          error: internal error process exited while connecting to monitor: \
            2013-04-22 14:24:54.214+0000: 2194219: debug : virFileClose:72 : \
            Closed fd 21
          2013-04-22 14:24:54.214+0000: 2194219: debug : virFileClose:72 : \
            Closed fd 27
          2013-04-22 14:24:54.215+0000: 2194219: debug : virFileClose:72 : \
            Closed fd 3
          2013-04-22 14:24:54.215+0000: 2194220: debug : virExec:602 : Run \
            hook 0x7feb8f600bf0 0x7feb86ef9300
          2013-04-22 14:24:54.215+0000: 2194220: debug : qemuProcessHook:2507 \
            : Obtaining domain lock
          2013-04-22 14:24:54.216+0000: 2194220: debug : \
            virDomainLockProcessStart:170 : plugin=0x7feb780261f0 \
            dom=0x7feb7802a360 paused=1 fd=0x7feb86ef8ec4
          2013-04-22 14:24:54.216+0000: 2194220: debug : \
            virDomainLockManagerNew:128 : plugin=0x7feb780261f0 \
            dom=0x7feb7802a360 withResources=1
          2013-04-22 14:24:54.216+0000: 2194220: debug : \
            virLockManagerPluginGetDriver:297 : plugin=0x7feb780261f0
          2013-04-22 14:24:54.216+0000: 2194220: debug : \
            virLockManagerNew:321 : driver=0x7feb8ef08640 type=0 nparams=5 \
            params=0x7feb86ef8d60 flags=0
          2013-04-22 14:24:54.216+000
      
      instead of (the output with this patch applied):
      
          virsh # start cd
          error: Reconnected to the hypervisor
          error: Failed to start domain cd
          error: internal error process exited while connecting to monitor: \
            char device redirected to /dev/pts/33 (label charserial0)
          qemu-system-x86_64: -drive file=/home/vm/systemrescuecd-x86-1.2.0.\
            iso,if=none,id=drive-ide0-1-0,readonly=on,format=raw,cache=none: \
            could not open disk image /home/vm/systemrescuecd-x86-1.2.0.iso: \
            Permission denied
      6d1b3edc
    • J
      e4bdba8d
    • G
      update input ip processing · 1e5306c7
      Gene Czarcinski 提交于
      1. Handle invalid ULong prefix specified.
      When parsing for @prefix as a ULong, a -2 can be returned
      if the specification is not a valid ULong.
      
      2.  Error out if address= is not specified.
      
      3.  Merge netmask process/tests under family tests.
      
      4. Max sure that prefix does not exceed maximum.
      .
      Signed-off-by: NGene Czarcinski <gene@czarc.net>
      1e5306c7
    • G
      create virSocketAddrGetIpPrefix utility function · bd7c7c1b
      Gene Czarcinski 提交于
      Create the utility function virSocketAddrGetIpPrefix() to
      determine the prefix for this network.  The code in this
      function was adapted from virNetworkIpDefPrefix().
      
      Update virNetworkIpDefPrefix() in src/conf/network_conf.c
      to use the new utility function.
      Signed-off-by: NGene Czarcinski <gene@czarc.net>
      bd7c7c1b
    • D
      Replace more cases of /system with /machine · 1e05073f
      Daniel P. Berrange 提交于
      The change in commit aed49863
      was incomplete, missing a couple of cases of /system. This
      caused failure to start VMs.
      Signed-off-by: NDaniel P. Berrange <berrange@redhat.com>
      1e05073f
  4. 22 4月, 2013 3 次提交
    • H
      sheepdog: Omit braces with a single-line body · 0f35e001
      Harry Wei 提交于
      libvirt/HACKING suggests omitting braces with a
      single-line body; this patch fixes the coding style
      problem for the Sheepdog storage backend driver.
      Signed-off-by: NHarry Wei <harryxiyou@gmail.com>
      Signed-off-by: NEric Blake <eblake@redhat.com>
      0f35e001
    • D
      Change default resource partition to /machine · aed49863
      Daniel P. Berrange 提交于
      After discussions with systemd developers it was decided that
      a better default policy for resource partitions is to have
      3 default partitions at the top level
      
         /system   - system services
         /machine - virtual machines / containers
         /user    - user login session
      
      This ensures that the default policy isolates guest from
      user login sessions & system services, so a mis-behaving
      guest can't consume 100% of CPU usage if other things are
      contending for it.
      
      Thus we change the default partition from /system to
      /machine
      Signed-off-by: NDaniel P. Berrange <berrange@redhat.com>
      aed49863
    • O
      qemu: Fix the wrong expression · a71ec988
      Osier Yang 提交于
      Wrong use of the parentheses causes "rc" always having a boolean value,
      either "1" or "0", and thus we can't get the detailed error message
      when it fails:
      
      Before (I only have 1 node):
      % virsh numatune f18 --nodeset 12
      error: Unable to change numa parameters
      error: unable to set numa tunable: Unknown error -1
      
      After:
      virsh numatune f18 --nodeset 12
      error: Unable to change numa parameters
      error: unable to set numa tunable: Invalid argument
      a71ec988