qemu-commits
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Qemu-commits] [qemu/qemu] 0374cb: hw/pci-host: Use the PCI_BUILD_BDF()


From: Peter Maydell
Subject: [Qemu-commits] [qemu/qemu] 0374cb: hw/pci-host: Use the PCI_BUILD_BDF() macro from 'h...
Date: Wed, 06 Jan 2021 03:43:51 -0800

  Branch: refs/heads/master
  Home:   https://github.com/qemu/qemu
  Commit: 0374cbd2f1b5bc68816c811bdaf266d98b21ecfe
      
https://github.com/qemu/qemu/commit/0374cbd2f1b5bc68816c811bdaf266d98b21ecfe
  Author: Philippe Mathieu-Daudé <f4bug@amsat.org>
  Date:   2021-01-04 (Mon, 04 Jan 2021)

  Changed paths:
    M hw/pci-host/bonito.c
    M hw/pci-host/pnv_phb4.c

  Log Message:
  -----------
  hw/pci-host: Use the PCI_BUILD_BDF() macro from 'hw/pci/pci.h'

We already have a generic PCI_BUILD_BDF() macro in "hw/pci/pci.h"
to pack these values, use it.

Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Acked-by: David Gibson <david@gibson.dropbear.id.au>
Acked-by: Michael S. Tsirkin <mst@redhat.com>
Message-Id: <20201012124506.3406909-3-philmd@redhat.com>


  Commit: d08b9c1b75c60c66ea65212a9dac90f58542e45e
      
https://github.com/qemu/qemu/commit/d08b9c1b75c60c66ea65212a9dac90f58542e45e
  Author: Philippe Mathieu-Daudé <f4bug@amsat.org>
  Date:   2021-01-04 (Mon, 04 Jan 2021)

  Changed paths:
    M hw/pci-host/uninorth.c

  Log Message:
  -----------
  hw/pci-host/uninorth: Use the PCI_FUNC() macro from 'hw/pci/pci.h'

We already have a generic PCI_FUNC() macro in "hw/pci/pci.h" to
extract the PCI function identifier, use it.

Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Acked-by: David Gibson <david@gibson.dropbear.id.au>
Acked-by: Michael S. Tsirkin <mst@redhat.com>
Message-Id: <20201012124506.3406909-4-philmd@redhat.com>


  Commit: 8d40def66d744a964c2675a19c11f644d288dd4f
      
https://github.com/qemu/qemu/commit/8d40def66d744a964c2675a19c11f644d288dd4f
  Author: Philippe Mathieu-Daudé <f4bug@amsat.org>
  Date:   2021-01-04 (Mon, 04 Jan 2021)

  Changed paths:
    M hw/hppa/dino.c
    M hw/i386/xen/xen-hvm.c
    M hw/isa/piix3.c
    M hw/mips/gt64xxx_pci.c
    M hw/pci-host/bonito.c
    M hw/pci-host/ppce500.c
    M hw/ppc/ppc4xx_pci.c
    M hw/sh4/sh_pci.c

  Log Message:
  -----------
  hw: Use the PCI_SLOT() macro from 'hw/pci/pci.h'

We already have a generic PCI_SLOT() macro in "hw/pci/pci.h"
to extract the PCI slot identifier, use it.

Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Acked-by: Paul Durrant <paul@xen.org>
Acked-by: David Gibson <david@gibson.dropbear.id.au>
Acked-by: Michael S. Tsirkin <mst@redhat.com>
Message-Id: <20201012124506.3406909-5-philmd@redhat.com>


  Commit: 4934e479f1806e69cfab637156fe136994619c03
      
https://github.com/qemu/qemu/commit/4934e479f1806e69cfab637156fe136994619c03
  Author: Philippe Mathieu-Daudé <f4bug@amsat.org>
  Date:   2021-01-04 (Mon, 04 Jan 2021)

  Changed paths:
    M hw/arm/virt.c
    M hw/pci-host/bonito.c
    M hw/pci-host/uninorth.c

  Log Message:
  -----------
  hw: Use the PCI_DEVFN() macro from 'hw/pci/pci.h'

We already have a generic PCI_DEVFN() macro in "hw/pci/pci.h"
to pack the PCI slot/function identifiers, use it.

Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Acked-by: David Gibson <david@gibson.dropbear.id.au>
Message-Id: <20201012124506.3406909-6-philmd@redhat.com>
Reviewed-by: Huacai Chen <chenhuacai@kernel.org>
Acked-by: Michael S. Tsirkin <mst@redhat.com>
Message-Id: <20201231224911.1467352-4-f4bug@amsat.org>


  Commit: ce3f3d3027078939f2b38cd0d371d44ba8a29026
      
https://github.com/qemu/qemu/commit/ce3f3d3027078939f2b38cd0d371d44ba8a29026
  Author: Philippe Mathieu-Daudé <f4bug@amsat.org>
  Date:   2021-01-04 (Mon, 04 Jan 2021)

  Changed paths:
    M hw/pci-host/bonito.c

  Log Message:
  -----------
  hw/pci-host/bonito: Display hexadecimal value with '0x' prefix

Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Reviewed-by: Huacai Chen <chenhuacai@kernel.org>
Message-Id: <20201231224911.1467352-3-f4bug@amsat.org>


  Commit: b4bb339b3ddb24601b5f914fb0bc7275a69d2c94
      
https://github.com/qemu/qemu/commit/b4bb339b3ddb24601b5f914fb0bc7275a69d2c94
  Author: Philippe Mathieu-Daudé <f4bug@amsat.org>
  Date:   2021-01-04 (Mon, 04 Jan 2021)

  Changed paths:
    M hw/pci-host/bonito.c

  Log Message:
  -----------
  hw/pci-host/bonito: Use pci_config_set_interrupt_pin()

Replace pci_set_byte(PCI_INTERRUPT_PIN) by
pci_config_set_interrupt_pin().

Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Reviewed-by: Huacai Chen <chenhuacai@kernel.org>
Message-Id: <20201231224911.1467352-5-f4bug@amsat.org>


  Commit: 5a4856ed78e8b83f6eece2efbbedccba857aaff7
      
https://github.com/qemu/qemu/commit/5a4856ed78e8b83f6eece2efbbedccba857aaff7
  Author: BALATON Zoltan <balaton@eik.bme.hu>
  Date:   2021-01-04 (Mon, 04 Jan 2021)

  Changed paths:
    M hw/isa/vt82c686.c

  Log Message:
  -----------
  vt82c686: Rename AC97/MC97 parts from VT82C686B to VIA

These parts are common between VT82C686B and VT8231 so can be shared
in the future. Rename them to VIA prefix accordingly.

Signed-off-by: BALATON Zoltan <balaton@eik.bme.hu>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Message-Id: 
<510ddb17836a2c2e68a27cf2dcaee420bc2efbc1.1609584215.git.balaton@eik.bme.hu>
Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>


  Commit: e634050544198545ded4b0a324d2e4c7f8c69809
      
https://github.com/qemu/qemu/commit/e634050544198545ded4b0a324d2e4c7f8c69809
  Author: BALATON Zoltan <balaton@eik.bme.hu>
  Date:   2021-01-04 (Mon, 04 Jan 2021)

  Changed paths:
    M hw/isa/vt82c686.c

  Log Message:
  -----------
  vt82c686: Remove unnecessary _DEVICE suffix from type macros

There's no reason to suffix everything with _DEVICE when the names are
already unique without it and shorter names are more readable.

Signed-off-by: BALATON Zoltan <balaton@eik.bme.hu>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Message-Id: 
<ea89683ebb3528c0f79ed99d3d3cfcefb63c3bfb.1609584216.git.balaton@eik.bme.hu>
Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>


  Commit: 0f79846147faf87ef17436d4255241ad32cc634c
      
https://github.com/qemu/qemu/commit/0f79846147faf87ef17436d4255241ad32cc634c
  Author: BALATON Zoltan <balaton@eik.bme.hu>
  Date:   2021-01-04 (Mon, 04 Jan 2021)

  Changed paths:
    M hw/isa/vt82c686.c

  Log Message:
  -----------
  vt82c686: Rename VT82C686B to VT82C686B_ISA

This is really the ISA bridge part so name the type accordingly.

Signed-off-by: BALATON Zoltan <balaton@eik.bme.hu>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Message-Id: 
<78db2ced4b41a8a775dbc6c97a90db683952c2cb.1609584216.git.balaton@eik.bme.hu>
Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>


  Commit: 07c6832cb2c65407e312e8bed893bf8d7ae770e6
      
https://github.com/qemu/qemu/commit/07c6832cb2c65407e312e8bed893bf8d7ae770e6
  Author: BALATON Zoltan <balaton@eik.bme.hu>
  Date:   2021-01-04 (Mon, 04 Jan 2021)

  Changed paths:
    M hw/isa/vt82c686.c
    M hw/mips/fuloong2e.c
    M include/hw/isa/vt82c686.h

  Log Message:
  -----------
  vt82c686: Remove vt82c686b_[am]c97_init() functions

These are legacy init functions that are just equivalent to directly
calling pci_create_simple so do that instead. Also rename objects to
lower case via-ac97 and via-mc97 matching naming of other devices.

Signed-off-by: BALATON Zoltan <balaton@eik.bme.hu>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Message-Id: 
<1c4373c8aeb6c4fb2a8df2c864b0e91a977a3d7b.1609584216.git.balaton@eik.bme.hu>
Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>


  Commit: 657fae258f98000ced6d50a4490a922a207e35f3
      
https://github.com/qemu/qemu/commit/657fae258f98000ced6d50a4490a922a207e35f3
  Author: BALATON Zoltan <balaton@eik.bme.hu>
  Date:   2021-01-04 (Mon, 04 Jan 2021)

  Changed paths:
    M hw/audio/meson.build
    A hw/audio/via-ac97.c
    M hw/isa/vt82c686.c

  Log Message:
  -----------
  vt82c686: Split off via-[am]c97 into separate file in hw/audio

The via-[am]c97 code is supposed to implement the audio part of VIA
south bridge chips so it is better placed under hw/audio/. Split it
off into a separate file.

Signed-off-by: BALATON Zoltan <balaton@eik.bme.hu>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Message-Id: 
<af083634e3b9efe67e6c4247cf0185d3fa7b1810.1609584216.git.balaton@eik.bme.hu>
Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>


  Commit: a250e1c6e775a2f14819a4412fed77a48253a473
      
https://github.com/qemu/qemu/commit/a250e1c6e775a2f14819a4412fed77a48253a473
  Author: BALATON Zoltan <balaton@eik.bme.hu>
  Date:   2021-01-04 (Mon, 04 Jan 2021)

  Changed paths:
    M hw/audio/via-ac97.c

  Log Message:
  -----------
  audio/via-ac97: Simplify code and set user_creatable to false

Remove some unneded, empty code and set user_creatable to false
(besides being not implemented yet, so does nothing anyway) it's also
normally part of VIA south bridge chips so no need to confuse users
showing them these devices.

Signed-off-by: BALATON Zoltan <balaton@eik.bme.hu>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Message-Id: 
<c7a5b1ee4c02e304ff70ebfbf269544f3c1f8412.1609584216.git.balaton@eik.bme.hu>
Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>


  Commit: 0bfda9a225b8b50a7e6bebd25ede9df6db8ddd83
      
https://github.com/qemu/qemu/commit/0bfda9a225b8b50a7e6bebd25ede9df6db8ddd83
  Author: BALATON Zoltan <balaton@eik.bme.hu>
  Date:   2021-01-04 (Mon, 04 Jan 2021)

  Changed paths:
    M hw/isa/vt82c686.c
    M hw/mips/fuloong2e.c
    M include/hw/isa/vt82c686.h

  Log Message:
  -----------
  vt82c686: Remove legacy vt82c686b_isa_init() function

Signed-off-by: BALATON Zoltan <balaton@eik.bme.hu>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Message-Id: 
<58d7585f979f154b1f1e69fdc026eed6dbc7996f.1609584216.git.balaton@eik.bme.hu>
Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>


  Commit: dc66439542c1772f2290c0908814580db9377f23
      
https://github.com/qemu/qemu/commit/dc66439542c1772f2290c0908814580db9377f23
  Author: BALATON Zoltan <balaton@eik.bme.hu>
  Date:   2021-01-04 (Mon, 04 Jan 2021)

  Changed paths:
    M hw/isa/vt82c686.c
    M hw/mips/fuloong2e.c
    M include/hw/isa/vt82c686.h

  Log Message:
  -----------
  vt82c686: Remove legacy vt82c686b_pm_init() function

Remove legacy vt82c686b_pm_init() function and also rename
VT82C686B_PM type name to match other device names.

Signed-off-by: BALATON Zoltan <balaton@eik.bme.hu>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Message-Id: 
<a70982b32f11222d335385b90749abb6cf2e2cce.1609584216.git.balaton@eik.bme.hu>
Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>


  Commit: ff413a1f7f65bb7fbd9489de29b04253f2fdaf1a
      
https://github.com/qemu/qemu/commit/ff413a1f7f65bb7fbd9489de29b04253f2fdaf1a
  Author: BALATON Zoltan <balaton@eik.bme.hu>
  Date:   2021-01-04 (Mon, 04 Jan 2021)

  Changed paths:
    M hw/isa/trace-events
    M hw/isa/vt82c686.c

  Log Message:
  -----------
  vt82c686: Convert debug printf to trace points

Drop DPRINTF and use trace functions instead. Two debug messages about
unimplemented registers could be converted to qemu_log_mask() but in
reality all registers are currently unimplemented (we just store and
return values of writable regs but do nothing with them). As we
already trace register access there's no need for additional debug
messages so these are just removed and a comment is added as a reminder.

Signed-off-by: BALATON Zoltan <balaton@eik.bme.hu>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Message-Id: 
<785854022a37035f66d89e70cb6ca1bc0e0d0163.1609584216.git.balaton@eik.bme.hu>
Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>


  Commit: 9b0fbae2cbf8ff47c731a476d4c4ec824394fb6b
      
https://github.com/qemu/qemu/commit/9b0fbae2cbf8ff47c731a476d4c4ec824394fb6b
  Author: BALATON Zoltan <balaton@eik.bme.hu>
  Date:   2021-01-04 (Mon, 04 Jan 2021)

  Changed paths:
    M hw/isa/vt82c686.c

  Log Message:
  -----------
  vt82c686: Remove unneeded includes and defines

These are not used or not needed.

Signed-off-by: BALATON Zoltan <balaton@eik.bme.hu>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Message-Id: 
<35cefcc3518a3395a796bb6ad6fbc308adc65266.1609584216.git.balaton@eik.bme.hu>
Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>


  Commit: 007b3103a393bb6123354e7d81650a761b62a66f
      
https://github.com/qemu/qemu/commit/007b3103a393bb6123354e7d81650a761b62a66f
  Author: BALATON Zoltan <balaton@eik.bme.hu>
  Date:   2021-01-04 (Mon, 04 Jan 2021)

  Changed paths:
    M hw/isa/vt82c686.c

  Log Message:
  -----------
  vt82c686: Use shorter name for local variable holding object state

Rename local variable holding object state for readability and
consistency.

Signed-off-by: BALATON Zoltan <balaton@eik.bme.hu>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Message-Id: 
<69655b23df2ecebbf0aff29726f4b4746f5b74de.1609584216.git.balaton@eik.bme.hu>
Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>


  Commit: 7886a674f13320c8e1a0744f3794e8caeeff874a
      
https://github.com/qemu/qemu/commit/7886a674f13320c8e1a0744f3794e8caeeff874a
  Author: BALATON Zoltan <balaton@eik.bme.hu>
  Date:   2021-01-04 (Mon, 04 Jan 2021)

  Changed paths:
    M hw/isa/vt82c686.c

  Log Message:
  -----------
  vt82c686: Rename superio config related parts

Use less confusing naming for superio config register handling related
parts that makes it clearer what belongs to this part.

Signed-off-by: BALATON Zoltan <balaton@eik.bme.hu>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Message-Id: 
<4d30a2b4b771b2ad651509885daae79d7c4fe7a8.1609584216.git.balaton@eik.bme.hu>
Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>


  Commit: 554d523785ef8681905ec13ad28a025ec0af40fe
      
https://github.com/qemu/qemu/commit/554d523785ef8681905ec13ad28a025ec0af40fe
  Author: Peter Maydell <peter.maydell@linaro.org>
  Date:   2021-01-04 (Mon, 04 Jan 2021)

  Changed paths:
    M docs/devel/clocks.rst
    M include/hw/clock.h

  Log Message:
  -----------
  clock: Introduce clock_ticks_to_ns()

The clock_get_ns() API claims to return the period of a clock in
nanoseconds. Unfortunately since it returns an integer and a
clock's period is represented in units of 2^-32 nanoseconds,
the result is often an approximation, and calculating a clock
expiry deadline by multiplying clock_get_ns() by a number-of-ticks
is unacceptably inaccurate.

Introduce a new API clock_ticks_to_ns() which returns the number
of nanoseconds it takes the clock to make a given number of ticks.
This function can do the complete calculation internally and
will thus give a more accurate result.

Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Tested-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Reviewed-by: Luc Michel <luc@lmichel.fr>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Message-Id: <20201215150929.30311-2-peter.maydell@linaro.org>
Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>


  Commit: 0ac1fb256742f665449c3dd02dd0ed7a5112cea1
      
https://github.com/qemu/qemu/commit/0ac1fb256742f665449c3dd02dd0ed7a5112cea1
  Author: Peter Maydell <peter.maydell@linaro.org>
  Date:   2021-01-04 (Mon, 04 Jan 2021)

  Changed paths:
    M target/mips/cpu.c

  Log Message:
  -----------
  target/mips: Don't use clock_get_ns() in clock period calculation

Currently the MIPS code uses the old clock_get_ns() API to
calculate a time length in nanoseconds:
 cpu->cp0_count_rate * clock_get_ns(MIPS_CPU(cpu)->clock)

This relies on the clock having a period which is an exact number
of nanoseconds.

Switch to the new clock_ticks_to_ns() function, which does the
multiplication internally at a higher precision.

Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Tested-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Reviewed-by: Luc Michel <luc@lmichel.fr>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Message-Id: <20201215150929.30311-3-peter.maydell@linaro.org>
Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>


  Commit: de6a65f11d7e5a2a93f2b75c0d434ab6ed7f68c8
      
https://github.com/qemu/qemu/commit/de6a65f11d7e5a2a93f2b75c0d434ab6ed7f68c8
  Author: Peter Maydell <peter.maydell@linaro.org>
  Date:   2021-01-04 (Mon, 04 Jan 2021)

  Changed paths:
    M docs/devel/clocks.rst
    M include/hw/clock.h

  Log Message:
  -----------
  clock: Remove clock_get_ns()

Remove the now-unused clock_get_ns() API and the CLOCK_PERIOD_TO_NS()
macro that only it was using.

Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Tested-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Reviewed-by: Luc Michel <luc@lmichel.fr>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Message-Id: <20201215150929.30311-4-peter.maydell@linaro.org>
Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>


  Commit: b7cd9c1e840d511319f326ee8cab772b3ac50a3b
      
https://github.com/qemu/qemu/commit/b7cd9c1e840d511319f326ee8cab772b3ac50a3b
  Author: Peter Maydell <peter.maydell@linaro.org>
  Date:   2021-01-04 (Mon, 04 Jan 2021)

  Changed paths:
    M docs/devel/clocks.rst
    M hw/core/clock.c
    M include/hw/clock.h
    M softmmu/qdev-monitor.c

  Log Message:
  -----------
  clock: Define and use new clock_display_freq()

It's common to want to print a human-readable indication of a clock's
frequency. Provide a utility function in the clock API to return a
string which is a displayable representation of the frequency,
and use it in qdev-monitor.c.

Before:

  (qemu) info qtree
  [...]
  dev: xilinx,zynq_slcr, id ""
    clock-in "ps_clk" freq_hz=3.333333e+07
    mmio 00000000f8000000/0000000000001000

After:

  dev: xilinx,zynq_slcr, id ""
    clock-in "ps_clk" freq_hz=33.3 MHz
    mmio 00000000f8000000/0000000000001000

Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Tested-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Reviewed-by: Luc Michel <luc@lmichel.fr>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Message-Id: <20201215150929.30311-5-peter.maydell@linaro.org>
Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>


  Commit: dea96332327f2651f726244cc5bb41c4f9350f8b
      
https://github.com/qemu/qemu/commit/dea96332327f2651f726244cc5bb41c4f9350f8b
  Author: Huacai Chen <chenhuacai@kernel.org>
  Date:   2021-01-04 (Mon, 04 Jan 2021)

  Changed paths:
    M hw/intc/loongson_liointc.c
    A include/hw/intc/loongson_liointc.h

  Log Message:
  -----------
  hw/intc: Rework Loongson LIOINTC

As suggested by Philippe Mathieu-Daudé, rework Loongson's liointc:
1, Move macro definitions to loongson_liointc.h;
2, Remove magic values and use macros instead;
3, Replace dead D() code by trace events.

Suggested-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Signed-off-by: Huacai Chen <chenhuacai@kernel.org>
Tested-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Message-Id: <20201221110538.3186646-2-chenhuacai@kernel.org>
Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>


  Commit: 313d1e910ac0eb12bbfe83f158cab60f600319f1
      
https://github.com/qemu/qemu/commit/313d1e910ac0eb12bbfe83f158cab60f600319f1
  Author: Huacai Chen <chenhuacai@kernel.org>
  Date:   2021-01-04 (Mon, 04 Jan 2021)

  Changed paths:
    M hw/mips/Kconfig
    A hw/mips/fw_cfg.c
    A hw/mips/fw_cfg.h
    M hw/mips/meson.build

  Log Message:
  -----------
  hw/mips: Implement fw_cfg_arch_key_name()

Implement fw_cfg_arch_key_name(), which returns the name of a
mips-specific key.

Co-developed-by: Jiaxun Yang <jiaxun.yang@flygoat.com>
Signed-off-by: Huacai Chen <chenhuacai@kernel.org>
Signed-off-by: Jiaxun Yang <jiaxun.yang@flygoat.com>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Message-Id: <1600742967-12933-7-git-send-email-chenhc@lemote.com>
[PMD: Add FW_CFG_MIPS Kconfig selector]
Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>


  Commit: d2245e2de0201d7dfaff66c6851934af2d23bbfc
      
https://github.com/qemu/qemu/commit/d2245e2de0201d7dfaff66c6851934af2d23bbfc
  Author: Huacai Chen <chenhuacai@kernel.org>
  Date:   2021-01-04 (Mon, 04 Jan 2021)

  Changed paths:
    M MAINTAINERS
    A hw/mips/loongson3_bootp.c
    A hw/mips/loongson3_bootp.h
    M hw/mips/meson.build

  Log Message:
  -----------
  hw/mips: Add Loongson-3 boot parameter helpers

Preparing to add Loongson-3 machine support, add Loongson-3's LEFI (a
UEFI-like interface for BIOS-Kernel boot parameters) helpers first.

Co-developed-by: Jiaxun Yang <jiaxun.yang@flygoat.com>
Signed-off-by: Huacai Chen <chenhuacai@kernel.org>
Signed-off-by: Jiaxun Yang <jiaxun.yang@flygoat.com>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Message-Id: <20201221110538.3186646-4-chenhuacai@kernel.org>
[PMD: Fixed typo]
Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>


  Commit: c76b409fef19dc60a48f02a8e12e7fb46b07caf7
      
https://github.com/qemu/qemu/commit/c76b409fef19dc60a48f02a8e12e7fb46b07caf7
  Author: Huacai Chen <chenhuacai@kernel.org>
  Date:   2021-01-04 (Mon, 04 Jan 2021)

  Changed paths:
    M MAINTAINERS
    M default-configs/devices/mips64el-softmmu.mak
    M hw/mips/Kconfig
    A hw/mips/loongson3_virt.c
    M hw/mips/meson.build

  Log Message:
  -----------
  hw/mips: Add Loongson-3 machine support

Add Loongson-3 based machine support, it use liointc as the interrupt
controler and use GPEX as the pci controller. Currently it can work with
both TCG and KVM.

As the machine model is not based on any exiting physical hardware, the
name of the machine is "loongson3-virt". It may be superseded in future
by a real machine model. If this happens, then a regular deprecation
procedure shall occur for "loongson3-virt" machine.

We now already have a full functional Linux kernel (based on Linux-5.4.x
LTS) here:

https://github.com/chenhuacai/linux

Of course the upstream kernel is also usable (the kvm host side and
guest side have both been upstream in Linux-5.9):

https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git

How to use QEMU/Loongson-3?
1, Download kernel source from the above URL;
2, Build a kernel with arch/mips/configs/loongson3_defconfig;
3, Boot a Loongson-3A4000 host with this kernel (for KVM mode);
4, Build QEMU-master with this patchset;
5, modprobe kvm (only necessary for KVM mode);
6, Use QEMU with TCG:
       qemu-system-mips64el -M loongson3-virt,accel=tcg -cpu Loongson-3A1000 
-kernel <path_to_kernel> -append ...
   Use QEMU with KVM:
       qemu-system-mips64el -M loongson3-virt,accel=kvm -cpu Loongson-3A4000 
-kernel <path_to_kernel> -append ...

   The "-cpu" parameter is optional here and QEMU will use the correct type for 
TCG/KVM automatically.

Co-developed-by: Jiaxun Yang <jiaxun.yang@flygoat.com>
Signed-off-by: Huacai Chen <chenhuacai@kernel.org>
Signed-off-by: Jiaxun Yang <jiaxun.yang@flygoat.com>
Message-Id: <20201221110538.3186646-5-chenhuacai@kernel.org>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
[PMD: Set TYPE_LOONGSON_MACHINE instance_size in TypeInfo,
      select FW_CFG_MIPS in Kconfig]
Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>


  Commit: c7784e42c7cce358ba9b21231dffe376f670cd5f
      
https://github.com/qemu/qemu/commit/c7784e42c7cce358ba9b21231dffe376f670cd5f
  Author: Huacai Chen <chenhuacai@kernel.org>
  Date:   2021-01-04 (Mon, 04 Jan 2021)

  Changed paths:
    M docs/system/target-mips.rst

  Log Message:
  -----------
  docs/system: Update MIPS machine documentation

Update MIPS machine documentation to add Loongson-3 based machine description.

Signed-off-by: Huacai Chen <chenhuacai@kernel.org>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Message-Id: <20201221110538.3186646-6-chenhuacai@kernel.org>
Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>


  Commit: dde989943b401aec3a063214278474d2f8c1a9de
      
https://github.com/qemu/qemu/commit/dde989943b401aec3a063214278474d2f8c1a9de
  Author: Jiaxun Yang <jiaxun.yang@flygoat.com>
  Date:   2021-01-04 (Mon, 04 Jan 2021)

  Changed paths:
    M hw/mips/fuloong2e.c
    M hw/mips/malta.c
    M hw/mips/mipssim.c

  Log Message:
  -----------
  hw/mips: Make bootloader addresses unsigned

Address should be unsigned anyway, otherwise it may carry
calculations wrongly.

Signed-off-by: Jiaxun Yang <jiaxun.yang@flygoat.com>
Tested-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Message-Id: <20201215064200.28751-2-jiaxun.yang@flygoat.com>
[PMD: Fixed typo and convert hw/mips/mipssim.c too]
Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>


  Commit: 5052b6e82ccad696f63b9e7107e49c53965f750c
      
https://github.com/qemu/qemu/commit/5052b6e82ccad696f63b9e7107e49c53965f750c
  Author: Jiaxun Yang <jiaxun.yang@flygoat.com>
  Date:   2021-01-04 (Mon, 04 Jan 2021)

  Changed paths:
    M hw/mips/malta.c

  Log Message:
  -----------
  hw/mips/malta: Use address translation helper to calculate bootloader_run_addr

So it will sign extend adresses properly.

Signed-off-by: Jiaxun Yang <jiaxun.yang@flygoat.com>
Tested-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Message-Id: <20201215064200.28751-3-jiaxun.yang@flygoat.com>
Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>


  Commit: df055c65e47e4f255921b4125221b8a9f8fccc00
      
https://github.com/qemu/qemu/commit/df055c65e47e4f255921b4125221b8a9f8fccc00
  Author: Jiaxun Yang <jiaxun.yang@flygoat.com>
  Date:   2021-01-04 (Mon, 04 Jan 2021)

  Changed paths:
    M hw/mips/fuloong2e.c
    M hw/mips/malta.c

  Log Message:
  -----------
  hw/mips: Use address translation helper to handle ENVP_ADDR

It will signed extend vaddr properly.

Signed-off-by: Jiaxun Yang <jiaxun.yang@flygoat.com>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Tested-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Message-Id: <20201215064200.28751-4-jiaxun.yang@flygoat.com>
Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>


  Commit: 6fed2a8ea4f50377d9c4aee051c7e75d9d70de5d
      
https://github.com/qemu/qemu/commit/6fed2a8ea4f50377d9c4aee051c7e75d9d70de5d
  Author: Jiaxun Yang <jiaxun.yang@flygoat.com>
  Date:   2021-01-04 (Mon, 04 Jan 2021)

  Changed paths:
    M hw/mips/fuloong2e.c

  Log Message:
  -----------
  hw/mips/fuloong2e: Remove define DEBUG_FULOONG2E_INIT

Seems useless....

Fixes: 051c190bce5 ("MIPS: Initial support of fulong mini pc (machine 
construction)")
Signed-off-by: Jiaxun Yang <jiaxun.yang@flygoat.com>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Reviewed-by: Huacai Chen <chenhuacai@kernel.org>
Message-Id: <20201224031750.52146-2-jiaxun.yang@flygoat.com>
Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>


  Commit: 94a37806c0269f9fa5554ebde3ffb141f22e187d
      
https://github.com/qemu/qemu/commit/94a37806c0269f9fa5554ebde3ffb141f22e187d
  Author: Jiaxun Yang <jiaxun.yang@flygoat.com>
  Date:   2021-01-04 (Mon, 04 Jan 2021)

  Changed paths:
    M hw/mips/fuloong2e.c

  Log Message:
  -----------
  hw/mips/fuloong2e: Replace faulty documentation links

Websites are downing, but GitHub may last forever.
Loongson even doesn't recogonize 2E as their products nowadays..

Signed-off-by: Jiaxun Yang <jiaxun.yang@flygoat.com>
Tested-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Reviewed-by: Huacai Chen <chenhuacai@kernel.org>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Message-Id: <20201224031750.52146-3-jiaxun.yang@flygoat.com>
Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>


  Commit: c0809fcfac3cf599d69f7e4610856a786d78ccad
      
https://github.com/qemu/qemu/commit/c0809fcfac3cf599d69f7e4610856a786d78ccad
  Author: Jiaxun Yang <jiaxun.yang@flygoat.com>
  Date:   2021-01-04 (Mon, 04 Jan 2021)

  Changed paths:
    M hw/mips/fuloong2e.c

  Log Message:
  -----------
  hw/mips/fuloong2e: Remove unused env entry

modetty is not handled by kernel and the parameter
here seems unreasonable.

Signed-off-by: Jiaxun Yang <jiaxun.yang@flygoat.com>
Message-Id: <20201224031750.52146-6-jiaxun.yang@flygoat.com>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
[PMD: Do not remove busclock]
Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>


  Commit: e41f27ec2a0c29a7b016af16992cf6e642a54002
      
https://github.com/qemu/qemu/commit/e41f27ec2a0c29a7b016af16992cf6e642a54002
  Author: Jiaxun Yang <jiaxun.yang@flygoat.com>
  Date:   2021-01-04 (Mon, 04 Jan 2021)

  Changed paths:
    M hw/mips/fuloong2e.c

  Log Message:
  -----------
  hw/mips/fuloong2e: Correct cpuclock in PROM environment

Missed in 3ca7639ff00 ("hw/mips/fuloong2e: Set CPU frequency
to 533 MHz"), we need to tell the kernel the correct clock.

Fixes: 3ca7639ff00 ("hw/mips/fuloong2e: Set CPU frequency to 533 MHz").
Signed-off-by: Jiaxun Yang <jiaxun.yang@flygoat.com>
Reviewed-by: Huacai Chen <chenhuacai@kernel.org>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Message-Id: <20201224031750.52146-7-jiaxun.yang@flygoat.com>
[PMD: Reworded]
Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>


  Commit: 457027298749333047bf81a856ce95ea5f9dccd9
      
https://github.com/qemu/qemu/commit/457027298749333047bf81a856ce95ea5f9dccd9
  Author: Jiaxun Yang <jiaxun.yang@flygoat.com>
  Date:   2021-01-04 (Mon, 04 Jan 2021)

  Changed paths:
    M tests/acceptance/boot_linux_console.py

  Log Message:
  -----------
  tests/acceptance: Test boot_linux_console for fuloong2e

The kernel comes from debian archive so it's trusted.

Invoking the test can be done as follows:

  $ avocado --show=app,console run -t machine:fuloong2e tests/acceptance/
   (1/1) 
tests/acceptance/boot_linux_console.py:BootLinuxConsole.test_mips64el_fuloong2e:
  console: [    0.000000] Initializing cgroup subsys cpuset
  console: [    0.000000] Initializing cgroup subsys cpu
  console: [    0.000000] Initializing cgroup subsys cpuacct
  console: [    0.000000] Linux version 3.16.0-6-loongson-2e 
(debian-kernel@lists.debian.org) (gcc version 4.8.4 (Debian 4.8.4-1) ) #1 
Debian 3.16.56-1+deb8u1 (2018-05-08)
  console: [    0.000000] memsize=256, highmemsize=0
  console: [    0.000000] CpuClock = 533080000
  console: [    0.000000] bootconsole [early0] enabled
  console: [    0.000000] CPU0 revision is: 00006302 (ICT Loongson-2)
  console: [    0.000000] FPU revision is: 00000501
  console: [    0.000000] Checking for the multiply/shift bug... no.
  console: [    0.000000] Checking for the daddiu bug... no.
  console: [    0.000000] Determined physical RAM map:
  console: [    0.000000]  memory: 0000000010000000 @ 0000000000000000 (usable)
  console: [    0.000000]  memory: 0000000004000000 @ 0000000010000000 
(reserved)
  console: [    0.000000]  memory: 0000000003ffffff @ 000000001c000001 
(reserved)
  console: [    0.000000] Initrd not found or empty - disabling initrd
  console: [    0.000000] Zone ranges:
  console: [    0.000000]   DMA      [mem 0x00000000-0x00ffffff]
  console: [    0.000000]   Normal   [mem 0x01000000-0x0fffffff]
  console: [    0.000000] Movable zone start for each node
  console: [    0.000000] Early memory node ranges
  console: [    0.000000]   node   0: [mem 0x00000000-0x0fffffff]
  console: [    0.000000] Reserving 0MB of memory at 0MB for crashkernel
  console: [    0.000000] Primary instruction cache 64kB, VIPT, direct mapped, 
linesize 32 bytes.
  console: [    0.000000] Primary data cache 64kB, 4-way, VIPT, no aliases, 
linesize 32 bytes
  console: [    0.000000] Unified secondary cache 512kB 4-way, linesize 32 
bytes.
  console: [    0.000000] Built 1 zonelists in Zone order, mobility grouping 
on.  Total pages: 16327
  console: [    0.000000] Kernel command line: printk.time=0 console=ttyS0
  PASS (2.27 s)

Signed-off-by: Jiaxun Yang <jiaxun.yang@flygoat.com>
Reviewed-by: Wainer dos Santos Moschetta <wainersm@redhat.com>
Reviewed-by: Willian Rampazzo <willianr@redhat.com>
Reviewed-by: Huacai Chen <chenhuacai@kernel.org>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Tested-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Message-Id: <20201224031750.52146-9-jiaxun.yang@flygoat.com>
[PMD: Added command line example]
Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>


  Commit: 2e0b5bbe813930021b2baab03c9d424c1c52d18b
      
https://github.com/qemu/qemu/commit/2e0b5bbe813930021b2baab03c9d424c1c52d18b
  Author: Peter Maydell <peter.maydell@linaro.org>
  Date:   2021-01-05 (Tue, 05 Jan 2021)

  Changed paths:
    M MAINTAINERS
    M default-configs/devices/mips64el-softmmu.mak
    M docs/devel/clocks.rst
    M docs/system/target-mips.rst
    M hw/arm/virt.c
    M hw/audio/meson.build
    A hw/audio/via-ac97.c
    M hw/core/clock.c
    M hw/hppa/dino.c
    M hw/i386/xen/xen-hvm.c
    M hw/intc/loongson_liointc.c
    M hw/isa/piix3.c
    M hw/isa/trace-events
    M hw/isa/vt82c686.c
    M hw/mips/Kconfig
    M hw/mips/fuloong2e.c
    A hw/mips/fw_cfg.c
    A hw/mips/fw_cfg.h
    M hw/mips/gt64xxx_pci.c
    A hw/mips/loongson3_bootp.c
    A hw/mips/loongson3_bootp.h
    A hw/mips/loongson3_virt.c
    M hw/mips/malta.c
    M hw/mips/meson.build
    M hw/mips/mipssim.c
    M hw/pci-host/bonito.c
    M hw/pci-host/pnv_phb4.c
    M hw/pci-host/ppce500.c
    M hw/pci-host/uninorth.c
    M hw/ppc/ppc4xx_pci.c
    M hw/sh4/sh_pci.c
    M include/hw/clock.h
    A include/hw/intc/loongson_liointc.h
    M include/hw/isa/vt82c686.h
    M softmmu/qdev-monitor.c
    M target/mips/cpu.c
    M tests/acceptance/boot_linux_console.py

  Log Message:
  -----------
  Merge remote-tracking branch 'remotes/philmd-gitlab/tags/mips-20210104' into 
staging

MIPS patches queue

- Use PCI macros (Philippe Mathieu-Daudé)
- Clean up VT82C686B south bridge (BALATON Zoltan)
- Introduce clock_ticks_to_ns() (Peter Maydell)
- Add Loongson-3 machine (Huacai Chen)
- Make addresses used by bootloader unsigned (Jiaxun Yang)
- Clean fuloong2e PROM environment (Jiaxun Yang)
- Add integration test of fuloong2e booting Linux (Jiaxun Yang)

# gpg: Signature made Mon 04 Jan 2021 22:37:48 GMT
# gpg:                using RSA key FAABE75E12917221DCFD6BB2E3E32C2CDEADC0DE
# gpg: Good signature from "Philippe Mathieu-Daudé (F4BUG) <f4bug@amsat.org>" 
[full]
# Primary key fingerprint: FAAB E75E 1291 7221 DCFD  6BB2 E3E3 2C2C DEAD C0DE

* remotes/philmd-gitlab/tags/mips-20210104: (35 commits)
  tests/acceptance: Test boot_linux_console for fuloong2e
  hw/mips/fuloong2e: Correct cpuclock in PROM environment
  hw/mips/fuloong2e: Remove unused env entry
  hw/mips/fuloong2e: Replace faulty documentation links
  hw/mips/fuloong2e: Remove define DEBUG_FULOONG2E_INIT
  hw/mips: Use address translation helper to handle ENVP_ADDR
  hw/mips/malta: Use address translation helper to calculate bootloader_run_addr
  hw/mips: Make bootloader addresses unsigned
  docs/system: Update MIPS machine documentation
  hw/mips: Add Loongson-3 machine support
  hw/mips: Add Loongson-3 boot parameter helpers
  hw/mips: Implement fw_cfg_arch_key_name()
  hw/intc: Rework Loongson LIOINTC
  clock: Define and use new clock_display_freq()
  clock: Remove clock_get_ns()
  target/mips: Don't use clock_get_ns() in clock period calculation
  clock: Introduce clock_ticks_to_ns()
  vt82c686: Rename superio config related parts
  vt82c686: Use shorter name for local variable holding object state
  vt82c686: Remove unneeded includes and defines
  ...

Signed-off-by: Peter Maydell <peter.maydell@linaro.org>


Compare: https://github.com/qemu/qemu/compare/52d25464605d...2e0b5bbe8139



reply via email to

[Prev in Thread] Current Thread [Next in Thread]