=== ben_r_ is now known as ben_r [02:30] apw: i've updated to 4.8.0-17-generic , but still no luck for the ntfs rw [07:14] ejat, reviewing configs, i cannot see when we had that turned on ... it looks like writability of ntfs was actually done via a fuse filesystem, which is unrelated to the kernel, so that is confusing [07:23] hi, regarding https://wiki.ubuntu.com/Kernel/FAQ#Kernel.2FFAQ.2FGeneralUbuntuDelta.What_differentiates_the_Ubuntu_Kernel_from_the_upstream_Linux_Kernel.3F [07:23] it seems wrong to included commits and tags like https://git.kernel.org/cgit/linux/kernel/git/stable/linux-stable.git/commit/?id=1d074db69c46d62ce82b331c2080e2fcb710bf4a [07:24] and using a "ckt" suffix would be more appropriate to reflect the nature of the kernel in its version string [07:24] wrong commit, I meant this one https://git.launchpad.net/~ubuntu-kernel/ubuntu/+source/linux/+git/xenial/commit/?h=master-next&id=b536d9ae195f9daa32cf2446df2a35787142cacb [07:29] ricotz, that is really a matter of taste to a large degree, that kernel has the vast majority of the stable 4.4.21 and its predecessors applied [07:29] ricotz, so is it wrong for its internal version strings to include that information [07:31] ricotz, it exposes its version in uname more like 4.4.0-36-generic #19-Ubuntu so you can tell it is an ubuntu build and track it back [07:31] apw, imho, it is wrong while it suggests it is based on the full 4.4.x stable release [07:31] this stable release version is exposed in logs though [07:32] ricotz, well the vast majority are applied so ... its pretty close, and it it was literally rebased on that and we reverted the delta it would be fine to call it that ? [07:32] apw, imo, yes [07:32] ricotz, as i say, i think you can make arguement either way, and indeed we have internally, but we feel that calling a kernel which has all of 4.4.21 applied 4.4.0 would be more inaccurate [07:33] ricotz, well i'd argue as those bits on disk would be identicle it is wrong to say one way of making it can be called 4.4.21 and the other not [07:33] what I am saying if someone it hit by a bug which an unapplied patch is triggering, it would be more painful to find out [07:34] s/triggering/fixing/ [07:35] ricotz, i can't deny that if we have dropped any there is a very slight possibility of people making assumptions rather thna actually checking what is applied [07:35] i actually do not think we have in 4.4 so far, we tend to drop them because they are already applied "early" rather than anything else [07:35] right, so using the ckt suffix to some extent would be it a bit more clear [07:36] the kernel advertises itself as -Ubuntu pretty strongly in the only real place we emit "loose dumps" of data, which is oops output [07:37] most any other collection of data is attached to system information [07:37] (I for myself assumed it would get all stable-release patches until now) [07:37] and that is the plan, aim and normally the execution [07:38] i have i think twice in recent memory seen a patch dropped for being already applied, and oobviously we have reverted a couple of actually broken patches [07:38] before they got fixed in -stable, but that seems like a positive [07:38] it might not hurt so much for non-lts kernels, but for the 4.4 lts with 5 year support those dropped patches might pile up [07:39] you know ubuntu is not a pure upstream kernel, no distro kernle is, there is always delta [07:39] having the 4.4.21 information tells you at least waht it is closest to in mainline terms, not including that just [07:39] means you have even less idea what it is similar to [07:39] I am aware of that, I am just arguing about the advertised base-kernel [07:40] I see [07:40] and you are arguing it should be 4.4.0 [07:40] no, I am not ;) [07:40] ok then you need to be clearer in what you are arguing :) [07:40] I mentioned to add some "ckt" suffix [07:41] and where do you want that, given we have a -Ubuntu suffix in the primary output userspace can ask for [07:42] and do keep in mind that we cannot completley arbitrarily mangle the version format [07:42] because _stupid_ userspace tools assume it is at least 3 dotted numbers, _and_ others assume they can just [07:42] like https://git.launchpad.net/~ubuntu-kernel/ubuntu/+source/linux/+git/trusty/commit/?id=a4e4e7cac5f9fcf4368ad5d22ac96285290562fc [07:42] use the string as a directory name in the filesystem [07:43] extraversion does not make it into the names exposed in the packaging, that is set in some of the older kernels [07:43] and you won't see it, because htat is where the abi number is [07:43] i guess we could put -ckt in the Makefile itself, but you'd never see it anywhere else [07:45] I know the kernel package name doesnt reflect the micro version [07:46] ok, I guess I will have to live with it ;) [07:46] ricotz, i'd like to remove the .0 there and expose the real micro in the binary packages, but that keeps slipping [07:52] apw, I see, that makes sense [07:52] we got into this .0 mess because userspace tooling is broken in the format assumptions so like ps would vomit if you made the built in version shorter [08:00] apw, oh, I assumed it was made that way to cope with "real" ABI breaks regarding dkms modules [08:01] the name of the abi directory isn't so critical as long as it changes when the abi changes [08:01] (and only ...) [08:15] apw: so its conclude not coming from Kernel issue/problem? [08:16] any suggestion how can i debug or get logs from ? [08:16] ejat, right now i believe that if it is a kernel issue it is not what i would have thought, as that fucnionality is marked dangerous and turned off [08:17] ejat, in all releases. it was pointed out to me that we used a fuse job to access ntfs r/w [08:17] not quite sure how that triggers [08:22] apw: i think i found some clues .. http://paste.ubuntu.com/23240235/ [08:23] i have suspected windows partition .. but i already full shutdown n restart [08:23] n off the fast restart [08:24] oh a good point, i wonder if with new never shutting down windows you can ever mount it r/w safely from outside [08:24] or maybe some updates/patches for Windows 10 Fast Ring Insider [08:25] but its work previous in LTS :( [08:26] it does not seem possible we could modify the filessytem is windows is hibernated [08:26] as it could have any of that data cached in memory and changing it would be fatal to the filesystme [08:27] but i've tried to disable the Fast Startup [08:28] maybe the cache is not cleaned yet ... [08:36] probabaly windows has reneabled it because you would be daft to not use it (in its mind) :) [09:03] lol ... u got a point .. im clearing off the cache in the disk n will try again shortly [09:34] apw: sorry to bothering u this few days ... finally rw work again after i clear the partition cache and disable the fast startup + shutdown [09:34] ejat, awsome that it is now working [09:35] but its kinda weird that i doesnt have to do that way previously as i can just reboot when ever i want to choose etiher ubuntu or windows [09:36] i doesn't need to disable the fast startup in 16.04 lts ... n its work flawlessly without proper shutdown (just only reboot will do the job to switch between OS) [09:36] that can only be broken to my mind, if windows is hibernated what is on disk cannot be changed safely ever [09:37] so something else must have changed, and i am willing to believe that windows changed something to be "faster" [09:39] :) [09:40] btw, thanks again .. im also thinking that there is "something" changed in windows ... as i use the fast insider ring === maxb_ is now known as maxb [13:43] rtg, EDAC_MM_EDAC=m therefore EDAC_GHES is not selected, EDAC_GHES is needed on arm64 v8.0+ for firmware first error handling of memory and CPU errors .. would you consider a patch to enable that for Y ? [13:43] apw, ^ [13:43] manjo, file away... [13:43] rtg, thanks [13:50] manjo, we're close enough to freeze now that we should have a bug for that and send the patch to the list, as it will likely need two acks etc as if it were an sru [13:50] apw, will open a bug and send it to ukml [13:54] apw, bug against package "linux" ? [13:56] apw, https://bugs.launchpad.net/ubuntu/+source/linux/+bug/1628111 [13:56] Ubuntu bug 1628111 in linux (Ubuntu) "[Yakkety] enable EDAC_GHES in kernel config" [Undecided,New] [14:02] manjo, yes, and i assume you referenced that in the patch [14:02] apw, yes . just letting you know I have the bug ☺ [14:02] oh don't worry i have wiped all knowledge of it [14:37] apw, config.common.ubuntu:CONFIG_HAVE_ACPI_APEI=y but when I do edit configs for arm64 it is not selected .. why is that ? [14:37] apw, it is depend is ACPI=y which is true [14:43] apw, nm I see why [14:43] configs are fenicety is normally the reason === PaulW2U_ is now known as PaulW2U [15:13] <_ami_> any pointers on why i get this warning messages sometimes when i send urbs for IN/OUT endpoint? [15:13] <_ami_> WARNING: CPU: 1 PID: 2866 at /build/linux-a2WvEb/linux-4.4.0/drivers/usb/core/urb.c [15:13] <_ami_> :338 usb_submit_urb+0x51/0x70() [15:13] <_ami_> [ +0.000002] URB ffff8801fd3ef780 submitted while active [15:14] <_ami_> i send interrupt IN & OUT whenever a sysfs file value is changed [15:14] _ami_, as that is a WARN_ON i assume it has some hints at that line to say what it thinks you are doing wrong [15:16] <_ami_> apw: it seems like i am resubmitting a urb which is yet to be used.. [15:16] reusing the same buffer perhaps ? [15:17] <_ami_> apw: indeed, it was the case. [15:17] <_ami_> just fixed :) [15:18] <_ami_> i was doing submitting interrupt in urb on _probe() too. just removed that code. now doing Interrupt in/out on device attribute setter function. :) [15:23] manjo, is that CONFIG_EDAC_MM_EDAC patch for Xenial or Yakkety ? [15:23] rtg, for Y [15:23] rtg, oops did I miss that [15:23] rtg, the bug report says Y .. [15:24] manjo, 'cause I set it [15:24] rtg, ok thanks .. sorry about that my patch should have said Y as well [15:35] <_ami_> apw: which is FASTER? creating and reusing control/bulk/interrupt urbs to send/recv data? or helper functions like usb_control/bulk/interrupt_msg()? i tend to believe that reusing same urb everytime is FASTER. just wonder why people use helper functions quite often? [15:36] <_ami_> people are LAZY or its just does not matter much? [15:36] reusing it is only safe if you can be sure it will never get queued etd [15:36] etc === daniel1 is now known as Odd_Bloke [15:36] helpers tend to stop you making mistakes like reusing them too early [15:37] <_ami_> aha, sync vs async dilemma [15:37] right, async is normally better when you have lots of data [15:37] and knowin when the urb has gone is error prone [15:38] <_ami_> got it., thanks, noted down. [15:39] manjo, I responded to your patch with an alternate. I'm not too keen on munging Kconfigs [15:40] rtg, trying your patch as we speak.. [15:41] manjo, thanks [15:41] rtg, it does not select EDAC_GHES [15:42] rtg, for that you also need to satisfy select HAVE_ACPI_APEI if ACPI in Kconfig. [15:43] manthen you really need 2 patches, a SAUCE patch to fix the Kconfig, and a patch to update the config settings. [15:43] manjo, ^^ [15:43] rtg, ok will resend [15:44] manjo, I expect that you'll want to upstream the Kconfig patch [15:44] rtg, right that is next [15:53] rtg, ah that patch for APIE in kconfig went upstream in Jul [15:53] rtg, https://lkml.org/lkml/2016/7/27/427 [15:53] rtg, wonder why it is missing in Ubuntu [15:54] manI assume you are basing this off of Yakkety master-next ? It has been rebased against 4.8-rc8 [15:54] manjo, ^^ damn tab completion [16:08] rtg, let me reclone my tree.. it is a cf right now [16:08] manjo, you shouldn't have to reclone. 'git fetch origin master-next;git reset --hard FETCH_HEAD" [16:09] rtg, that is what I did and it blew up on my face [16:09] rtg, recloning as we speak [16:09] manjo, how did it blow up ? [16:09] git fetch origin master-next doenst make sense those are all remote names [16:12] apw, git fetch master-next; git rebase master-next ? [16:13] git fetch origin; then maybe a rebase yes [16:13] but if the upstream is rebasing which it is you need to be more careful [16:13] and use --onto === daniel is now known as Guest11181 === Guest11181 is now known as Odd_Bloke === Elimin8r is now known as Elimin8er === Serge is now known as Guest37289 === xnox_ is now known as xnox === leftyfb_ is now known as leftyfb === sforshee` is now known as sforshee === Elimin8r is now known as Elimin8er === Guest37289 is now known as hallyn === ben_r_ is now known as ben_r === eichiro_ is now known as eichiro === kees_ is now known as kees === spossiba_ is now known as spossiba === sforshee` is now known as sforshee === ghostcube_ is now known as ghostcube === mamarley is now known as Guest12142 === mamarley_ is now known as mamarley === sts is now known as Guest67929 === PaulW2U is now known as Guest66500 === PaulW2U_ is now known as PaulW2U