On 03/04/2015 01:35 AM, Takashi Iwai wrote:
At Wed, 04 Mar 2015 01:01:32 -0800, Alnie wrote:
On 03/04/2015 12:00 AM, Takashi Iwai wrote:
At Tue, 03 Mar 2015 18:12:31 -0800, Alnie wrote:
My suggestion isn't about a compile option but that you add some debug printk() calls manually around some codes. We need to know the value written and read by azx_write*() and azx_read*() calls. Especially the value read in pci_azx_read*() is more interesting. You can try to modify sound/pci/hda/hda_intel.c and add a printk() to each pci_azx_read*() function for printing the value to be returned. Beware that this will likely flood many messages, so just try once.
Takashi
I can not find any reference to pci_azx_read in hda_intel.c
You must be using a too old kernel, then. Please use the latest kernel for debugging.
Takashi
Ok. I now have latest kernel.
Here is a small portion...
/* PCI register access. */ static void pci_azx_writel(u32 value, u32 __iomem *addr) { writel(value, addr); }
static u32 pci_azx_readl(u32 __iomem *addr) { return readl(addr); }
Can you show me how I can properly place printk without breaking things and produce relevant messages?
Something like:
static u32 pci_azx_readl(u32 __iomem *addr) { u32 val = readl(addr); pr_info("XXX readl %p %x\n", addr, val); return val; }
But since there are quite lots of accesses, it might be safer to use the ratelimited version. Use like the following: pr_info_ratelimited("XXX readl %p %x\n", addr, val);
Also there are variants pci_azx_readw() and _readb().
Takashi
Ok, I was only able to guess through the code for the read values. If you need write information too please show me how. here are the mods I made...
/* PCI register access. */ static void pci_azx_writel(u32 value, u32 __iomem *addr) { writel(value, addr); }
static u32 pci_azx_readl(u32 __iomem *addr) { u32 val = readl(addr); pr_info_ratelimited("XXX readl %p %x\n", addr, val); return val; }
static void pci_azx_writew(u16 value, u16 __iomem *addr) { writew(value, addr); }
static u16 pci_azx_readw(u16 __iomem *addr) { u16 val = readw(addr); pr_info_ratelimited("XXX readw %p %x\n", addr, val); return val; }
static void pci_azx_writeb(u8 value, u8 __iomem *addr) { writeb(value, addr); }
static u8 pci_azx_readb(u8 __iomem *addr) { u8 val = readb(addr); pr_info_ratelimited("XXX readb %p %x\n", addr, val); return val; }
and i was able to produce this...
dmesg | grep -i read [ 0.764102] tpm_tis 00:05: A TPM error (7) occurred attempting to read a pcr value [ 0.769674] Write protecting the kernel read-only data: 12288k [ 1.729952] sd 0:0:0:0: [sda] Write cache: enabled, read cache: enabled, doesn't support DPO or FUA [ 2.071100] sd 1:0:0:0: [sdb] Write cache: enabled, read cache: enabled, doesn't support DPO or FUA [ 2.778875] EXT4-fs (sda5): INFO: recovery required on readonly filesystem [ 2.817982] EXT4-fs (sda5): orphan cleanup on readonly fs [ 4.662442] XXX readw ffffc90004f30000 4401 [ 4.662466] XXX readl ffffc90004f30008 1 [ 4.662468] XXX readb ffffc90004f30008 1 [ 4.662609] XXX readw ffffc90004f70000 3300 [ 4.662631] XXX readl ffffc90004f70008 0 [ 4.662635] XXX readb ffffc90004f70008 0 [ 4.663483] XXX readb ffffc90004f30008 0 [ 4.663492] XXX readb ffffc90004f70008 0 [ 4.663496] XXX readb ffffc90004f70008 0 [ 4.664138] XXX readb ffffc90004f30008 0 [ 4.664142] XXX readb ffffc90004f30008 0 [ 4.664150] XXX readb ffffc90004f70008 1 [ 4.665173] XXX readb ffffc90004f30008 1 [ 4.665368] XXX readb ffffc90004f70008 1 [ 4.665372] XXX readw ffffc90004f7000e 2 [ 4.665378] XXX readl ffffc90004f70020 0 [ 4.666389] XXX readw ffffc90004f3000e 1 [ 4.666394] XXX readl ffffc90004f30020 0 [ 4.668203] XXX readw ffffc90004f70068 0 [ 4.668209] XXX readw ffffc90004f70068 0 [ 4.668213] XXX readw ffffc90004f70068 0 [ 4.668216] XXX readw ffffc90004f70068 0 [ 4.668221] XXX readw ffffc90004f70068 0 [ 4.668226] XXX readw ffffc90004f70068 0 [ 4.668412] XXX readl ffffc90004f70020 c0000000 [ 4.668420] XXX readl ffffc90004f70008 1 [ 4.671683] XXX readl ffffc90004f70020 0 [ 4.674569] XXX readl ffffc90004f30064 14f15051 [ 4.674608] XXX readl ffffc90004f30064 14f15051 [ 4.674651] XXX readl ffffc90004f30064 0 [ 5.624194] IPv6: ADDRCONF(NETDEV_UP): eth0: link is not ready [ 7.297049] IPv6: ADDRCONF(NETDEV_CHANGE): eth0: link becomes ready [ 15.003841] pci_azx_readl: 160 callbacks suppressed [ 15.003845] XXX readl ffffc90004f30024 0 [ 15.003911] XXX readl ffffc90004f30024 0 [ 16.334821] pci_azx_readb: 12 callbacks suppressed [ 16.334826] XXX readb ffffc90004f30080 0 [ 16.334829] XXX readb ffffc90004f30080 0 [ 16.334834] XXX readb ffffc90004f30080 1 [ 16.334839] XXX readb ffffc90004f30080 0 [ 16.334844] XXX readb ffffc90004f30080 0 [ 16.334846] XXX readl ffffc90004f30080 40000 [ 16.334849] XXX readl ffffc90004f30070 31303000 [ 16.334852] XXX readl ffffc90004f30080 140000 [ 16.334857] pci_azx_readw: 3599 callbacks suppressed [ 16.334858] XXX readw ffffc90004f30068 2 [ 16.334860] XXX readw ffffc90004f30068 2 [ 16.334862] XXX readw ffffc90004f30068 0 [ 16.334864] XXX readw ffffc90004f30068 1 [ 16.334868] XXX readw ffffc90004f30068 1 [ 16.334871] XXX readw ffffc90004f30068 1 [ 16.334874] XXX readw ffffc90004f30068 1 [ 16.334877] XXX readw ffffc90004f30068 1 [ 16.334880] XXX readw ffffc90004f30068 1 [ 16.334884] XXX readw ffffc90004f30068 1 [ 16.334902] XXX readl ffffc90004f30064 0 [ 16.334943] XXX readl ffffc90004f30064 0 [ 16.334985] XXX readl ffffc90004f30064 11 [ 16.340093] XXX readl ffffc90004f30064 0 [ 16.340115] XXX readb ffffc90004f30080 1c [ 16.340118] XXX readb ffffc90004f30080 0 [ 16.340123] XXX readb ffffc90004f30080 1 [ 16.340128] XXX readb ffffc90004f30080 0 [ 16.340132] XXX readb ffffc90004f30080 0 [ 16.340134] XXX readl ffffc90004f30080 40000 [ 20.018443] pci_azx_readl: 148 callbacks suppressed [ 20.018448] XXX readl ffffc90004f30030 15f6582b [ 20.018709] XXX readl ffffc90004f30030 15f6715a [ 20.018716] XXX readl ffffc90004f30030 15f67203 [ 20.370026] XXX readl ffffc90004f30030 16770835 [ 20.370737] XXX readl ffffc90004f30030 16774b57 [ 20.370752] XXX readl ffffc90004f30030 16774cd3 [ 20.721866] XXX readl ffffc90004f30030 16f7d0db [ 20.722577] XXX readl ffffc90004f30030 16f813de [ 20.722592] XXX readl ffffc90004f30030 16f81544 [ 21.073497] XXX readl ffffc90004f30030 17788616 [ 21.447607] pci_azx_readb: 75 callbacks suppressed [ 21.447617] XXX readb ffffc90004f30100 1e [ 21.447624] XXX readb ffffc90004f30100 2 [ 21.447629] XXX readb ffffc90004f30100 2 [ 21.447633] XXX readb ffffc90004f30100 2 [ 21.447637] XXX readb ffffc90004f30100 0 [ 21.879992] XXX readb ffffc90004f30100 0 [ 21.879998] XXX readb ffffc90004f30100 0 [ 21.880032] XXX readb ffffc90004f30100 1 [ 21.880039] XXX readb ffffc90004f30100 0 [ 21.880043] XXX readb ffffc90004f30100 0 [ 21.880051] pci_azx_readw: 855 callbacks suppressed [ 21.880052] XXX readw ffffc90004f30110 bf [ 21.880075] XXX readw ffffc90004f30110 bf [ 25.032271] pci_azx_readl: 896 callbacks suppressed [ 25.032281] XXX readl ffffc90004f30030 1d218b30 [ 25.033006] XXX readl ffffc90004f30030 1d21d08e [ 25.033021] XXX readl ffffc90004f30030 1d21d1f9 [ 25.393756] XXX readl ffffc90004f30030 1da5db6d [ 25.394488] XXX readl ffffc90004f30030 1da62083 [ 25.394503] XXX readl ffffc90004f30030 1da62206 [ 25.755381] XXX readl ffffc90004f30030 1e2a394b [ 25.755746] XXX readl ffffc90004f30030 1e2a5bab [ 25.755754] XXX readl ffffc90004f30030 1e2a5c65 [ 26.116985] XXX readl ffffc90004f30030 1eae9483 [ 29.677529] pci_azx_readb: 11 callbacks suppressed [ 29.677540] XXX readb ffffc90004f30100 1e [ 29.677547] XXX readb ffffc90004f30100 2 [ 29.677551] XXX readb ffffc90004f30100 2 [ 29.677556] XXX readb ffffc90004f30100 2 [ 29.677560] XXX readb ffffc90004f30100 2 [ 29.677565] XXX readb ffffc90004f30100 0
-Alnie