Skip to contentSkip to conversation

ipmi: setting OS name as Linux in BMC

PATCH Superseded [email protected] 2 participants 4 messages Jun 26, 2012 1 message not in the archive IPMI

v1 of 2 Superseded Latest is v2 →

lore
Share & tools
lore

A message that is not in the archive (sent off-list or not yet received): [email protected]

[email protected]

author
#1 lore raw

PATCH 1/1 ipmi: setting OS name as Linux in BMC

There is an option in IPMI Spec using which BMC can be made aware of the OS name that it is talking to. IPMI_Spec-IPMI2_0E4_061209 - ref - 22.14a , [Set System Info]

This patch will update the OS name as "Linux" in BMC during IPMI Driver initialization. In the current patch, declaration of the OS name is done only for the Volatile param.

Signed-off-by Srinivas Gowda G <[email protected]>
---
 drivers/char/ipmi/ipmi_si_intf.c |   72 ++++++++++++++++++++++++++++++++++++++
 include/linux/ipmi_msgdefs.h     |    1 +
 2 files changed, 73 insertions(+), 0 deletions(-)
drivers/char/ipmi/ipmi_si_intf.c +72 -0
index 1e638ff..df995b4 100644
@@ -2709,6 +2709,71 @@ static int try_get_dev_id(struct smi_info *smi_info)
 	return rv;
 }
 
+/*
+ * Set the Operating System Name as "Linux" using "Set System Info" command.
+ * Only setting Volatile variable - Parameter 4 
+ */
+static int try_set_system_info_os_name(struct smi_info *smi_info)
+{
+	unsigned char         msg[11];
+	unsigned char         *resp;
+	unsigned char         param_select;
+	unsigned char	      set_selector;		
+	unsigned char         string_encode;
+	unsigned char         str_len;
+	unsigned long         resp_len;
+	int                   rv = 0;
+
+	resp = kmalloc(IPMI_MAX_MSG_LENGTH, GFP_KERNEL);
+	if (!resp)
+		return -ENOMEM;
+
+	param_select = 4;	/* parameter number for volatile OS name */
+	set_selector = 0;	/* set selector, block 0 */
+	string_encode = 0;	/* ASCII Encoding */
+	str_len = 5;		/* length of ASCII string - "Linux" */
+
+	msg[0] = IPMI_NETFN_APP_REQUEST << 2;
+	msg[1] = IPMI_SET_SYSTEM_INFO;
+	msg[2] = param_select;
+	msg[3] = set_selector;
+	msg[4] = string_encode;	
+	msg[5] = str_len;
+	msg[6] =  'L';	
+	msg[7] =  'i';	
+	msg[8] =  'n';	
+	msg[9] =  'u';	
+	msg[10] = 'x';	
+
+	smi_info->handlers->start_transaction(smi_info->si_sm, msg, 11);
+
+	rv = wait_for_msg_done(smi_info);
+	if (rv)
+		goto out;
+
+	resp_len = smi_info->handlers->get_result(smi_info->si_sm,
+						  resp, IPMI_MAX_MSG_LENGTH);
+
+	if (resp_len < 3 ||
+			resp[0] != (IPMI_NETFN_APP_REQUEST | 1) << 2 ||
+			resp[1] != IPMI_SET_SYSTEM_INFO   ||
+			resp[2] != 0) {
+		if (resp_len == 3 )
+			 rv = resp[2]; 
+		printk(KERN_WARNING PFX "Failed to set OS name as Linux: 0x%X\n", rv);
+
+		rv = -EINVAL;
+		goto out;
+	}
+	else
+		/* Volatile Opertaing System name */
+		printk(KERN_INFO PFX "OS Name successfully set as Linux\n");
+
+ out:
+	kfree(resp);
+	return rv;
+}
+
 static int try_enable_event_buffer(struct smi_info *smi_info)
 {
 	unsigned char         msg[3];

@@ -3216,6 +3281,13 @@ static int try_smi_init(struct smi_info *new_smi)
 	new_smi->intf_num = smi_num;
 	smi_num++;
 
+	/*
+ 	 * Set the OS in BMC as "Linux" using "Set System Info" Command 
+ 	 */
+	rv = try_set_system_info_os_name(new_smi);
+	if(rv) 
+		printk(KERN_WARNING PFX "Could not set OS Name in BMC\n");
+		
 	rv = try_enable_event_buffer(new_smi);
 	if (rv == 0)
 		new_smi->has_event_buffer = 1;
include/linux/ipmi_msgdefs.h +1 -0
index df97e6e..b5ea664 100644
@@ -57,6 +57,7 @@
 #define IPMI_GET_BMC_GLOBAL_ENABLES_CMD	0x2f
 #define IPMI_READ_EVENT_MSG_BUFFER_CMD	0x35
 #define IPMI_GET_CHANNEL_INFO_CMD	0x42
+#define IPMI_SET_SYSTEM_INFO		0x58
 
 /* Bit for BMC global enables. */
 #define IPMI_BMC_RCV_MSG_INTR     0x01
Signature · 5 lines
1.7.1


Thanks,
G
#2 lore raw

The idea here is fine, but it's probably more appropriate to do this in ipmi_msghandler.c, not ipmi_si_intf.c, since the latter only handles some interface types.

-corey

Quoted 116 lines · On 06/26/2012 08:24 AM, [email protected] wrote:

There is an option in IPMI Spec using which BMC can be made aware of the OS name that it is talking to. IPMI_Spec-IPMI2_0E4_061209 - ref - 22.14a , [Set System Info]

This patch will update the OS name as "Linux" in BMC during IPMI Driver initialization. In the current patch, declaration of the OS name is done only for the Volatile param.

Signed-off-by Srinivas Gowda G<[email protected]>
---
  drivers/char/ipmi/ipmi_si_intf.c |   72 ++++++++++++++++++++++++++++++++++++++
  include/linux/ipmi_msgdefs.h     |    1 +
  2 files changed, 73 insertions(+), 0 deletions(-)
drivers/char/ipmi/ipmi_si_intf.c +72 -0
index 1e638ff..df995b4 100644
@@ -2709,6 +2709,71 @@ static int try_get_dev_id(struct smi_info *smi_info)
  	return rv;
  }
 
+/*
+ * Set the Operating System Name as "Linux" using "Set System Info" command.
+ * Only setting Volatile variable - Parameter 4
+ */
+static int try_set_system_info_os_name(struct smi_info *smi_info)
+{
+	unsigned char         msg[11];
+	unsigned char         *resp;
+	unsigned char         param_select;
+	unsigned char	      set_selector;		
+	unsigned char         string_encode;
+	unsigned char         str_len;
+	unsigned long         resp_len;
+	int                   rv = 0;
+
+	resp = kmalloc(IPMI_MAX_MSG_LENGTH, GFP_KERNEL);
+	if (!resp)
+		return -ENOMEM;
+
+	param_select = 4;	/* parameter number for volatile OS name */
+	set_selector = 0;	/* set selector, block 0 */
+	string_encode = 0;	/* ASCII Encoding */
+	str_len = 5;		/* length of ASCII string - "Linux" */
+
+	msg[0] = IPMI_NETFN_APP_REQUEST<<  2;
+	msg[1] = IPMI_SET_SYSTEM_INFO;
+	msg[2] = param_select;
+	msg[3] = set_selector;
+	msg[4] = string_encode;	
+	msg[5] = str_len;
+	msg[6] =  'L';	
+	msg[7] =  'i';	
+	msg[8] =  'n';	
+	msg[9] =  'u';	
+	msg[10] = 'x';	
+
+	smi_info->handlers->start_transaction(smi_info->si_sm, msg, 11);
+
+	rv = wait_for_msg_done(smi_info);
+	if (rv)
+		goto out;
+
+	resp_len = smi_info->handlers->get_result(smi_info->si_sm,
+						  resp, IPMI_MAX_MSG_LENGTH);
+
+	if (resp_len<  3 ||
+			resp[0] != (IPMI_NETFN_APP_REQUEST | 1)<<  2 ||
+			resp[1] != IPMI_SET_SYSTEM_INFO   ||
+			resp[2] != 0) {
+		if (resp_len == 3 )
+			 rv = resp[2];
+		printk(KERN_WARNING PFX "Failed to set OS name as Linux: 0x%X\n", rv);
+
+		rv = -EINVAL;
+		goto out;
+	}
+	else
+		/* Volatile Opertaing System name */
+		printk(KERN_INFO PFX "OS Name successfully set as Linux\n");
+
+ out:
+	kfree(resp);
+	return rv;
+}
+
  static int try_enable_event_buffer(struct smi_info *smi_info)
  {
  	unsigned char         msg[3];

@@ -3216,6 +3281,13 @@ static int try_smi_init(struct smi_info *new_smi)
  	new_smi->intf_num = smi_num;
  	smi_num++;
 
+	/*
+ 	 * Set the OS in BMC as "Linux" using "Set System Info" Command
+ 	 */
+	rv = try_set_system_info_os_name(new_smi);
+	if(rv)
+		printk(KERN_WARNING PFX "Could not set OS Name in BMC\n");
+		
  	rv = try_enable_event_buffer(new_smi);
  	if (rv == 0)
  		new_smi->has_event_buffer = 1;
include/linux/ipmi_msgdefs.h +1 -0
index df97e6e..b5ea664 100644
@@ -57,6 +57,7 @@
  #define IPMI_GET_BMC_GLOBAL_ENABLES_CMD	0x2f
  #define IPMI_READ_EVENT_MSG_BUFFER_CMD	0x35
  #define IPMI_GET_CHANNEL_INFO_CMD	0x42
+#define IPMI_SET_SYSTEM_INFO		0x58
 
  /* Bit for BMC global enables. */
  #define IPMI_BMC_RCV_MSG_INTR     0x01
author
#3 lore raw
Quoted 121 lines · Corey Minyard

The idea here is fine, but it's probably more appropriate to do this in ipmi_msghandler.c, not ipmi_si_intf.c, since the latter only handles some interface types.

-corey

Quoted 114 lines · On 06/26/2012 08:24 AM, [email protected] wrote:

There is an option in IPMI Spec using which BMC can be made aware of the OS name that it is talking to. IPMI_Spec-IPMI2_0E4_061209 - ref - 22.14a , [Set System Info]

This patch will update the OS name as "Linux" in BMC during IPMI Driver initialization. In the current patch, declaration of the OS name is done only for the Volatile param.

Signed-off-by Srinivas Gowda G<[email protected]>
---
  drivers/char/ipmi/ipmi_si_intf.c |   72 ++++++++++++++++++++++++++++++++++++++
  include/linux/ipmi_msgdefs.h     |    1 +
  2 files changed, 73 insertions(+), 0 deletions(-)
drivers/char/ipmi/ipmi_si_intf.c +72 -0
index 1e638ff..df995b4 100644
@@ -2709,6 +2709,71 @@ static int try_get_dev_id(struct smi_info *smi_info)
  	return rv;
  }
 
+/*
+ * Set the Operating System Name as "Linux" using "Set System Info" command.
+ * Only setting Volatile variable - Parameter 4
+ */
+static int try_set_system_info_os_name(struct smi_info *smi_info)
+{
+	unsigned char         msg[11];
+	unsigned char         *resp;
+	unsigned char         param_select;
+	unsigned char	      set_selector;		
+	unsigned char         string_encode;
+	unsigned char         str_len;
+	unsigned long         resp_len;
+	int                   rv = 0;
+
+	resp = kmalloc(IPMI_MAX_MSG_LENGTH, GFP_KERNEL);
+	if (!resp)
+		return -ENOMEM;
+
+	param_select = 4;	/* parameter number for volatile OS name */
+	set_selector = 0;	/* set selector, block 0 */
+	string_encode = 0;	/* ASCII Encoding */
+	str_len = 5;		/* length of ASCII string - "Linux" */
+
+	msg[0] = IPMI_NETFN_APP_REQUEST<<  2;
+	msg[1] = IPMI_SET_SYSTEM_INFO;
+	msg[2] = param_select;
+	msg[3] = set_selector;
+	msg[4] = string_encode;	
+	msg[5] = str_len;
+	msg[6] =  'L';	
+	msg[7] =  'i';	
+	msg[8] =  'n';	
+	msg[9] =  'u';	
+	msg[10] = 'x';	
+
+	smi_info->handlers->start_transaction(smi_info->si_sm, msg, 11);
+
+	rv = wait_for_msg_done(smi_info);
+	if (rv)
+		goto out;
+
+	resp_len = smi_info->handlers->get_result(smi_info->si_sm,
+						  resp, IPMI_MAX_MSG_LENGTH);
+
+	if (resp_len<  3 ||
+			resp[0] != (IPMI_NETFN_APP_REQUEST | 1)<<  2 ||
+			resp[1] != IPMI_SET_SYSTEM_INFO   ||
+			resp[2] != 0) {
+		if (resp_len == 3 )
+			 rv = resp[2];
+		printk(KERN_WARNING PFX "Failed to set OS name as Linux: 0x%X\n", rv);
+
+		rv = -EINVAL;
+		goto out;
+	}
+	else
+		/* Volatile Opertaing System name */
+		printk(KERN_INFO PFX "OS Name successfully set as Linux\n");
+
+ out:
+	kfree(resp);
+	return rv;
+}
+
  static int try_enable_event_buffer(struct smi_info *smi_info)
  {
  	unsigned char         msg[3];

@@ -3216,6 +3281,13 @@ static int try_smi_init(struct smi_info *new_smi)
  	new_smi->intf_num = smi_num;
  	smi_num++;
 
+	/*
+ 	 * Set the OS in BMC as "Linux" using "Set System Info" Command
+ 	 */
+	rv = try_set_system_info_os_name(new_smi);
+	if(rv)
+		printk(KERN_WARNING PFX "Could not set OS Name in BMC\n");
+		
  	rv = try_enable_event_buffer(new_smi);
  	if (rv == 0)
  		new_smi->has_event_buffer = 1;
include/linux/ipmi_msgdefs.h +1 -0
index df97e6e..b5ea664 100644
@@ -57,6 +57,7 @@
  #define IPMI_GET_BMC_GLOBAL_ENABLES_CMD	0x2f
  #define IPMI_READ_EVENT_MSG_BUFFER_CMD	0x35
  #define IPMI_GET_CHANNEL_INFO_CMD	0x42
+#define IPMI_SET_SYSTEM_INFO		0x58
Quoted 3 lines

include/linux/ipmi_msgdefs.h line 63

Quoted 2 lines
include/linux/ipmi_msgdefs.h +0 -0
  /* Bit for BMC global enables. */
  #define IPMI_BMC_RCV_MSG_INTR     0x01

Corey,
Thanks for the comments.

I was looking at the way BMC presence is detected.
Thought it would be appropriate that BMC is told about the OS information
-- Soon after ipmi_si driver detects BMC(Get_Device_ID) and before the driver starts doing anything useful. Looked like a quick clean getaway..!

Looking at the ipmi_msghandler, I don't see how we will be able to do this during the driver initialization process. I could probably do it inside ipmi_register_smi in ipmi_msghandler. But I still prefer doing this in ipmi_si itself. Please let me know your thoughts on this.

Thanks,
G

#4 lore raw
Quoted 1 line · On 06/26/2012 11:02 AM, [email protected] wrote:

Corey, Thanks for the comments. I was looking at the way BMC presence is detected. Thought it would be appropriate that BMC is told about the OS information -- Soon after ipmi_si driver detects BMC(Get_Device_ID) and before the driver starts doing anything useful. Looked like a quick clean getaway..! Looking at the ipmi_msghandler, I don't see how we will be able to do this during the driver initialization process. I could probably do it inside ipmi_register_smi in ipmi_msghandler. But I still prefer doing this in ipmi_si itself. Please let me know your thoughts on this. Thanks, G

The trouble is that there are other interfaces (that are unfortunately not in the mainstream kernel, but that's a different story) that would not benefit from this if you put it in ipmi_si. That's a low-level interface, this is a higher-level function and belongs.

Look at get_guid() called from ipmi_register_smi() in ipmi_msghandler.c. It's not that hard.

-corey