OpenE2K/qemu-e2k
13
4
Fork 0
Code Issues 4 Pull Requests Projects 2 Releases Activity

pseries: Add a routine to find a stable "default" vty and use it

Browse Source 
In vty_lookup() we have a special case for supporting early debug in
the kernel. This accepts reg == 0 as a special case to mean "any vty".

We implement this by searching the vtys on the bus and returning the
first we find. This means that the vty we chose depends on the order
the vtys are specified on the QEMU command line - because that determines
the order of the vtys on the bus.

We'd rather the command line order was irrelevant, so instead return
the vty with the lowest reg value. This is still a guess as to what the
user really means, but it is at least stable WRT command line ordering.

Signed-off-by: Michael Ellerman <michael@ellerman.id.au>
Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
Signed-off-by: Alexander Graf <agraf@suse.de>

[agraf] fix braces
This commit is contained in:
David Gibson 2011-12-12 18:24:33 +00:00 committed by Alexander Graf
parent 05c194384f
commit 98331f8ad6
1 changed files with 38 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

47
hw/spapr_vty.c
View File

@ -149,24 +149,53 @@ static VIOsPAPRDeviceInfo spapr_vty = {
}, },
}; };
static VIOsPAPRDevice *spapr_vty_get_default(VIOsPAPRBus *bus)
{
VIOsPAPRDevice *sdev, *selected;
DeviceState *iter;
/*
* To avoid the console bouncing around we want one VTY to be
* the "default". We haven't really got anything to go on, so
* arbitrarily choose the one with the lowest reg value.
*/
selected = NULL;
QTAILQ_FOREACH(iter, &bus->bus.children, sibling) {
/* Only look at VTY devices */
if (iter->info != &spapr_vty.qdev) {
continue;
}
sdev = DO_UPCAST(VIOsPAPRDevice, qdev, iter);
/* First VTY we've found, so it is selected for now */
if (!selected) {
selected = sdev;
continue;
}
/* Choose VTY with lowest reg value */
if (sdev->reg < selected->reg) {
selected = sdev;
}
}
return selected;
}
static VIOsPAPRDevice *vty_lookup(sPAPREnvironment *spapr, target_ulong reg) static VIOsPAPRDevice *vty_lookup(sPAPREnvironment *spapr, target_ulong reg)
{ {
VIOsPAPRDevice *sdev; VIOsPAPRDevice *sdev;
sdev = spapr_vio_find_by_reg(spapr->vio_bus, reg); sdev = spapr_vio_find_by_reg(spapr->vio_bus, reg);
if (!sdev && reg == 0) { if (!sdev && reg == 0) {
DeviceState *qdev;
/* Hack for kernel early debug, which always specifies reg==0. /* Hack for kernel early debug, which always specifies reg==0.
* We search all VIO devices, and grab the first available vty * We search all VIO devices, and grab the vty with the lowest
* device. This attempts to mimic existing PowerVM behaviour * reg. This attempts to mimic existing PowerVM behaviour
* (early debug does work there, despite having no vty with * (early debug does work there, despite having no vty with
* reg==0. */ * reg==0. */
QTAILQ_FOREACH(qdev, &spapr->vio_bus->bus.children, sibling) { return spapr_vty_get_default(spapr->vio_bus);
if (qdev->info == &spapr_vty.qdev) {
return DO_UPCAST(VIOsPAPRDevice, qdev, qdev);
}
}
} }
return sdev; return sdev;