On Thu, 12 Jan 2012 00:02:33 +0200, "Michael S. Tsirkin" <mst@xxxxxxxxxx> wrote: > Look, we have a race currently. Let us not tie a bug fix to a huge > rewrite with unclear performance benefits, please. In theory, yes. In practice, we bandaid it. I think in the short term we change ->get to get the entire sequence twice, and check it's the same. Theoretically, still racy, but it does cut the window. And we haven't seen the bug yet, either. In the longer term, we fix it properly: 1) Make it readonly, prevents one class of problems. 2) Treat it as constant if drv->config_changed is NULL (we can do this now, in fact) and ignore the config interrupt. 3) Use a generation counter on the config, odd means wait. Cheers, Rusty. _______________________________________________ Virtualization mailing list Virtualization@xxxxxxxxxxxxxxxxxxxxxxxxxx https://lists.linuxfoundation.org/mailman/listinfo/virtualization
