Thread (15 messages) flat view 15 messages, 5 authors, 2015-05-18

Re: [PATCH net-next v3 2/2] net: Adding support for Cavium ThunderX network controller

From: David Miller <davem@davemloft.net>
Date: 2015-05-18 23:52:34
Also in: linux-arm-kernel, lkml

From: David Daney <redacted>
Date: Mon, 18 May 2015 14:52:30 -0700
As I see it there are two options:

1) Keep the writeq_relaxed()/readq_relaxed(), but add a comment about
why they are safe.

2) Change the patch so that we are using writeq()/readq() and suffer a
decrease in performance.
I want a detailed comment added, so #1 would be my choice.

There are more than one set of register accessors, so make sure you
annotate them all.

Thanks.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help