From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:48836) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1WNnYg-0001BA-5B for qemu-devel@nongnu.org; Wed, 12 Mar 2014 14:07:19 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1WNnYb-0005mq-4j for qemu-devel@nongnu.org; Wed, 12 Mar 2014 14:07:14 -0400 Message-ID: <5320A24A.3010105@suse.de> Date: Wed, 12 Mar 2014 19:07:06 +0100 From: =?ISO-8859-15?Q?Andreas_F=E4rber?= MIME-Version: 1.0 References: <1392904246-15575-1-git-send-email-aik@ozlabs.ru> <1392904246-15575-3-git-send-email-aik@ozlabs.ru> In-Reply-To: <1392904246-15575-3-git-send-email-aik@ozlabs.ru> Content-Type: text/plain; charset=ISO-8859-15 Content-Transfer-Encoding: quoted-printable Subject: Re: [Qemu-devel] [PATCH v5 2/6] qdev: introduce FWPathProvider interface List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Alexey Kardashevskiy , qemu-devel@nongnu.org Cc: Paolo Bonzini , qemu-ppc@nongnu.org, Alexander Graf Am 20.02.2014 14:50, schrieb Alexey Kardashevskiy: > QEMU supports firmware names for all devices in the QEMU tree but > some architectures expect some parts of firmware path names in differen= t > format. >=20 > This introduces a firmware-pathname-change interface definition. > If some machines needs to redefine the firmware path format, it has > to add the TYPE_FW_PATH_PROVIDER interface to an object that is above > the device on the QOM tree (typically /machine). >=20 > Signed-off-by: Alexey Kardashevskiy > Signed-off-by: Paolo Bonzini I do not see a reply to my question on v4 and didn't catch bonzini on IRC - guessing the two lines should be swapped since both v4 and v5 were submitted by Alexey? > --- > Changes: > v5: > * fixed code design > * added license headers >=20 > v4: > * added fw-path-provider.o into tests/Makefile > * fixed 80chars warning from checkpatch.pl > --- > hw/core/Makefile.objs | 1 + > hw/core/fw-path-provider.c | 51 +++++++++++++++++++++++++++++++++++= ++++++++ > hw/core/qdev.c | 18 ++++++++++++++- > include/hw/fw-path-provider.h | 49 +++++++++++++++++++++++++++++++++++= ++++++ > tests/Makefile | 1 + > 5 files changed, 119 insertions(+), 1 deletion(-) > create mode 100644 hw/core/fw-path-provider.c > create mode 100644 include/hw/fw-path-provider.h >=20 > diff --git a/hw/core/Makefile.objs b/hw/core/Makefile.objs > index 9e324be..9f75ea3 100644 > --- a/hw/core/Makefile.objs > +++ b/hw/core/Makefile.objs > @@ -1,5 +1,6 @@ > # core qdev-related obj files, also used by *-user: > common-obj-y +=3D qdev.o qdev-properties.o > +common-obj-y +=3D fw-path-provider.o > # irq.o needed for qdev GPIO handling: > common-obj-y +=3D irq.o > common-obj-y +=3D hotplug.o > diff --git a/hw/core/fw-path-provider.c b/hw/core/fw-path-provider.c > new file mode 100644 > index 0000000..b117157 > --- /dev/null > +++ b/hw/core/fw-path-provider.c > @@ -0,0 +1,51 @@ > +/* > + * Firmware patch provider class and helpers. > + * > + * This program is free software; you can redistribute it and/or modi= fy > + * it under the terms of the GNU General Public License as published = by > + * the Free Software Foundation; under version 2 of the License. > + * > + * This program is distributed in the hope that it will be useful, > + * but WITHOUT ANY WARRANTY; without even the implied warranty of > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the > + * GNU General Public License for more details. > + * > + * You should have received a copy of the GNU General Public License > + * along with this program; if not, see . > + */ > + > +#include "hw/fw-path-provider.h" > + > +char *fw_path_provider_get_dev_path(FWPathProvider *p, BusState *bus, > + DeviceState *dev) > +{ > + FWPathProviderClass *k =3D FW_PATH_PROVIDER_GET_CLASS(p); > + > + return k->get_dev_path(p, bus, dev); > +} > + > +char *fw_path_provider_try_get_dev_path(Object *o, BusState *bus, > + DeviceState *dev) > +{ > + FWPathProvider *p =3D (FWPathProvider *) > + object_dynamic_cast(o, TYPE_FW_PATH_PROVIDER); > + > + if (p) { > + return fw_path_provider_get_dev_path(p, bus, dev); > + } > + > + return NULL; > +} > + > +static const TypeInfo fw_path_provider_info =3D { > + .name =3D TYPE_FW_PATH_PROVIDER, > + .parent =3D TYPE_INTERFACE, > + .class_size =3D sizeof(FWPathProviderClass), > +}; > + > +static void fw_path_provider_register_types(void) > +{ > + type_register_static(&fw_path_provider_info); > +} > + > +type_init(fw_path_provider_register_types) > diff --git a/hw/core/qdev.c b/hw/core/qdev.c > index ae30163..dd993b5 100644 > --- a/hw/core/qdev.c > +++ b/hw/core/qdev.c > @@ -26,6 +26,7 @@ > this API directly. */ > =20 > #include "hw/qdev.h" > +#include "hw/fw-path-provider.h" > #include "sysemu/sysemu.h" > #include "qapi/error.h" > #include "qapi/qmp/qerror.h" > @@ -529,6 +530,18 @@ static char *bus_get_fw_dev_path(BusState *bus, De= viceState *dev) > return NULL; > } > =20 > +static char *qdev_get_fw_dev_path_from_handler(BusState *bus, DeviceSt= ate *dev) > +{ > + Object *obj =3D OBJECT(dev); > + char *d =3D NULL; > + > + while (!d && obj->parent) { > + obj =3D obj->parent; > + d =3D fw_path_provider_try_get_dev_path(obj, bus, dev); > + } > + return d; > +} > + > static int qdev_get_fw_dev_path_helper(DeviceState *dev, char *p, int = size) > { > int l =3D 0; > @@ -536,7 +549,10 @@ static int qdev_get_fw_dev_path_helper(DeviceState= *dev, char *p, int size) > if (dev && dev->parent_bus) { > char *d; > l =3D qdev_get_fw_dev_path_helper(dev->parent_bus->parent, p, = size); > - d =3D bus_get_fw_dev_path(dev->parent_bus, dev); > + d =3D qdev_get_fw_dev_path_from_handler(dev->parent_bus, dev); > + if (!d) { > + d =3D bus_get_fw_dev_path(dev->parent_bus, dev); > + } > if (d) { > l +=3D snprintf(p + l, size - l, "%s", d); > g_free(d); > diff --git a/include/hw/fw-path-provider.h b/include/hw/fw-path-provide= r.h > new file mode 100644 > index 0000000..8889d8b > --- /dev/null > +++ b/include/hw/fw-path-provider.h > @@ -0,0 +1,49 @@ > +/* > + * Firmware patch provider class and helpers definitions. > + * > + * This program is free software; you can redistribute it and/or modi= fy > + * it under the terms of the GNU General Public License as published = by > + * the Free Software Foundation; under version 2 of the License. > + * > + * This program is distributed in the hope that it will be useful, > + * but WITHOUT ANY WARRANTY; without even the implied warranty of > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the > + * GNU General Public License for more details. > + * > + * You should have received a copy of the GNU General Public License > + * along with this program; if not, see . > + */ > + > +#ifndef FW_PATH_PROVIDER_H > +#define FW_PATH_PROVIDER_H 1 > + > +#include "qemu-common.h" > +#include "qom/object.h" > + > +#define TYPE_FW_PATH_PROVIDER "fw-path-provider" > + > +#define FW_PATH_PROVIDER_CLASS(klass) \ > + OBJECT_CLASS_CHECK(FWPathProviderClass, (klass), TYPE_FW_PATH_PRO= VIDER) > +#define FW_PATH_PROVIDER_GET_CLASS(obj) \ > + OBJECT_GET_CLASS(FWPathProviderClass, (obj), TYPE_FW_PATH_PROVIDER= ) > +#define FW_PATH_PROVIDER(obj) \ > + INTERFACE_CHECK(FWPathProvider, (obj), TYPE_FW_PATH_PROVIDER) > + > +typedef struct FWPathProvider { > + Object parent_obj; > +} FWPathProvider; Unused? > + > +typedef void (*StreamCanPushNotifyFn)(void *opaque); Unused? Hoping we can consider this a bugfix and get done during freeze. Andreas > + > +typedef struct FWPathProviderClass { > + InterfaceClass parent_class; > + > + char *(*get_dev_path)(FWPathProvider *p, BusState *bus, DeviceStat= e *dev); > +} FWPathProviderClass; > + > +char *fw_path_provider_get_dev_path(FWPathProvider *p, BusState *bus, > + DeviceState *dev); > +char *fw_path_provider_try_get_dev_path(Object *o, BusState *bus, > + DeviceState *dev); > + > +#endif /* FW_PATH_PROVIDER_H */ > diff --git a/tests/Makefile b/tests/Makefile > index 9a7d2f1..0692b11 100644 > --- a/tests/Makefile > +++ b/tests/Makefile > @@ -165,6 +165,7 @@ tests/test-int128$(EXESUF): tests/test-int128.o > tests/test-qdev-global-props$(EXESUF): tests/test-qdev-global-props.o = \ > hw/core/qdev.o hw/core/qdev-properties.o hw/core/hotplug.o\ > hw/core/irq.o \ > + hw/core/fw-path-provider.o \ > $(qom-core-obj) \ > $(test-qapi-obj-y) \ > libqemuutil.a libqemustub.a >=20 --=20 SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 N=FCrnberg, Germany GF: Jeff Hawn, Jennifer Guild, Felix Imend=F6rffer; HRB 16746 AG N=FCrnbe= rg