SerialICE February 2013

serialice@serialice.com

1 participants
29 discussions

Patch set updated for serialice: 07cdc0d SerialICE: update qemu-0.15.x/build.sh for glibc 2.17
by Idwer Vollering 25 Feb '13

25 Feb '13

Idwer Vollering (vidwer(a)gmail.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/2504 -gerrit commit 07cdc0d19c5e35b54145444262849338c7d7a595 Author: Idwer Vollering <vidwer(a)gmail.com> Date: Mon Feb 25 03:49:04 2013 +0100 SerialICE: update qemu-0.15.x/build.sh for glibc 2.17 glibc 2.17 and above use librt. Update qemu-0.15.x/build.sh to compile when glibc >= 2.17 is installed. This solves linker errors similar to this: (32-bit archlinux installation with glibc 2.17-3, binutils 2.23.1-3, glib2 2.34.3-1) /usr/bin/ld: ../qemu-timer.o: undefined reference to symbol 'timer_settime@@GLIBC_2.2' /usr/bin/ld: note: 'timer_settime@@GLIBC_2.2' is defined in DSO /usr/lib/librt.so.1 so try adding it to the linker command line /usr/lib/librt.so.1: could not read symbols: Invalid operation Change-Id: I2d335d065457788ef8aada7d0d500e84ba31e4d0 Signed-off-by: Idwer Vollering <vidwer(a)gmail.com> --- qemu-0.15.x/build.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/qemu-0.15.x/build.sh b/qemu-0.15.x/build.sh index 2cac6f5..3dfd3a5 100644 --- a/qemu-0.15.x/build.sh +++ b/qemu-0.15.x/build.sh @@ -1,5 +1,5 @@ #!/bin/sh -./configure --extra-ldflags="-ldl" --disable-kvm --disable-sdl --enable-serialice \ +./configure --extra-ldflags="-ldl -lrt" --disable-kvm --disable-sdl --enable-serialice \ --target-list="x86_64-softmmu, i386-softmmu" make

1 0

Patch merged into serialice/master: d97f5b2 Fix some Intel ICH7 filters
by gerrit＠coreboot.org 25 Feb '13

25 Feb '13

the following patch was just integrated into master: commit d97f5b23aa8945c5b4a34e10f011d26859b430b4 Author: Kyösti Mälkki <kyosti.malkki(a)gmail.com> Date: Fri Feb 22 20:10:08 2013 +0200 Fix some Intel ICH7 filters The enable bits to route serial COM port ranges to LPC are a byte register 0x82. Remove incorrect AC'97 device mappings. Change-Id: If7b89838d94d953ac19d97d5573bc866bb71d2ba Signed-off-by: Kyösti Mälkki <kyosti.malkki(a)gmail.com> Reviewed-on: http://review.coreboot.org/2486 Tested-by: build bot (Jenkins) Reviewed-by: Stefan Reinauer <stefan.reinauer(a)coreboot.org> Build-Tested: build bot (Jenkins) at Fri Feb 22 20:01:08 2013, giving +1 Reviewed-By: Stefan Reinauer <stefan.reinauer(a)coreboot.org> at Sat Feb 23 01:15:47 2013, giving +2 See http://review.coreboot.org/2486 for details. -gerrit

1 0

Patch merged into serialice/master: e430588 Add Commell LV-672
by gerrit＠coreboot.org 25 Feb '13

25 Feb '13

the following patch was just integrated into master: commit e4305885e964c333fa96b8759c23a34405423b5f Author: Kyösti Mälkki <kyosti.malkki(a)gmail.com> Date: Tue Feb 19 23:50:10 2013 +0200 Add Commell LV-672 Change-Id: I56a2eb48a53c20a055670cab06683a938ec18d5a Signed-off-by: Kyösti Mälkki <kyosti.malkki(a)gmail.com> Reviewed-on: http://review.coreboot.org/2478 Tested-by: build bot (Jenkins) Reviewed-by: Martin Roth <martin.roth(a)se-eng.com> Reviewed-by: Stefan Reinauer <stefan.reinauer(a)coreboot.org> Build-Tested: build bot (Jenkins) at Fri Feb 22 19:55:11 2013, giving +1 Reviewed-By: Stefan Reinauer <stefan.reinauer(a)coreboot.org> at Sat Feb 23 01:15:13 2013, giving +2 See http://review.coreboot.org/2478 for details. -gerrit

1 0

Patch merged into serialice/master: 5ad5edc Add MSI MS-7133
by gerrit＠coreboot.org 25 Feb '13

25 Feb '13

the following patch was just integrated into master: commit 5ad5edc94020fbad9c39902f33a567371abb1b47 Author: Kyösti Mälkki <kyosti.malkki(a)gmail.com> Date: Sat Nov 10 18:59:43 2012 +0200 Add MSI MS-7133 Change-Id: I05f50ef3867d2ef0b9ad77d5afd9f49ca5856a77 Signed-off-by: Kyösti Mälkki <kyosti.malkki(a)gmail.com> Reviewed-on: http://review.coreboot.org/2477 Tested-by: build bot (Jenkins) Reviewed-by: Paul Menzel <paulepanter(a)users.sourceforge.net> Reviewed-by: Stefan Reinauer <stefan.reinauer(a)coreboot.org> Build-Tested: build bot (Jenkins) at Fri Feb 22 19:34:22 2013, giving +1 Reviewed-By: Stefan Reinauer <stefan.reinauer(a)coreboot.org> at Sat Feb 23 01:14:39 2013, giving +2 See http://review.coreboot.org/2477 for details. -gerrit

1 0

New patch to review for serialice: bdb5b3b Drop io_base_post
by Kyösti Mälkki 25 Feb '13

25 Feb '13

Kyösti Mälkki (kyosti.malkki(a)gmail.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/2507 -gerrit commit bdb5b3bfd747d0fb76dd558f6e15d32b751ca7f9 Author: Kyösti Mälkki <kyosti.malkki(a)gmail.com> Date: Mon Feb 25 14:50:06 2013 +0200 Drop io_base_post Add a field .decode in filter to control if it operates on a set of fixed addresses or a range. Change-Id: I56a1403dcd0fd887dec6ff9cb3b2b603ed1ce285 Signed-off-by: Kyösti Mälkki <kyosti.malkki(a)gmail.com> --- SerialICE/simba/core_io.lua | 27 ++++++++++++++------------- SerialICE/simba/hooks.lua | 3 ++- 2 files changed, 16 insertions(+), 14 deletions(-) diff --git a/SerialICE/simba/core_io.lua b/SerialICE/simba/core_io.lua index 4234239..5781b5a 100644 --- a/SerialICE/simba/core_io.lua +++ b/SerialICE/simba/core_io.lua @@ -13,19 +13,19 @@ function io_undefined(f, action) end function io_post(f, action) - if (action.write) then - printk(f, action, "out%s %04x <= %s\n", size_suffix(action.size), action.addr, size_data(action.size, action.data)) - else - printk(f, action, " in%s %04x => %s\n", size_suffix(action.size), action.addr, size_data(action.size, action.data)) - end - return true -end - -function io_base_post(f, action) - if (action.write) then - printk(f, action, "[%04x] <= %s\n", action.addr - f.base, size_data(action.size, action.data)) - else - printk(f, action, "[%04x] => %s\n", action.addr - f.base, size_data(action.size, action.data)) + local size = size_data(action.size, action.data) + if not f.decode or f.decode == F_FIXED then + if (action.write) then + printk(f, action, "out%s %04x <= %s\n", size_suffix(action.size), action.addr, size) + else + printk(f, action, " in%s %04x => %s\n", size_suffix(action.size), action.addr, size) + end + elseif f.decode == F_RANGE then + if (action.write) then + printk(f, action, "[%04x] <= %s\n", action.addr - f.base, size) + else + printk(f, action, "[%04x] => %s\n", action.addr - f.base, size) + end end return true end @@ -35,6 +35,7 @@ filter_io_fallback = { name = "IO", pre = io_undefined, post = io_post, + decode = F_FIXED, base = 0x0, size = 0x10000, } diff --git a/SerialICE/simba/hooks.lua b/SerialICE/simba/hooks.lua index c7cc736..29f574a 100644 --- a/SerialICE/simba/hooks.lua +++ b/SerialICE/simba/hooks.lua @@ -149,7 +149,8 @@ function generic_io_bar(bar) local f = {} f.id = -1 f.pre = handle_action - f.post = io_base_post + f.post = io_post + f.decode = F_RANGE f.hide = true f.name = bar.name f.size = bar.size

1 0

New patch to review for serialice: 27e1474 Drop mem_base_post
by Kyösti Mälkki 25 Feb '13

25 Feb '13

Kyösti Mälkki (kyosti.malkki(a)gmail.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/2506 -gerrit commit 27e1474304fc60c8304308e74472a4c5b8654b25 Author: Kyösti Mälkki <kyosti.malkki(a)gmail.com> Date: Mon Feb 25 14:49:52 2013 +0200 Drop mem_base_post Add a field .decode in filter to control if it operates on a set of fixed addresses or a range. Change-Id: I7bf7c323a339427357b3fc69dc2e2a84c14c3dd8 Signed-off-by: Kyösti Mälkki <kyosti.malkki(a)gmail.com> --- SerialICE/simba/core_io.lua | 32 ++++++++++++++++++-------------- SerialICE/simba/hooks.lua | 3 ++- SerialICE/simba/i82801.lua | 3 ++- SerialICE/simba/mmio.lua | 6 ++++-- 4 files changed, 26 insertions(+), 18 deletions(-) diff --git a/SerialICE/simba/core_io.lua b/SerialICE/simba/core_io.lua index 0b823cc..4234239 100644 --- a/SerialICE/simba/core_io.lua +++ b/SerialICE/simba/core_io.lua @@ -1,3 +1,7 @@ + +F_FIXED = 1 +F_RANGE = 2 + -- ********************************************************** -- @@ -46,10 +50,19 @@ function mem_undefined(f, action) end function mem_post(f, action) - if (action.write) then - printk(f, action, "write%s %08x <= %s", size_suffix(action.size), action.addr, size_data(action.size, action.data)) - else - printk(f, action, " read%s %08x => %s", size_suffix(action.size), action.addr, size_data(action.size, action.data)) + local size = size_data(action.size, action.data) + if not f.decode or f.decode == F_FIXED then + if (action.write) then + printk(f, action, "write%s %08x <= %s", size_suffix(action.size), action.addr, size) + else + printk(f, action, " read%s %08x => %s", size_suffix(action.size), action.addr, size) + end + elseif f.decode == F_RANGE then + if (action.write) then + printk(f, action, "[%08x] <= %s", bit32.band(action.addr, (f.size - 1)), size) + else + printk(f, action, "[%08x] => %s", bit32.band(action.addr, (f.size - 1)), size) + end end if action.to_hw then printf(" *") @@ -58,21 +71,12 @@ function mem_post(f, action) return true end -function mem_base_post(f, action) - if (action.write) then - printk(f, action, "[%08x] <= %s\n", bit32.band(action.addr, (f.size - 1)), size_data(action.size, action.data)) - else - printk(f, action, "[%08x] => %s\n", bit32.band(action.addr, (f.size - 1)), size_data(action.size, action.data)) - end - return true -end - - filter_mem_fallback = { id = -1, name = "MEM", pre = mem_undefined, post = mem_post, + decode = F_FIXED, base = 0x0, size = 0x100000000 } diff --git a/SerialICE/simba/hooks.lua b/SerialICE/simba/hooks.lua index fa61ab4..c7cc736 100644 --- a/SerialICE/simba/hooks.lua +++ b/SerialICE/simba/hooks.lua @@ -168,7 +168,8 @@ function generic_mmio_bar(bar) local f = {} f.id = -1 f.pre = handle_action - f.post = mem_base_post + f.post = mem_post + f.decode = F_RANGE f.hide = true f.name = bar.name f.size = bar.size diff --git a/SerialICE/simba/i82801.lua b/SerialICE/simba/i82801.lua index ddc9a6e..28b506f 100644 --- a/SerialICE/simba/i82801.lua +++ b/SerialICE/simba/i82801.lua @@ -149,7 +149,8 @@ filter_ich7_tpm = { id = -1, name = "ICH7 TPM", pre = mem_target_only, - post = mem_base_post, + post = mem_post, + decode = F_RANGE, base = 0xfed40000, size = 0x00001000, hide = true diff --git a/SerialICE/simba/mmio.lua b/SerialICE/simba/mmio.lua index 695315f..681953b 100644 --- a/SerialICE/simba/mmio.lua +++ b/SerialICE/simba/mmio.lua @@ -20,7 +20,8 @@ filter_lapic = { id = -1, name = "LAPIC", pre = mem_lapic, - post = mem_base_post, + post = mem_post, + decode = F_RANGE, hide = true, base = 0xfee00000, size = 0x00010000, @@ -31,7 +32,8 @@ filter_ioapic = { id = -1, name = "IOAPIC", pre = mem_target_only, - post = mem_base_post, + post = mem_post, + decode = F_RANGE, hide = true, base = 0xfec00000, size = 0x00010000,

1 0

New patch to review for serialice: 95002ca Add Intel i845 filter
by Kyösti Mälkki 25 Feb '13

25 Feb '13

Kyösti Mälkki (kyosti.malkki(a)gmail.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/2505 -gerrit commit 95002caadf022d3a62bfffab616a0dce291a070d Author: Kyösti Mälkki <kyosti.malkki(a)gmail.com> Date: Mon Feb 25 14:53:13 2013 +0200 Add Intel i845 filter Catch access to RCOMP bar registers on i845 northbridge. Change-Id: If73fb787da491b7c22327debc6034dbc30d6e43f Signed-off-by: Kyösti Mälkki <kyosti.malkki(a)gmail.com> --- SerialICE/simba/asrock_775i65g.lua | 2 ++ SerialICE/simba/asus_p4p800_vm.lua | 2 ++ SerialICE/simba/intel_bars.lua | 12 ++++++++++++ 3 files changed, 16 insertions(+) diff --git a/SerialICE/simba/asrock_775i65g.lua b/SerialICE/simba/asrock_775i65g.lua index 7021554..7aeb68d 100644 --- a/SerialICE/simba/asrock_775i65g.lua +++ b/SerialICE/simba/asrock_775i65g.lua @@ -1,10 +1,12 @@ dofile("i82801.lua") +dofile("intel_bars.lua") function do_mainboard_setup() do_default_setup() enable_hook_i82801dx() + northbridge_i845() -- Apply mainboard hooks last, so they are the first ones to check --enable_hook(io_hooks, filter_mainboard) diff --git a/SerialICE/simba/asus_p4p800_vm.lua b/SerialICE/simba/asus_p4p800_vm.lua index aad1932..fbc8c44 100644 --- a/SerialICE/simba/asus_p4p800_vm.lua +++ b/SerialICE/simba/asus_p4p800_vm.lua @@ -72,11 +72,13 @@ filter_mainboard = { } dofile("i82801.lua") +dofile("intel_bars.lua") function do_mainboard_setup() do_default_setup() enable_hook_i82801dx() + northbridge_i845() -- Apply mainboard hooks last, so they are the first ones to check enable_hook(io_hooks, filter_mainboard) diff --git a/SerialICE/simba/intel_bars.lua b/SerialICE/simba/intel_bars.lua index 1e42cac..0841a23 100644 --- a/SerialICE/simba/intel_bars.lua +++ b/SerialICE/simba/intel_bars.lua @@ -1,3 +1,15 @@ +-- ********************************************************** +-- Intel i845 + +dev_i845_mch2 = { + pci_dev = pci_bdf(0x0,0x6,0x0,0x0), + name = "MCH", + bar = {} +} + +function northbridge_i845() + add_mem_bar(dev_i845_mch2, 0x10, "RCOMP", 0x1000) +end -- ********************************************************** -- Intel e7505

1 0

New patch to review for serialice: a96cec2 SerialICE: update qemu-0.15.x/build.sh
by Idwer Vollering 25 Feb '13

25 Feb '13

Idwer Vollering (vidwer(a)gmail.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/2504 -gerrit commit a96cec2f09cdf0c1d91fa9be3c4b65c3651d251d Author: Idwer Vollering <vidwer(a)gmail.com> Date: Mon Feb 25 03:49:04 2013 +0100 SerialICE: update qemu-0.15.x/build.sh More recent (newer) builds of glibc need to be able to find, and use, librt. Update qemu-0.15.x/build.sh to make building work again. Solve linker errors that look similar to this (32-bit archlinux installation): /usr/bin/ld: ../qemu-timer.o: undefined reference to symbol 'timer_settime@@GLIBC_2.2' /usr/bin/ld: note: 'timer_settime@@GLIBC_2.2' is defined in DSO /usr/lib/librt.so.1 so try adding it to the linker command line /usr/lib/librt.so.1: could not read symbols: Invalid operation Change-Id: I2d335d065457788ef8aada7d0d500e84ba31e4d0 Signed-off-by: Idwer Vollering <vidwer(a)gmail.com> --- qemu-0.15.x/build.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/qemu-0.15.x/build.sh b/qemu-0.15.x/build.sh index 2cac6f5..3dfd3a5 100644 --- a/qemu-0.15.x/build.sh +++ b/qemu-0.15.x/build.sh @@ -1,5 +1,5 @@ #!/bin/sh -./configure --extra-ldflags="-ldl" --disable-kvm --disable-sdl --enable-serialice \ +./configure --extra-ldflags="-ldl -lrt" --disable-kvm --disable-sdl --enable-serialice \ --target-list="x86_64-softmmu, i386-softmmu" make

1 0

New patch to review for serialice: 90b3c55 Fix replay of old logfiles
by Kyösti Mälkki 25 Feb '13

25 Feb '13

Kyösti Mälkki (kyosti.malkki(a)gmail.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/2503 -gerrit commit 90b3c55c24950702738419ecff7a8569e05e4ef4 Author: Kyösti Mälkki <kyosti.malkki(a)gmail.com> Date: Mon Feb 25 03:05:13 2013 +0200 Fix replay of old logfiles Old serialice logfiles had removed I/O accesses of PCI configuration register accesses. Replayer script needs to inject those lines back in. This was broken, as pci_bdf() no longer converts to the format required for I/O 0xcf8 register. Change-Id: I9ba2b76a8353895ac431eaaa355e0d8ec8381e02 Signed-off-by: Kyösti Mälkki <kyosti.malkki(a)gmail.com> --- SerialICE/simba/replay.lua | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/SerialICE/simba/replay.lua b/SerialICE/simba/replay.lua index fcfabf3..f3ba17f 100644 --- a/SerialICE/simba/replay.lua +++ b/SerialICE/simba/replay.lua @@ -11,6 +11,10 @@ SerialICE_mainboard = "undetected" regs = { eax, ebc, ecx, edx, cs=0, eip=0, ds, es, ss, gs, fs, } ids = { parent, this, } +function pci_bdf_noext(bus, dev, func, reg) + return bus*65536 + dev*2048 + func*256 + reg +end + function replay_io(dir_wr, addr, size, data) pre_action(io_action, dir_wr, addr, size, data) walk_pre_hooks(io_hooks, io_action) @@ -102,7 +106,7 @@ function parse_pci(line) local ndata = tonumber(data,16) local nsize = string.len(data)/2 - replay_io(true, 0xcf8, 4, pci_bdf(tonumber(bus,16), tonumber(dev,16), tonumber(fn,16), nreg)) + replay_io(true, 0xcf8, 4, pci_bdf_noext(tonumber(bus,16), tonumber(dev,16), tonumber(fn,16), nreg)) if string.find("<=", dir) then replay_io(true, 0xcfc + noff, nsize, ndata) else

1 0

Patch merged into serialice/master: 31de5de Add some Intel ICH6 filters
by gerrit＠coreboot.org 23 Feb '13

23 Feb '13

the following patch was just integrated into master: commit 31de5de7953ae29a91c96821566f52fb4e744063 Author: Kyösti Mälkki <kyosti.malkki(a)gmail.com> Date: Wed Feb 20 00:08:20 2013 +0200 Add some Intel ICH6 filters Only the most useful BARs added. Change-Id: I4f3b87f939849fe814045604fee9bc392159aa21 Signed-off-by: Kyösti Mälkki <kyosti.malkki(a)gmail.com> Reviewed-on: http://review.coreboot.org/2476 Tested-by: build bot (Jenkins) Reviewed-by: Paul Menzel <paulepanter(a)users.sourceforge.net> Reviewed-by: Stefan Reinauer <stefan.reinauer(a)coreboot.org> Build-Tested: build bot (Jenkins) at Fri Feb 22 19:39:55 2013, giving +1 Reviewed-By: Stefan Reinauer <stefan.reinauer(a)coreboot.org> at Sat Feb 23 01:12:54 2013, giving +2 See http://review.coreboot.org/2476 for details. -gerrit

1 0

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

SerialICE February 2013