On Fri, Aug 11, 2017 at 5:31 PM, John Stultz <john.stu...@linaro.org> wrote:
> On Fri, Aug 11, 2017 at 5:10 PM, Wei Wang <wei...@google.com> wrote:
>>> If after Cong's fix, the issue still happens, could you help try the
>>> patch attached and collect al
On Fri, Aug 11, 2017 at 5:10 PM, Wei Wang wrote:
>> If after Cong's fix, the issue still happens, could you help try the
>> patch attached and collect all logs when you try the reproduce the
>> issue? It would be great to have logs for both success case and the
>> failure case.
On Wed, Aug 9, 2017 at 10:41 PM, Wei Wang wrote:
> Hi John,
>
> Is it possible to try the attached patch?
Thanks so much for the quick turn around!
So I dropped all the reverts you suggested, and applied this one
against 4.13-rc4, but I'm still seeing the problematic
On Wed, Aug 9, 2017 at 5:36 PM, Wei Wang <wei...@google.com> wrote:
> On Wed, Aug 9, 2017 at 4:44 PM, John Stultz <john.stu...@linaro.org> wrote:
>> On Wed, Aug 9, 2017 at 4:34 PM, Cong Wang <xiyou.wangc...@gmail.com> wrote:
>>> (Cc'ing Wei whose commit was blam
On Wed, Aug 9, 2017 at 5:36 PM, Wei Wang wrote:
>
> Does your USB adapter get an IPv6 address?
Yes, it does.
> If you see the problem starts to happen on commit
> 9514528d92d4cbe086499322370155ed69f5d06c, could you try reverting all
> the following commits:
> (from new to
On Wed, Aug 9, 2017 at 4:34 PM, Cong Wang <xiyou.wangc...@gmail.com> wrote:
> (Cc'ing Wei whose commit was blamed)
>
> On Mon, Aug 7, 2017 at 2:15 PM, John Stultz <john.stu...@linaro.org> wrote:
>> On Mon, Aug 7, 2017 at 2:05 PM, John Stultz <john.stu...@linaro.o
On Mon, Aug 7, 2017 at 2:05 PM, John Stultz <john.stu...@linaro.org> wrote:
> So, with recent testing with my HiKey board, I've been noticing some
> quirky behavior with my USB eth adapter.
>
> Basically, pluging the usb eth adapter in and then removing it, when
> plugging it
So, with recent testing with my HiKey board, I've been noticing some
quirky behavior with my USB eth adapter.
Basically, pluging the usb eth adapter in and then removing it, when
plugging it back in I often find that its not detected, and the system
slowly spits out the following message over and
On Thu, Jun 1, 2017 at 5:26 PM, Yan, Zheng wrote:
> On Thu, Jun 1, 2017 at 6:22 PM, Arnd Bergmann wrote:
>> On Thu, Jun 1, 2017 at 11:56 AM, Yan, Zheng wrote:
>>> On Sat, Apr 8, 2017 at 8:57 AM, Deepa Dinamani
>>>
On Tue, Dec 6, 2016 at 10:23 AM, Tejun Heo wrote:
> Hello,
>
> On Tue, Dec 06, 2016 at 10:13:53AM -0800, Andy Lutomirski wrote:
>> > Delegation is an explicit operation and reflected in the ownership of
>> > the subdirectories and cgroup interface files in them. The
>> >
On Tue, Nov 22, 2016 at 4:57 PM, John Stultz <john.stu...@linaro.org> wrote:
> On Tue, Nov 8, 2016 at 4:12 PM, Andy Lutomirski <l...@amacapital.net> wrote:
>> On Tue, Nov 8, 2016 at 4:03 PM, Alexei Starovoitov
>> <alexei.starovoi...@gmail.com> wrote:
>>>
On Tue, Nov 8, 2016 at 4:12 PM, Andy Lutomirski wrote:
> On Tue, Nov 8, 2016 at 4:03 PM, Alexei Starovoitov
> wrote:
>> On Tue, Nov 08, 2016 at 03:51:40PM -0800, Andy Lutomirski wrote:
>>>
>>> I hate to say it, but I think I may see a problem.
; Reviewed-by: Josh Triplett <j...@joshtriplett.org>
> Acked-by: Richard Cochran <richardcoch...@gmail.com>
> Acked-by: Thomas Gleixner <t...@linutronix.de>
Ok.. ran the series through the kselftest/timers series and the results look ok.
Acked-by: John Stultz <john.stu...@linaro.org>
thanks
-john
On Tue, Nov 8, 2016 at 10:19 AM, Nicolas Pitre <nicolas.pi...@linaro.org> wrote:
> On Tue, 8 Nov 2016, John Stultz wrote:
>
>> One spot of concern is that the
>> tools/testing/selftests/timers/posix_timers.c test hangs testing
>> virtual itimers. Looking through
On Mon, Nov 7, 2016 at 2:14 PM, Nicolas Pitre wrote:
> Some embedded systems have no use for them. This removes about
> 22KB from the kernel binary size when configured out.
>
> Corresponding syscalls are routed to a stub logging the attempt to
> use those syscalls
an Vecera <ivec...@redhat.com>
Cc: linux-...@vger.kernel.org
Cc: netdev@vger.kernel.org
Cc: stable <sta...@vger.kernel.org> #4.4+
Reported-by: Yongqin Liu <yongqin@linaro.org>
Signed-off-by: John Stultz <john.stu...@linaro.org>
---
drivers/net/usb/asix_common.c | 2 +-
1 f
On Wed, May 11, 2016 at 3:00 PM, Dean Jenkins wrote:
>
> Your observations are consistent with missing URBs from the USB host
> controller.
>
> Here is a summary of what I think is happening in your case:
>
> Good case:
> URB #1: 1514 octets of 1514 Ethernet frame (A)
>
On Tue, May 3, 2016 at 2:16 PM, Dean Jenkins wrote:
>
> [ 239.027993] asix 1-1.1:1.0 eth0: asix_rx_fixup() Data Header
> synchronisation was lost, remaining 988
>
> This error message consistently shows the remaining value to be 988, at
> least for the 3 examples
On Tue, May 3, 2016 at 2:16 PM, Dean Jenkins wrote:
> A good test would be to run "ping -c 1 -s $packet_length $ip_address" inside
> a script which has a loop with an increasing payload length $packet_length
> with a small delay between ping calls. This will show whether
On Fri, May 6, 2016 at 8:00 AM, Dean Jenkins wrote:
> My conclusion is that your USB to Ethernet Adaptor is not running at high
> speed (480Mbps) mode which is causing a partial loss (corruption) of
> Ethernet frames across the USB link. A USB Protocol Analyser or
On Thu, May 5, 2016 at 1:11 AM, Dean Jenkins <dean_jenk...@mentor.com> wrote:
> On 05/05/16 00:45, John Stultz wrote:
>>
>> Just as a sample point, I have managed to reproduce exactly this issue
>> on an x86_64 system by simply scp'ing a large file.
>
> Please
-next.
>
> John had qualms about [2/3], but it looks like a straightforward
> substitution in areas which will get plenty of testing
Yea. My main concern is just not stepping on any other maintainers toes.
> I'll grab the patches for now to get them some external testing.
I'd be just as happy if the set went through you Andrew.
For the set: Acked-by: John Stultz <john.stu...@linaro.org>
thanks
-john
On Tue, May 3, 2016 at 3:54 AM, Dean Jenkins wrote:
> On 03/05/16 11:04, Guodong Xu wrote:
>>
>> did you test on ARM 64-bit system or ARM 32-bit? I ask because HiKey
>> is an ARM 64-bit system. I suggest we should be careful on that. I saw
>> similar issues when
On Wed, May 4, 2016 at 12:24 PM, Deepa Dinamani wrote:
> struct timespec is not y2038 safe.
> Even though timespec might be sufficient to represent
> timeouts, use struct timespec64 here as the plan is to
> get rid of all timespec reference in the kernel.
>
> The patch
On Tue, May 3, 2016 at 7:42 AM, David B. Robins <li...@davidrobins.net> wrote:
> On 2016-05-03 00:55, John Stultz wrote:
>>
>> Looking through the commits since the v4.1 kernel where we didn't see
>> this, I narrowed the regression down, and reverting the following tw
In testing with HiKey, we found that since commit 3f30b158eba5c60
(asix: On RX avoid creating bad Ethernet frames), we're seeing lots of
noise during network transfers:
[ 239.027993] asix 1-1.1:1.0 eth0: asix_rx_fixup() Data Header
synchronisation was lost, remaining 988
[ 239.037310] asix
WiFi as wlan0 is
> registered.
>
> Signed-off-by: Bjorn Andersson <bjorn.anders...@linaro.org>
I've been using this with my nexus7 tree, and its avoided issues I was
seeing without it.
Tested-by: John Stultz <john.stu...@linaro.org>
thanks
-john
Hey Thomas,
So again, here is Christopher's cross-timestamp
infrastructure patchset which I wanted to send along for 4.6.
(Including the minor tweaks you suggested). These apply
against tip/timers/core.
Let me know if you have any objections.
thanks
-john
Cc: Prarit Bhargava
effrey.t.kirs...@intel.com
Cc: netdev@vger.kernel.org
Reviewed-by: Thomas Gleixner <t...@linutronix.de>
Signed-off-by: Christopher S. Hall <christopher.s.h...@intel.com>
[jstultz: Reworked to remove extra structures and simplify calling]
Signed-off-by:
things up,
fixed cycles_t/cycle_t confusion.]
Signed-off-by: John Stultz <john.stu...@linaro.org>
---
include/linux/timekeeping.h | 18 ++
kernel/time/timekeeping.c | 30 ++
2 files changed, 48 insertions(+)
diff --git a/include/linux/timekeepi
@intel.com
Cc: netdev@vger.kernel.org
Reviewed-by: Thomas Gleixner <t...@linutronix.de>
Signed-off-by: Christopher S. Hall <christopher.s.h...@intel.com>
Signed-off-by: John Stultz <john.stu...@linaro.org>
---
include/linux/pps_kernel.h | 17 ++---
kernel/time/timek
kevin.j.cla...@intel.com
Cc: h...@zytor.com
Cc: jeffrey.t.kirs...@intel.com
Cc: netdev@vger.kernel.org
Reviewed-by: Thomas Gleixner <t...@linutronix.de>
Signed-off-by: Christopher S. Hall <christopher.s.h...@intel.com>
Signed-off-by: John Stultz <john.stu...@linaro.org>
---
kernel/time/
ff-by: Christopher S. Hall <christopher.s.h...@intel.com>
[jstultz: Fixed up cycles_t/cycle_t type confusion]
Signed-off-by: John Stultz <john.stu...@linaro.org>
---
include/linux/timekeeper_internal.h | 2 +
include/linux/timekeeping.h | 5 ++
kernel/time/timekeeping.c
i <l...@amacapital.net>
Cc: kevin.b.stan...@intel.com
Cc: kevin.j.cla...@intel.com
Cc: h...@zytor.com
Cc: jeffrey.t.kirs...@intel.com
Cc: netdev@vger.kernel.org
Acked-by: Richard Cochran <richardcoch...@gmail.com>
Signed-off-by: Christopher S. Hall <christopher.s.h...@intel.com>
[
stopher S. Hall <christopher.s.h...@intel.com>
[jstultz: Tweaked to fix build issue, also reworked math for
64bit division on 32bit systems, as well as !CONFIG_CPU_FREQ build
fixes]
Signed-off-by: John Stultz <john.stu...@linaro.org>
---
arch/x86/include/asm/cpufeature.h | 2 +
zytor.com
Cc: jeffrey.t.kirs...@intel.com
Cc: netdev@vger.kernel.org
Acked-by: Jeff Kirsher <jeffrey.t.kirs...@intel.com>
Signed-off-by: Christopher S. Hall <christopher.s.h...@intel.com>
[jstultz: Reworked to use new interface, commit message tweaks]
Signed-off-by: John Stultz <john.stu..
On Thu, Mar 3, 2016 at 2:27 PM, Jeff Kirsher
wrote:
> Since you are making changes to patch 6's title, then you should fix
> this patch title as well, it should be:
>
> [PATCH 8/8] e1000e: Adds hardware supported...
Ack
-john
On Thu, Mar 3, 2016 at 2:21 AM, Thomas Gleixner <t...@linutronix.de> wrote:
> On Wed, 2 Mar 2016, John Stultz wrote:
>
> Subject: x86: tsc: Always Running ...
>
> Please make that:
>
> Subject: x86/tsc: Always Running ...
Will do.
>
>>
>> +#else
>
things up,
fixed cycles_t/cycle_t confusion.]
Signed-off-by: John Stultz <john.stu...@linaro.org>
---
include/linux/timekeeping.h | 18 ++
kernel/time/timekeeping.c | 30 ++
2 files changed, 48 insertions(+)
diff --git a/include/linux/timekeepi
Hey Thomas,
So here is Christopher's cross-timestamp infrastructure
patchset which I wanted to send along for 4.6. These apply against
tip/timers/core.
Let me know if you have any objections.
thanks
-john
Cc: Prarit Bhargava
Cc: Richard Cochran
@intel.com
Cc: netdev@vger.kernel.org
Reviewed-by: Thomas Gleixner <t...@linutronix.de>
Signed-off-by: Christopher S. Hall <christopher.s.h...@intel.com>
Signed-off-by: John Stultz <john.stu...@linaro.org>
---
include/linux/pps_kernel.h | 17 ++---
kernel/time/timek
effrey.t.kirs...@intel.com
Cc: netdev@vger.kernel.org
Reviewed-by: Thomas Gleixner <t...@linutronix.de>
Signed-off-by: Christopher S. Hall <christopher.s.h...@intel.com>
[jstultz: Reworked to remove extra structures and simplify calling]
Signed-off-by:
ff-by: Christopher S. Hall <christopher.s.h...@intel.com>
[jstultz: Fixed up cycles_t/cycle_t type confusion]
Signed-off-by: John Stultz <john.stu...@linaro.org>
---
include/linux/timekeeper_internal.h | 2 +
include/linux/timekeeping.h | 5 ++
kernel/time/timekeeping.c
kevin.j.cla...@intel.com
Cc: h...@zytor.com
Cc: jeffrey.t.kirs...@intel.com
Cc: netdev@vger.kernel.org
Reviewed-by: Thomas Gleixner <t...@linutronix.de>
Signed-off-by: Christopher S. Hall <christopher.s.h...@intel.com>
Signed-off-by: John Stultz <john.stu...@linaro.org>
---
kernel/time/
stopher S. Hall <christopher.s.h...@intel.com>
[jstultz: Tweaked to fix build issue, also reworked math for
64bit division on 32bit systems, as well as !CONFIG_CPU_FREQ build
fixes]
Signed-off-by: John Stultz <john.stu...@linaro.org>
---
arch/x86/include/asm/cpufeature.h | 2 +
zytor.com
Cc: jeffrey.t.kirs...@intel.com
Cc: netdev@vger.kernel.org
Acked-by: Jeff Kirsher <jeffrey.t.kirs...@intel.com>
Signed-off-by: Christopher S. Hall <christopher.s.h...@intel.com>
[jstultz: Reworked to use new interface, commit message tweaks]
Signed-off-by: John Stultz <john.stu..
i <l...@amacapital.net>
Cc: kevin.b.stan...@intel.com
Cc: kevin.j.cla...@intel.com
Cc: h...@zytor.com
Cc: jeffrey.t.kirs...@intel.com
Cc: netdev@vger.kernel.org
Acked-by: Richard Cochran <richardcoch...@gmail.com>
Signed-off-by: Christopher S. Hall <christopher.s.h...@intel.com>
[
On Wed, Feb 24, 2016 at 2:56 AM, Thomas Gleixner wrote:
> On Mon, 22 Feb 2016, Christopher S. Hall wrote:
>> +{
>> + struct timekeeper *tk = _core.timekeeper;
>> + bool interp_forward;
>> + u64 corr_raw, corr_real;
>> + int ret;
>
> Once more:
>
>
On Mon, Feb 22, 2016 at 10:33 AM, Christopher Hall
wrote:
> I just sent another patchset (v8). I corrected the comment problems pointed
> out by Richard Cochran. I also changed the arch/x86 code to use "non-stop"
> TSC rather than "invariant" TSC. They are *exactly*
On Thu, Feb 18, 2016 at 2:17 PM, Richard Cochran
wrote:
> On Fri, Feb 12, 2016 at 12:25:26PM -0800, Christopher S. Hall wrote:
>> /**
>> * get_device_system_crosststamp - Synchronously capture system/device
>> timestamp
>> - * @sync_devicetime: Callback to get
On Fri, Feb 12, 2016 at 12:25 PM, Christopher S. Hall
wrote:
> Modern Intel hardware adds an Always Running Timer (ART) that allows the
> network and audio device clocks to precisely cross timestamp the device
> clock with the system clock. This allows a precise
On Mon, Jan 4, 2016 at 4:45 AM, Christopher S. Hall
wrote:
> ACKNOWLEDGMENT: The original correlated clock source and cross
> timestamp code was developed by Thomas Gleixner
> . It has changed considerably and any mistakes are
> mine.
>
> The
On Mon, Jan 4, 2016 at 4:45 AM, Christopher S. Hall
wrote:
> The code in ktime_get_snapshot() is a superset of the code in
> ktime_get_raw_and_real() code. Changes the latter to call the former. A
> side effect of this is that ktime_get_raw_and_real() returns two
On Mon, Jan 4, 2016 at 4:45 AM, Christopher S. Hall
wrote:
> @@ -13,6 +13,9 @@
> /**
> * struct tk_read_base - base structure for timekeeping readout
> * @clock: Current clocksource used for timekeeping.
> + * @cs_seq:Clocksource sequence is incremented
On Tue, Nov 3, 2015 at 11:18 AM, Stanton, Kevin B
<kevin.b.stan...@intel.com> wrote:
> On Wed, 21 Oct 2015, Thomas Gleixner wrote:
>> On Tue, 20 Oct 2015, John Stultz wrote:
>>> Being able to have various hardware sharing a time base is quite
>>> useful, and m
On Tue, Oct 20, 2015 at 12:11 PM, Thomas Gleixner wrote:
> On Tue, 20 Oct 2015, Richard Cochran wrote:
>> On Tue, Oct 20, 2015 at 01:51:13PM +0200, Richard Cochran wrote:
>> > You can, in fact, achieve "proper" correlation by sampling. As John
>> > said, the question is
On Mon, Oct 19, 2015 at 5:18 PM, Christopher Hall
wrote:
> On Thu, 15 Oct 2015 01:15:57 -0700, Thomas Gleixner
> wrote:
>>>
>>> >
>>> > > +#define SHADOW_HISTORY_DEPTH 7
>>> >
>>> > And that number is 7 because?
>>>
>>> Due to power of 2 it will
person
>> > to merge them.
>>
>> I'm happy with this going via a tree other than mine, and for the
>
> I think it should go via John Stultz timekeeping tree.
I've queued the set for testing.
thanks
-john
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
On Thu, Sep 3, 2015 at 4:20 PM, Hall, Christopher S
wrote:
> For example, supply the ART value as an argument and, in the case of the
> realtime
> clock, keep a short history of clock changes. It would fail in cases where
> there
> are a lot of calls to
On Mon, Jul 27, 2015 at 8:44 PM, John Stultz john.stu...@linaro.org wrote:
On Mon, Jul 27, 2015 at 5:46 PM, Christopher Hall
christopher.s.h...@intel.com wrote:
* counter_to_rawmono64
* counter_to_mono64
* counter_to_realtime64
Enables drivers to translate a captured system clock counter
On Mon, Jul 27, 2015 at 5:46 PM, Christopher Hall
christopher.s.h...@intel.com wrote:
* counter_to_rawmono64
* counter_to_mono64
* counter_to_realtime64
Enables drivers to translate a captured system clock counter to system
time. This is useful for network and audio devices that capture
On Mon, Jul 27, 2015 at 5:46 PM, Christopher Hall
christopher.s.h...@intel.com wrote:
+static bool checked_art_to_tsc(cycle_t *tsc)
+{
+ if (!has_art())
+ return false;
+ *tsc = art_to_tsc(*tsc);
+ return true;
+}
+
+static int art_to_rawmono64(struct
the FRV arch could be trivially converted
to GENERIC_TIME.
Would you consider the following, totally untested patch?
Signed-off-by: John Stultz [EMAIL PROTECTED]
Kconfig |4 ++
kernel/time.c | 81 --
2 files changed, 4 insertions
Both of these were seen on my laptop w/ the current (as of this writing)
-git tree using the e1000 driver after a suspend/resume cycle.
thanks
-john
BUG: warning at net/core/dev.c:1171/skb_checksum_help()
[c0103d69] show_trace_log_lvl+0x149/0x170
[c01052bb] show_trace+0x1b/0x20
[c01052e4]
64 matches
Mail list logo