linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH for v5.9] sch_cake: Replace HTTP links with HTTPS ones
@ 2020-07-19 12:22 Alexander A. Klimov
  2020-07-19 22:17 ` David Miller
  0 siblings, 1 reply; 2+ messages in thread
From: Alexander A. Klimov @ 2020-07-19 12:22 UTC (permalink / raw)
  To: toke, jhs, xiyou.wangcong, jiri, davem, kuba, cake, netdev, linux-kernel
  Cc: Alexander A. Klimov

Rationale:
Reduces attack surface on kernel devs opening the links for MITM
as HTTPS traffic is much harder to manipulate.

Deterministic algorithm:
For each file:
  If not .svg:
    For each line:
      If doesn't contain `\bxmlns\b`:
        For each link, `\bhttp://[^# \t\r\n]*(?:\w|/)`:
	  If neither `\bgnu\.org/license`, nor `\bmozilla\.org/MPL\b`:
            If both the HTTP and HTTPS versions
            return 200 OK and serve the same content:
              Replace HTTP with HTTPS.

Signed-off-by: Alexander A. Klimov <grandmaster@al2klimov.de>
---
 Continuing my work started at 93431e0607e5.
 See also: git log --oneline '--author=Alexander A. Klimov <grandmaster@al2klimov.de>' v5.7..master
 (Actually letting a shell for loop submit all this stuff for me.)

 If there are any URLs to be removed completely
 or at least not (just) HTTPSified:
 Just clearly say so and I'll *undo my change*.
 See also: https://lkml.org/lkml/2020/6/27/64

 If there are any valid, but yet not changed URLs:
 See: https://lkml.org/lkml/2020/6/26/837

 If you apply the patch, please let me know.

 Sorry again to all maintainers who complained about subject lines.
 Now I realized that you want an actually perfect prefixes,
 not just subsystem ones.
 I tried my best...
 And yes, *I could* (at least half-)automate it.
 Impossible is nothing! :)


 net/sched/sch_cake.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/net/sched/sch_cake.c b/net/sched/sch_cake.c
index ebaeec1e5c82..2f6c0daa2337 100644
--- a/net/sched/sch_cake.c
+++ b/net/sched/sch_cake.c
@@ -363,7 +363,7 @@ static const u8 bulk_order[] = {1, 0, 2, 3};
 #define REC_INV_SQRT_CACHE (16)
 static u32 cobalt_rec_inv_sqrt_cache[REC_INV_SQRT_CACHE] = {0};
 
-/* http://en.wikipedia.org/wiki/Methods_of_computing_square_roots
+/* https://en.wikipedia.org/wiki/Methods_of_computing_square_roots
  * new_invsqrt = (invsqrt / 2) * (3 - count * invsqrt^2)
  *
  * Here, invsqrt is a fixed point number (< 1.0), 32bit mantissa, aka Q0.32
-- 
2.27.0


^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH for v5.9] sch_cake: Replace HTTP links with HTTPS ones
  2020-07-19 12:22 [PATCH for v5.9] sch_cake: Replace HTTP links with HTTPS ones Alexander A. Klimov
@ 2020-07-19 22:17 ` David Miller
  0 siblings, 0 replies; 2+ messages in thread
From: David Miller @ 2020-07-19 22:17 UTC (permalink / raw)
  To: grandmaster
  Cc: toke, jhs, xiyou.wangcong, jiri, kuba, cake, netdev, linux-kernel


Please don't do this "for v5.9" stuff.

There is no precendence for this and it looks like -stable patch
series to just about anyone.

There are well defined, established, ways to write Subject lines for
proper patch submissions.

Please do not invent your own way of doing this.  It is very
frustrating for me personally, especially with the amount of
patches I have to process and review, to watch how you wildly
change your patch submission formatting over and over again.

Just do what other developers are doing and you'll be fine.  Don't
invent your own way.

I'm ignoring all of your "for v5.9" formatted patches, sorry.

Thanks.

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2020-07-19 22:18 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-07-19 12:22 [PATCH for v5.9] sch_cake: Replace HTTP links with HTTPS ones Alexander A. Klimov
2020-07-19 22:17 ` David Miller

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).