diff options
author | jimharris <jimharris@FreeBSD.org> | 2012-12-05 20:21:33 +0000 |
---|---|---|
committer | jimharris <jimharris@FreeBSD.org> | 2012-12-05 20:21:33 +0000 |
commit | 3bda4941ce7aaff4a522572d3fcd2d1d4125c498 (patch) | |
tree | f19e988d10929804f1e9e11544bb656d60abad3b /contrib/gcclibs/libcpp/files.c | |
parent | 377b89c55f75aba70c2c66bfc96fb4c4af7311ae (diff) | |
download | FreeBSD-src-3bda4941ce7aaff4a522572d3fcd2d1d4125c498.zip FreeBSD-src-3bda4941ce7aaff4a522572d3fcd2d1d4125c498.tar.gz |
Don't call bus_dmamap_load in CAM_DIR_NONE case, since there is nothing
to map, and technically this isn't allowed.
Functionally, it works OK (at least on x86) to call bus_dmamap_load with
a NULL data pointer and zero length, so this is primarily for correctness
and consistency with other drivers.
While here, remove check in isci_io_request_construct for nseg==0.
Previously, bus_dmamap_load would pass nseg==1, even for case where
buffer is NULL and length = 0, which allowed CAM_DIR_NONE CCBs
to get processed. This check is not correct though, and needed to be
removed both for the changes elsewhere in this patch, as well as jeff's
preliminary bus_dmamap_load_ccb patch (which uncovered all of this in
the first place).
MFC after: 3 days
Diffstat (limited to 'contrib/gcclibs/libcpp/files.c')
0 files changed, 0 insertions, 0 deletions