• Andrea Merello's avatar
    rtl818x_pci: fix possible RX descriptor invalid data read · 325ed9ff
    Andrea Merello authored
    RX descriptor data must be read only if the descriptor has been fully
    updated by HW.
    There is a "ownership" flag in the descriptor itself to test this.
    
    The driver code contains a read for the "ownership" flag and, after
    it, other read access for descriptor data.
    
    This is in DMA coherent memory, that is _not_ guaranteed to be immune
    to instruction reordering, thus it is possible that the descriptor
    data is read _before_ the "ownership" flag.
    
    This can theoretically lead to a DMA/CPU race that may end up with the
    driver reading the data when it is still not valid, and the "ownership"
    bit just after enough time that the HW make the whole descriptor valid.
    The driver will in this case believe the data is valid, but it will use
    the invalid data read earlier.
    
    In order to avoid this, this patch adds a rmb() to force the "ownership"
    bit read to be issued before other descriptor data reads are attempted.
    Signed-off-by: default avatarAndrea Merello <andrea.merello@gmail.com>
    Signed-off-by: default avatarJohn W. Linville <linville@tuxdriver.com>
    325ed9ff
dev.c 54.5 KB