On Tue, Nov 26 2013 at 11:15:25 AM, Greg Kroah-Hartman <gre...@linuxfoundation.org> wrote: > On Tue, Nov 26, 2013 at 10:14:21AM -0800, Ashutosh Dixit wrote: >> Endianness issues are now consistent as per the documentation in >> host/mic_virtio.h. Note that the host can be both BE or LE whereas the >> card is always LE. >> >> Memory space sparse warnings are fixed for now by using __force. This is >> sufficient for now since the driver depends on x86 but will need to be >> revisited if we support other architectures which treat I/O memory >> differently from system memory. > > Can you split the endian fixes up from the user pointer fixes to make it > easier to review/apply?
We've just submitted v3 where we now have separate patches (5 and 6) for endianness and sparse memory access fixes. > There's no need for this for 3.13-final, right? No bug fixes are here > that I can tell. That's correct. I've reordered the patches so that v3 patches 1-4 contain the real bug fixes. We'd like to apply patches 5 and 6 for 3.13-final too but if it is not possible we can drop one or both of them for now for further review. > And don't use __force, really, can't you fix this some other way? As stated in the description for patch 6, because of the dependence of the MIC drivers on the virtio infrastructure there is no simple way to fix these memory space issues at present. Since the sparse warnings do not represent real issues the warnings are being suppressed using __force. >> + vq = vring_new_virtqueue(index, le16_to_cpu(config.num), >> + MIC_VIRTIO_RING_ALIGN, vdev, false, >> + (void __force *)va, mic_notify, callback, >> + name); > > Why __force a void * here? That feels wrong. In all instances in patch 6 __force is used to strip the __iomem attribute (e.g. va is declared as 'void __iomem *va;'). thanks, ashutosh -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
