----- Original Message ----- > Marc-André Lureau <[email protected]> writes: > > > Hi > > > > ----- Original Message ----- > >> Marc-André Lureau <[email protected]> writes: > >> > >> > ----- Original Message ----- > >> >> Signed-off-by: Markus Armbruster <[email protected]> > >> >> --- > >> >> hw/misc/ivshmem.c | 1 + > >> >> 1 file changed, 1 insertion(+) > >> >> > >> >> diff --git a/hw/misc/ivshmem.c b/hw/misc/ivshmem.c > >> >> index 83d7bd3..a6bfdde 100644 > >> >> --- a/hw/misc/ivshmem.c > >> >> +++ b/hw/misc/ivshmem.c > >> >> @@ -939,6 +939,7 @@ static void pci_ivshmem_realize(PCIDevice *dev, > >> >> Error > >> >> **errp) > >> >> memory_region_add_subregion(&s->bar, 0, mr); > >> >> pci_register_bar(PCI_DEVICE(s), 2, attr, &s->bar); > >> >> } else if (s->server_chr != NULL) { > >> >> + /* FIXME replace this test, it works basically by chance */ > >> > > >> > I wouldn't say it like that, but ack anyway. > >> > >> I'm happy to consider a rephrased comment. > > > > Although Paolo used "chance" too, I think the proper comment would be > > the following: > > > > FIXME: only pty and socket chardevs set chr->filename, other set to > > the default---the backend name. > > I'm afraid that doesn't convey what needs fixing. > > > ie, to me it's not just by "chance" :) > > Well, it's certainly not by design! > > What about: > > + /* FIXME do not rely on what chr drivers put into filename */
ack > > [...] >
