qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: li guang <lig.fnst@cn.fujitsu.com>
To: qemu-devel@nongnu.org
Cc: Anthony Liguori <anthony@codemonkey.ws>
Subject: Re: [Qemu-devel] [PATCH][RFC 0/14] implement power chip
Date: Mon, 18 Mar 2013 14:12:03 +0800	[thread overview]
Message-ID: <1363587123.21129.138.camel@liguang.fnst.cn.fujitsu.com> (raw)
In-Reply-To: <1363309173.21129.125.camel@liguang.fnst.cn.fujitsu.com>

ping ...


在 2013-03-15五的 08:59 +0800,li guang写道:
> Hi, Anthony
> 
> Sorry to bother you,
> can you please help to see it this work is
> worth to go on, or ...
> 
> or someone else can help to see this patch-set.
> 
> Thanks All.
> 
> 
> 在 2013-03-13三的 16:01 +0800,liguang写道:
> > By now, all devices of QEMU do not have much more
> > power management consideration, for example, if
> > system do suspend, it will call all registered notifiers,
> > this was loosely required, and the code to do power management
> > state transition seems just do 'ugly emulation', rather than be
> > conscious with whole system devices, same condition with reset(it
> > has been embedded in DeviceClass, good!),
> > shutdown, in real world, commonly all devices' power are controlled
> > by a power chip, then all power sequence can be done just
> > issue commands to this chip.
> > so, I come across an idea to implement qdev'ed power device, and
> > make all qdev struct of devices aware of self power management(add
> > on/off/wakeup/suspend ... filed for DeviceClass), this will
> > bring tidy power management, and the emulation will more like what
> > happened in real world.
> > 
> > Of course, it's only a patch-set for RFC, I'd like to ask all 
> > developers to help correct this idea, if it's worth to implement, 
> > I'll go head to refactor more.
> > 
> > Li Guang (14)
> > 	 gitignore: ignore more files
> > 	 qdev: add power management method
> > 	 qdev: remove redundant abort()
> > 	 qdev: add power on/off/suspend/wakeup handler
> > 	 power: add power chip emulation
> > 	 sysemu: remove PowerReason in sysemu.h
> > 	 acpi: refactor acpi wakeup function
> > 	 ich9: make lpc's reset also do pm_reset
> > 	 ich9: do lpc's power on by reset function
> > 	 piix4: refactor piix4's power callbacks
> > 	 pckbd: refactor pckbd's power callbacks
> > 	 ps2: call ps2_{kbd,mouse}_reset in kbd_reset
> > 	 parallel: refactor parallel_reset function
> > 	 uhci: refactor uhci's power callbacks
> > 
> > .gitignore              |   3 +++
> > Makefile.objs           |   1 +
> > hw/acpi.c               |  20 +++++++++-----------
> > hw/acpi.h               |   3 ++-
> > hw/acpi_ich9.c          |   4 ++--
> > hw/ich9.h               |   1 +
> > hw/lpc_ich9.c           |  12 ++++++++++-
> > hw/parallel.c           |  10 ++++++----
> > hw/pckbd.c              |  25 ++++++++++++----------
> > hw/piix4.c              |  14 ++++++++++++--
> > hw/ps2.c                |   8 ++++----
> > hw/ps2.h                |   2 ++
> > hw/qdev-core.h          |  15 ++++++
> > hw/qdev.c               |  99 +--
> > hw/usb/hcd-uhci.c       |  10 ++++++++++
> > include/sysemu/sysemu.h |   7 +------
> > power.c                 | 133 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++
> > power.h                 |  41 +++++++++++++++++
> > 18 files changed, 365 insertions(+), 43 deletions(-)
> >  create mode 100644 power.c
> >  create mode 100644 power.h
> > 
> > 
> > 
> 

  reply	other threads:[~2013-03-18  6:13 UTC|newest]

Thread overview: 37+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-03-13  8:01 [Qemu-devel] [PATCH][RFC 0/14] implement power chip liguang
2013-03-13  8:01 ` [Qemu-devel] [PATCH][RFC 01/14] gitignore: ignore more files liguang
2013-03-21  6:24   ` li guang
2013-03-21 10:43     ` Peter Maydell
2013-03-13  8:01 ` [Qemu-devel] [PATCH][RFC 02/14] qdev: add power management method liguang
2013-03-18  8:25   ` Andreas Färber
2013-03-18  8:29     ` li guang
2013-03-13  8:01 ` [Qemu-devel] [PATCH][RFC 03/14] qdev: remove redundant abort() liguang
2013-03-18  8:26   ` Andreas Färber
2013-03-21  6:24     ` li guang
2013-03-13  8:01 ` [Qemu-devel] [PATCH][RFC 04/14] qdev: add power on/off/suspend/wakeup handler liguang
2013-03-18  8:31   ` Andreas Färber
2013-03-18  8:34     ` li guang
2013-03-13  8:01 ` liguang
2013-03-13  8:01 ` [Qemu-devel] [PATCH][RFC 06/14] sysemu: remove PowerReason in sysemu.h liguang
2013-03-13  8:01 ` [Qemu-devel] [PATCH][RFC 07/14] acpi: refactor acpi wakeup function liguang
2013-03-13  8:01 ` [Qemu-devel] [PATCH][RFC 08/14] ich9: make lpc's reset also do pm_reset liguang
2013-03-13  8:01 ` [Qemu-devel] [PATCH][RFC 09/14] ich9: do lpc's power on by reset function liguang
2013-03-13  8:01 ` [Qemu-devel] [PATCH][RFC 10/14] piix4: refactor piix4's power callbacks liguang
2013-03-13  8:01 ` [Qemu-devel] [PATCH][RFC 11/14] pckbd: refactor pckbd's " liguang
2013-03-13  8:01 ` [Qemu-devel] [PATCH][RFC 12/14] ps2: call ps2_{kbd, mouse}_reset in kbd_reset liguang
2013-03-13  8:01 ` [Qemu-devel] [PATCH][RFC 13/14] parallel: refactor parallel_reset function liguang
2013-03-13  8:01 ` [Qemu-devel] [PATCH][RFC 14/14] uhci: refactor uhci's power callbacks liguang
2013-03-15  0:59 ` [Qemu-devel] [PATCH][RFC 0/14] implement power chip li guang
2013-03-18  6:12   ` li guang [this message]
2013-03-18  8:34 ` Andreas Färber
2013-03-18  8:41   ` li guang
2013-03-18 11:07 ` Peter Maydell
2013-03-19  0:55   ` li guang
2013-03-19  9:05     ` Peter Maydell
2013-03-19  9:31       ` li guang
2013-03-19 10:15         ` Peter Maydell
2013-03-20  0:56           ` li guang
2013-03-20 10:50             ` Peter Maydell
2013-03-21  0:36               ` li guang
2013-03-21 10:41                 ` Peter Maydell
2013-03-22  0:31                   ` li guang

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1363587123.21129.138.camel@liguang.fnst.cn.fujitsu.com \
    --to=lig.fnst@cn.fujitsu.com \
    --cc=anthony@codemonkey.ws \
    --cc=qemu-devel@nongnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).