On Wed, Oct 18, 2017 at 10:09 AM, Casey Leedom <leedom@xxxxxxxxxxx> wrote: > | From: Luis R. Rodriguez <mcgrof@xxxxxxxxxx> > | Sent: Tuesday, October 17, 2017 6:14:20 PM > | > | You referred to commit 2355869. What tree and more importantly what patch > | title? > > Ah, my bad. Sorry, I had the impression that git commit IDs were > preserved across repositories. That's in Linus' main repository: > > commit 235586939d7fe4833ada9e988f92af543ee6851f > Author: Luis R. Rodriguez <mcgrof@xxxxxxxxxx> > Date: Fri Sep 8 16:17:00 2017 -0700 > > kmod: split out umh code into its own file > > Patch series "kmod: few code cleanups to split out umh code" > > The usermode helper has a provenance from the old usb code which first > required a usermode helper. Eventually this was shoved into kmod.c and > the kernel's modprobe calls was converted over eventually to share the > same code. Over time the list of usermode helpers in the kernel has grown > -- so kmod is just but one user of the API. > ... > > But, that was just a random guess since it was the largest recent change in > this area. Komali tried to apply a reverse patch to see if that guess > worked out but that didn't apply cleanly and trying to reset to 2355869^ led > to other problems with systemd-udev.service running constantly reloading the > driver with a RHEL7 installation ... which may be a hint about what's going > on ... I'm going to recommend that Komali do a bisect to see if she can > figure out where this new behavior started ... I'd recommend to do a full bisect, I *highly* doubt the above commit is the issue given it really should not have introduced any functional changes as its just a code shuffle, moving code from one file to another. Specially for the type of change you are describing, that requires significant changes. So save your time and just focus on a proper bisect. Luis
