From: Anthony Liguori <aliguori@us.ibm.com>
To: Jan Kiszka <jan.kiszka@siemens.com>
Cc: qemu-devel <qemu-devel@nongnu.org>
Subject: Re: [Qemu-devel] [PATCH] pcnet: Add link state support
Date: Tue, 01 Nov 2011 17:20:22 -0500 [thread overview]
Message-ID: <4EB070A6.1090300@us.ibm.com> (raw)
In-Reply-To: <4E8ED40D.5010105@siemens.com>
On 10/07/2011 05:27 AM, Jan Kiszka wrote:
> Update lnkst on link state changes so that guests can obtain this
> information via reading back the LED output pin. Works for Linux but
> not for guests that depend on the missing PHY.
>
> Signed-off-by: Jan Kiszka<jan.kiszka@siemens.com>
Applied. Thanks.
Regards,
Anthony Liguori
> ---
> hw/lance.c | 1 +
> hw/pcnet-pci.c | 1 +
> hw/pcnet.c | 7 +++++++
> hw/pcnet.h | 1 +
> 4 files changed, 10 insertions(+), 0 deletions(-)
>
> diff --git a/hw/lance.c b/hw/lance.c
> index d83e7f5..93d5fda 100644
> --- a/hw/lance.c
> +++ b/hw/lance.c
> @@ -97,6 +97,7 @@ static NetClientInfo net_lance_info = {
> .size = sizeof(NICState),
> .can_receive = pcnet_can_receive,
> .receive = pcnet_receive,
> + .link_status_changed = pcnet_set_link_status,
> .cleanup = lance_cleanup,
> };
>
> diff --git a/hw/pcnet-pci.c b/hw/pcnet-pci.c
> index cab1116..548bb50 100644
> --- a/hw/pcnet-pci.c
> +++ b/hw/pcnet-pci.c
> @@ -287,6 +287,7 @@ static NetClientInfo net_pci_pcnet_info = {
> .size = sizeof(NICState),
> .can_receive = pcnet_can_receive,
> .receive = pcnet_receive,
> + .link_status_changed = pcnet_set_link_status,
> .cleanup = pci_pcnet_cleanup,
> };
>
> diff --git a/hw/pcnet.c b/hw/pcnet.c
> index add3ec2..cba253b 100644
> --- a/hw/pcnet.c
> +++ b/hw/pcnet.c
> @@ -1197,6 +1197,13 @@ ssize_t pcnet_receive(VLANClientState *nc, const uint8_t *buf, size_t size_)
> return size_;
> }
>
> +void pcnet_set_link_status(VLANClientState *nc)
> +{
> + PCNetState *d = DO_UPCAST(NICState, nc, nc)->opaque;
> +
> + d->lnkst = nc->link_down ? 0 : 0x40;
> +}
> +
> static void pcnet_transmit(PCNetState *s)
> {
> target_phys_addr_t xmit_cxda = 0;
> diff --git a/hw/pcnet.h b/hw/pcnet.h
> index 52cc52e..edc81c9 100644
> --- a/hw/pcnet.h
> +++ b/hw/pcnet.h
> @@ -58,6 +58,7 @@ uint32_t pcnet_ioport_readl(void *opaque, uint32_t addr);
> uint32_t pcnet_bcr_readw(PCNetState *s, uint32_t rap);
> int pcnet_can_receive(VLANClientState *nc);
> ssize_t pcnet_receive(VLANClientState *nc, const uint8_t *buf, size_t size_);
> +void pcnet_set_link_status(VLANClientState *nc);
> void pcnet_common_cleanup(PCNetState *d);
> int pcnet_common_init(DeviceState *dev, PCNetState *s, NetClientInfo *info);
> extern const VMStateDescription vmstate_pcnet;
prev parent reply other threads:[~2011-11-01 22:20 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-10-07 10:27 [Qemu-devel] [PATCH] pcnet: Add link state support Jan Kiszka
2011-10-07 12:06 ` Jan Kiszka
2011-11-01 22:20 ` Anthony Liguori [this message]
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=4EB070A6.1090300@us.ibm.com \
--to=aliguori@us.ibm.com \
--cc=jan.kiszka@siemens.com \
--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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.