* [PATCH net-next] docs: networking: timestamping: replace tabs with spaces in code blocks
@ 2020-07-09 20:22 Vladimir Oltean
2020-07-15 19:17 ` Jakub Kicinski
0 siblings, 1 reply; 3+ messages in thread
From: Vladimir Oltean @ 2020-07-09 20:22 UTC (permalink / raw)
To: netdev, davem; +Cc: richardcochran, sorganov, andrew
Reading the document in vim is currently not a pleasant experience. Its
rst syntax highlighting is confused by the "*/" sequences which it's not
interpreting as part of the code blocks for some reason.
Replace the tabs with spaces, so that syntax highlighters (at least the
one in vim) have a better idea where code blocks start and where they
end.
Signed-off-by: Vladimir Oltean <olteanv@gmail.com>
---
Documentation/networking/timestamping.rst | 114 +++++++++++-----------
1 file changed, 57 insertions(+), 57 deletions(-)
diff --git a/Documentation/networking/timestamping.rst b/Documentation/networking/timestamping.rst
index 03f7beade470..5fa4e2274dd9 100644
--- a/Documentation/networking/timestamping.rst
+++ b/Documentation/networking/timestamping.rst
@@ -257,13 +257,13 @@ setsockopt::
struct msghdr *msg;
...
- cmsg = CMSG_FIRSTHDR(msg);
- cmsg->cmsg_level = SOL_SOCKET;
- cmsg->cmsg_type = SO_TIMESTAMPING;
- cmsg->cmsg_len = CMSG_LEN(sizeof(__u32));
+ cmsg = CMSG_FIRSTHDR(msg);
+ cmsg->cmsg_level = SOL_SOCKET;
+ cmsg->cmsg_type = SO_TIMESTAMPING;
+ cmsg->cmsg_len = CMSG_LEN(sizeof(__u32));
*((__u32 *) CMSG_DATA(cmsg)) = SOF_TIMESTAMPING_TX_SCHED |
- SOF_TIMESTAMPING_TX_SOFTWARE |
- SOF_TIMESTAMPING_TX_ACK;
+ SOF_TIMESTAMPING_TX_SOFTWARE |
+ SOF_TIMESTAMPING_TX_ACK;
err = sendmsg(fd, msg, 0);
The SOF_TIMESTAMPING_TX_* flags set via cmsg will override
@@ -273,7 +273,7 @@ Moreover, applications must still enable timestamp reporting via
setsockopt to receive timestamps::
__u32 val = SOF_TIMESTAMPING_SOFTWARE |
- SOF_TIMESTAMPING_OPT_ID /* or any other flag */;
+ SOF_TIMESTAMPING_OPT_ID /* or any other flag */;
err = setsockopt(fd, SOL_SOCKET, SO_TIMESTAMPING, &val, sizeof(val));
@@ -354,14 +354,14 @@ SOL_SOCKET, cmsg_type SCM_TIMESTAMPING, and payload of type
For SO_TIMESTAMPING_OLD::
- struct scm_timestamping {
- struct timespec ts[3];
- };
+ struct scm_timestamping {
+ struct timespec ts[3];
+ };
For SO_TIMESTAMPING_NEW::
- struct scm_timestamping64 {
- struct __kernel_timespec ts[3];
+ struct scm_timestamping64 {
+ struct __kernel_timespec ts[3];
Always use SO_TIMESTAMPING_NEW timestamp to always get timestamp in
struct scm_timestamping64 format.
@@ -468,11 +468,11 @@ Hardware time stamping must also be initialized for each device driver
that is expected to do hardware time stamping. The parameter is defined in
include/uapi/linux/net_tstamp.h as::
- struct hwtstamp_config {
- int flags; /* no flags defined right now, must be zero */
- int tx_type; /* HWTSTAMP_TX_* */
- int rx_filter; /* HWTSTAMP_FILTER_* */
- };
+ struct hwtstamp_config {
+ int flags; /* no flags defined right now, must be zero */
+ int tx_type; /* HWTSTAMP_TX_* */
+ int rx_filter; /* HWTSTAMP_FILTER_* */
+ };
Desired behavior is passed into the kernel and to a specific device by
calling ioctl(SIOCSHWTSTAMP) with a pointer to a struct ifreq whose
@@ -505,42 +505,42 @@ not been implemented in all drivers.
::
- /* possible values for hwtstamp_config->tx_type */
- enum {
- /*
- * no outgoing packet will need hardware time stamping;
- * should a packet arrive which asks for it, no hardware
- * time stamping will be done
- */
- HWTSTAMP_TX_OFF,
-
- /*
- * enables hardware time stamping for outgoing packets;
- * the sender of the packet decides which are to be
- * time stamped by setting SOF_TIMESTAMPING_TX_SOFTWARE
- * before sending the packet
- */
- HWTSTAMP_TX_ON,
- };
-
- /* possible values for hwtstamp_config->rx_filter */
- enum {
- /* time stamp no incoming packet at all */
- HWTSTAMP_FILTER_NONE,
-
- /* time stamp any incoming packet */
- HWTSTAMP_FILTER_ALL,
-
- /* return value: time stamp all packets requested plus some others */
- HWTSTAMP_FILTER_SOME,
-
- /* PTP v1, UDP, any kind of event packet */
- HWTSTAMP_FILTER_PTP_V1_L4_EVENT,
-
- /* for the complete list of values, please check
- * the include file include/uapi/linux/net_tstamp.h
- */
- };
+ /* possible values for hwtstamp_config->tx_type */
+ enum {
+ /*
+ * no outgoing packet will need hardware time stamping;
+ * should a packet arrive which asks for it, no hardware
+ * time stamping will be done
+ */
+ HWTSTAMP_TX_OFF,
+
+ /*
+ * enables hardware time stamping for outgoing packets;
+ * the sender of the packet decides which are to be
+ * time stamped by setting SOF_TIMESTAMPING_TX_SOFTWARE
+ * before sending the packet
+ */
+ HWTSTAMP_TX_ON,
+ };
+
+ /* possible values for hwtstamp_config->rx_filter */
+ enum {
+ /* time stamp no incoming packet at all */
+ HWTSTAMP_FILTER_NONE,
+
+ /* time stamp any incoming packet */
+ HWTSTAMP_FILTER_ALL,
+
+ /* return value: time stamp all packets requested plus some others */
+ HWTSTAMP_FILTER_SOME,
+
+ /* PTP v1, UDP, any kind of event packet */
+ HWTSTAMP_FILTER_PTP_V1_L4_EVENT,
+
+ /* for the complete list of values, please check
+ * the include file include/uapi/linux/net_tstamp.h
+ */
+ };
3.1 Hardware Timestamping Implementation: Device Drivers
--------------------------------------------------------
@@ -555,10 +555,10 @@ to the shared time stamp structure of the skb call skb_hwtstamps(). Then
set the time stamps in the structure::
struct skb_shared_hwtstamps {
- /* hardware time stamp transformed into duration
- * since arbitrary point in time
- */
- ktime_t hwtstamp;
+ /* hardware time stamp transformed into duration
+ * since arbitrary point in time
+ */
+ ktime_t hwtstamp;
};
Time stamps for outgoing packets are to be generated as follows:
--
2.25.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH net-next] docs: networking: timestamping: replace tabs with spaces in code blocks
2020-07-09 20:22 [PATCH net-next] docs: networking: timestamping: replace tabs with spaces in code blocks Vladimir Oltean
@ 2020-07-15 19:17 ` Jakub Kicinski
2020-07-15 19:29 ` Matthew Wilcox
0 siblings, 1 reply; 3+ messages in thread
From: Jakub Kicinski @ 2020-07-15 19:17 UTC (permalink / raw)
To: Vladimir Oltean, Jonathan Corbet
Cc: netdev, davem, richardcochran, sorganov, andrew, linux-doc
On Thu, 9 Jul 2020 23:22:10 +0300 Vladimir Oltean wrote:
> Reading the document in vim is currently not a pleasant experience. Its
> rst syntax highlighting is confused by the "*/" sequences which it's not
> interpreting as part of the code blocks for some reason.
>
> Replace the tabs with spaces, so that syntax highlighters (at least the
> one in vim) have a better idea where code blocks start and where they
> end.
>
> Signed-off-by: Vladimir Oltean <olteanv@gmail.com>
Dunno about this change, there seems to be plenty examples of using
tabs for indentation of code block :(
Jon, any guidance? It seems to me the document renders correctly [1],
so the onus is on the editor to fix the RST highlight..
[1] https://www.kernel.org/doc/html/latest/networking/timestamping.html
> diff --git a/Documentation/networking/timestamping.rst b/Documentation/networking/timestamping.rst
> index 03f7beade470..5fa4e2274dd9 100644
> --- a/Documentation/networking/timestamping.rst
> +++ b/Documentation/networking/timestamping.rst
> @@ -257,13 +257,13 @@ setsockopt::
>
> struct msghdr *msg;
> ...
> - cmsg = CMSG_FIRSTHDR(msg);
> - cmsg->cmsg_level = SOL_SOCKET;
> - cmsg->cmsg_type = SO_TIMESTAMPING;
> - cmsg->cmsg_len = CMSG_LEN(sizeof(__u32));
> + cmsg = CMSG_FIRSTHDR(msg);
> + cmsg->cmsg_level = SOL_SOCKET;
> + cmsg->cmsg_type = SO_TIMESTAMPING;
> + cmsg->cmsg_len = CMSG_LEN(sizeof(__u32));
> *((__u32 *) CMSG_DATA(cmsg)) = SOF_TIMESTAMPING_TX_SCHED |
> - SOF_TIMESTAMPING_TX_SOFTWARE |
> - SOF_TIMESTAMPING_TX_ACK;
> + SOF_TIMESTAMPING_TX_SOFTWARE |
> + SOF_TIMESTAMPING_TX_ACK;
> err = sendmsg(fd, msg, 0);
>
> The SOF_TIMESTAMPING_TX_* flags set via cmsg will override
> @@ -273,7 +273,7 @@ Moreover, applications must still enable timestamp reporting via
> setsockopt to receive timestamps::
>
> __u32 val = SOF_TIMESTAMPING_SOFTWARE |
> - SOF_TIMESTAMPING_OPT_ID /* or any other flag */;
> + SOF_TIMESTAMPING_OPT_ID /* or any other flag */;
> err = setsockopt(fd, SOL_SOCKET, SO_TIMESTAMPING, &val, sizeof(val));
>
>
> @@ -354,14 +354,14 @@ SOL_SOCKET, cmsg_type SCM_TIMESTAMPING, and payload of type
>
> For SO_TIMESTAMPING_OLD::
>
> - struct scm_timestamping {
> - struct timespec ts[3];
> - };
> + struct scm_timestamping {
> + struct timespec ts[3];
> + };
>
> For SO_TIMESTAMPING_NEW::
>
> - struct scm_timestamping64 {
> - struct __kernel_timespec ts[3];
> + struct scm_timestamping64 {
> + struct __kernel_timespec ts[3];
>
> Always use SO_TIMESTAMPING_NEW timestamp to always get timestamp in
> struct scm_timestamping64 format.
> @@ -468,11 +468,11 @@ Hardware time stamping must also be initialized for each device driver
> that is expected to do hardware time stamping. The parameter is defined in
> include/uapi/linux/net_tstamp.h as::
>
> - struct hwtstamp_config {
> - int flags; /* no flags defined right now, must be zero */
> - int tx_type; /* HWTSTAMP_TX_* */
> - int rx_filter; /* HWTSTAMP_FILTER_* */
> - };
> + struct hwtstamp_config {
> + int flags; /* no flags defined right now, must be zero */
> + int tx_type; /* HWTSTAMP_TX_* */
> + int rx_filter; /* HWTSTAMP_FILTER_* */
> + };
>
> Desired behavior is passed into the kernel and to a specific device by
> calling ioctl(SIOCSHWTSTAMP) with a pointer to a struct ifreq whose
> @@ -505,42 +505,42 @@ not been implemented in all drivers.
>
> ::
>
> - /* possible values for hwtstamp_config->tx_type */
> - enum {
> - /*
> - * no outgoing packet will need hardware time stamping;
> - * should a packet arrive which asks for it, no hardware
> - * time stamping will be done
> - */
> - HWTSTAMP_TX_OFF,
> -
> - /*
> - * enables hardware time stamping for outgoing packets;
> - * the sender of the packet decides which are to be
> - * time stamped by setting SOF_TIMESTAMPING_TX_SOFTWARE
> - * before sending the packet
> - */
> - HWTSTAMP_TX_ON,
> - };
> -
> - /* possible values for hwtstamp_config->rx_filter */
> - enum {
> - /* time stamp no incoming packet at all */
> - HWTSTAMP_FILTER_NONE,
> -
> - /* time stamp any incoming packet */
> - HWTSTAMP_FILTER_ALL,
> -
> - /* return value: time stamp all packets requested plus some others */
> - HWTSTAMP_FILTER_SOME,
> -
> - /* PTP v1, UDP, any kind of event packet */
> - HWTSTAMP_FILTER_PTP_V1_L4_EVENT,
> -
> - /* for the complete list of values, please check
> - * the include file include/uapi/linux/net_tstamp.h
> - */
> - };
> + /* possible values for hwtstamp_config->tx_type */
> + enum {
> + /*
> + * no outgoing packet will need hardware time stamping;
> + * should a packet arrive which asks for it, no hardware
> + * time stamping will be done
> + */
> + HWTSTAMP_TX_OFF,
> +
> + /*
> + * enables hardware time stamping for outgoing packets;
> + * the sender of the packet decides which are to be
> + * time stamped by setting SOF_TIMESTAMPING_TX_SOFTWARE
> + * before sending the packet
> + */
> + HWTSTAMP_TX_ON,
> + };
> +
> + /* possible values for hwtstamp_config->rx_filter */
> + enum {
> + /* time stamp no incoming packet at all */
> + HWTSTAMP_FILTER_NONE,
> +
> + /* time stamp any incoming packet */
> + HWTSTAMP_FILTER_ALL,
> +
> + /* return value: time stamp all packets requested plus some others */
> + HWTSTAMP_FILTER_SOME,
> +
> + /* PTP v1, UDP, any kind of event packet */
> + HWTSTAMP_FILTER_PTP_V1_L4_EVENT,
> +
> + /* for the complete list of values, please check
> + * the include file include/uapi/linux/net_tstamp.h
> + */
> + };
>
> 3.1 Hardware Timestamping Implementation: Device Drivers
> --------------------------------------------------------
> @@ -555,10 +555,10 @@ to the shared time stamp structure of the skb call skb_hwtstamps(). Then
> set the time stamps in the structure::
>
> struct skb_shared_hwtstamps {
> - /* hardware time stamp transformed into duration
> - * since arbitrary point in time
> - */
> - ktime_t hwtstamp;
> + /* hardware time stamp transformed into duration
> + * since arbitrary point in time
> + */
> + ktime_t hwtstamp;
> };
>
> Time stamps for outgoing packets are to be generated as follows:
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH net-next] docs: networking: timestamping: replace tabs with spaces in code blocks
2020-07-15 19:17 ` Jakub Kicinski
@ 2020-07-15 19:29 ` Matthew Wilcox
0 siblings, 0 replies; 3+ messages in thread
From: Matthew Wilcox @ 2020-07-15 19:29 UTC (permalink / raw)
To: Jakub Kicinski
Cc: Vladimir Oltean, Jonathan Corbet, netdev, davem, richardcochran,
sorganov, andrew, linux-doc
On Wed, Jul 15, 2020 at 12:17:17PM -0700, Jakub Kicinski wrote:
> On Thu, 9 Jul 2020 23:22:10 +0300 Vladimir Oltean wrote:
> > Reading the document in vim is currently not a pleasant experience. Its
> > rst syntax highlighting is confused by the "*/" sequences which it's not
> > interpreting as part of the code blocks for some reason.
> >
> > Replace the tabs with spaces, so that syntax highlighters (at least the
> > one in vim) have a better idea where code blocks start and where they
> > end.
> >
> > Signed-off-by: Vladimir Oltean <olteanv@gmail.com>
>
> Dunno about this change, there seems to be plenty examples of using
> tabs for indentation of code block :(
>
> Jon, any guidance? It seems to me the document renders correctly [1],
> so the onus is on the editor to fix the RST highlight..
vim's rst mode really really really wants to use spaces instead of tabs.
It's screwed me over a bunch of times, so I eventually just disabled it.
I think we should probably stop fighting it and use spaces in rst files
instead of tabs, but that's not my call to make.
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2020-07-15 19:29 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-07-09 20:22 [PATCH net-next] docs: networking: timestamping: replace tabs with spaces in code blocks Vladimir Oltean
2020-07-15 19:17 ` Jakub Kicinski
2020-07-15 19:29 ` Matthew Wilcox
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.