# git rev-parse -q --verify a3ac7917b73070010c05b4485b8582a6c9cd69b6^{commit}
a3ac7917b73070010c05b4485b8582a6c9cd69b6
already have revision, skipping fetch
# git checkout -q -f -B kisskb a3ac7917b73070010c05b4485b8582a6c9cd69b6
# git clean -qxdf
# < git log -1
# commit a3ac7917b73070010c05b4485b8582a6c9cd69b6
# Author: Linus Torvalds <torvalds@linux-foundation.org>
# Date:   Mon Mar 25 14:49:00 2019 -0700
# 
#     Revert "parport: daisy: use new parport device model"
#     
#     This reverts commit 1aec4211204d9463d1fd209eb50453de16254599.
#     
#     Steven Rostedt reports that it causes a hang at bootup and bisected it
#     to this commit.
#     
#     The troigger is apparently a module alias for "parport_lowlevel" that
#     points to "parport_pc", which causes a hang with
#     
#         modprobe -q -- parport_lowlevel
#     
#     blocking forever with a backtrace like this:
#     
#         wait_for_completion_killable+0x1c/0x28
#         call_usermodehelper_exec+0xa7/0x108
#         __request_module+0x351/0x3d8
#         get_lowlevel_driver+0x28/0x41 [parport]
#         __parport_register_driver+0x39/0x1f4 [parport]
#         daisy_drv_init+0x31/0x4f [parport]
#         parport_bus_init+0x5d/0x7b [parport]
#         parport_default_proc_register+0x26/0x1000 [parport]
#         do_one_initcall+0xc2/0x1e0
#         do_init_module+0x50/0x1d4
#         load_module+0x1c2e/0x21b3
#         sys_init_module+0xef/0x117
#     
#     Supid says:
#      "Due to the new device model daisy driver will now try to find the
#       parallel ports while trying to register its driver so that it can bind
#       with them. Now, since daisy driver is loaded while parport bus is
#       initialising the list of parport is still empty and it tries to load
#       the lowlevel driver, which has an alias set to parport_pc, now causes
#       a deadlock"
#     
#     But I don't think the daisy driver should be loaded by the parport
#     initialization in the first place, so let's revert the whole change.
#     
#     If the daisy driver can just initialize separately on its own (like a
#     driver should), instead of hooking into the parport init sequence
#     directly, this issue probably would go away.
#     
#     Reported-and-bisected-by: Steven Rostedt (VMware) <rostedt@goodmis.org>
#     Reported-by: Michal Kubecek <mkubecek@suse.cz>
#     Acked-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
#     Cc: Sudip Mukherjee <sudipm.mukherjee@gmail.com>
#     Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
# < /opt/cross/kisskb/fe-x86-64-core-i7-2017.05/bin/x86_64-linux-gcc --version
# < /opt/cross/kisskb/fe-x86-64-core-i7-2017.05/bin/x86_64-linux-ld --version
# < git log --format=%s --max-count=1 a3ac7917b73070010c05b4485b8582a6c9cd69b6
# < make -s -j 10 ARCH=um O=/kisskb/build/linus_um-defconfig_um-x86_64 CROSS_COMPILE=/opt/cross/kisskb/fe-x86-64-core-i7-2017.05/bin/x86_64-linux- SUBARCH=x86_64 defconfig
# make -s -j 10 ARCH=um O=/kisskb/build/linus_um-defconfig_um-x86_64 CROSS_COMPILE=/opt/cross/kisskb/fe-x86-64-core-i7-2017.05/bin/x86_64-linux- SUBARCH=x86_64 
/kisskb/src/arch/um/kernel/skas/uaccess.c: In function 'do_op_one_page':
/kisskb/src/arch/um/kernel/skas/uaccess.c:62:10: warning: unused variable 'buf' [-Wunused-variable]
  jmp_buf buf;
          ^
/kisskb/src/arch/um/os-Linux/umid.c: In function 'is_umdir_used':
/kisskb/src/arch/um/os-Linux/umid.c:138:2: warning: ISO C90 forbids variable length array 'file' [-Wvla]
  char file[strlen(uml_dir) + UMID_LEN + sizeof("/pid\0")];
  ^
/kisskb/src/arch/um/os-Linux/umid.c: In function 'create_pid_file':
/kisskb/src/arch/um/os-Linux/umid.c:213:2: warning: ISO C90 forbids variable length array 'file' [-Wvla]
  char file[strlen(uml_dir) + UMID_LEN + sizeof("/pid\0")];
  ^
/kisskb/src/arch/um/os-Linux/umid.c: In function 'remove_umid_dir':
/kisskb/src/arch/um/os-Linux/umid.c:388:2: warning: ISO C90 forbids variable length array 'dir' [-Wvla]
  char dir[strlen(uml_dir) + UMID_LEN + 1], err;
  ^
/kisskb/src/mm/mprotect.c: In function 'change_pte_range':
/kisskb/src/mm/mprotect.c:42:20: warning: unused variable 'mm' [-Wunused-variable]
  struct mm_struct *mm = vma->vm_mm;
                    ^
  LINK linux
Completed OK
# rm -rf /kisskb/build/linus_um-defconfig_um-x86_64
# Build took: 0:00:48.863210