On Thu, Jan 26, 2023 at 01:28:15PM +0000, Reshetova, Elena wrote: > > This is exactly what I said. You presented me the cases which exist in > > your invented world. Mentioned unhandled page fault doesn't exist in real > > world. If PCI device doesn't work, it needs to be replaced/blocked and not > > left to be operable and accessible from the kernel/user. > > Can we really assure correct operation of *all* pci devices out there? > How would such an audit be performed given a huge set of them available? > Isnt it better instead to make a small fix in the kernel behavior that would guard > us from such potentially not correctly operating devices? We assume that hardware works according to the spec; that's why we have a specification. Otherwise, things would be pretty insane, and would lead to massive bloat *everywhere*. If there are broken PCI devices out there, then we can blacklist the PCI device. If a manufacturer is consistently creating devices which don't obey the spec, we could block all devices from that manufacturer, and have an explicit white list for those devices from that manufacturer that actually work. If we can't count on a floating point instruction to return the right value, what are we supposed to do? Create a code which double checks every single floating point instruction just in case 2 + 2 = 3.99999999? :-) Ultimately, changing the trust boundary what is considered is a fundamentally hard thing, and to try to claim that code that assumes that things inside the trust boundary are, well, trusted, is not a great way to win friends and influence people. > Let's forget the trust angle here (it only applies to the Confidential Computing > threat model and you clearly implying the existing threat model instead) and stick just to > the not-correctly operating device. What you are proposing is to fix *unknown* bugs > in multitude of pci devices that (in case of this particular MSI bug) can > lead to two different values being read from the config space and kernel incorrectly > handing this situation. I don't think that's what people are saying. If there are buggy PCI devices, we can put them on block lists. But checking that every single read from the config space is unchanged is not something we should do, period. > Isn't it better to do the clear fix in one place to ensure such > situation (two subsequent reads with different values) cannot even happen in theory? > In security we have a saying that fixing a root cause of the problem is the most efficient > way to mitigate the problem. The root cause here is a double-read with different values, > so if it can be substituted with an easy and clear patch that probably even improves > performance as we do one less pci read and use cached value instead, where is the > problem in this particular case? If there are technical issues with the patch, of course we > need to discuss it/fix it, but it seems we are arguing here about whenever or not we want > to be fixing kernel code when we notice such cases... Well, if there is a performance win to cache a read from config space, then make the argument from a performance perspective. But caching values takes memory, and will potentially bloat data structures. It's not necessarily cost-free to caching every single config space variable to prevent double-read from either buggy or malicious devices. So it's one thing if we make each decision from a cost-benefit perspective. But then it's a *optimization*, not a *bug-fix*, and it also means that we aren't obligated to cache every single read from config space, lest someone wag their fingers at us saying, "Buggy! Your code is Buggy!". Cheers, - Ted
