Royce Lv has posted comments on this change. Change subject: udev: Race fix- load and trigger dev rule(#891300) ......................................................................
Patch Set 1: I would prefer that you didn't submit this 1.I can't access the bugzilla. 2.As I have explained in https://bugzilla.redhat.com/show_bug.cgi?id=844130: in reload cmd in RHEL udev-173 is : # udevadm control --reload-rules but in fc17-udev-182 is: #udevadm control --reload Can check the source code here: www.kernel.org/pub/linux/utils/kernel/hotplug/udev-182.tar.bz2 (udevd.c/udev-rules.c) According to changelog 181: The 'udevadm control --reload-rules' option has been renamed to '--reload'. It now also reloads the kernel module configuration. Thought in source code udev-182 it still restores --reload-rules, but did not expose it to man. So I'm afraid there will be some gap of this cmd in the future. -- To view, visit http://gerrit.ovirt.org/10758 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: If3b2008a3d9df2dcaf54190721c2dd9764338627 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Vered Volansky <[email protected]> Gerrit-Reviewer: Ayal Baron <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Federico Simoncelli <[email protected]> Gerrit-Reviewer: Lee Yarwood <[email protected]> Gerrit-Reviewer: Royce Lv <[email protected]> Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
