Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751324AbaJOTlq (ORCPT ); Wed, 15 Oct 2014 15:41:46 -0400 Received: from mail-vc0-f174.google.com ([209.85.220.174]:39507 "EHLO mail-vc0-f174.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750721AbaJOTln (ORCPT ); Wed, 15 Oct 2014 15:41:43 -0400 MIME-Version: 1.0 In-Reply-To: References: <1409899047-13045-1-git-send-email-mcgrof@do-not-panic.com> <20140905141241.GC10455@mtj.dyndns.org> <20140905164405.GA28964@core.coreip.homeip.net> <20140905174925.GA12991@mtj.dyndns.org> <20140905224047.GC15723@mtj.dyndns.org> <20140909011059.GB11706@mtj.dyndns.org> <1410241109.2028.22.camel@jarvis.lan> <1410291346.13298.16.camel@jarvis.lan> Date: Wed, 15 Oct 2014 12:41:42 -0700 Message-ID: Subject: Re: [systemd-devel] [RFC v2 3/6] kthread: warn on kill signal if not OOM From: Anatol Pomozov To: Tom Gundersen Cc: "Luis R. Rodriguez" , Tejun Heo , One Thousand Gnomes , Takashi Iwai , Kay Sievers , Sreekanth Reddy , James Bottomley , Praveen Krishnamoorthy , hare , Nagalakshmi Nandigama , Wu Zhangjin , Tetsuo Handa , "mpt-fusionlinux.pdl" , Tim Gardner , Benjamin Poirier , Santosh Rastapur , Casey Leedom , Hariprasad S , Pierre Fersing , Arjan van de Ven , Abhijit Mahajan , systemd Mailing List , Linux SCSI List , "netdev@vger.kernel.org" , Dmitry Torokhov , Oleg Nesterov , "linux-kernel@vger.kernel.org" , Andrew Morton , Joseph Salisbury Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi On Fri, Oct 10, 2014 at 3:45 PM, Tom Gundersen wrote: > On Fri, Oct 10, 2014 at 11:54 PM, Anatol Pomozov > wrote: >> 1) Why not to make the timeout configurable through config file? There >> is already udev.conf you can put config option there. Thus people with >> modprobe issues can easily "fix" the problem. And then decrease >> default timeout back to 30 seconds. I agree that long module loading >> (more than 30 secs) is abnormal and should be investigated by driver >> authors. > > We can already configure this either on the udev or kernel > commandline, is that not sufficient (I don't object to also adding it > to the config file, just asking)? I did not know that udev timeout can be configured via kernel cmd. And because other people ask about changing timeout they most like did not know about it neither. Actually looking at http://www.freedesktop.org/software/systemd/man/kernel-command-line.html I do not see where it mentions udev timeout. I think adding configuration to the right place (udev config file) and adding documentation to make the option more discoverable will solve the topic starter issue. Now anyone can easily set timeout they want. The default timeout can go back to 30 sec in this case. >> 2) Could you add 'echo w > /proc/sysrq-trigger' to udev code right >> before killing the "modprobe" thread? sysrq will print information >> about stuck threads (including modprobe itself) this will make >> debugging easier. e.g. dmesg here >> https://bugs.archlinux.org/task/40454 says nothing where the threads >> were stuck. > > Are the current warnings (in udev git) sufficient (should tell you > which module is taking long, but still won't tell you which kernel > thread of course)? True. module name should be enough. In this case to debug the issue user needs: - disable failing udev rule (or blacklist module?) - reboot, it will let the user get into shell - modprobe the failing module - use sysrq-trigger to get more information about stuck process So it is more matter of easier problem debugging. Not critical but it will be useful imho. This feature can be configured via udev.conf -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/