* [BUG] Serial/dma stall/failure after "dmaengine: fsl-edma: extract common fsl-edma code (no changes in behavior intended)"
@ 2018-07-04 8:54 Krzysztof Kozlowski
2018-07-05 1:10 ` Randy Dunlap
2018-07-05 7:35 ` Vinod
0 siblings, 2 replies; 12+ messages in thread
From: Krzysztof Kozlowski @ 2018-07-04 8:54 UTC (permalink / raw)
To: Vinod Koul, dmaengine, Angelo Dureghello
Cc: Dan Williams, linux-kernel, Stefan Agner
[-- Attachment #1: Type: text/plain, Size: 3537 bytes --]
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.
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
[-- Attachment #2: vf_defconfig --]
[-- Type: application/octet-stream, Size: 6644 bytes --]
CONFIG_KERNEL_LZO=y
CONFIG_SYSVIPC=y
CONFIG_FHANDLE=y
CONFIG_IRQ_DOMAIN_DEBUG=y
CONFIG_NO_HZ_IDLE=y
CONFIG_HIGH_RES_TIMERS=y
CONFIG_TASKSTATS=y
CONFIG_TASK_DELAY_ACCT=y
CONFIG_TASK_XACCT=y
CONFIG_TASK_IO_ACCOUNTING=y
CONFIG_IKCONFIG=y
CONFIG_IKCONFIG_PROC=y
CONFIG_LOG_BUF_SHIFT=16
CONFIG_CGROUPS=y
CONFIG_NAMESPACES=y
CONFIG_BLK_DEV_INITRD=y
# CONFIG_RD_BZIP2 is not set
# CONFIG_RD_LZMA is not set
# CONFIG_RD_XZ is not set
CONFIG_KALLSYMS_ALL=y
CONFIG_EMBEDDED=y
CONFIG_PERF_EVENTS=y
# CONFIG_SLUB_DEBUG is not set
# CONFIG_COMPAT_BRK is not set
CONFIG_MODULES=y
CONFIG_MODULE_UNLOAD=y
CONFIG_MODVERSIONS=y
CONFIG_MODULE_SRCVERSION_ALL=y
# CONFIG_BLK_DEV_BSG is not set
# CONFIG_IOSCHED_DEADLINE is not set
CONFIG_ARCH_MXC=y
CONFIG_SOC_VF610=y
CONFIG_ARCH_VEXPRESS=y
CONFIG_SWP_EMULATE=y
CONFIG_VMSPLIT_2G=y
CONFIG_PREEMPT_VOLUNTARY=y
CONFIG_AEABI=y
CONFIG_CMA=y
CONFIG_KEXEC=y
# CONFIG_ATAGS_PROC is not set
CONFIG_CPU_IDLE=y
# CONFIG_CPU_IDLE_GOV_LADDER is not set
CONFIG_VFP=y
CONFIG_NEON=y
CONFIG_KERNEL_MODE_NEON=y
CONFIG_BINFMT_MISC=y
# CONFIG_COREDUMP is not set
CONFIG_PM_WAKELOCKS=y
CONFIG_NET=y
CONFIG_PACKET=y
CONFIG_UNIX=y
CONFIG_INET=y
CONFIG_IP_ADVANCED_ROUTER=y
CONFIG_IP_MULTIPLE_TABLES=y
CONFIG_IP_PNP=y
CONFIG_IP_PNP_DHCP=y
# CONFIG_INET_XFRM_MODE_TRANSPORT is not set
# CONFIG_INET_XFRM_MODE_TUNNEL is not set
# CONFIG_INET_XFRM_MODE_BEET is not set
# CONFIG_INET_LRO is not set
# CONFIG_INET_DIAG is not set
# CONFIG_IPV6 is not set
CONFIG_BRIDGE=y
# CONFIG_BRIDGE_IGMP_SNOOPING is not set
CONFIG_WIRELESS_EXT_OUT_OF_TREE=y
CONFIG_CFG80211=m
CONFIG_MAC80211=m
CONFIG_RFKILL=y
CONFIG_RFKILL_INPUT=y
CONFIG_DEVTMPFS=y
CONFIG_DEVTMPFS_MOUNT=y
# CONFIG_STANDALONE is not set
CONFIG_DMA_CMA=y
CONFIG_CONNECTOR=y
CONFIG_MTD=y
CONFIG_MTD_CMDLINE_PARTS=y
CONFIG_MTD_JEDECPROBE=y
CONFIG_MTD_PHYSMAP_OF=y
CONFIG_MTD_BLOCK2MTD=y
CONFIG_MTD_NAND=y
CONFIG_MTD_NAND_VF610_NFC=y
CONFIG_MTD_UBI=y
CONFIG_MTD_UBI_FASTMAP=y
CONFIG_MTD_UBI_BLOCK=y
CONFIG_BLK_DEV_LOOP=y
CONFIG_BLK_DEV_RAM=y
CONFIG_BLK_DEV_RAM_COUNT=1
CONFIG_BLK_DEV_RAM_SIZE=16384
CONFIG_VIRTIO_BLK=y
CONFIG_SCSI=y
# CONFIG_SCSI_PROC_FS is not set
CONFIG_BLK_DEV_SD=y
CONFIG_SCSI_SCAN_ASYNC=y
# CONFIG_SCSI_LOWLEVEL is not set
CONFIG_NETDEVICES=y
CONFIG_VIRTIO_NET=y
# CONFIG_NET_VENDOR_ARC is not set
# CONFIG_NET_CADENCE is not set
# CONFIG_NET_VENDOR_BROADCOM is not set
# CONFIG_NET_VENDOR_CIRRUS is not set
# CONFIG_NET_VENDOR_EZCHIP is not set
# CONFIG_NET_VENDOR_FARADAY is not set
CONFIG_FSL_L2_SWITCH=y
# CONFIG_NET_VENDOR_HISILICON is not set
# CONFIG_NET_VENDOR_INTEL is not set
# CONFIG_NET_VENDOR_MARVELL is not set
# CONFIG_NET_VENDOR_MICREL is not set
# CONFIG_NET_VENDOR_NATSEMI is not set
# CONFIG_NET_VENDOR_QUALCOMM is not set
# CONFIG_NET_VENDOR_RENESAS is not set
# CONFIG_NET_VENDOR_ROCKER is not set
# CONFIG_NET_VENDOR_SAMSUNG is not set
# CONFIG_NET_VENDOR_SEEQ is not set
# CONFIG_NET_VENDOR_SMSC is not set
# CONFIG_NET_VENDOR_STMICRO is not set
# CONFIG_NET_VENDOR_SYNOPSYS is not set
# CONFIG_NET_VENDOR_VIA is not set
# CONFIG_NET_VENDOR_WIZNET is not set
CONFIG_MICREL_PHY=y
CONFIG_USB_NET_DRIVERS=m
CONFIG_USB_RTL8152=m
CONFIG_USB_USBNET=m
# CONFIG_USB_NET_CDC_NCM is not set
# CONFIG_USB_NET_NET1080 is not set
# CONFIG_USB_NET_CDC_SUBSET is not set
# CONFIG_USB_NET_ZAURUS is not set
CONFIG_ATH_CARDS=m
# CONFIG_ATH9K_BTCOEX_SUPPORT is not set
CONFIG_ATH9K_HTC=m
# CONFIG_RTL_CARDS is not set
CONFIG_INPUT_POLLDEV=y
# CONFIG_INPUT_MOUSEDEV is not set
CONFIG_INPUT_EVDEV=y
# CONFIG_KEYBOARD_ATKBD is not set
CONFIG_KEYBOARD_GPIO=y
# CONFIG_INPUT_MOUSE is not set
# CONFIG_SERIO is not set
CONFIG_VT_HW_CONSOLE_BINDING=y
# CONFIG_LEGACY_PTYS is not set
# CONFIG_DEVKMEM is not set
CONFIG_SERIAL_FSL_LPUART=y
CONFIG_SERIAL_FSL_LPUART_CONSOLE=y
CONFIG_VIRTIO_CONSOLE=y
CONFIG_HW_RANDOM=y
CONFIG_HW_RANDOM_VIRTIO=y
CONFIG_I2C=y
# CONFIG_I2C_COMPAT is not set
CONFIG_I2C_CHARDEV=y
CONFIG_I2C_MUX=y
# CONFIG_I2C_HELPER_AUTO is not set
CONFIG_I2C_ALGOBIT=y
CONFIG_I2C_IMX=y
CONFIG_SPI=y
CONFIG_SPI_FSL_DSPI=y
CONFIG_SPI_SPIDEV=y
CONFIG_GPIO_SYSFS=y
CONFIG_BATTERY_BQ27XXX=y
CONFIG_POWER_RESET_GPIO=y
CONFIG_POWER_RESET_GPIO_RESTART=y
CONFIG_POWER_RESET_SYSCON=y
CONFIG_SENSORS_IIO_HWMON=y
CONFIG_WATCHDOG=y
CONFIG_IMX2_WDT=y
CONFIG_REGULATOR_ANATOP=y
CONFIG_REGULATOR_GPIO=y
# CONFIG_HID is not set
# CONFIG_USB_HID is not set
CONFIG_USB=y
CONFIG_USB_EHCI_HCD=y
CONFIG_USB_ACM=m
CONFIG_USB_STORAGE=y
CONFIG_USB_CHIPIDEA=y
CONFIG_USB_CHIPIDEA_UDC=y
CONFIG_USB_CHIPIDEA_HOST=y
CONFIG_USB_SERIAL=y
CONFIG_USB_SERIAL_CONSOLE=y
CONFIG_USB_SERIAL_GENERIC=y
CONFIG_USB_SERIAL_FTDI_SIO=y
CONFIG_USB_SERIAL_PL2303=y
CONFIG_USB_SERIAL_OPTION=m
CONFIG_NOP_USB_XCEIV=y
CONFIG_USB_MXS_PHY=y
CONFIG_USB_GADGET=y
CONFIG_USB_FSL_USB2=y
CONFIG_USB_CONFIGFS=m
CONFIG_USB_CONFIGFS_SERIAL=y
CONFIG_USB_CONFIGFS_ACM=y
CONFIG_USB_CONFIGFS_OBEX=y
CONFIG_USB_CONFIGFS_NCM=y
CONFIG_USB_CONFIGFS_ECM=y
CONFIG_USB_CONFIGFS_RNDIS=y
CONFIG_USB_CONFIGFS_EEM=y
CONFIG_USB_CONFIGFS_F_FS=y
CONFIG_USB_CONFIGFS_F_HID=y
CONFIG_MMC=y
CONFIG_MMC_ARMMMCI=y
CONFIG_MMC_SDHCI=y
CONFIG_MMC_SDHCI_PLTFM=y
CONFIG_MMC_SDHCI_ESDHC_IMX=y
CONFIG_LEDS_CLASS=y
CONFIG_LEDS_GPIO=y
CONFIG_LEDS_PWM=y
CONFIG_LEDS_TRIGGER_TIMER=y
CONFIG_LEDS_TRIGGER_ONESHOT=y
CONFIG_LEDS_TRIGGER_HEARTBEAT=y
CONFIG_LEDS_TRIGGER_BACKLIGHT=y
CONFIG_LEDS_TRIGGER_GPIO=y
CONFIG_RTC_CLASS=y
CONFIG_RTC_DRV_SNVS=y
CONFIG_DMADEVICES=y
CONFIG_FSL_EDMA=y
# CONFIG_MX3_IPU is not set
CONFIG_VIRTIO_MMIO=y
CONFIG_VIRTIO_MMIO_CMDLINE_DEVICES=y
# CONFIG_IOMMU_SUPPORT is not set
CONFIG_SOC_BUS_VF610=y
CONFIG_EXTCON_USB_GPIO=y
CONFIG_IIO=y
CONFIG_VF610_ADC=y
CONFIG_VF610_DAC=y
CONFIG_IIO_SYSFS_TRIGGER=y
CONFIG_PWM=y
CONFIG_PWM_FSL_FTM=y
CONFIG_NVMEM=y
CONFIG_NVMEM_VF610_OCOTP=y
CONFIG_EXT4_FS=y
CONFIG_AUTOFS4_FS=y
CONFIG_FUSE_FS=y
CONFIG_CUSE=y
CONFIG_VFAT_FS=y
CONFIG_NTFS_FS=y
CONFIG_NTFS_RW=y
CONFIG_TMPFS=y
CONFIG_TMPFS_POSIX_ACL=y
CONFIG_UBIFS_FS=y
CONFIG_NFS_FS=y
# CONFIG_NFS_V2 is not set
# CONFIG_NFS_V3 is not set
CONFIG_NFS_V4=y
CONFIG_ROOT_NFS=y
CONFIG_NLS_DEFAULT="cp437"
CONFIG_NLS_CODEPAGE_437=y
CONFIG_NLS_ASCII=y
CONFIG_NLS_ISO8859_1=y
CONFIG_NLS_UTF8=y
CONFIG_PRINTK_TIME=y
CONFIG_MESSAGE_LOGLEVEL_DEFAULT=2
CONFIG_LOCKUP_DETECTOR=y
CONFIG_DEFAULT_HUNG_TASK_TIMEOUT=10
# CONFIG_SCHED_DEBUG is not set
CONFIG_STACKTRACE=y
# CONFIG_DEBUG_BUGVERBOSE is not set
# CONFIG_FTRACE is not set
# CONFIG_ARM_UNWIND is not set
CONFIG_DEBUG_USER=y
CONFIG_SECURITYFS=y
CONFIG_CRYPTO_ECB=y
CONFIG_CRYPTO_CMAC=y
CONFIG_CRYPTO_HMAC=y
CONFIG_CRYPTO_MD4=y
CONFIG_CRYPTO_MD5=y
CONFIG_CRYPTO_SHA256=y
CONFIG_CRYPTO_ARC4=y
CONFIG_CRYPTO_DES=y
# CONFIG_CRYPTO_HW is not set
CONFIG_CRC_CCITT=m
CONFIG_CRC_T10DIF=y
CONFIG_XZ_DEC=y
[-- Attachment #3: boot.log --]
[-- Type: text/x-log, Size: 10047 bytes --]
[ 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.
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [BUG] Serial/dma stall/failure after "dmaengine: fsl-edma: extract common fsl-edma code (no changes in behavior intended)"
2018-07-04 8:54 [BUG] Serial/dma stall/failure after "dmaengine: fsl-edma: extract common fsl-edma code (no changes in behavior intended)" Krzysztof Kozlowski
@ 2018-07-05 1:10 ` Randy Dunlap
2018-07-05 7:37 ` Vinod
2018-07-05 7:35 ` Vinod
1 sibling, 1 reply; 12+ messages in thread
From: Randy Dunlap @ 2018-07-05 1:10 UTC (permalink / raw)
To: Krzysztof Kozlowski, Vinod Koul, dmaengine, Angelo Dureghello
Cc: Dan Williams, linux-kernel, Stefan Agner
On 07/04/18 01: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.
>
> 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
BTW, there is also this warning in linux-next:
WARNING: modpost: missing MODULE_LICENSE() in drivers/dma/fsl-edma-common.o
--
~Randy
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [BUG] Serial/dma stall/failure after "dmaengine: fsl-edma: extract common fsl-edma code (no changes in behavior intended)"
2018-07-04 8:54 [BUG] Serial/dma stall/failure after "dmaengine: fsl-edma: extract common fsl-edma code (no changes in behavior intended)" Krzysztof Kozlowski
2018-07-05 1:10 ` Randy Dunlap
@ 2018-07-05 7:35 ` Vinod
2018-07-05 8:12 ` Angelo Dureghello
1 sibling, 1 reply; 12+ messages in thread
From: Vinod @ 2018-07-05 7:35 UTC (permalink / raw)
To: Angelo Dureghello
Cc: dmaengine, Krzysztof Kozlowski, Dan Williams, linux-kernel, Stefan Agner
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 ?
>
> 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
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [BUG] Serial/dma stall/failure after "dmaengine: fsl-edma: extract common fsl-edma code (no changes in behavior intended)"
2018-07-05 1:10 ` Randy Dunlap
@ 2018-07-05 7:37 ` Vinod
0 siblings, 0 replies; 12+ messages in thread
From: Vinod @ 2018-07-05 7:37 UTC (permalink / raw)
To: Randy Dunlap
Cc: Krzysztof Kozlowski, dmaengine, Angelo Dureghello, Dan Williams,
linux-kernel, Stefan Agner
On 04-07-18, 18:10, Randy Dunlap wrote:
> On 07/04/18 01:54, Krzysztof Kozlowski wrote:
>
> BTW, there is also this warning in linux-next:
> WARNING: modpost: missing MODULE_LICENSE() in drivers/dma/fsl-edma-common.o
That has been fixed.
--
~Vinod
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [BUG] Serial/dma stall/failure after "dmaengine: fsl-edma: extract common fsl-edma code (no changes in behavior intended)"
2018-07-05 7:35 ` Vinod
@ 2018-07-05 8:12 ` Angelo Dureghello
2018-07-05 12:45 ` Angelo Dureghello
0 siblings, 1 reply; 12+ messages in thread
From: Angelo Dureghello @ 2018-07-05 8:12 UTC (permalink / raw)
To: Vinod
Cc: dmaengine, Krzysztof Kozlowski, Dan Williams, linux-kernel, Stefan Agner
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
> >
> > 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
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [BUG] Serial/dma stall/failure after "dmaengine: fsl-edma: extract common fsl-edma code (no changes in behavior intended)"
2018-07-05 8:12 ` Angelo Dureghello
@ 2018-07-05 12:45 ` Angelo Dureghello
2018-07-05 14:29 ` Stefan Agner
2018-07-05 15:55 ` Stefan Agner
0 siblings, 2 replies; 12+ messages in thread
From: Angelo Dureghello @ 2018-07-05 12:45 UTC (permalink / raw)
To: Vinod
Cc: dmaengine, Krzysztof Kozlowski, Dan Williams, linux-kernel, Stefan Agner
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.
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
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [BUG] Serial/dma stall/failure after "dmaengine: fsl-edma: extract common fsl-edma code (no changes in behavior intended)"
2018-07-05 12:45 ` Angelo Dureghello
@ 2018-07-05 14:29 ` Stefan Agner
2018-07-05 15:55 ` Stefan Agner
1 sibling, 0 replies; 12+ messages in thread
From: Stefan Agner @ 2018-07-05 14:29 UTC (permalink / raw)
To: Vinod, Angelo Dureghello
Cc: dmaengine, Krzysztof Kozlowski, Dan Williams, linux-kernel
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.
Vinod, before reverting, let me quickly look into it.
--
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
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [BUG] Serial/dma stall/failure after "dmaengine: fsl-edma: extract common fsl-edma code (no changes in behavior intended)"
2018-07-05 12:45 ` Angelo Dureghello
2018-07-05 14:29 ` Stefan Agner
@ 2018-07-05 15:55 ` Stefan Agner
2018-07-05 16:23 ` Angelo Dureghello
1 sibling, 1 reply; 12+ messages in thread
From: Stefan Agner @ 2018-07-05 15:55 UTC (permalink / raw)
To: Angelo Dureghello, Vinod
Cc: dmaengine, Krzysztof Kozlowski, Dan Williams, linux-kernel
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
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [BUG] Serial/dma stall/failure after "dmaengine: fsl-edma: extract common fsl-edma code (no changes in behavior intended)"
2018-07-05 15:55 ` Stefan Agner
@ 2018-07-05 16:23 ` Angelo Dureghello
2018-07-06 6:05 ` Vinod
0 siblings, 1 reply; 12+ messages in thread
From: Angelo Dureghello @ 2018-07-05 16:23 UTC (permalink / raw)
To: Stefan Agner
Cc: Vinod, dmaengine, Krzysztof Kozlowski, Dan Williams, linux-kernel
Hi Stafan,
On Thu, Jul 05, 2018 at 05:55:31PM +0200, Stefan Agner wrote:
> 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?
>
Thanks for looking into it.
I have spent a lot of time on this patch and really would have
dma for Coldfire available. So, have ordered a Colibri / v50 board.
Should receive it in few days and should be able to debug this issue,
but as said, i will be off for some weeks so looks like it is probably
better to revert the patch.
My initial submit was a separate driver, to avoid such issues where
i cannot test, but it resulted in too much duplicated code.
Sure, i can send the full patch to you/arm with all the fixes included
until now.
Best regards,
Angelo
> --
> 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
> --
> 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
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [BUG] Serial/dma stall/failure after "dmaengine: fsl-edma: extract common fsl-edma code (no changes in behavior intended)"
2018-07-05 16:23 ` Angelo Dureghello
@ 2018-07-06 6:05 ` Vinod
2018-07-06 6:16 ` Krzysztof Kozlowski
0 siblings, 1 reply; 12+ messages in thread
From: Vinod @ 2018-07-06 6:05 UTC (permalink / raw)
To: Angelo Dureghello
Cc: Stefan Agner, dmaengine, Krzysztof Kozlowski, Dan Williams, linux-kernel
On 05-07-18, 18:23, Angelo Dureghello wrote:
> Hi Stafan,
>
> On Thu, Jul 05, 2018 at 05:55:31PM +0200, Stefan Agner wrote:
> > 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?
> >
>
> Thanks for looking into it.
>
> I have spent a lot of time on this patch and really would have
> dma for Coldfire available. So, have ordered a Colibri / v50 board.
>
> Should receive it in few days and should be able to debug this issue,
> but as said, i will be off for some weeks so looks like it is probably
> better to revert the patch.
>
> My initial submit was a separate driver, to avoid such issues where
> i cannot test, but it resulted in too much duplicated code.
>
> Sure, i can send the full patch to you/arm with all the fixes included
> until now.
Okay dropped now from -next. I still keeping topic/fsl around and
collect other fixes for you guys to check.
One way would be to split to common patch into multiple patch and check
regression. That should help quickly identify the issue.
--
~Vinod
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [BUG] Serial/dma stall/failure after "dmaengine: fsl-edma: extract common fsl-edma code (no changes in behavior intended)"
2018-07-06 6:05 ` Vinod
@ 2018-07-06 6:16 ` Krzysztof Kozlowski
2018-07-06 8:55 ` Angelo Dureghello
0 siblings, 1 reply; 12+ messages in thread
From: Krzysztof Kozlowski @ 2018-07-06 6:16 UTC (permalink / raw)
To: Vinod
Cc: Angelo Dureghello, Stefan Agner, dmaengine, Dan Williams, linux-kernel
On 6 July 2018 at 08:05, Vinod <vkoul@kernel.org> wrote:
> On 05-07-18, 18:23, Angelo Dureghello wrote:
>> Hi Stafan,
>>
>> On Thu, Jul 05, 2018 at 05:55:31PM +0200, Stefan Agner wrote:
>> > 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?
>> >
>>
>> Thanks for looking into it.
>>
>> I have spent a lot of time on this patch and really would have
>> dma for Coldfire available. So, have ordered a Colibri / v50 board.
>>
>> Should receive it in few days and should be able to debug this issue,
>> but as said, i will be off for some weeks so looks like it is probably
>> better to revert the patch.
>>
>> My initial submit was a separate driver, to avoid such issues where
>> i cannot test, but it resulted in too much duplicated code.
>>
>> Sure, i can send the full patch to you/arm with all the fixes included
>> until now.
>
> Okay dropped now from -next. I still keeping topic/fsl around and
> collect other fixes for you guys to check.
>
> One way would be to split to common patch into multiple patch and check
> regression. That should help quickly identify the issue.
Even with show -C20% -M20% I gave up when trying to spot possible
differences in the commit. I understand that logically it is one
change and it makes sense... but it touches, moves and adds too much
making review extremely difficult. I think the splitting into smaller
chunks would be beneficial anyway.
Today's next boots fine.
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [BUG] Serial/dma stall/failure after "dmaengine: fsl-edma: extract common fsl-edma code (no changes in behavior intended)"
2018-07-06 6:16 ` Krzysztof Kozlowski
@ 2018-07-06 8:55 ` Angelo Dureghello
0 siblings, 0 replies; 12+ messages in thread
From: Angelo Dureghello @ 2018-07-06 8:55 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Vinod, Stefan Agner, dmaengine, Dan Williams, linux-kernel
Hi,
On Fri, Jul 06, 2018 at 08:16:29AM +0200, Krzysztof Kozlowski wrote:
> On 6 July 2018 at 08:05, Vinod <vkoul@kernel.org> wrote:
> > On 05-07-18, 18:23, Angelo Dureghello wrote:
> >> Hi Stafan,
> >>
> >> On Thu, Jul 05, 2018 at 05:55:31PM +0200, Stefan Agner wrote:
> >> > 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?
> >> >
> >>
> >> Thanks for looking into it.
> >>
> >> I have spent a lot of time on this patch and really would have
> >> dma for Coldfire available. So, have ordered a Colibri / v50 board.
> >>
> >> Should receive it in few days and should be able to debug this issue,
> >> but as said, i will be off for some weeks so looks like it is probably
> >> better to revert the patch.
> >>
> >> My initial submit was a separate driver, to avoid such issues where
> >> i cannot test, but it resulted in too much duplicated code.
> >>
> >> Sure, i can send the full patch to you/arm with all the fixes included
> >> until now.
> >
> > Okay dropped now from -next. I still keeping topic/fsl around and
> > collect other fixes for you guys to check.
> >
> > One way would be to split to common patch into multiple patch and check
> > regression. That should help quickly identify the issue.
>
> Even with show -C20% -M20% I gave up when trying to spot possible
> differences in the commit. I understand that logically it is one
> change and it makes sense... but it touches, moves and adds too much
> making review extremely difficult. I think the splitting into smaller
> chunks would be beneficial anyway.
>
> Today's next boots fine.
>
Good, will resend the patch once splitted in more commits, and
tested on V50 Vybrid.
Regards,
Angelo
> Best regards,
> Krzysztof
> --
> 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
^ permalink raw reply [flat|nested] 12+ messages in thread
end of thread, other threads:[~2018-07-06 8:55 UTC | newest]
Thread overview: 12+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-07-04 8:54 [BUG] Serial/dma stall/failure after "dmaengine: fsl-edma: extract common fsl-edma code (no changes in behavior intended)" Krzysztof Kozlowski
2018-07-05 1:10 ` Randy Dunlap
2018-07-05 7:37 ` Vinod
2018-07-05 7:35 ` Vinod
2018-07-05 8:12 ` Angelo Dureghello
2018-07-05 12:45 ` Angelo Dureghello
2018-07-05 14:29 ` Stefan Agner
2018-07-05 15:55 ` Stefan Agner
2018-07-05 16:23 ` Angelo Dureghello
2018-07-06 6:05 ` Vinod
2018-07-06 6:16 ` Krzysztof Kozlowski
2018-07-06 8:55 ` Angelo Dureghello
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).