lkml.org 
[lkml]   [2015]   [May]   [28]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH v6 1/2] PCI/MSI: Don't disable MSI/MSI-X at shutdown
    On Thu, May 14, 2015 at 11:53:34AM +0200, Michael S. Tsirkin wrote:
    > > you are almost certainly going to subject it to a
    > > screaming MSI interrupt and there almost certainly will
    > > not be code to disable MSIs as they are disabled by at
    > > boot up by default.
    > >
    > > Eric
    >
    > OTOH if you do disable MSI but leave device functioning you will just
    > get screaming INT#x which is even worse because it will end up disabling
    > INT#x which is shared, and so breaking multiple devices and not just
    > this one.

    Eric, can you comment on this please?
    To list the points again:
    1- if device is properly shut down there's no need to disable MSIs
    2- if device is not properly shut down disabling MSIs will cause
    screaming INT#x interrupts
    3- in both cases if we leave MSI on then we can be sure we won't
    get screaming interrupts since we now disable bus mastering
    which suppresses MSIs (but not INT#x)

    Looking at the commit that added the disable, that is d52877c7b1af, you
    will see that historically it preceded b566a22c23. So the screaming MSI
    problem that d52877c7b1af tried to address is now gone, addressed by
    b566a22c23.

    What do you think?

    > --
    > MST


    \
     
     \ /
      Last update: 2015-05-28 19:21    [W:3.713 / U:0.008 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site