Re: [dpdk-dev] Virtio pci legacy support

[Tiwei Bie <tiwei.bie@intel.com>]

On Tue, Nov 12, 2019 at 09:23:31AM -0800, Stephen Hemminger wrote:
> Since Linux kernel is going to drop support for iopl real soon now.
> Is there any ongoing effort to either drop Legacy virtio support, 

I think it's not a good idea to drop the legacy virtio
support at present.

> or replace iopl with ioperm,

+1. This looks like the most straightforward way for now.

> or at least test with iopl failing?

The iopl failing test was just moved to rte_pci_ioport_map()
by below series:

https://patches.dpdk.org/cover/61627/

> 
> Note: it looks like the current code does not match comment.
> 
> /*
>  * Request iopl privilege for all RPL, returns 0 on success
>  * iopl() call is mostly for the i386 architecture. For other architectures,
>  * return -1 to indicate IO privilege can't be changed in this way.
>  */
> int
> rte_eal_iopl_init(void)
> {
> #if defined(RTE_ARCH_X86)
> 	if (iopl(3) != 0)
> 		return -1;
> #endif
> 	return 0;
> }