From mboxrd@z Thu Jan  1 00:00:00 1970
From: Mike Wray <mike.wray@hpl.hp.com>
Subject: Re: PATCH: ability to send sysrqs to Linux domains
Date: Thu, 09 Dec 2004 17:43:59 +0000
Message-ID: <41B88EDF.9030906@hpl.hp.com>
References: <cp9drl$qoj$1@sea.gmane.org>
Mime-Version: 1.0
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: 7bit
Return-path: <xen-devel-admin@lists.sourceforge.net>
In-Reply-To: <cp9drl$qoj$1@sea.gmane.org>
Sender: xen-devel-admin@lists.sourceforge.net
Errors-To: xen-devel-admin@lists.sourceforge.net
List-Unsubscribe: <https://lists.sourceforge.net/lists/listinfo/xen-devel>,
	<mailto:xen-devel-request@lists.sourceforge.net?subject=unsubscribe>
List-Post: <mailto:xen-devel@lists.sourceforge.net>
List-Help: <mailto:xen-devel-request@lists.sourceforge.net?subject=help>
List-Subscribe: <https://lists.sourceforge.net/lists/listinfo/xen-devel>,
	<mailto:xen-devel-request@lists.sourceforge.net?subject=subscribe>
List-Archive: <http://sourceforge.net/mailarchive/forum.php?forum=xen-devel>
To: Matthew Bloch <matthew@bytemark.co.uk>
Cc: xen-devel@lists.sourceforge.net
List-Id: xen-devel@lists.xenproject.org

Matthew Bloch wrote:
> Here's the promised patch against a recent 2.01 snapshot to allow you to 
> send sysrqs to running Linux kernels.  I went with Keir's suggestion of 
> adding a sub-reason code to the shutdown message with a 1-byte payload 
> specifying which key was pressed.  Hope it's acceptable for the main 
> tree; I will happily alter it to spec if any of the maintainers have 
> suggestions for better structuring it.

There's a slight problem with the client/server python code.
On the client side you have

def xend_domain_shutdown(self, id, reason, key=None):
         return self.xendPost(self.domainurl(id),
                              {'op'      : 'shutdown',
                               'reason'  : reason,
                               'key'     : key })

while on the server side you have

  fn = FormFn(self.xd.domain_shutdown,
              [['dom', 'str'],
               ['reason', 'str'],
               ['key', 'int']])

which means 'key' has to be an int.
So passing 'None' is going to cause an error.

Mike


-------------------------------------------------------
SF email is sponsored by - The IT Product Guide
Read honest & candid reviews on hundreds of IT Products from real users.
Discover which products truly live up to the hype. Start reading now. 
http://productguide.itmanagersjournal.com/