Re: [Qemu-devel] [PATCH v3 6/9] rocker: add new rocker switch device

2015-01-20 Thread Jason Wang
On Fri, Jan 16, 2015 at 5:48 PM, Scott Feldman sfel...@gmail.com wrote: On Fri, Jan 16, 2015 at 1:15 AM, Jason Wang jasow...@redhat.com wrote: On 01/11/2015 11:57 AM, sfel...@gmail.com wrote: Each port is a netdev and can be paired with using -netdev id=port name. Signed-off-by: Scott

Re: [Qemu-devel] [PATCH v3 6/9] rocker: add new rocker switch device

2015-01-16 Thread Jason Wang
On 01/11/2015 11:57 AM, sfel...@gmail.com wrote: From: Scott Feldman sfel...@gmail.com Rocker is a simulated ethernet switch device. The device supports up to 62 front-panel ports and supports L2 switching and L3 routing functions, as well as L2/L3/L4 ACLs. The device presents a single PCI

Re: [Qemu-devel] [PATCH v3 6/9] rocker: add new rocker switch device

2015-01-16 Thread Scott Feldman
On Fri, Jan 16, 2015 at 1:15 AM, Jason Wang jasow...@redhat.com wrote: On 01/11/2015 11:57 AM, sfel...@gmail.com wrote: Each port is a netdev and can be paired with using -netdev id=port name. Signed-off-by: Scott Feldman sfel...@gmail.com Signed-off-by: Jiri Pirko j...@resnulli.us ---

Re: [Qemu-devel] [PATCH v3 6/9] rocker: add new rocker switch device

2015-01-15 Thread Paolo Bonzini
On 15/01/2015 00:20, Scott Feldman wrote: I tried setting .impl.max_access_size=4 and the Linux driver self-test fails on 64-bit reg write/read. 32-bit regs writes/reads are OK. On the 64-bit write/read, the self test writes a value and expects to read back 2x the value written. After

Re: [Qemu-devel] [PATCH v3 6/9] rocker: add new rocker switch device

2015-01-14 Thread Scott Feldman
On Mon, Jan 12, 2015 at 4:57 AM, Paolo Bonzini pbonz...@redhat.com wrote: On 11/01/2015 04:57, sfel...@gmail.com wrote: +static const MemoryRegionOps rocker_mmio_ops = { +.read = rocker_mmio_read, +.write = rocker_mmio_write, +.endianness = DEVICE_LITTLE_ENDIAN, +.valid = {

Re: [Qemu-devel] [PATCH v3 6/9] rocker: add new rocker switch device

2015-01-12 Thread Paolo Bonzini
On 11/01/2015 04:57, sfel...@gmail.com wrote: +static const MemoryRegionOps rocker_mmio_ops = { +.read = rocker_mmio_read, +.write = rocker_mmio_write, +.endianness = DEVICE_LITTLE_ENDIAN, +.valid = { +.min_access_size = 4, +.max_access_size = 8, +},