This rather large patch rewrites the virStorageBackendISCSIFindLUNs() function
to only rely on sysfs for finding LUNs, given a session number.  Along the way,
it also fixes the bug where we wouldn't find LUNs for older kernels (with the
block:sda format), and also fixes a race condition where we could try
to find the LUN before udev has finished connecting it.

This patch fixes a few different bugs:
1) We weren't finding LUNs on pre 2.6.24 kernels
2) libvirtd sysfs scanning was racing with udev, so we didn't always see LUNs
3) We weren't showing the 0'th LUN, although that can be a valid LUN

Signed-off-by: Chris Lalancette <clalance@redhat.com>

A small bugfix; we only need to call the iscsiadm sendtarget command when we are
first logging in; we don't need to do it for logout.  Move the sendtarget
command into the Login() function.

Signed-off-by: Chris Lalancette <clalance@redhat.com>

Older versions of iscsiadm didn't support the "-P 0" flag to the "iscsiadm
--mode session" command.  However, just running "iscsiadm --mode session" seems
to work on all version of iscsiadm commands back to FC-6, so just use that.

Signed-off-by: Chris Lalancette <clalance@redhat.com>

In src/util.c, virLog is just a wrapper around fprintf(stderr).  Make sure to
put line breaks at the end of lines that use virLog() (noticed during testing).

Signed-off-by: Chris Lalancette <clalance@redhat.com>

This patch changes things around so that virStorageBackendRunProgRegex() does
*not* virStorageReportError() if the fork()/exec() process it spawned returned a
!= 0 exit code.  Rather, it returns the exitcode in this case, and it is up to
the higher level to determine whether this is a fatal error or not.  The use
case for this change is in the iSCSI stuff; older versions of iscsiadm tools
would return a failure when getting the session number, despite the command
succeeding.

Signed-off-by: Chris Lalancette <clalance@redhat.com>

adding autogen.sh to EXTRA_DIST
* Makefile.am: adding autogen.sh to EXTRA_DIST
Daniel

* src/qemu_conf.c: patch from Kaitlin Rupert, dynamic devices
  for network should be defined in an 'interface' node not 'net'
Daniel

* configure.in NEWS libvirt.spec* doc/* po/*: making libvirt-0.4.3
  release
Daniel

Fix misc compile warnings

Fix saving of iptables rules

Deal with inactive virtual network in domain creation

Adding Kaitlin Rupert in the AUTHORS list
Daniel

* src/qemu_driver.c: patch from Kaitlin Rupert, changing vcpu or
  memory of an active domain is a NO_SUPPORT not INTERNAL_ERROR
Daniel

po regeneration
* po/*: update and regenerated
Daniel

Fix RPM build

* python/libvir.py python/libvirt-python-api.xml: more python
  cleanups by Cole Robinson
Daniel

* configure.in: patch  from Cole Robinson to reactivate storage
  support
Daniel

* python/generator.py python/libvir.c python/libvirt-python-api.xml:
  Apply patch from Cole Robinson fixing UUIDString for python
Daniel

Pass syntax check for XM block devices over to XenD
* src/xm_internal.c: Pass responsibility for checking syntax
  of XM block devices over the XenD (see also Xen cset 17617).
  (Hiroyuki Kaguchi).

virDomainMemoryPeek API
        * include/libvirt/libvirt.h.in, src/libvirt.c, src/driver.h,
          src/libvirt_sym.version: New virDomainMemoryPeek API.
        * qemud/remote.c, qemud/remote_protocol.x, src/remote_internal.c:
          Support for remote.
        * src/qemu_driver.c: QEMU driver implementation of API.
        * src/test.c: Test driver (null) implementation of API.
        * docs/hvsupport.html.in: Document API.
        * libvirt.spec.in: New path /var/cache/libvirt for temporary
          storage of memory images.
        * qemud/libvirtd.init.in: Remove any old temp files in
          /var/cache/libvirt on restarts.
        * src/Makefile.am: make install creates /var/cache/libvirt.
        * configure.in: Detect mkdir -p.

Mon Jun  9 15:42:34 PST 2008 David L. Leskovec <dlesko@linux.vnet.ibm.com>

* src/lxc_driver.c: Make console element is output only.  Always open new
PTY when starting a container.
Fix string overrun when storing console name in VM def struct

Better error messages in xend driver.
* src/xend_internal.c: Change virXendError function to take
varargs.  Better error messages throughout.

Switch over remaining driver code to use memory alloc apis

Switch daemon over to use internal memory allocation APIs

virDomainBlockPeek QEMU and remote support
* qemud/remote.c, qemud/remote_protocol.x, src/remote_internal.c:
  Remote support.
* qemud/remote_dispatch_localvars.h,
  qemud/remote_dispatch_proc_switch.h,
  qemud/remote_dispatch_prototypes.h,
  qemud/remote_protocol.c,
  qemud/remote_protocol.h:
  Generated files for remote support.
* src/xen_unified.c, src/driver.h, src/libvirt.c: Small fix -
  pass flags around internally.
* src/qemu_driver.c: Support for QEMU.
* src/xend_internal.c: Remove redundant fstat call from Xen.

virDomainBlockPeek call
* configure.in: Document AC_SYS_LARGEFILE.
* docs/hvsupport.html.in: Document HV support for virDomainBlockPeek.
* include/libvirt/libvirt.h.in, src/driver.h, src/libvirt.c,
src/libvirt_sym.version: Add virDomainBlockPeek infrastructure.
* src/qemu_driver.c, src/test.c: Null versions of this call.
* src/xen_unified.c, src/xend_internal.c, src/xend_internal.h,
src/xm_internal.c, src/xm_internal.h: Xen implementation.
* tests/sexpr2xmldata/sexpr2xml-curmem.xml,
tests/sexpr2xmldata/sexpr2xml-no-source-cdrom.xml: XML output
has been reordered slightly in the Xen driver, but should be
functionally the same.

Fix a few issues related to restart of libvirtd with containers running.

Mon May 12 23:32:21 PST 2008 David L. Leskovec <dlesko@linux.vnet.ibm.com>

* src/lxc_driver.c: Add sanity of tty pid before kill()
Ignore ECHILD errors during VM cleanup
Call functions to store tty pid and cleanup tty pid file
* src/lxc_conf.h: Add function to verify container process exists
Add facilities to manage storing the tty forward process pid
* src/lxc_conf.c: Add function to verify container process exists
Call function to verify container process during config load
Add facilities to manage storing the tty forward process pid
Call function to load tty pid during load config

* src/xm_internal.c: If vcpu cpuset attribute is invalid
then display an error instead of silently ignoring it
(Hiroyuki Kaguchi).

General tips for contributing patches
* HACKING: Added section "general tips for contributing patches"
to this file.

Only run coverage tests if lcov is installed

Fix make dist

Fix warning with OOM testing is disabled

Switch to using lcov for code coverage repots

Fix parted check & disable storage driver if daemon isn't present

Convert to use new memory allocation APIs

Pass allocation number to callback

Convert to use new memory  allocation APIs

Fix missing check for memory allocation failure

Convert test suites over to new test infrastructure

Misc OOM / memory leak fixes

Generic test suite helpers for OOM testing

Generic OOM testing hooks

Don't add trailing blanks in dominfo output

Fix python code generation for storage APIs

Fix make syntax-check caused by trailing blanks

Fix virsh dominfo test.
* tests/virshdata/dominfo-fc4.txt: 'virsh dominfo' test
matches actual output with autostart status.

Display autostart status in virsh dominfo command.
* src/virsh.c: Display autostart status in virsh dominfo
command (Shigeki Sakamoto).

Fix use of header files in disk storage backend.
* src/storage_backend_disk.c: Include internal.h to resolve use
of header files.

Deprecate the conn, dom and net fields in the _virError structure.
* include/libvirt/libvirt.h.in: Add VIR_DEPRECATED macro.
* src/internal.h: Internal code is allowed to use deprecated
features, so override VIR_DEPRECATED here.
* include/libvirt/virterror.h: Deprecate the conn, dom and net
fields in the _virError structure.

Standardize use of header files, making internal.h primary.
* qemud/internal.h, qemud/qemud.h: Rename this file so it
doesn't conflict with src/internal.h.
* HACKING: Document how header files should be used.
* qemud/Makefile.am: Add src/ directory to includes.
* qemud/event.c, qemud/mdns.c, qemud/qemud.c, qemud/remote.c,
qemud/remote_protocol.c, qemud/remote_protocol.h,
qemud/remote_protocol.x, src/buf.c, src/libvirt.c,
src/nodeinfo.c, src/qemu_conf.c, src/qemu_driver.c,
src/stats_linux.c, src/storage_backend.c, src/storage_backend_fs.c,
src/storage_backend_iscsi.c, src/storage_backend_logical.c,
src/storage_conf.c, src/storage_driver.c, src/util.c,
src/util.h, src/virsh.c, src/virterror.c, src/xend_internal.c,
src/xml.c, tests/reconnect.c, tests/xmlrpctest.c,
tests/qparamtest.c: Standardize use of header files.
* docs/*, po/*: Rebuild docs.

Fix type-punning warning in remote code.
* qemud/rpcgen_fix.pl: Case-insensitive string match on identifiers.
* qemud/remote_protocol.c: Update this generated file.

Fix misc leaks in qparams code, support ; as param separator. Add test suite

Refactor QEMU command line building code for readability

Apply CPU pinning at startup for QEMU guests

Add support for VCPU pinning in QEMU driver

Added missing config.h import

Fix removal of iptable forwarding rules

Remove duplicated error messages

Fix misleading comment about numactl

Quote all args to configure.in macros

Implement NUMA info/apis in QEMU driver

Wire up free memory APIs to remote driver/daemon

Fix misc memory leaks

"copy-paste" errors
* docs/formatdomain.html docs/formatdomain.html.in: Anton Protopopov
  pointed out a few "copy-paste" errors
daniel

Fix exit status check with qemu -help

Fixed couple of memory leaks wrt to virXXXDestroy APIs, and clarify docs to match reality

Don't free C object after destroy method in python

Run qemu with -help when probing for args

Avoid initializing driver if pidfie is claimed. Always claim pidfile as root

Notes on the use of listen_tls/listen_tcp
* docs/remote.html docs/remote.html.in: patch from Kenneth Nagin
  with Notes on the use of listen_tls/listen_tcp
Daniel

plug two leaks

* src/qemu_conf.c (qemudParseXML): Free "obj" unconditionally.

Make QEMU test suite independant of host OS capabilities

* src/xml.c: Include "c-ctype.h".

start using c-ctype functions

Up to now, we've been avoiding ctype functions like isspace, isdigit,
etc.  because they are locale-dependent.  Now that we have the c-ctype
functions, we can start using *them*, to make the code more readable
with changes like these:

-        /* This may not work on EBCDIC. */
-        if ((*p >= 'a' && *p <= 'z') ||
-            (*p >= 'A' && *p <= 'Z') ||
-            (*p >= '0' && *p <= '9'))
+        if (c_isalnum(*p))

-    while ((*cur >= '0') && (*cur <= '9')) {
+    while (c_isdigit(*cur)) {

Also, some macros in conf.c used names that conflicted with
standard meaning of "BLANK" and "SPACE", so I've adjusted them
to be in line with the definition of e.g., isblank.
In addition, I've wrapped those statement macros with do {...} while (0),
so that we can't forget the ";" after a use.  There was one like that
already (fixed below).  The missing semicolon would mess up automatic
indenting.
* src/buf.c (virBufferURIEncodeString):
* src/conf.c (IS_EOL, SKIP_BLANKS_AND_EOL, SKIP_BLANKS)
(virConfParseLong, virConfParseValue, virConfParseName)
(virConfParseSeparator, virConfParseStatement, IS_BLANK, IS_CHAR)
(IS_DIGIT, IS_SPACE, SKIP_SPACES):
* src/nodeinfo.c:
* src/qemu_conf.c (qemudParseInterfaceXML):
* src/qemu_driver.c (qemudDomainBlockStats):
* src/sexpr.c:
* src/stats_linux.c:
* src/util.c (virParseNumber, virDiskNameToIndex):
* src/uuid.c (hextobin, virUUIDParse):
* src/virsh.c:
* src/xml.c (parseCpuNumber, virParseCpuSet):

avoid a double-free bug

* src/qemu_conf.c (qemudParseXML): Ensure that "obj" is either
NULL or a valid malloc'd pointer before we might "goto error"
where it is freed.

Remove use of strcmp()

Fixe default disk bus selection logic

Support Xenner bootloader

Pass -name arg to QEMU when available

Add support for Xen input device in QEMU driver for xenner

Support paravirt disk configuration for xenner guests

avoid using STREQLEN with a literal; use STRPREFIX instead

Likewise for STRNEQLEN -> !STRPREFIX.
* src/nodeinfo.c (linuxNodeInfoCPUPopulate):
* src/qemu_conf.c (qemudNetworkIfaceConnect):
(qemudParseInterfaceXML):
* src/qemu_driver.c (qemudDomainBlockStats):
* src/remote_internal.c (call):
* src/stats_linux.c (xenLinuxDomainDeviceID):
* src/xend_internal.c (xend_parse_sexp_desc):
(xend_get, sexpr_to_xend_topology):
* src/xm_internal.c (xenXMConfigCacheRefresh)
(xenXMDomainFormatXML):

Update Stefan's email addr

Fix default config param

* Makefile.maint (sc_prohibit_strcmp): Also prohibit strncmp.

Change libvirtd description from qemu only to various VM(i.e. Xen, LXC)
Fix typos and comments

Ignore generated test binaries

replace two indentation TABs with spaces

remove trailing blanks

exempt gnulib/ from "make syntax-check" strcmp prohibition

* .x-sc_prohibit_strcmp: New file.
* Makefile.am (EXTRA_DIST): Add .x-sc_prohibit_strcmp.

Added listen_addr config param for daemon (Stefan  de Konink)

Remove all use of strcmp, strncmp in favour of STREQ, STREQLEN, STRPREFIX

avoid "not a string literal..." warnings

* src/qemu_conf.c (qemudParseInterfaceXML): Add "%s".
(qemudBuildCommandLine, qemudGenerateXML): Likewise.

Fix some stray leading spaces in Changelog entry for my last commit

Enable libvirtd drivers to handle signals, in lxc, sigchld triggers vm cleanup
Mon May 12 23:32:21 PST 2008 David L. Leskovec <dlesko@linux.vnet.ibm.com>

* qemud/qemud.c: get siginfo with signals, distribute to drivers that
    register a handler
* src/driver.h: add sighandler function to state driver table
* src/internal.h: define virStateSigDispatcher functions
* src/libvirt.c: add __virStateSigDispatcher function
* src/libvirt_sym.version: add __virStateSigDispatcher
* src/lxc_driver.c: add sig handler function, cleanup vm when sigchld
    received from container process
* src/qemu_driver.c: NULL for sig handler (no handler) in state driver
* src/remote_internal.c: NULL for sig handler (no handler) in state driver
* src/storage_driver.c: NULL for sig handler (no handler) in state driver

Add support for -drive QEMU syntax, and virtio bus / disk type

Add new files from gnulib.

* gnulib/lib/c-ctype.c: New file.
* gnulib/lib/c-ctype.h: Likewise.
* gnulib/tests/test-c-ctype.c: Likewise.
* gnulib/tests/test-vc-list-files-git.sh: Likewise.
* gnulib/tests/test-vc-list-files-cvs.sh: Likewise.

Use gnulib's c-ctype.h, not <ctype.h>.

# Convert uses of isspace to c_isspace, isdigit to c_isdigit, etc.
re=$(man isspace|grep is.....,.is|sed 's/ -.*//' \
  |tr -s ', \n' \||sed 's/^|//;s/|$//')
git grep -l -E "$re"|grep -Ev 'Chan|gnulib' \
  |xargs perl -pi -e 's/\b('"$re"')\b/c_$1/g'
# Remove all uses of to_uchar
git grep -l to_uchar|xargs perl -pi -e 's/to_uchar\((.*?)\)/$1/g'
* src/util.h (to_uchar): Remove definition.
(TOLOWER): Remove definition.
(__virMacAddrCompare): Use c_tolower, not TOLOWER.
Globally:
  Where needed, change <ctype.h> to <c-ctype.h>.
  Remove unnecessary inclusion of <ctype.h>.
Ensure the global changes are never needed again:
* Makefile.maint (sc_avoid_ctype_macros): Prohibit use of ctype
macros.   Recommend c-ctype.h instead.
(sc_prohibit_c_ctype_without_use): New rule.
(sc_prohibit_ctype_h): New rule.  Disallow use of <ctype.h>.

Prepare to use gnulib's c-type module.

* bootstrap: Move module list into separate variable w/less syntax.
(modules): Add c-ctype.
Update the following from gnulib:
* build-aux/useless-if-before-free:
* build-aux/vc-list-files:
* gnulib/lib/Makefile.am:
* gnulib/lib/getaddrinfo.h:
* gnulib/m4/gnulib-cache.m4:
* gnulib/m4/gnulib-comp.m4:
* gnulib/m4/inet_ntop.m4:
* gnulib/tests/Makefile.am:
* gnulib/tests/test-getaddrinfo.c:

autostart for running Xen domains
* src/xen_unified.c src/xend_internal.c src/xend_internal.h: patch
  from Cole Robinson implementing autostart for running Xen domains
Daniel