Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
MIME-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit
Date:   Thu, 05 Jul 2018 17:55:31 +0200
From:   Stefan Agner <stefan@agner.ch>
To:     Angelo Dureghello <angelo@sysam.it>, Vinod <vkoul@kernel.org>
Cc:     dmaengine@vger.kernel.org, Krzysztof Kozlowski <krzk@kernel.org>,
        Dan Williams <dan.j.williams@intel.com>,
        linux-kernel@vger.kernel.org
Subject: Re: [BUG] Serial/dma stall/failure after "dmaengine: fsl-edma:
 extract common fsl-edma code (no changes in behavior intended)"
In-Reply-To: <20180705124526.GA3620@jerusalem>
References: <CAJKOXPcVFrkEA4YV3gi-gPYhtTXHeKtv4CvL29gstfTacz2sMg@mail.gmail.com>
 <20180705073552.GI22377@vkoul-mobl> <20180705081253.GA3513@jerusalem>
 <20180705124526.GA3620@jerusalem>
Message-ID: <7d2bc5bc8cd01347acf1858ba816b12d@agner.ch>
User-Agent: Roundcube Webmail/1.3.4
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

Hi Vinod, Hi Angelo,

On 05.07.2018 14:45, Angelo Dureghello wrote:
> Hi Vinod,
> 
> On Thu, Jul 05, 2018 at 10:12:53AM +0200, Angelo Dureghello wrote:
>> Hi Vinod,
>>
>> On Thu, Jul 05, 2018 at 01:05:52PM +0530, Vinod wrote:
>> > On 04-07-18, 10:54, Krzysztof Kozlowski wrote:
>> > > Hi,
>> > >
>> > > The commit 6ad069123f03bebe4315dea13d44845854ca6043 ("dmaengine:
>> > > fsl-edma: extract common fsl-edma code (no changes in behavior
>> > > intended)"), even though marked as no changes in behavior intended...
>> > > make serial console with DMA broken after boot. The console just hangs
>> > > - is not responsive even to SysRq. Usually after finishing boot -
>> > > before or after login prompt. Sometimes login is allowed and then it
>> > > hangs during printing dmesg.
>> > >
>> > > Board: Toradex Colibri VF50 (NXP VF500, Cortex A5, serial configured
>> > > with DMA) on Iris Carrier.
>> >
>> > Angelo ?
>> >
>> sorry for this. As said i couldn't test it in any of those arm boards
>> but i take a look.
>>
>> The only code part that changes is the initial setting up of the registers.
>> I am checking that.
>>
>> Regards,
>> Angelo
>>
> 
> I cannot find anything obviously wrong.
> And i cannot test on Vybrid.
> I will try to get a Vybrid V50 board to test this issue. It would
> require some days and also i wil be 3 weeks off on July.
> 
> So please revert my patch. 

I did not found the issue quickly. But I must say that I gave up pretty
quickly. There are too many changes in a single patch which makes it
hard to figure out what could be wrong. I'd rather prefer if we could
drop that patch again and go through another review phase.

Angelo, as far as I can see the patch has not been sent to LKML or the
ARM mailing list. Especially since you do not have such a device it
would have been nice to also send it to the ARM mailing list...

Can you resend your last revision with CC to me/ARM mailing list?

--
Stefan

> 
> Regards,
> Angelo
> 
>> > >
>> > > Commit found with bisect:
>> > > git bisect start
>> > > # bad: [bce40927669338cc4a361fc97d46bf650573ce5c] Add linux-next
>> > > specific files for 20180704
>> > > git bisect bad bce40927669338cc4a361fc97d46bf650573ce5c
>> > > # good: [410da1e12ffed61129d61df5b7adce4d08c7f17c] net/smc: fix up
>> > > merge error with poll changes
>> > > git bisect good 410da1e12ffed61129d61df5b7adce4d08c7f17c
>> > > # good: [decd57900d6fd743f7ddc1329400f4bd9cf54284] Merge
>> > > remote-tracking branch 'crypto/master'
>> > > git bisect good decd57900d6fd743f7ddc1329400f4bd9cf54284
>> > > # good: [e9484bb660753ca8a35d219db000fff967b70a2c] Merge
>> > > remote-tracking branch 'tip/auto-latest'
>> > > git bisect good e9484bb660753ca8a35d219db000fff967b70a2c
>> > > # bad: [20adee487b5f252d3f38197d43487c004273d3d0] Merge
>> > > remote-tracking branch 'slave-dma/next'
>> > > git bisect bad 20adee487b5f252d3f38197d43487c004273d3d0
>> > > # good: [2df2577dce1d5fd98cac342fe857677a0556421b] Merge
>> > > remote-tracking branch 'tty/tty-next'
>> > > git bisect good 2df2577dce1d5fd98cac342fe857677a0556421b
>> > > # good: [03b17ed8410a61f4f337c7bcfded1f744182edd5] spi: spi-fsl-dspi:
>> > > Fix imprecise abort on VF500 during probe
>> > > git bisect good 03b17ed8410a61f4f337c7bcfded1f744182edd5
>> > > # good: [85e1d42663a0c163002961d2685be952067b0dc2] staging:
>> > > mt7621-eth: Fix memory leak in mtk_add_mac() error path
>> > > git bisect good 85e1d42663a0c163002961d2685be952067b0dc2
>> > > # good: [b244f917acb7e1f492c40f6f4cf7be860a5065a5] staging: rtl8188eu:
>> > > remove blank lines
>> > > git bisect good b244f917acb7e1f492c40f6f4cf7be860a5065a5
>> > > # good: [b244f917acb7e1f492c40f6f4cf7be860a5065a5] staging: rtl8188eu:
>> > > remove blank lines
>> > > git bisect good b244f917acb7e1f492c40f6f4cf7be860a5065a5
>> > > # good: [b244f917acb7e1f492c40f6f4cf7be860a5065a5] staging: rtl8188eu:
>> > > remove blank lines
>> > > git bisect good b244f917acb7e1f492c40f6f4cf7be860a5065a5
>> > > # good: [6c796f514b7f8d0fef4733b971a5ded9e9a74490] Merge
>> > > remote-tracking branch 'staging/staging-next'
>> > > git bisect good 6c796f514b7f8d0fef4733b971a5ded9e9a74490
>> > > # good: [14dfa379ffd3ffd73729a72c289e9db6780a60ab] Merge branch
>> > > 'topic/ste' into next
>> > > git bisect good 14dfa379ffd3ffd73729a72c289e9db6780a60ab
>> > > # good: [c1199875d327a9b3bbb3fb1cca0d467d3e625fbb] dmaengine:
>> > > imx-sdma: Use GFP_NOWAIT for dma allocations
>> > > git bisect good c1199875d327a9b3bbb3fb1cca0d467d3e625fbb
>> > > # bad: [15efb0b488596a929bc00ce23aeb746ee6a7c14d] dmaengine: fsl-edma:
>> > > fix trailing whitespace
>> > > git bisect bad 15efb0b488596a929bc00ce23aeb746ee6a7c14d
>> > > # bad: [5c1355ad87a2c138049bfafa753b44046492aebc] dmaengine: fsl-edma:
>> > > add ColdFire mcf5441x edma support
>> > > git bisect bad 5c1355ad87a2c138049bfafa753b44046492aebc
>> > > # bad: [6ad069123f03bebe4315dea13d44845854ca6043] dmaengine: fsl-edma:
>> > > extract common fsl-edma code (no changes in behavior intended)
>> > > git bisect bad 6ad069123f03bebe4315dea13d44845854ca6043
>> > > # first bad commit: [6ad069123f03bebe4315dea13d44845854ca6043]
>> > > dmaengine: fsl-edma: extract common fsl-edma code (no changes in
>> > > behavior intended)
>> > >
>> > > Defconfig and boot log attached. I could not find any obvious error in boot log.
>> > >
>> > > Let me know if you need more data.
>> > >
>> > > Best regards,
>> > > Krzysztof
>> >
>> >
>> > > [    0.000000] Memory policy: Data cache writeback
>> > > [    0.000000] cma: Reserved 16 MiB at 0x87000000
>> > > [    0.000000] CPU: All CPU(s) started in SVC mode.
>> > > [    0.000000] random: get_random_bytes called from start_kernel+0xa0/0x3fc with crng_init=0
>> > > [    0.000000] Built 1 zonelists, mobility grouping on.  Total pages: 32512
>> > > [    0.000000] Kernel command line: ip=:::::eth0: root=/dev/nfs console=tty1 console=ttyLP0,115200n8 consoleblank=0
>> > > [    0.000000] Dentry cache hash table entries: 16384 (order: 4, 65536 bytes)
>> > > [    0.000000] Inode-cache hash table entries: 8192 (order: 3, 32768 bytes)
>> > > [    0.000000] Memory: 103836K/131072K available (6144K kernel code, 192K rwdata, 1604K rodata, 1024K init, 142K bss, 10852K reserved, 16384K cma-reserved)
>> > > [    0.000000] Virtual kernel memory layout:
>> > > [    0.000000]     vector  : 0xffff0000 - 0xffff1000   (   4 kB)
>> > > [    0.000000]     fixmap  : 0xffc00000 - 0xfff00000   (3072 kB)
>> > > [    0.000000]     vmalloc : 0x88800000 - 0xff800000   (1904 MB)
>> > > [    0.000000]     lowmem  : 0x80000000 - 0x88000000   ( 128 MB)
>> > > [    0.000000]     modules : 0x7f000000 - 0x80000000   (  16 MB)
>> > > [    0.000000]       .text : 0x(ptrval) - 0x(ptrval)   (7136 kB)
>> > > [    0.000000]       .init : 0x(ptrval) - 0x(ptrval)   (1024 kB)
>> > > [    0.000000]       .data : 0x(ptrval) - 0x(ptrval)   ( 193 kB)
>> > > [    0.000000]        .bss : 0x(ptrval) - 0x(ptrval)   ( 143 kB)
>> > > [    0.000000] SLUB: HWalign=64, Order=0-3, MinObjects=0, CPUs=1, Nodes=1
>> > > [    0.000000] NR_IRQS: 16, nr_irqs: 16, preallocated irqs: 16
>> > > [    0.000058] sched_clock: 64 bits at 132MHz, resolution 7ns, wraps every 2199023255551ns
>> > > [    0.000133] clocksource: arm_global_timer: mask: 0xffffffffffffffff max_cycles: 0x1e7178659b, max_idle_ns: 440795204301 ns
>> > > [    0.000232] Switching to timer-based delay loop, resolution 7ns
>> > > [    0.001506] Console: colour dummy device 80x30
>> > > [    0.002451] console [tty1] enabled
>> > > [    0.002550] Calibrating delay loop (skipped), value calculated using timer frequency.. 264.00 BogoMIPS (lpj=1320000)
>> > > [    0.002649] pid_max: default: 32768 minimum: 301
>> > > [    0.003151] Mount-cache hash table entries: 1024 (order: 0, 4096 bytes)
>> > > [    0.003263] Mountpoint-cache hash table entries: 1024 (order: 0, 4096 bytes)
>> > > [    0.005190] CPU: Testing write buffer coherency: ok
>> > > [    0.007551] Setting up static identity map for 0x80100000 - 0x80100060
>> > > [    0.010355] devtmpfs: initialized
>> > > [    0.027419] VFP support v0.3: implementor 41 architecture 2 part 30 variant 5 rev 1
>> > > [    0.028335] clocksource: jiffies: mask: 0xffffffff max_cycles: 0xffffffff, max_idle_ns: 19112604462750000 ns
>> > > [    0.028470] futex hash table entries: 256 (order: -1, 3072 bytes)
>> > > [    0.031116] pinctrl core: initialized pinctrl subsystem
>> > > [    0.034483] NET: Registered protocol family 16
>> > > [    0.038696] DMA: preallocated 256 KiB pool for atomic coherent allocations
>> > > [    0.042265] cpuidle: using governor menu
>> > > [    0.042687] hw-breakpoint: found 2 (+1 reserved) breakpoint and 1 watchpoint registers.
>> > > [    0.042783] hw-breakpoint: maximum watchpoint size is 4 bytes.
>> > > [    0.055428] vf610-pinctrl 40048000.iomuxc: initialized IMX pinctrl driver
>> > > [    0.125877] +V3.3: supplied by 3.3V
>> > > [    0.128267] SCSI subsystem initialized
>> > > [    0.129203] usbcore: registered new interface driver usbfs
>> > > [    0.129526] usbcore: registered new interface driver hub
>> > > [    0.129925] usbcore: registered new device driver usb
>> > > [    0.132755] i2c i2c-0: IMX I2C adapter registered
>> > > [    0.133005] i2c i2c-0: using dma0chan0 (tx) and dma0chan1 (rx) for DMA transfers
>> > > [    0.133778] pps_core: LinuxPPS API ver. 1 registered
>> > > [    0.133856] pps_core: Software ver. 5.3.6 - Copyright 2005-2007 Rodolfo Giometti <giometti@linux.it>
>> > > [    0.134029] PTP clock support registered
>> > > [    0.138089] clocksource: Switched to clocksource arm_global_timer
>> > > [    0.178733] NET: Registered protocol family 2
>> > > [    0.180526] tcp_listen_portaddr_hash hash table entries: 512 (order: 0, 4096 bytes)
>> > > [    0.180694] TCP established hash table entries: 1024 (order: 0, 4096 bytes)
>> > > [    0.180788] TCP bind hash table entries: 1024 (order: 0, 4096 bytes)
>> > > [    0.180869] TCP: Hash tables configured (established 1024 bind 1024)
>> > > [    0.181366] UDP hash table entries: 256 (order: 0, 4096 bytes)
>> > > [    0.181484] UDP-Lite hash table entries: 256 (order: 0, 4096 bytes)
>> > > [    0.182137] NET: Registered protocol family 1
>> > > [    0.183893] RPC: Registered named UNIX socket transport module.
>> > > [    0.183996] RPC: Registered udp transport module.
>> > > [    0.184043] RPC: Registered tcp transport module.
>> > > [    0.184085] RPC: Registered tcp NFSv4.1 backchannel transport module.
>> > > [    0.186339] hw perfevents: enabled with armv7_cortex_a5 PMU driver, 3 counters available
>> > > [    0.190606] Initialise system trusted keyrings
>> > > [    0.191215] workingset: timestamp_bits=30 max_order=15 bucket_order=0
>> > > [    0.211916] NFS: Registering the id_resolver key type
>> > > [    0.212112] Key type id_resolver registered
>> > > [    0.212167] Key type id_legacy registered
>> > > [    0.212344] ntfs: driver 2.1.32 [Flags: R/W].
>> > > [    0.214186] fuse init (API version 7.27)
>> > > [    0.228202] Key type asymmetric registered
>> > > [    0.228304] Asymmetric key parser 'x509' registered
>> > > [    0.228413] io scheduler noop registered
>> > > [    0.229036] io scheduler cfq registered (default)
>> > > [    0.229121] io scheduler mq-deadline registered
>> > > [    0.229171] io scheduler kyber registered
>> > > [    0.244133] 40027000.serial: ttyLP0 at MMIO 0x40027000 (irq = 19, base_baud = 4125000) is a FSL_LPUART
>> > > [    0.769644] console [ttyLP0] enabled
>> > > [    0.775130] 40028000.serial: ttyLP1 at MMIO 0x40028000 (irq = 20, base_baud = 4125000) is a FSL_LPUART
>> > > [    0.786693] 40029000.serial: ttyLP2 at MMIO 0x40029000 (irq = 21, base_baud = 4125000) is a FSL_LPUART
>> > > [    0.802286] brd: module loaded
>> > > [    0.830071] loop: module loaded
>> > > [    0.836930] nand: device found, Manufacturer ID: 0xc2, Chip ID: 0xf1
>> > > [    0.843499] nand: Macronix NAND 128MiB 3,3V 8-bit
>> > > [    0.848347] nand: 128 MiB, SLC, erase size: 128 KiB, page size: 2048, OOB size: 64
>> > > [    0.856436] Bad block table found at page 65472, version 0x01
>> > > [    0.862594] Bad block table found at page 65408, version 0x01
>> > > [    0.868726] nand_read_bbt: bad block at 0x0000013c0000
>> > > [    0.874135] 6 fixed-partitions partitions found on MTD device vf610_nfc
>> > > [    0.880908] Creating 6 MTD partitions on "vf610_nfc":
>> > > [    0.886046] 0x000000000000-0x000000020000 : "vf-bcb"
>> > > [    0.892637] 0x000000020000-0x000000180000 : "u-boot"
>> > > [    0.899247] 0x000000180000-0x0000001c0000 : "u-boot-env"
>> > > [    0.905999] 0x0000001c0000-0x000004080000 : "ubi1"
>> > > [    0.913304] 0x000004080000-0x000007f40000 : "ubi2"
>> > > [    0.920544] 0x000007f40000-0x000007f80000 : "reserved"
>> > > [    0.932515] libphy: Fixed MDIO Bus: probed
>> > > [    0.951887] libphy: fec_enet_mii_bus: probed
>> > > [    0.958566] ehci_hcd: USB 2.0 'Enhanced' Host Controller (EHCI) Driver
>> > > [    0.965935] usbcore: registered new interface driver usb-storage
>> > > [    0.972581] usbcore: registered new interface driver usbserial_generic
>> > > [    0.979465] usbserial: USB Serial support registered for generic
>> > > [    0.985705] usbcore: registered new interface driver ftdi_sio
>> > > [    0.991776] usbserial: USB Serial support registered for FTDI USB Serial Device
>> > > [    0.999432] usbcore: registered new interface driver pl2303
>> > > [    1.005220] usbserial: USB Serial support registered for pl2303
>> > > [    1.024975] snvs_rtc 400a7000.snvs:snvs-rtc-lp: rtc core: registered 400a7000.snvs:snvs-rtc-lp as rtc0
>> > > [    1.034957] i2c /dev entries driver
>> > > [    1.043842] imx2-wdt 4003e000.wdog: timeout 60 sec (nowayout=0)
>> > > [    1.051554] sdhci: Secure Digital Host Controller Interface driver
>> > > [    1.057817] sdhci: Copyright(c) Pierre Ossman
>> > > [    1.062382] sdhci-pltfm: SDHCI platform and OF driver helper
>> > > [    1.069803] sdhci-esdhc-imx 400b2000.esdhc: Got CD GPIO
>> > > [    1.104211] mmc0: SDHCI controller on 400b2000.esdhc [400b2000.esdhc] using ADMA
>> > > [    1.128652] NET: Registered protocol family 17
>> > > [    1.133569] Key type dns_resolver registered
>> > > [    1.138440] Registering SWP/SWPB emulation handler
>> > > [    1.145779] registered taskstats version 1
>> > > [    1.150109] Loading compiled-in X.509 certificates
>> > > [    1.159775] VCC_USB[1-4]: supplied by 5V
>> > > [    1.169813] ci_hdrc ci_hdrc.1: EHCI Host Controller
>> > > [    1.175111] ci_hdrc ci_hdrc.1: new USB bus registered, assigned bus number 1
>> > > [    1.208308] ci_hdrc ci_hdrc.1: USB 2.0 started, EHCI 1.00
>> > > [    1.216835] hub 1-0:1.0: USB hub found
>> > > [    1.220977] hub 1-0:1.0: 1 port detected
>> > > [    1.230260] snvs_rtc 400a7000.snvs:snvs-rtc-lp: setting system clock to 1970-01-01 00:00:00 UTC (0)
>> > > [    1.338443] Micrel KSZ8041 400d1000.ethernet-1:00: attached PHY driver [Micrel KSZ8041] (mii_bus:phy_addr=400d1000.ethernet-1:00, irq=POLL)
>> > > [    1.368009] mmc0: new high speed SDHC card at address aaaa
>> > > [    1.377866] mmcblk0: mmc0:aaaa SL16G 14.8 GiB
>> > > [    1.390488]  mmcblk0: p1 p2
>> > > [    1.748191] usb 1-1: new high-speed USB device number 2 using ci_hdrc
>> > > [    3.449105] fec 400d1000.ethernet eth0: Link is Up - 100Mbps/Full - flow control rx/tx
>> > > [    3.508218] Sending DHCP requests ., OK
>> > > [    3.572142] IP-Config: Got DHCP answer from 192.168.10.1, my address is 192.168.10.17
>> > > [    3.580129] IP-Config: Complete:
>> > > [    3.583423]      device=eth0, hwaddr=00:14:2d:2c:a1:2e, ipaddr=192.168.10.17, mask=255.255.255.0, gw=192.168.10.1
>> > > [    3.593813]      host=192.168.10.17, domain=, nis-domain=(none)
>> > > [    3.599833]      bootserver=192.168.10.1, rootserver=192.168.10.1, rootpath=/srv/nfs/colibri_vf,v4,tcp
>> > > [    3.599854]      nameserver0=192.168.10.1
>> > > [    3.676511] VFS: Mounted root (nfs4 filesystem) on device 0:13.
>> > > [    3.684257] devtmpfs: mounted
>> > > [    3.693865] Freeing unused kernel memory: 1024K
>> > > [    3.700956] random: fast init done
>> > > [    4.950401] systemd[1]: System time before build time, advancing clock.
>> > > [    5.136393] random: systemd: uninitialized urandom read (16 bytes read)
>> > > [    5.186057] systemd[1]: systemd 232 running in system mode. (-PAM -AUDIT -SELINUX -IMA -APPARMOR -SMACK +SYSVINIT +UTMP -LIBCRYPTSETUP -GCRYPT -GNUTLS +ACL +XZ -LZ4 -SECCOMP +BLKID -ELFUTILS +KMOD -IDN)
>> > > [    5.206664] systemd[1]: Detected architecture arm.
>> > >
>> >
>> >
>> > --
>> > ~Vinod
>> --
>> To unsubscribe from this list: send the line "unsubscribe dmaengine" in
>> the body of a message to majordomo@vger.kernel.org
>> More majordomo info at  http://vger.kernel.org/majordomo-info.html