/srv/irclogs.ubuntu.com/2010/04/27/#ubuntu-kernel.txt

crimsunBenC: pong, sorry for missing you. Spotty irc presence for an indeterminate time.00:31
BenCcrimsun: no problem, I actually figured out what I wanted02:10
BenCcrimsun: I was trying to figure out how the period/buffer stuff worked for alsa capture drivers, and I finally found the right documentation02:11
BenCcrimsun: so my driver is working properly now :)02:13
_pHI_hey! i was curious if TRIM support for SSDs was backported into the kernel that will ship with 10.04?02:37
_pHI_afaik it only arrived with 2.6.3302:37
_pHI_any help would be great...02:55
=== lifeless_ is now known as lifeless
=== ericm-afk is now known as ericm|ubuntu
toabctlhi11:39
toabctlis it possible to create a devicenode (with udev rule) when a driver is loaded? e.g. if "mordprobe drivername": mknod ...11:39
persiatoabctl: If the module discovers/enables some device, yes.11:42
toabctlpersia, no. the device is a lcd-display (char device) and this won't be discovered11:43
toabctlpersia, i can create the device-node by hand, but i would like to use udev to create the devicenode if the driver is loaded11:43
toabctlpersia, my udev rule looks like this: DRIVER=="mylcdmodule", NAME="my-lcd"11:44
persiaHow is the LCD connected?11:44
ogralook at /lib/udev/rules.d/README11:44
toabctlpersia, over gpios (it's a handmade solution)11:45
persiaOh, that's trickier indeed.  Someone more knowledgeable than I needs to answer you.11:45
toabctlpersia, i just thought that udev can do some action if a driver is loaded.11:46
persiaI think you can, but I think you need a connect event: I hope I'm mistaken, so you can have an easy solution.11:46
ograyes, you need a kernel event 11:46
toabctlogra, and a "module load" does not emit an event?11:47
toabctlogra, i have to add the event to the module source code?11:47
ograwell, look at /var/log/udev when you load the module 11:48
ograyou should see an "add" or "change" event11:49
toabctlogra, thx. "udevadm monitor" is the command to help to debug the problem11:52
ograah, right11:52
ograi forgot about that one11:53
ograthough the log should show essentially the same11:53
toabctlogra, but how to handle the event? i got: UDEV  [1272365579.494531] add      /module/meteo40_lcd (module)11:54
ograman udev should help you11:55
persiatoabctl: If you have an event, just make a rule that matches it, and create your device.11:56
toabctlpersia, i know that, but what is the event in my case? and how to match it?11:56
persiaThe event is "add /module/meteo40_lcd", from what you said12:00
toabctlpersia, ACTION=="add", but what keyword should i use for "/module/meteo40_lcd" ? NAME?KERNEL?12:09
persiaI'd go with trial and error on that one :)12:10
persia(although someone more knowledgeable may answer)12:10
toabctlpersia, just if you are interessted: you can do: udevadm info -a -p /class/meteo40lcd12:28
toabctlthen you see the information you need to write the correct rule.12:28
persiatoabctl: Cool!  Thanks for the hint.12:34
* persia may have to add some rules for Displaylink devices in maverick12:35
=== mdomsch is now known as mdomsch_BOS
exltI would like to know if 2.6.32.12 will be rolled into the Lucid release - they *finally* got the stable XFS patchset included, which fixes massive instability with the current version (we have been custom patching for quite a few releases..)13:25
smbexlt, IT will13:25
smbNot into release13:26
exlt\o/13:26
smbBit it will get into update13:26
exltah, ok13:26
exltsmb would you have a rough timeline of when that update might go out?  just a guess is totally fine, so I can tell those at $JOB13:27
* persia expects "a few weeks" is probably the right amount of precision to give in an official answer, to save smb needing to give a more correct answer on which someone might actually rely13:28
smbexlt, There likely will be a security update in between. And UDS is ahead which slows things down13:28
exltthat all sounds perfect to me  :)13:29
smbSo yes some weeks sounds like something to be expected13:30
* akgraner waves - Rikki said - She can connect to the internet wireless now - And even posted - "Who has the best Kernel Team? Ubuntu!" Y'all Rock! Thanks JFo and whomever else that helped!!!14:04
JFoheh14:05
JFoI saw that14:05
JFoshe is a mess14:05
JFoso now Tim knows that PPA kernel fixes her14:05
akgranerJFo, I told her it's cause I helped her file the bug :-) - she laughed14:05
akgranerGreat - I am writing up how a simple FB post  - can open all kinds on adventures from Learning how to file a bug, to showing developer channels aren't full scary unfriendly people but awesome helpful folks, and the list goes on..  - so thanks y'all :-)14:09
* akgraner waves bye - laters14:09
arandI reported Bug #514498 a while ago, and I was wondering whether or not any useful info at all might be extracted from the dd-copy?14:41
ubot3Malone bug 514498 in linux "whole filesystem lost to corruption " [Medium,Triaged] https://launchpad.net/bugs/51449814:41
=== kamalmostafa is now known as kamalm
JFoarand, good question :)15:53
JFoapw, cnd  do you know of anything we might be able to take away from a corrupted FS?15:54
arandJFo: Ah, you the triager?15:54
JFoarand has a dd image of it15:54
cndI don't really know filesystems at all15:54
JFoarand, I am15:54
JFocsurbhi, how about you?15:54
JFoI know you know filesystems :-)15:55
JFocnd, meant to put csurbhi there instead of you... sorry about that15:56
cndJFo: np :)15:56
arandI'm not even sure if corruption after hard poweroff is "just the way things are", or were for the jaunty version of the kernel...15:59
psurbhiarand, I suspect the corruption occurs because of the hard poweroff16:02
psurbhii can see this is lucid too, when the journal gets corrupted for instance.16:02
arandpsurbhi: Yes, I suspected so as well, and it can be that repeated instances, even though fsck succeeds and everything seems fine, might "accumulate" corruption? Or would it always be a one-off fail?16:04
psurbhiif fsck succeeds, then fs should not be corrupted at that point16:05
psurbhifsck should tell you when the fs is corrupted and beyond correction16:05
psurbhiarand, i will come back to you after looking what could be looked at to fix this up16:08
psurbhithanks!16:08
JFothank you for looking psurbhi 16:09
arandpsurbhi: ok cheers, if I'm not around here I follow the bug.16:09
JFothanks for reminding me arand :)16:09
arandJFo: Yea, I wanted to make sure I didn't just forget it either ;)16:15
=== ogasawara_ is now known as ogasawara
DinkHello, I am trying to use launchpad to compile only a specific flavour kernel. I have edited i386.mk to only contain the flavour I want. How do I prepare this for launchpadd ppa to use? Are there any other files I need to edit ? Any help would be appreciated.19:33
aboganiDink: What should do your specific flavour?19:41
Dinkits a strip down kernel for my netbook19:44
DinkI have never done this via .deb/ppa . I have done this before via rpm/opensuse build service19:45
aboganiDink: Why don't you create a -generic with a bigger number version?19:45
DinkI just found this link ... https://wiki.ubuntu.com/KernelTeam/KernelMaintenance#Preparing%20an%20upload19:45
Dinkabogani, I do not want it to interfere with ubuntu's generic hence the different flavour19:45
Dinkat time I do go back and fourth for testing etc, to me makes it easier19:46
DinkI am using the generic as my base19:46
aboganiDink: If you place that into you personal PPA you ever don't interfere with others.19:46
DinkAhh I think I got it. Had to edit the first line in my changelog also20:01
kamalmI'm preparing my first patch submission to LKML -- the patch has already been applied to Lucid.  I understand that I should put "Signed-off-by: ...myself..." in the patch description (as I did for the Lucid patch).   I don't think anyone else has "signed off" on this patch, but apw did state "Acked-by: Andy W...".20:13
kamalmQuestion: Should I copy apw's Acked-by: line into the version of the patch going to LKML?  Otherwise, is just my "S-o-b:" line sufficient for a submission to LKML?20:13
achiangkamalm: depends on whether apw was ack'ing your patch in the context of the lucid kernel or in the context of upstream.20:15
kamalmachiang: ah ha, I guess I should ask him.  :-)20:15
kamalmAnother question: I'm preparing the patch for LKML against the "mainline" tree (/pub/scm / linux/kernel/git/torvalds/linux-2.6.git) -- is the proper tree to base my patch from?20:16
achiangkamalm: in general, acked-by is somewhat "weak" tag. depending on the subsystem maintainer, they may interpret it differently. the most common use is if the maintainer has a few trusted reviewers who add their acked-bys.20:17
achiangkamalm: if i were to go ack a bunch of networking patches, e.g., i'm sure i would get yelled at by davem since i have no expertise there20:17
kamalmachiang: well maybe I just don't need to apply that Acked-by: to my LKML submission at all?20:17
achiangkamalm: in general yes, prepare against linus's tree, but again, sometimes depends on the subsystem20:18
achiangkamalm: right, without seeing your patch, it's hard to say20:18
achiangkamalm: care to post your patch (or a pointer)?20:18
kamalmcertainly -- let me find it here.20:19
kamalmachiang: https://lists.ubuntu.com/archives/kernel-team/2010-April/010181.html20:20
kamalmachiang: https://lists.ubuntu.com/archives/kernel-team/2010-April/010188.html  <--- andy's Acked-by response20:21
achiangkamalm: ok, so ACPI patch. for acpi, working off of linus's tree is fine20:21
achiangkamalm: as for adding andy's acked-by, i'm not entirely sure. you'd have to ask him, i guess20:22
achiangkamalm: you can add mine though. :)20:22
achiangAcked-by: Alex Chiang <achiang@canonical.com>20:22
kamalmachiang: oh, okay, I'll add yours and not add andy's, since he's not around to ask.  thanks very much for the advice.  :-)20:23
achiangkamalm: i'm actually waiting on this one: https://patchwork.kernel.org/patch/94711/20:23
achiangkamalm: i'll probably poke lenb today; he's been travelling so i wanted to give him some time to work through his backlog20:24
kamalmachiang: ah, your thinkpad patch is exactly the same issue eh?  I'm wondering whether it would be possible to move the DMI-detection for this down into udev and just have the kernel respond to a binary switch to do the sci_en thing or not based on whatever udev said to do.  does that sound plausible to you?20:25
achiangkamalm: for your patch, please do cc: stable@kernel.org too20:25
achiangkamalm: hm, what you suggest sounds possible, but it's not common for how we handle acpi quirks20:26
kamalmachiang: ok, will do -- does cc'ing stable@ mean that I'm requesting that the patch be applied to the stable kernel as well as mainline then?20:26
achiangyes, that's what you're requesting20:26
achiangkamalm: in the context of these bugs, we're basically getting little band-aids upstream to help distros out. the fix going forward will be this patch:https://patchwork.kernel.org/patch/93560/ 20:27
achiangkamalm: so basically, get this quirks added upstream, get them into stable to help out distros, and hopefully with mjg59's fix we don't have to add to the DMI table any more in the future20:28
kamalmachiang: the detect-it-in-udev concept is how the keyboard map quirks seem to work, which is what gave me the idea of doing the sci_en that way also -- i had heard about the "forthcoming better way" -- thanks for the pointer -- no need to pursue a better band-aid mechanism then.  :-)20:29
cndkamalm: achiang: you don't cc stable@kernel.org directly though21:18
cndyou add a "Cc: stable@kernel.org" line to the s-o-b area21:19
cndyou can ask csurbhi all about the fun you'll get from gregkh if you start CC'ing patches directly to stable@kernel.org21:19
achiangcnd: hm, i add the cc to the sign-off area and also in my mailer21:20
achiangcnd: i haven't been yelled at yet. ;)21:20
cndachiang: if you have the Cc in the s-o-b area, it will get sent to stable@kernel.org when linus applies it to his tree21:21
cndso that way only patches which are actually applied get sent21:21
achiangcnd: interesting, i hadn't realized that21:21
kamalmcnd: yes thanks for that tip!  (very timely too!)21:22
achiangdoes linus have a special git hook to do that?21:22
cndif you cc stable@kernel.org directly, then gregkh gets a bunch of patches that may or may not actually be in linus' tree21:22
cndachiang: I'm not sure21:22
cndand it may be further down the line at the subsystem maintainers level for all I know21:22
cndbut I have had a patch accepted to stable that way, so I can vouch for it :)21:22
achiangbecause that's not standard kernel patch flow at least not in the areas i'm familiar with -- simply adding a git tag to the signed off area doesn't automatically result in an email21:23
cndachiang: I have a feeling if you start to send a few more you may hear from gregkh about it :)21:23
achiangmaybe stable is special21:23
cndstable isn't like any other subsystem though21:23
cndit's rather *special*21:23
achiangyup, i'm guessing a fancy hook21:24
achiangcnd: anyhow, good info to have, thanks21:24
=== lifeless_ is now known as lifeless
cndonly stuff that is applied upstream should make it to the stable trees21:24
cndin general, if you're unsure, check the Documentation directory of the tree21:25
cndit has a whole txt file devoted to stable tree submissions21:25
cndall about what's appropriate and how to get changes accepted21:25
cndfor example, it's always good to CC the subsystem maintainers if you're taking an upstream patch and asking it to be applied to stable21:26
cndand to add a note about regression potential in that case too21:26
cndif it's a rather large change, it may be easier to get the subsystem maintainer to push it to stable as well21:26
cndthere's a little bit of a trust issue there, for better or worse depending on your point of view21:26
kamalmI discovered that scripts/get_maintainer.pl generates a nice list of all the right people/lists to email, given a patch file -- good stuff.21:27
cndkamalm: hmmm, hadn't noticed that file, good find21:28
cndkamalm: also, run your patches through scripts/checkpatch.pl before submitting21:28
cndit finds format issues21:28
kamalmcnd: re checkpatch.pl - neato! its happy with my patch21:30
cndgreat!21:30
achiangi've discovered that turning 'let c_space_errors=1' on in my .vimrc helps quite a bit21:39
cndachiang: is that what turns ws red in git diff?21:42
cndI wouldn't mind having that on all the time too...21:42
achiangcnd: no, that is something you set in your ~/.gitconfig21:42
achiangcnd: oh, nm -- if you colorize git, then you get whitespace color for free without twiddling your .gitconfig21:43
cndachiang: yeah, I get it automatically in git diff21:43
cndjust not in vim21:43
achiangbut you can set other whitespace options in there: core.whitespace (not common) and apply.whitespace (much more common)21:44
achiangcnd: that vim option lets you see whitespace errors in your editor, so you can write perfectly formatted patches. :)21:44
achiangit can be a little annoying when you view files in the kernel with broken whitespace21:45
cndheh21:45
achiangkamalm: next time, reverse the order of your git tags. :)22:12
kamalmum... what do you mean?22:13
achiangcc: stable goes first, cc: me next, s-o-b last22:13
achiangyou read that sign off area from top to bottom, so the top has informational stuff, like cc's, reviewed-by:, acked-by: etc.22:14
achiangthen in order of patch delivery, you add s-o-bs22:14
achiangso original author is topmost, and last hop in the delivery path is you22:14
kamalmachiang: oh, I guess I didn't even realized that the order there even mattered.  I'll try to remember that for next time. thanks!22:15
achiangkamalm: you're right, it doesn't matter so much, but it is a nicety that makes it a little easier to read. that's all. :)22:15
achiangthe s-o-bs are important to get in order22:15
kamalmachiang: got it -- luckily, there's only one s-o-b for this one!22:16
achiangyup22:16
cndkamalm: it's a little overwhelming, all the processes :)22:24
cndrest assured, three months in (like me), you'll still be spinning around, but you'll have a better idea of how things go :)22:25
kamalmcnd: sure is overwhelming, but yes, its all becoming clearer with each passing day -- thanks very much to you and achiang for the help!22:25
* maco didn't know it mattered either22:26
achiangthink of the s-o-b's as a path a patch takes to get upstream22:26
achiangeach person involved in the patch submission appends his/her sob to the path22:26

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!