if no interface is specified, use the dbus interface by default

This is a patch that should only be on openbmc, not sent upstream
because the dbus interface is really only an openbmc thing.

Signed-

if no interface is specified, use the dbus interface by default

This is a patch that should only be on openbmc, not sent upstream
because the dbus interface is really only an openbmc thing.

Signed-off-by: Vernon Mauery <vernon.mauery@linux.intel.com>

show more ...

plugins/dbus: Use default dbus connection

... so we'll use the system bus if run as UID 0.

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

plugins/dbus: Fix ccode response

We need to copy it in from the dbus reply.

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

plugins/dbus: fix missing semicolon

Reported-by: Chris Austen <austenc@us.ibm.com>

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

plugins/dbus: Split netfn from lun, and ccode from data

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

plugins: Add a backend for the OpenBMC dbus interface

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

ID:390 - Support for new Communication Interface (USB Medium)

This feature request is for adding support for USB Medium as an interface in
IPMITool. AMI BMC (OEM feature) provides Virtual USB device

ID:390 - Support for new Communication Interface (USB Medium)

This feature request is for adding support for USB Medium as an interface in
IPMITool. AMI BMC (OEM feature) provides Virtual USB devices in the host as
part of its feature list. IPMITool can use this 'Virtual USB device' as a
medium for Inband communication with BMC.

Just like any other interface, use can just give 'ipmitool -I usb xxxx' to
communicate with the BMC via USB Interface.

show more ...

Remove trailing white-spaces in src/plugins/dummy/dummy.c

ID:384 - Fix compilation under cygwin64

Patch fixes several compile-time errors which appear during IPMITOOL
compilation in Cygwin64 environment.

Commit for: Dmitry Bazhenov

ID:383 - Fix compile-time error in src/plugins/lan/lan.c

Patch fixes an error of incompatible pointer type which appears during LAN
plugin compilation.

Commit for: Dmitry Bazhenov

ID:369 - Fix lanplus interface bridging and response matching

Request/response matching for bridged and double-bridged requests is broken.
This patch reworks the sending and command construction cod

ID:369 - Fix lanplus interface bridging and response matching

Request/response matching for bridged and double-bridged requests is broken.
This patch reworks the sending and command construction code, and fixes the
response matching problems.
Since the polling code is retried several times, it was moved into a separate
function in order to make the code more readable.

Commit for: Dmitry Bazhenov

show more ...

ID:368 - Fix handling of bridging-related parameters

1. my_addr is not set if an interface does not expose set_my_addr.
Currently, the only interface which requires some special handling to set

ID:368 - Fix handling of bridging-related parameters

1. my_addr is not set if an interface does not expose set_my_addr.
Currently, the only interface which requires some special handling to set
my_addr is OpenIPMI. But changing of my_addr still needed for other interfaces.
So, we must set it regardless of presence of set_my_addr().
2. Since set_my_addr() for serial interfaces only sets my_addr, we remove them
as redundand.
3. Bridging is enabled when either trasit_addr or target_addr is not 0.
Currentle transit_addr is not regarded.
4. target_lun does not relate to briging. It is needed for "raw" command. We
set it regardles of bridging.

Commit for: Dmitry Bazhenov

show more ...

ID:367 - Fix building of non-bridged LAN interface commands

When bridging is used (-t specifies an address which is different from 20h),
presession command must go to BMC_SLAVE_ADDR, not to target_a

ID:367 - Fix building of non-bridged LAN interface commands

When bridging is used (-t specifies an address which is different from 20h),
presession command must go to BMC_SLAVE_ADDR, not to target_addr.
BMC_SLAVE_ADDR is always used as the responder address in outer Send Message
request, when message is forwarded. Use the same approach for non-bridged
commands.

Commit for: Dmitry Bazhenov

show more ...

ID:366 - Properly clean LAN and LAN+ interfaces on close

When closing, LAN and LAN+ interfaces do not do proper cleaning of request
entries. Some pointers remain non NULL. This may cause bad memory

ID:366 - Properly clean LAN and LAN+ interfaces on close

When closing, LAN and LAN+ interfaces do not do proper cleaning of request
entries. Some pointers remain non NULL. This may cause bad memory references.

Commit for: Dmitry Bazhenov

show more ...

ID:365 - Fix for ipmitool crash when using serial-terminal interface

Negative value returned by serial_read_line() function was not properly
recognized, since the storage type was unsigned. This cau

ID:365 - Fix for ipmitool crash when using serial-terminal interface

Negative value returned by serial_read_line() function was not properly
recognized, since the storage type was unsigned. This caused ipmitool crash in
some sutuations. This patch fixes the problem.

Commit for: Dmitry Bazhenov

show more ...

ID:364 - Fix for serial-basic interface bridging

Response matching for bridged serial-basic interface request is broken.
This patch fixes the problem.

Commit for: Dmitry Bazhenov

ID:319 - Interface safe re-open

Currently, interface-management code in the ipmitool does not allow safe
interface re-opening (i.e. closing and opening again). It is because the session
is allocated

ID:319 - Interface safe re-open

Currently, interface-management code in the ipmitool does not allow safe
interface re-opening (i.e. closing and opening again). It is because the session
is allocated in the interface setup callback while is freed in the close
callback. So, normal re-opening of the interface, which can be required for
example durng the HPM.1 upgrade, leads to segmentation fault. That's why in the
ipmi_hpmfwupg.c instead of normal closing interface, directly access the
interface data for subsequent re-opening.

Commit for Dmitry Bazhenov

show more ...

ID:373 - Fix compilation of IMB on Windows

Commit fixes compilation of IMB driver under Cygwin.
* don't redefine PAGESIZE if defined already
* fix missing include of <asm/socket.h> - required by IO

ID:373 - Fix compilation of IMB on Windows

Commit fixes compilation of IMB driver under Cygwin.
* don't redefine PAGESIZE if defined already
* fix missing include of <asm/socket.h> - required by IO calls

show more ...

Replace s6_addr16 with s6_addr since Mac OS X does not have it

ID:355 - Fix compiler warnings

Commit removes all unused variables reported by compiler.

ID: 355 - Fix compiler warnings for Dummy Interface

Commit fixes compiler warnings for Dummy Interface:
* fix variable name
* remove unused variables

ID:355 - Fix ``obsolete use of designated initializer with ‘:’'' warning

Commit fixes compiler warning ``obsolete use of designated initializer with
‘:’''. Use of ``fieldname: value'' has been obsol

ID:355 - Fix ``obsolete use of designated initializer with ‘:’'' warning

Commit fixes compiler warning ``obsolete use of designated initializer with
‘:’''. Use of ``fieldname: value'' has been obsolete since GCC 2.5.

show more ...

ID:354 - forcefully switch to C99 and more strict CFLAGS

Commit changes CFLAGS which include C99 and more strict flags. Watch out for
falling rocks.

ID:354 - replace/drop caddr_t in IMB

Commit replaces caddr_t in IMB driver with char *. This is a wild guess
replacement, because caddr_t doesn't have to be char *. However, it shouldn't be
used and

ID:354 - replace/drop caddr_t in IMB

Commit replaces caddr_t in IMB driver with char *. This is a wild guess
replacement, because caddr_t doesn't have to be char *. However, it shouldn't be
used and so shouldn't mmap(). So we'll have to see how things work out.

http://computer-programming-forum.com/47-c-language/556a90938d01f023.htm

show more ...

ID:354 - Replace obsolete u_int[0-9]+_t with standardized types in Free iface