Bug 918296 - add cxoem command to ipmitool
Summary: add cxoem command to ipmitool
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: ipmitool
Version: 18
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Ales Ledvinka
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: 919591 919592
TreeView+ depends on / blocked
 
Reported: 2013-03-05 20:11 UTC by Jeff Bastian
Modified: 2016-09-20 04:33 UTC (History)
5 users (show)

Fixed In Version: ipmitool-1.8.12-16.fc20
Clone Of:
Environment:
Last Closed: 2013-11-05 16:23:01 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)
patch to add cxoem command to ipmitool-1.8.12-9.fc18 (145.85 KB, patch)
2013-03-05 20:11 UTC, Jeff Bastian
no flags Details | Diff
patch to add cxoem command to ipmitool-1.8.12-9.fc18 (125.63 KB, patch)
2013-04-18 22:51 UTC, Jeff Bastian
no flags Details | Diff
cx6: additional patch with latest changes from upstream (40.68 KB, patch)
2013-06-17 13:35 UTC, Jeff Bastian
no flags Details | Diff
cx6 single patch (135.05 KB, patch)
2013-06-17 13:38 UTC, Jeff Bastian
no flags Details | Diff

Description Jeff Bastian 2013-03-05 20:11:55 UTC
Created attachment 705662 [details]
patch to add cxoem command to ipmitool-1.8.12-9.fc18

Description of problem:
Calxeda has added a 'cxoem' command to ipmitool to manage their servers.  For example:

$ ipmitool -I lanplus -H host.example.com -U user -P passwd cxoem info basic
Calxeda SoC (0x0096CD)
  Firmware Version: ECX-1000-v2.1.5
  SoC Version: v1.1.3
  Timestamp (1359478645): Tue Jan 29 10:57:25 2013


The patches are available in their git repo:
  http://sources.calxeda.com/gitweb/?p=ipmitool.git;a=summary

Their patches are based on ipmitool-1.8.11, but Fedora 18 is on 1.8.12, so I've adapted the patches to apply cleanly to the F18 sources.  See attached.

Please add the 'cxoem' command to ipmitool.


Version-Release number of selected component (if applicable):
ipmitool-1.8.12-9.fc18

Comment 1 Jeff Bastian 2013-03-05 21:20:19 UTC
Koji scratch build of the packages:
  http://koji.fedoraproject.org/koji/taskinfo?taskID=5082831

And I've mirrored the build here:
  http://people.redhat.com/~jbastian/ipmitool-cxoem/

Comment 3 Jeff Bastian 2013-03-08 20:27:28 UTC
Note: I've opened 3 new packages reviews, 2 of which depend on the cxoem patch for ipmitool:
  bug 919591: python-pyipmi
  bug 919592: cxmanage

The third is also needed by cxmanage, but it's not dependent on the cxoem patch:
  bug 919590: python-tftpy

Comment 4 Jeff Bastian 2013-04-18 22:51:51 UTC
Created attachment 737460 [details]
patch to add cxoem command to ipmitool-1.8.12-9.fc18

Attached is a slimmer version of the patch to add the cxoem command.

43 commits from Calxeda were included in this version -- only those commits that touched ipmi_cxoem.c or ipmi_cxoem.h -- and 16 commits were excluded.  See the comments on the top of the patch for more details.

Comment 5 Jeff Bastian 2013-06-17 13:35:16 UTC
Created attachment 762027 [details]
cx6: additional patch with latest changes from upstream

This patch builds on top of the patch from comment 4.  It contains the latest upstream changes from the past few months bringing the Fedora package to Calxeda version "cx6".  (The patch in comment 4 was cx3.)

Comment 6 Jeff Bastian 2013-06-17 13:38:29 UTC
Created attachment 762028 [details]
cx6 single patch

Or, to simplify things, this attachment is a single patch that adds all Calxeda changes up through cx6 to Fedora 18 ipmitool-1.8.12-9

Comment 7 Jeff Bastian 2013-06-17 13:40:59 UTC
(In reply to Jeff Bastian from comment #6)
> Or, to simplify things, this attachment is a single patch that adds all
> Calxeda changes up through cx6 to Fedora 18 ipmitool-1.8.12-9

Let me clarify: not *all* Calxeda changes.  Just those their commits that touch ipmi_cxoem.[ch] and related files.

There are some other commits that modify the SEL code and others; see the comments at the top of ipmitool-cxoem-cx3-slim.patch


Note You need to log in before you can comment on or make changes to this bug.