-
Notifications
You must be signed in to change notification settings - Fork 0
/
Copy pathhate.json
1254 lines (1254 loc) · 109 KB
/
hate.json
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
284
285
286
287
288
289
290
291
292
293
294
295
296
297
298
299
300
301
302
303
304
305
306
307
308
309
310
311
312
313
314
315
316
317
318
319
320
321
322
323
324
325
326
327
328
329
330
331
332
333
334
335
336
337
338
339
340
341
342
343
344
345
346
347
348
349
350
351
352
353
354
355
356
357
358
359
360
361
362
363
364
365
366
367
368
369
370
371
372
373
374
375
376
377
378
379
380
381
382
383
384
385
386
387
388
389
390
391
392
393
394
395
396
397
398
399
400
401
402
403
404
405
406
407
408
409
410
411
412
413
414
415
416
417
418
419
420
421
422
423
424
425
426
427
428
429
430
431
432
433
434
435
436
437
438
439
440
441
442
443
444
445
446
447
448
449
450
451
452
453
454
455
456
457
458
459
460
461
462
463
464
465
466
467
468
469
470
471
472
473
474
475
476
477
478
479
480
481
482
483
484
485
486
487
488
489
490
491
492
493
494
495
496
497
498
499
500
501
502
503
504
505
506
507
508
509
510
511
512
513
514
515
516
517
518
519
520
521
522
523
524
525
526
527
528
529
530
531
532
533
534
535
536
537
538
539
540
541
542
543
544
545
546
547
548
549
550
551
552
553
554
555
556
557
558
559
560
561
562
563
564
565
566
567
568
569
570
571
572
573
574
575
576
577
578
579
580
581
582
583
584
585
586
587
588
589
590
591
592
593
594
595
596
597
598
599
600
601
602
603
604
605
606
607
608
609
610
611
612
613
614
615
616
617
618
619
620
621
622
623
624
625
626
627
628
629
630
631
632
633
634
635
636
637
638
639
640
641
642
643
644
645
646
647
648
649
650
651
652
653
654
655
656
657
658
659
660
661
662
663
664
665
666
667
668
669
670
671
672
673
674
675
676
677
678
679
680
681
682
683
684
685
686
687
688
689
690
691
692
693
694
695
696
697
698
699
700
701
702
703
704
705
706
707
708
709
710
711
712
713
714
715
716
717
718
719
720
721
722
723
724
725
726
727
728
729
730
731
732
733
734
735
736
737
738
739
740
741
742
743
744
745
746
747
748
749
750
751
752
753
754
755
756
757
758
759
760
761
762
763
764
765
766
767
768
769
770
771
772
773
774
775
776
777
778
779
780
781
782
783
784
785
786
787
788
789
790
791
792
793
794
795
796
797
798
799
800
801
802
803
804
805
806
807
808
809
810
811
812
813
814
815
816
817
818
819
820
821
822
823
824
825
826
827
828
829
830
831
832
833
834
835
836
837
838
839
840
841
842
843
844
845
846
847
848
849
850
851
852
853
854
855
856
857
858
859
860
861
862
863
864
865
866
867
868
869
870
871
872
873
874
875
876
877
878
879
880
881
882
883
884
885
886
887
888
889
890
891
892
893
894
895
896
897
898
899
900
901
902
903
904
905
906
907
908
909
910
911
912
913
914
915
916
917
918
919
920
921
922
923
924
925
926
927
928
929
930
931
932
933
934
935
936
937
938
939
940
941
942
943
944
945
946
947
948
949
950
951
952
953
954
955
956
957
958
959
960
961
962
963
964
965
966
967
968
969
970
971
972
973
974
975
976
977
978
979
980
981
982
983
984
985
986
987
988
989
990
991
992
993
994
995
996
997
998
999
1000
[
{
"text": "Why are you making up these completely invalid arguments? Because you are making them up....And given this *fact*, your denial that \"PCI reboot should never be used\" is counterfactual. It may be true in some theoretical \"this is how the world should work\" universe, but in the real world it is just BS. Why are you so deep in denial about this?",
"hate": 0.27520477805473287
},
{
"text": "A computer is like air conditioning - it becomes useless if you open Windows",
"hate": 0.6
},
{
"text": "What drugs are you on? Your example is moronic, and against all _documented_ uses of chroot.",
"hate": 0.3011801299692075
},
{
"text": "Oh christ. This is exactly what the scheduler has ALWAYS ALREADY DONE FOR YOU. ... Stop doing it. You get *zero* advantages from just doing what the scheduler natively does for you, and the scheduler does it *better*.",
"hate": 0.401778212265373
},
{
"text": "Really. Shut up.... And if you aren't ok with \"wasting time\" on trying to give that kind of reassurances to users, then you shouldn't be working on the kernel. I'm serious about this. You really *need* to understand that. Your job as a kernel developer is very much to support the users. Not try to make it easy for *you* at the cost of being nasty for *them*.",
"hate": 0.40353982951892287
},
{
"text": "Andy, you need to lay off the drugs.",
"hate": 0.4465161295570569
},
{
"text": "Because you screwed up that pull request, and I argue that you screwed up exactly *because* it's ambiguous and confusing.",
"hate": 0.4482903428839735
},
{
"text": "And that audit code really is aushit. I think I found a bug in it while just scanning it:",
"hate": 0.4497502159878837
},
{
"text": "Congratulations, you seem to have found a whole new and unique way of screwing up ;) Linus \"my mom called me 'special' too\" Torvalds",
"hate": 0.4613196205893423
},
{
"text": "Are we trying to win some obfuscated C contest here?",
"hate": 0.4868557751547127
},
{
"text": "Seriously. People who use BUG() statements like some kind of assert() are a menace to society. They kill machines.",
"hate": 0.49000780599437455
},
{
"text": "David, what the heck are you doing? ... Seriously. Those commits now have TOTALLY MISLEADING summary messages. ...",
"hate": 0.49979846957843743
},
{
"text": "If most of the oopses you decode are on your own machine with your own kernel, you might want to try to learn to be more careful when writing code. And I'm not even kidding.",
"hate": 0.5207378235075887
},
{
"text": "Grr. You missed the branch name. I can see from the SHA1 (and historical pull requests) that you meant the usual 'v4l_for_linus' branch, but please be more careful.",
"hate": 0.5254318615350907
},
{
"text": "David, I want to make it very clear that if you *ever* suggest another big include file cleanup, I will say \"f*ck no\" and block you from my emails forever. Ok? So don't bother. We're done with these kinds of games. Forever. It's not worth it, don't ever suggest it again for some other \"cleanup\".",
"hate": 0.5272125566149333
},
{
"text": "Christ. This is so ugly that it's almost a work of art.",
"hate": 0.5435223306059218
},
{
"text": "WRONG. Alan, you're not getting it. Loading firmware as part of suspend/resume is WRONG.",
"hate": 0.5482156506373634
},
{
"text": "But dammit, if you build with debug_info and then strip the end result, you're just insane. You made your build take ten times longer, use ten times more diskspace, and then you throw it all away. Crazy.",
"hate": 0.5584212573284328
},
{
"text": "WHAT? NONE OF WHAT YOU SAY MAKES ANY SENSE.",
"hate": 0.5609879912918221
},
{
"text": "That's just disgusting crazy talk. Christ, David, get a grip on yourself. ...",
"hate": 0.5617764857509643
},
{
"text": "Ok. So your commit message and explanation was pure and utter tripe,",
"hate": 0.56989378938028
},
{
"text": "NONE of your statements made any sense at all, since everything you talk about could have been done with a separate project. The only thing the lock-step does is to generate the kind of dependency that I ABSOLUTELY DETEST,",
"hate": 0.5835065754744607
},
{
"text": "Ugh. Your diffstat is crap, because you don't show the inexact renames that are very abundant in the nouveau driver.",
"hate": 0.5854653490310119
},
{
"text": "You do realize that none of your arguments touched the \"why should Linus merge the tree\" question at all? Everything you said was about how it's more convenient for you and Ingo, not at all about why it should be better for anybody else. ... You're the only one working on it, so being convenient for you is the primary issue. Arguments like that actively make me not want to merge it, ...",
"hate": 0.5858136278335135
},
{
"text": "...So stop complaining. Reverts really *are* just patches, Greg is 100% right, and you are simply wrong. ...and the fact that you *continue* to complain just makes you look stupid.",
"hate": 0.5878761330573111
},
{
"text": "See what my complaint is? Not this half-assery that used to be a small random detail, and that the patch makes into an institutionalized and explicit half-assery. (And Mikhail - I'm not ragging on you, even if I'm ragging on the patch. I understand why you did it the way you did, and it makes sense exactly in the \"let's reinstate old hackery\" model. I just think we can and should do better than that, now that the \"exchange\" vs \"move over\" semantics are so explicit)",
"hate": 0.5893440056351167
},
{
"text": "Adding Andrea to the Cc, because he's the author of that horridness. Putting Steven's test-case here as an attachement for Andrea, maybe that makes him go \"Ahh, yes, silly case\". Also added Kirill, because he was involved the last _PAGE_NUMA debacle.",
"hate": 0.5894206644406071
},
{
"text": "Bullshit, Andrea. That's *exactly* what you said in the commit message for the broken patch that I complained about. ... and I pointed out that your commit message was garbage, and that it's not at all as easy as you claim, and that your patch was broken, and your description was even more broken.... Your commit message was garbage, and actively misleading. Don't make excuses.",
"hate": 0.5917782985862905
},
{
"text": "That's f*cking sad. You know *why* it's sad? ... Now, that should make you think about THE ABSOLUTE CRAP YOU MARK FOR -stable! ... Listen to yourself. In fact, there is a damn good solution\": don't mark crap for stable, and don't send crap to me after -rc4. ... Greg, the reason you get a lot of stable patches seems to be that you make it easy to act as a door-mat. ... You may need to learn to shout at people.",
"hate": 0.5947676945714664
},
{
"text": "Don't try to change the rules because you think you are \"clever\". You're only making things worse.",
"hate": 0.5958461581038789
},
{
"text": "Please, Debabrata, humor me, and just try the patch. And try reading the source code. Because your statement is BS.",
"hate": 0.5966823729721406
},
{
"text": "What a crock. That direct-IO code is hack-upon-hack. Whoever wrote it should be shot. ...",
"hate": 0.5969650366936845
},
{
"text": "Why? You've made this statement over and over and over again, and I've dismissed it over and over and over again because I simply don't think it's true. It's simply a statement with nothing to back it up. Why repeat it? THAT is my main contention. I told you why I think it's actually actively untrue. ... So you make these unsubstantiated claims about how much easier it is, and they make no sense. You never explain *why* it's so magically easier. ... Anyway, I'm done arguing. You can do what you want, but just stop misrepresenting it as being \"linux-next\" material unless you are willing to actually explain why it should be so.",
"hate": 0.6018116750393498
},
{
"text": "Did anybody ever actually look at this sh*t-for-brains patch? Yeah, I'm grumpy. But I'm wasting time looking at patches that have new code in them that is stupid and retarded. This is the VM, guys, we don't add stupid and retarded code. LOOK at the code, for chrissake. Just look at it. And if you don't see why the above is stupid and retarded, you damn well shouldn't be touching VM code.",
"hate": 0.6033242178323879
},
{
"text": "Umm. I think your argument is totally braindead and wrong. My counter-argument is very simple: \"So what?\"",
"hate": 0.6065125267770252
},
{
"text": "The games with \"max_block\" are hilarious. In a really sad way. That whole blkdev_get_blocks() function is pure and utter shit.",
"hate": 0.609408807546925
},
{
"text": "I don't mind it if it's *one* line, and if people realize that the commentary in the commit in question was pure and utter shit. ... So really, I don't see the point of even a oneliner message. You guys know who the user is. There's no value in the message. Either you fix the user or you don't.",
"hate": 0.609458734134122
},
{
"text": "You make no sense. The commits you list were all on top of plain 4.0-rc2.",
"hate": 0.6122008243719
},
{
"text": "It appears Intel is fixing their braindamage",
"hate": 0.6132644828017194
},
{
"text": "BS. ...And you ignored the real issue: special-casing idle is *stupid*. It's more complicated, and gives fewer cases where it helps. It's simply fundamentally stupid and wrong.",
"hate": 0.6137721099390107
},
{
"text": "Yes, yes, it may \"work\", but I'm not pulling that kind of hack just before a release....But dammit, using this kind of hackery, ... is just not acceptable.",
"hate": 0.6153743800086194
},
{
"text": "We should definitely drop it. The feature is an abomination. I thought gcc only allowed them at the end of structs, in the middle of a struct it's just f*cking insane beyond belief.",
"hate": 0.6167289494632293
},
{
"text": "Your arguments only make sense if you accept those insane assumptions to begin with. And I don't.",
"hate": 0.6176119930963673
},
{
"text": "... Christ. That code is a mess. ...",
"hate": 0.6192799969735421
},
{
"text": "Kay, this needs to be fixed. ... Of course, I'd also suggest that whoever was the genius who thought it was a good idea to read things ONE F*CKING BYTE AT A TIME with system calls for each byte should be retroactively aborted. Who the f*ck does idiotic things like that? How did they noty die as babies, considering that they were likely too stupid to find a tit to suck on?",
"hate": 0.6210100248236998
},
{
"text": "I'm ok with coding, I find your particular patch horrible. You add a dynamic allocator that will work *horribly* badly if people actually start using it for more complex cases, and then you use that for just about the least interesting case. And the way you do the dynamic allocator, nobody can ever allocate one of the wait-queue entries *efficiently* by just knowing that they are a leaf and there is never any recursive allocation....",
"hate": 0.6253973681379242
},
{
"text": "... Stop the fear mongering already. So here's what I would suggest, and it is based on REAL SECURITY and on PUTTING THE USER FIRST instead of your continual \"let's please microsoft by doing idiotic crap\" approach. ... Quite frankly, *you* are what he key-hating crazies were afraid of. You peddle the \"control, not security\" crap-ware. The whole \"MS owns your machine\" is *exactly* the wrong way to use keys.",
"hate": 0.6257981606555768
},
{
"text": "Hell no.... In exactly *WHAT* crazy universe does that make sense as an argument? It's like saying \"I put literal shit on your plate, because there are potentially nutritious sausages that look superficially a bit like the dogshit I served you\". Seriously. ... It's *exactly* the same argument as \"dog poop superficially looks like good sausages\". Is that really your argument? There is never an excuse for \"usub_overflow()\". It's that simple. No amount of _other_ overflow functions make that shit palatable.",
"hate": 0.6295136856419652
},
{
"text": "Oh, please, that's a British-level understatement. It's like calling WWII \"a small bother\". That's too ugly to live.",
"hate": 0.6301846210610489
},
{
"text": "Has Chris Ball been told what an incredible pain this kind of crap is, and that there's a damn good reason why WE DO NOT REBASE PUBLIC TREES THAT OTHERS MAY BE BASING THEIR DEVELOPMENT ON! Chris, can you hear me shouting? Don't do that.",
"hate": 0.630892693369288
},
{
"text": "Bullshit. That expectation is just a fact. ... We do not say \"user mode shouldn't\". Seriously. EVER. User mode *does*, and we deal with it. Learn it now, and stop ever saying that again. This is really starting to annoy me. Kernel developers who say \"user mode should be fixes to not do that\" should go somewhere else.",
"hate": 0.6344161953611994
},
{
"text": "Christoph, stop arguing. Trust me, Paul knows memory ordering. You clearly do *not*.",
"hate": 0.6383285547138791
},
{
"text": "Stop being a moron. Go back and read the \"nobody can work with you\".",
"hate": 0.6389533107811186
},
{
"text": "But dammit, every single discussion I see, you use some *other* argument for it, like \"people don't have initrd\" or whatever crazy crap. That's what I was objecting to.",
"hate": 0.6394392385636707
},
{
"text": "No. Really. No. ... Thomas, you're in denial. ... Your argument \"it has a question mark in front of it\" objection is bogus. ... I'm just saying that your arguments to ignore CPU0 are pretty damn weak.",
"hate": 0.6418009529098407
},
{
"text": "I'm a moron.",
"hate": 0.6428015433596385
},
{
"text": "You messed up the pull request too.. The branch name is missing from that git line, even if you did mention it a few lines earlier...",
"hate": 0.6442688807221422
},
{
"text": "How f*cking hard is it for you to understand? Stop arguing about what MS wants. We do not care. We care bout the *user*. You are continually missing the whole point of security, and then you make some idiotic arguments about what MS wants you to do. It's irrelevant. The only thing that matters is what our *users* want us to do, and protecting *their* rights. As long as you seem to treat this as some kind of \"let's please MS, not our users\" issue, all your arguments are going to be crap.",
"hate": 0.6470514200973677
},
{
"text": "If you really think that, I hope to God that you have nothing to do with the C standard or any actual compiler I ever use. Because such a standard or compiler would be shit. It's sadly not too uncommon",
"hate": 0.647358037294621
},
{
"text": "Well, that's one way of reading that callchain. I think it's the *wrong* way of reading it, though. Almost dishonestly so.",
"hate": 0.6497172173159032
},
{
"text": "Greg, this is BS. ... so now you've re-introduced part of the problem, and marked it for stable too. The commit log shows nothing useful. ... And it really _isn't_ a good idea. ... Don't do this shit.",
"hate": 0.6507907365954533
},
{
"text": "And the commit seems to be pure shit. Why is it pure shit? Look at what users are left. THERE ARE NO USERS THAT SET THAT FIELD ANY MORE! ... I've pulled the changes for now, but I absolutely *detest* seeing things like this. ...",
"hate": 0.6520034837534499
},
{
"text": "Christ, why can't people learn?",
"hate": 0.6547076153082676
},
{
"text": "and this, btw, is just another example of why MCE hardware designers are f*cking morons that should be given extensive education about birth control and how not to procreate.",
"hate": 0.6558956701473204
},
{
"text": "Yeah, I'm a f*cking moron.",
"hate": 0.658394270909275
},
{
"text": "Oh christ. What insane version of gcc is that? Can you please make a gcc bug-report? ... is just so fricking stupid that it's outright buggy. That's just crazy. It's demented. It's an \"and\" with all bits set.",
"hate": 0.6633325780254008
},
{
"text": "Absolutely. Anybody who does that is just terminally confused. \"return()\" is in no way a function. ... Here's an example of a really bad use of \"sizeof\" that doesn't have the parenthesis around the argument: sizeof(*p)->member. Quite frankly, if you do this, you should be shot. ... And let's face it: if you write your code so that it's easy to parse for a machine, and ignore how easy it is to parse for a human, I don't want you writing kernel code. ...",
"hate": 0.6648610850175812
},
{
"text": "Ok, this code is a rats nest. ... The code is crazy. It's an unreadable mess. Not surprising that it then also is buggy.... Looking at the code, I don't think it has been written by a human. ... Some of that code is clearly pure garbage. ... In fact, it's *all* crap.",
"hate": 0.6655628135523088
},
{
"text": "I've pulled this, but I was pretty close to saying \"screw this shit\". Look at commit 9a630d15f16d, and pray tell me why those kinds of commit logs are excusable? That commit message is totally worthless noise. ... Seriously. ... That commit 9a630d15f16d is pure garbage. It's not the only crappy one, but it really does stand out. ...I'd really prefer it to talk about what it merges and why, but it's still *much* better than your completely information-free merge message.",
"hate": 0.666816599355471
},
{
"text": "Christ, Mel. Your reasons in b22d127a39dd are weak as hell, and then you come up with *THIS* shit instead: ... Heck no. In fact, not a f*cking way in hell. Look yourself in the mirror, Mel. This patch is ugly, and *guaranteed* to result in subtle locking issues, and then you have the *gall* to quote the \"uhh, that's a bit ugly due to some trivial duplication\" thing in commit... compared to the diseased abortion you just posted...",
"hate": 0.6688304255216247
},
{
"text": "I really dislike this one.... The other patches look sane, this one I really don't like. You may have good reasons for it, but it's disgusting.",
"hate": 0.6700635035159841
},
{
"text": "So you're potentially making things worse for just about everybody, in order to fix a problem that almost nobody can actually see. And apparently you don't even know the problem.. and as I already explained, THAT IS PURE AND UTTER BULLSHIT. It may make things WORSE. On all the things you haven't run to check that it does better. You just stated something that is not at all necessarily true. .... That's pure bullshit. ... And yet you go on to say that it will fix performance problems THAT WE DON'T EVEN KNOW ABOUT! After seeing *proof* to the exact contrary behavior! What f*cking planet are you from, again? Christ, that's hubris. ...",
"hate": 0.6735483994285129
},
{
"text": "Ingo, stop this idiotic nonsense. You seem to think that \"kvmtool is useful for kernel\" is somehow relevant. IT IS TOTALLY IRRELEVANT.",
"hate": 0.6749470293279405
},
{
"text": "Ok, I'm sorry, but that's just pure bullshit then. ... This code is pure and utter garbage. It's beyond the pale how crazy it is.",
"hate": 0.6760944134005538
},
{
"text": "So stop spouting garbage.",
"hate": 0.6763883775150119
},
{
"text": "Christ, we should just try to get rid of the personality bits entirely, they are completely insane",
"hate": 0.6798668047659948
},
{
"text": "Quite frankly, this is f*cking moronic.",
"hate": 0.6876318867401255
},
{
"text": "Grr. I've pulled it, but looking at that history, it's just pure and utter f*cking garbage.",
"hate": 0.6902609711816332
},
{
"text": "That statement is so nonsensical that I can't get past it. When you understand why it is nonsensical, you understand why the bit is cleared. Feel free to bring this up again without that idiotic statement as an argument.",
"hate": 0.6908697958193426
},
{
"text": "So I'm not saying \"ifconfig is wonderful\". It's not. But I *am* saying that \"changing user interfaces and then expecting people to change is f*cking stupid\".... Because people who think that \"we'll just redesign everything\" are actually f*cking morons. Really. There's a real reason the kernel has the \"no regression\" policy. And that reason is that I'm not a moron.",
"hate": 0.6909405395866941
},
{
"text": "I'll let you think about just how stupid that comment was for a moment.",
"hate": 0.69095835350407
},
{
"text": "And what *possible* situation could make that \"_once()\" version ever be valid? None. It's bogus. It's crap. It's insane. There is no way that it is *ever* a valid question to even ask.",
"hate": 0.691796243178145
},
{
"text": "...You did *two* of the merges within hours of each other! ... That's just crazy. The fact that you then say that you have some kind of *excuse* for that craziness is just sad. Stop doing that. It's stupid. It just makes it harder for everybody to see what you are doing. ...Can't you see how crazy that is?",
"hate": 0.6974132745024295
},
{
"text": "Yes it damn well is. Stop the f*cking stupid arguments, and instead listen to what I say. Here. Let me bold-face the most important part for you, so that you don't miss it in all the other crap: ... Nothing else. Seriously. Your \"you can't do it because we copy backwards\" arguments are pure and utter garbage, ... You're complicating the whole thing for no good reason. ...",
"hate": 0.698227424180082
},
{
"text": "Wrong. ... so you're just full of it. ... Checking the MCE data is stupid and wrong. Stop doing it, and stop making idiotic excuses for it. ...you are just doing moronic things. Stop doing stupid things.",
"hate": 0.7003878157022334
},
{
"text": "The reading comprehension here is abysmal. ...And none of that matters for my argument AT ALL.",
"hate": 0.7005104776547095
},
{
"text": "So get your act together, and push back on the people you are supposed to manage. Because this is *not* acceptable for post-rc5, and I'm giving this single warning. Next time, I'll just ignore the sh*t you send me. Comprende?",
"hate": 0.7013781127365946
},
{
"text": "Is this whole thread still just for the crazy and pointless \"max_sane_readahead()\"? Or is there some *real* reason we should care? Because if it really is just for max_sane_readahead(), then for the love of God, let us just do this ... and bury this whole idiotic thread.",
"hate": 0.7097152222724888
},
{
"text": "What part of \"We don't break user space\" do you have trouble understanding? ... End of discussion. I don't understand why people have such a hard time understanding such a simple concept. ... Seriously, IT IS THAT SIMPLE.",
"hate": 0.7116556669807639
},
{
"text": "Absolutely not. I will not take this, and it's stupid in the extreme. ...That's just crazy talk. ... So I don't know how many ways I can say \"NO\", but I'll not take anythign like this. It's *completely* wrong.",
"hate": 0.716957130137101
},
{
"text": "That's a technical issue, Stefani. ... And when Fengguang's automatic bug tester found the problem, YOU STARTED ARGUING WITH HIM. Christ, well *excuuse* me for being fed up with this pointless discussion.",
"hate": 0.7183062080139838
},
{
"text": "Really. That's it. Claiming that that is \"complicated\" and needs a helper function is not something sane people do. A fifth-grader that isn't good at math can understand that. In contrast, nobody sane understands \"usub_overflow(a, b, &res)\". So really. Stop making inane arguments.",
"hate": 0.7193511040393217
},
{
"text": "What BS is that? If you use an \"atomic_store_explicit()\", by definition you're either (a) f*cking insane (b) not doing sequential non-synchronizing code ... and a compiler that assumes that the programmer is insane may actually be correct more often than not, but it's still a shit compiler. Agreed? So I don't see how any sane person can say that speculative writes are ok. They are clearly not ok. Speculative stores are a bad idea in general. They are completely invalid for anything that says \"atomic\". This is not even worth discussing.",
"hate": 0.7208914928670503
},
{
"text": "This is the kind of totally bogus crap that no sane person should ever spout. Stop it.",
"hate": 0.7228884292942104
},
{
"text": "And you're happy shilling for a broken model? ... Your arguments constantly seem to miss that rather big point. You think this is about bending over when MS whispers sweet nothings in your ear.. ... You, on the other hand, seem to have drunk the cool-aid on the whole \"let's control the user\" crap. Did you forget what security was all about?",
"hate": 0.724450098397838
},
{
"text": "No it's not. ... which is entirely and utterly pointless. Christ, the amount of confusion in that tree. ... Don't do this kind of thing. That branch is pointless, and just confused you.",
"hate": 0.7260907702568797
},
{
"text": "Ugh. I pulled it, but things like this makes me want to dig my eyes out with a spoon:...",
"hate": 0.7282438863837265
},
{
"text": "No, it really isn't. You still seem to be in denial: ... NO YOU DID NOT! Stop claiming that. You didn't actually test what you sent me. YOU TESTED SOMETHING ENTIRELY DIFFERENT. Do you really not see the difference? Because that's a honking big difference. ...",
"hate": 0.730275869385983
},
{
"text": "NOOO!... Get rid of the f*cking size checks etc on READ_ONCE() and friends. ... Hell f*cking no. The \"something like so\" is huge and utter crap, because the barrier is on the wrong side.",
"hate": 0.7311487138680152
},
{
"text": "Ok, I'm used to fixing up your whitespace and lack of capitalization, but you're getting so incoherent that I can no longer even parse it well enough to fix it up. English *is* your first language, right?",
"hate": 0.7338889133156115
},
{
"text": "Oh christ, I also cannot be bothered to continue arguing with you since you seemingly randomly drop other people from the discussion. So don't expect any more replies from me.",
"hate": 0.7345173280929542
},
{
"text": "Ingo, stop it already! This is *exactly* the kind of \"blame everybody else than yourself\" behavior that I was talking about earlier. ... Ingo, look your code in the mirror some day, and ask yourself: why do you think this fixes a \"regression\"? ...So by trying to make up for vsyscalls only in your numbers, you are basically trying to lie about regressions, and try to cover up the schednuma regression by fixing something else. ... See? That's bogus. When you now compare numbers, YOU ARE LYING. You have introduced a performance regression, and then trying to hide it by making something else go faster. ...The same is true of all your arguments about Mel's numbers wrt THP etc. Your arguments are misleading - either intentionally, of because you yourself didn't think things through. ...",
"hate": 0.7351130067196627
},
{
"text": "I never *ever* want to see this code ever again. Sorry, but last time was too f*cking painful.",
"hate": 0.7367643369256935
},
{
"text": "This looks totally invalid....Your patch is horribly wrong.",
"hate": 0.7401150258921102
},
{
"text": "No it's not. Please fix your crappy script. First off, that '#' is wrong. It should be a space.",
"hate": 0.7426792635047614
},
{
"text": "And no, we should *not* play games with \"tlb->local.next\". That just sounds completely and utterly insane. That's a hack, it's unclear, it's stupid, and it's connected to a totally irrelevant implementation detail, namely that random RCU freeing. Set a flag, for chrissake.",
"hate": 0.743261689308738
},
{
"text": "What the F*CK is wrong with people?",
"hate": 0.7451454396703296
},
{
"text": "...And a C++ person who says that \"(vodi *)0\" is just any \"void *\" is a *fucking*moron*, ...There is absolutely *zero* excuse for the idiotic traditional C++ brain damage of saying \"NULL cannot be '(void *)0', because 'void *' will warn\". Anybody who says that is lying. ... The C++ people? They are morons, and they never got it, and in fact they spent much of their limited mental effort arguing against it. ... The whole \"it's a stronger type system, so NULL has to be 0\" is pure and utter garbage. It's wrong. It's stupid. ... Yeah, I'm not a fan of C++. It's a cruddy language.",
"hate": 0.7454519550857
},
{
"text": "No, guys. That cannot work. It's a completely moronic idea.",
"hate": 0.7462408239078641
},
{
"text": "Yeah, it's a hack, and it's wrong, and we should figure out how to do it right.",
"hate": 0.7463157575296538
},
{
"text": "Grr. This is still bullshit. Doing this: ... is fundamentally crap ... So doing *any* of these calculations in bytes is pure and utter crap. ... Anything that works in bytes is simply pure crap. And don't talk to me about 64-bit math and doing it in \"u64\" or \"loff_t\", that's just utterly moronic too. ... So the math is confused, the types are confused, and the naming is confused. Please, somebody check this out, because now *I* am confused. And btw, that whole commit happened too f*cking late too. ... I'm grumpy, because all of this code is UTTER SH*T, and it was sent to me. Why?",
"hate": 0.7469102462898389
},
{
"text": "My point is that I have sixteen pointless messages in my mbox, half of which are due to just your argumentative nature.",
"hate": 0.7469964835334935
},
{
"text": "So this is definitely crap. You can't return an error. ... Same deal. Returning an error is wrong.",
"hate": 0.74723713946293
},
{
"text": "This seems to be just pure stupid. ...Even the help message is pure and utter garbage ... Asking the user questions that make no f*cking sense to ask is stupid. And I'm not knowingly pulling stupid crap.",
"hate": 0.7481964958616176
},
{
"text": "And if you cannot understand what tens of people have tried to explain to you, you are just f*cking stupid.",
"hate": 0.7488398264300431
},
{
"text": "They may be readable, but they are total shit. ... So no. Hell no.",
"hate": 0.7490401448416896
},
{
"text": "And this is just insanity. The \"barrier()\" cannot *possibly* do anything sane. If it really makes a difference, there is again some serious problem with the whole f*cking thing. NAK on the patch until sanity is restored. This is just total voodoo programming.",
"hate": 0.7493741531193527
},
{
"text": "This is why I don't like it when I see Torvald talk about \"proving\" things. It's bullshit.",
"hate": 0.7511155197081347
},
{
"text": "I also call bullshit on your \"it will surely be fixed when we know what's the right fix\" excuses. The fact is, you've spent the last several months blaming everybody but yourself, and actively told people to stop blaming you: ... despite having clearly seen the patch (you *replied* to it, for chissake, and I even told you in that same thread why that reply was wrong at the time). ... Kay, you are so full of sh*t that it's not funny. You're refusing to acknowledge your bugs, you refuse to fix them even when a patch is sent to you, and then you make excuses for the fact that we have to work around *your* bugs, and say that we should have done so from the very beginning. Yes, doing it in the kernel is \"more robust\". But don't play games, and stop the lying.",
"hate": 0.7526992070200629
},
{
"text": "Rik, *LOOK* at the code like I asked you to, instead of making excuses for it. I'm not necessarily arguing with what the code tries to do. I'm arguing with the fact that the code is pure and utter *garbage*. It has two major (and I mean *MAJOR*) problems, both of which individually should make you ashamed for ever posting that piece of shit: The obvious-without-even-understanding-semantics problem: - it's humongously stupidly written. It calculates that 'flush_remote' flag WHETHER IT GETS USED OR NOT. Christ. I can kind of expect stuff like that in driver code etc, but in VM routines? Yes, the compiler may be smart enough to actually fix up the idiocy. That doesn't make it less stupid. The more-subtle-but-fundamental-problem: - regardless of how stupidly written it is on a very superficial level, it's even more stupid in a much more fundamental way. ... In other words, everything that was added by that patch is PURE AND UTTER SHIT. And THAT is what I'm objecting to. ... Because everything I see about \"flush_remote\" looks just wrong, wrong, wrong. And if there really is some reason for that whole flush_remote braindamage, then we have much bigger problems ... So that patch should be burned, and possibly used as an example of horribly crappy code for later generations. At no point should it be applied.",
"hate": 0.7537096548107397
},
{
"text": "Umm. That just smells like BS to me. ... Also, your protection claim seems to be invalidated by the actual code. ... So your claim that it hedges around it by looking at the inquiry data is pure crap. It's simply not true. ... So no, I simply don't see the careful testing or the checking that you claim exists.",
"hate": 0.7537251528715909
},
{
"text": "Ugh: ... Can we please move that abortion into arch/powerpc/? Instead of making generic code even uglier..",
"hate": 0.7552048443254205
},
{
"text": "Why do you think it's not acceptable? Why do you raise a stink *one* day after the patch - that seems to not be very important - is sent out?... I don't think the patch is necessarily wrong, but I don't see why it would be critical, and I *definitely* don't see why the f*ck you are making a big deal of it. Go away, stop bothering people.",
"hate": 0.7560838366009982
},
{
"text": "Tejun, absolutely nothing \"justifies\" things if they break. ...And if nothing breaks, you don't need the excuses. In other words, I'll happily pull this, but your excuses for it are wrong-headed. There is no \"crazyness justifies this\". That's crap. ... None of this \"the interface is crazy, so we can change it\". Because that is pure and utter BS. Whether the interface is crazy or not is *entirely* irrelevant to whether it can be changed or not. The only thing that matters is whether people actually _trigger_ the issue you have in reality, not whether the issue is crazy.",
"hate": 0.7562240955547168
},
{
"text": "You're a fucking moron. ... So just reverting it from stable, *WITHOUT LEARNING THE LESSON*, is not a no-op at all, it's a sign of being a f*cking moron. I'm stupider for just reading your email. Go away.",
"hate": 0.7573115381946609
},
{
"text": ".. so your whole argument is bogus, because it doesn't actually fix anything else.... You're not fixing the problem, you're fixing one unimportant detail that isn't worth fixing that way.",
"hate": 0.7580508860722449
},
{
"text": "NAK NAK NAK. Ingo, please don't take any of these patches if they are starting to make NUMA scheduling be some arch-specific crap. Peter - you're way off base. You are totally and utterly wrong for several reasons: ...so making some arch-specific NUMA crap is *idiotic*. Your argument ...is pure crap. ... Christ, people. ...is just f*cking moronic. ... Stop the idiocy. ...",
"hate": 0.7613214610690521
},
{
"text": "That's absolutely insane. It's insane for two reasons: - it's not SUFFICIENT. ... - it's POINTLESS and WRONG. ...",
"hate": 0.7622273445742606
},
{
"text": "Ugh. Ok, so that's a disgusting hack, but it's better than messing up the generic PCI subsystem. At least it's a disgusting hack in the IOV code.",
"hate": 0.7625617442861903
},
{
"text": "The softirq semantics are perfectly fine. Don't blame softirq for the fact that irq_exit() has had shit-for-brains for a long time. ... Don't blame the wrong code here.",
"hate": 0.764044129341103
},
{
"text": "Bullshit. That's exactly the wrong kind of thinking. ... This whole discussion has been f*cking moronic. The \"security\" arguments have been utter shite with clearly no thinking behind it, the feature is total crap ... and I'm seriously considering just getting rid of this idiotic dmesg_restrict thing entirely. Your comment is the very epitome of bad security thinking.",
"hate": 0.7643113407762194
},
{
"text": "Finally, people - your merge messages suck. Leaving the list of conflicting files without talking about what the conflict actually was is *not* a good merge message. Len, you're not the only one that does this, but it is yet another reason why I absolutely detest some of the merges I see: they are just very uninformative, and don't add anything useful to the tree - quite the reverse. They hide a problem, without actually talking about what the problem was. ...And yes, sometimes my merge messages suck too (although I've tried very hard to become better at them).",
"hate": 0.7644553813120065
},
{
"text": "Ugh, looking more at the patch, I'm getting more and more convinces that it is pure and utter garbage. ... WTF? ... This is crap, guys. Seriously. Stop playing russian rulette with this code.",
"hate": 0.7664637795638153
},
{
"text": "Guys, this is not a dick-sucking contest. ... If Red Hat wants to deep-throat Microsoft, that's *your* issue.",
"hate": 0.7680006651105272
},
{
"text": "sizeof without parenthesis is an abomination, and should never be used. ... The sane solution is: just add the f*cking parenthesis, and don't use the parsing oddity. ... And talking about \"prefix operators\" is a moronic thing to do. ... Think of it as a function, and get over your idiotic pissing match over how long you've both known C. That's irrelevant. ...",
"hate": 0.7684528654812404
},
{
"text": "Ugh. This patch makes my eyes bleed. ... So I guess this patch fixes things, but it does make me go \"That's really *really* ugly\".",
"hate": 0.7714453966473782
},
{
"text": "No. I think it makes sense to put a big warning on any users you find, and fart in the general direction of any developer who did that broken pattern. Because code like that is obviously crap.",
"hate": 0.7720327748326408
},
{
"text": "We've been here before, haven't we? There's so much wrong with this that it's not funny. ... And I know you can do them, because you've done them in the past. So please don't continue to do the above.",
"hate": 0.7725198031833707
},
{
"text": "So adding the loop looks like just voodoo programming, not actually fixing anything.",
"hate": 0.7767215408126428
},
{
"text": "... Christ, people. Learn C, instead of just stringing random characters together until it compiles (with warnings). ... There are arguments for them, but they are from weak minds. ... A later patch then added onto the pile of manure by adding *another* broken array argument, ...It's basically just lying about what is going on, and the only thing it documents is \"I don't know how to C\". ... Please people. ...",
"hate": 0.777722536063855
},
{
"text": "... However, please don't use the *INSANE* ARM \"v8\" naming. There must be something in the water in Cambridge, but the ARM \"version\" naming is just totally broken. ...maybe it all makes sense if you have drunk the ARM cool-aid and have joined the ARM cult, but to sane people and outsiders, it is just a f*cking mess. So - aarch64 is just another druggie name that the ARM people came up with after drinking too much of the spiked water in cambridge. ... - armv8 is totally idiotic and incomprehensible to anybody else...complete and utter nonsense. It's only confusing. Christ. Seriously. The insanity is so strong in the ARM version names that it burns. If it really makes sense to anybody ... you need to have your head examined. So please don't use \"aarch64\", because it's just f*cking crazy. And please don't use \"armv8\", because it's just completely retarded and confused.",
"hate": 0.7811281591516777
},
{
"text": "Stop right there. ... This is about your patch BREAKING EXISTING BINARIES. So stop the f*&^ing around already. The patch was shown to be broken, stop making excuses, and stop blathering. End of story. Binary compatibility is more important than *any* of your patches. If you continue to argue anything else or making excuses, I'm going to ask people to just ignore your patches entirely. Seriously. ... Dammit, I'm continually surprised by the *idiots* out there that don't understand that binary compatibility is one of the absolute top priorities. ... Breaking existing binaries ... is just about the *worst* offense any kernel developer can do. Because that shows that they don't understand what the whole *point* of the kernel was after all. We're not masturbating around with some research project. We never were. Even when Linux was young, the whole and only point was to make a *usable* system. It's why it's not some crazy drug-induced microkernel or other random crazy thing. Really.",
"hate": 0.7825150243941044
},
{
"text": "Whee. Third time is the charm. I didn't know my email address was *that* hard to type in correctly.Usually it's the \"torvalds\" that trips people up, but you had some issues with \"foundation\", didn't you ;)",
"hate": 0.7830813361037369
},
{
"text": "Stop this idiocy. ... The fact that you then continually try to make this a \"kernel issue\" is just disgusting. Talking about the kernel solving it \"properly\" is pretty dishonest, when the kernel wasn't the problem in the first place.",
"hate": 0.7841595442902811
},
{
"text": "...which at least isn't butt ugly. ... Who is in charge of the whole 'is_virtfn' mess? How realistic is it to fix that crud?",
"hate": 0.7865562146315987
},
{
"text": "Sorry, you're wrong. And Rafael *told* you why you are wrong, and you ignored him and talked about \"exec\" some more. So go back and read Rafael's email. ... So please, read the emails. People actually *agree* that the name may be a bit misleading, but instead of harping on bogus issues, just read the emails from people like Rafael about it. So STOP with this idiocy already. ... Seriously. Get that through your head already.",
"hate": 0.7873434558648165
},
{
"text": "Your original email used \"[email protected]\", which goes into a kind-of-black hole. Please fix whatever crazy-old address book you have - that address is old, old, old. Oh, and your *new* email had totally broken email headers too. WTF? That ... is just pure and utter garbage. What the hell is wrong with your email setup? When I reply to that email, I sure as hell want to reply to *you*, not to *me*. So fix your email, right now it's terminally broken. Will look at the pull requests now that I actually see them, and when I'm over being upset by your idiotic email issues.",
"hate": 0.7895454699421509
},
{
"text": "Really. Stop this idiocy. We have gone through this before. It's a disaster.",
"hate": 0.7899533604712485
},
{
"text": "I think that code is bad, and you should feel bad.",
"hate": 0.7907568143335979
},
{
"text": "I really get the feeling that somebody needs to go over this patch-series with a fine comb to fix these kinds of ugly things",
"hate": 0.7918722805993774
},
{
"text": "Why? This change looks completely and utterly bogus.... Guys, this is crap. ... That's utter bullshit, guys. ...Exposing it at all is a disgrace. making it \"default y\" is doubly so. ... I'm not pulling crap like this. Get your act together. Why the heck should _I_ be the one that notices that this commit is insane and stupid? Yes, this is a pet peeve of mine. ... This cavalier attitude about asking people idiotic questions MUST STOP. Seriously. This is not some \"small harmless bug\". This mindset of crazy questions is a major issue!",
"hate": 0.796587186134485
},
{
"text": "I can't even begin to say whether this is a good solution or not, because that if-conditional makes me want to go out and kill some homeless people to let my aggressions out. Can we please agree to *never* write code like this? Ever?",
"hate": 0.7975509400529777
},
{
"text": "Mark, I pulled this, but I was *this* close to unpulling it because it's such an unholy mess. You seem to do the crazy \"daily pull\" crap that NOBODY IS EVER SUPPOSED TO DO. There are lots of totally pointless ... merge commits, and no, that's not ok.... Just don't do it. There's no excuse. The *only* time you should merge is when a sub lieutenant asks you to - and if you have people who work with you and ask you to do pointless merges almost every day, just tell them to shut the f*ck up already!...But dammit, don't then do development on top of that kind of crap - use that branch *only* for linux-next, not for anything else, and don't ask me to pull that polluted piece of sh*t. ...But never *ever* have those stupid pointless \"Merge remote-tracking branch 'regulator/for-linus' into regulator-next\" in the branch you actually use for development, and the branch you send to me.",
"hate": 0.7987151585936965
},
{
"text": "... I don't understand why this seems to be so hard for people to understand. ...this whole thread is a wonderful example of how F*CKING STUPID it was to even consider it. ... SO STOP DOING ABI CHANGES. WE DON'T DO THEM. ...but anybody who does it on purpose \"just because\" should not be involved in kernel development (or library development for that matter).",
"hate": 0.8008857942638532
},
{
"text": "Why? You're wrong. I mean, anybody who disagrees with me is pretty much wrong just on pure principles, but I actually mean a deeper kind of wrong than that. I mean a very objective \"you're clearly wrong\". ... .. and then you use a totally bogus example to try to \"prove\" your point. ... Your example is pure and utter shit, since you still get confused about what is actually const and what isn't. ... But that argument is BULLSHIT, because the fact is, the function *doesn't* do what you try to claim it does.",
"hate": 0.8011116731689472
},
{
"text": "That's insane. ... It is simply not sensible to have a \"wait_for_unlock()\" that then synchronizes loads or stores that happened *before* the wait. That's some crazy voodoo programming. ... Or just take the damn lock, and don't play any games at all.",
"hate": 0.8014264696501292
},
{
"text": "*NEITHER* of your points actually address my issue. ... IOW, why the hell do you set a name that is so useful that no sane person would ever want to use it? ... So let me be more blunt and direct: which part of \"that's f*cking stupid\" do you disagree with? So instead of making drivers do stupid things because you've made the input layer names so damn useless, why don't you make the input layer use better naming? Doesn't that seem to be the *smart* thing to do?",
"hate": 0.801731111695407
},
{
"text": "This was obviously brought on by my frustration with the currently nasty do_notify_resume() always returning to iret for the task_work case, and PeterZ's patch that fixed that, but made the asm mess even *worse*.",
"hate": 0.8017644563947721
},
{
"text": "Stop this idiocy. ... And that disgusting \"overflow_usub()\" in no way makes the code more readable. EVER. So stop just making things up.... It wasn't more efficient, it wasn't more legible, and it simply had no excuse for it. Stop making excuses for shit.",
"hate": 0.8022031025193204
},
{
"text": "Basically, I absolutely hate the notion of us doing something unsynchronized, when I can see us undoing a mmap that another thread is doing. It's wrong. You also didn't react to all the *other* things that were wrong in that patch-set. The games you play with !fatal_signal_pending() etc are just crazy. End result: I absolutely detest the whole thing. I told you what I consider an acceptable solution instead, that is much simpler and doesn't have any of the problems of your patchset.",
"hate": 0.8026464596433878
},
{
"text": "Hmm. Less vomit-inducing, except for this part:...Ugh, that just *screams* for a helper function.",
"hate": 0.802683443638446
},
{
"text": "... And it really is all stupidly and badly done. I bet we can make that code faster without really changing the end result at all, just changing the algorithm. ... In fact, looking at select_idle_sibling(), I just want to puke. The thing is crazy. ... instead it does totally f*cking insane things ... The code is shit. Just fix the shit, instead of trying to come up with some totally different model. Ok? ...",
"hate": 0.8038211248696474
},
{
"text": "Duh. That is just broken. ... That's just stupid.",
"hate": 0.8049355034388979
},
{
"text": "No. I'm not pulling 3000 lines of broken defconfig crap. That defconfig update is pure and utter shit, and changes from the minimal kind of defconfig to the old-style \"everything and the kitchen sink\" crap. We started doing that defconfig minimization for a reason. See commit 8b1bb90701f9 for example. We're not going back to the bad old days, and I'm not pulling thousands of lines or pure garbage noise just because you were too lazy to do it right.",
"hate": 0.806512451074602
},
{
"text": "No it's not. Thomas, stop this crap already. Look at the f*cking code carefully instead of just dismissing cases. ... So, Christ, Thomas, you have now *twice* dismissed a real concern with totally bogus \"that can never happen\" by explaining some totally unrelated *simple* case rather than the much more complex case. So please. Really. Truly look at the code and thing about it, or shut the f*ck up. No more of this shit where you glance at the code, find some simple case, and say \"that can't happen\", and dismiss the bug-report.",
"hate": 0.8082092019063171
},
{
"text": "This patch is insane. This is pure garbage. Anybody who thinks this: ... is a good idea should be shot. Don't do it. ...That's just f*cking insane. Stop this kind of idiocy. The code looks bad, and the logic is pure shit too.",
"hate": 0.8089388655340428
},
{
"text": "Ok, so I'm looking at the code generation and your compiler is pure and utter *shit*. ... Lookie here, your compiler does some absolutely insane things with the spilling, including spilling a *constant*. For chrissake, that compiler shouldn't have been allowed to graduate from kindergarten. We're talking \"sloth that was dropped on the head as a baby\" level retardation levels here: ... Because it damn well is some seriously crazy shit. However, that constant spilling part just counts as \"too stupid to live\". ... ... This is your compiler creating completely broken code.",
"hate": 0.8093202085297696
},
{
"text": "No it didn't. There was nothing accidental about it, and it doesn't even change it the way you claim.... Your explanation makes no sense for _another_ reason.... ... So tell us more about those actual problems, because your patch and explanation is clearly wrong. ... So this whole thing makes no sense what-so-ever.",
"hate": 0.8102418937082152
},
{
"text": "Stop the idiotic arguing already.",
"hate": 0.810709046585318
},
{
"text": "It's misleading crap. Really. Just do a quick grep for that bit, and you see just *how* confused people are about it:...think about it. Just *THINK* about how broken that code is. The whole thing is a disaster. _PAGE_NUMA must die. It's shit.",
"hate": 0.8113774269329408
},
{
"text": "Not pulled, because your hamster smells of eldeberries. This is not just bugfixes. In fact, as far as I can tell, this *introduces* bugs, ... I'm f*cking tired of people having problems understanding \"we're past rc5\". If it's not something you would call stable material, you shouldn't send it to me.",
"hate": 0.8116823499648296
},
{
"text": "Side note, you'll obviously also need to fix the actual bogus 'gp_init_delay' use in kernel/rcu/tree.c. That code is horrible.",
"hate": 0.8127415613736015
},
{
"text": "Kees, you don't seem to understand. Breaking applications is unacceptable. End of story. It's broken them. Get over it. ... Your \"IT HAS TO BE DONE AT BOOT TIME, THE SKY IS FALLING, NOTHING ELSE IS ACCEPTABLE!\" ranting is a disease. Stop it.",
"hate": 0.8129376023678443
},
{
"text": "So please, just remove that idiotic \"if (!event->attr.exclude_guest)\" test. It's wrong. It cannot possibly do the right thing. It is totally misdesigned, exactly because you don't even know beforehand if somebody uses virtualization or not.",
"hate": 0.8129752569843693
},
{
"text": "No. I pulled, and immediately unpulled again. This is complete shit, and the compiler even tells you so: ... I'm not taking \"cleanups\" like this. And I certainly don't appreciate being sent completely bogus shit pull requests at the end of the merge cycle.",
"hate": 0.8132106003206262
},
{
"text": "What the F*CK, guys? This piece-of-shit commit is marked for stable, but you clearly never even test-compiled it, did you? ...The declaration for gate_desc is very very different for 32-bit and 64-bit x86 for whatever braindamaged reasons. Seriously, WTF? I made the mistake of doing multiple merges back-to-back with the intention of not doing a full allmodconfig build in between them, and now I have to undo them all because this pull request was full of unbelievable shit. And why the hell was this marked for stable even *IF* it hadn't been complete and utter tripe? It even has a comment in the commit message about how this probably doesn't matter. So it's doubly crap: it's *wrong*, and it didn't actually fix anything to begin with. There aren't enough swear-words in the English language, so now I'll have to call you perkeleen vittup\u00e4\u00e4 just to express my disgust and frustration with this crap.",
"hate": 0.8139540249123025
},
{
"text": "This looks completely broken to me. ... Wtf? Am I missing something?",
"hate": 0.8140783660458883
},
{
"text": ".. and apparently this whole paragraph is completely bogus. It *does* break things, and for normal people. That's what the bug report is all about. So don't waffle about it.... Wrong. We don't break existing setups, and your attitude needs fixing. ... The problem needs to be solved some other way, and developers need to f*cking stop with the \"we can break peoples setups\" mentality./ Hans, seriously. You have the wrong mental model. Fix it.",
"hate": 0.8148479940696665
},
{
"text": "Yeah, that would be a no. I finally got to look at the new architectures and be ready to pull them, and you just made sure I won't pull this. This is exactly the kind of crap I don't want to see in *any* pull requests, ... Why the f*ck are you doing back-merges? There is no excuse for even a single one. And here you have just about one back-merge per commit. No, no no.",
"hate": 0.814898612788688
},
{
"text": "No. Your repository is bogus. I don't know what the hell you have done or why you have done it, but you have actually rebased *my* 4-3-rc7 commit that updates the Makefile from rc6 to rc7... and there is no way I will take things like this.",
"hate": 0.8157836516258494
},
{
"text": "No. Your pull requests are just illogical. I have yet to see a single reason why it should be merged. ... That's total bullshit. ... Again, total *bullshit*. ... ... Ingo, it's not us being silly, it is *you*. ... So here, let me state it very very clearly: I will not be merging kvmtool. ... In other words, I don't see *any* advantage to merging kvmtool. I think merging it would be an active mistake, and would just tie two projects together that just shouldn't be tied together. So nobody is \"hurting useful code\", except perhaps you. Explain to me why I'm wrong. I don't think you can. You certainly haven't so far.",
"hate": 0.8163050498769295
},
{
"text": "Yes, I'm upset. Very upset. ... So your question \"why would pulseaudio care\" is totally *irrelevant*, senseless, and has nothing to do with anything.",
"hate": 0.8163674033539857
},
{
"text": "I absolutely detest these types. I realize that we already have a few users, but just looking at these diffs *hurts*. It's disgusting.",
"hate": 0.8197295535360578
},
{
"text": "Ugh. I dislike this RCU'ism. It's bad code. It doesn't just look ugly and complex, it's also not even clever. It is possible that the compiler can fix up this horrible stuff and turn it into the nice clever stuff, but I dunno.",
"hate": 0.8198414293995426
},
{
"text": "Grr. I hate it when people do this. Your merge message sucks.",
"hate": 0.8207332940990693
},
{
"text": "Christ. This one is too ugly to live. I'm not pulling crap like this. It's f*^&ing stupid to take a lock, calculate a bitqueue, and just generally be an absolute ass-hat about things for waiting for a bit that is already set 99.999% of the time.",
"hate": 0.8215065507011629
},
{
"text": "Please don't. That thing is too ugly to exist. It also looks completely and utterly buggy. There's no way I'm taking it. If switch-names is suddenly conditional, what the f*ck happens to the name hash which is unconditionally done with a swap() right afterwards. There's no way that patch is correct",
"hate": 0.8215622542634073
},
{
"text": "The fact that it doesn't even compile makes me doubt your statement that it has been in linux-next. ... I fixed it up properly in the merge, but please try to figure out how the hell this passed through the cracks.",
"hate": 0.8216988903895655
},
{
"text": "Umm. We had oopses showing it. Several times. ... .. and you and Andi repeatedly refused to make the code more robust when I asked. Which is why I don't work with Andi or you directly any more, ... Every time there were just excuses. Like now. ... I'm done with your crazy unwinder games. ... But this patch I NAK'ed because the code is not readable, and the infrastructure is not bearable. Live with it.",
"hate": 0.8239366893463678
},
{
"text": "Oww, oww, oww. DAMMIT. ...So I'm pissed off. This patch was clearly never tested anywhere. Why was it sent to me?...Grr. Consider yourself cursed at. Saatana.",
"hate": 0.8248763577834479
},
{
"text": "Please don't do these ugly and pointless preprocessor macro expanders that hide what the actual operation is. And this is really ugly. Again it's also then hidden behind the ugly macro. ...",
"hate": 0.824927722167148
},
{
"text": "Christ guys. This whole thread is retarded. The *ONLY* reason people seem to have for reverting that is a \"ooh, my feelings are hurt by how this was done, and now I'm a pissy bitch and I want to get this reverted\". Stop the f*cking around already. ... And if your little feelings got hurt, get your mommy to tuck you in, don't email me about it. Because I'm not exactly known for my deep emotional understanding and supportive personality, am I?",
"hate": 0.8283600499747166
},
{
"text": "Hell no. Stop with the random BUG_ON() additions. ... Dammit, there's no reason to add a BUG_ON() here in the first place, and the reason of \"but but it's an unused error return\": is f*cking retarded. Stop this idiocy. We don't write crap code just to satisfy some random coding standard or shut up a compiler error.... NO NO NO. ... Really. I'm getting very tired indeed of people adding BUG_ON's like that. Stop it.",
"hate": 0.828849409027625
},
{
"text": "Guys, stop it now. Your \"problem\" isn't what any sane person cares about, and isn't what I started the RFC for. Seriously. NOBODY CARES. ... Stop whining.",
"hate": 0.8299087057272497
},
{
"text": "Ugh. I absolutely detest this patch. If we're going to leave the TLB dirty, then dammit, leave it dirty. Don't play some half-way games....",
"hate": 0.8314109841244047
},
{
"text": "No they aren't. You think they are, and then you find one case, and ignore all the others. ... So no, your patch doesn't change the fundamental issue in any way, shape, or form. I asked you to stop emailing me with these broken \"let's fix one special case, because I can't be bothered to understand the big picture\" patches. This was another one of those hacky sh*t patches that doesn't actually change the deeper issue. Stop it. Seriously. This idiotic thread has been going on for too long.",
"hate": 0.8323267749129
},
{
"text": "Stop this \"we can break stuff\" crap. Who maintains udev? Regressions are not acceptable. I'm not going to change the kernel because udev broke, f*ck it. Seriously. More projects need to realize that regressions are totally and utterly unacceptable. ... That just encourages those package maintainers to be shit maintainers. ... And stop blaming the kernel for user space breakage!...",
"hate": 0.8326730446019563
},
{
"text": "You are the one who seems to just want to add hack upon hack to things. THAT is what I really hate. It's not only in bad taste, it *will* come back and bite us some day.",
"hate": 0.8348503697757049
},
{
"text": "But your (and the current C standards) attempt to define this with some kind of syntactic dependency carrying chain will _inevitably_ get this wrong, and/or be too horribly complex to actually be useful. Seriously, don't do it. ... So just give it up. It's a fundamentally broken model. It's *wrong*, but even more importantly, it's not even *useful*, ...I really really really think you need to do this at a higher conceptual level, get away from all these idiotic \"these operations maintain the chain\" crap.",
"hate": 0.8349956577323447
},
{
"text": "No. You guys need to realize that I'm not talking crap like this this late. This is not major bugfixes. I already looked away once just because it's a new filesystem, but enough is enough. This is way way WAY too late to start sendign \"enhancements\". Seriously.",
"hate": 0.8350117641043012
},
{
"text": ".. why did that commit ever even get far enough to get to me? ... Either way, it shows a rather distinct lack of actual testing, wouldn't you say? I really see no excuse for crap like this. ...Linus \"not happy\" Torvalds",
"hate": 0.838218598944263
},
{
"text": "I would love to blame gcc, but no, I think the code is crap. ... And gcc would be completely correct. That test is moronic.",
"hate": 0.8385927406257188
},
{
"text": "Grr. Most of these patches have the same stupid problem: why the *hell* do you repeat the single-line top-level description in both the Subject line and the body of the email? It only results in stupid duplicate lines in the commit logs. This is a disease. I don't know who the heck started doing it, but it's WRONG. It's stupid. What broken piece-of-shit tool is it that does this braindamage? Fix it. Stop sending these broken commit messages to people. I'm grumpy, yes, because this is a common problem. I see it all over the place, and it makes our commit logs look f*cking retarded. ...",
"hate": 0.8389756777018744
},
{
"text": "If this comes from some man-page, then the man-page is just full of sh*t, and is being crazy. ...So NAK NAK NAK. This is insane and completely wrong. And the bugzilla is crazy too. Why would anybody think that readahead() is the same as read()?",
"hate": 0.8397813542139798
},
{
"text": "Ingo, stop doing this kind of crap. ... You seem to be in total denial. ... Stop it. That kind of \"head-in-the-sand\" behavior is not conducive to good code, ... Seriously. If you can't make the non-THP case go faster, don't even bother sending out the patches. Similarly, if you cannot take performance results from others, don't even bother sending out the patches. ... So stop ignoring the feedback, and stop shooting the messenger. Look at the numbers, and admit that there is something that needs to be fixed.",
"hate": 0.8398371907439319
},
{
"text": "This is disgusting. Many (most?) __gup_fast() users just want a single page, and the stupid overhead of the multi-page version is already unnecessary. This just makes things much worse.",
"hate": 0.8401642774498181
},
{
"text": "So I absolutely *hate* how this was done.... I'm pulling it this time, but quite frankly, next time I see this kind of ugly AND TOTALLY POINTLESS layering violation, I will just drop the stupid pull request. ... In other words, this was NOT OK. This was stupid and wrong, and violated all sanity. ... What the hell was going on here?",
"hate": 0.8407231992027466
},
{
"text": "This has so much wrong that I don't know where to start.",
"hate": 0.8417064101827453
},
{
"text": "NO I AM NOT! Dammit, this feature is f*cking brain-damaged. ...But even apart from the Xen case, it was just a confusing hell. Like Yoda said: \"Either they are the same or they are not. There is no 'try'\". So pick one solution. Don't try to pick the mixed-up half-way case that is a disaster and makes no sense.",
"hate": 0.8424148553127997
},
{
"text": "So I have to say, I hate this entire series. ... So quite frankly, I think the whole series is total and utter garbage. And there isn't even any *explanation\" for the garbage. You say that you are unifying things, but dammit, in order to unify them you end up *adding*new*f&^#ing*code*. ... So a honking big NAK on this whole series unless you can explain with numbers and show with numbers what the advantage of the abortion is.",
"hate": 0.8425100124780881
},
{
"text": "Dammit, this is pure shit, and after having to deal with yet another pointless merge conflict due to stupid \"cleanups\" in Makefiles, IT DOES NOT EVEN COMPILE. And no, that's not due to a merge error of mine. It was that way in your tree. Hulk angry. Hulk smash. I fixed it up in the merge, but I shouldn't need to. This should have been caught in -next, and even if you compile for ARM as your primary target, I know *damn* well that no sane ARM developer actually compiles *on* ARM (because there are no machines where it's worth the pain), so you should make sure that the x86-64 build works too. If I can find compile errors within a couple of minutes of pulling and it's not a merge error of mine, the tree I'm pulling from is clearly crap. So I'm more than a bit grumpy. Get your act together, and don't send me any more shit. In fact, I would suggest you send nothing but obvious fixes from now on in this release. Because I won't be taking anything else.",
"hate": 0.8434476248737879
},
{
"text": "You have also marked 3.18-rc1 bad *twice*, along with the network merge, and the tty merge. That's just odd. But it doesn't make the bisect wrong, it just means that you fat-fingered thing and marked the same thing bad a couple of times. Nothing to worry about, unless it's a sign of early Parkinsons...",
"hate": 0.8441764538366523
},
{
"text": "Rafael, please don't *ever* write that crap again. ... Seriously. Why do I even have to mention this? Why do I have to explain this to somebody pretty much *every* f*cking merge window? This is not a new rule. ... So you should be well acquainted with the rule, and I'm surprised to hear that kind of utter garbage from you in particular. ...",
"hate": 0.8443138084082528
},
{
"text": "Ugh, so I pulled this, but I'm going to unpull it, because I dislike your new \"i_mmap_lastmap\" field.... makes me just gouge my eyes out. It's not only uglifying generic code, it's _stupid_ even when it's used....But the fact that it adds code to the generic file just adds insult to injury and makes me go \"no, I don't want to pull this\".",
"hate": 0.844400180291415
},
{
"text": "No. Really. ... No. The whole concept of \"drop the lock in the middle\" is *BROKEN*. It's seriously crap. It's not just a bug, it's a really fundamentally wrong thing to do. ... No. That's still wrong. You can have two people holding a write-lock. Seriously. That's *shit*",
"hate": 0.8448858194517508
},
{
"text": "I did look at it, but the thing is horrible. I started on this something like ten times, and always ended up running away screaming.",
"hate": 0.8457555416068469
},
{
"text": "Ugh. That's horrid. Do we need to even support O_DIRECT in that case? ... In general, it's really a horrible thing to use, and tends to be a big red sign that \"somebody misdesigned this badly\"",
"hate": 0.8458575409888278
},
{
"text": "Why does this have the crappy cputime scaling overflow code, ... WTF happened here? I and others spent efforts so that we wouldn't need this kind of crap.",
"hate": 0.8474158356369889
},
{
"text": "Ugh. This is way late in the release, and the patch makes me go: \"This is completely insane\", which doesn't really help...This is just pure bullshit....So the above locking change is at least terminally stupid, and at most a sign of something much much worse.... there is no way in hell I will apply this obviously crap patch this late in the game. Because this patch is just inexcusable crap, and it should *not* have been sent to me in this state. ...",
"hate": 0.8475229805962871
},
{
"text": "Ugh. I hate that. It looks bad, but it's also pointless. ...Compilers that warn for the good kind of safe range tests should be taken out and shot. ... so I just detest that buggy piece of crap for *so* many reasons. It's also sad that a one-liner commit that claims to \"fix\" something was this broken to begin with. Grr. Honza, not good.",
"hate": 0.8480862125699891
},
{
"text": "Fuck no. ... You are just making shit up. Bad shit. Get off the drugs, because it's not the good kind.... Cry me a river. ... Bullshit. This whole thread is now marked as \"muted\" for me, because I can't take the BS any more. You make no sense. ...You're crazy. Go away. Or don't. I won't be seeing your emails anyway, so why would I care?",
"hate": 0.8481493300006194
},
{
"text": "Your arguments make no sense. ... NO IT DOES NOT. Christ, Paul. ... You have turned it into something else in your mind. But your mind is WRONG. ... I really don't understand your logic. ... That is NOT WHAT I WANT AT ALL.",
"hate": 0.8502581437028554
},
{
"text": "Because code like this is just crap: ... really. It's just crazy. It makes no sense. It's all just avoiding the warning, it's not making the code any better.",
"hate": 0.8506653143731778
},
{
"text": "Christ people. I already reported that it DOES NOT EVEN COMPILE. ... Alan apparently doesn't care about the patch he wrote to even bother fixing that, and the only person who does seem to care enough to carry two fixes around (Andrew) apparently doesn't feel that he's comfortable forwarding it to me ... I'm not picking up random patches from people who don't care enough about those patches to even bother fixing compile errors when reportyed and didn't even send them to me to begin with.",
"hate": 0.8521833259539588
},
{
"text": "The whole \"it's more convenient to use sleeping locks\" argument is PURE AND UTTER SHIT when it comes to really core code. ... Seriously. Your argument is bad, but more importantly, it is *dangerously* bad. It's crap that results in bad code: and the bad code is almost impossible to fix up later...",
"hate": 0.8521992066383708
},
{
"text": "No. You think *WRONG*. ... YOUR CODE IS WRONG, AND REALITY SHOWS THAT YOUR DEFAULT IS CRAP. Really. ... BS. The only reason for your interface was that it was simpler to use. You broke that. And you broke that for no good reason. .... So your \"default\" is not actually safe. It breaks real cases, and doesn't add any security. It's broken.",
"hate": 0.8532715024553228
},
{
"text": "...it's a pointless and wrong example....So your argument is *shit*. Why do you continue to argue it?...It's really not that complicated....Really, why is so hard to understand?",
"hate": 0.8534513412321592
},
{
"text": "You're doing completelt faulty math, and you haven't thought it through. ...That's *insane*. It's crap. All just to try to avoid one page copy. Don't do it. ...Really, you need to rethink your whole \"zerocopy\" model. It's broken. Nobody sane cares. You've bought into a model that Sun already showed doesn't work. ...",
"hate": 0.8542957166678475
},
{
"text": "... Why has this been explicitly designed to be as ugly as humanly possible, and to be so inconvenient to parse too? ... So here's a serious suggestion: aim for \"line oriented\". ... ...That's stupid. Don't do it. ...Because this is just pure and utter *shit*: ...This part gets a big NAK from me. I don't see the point of this. It's pure crap. There's no advantage. And the \"use an uint64_t\" is a classic case of just being a f*cking moron. ...This is the kind of thinking and code that just makes me go \"No way, not today, not ever\". It's *stupid*.",
"hate": 0.8543501607268051
},
{
"text": "...End of discussion. Seriously. Your whinging about \"support costs\" is just crying over the fact that you have users. Deal with it. ...And dammit, I really never *ever* want to hear arguments against fixing regressions ever again. It really is the #1 rule for the kernel. There is *no* excuse for that NAK. There is only \"sorry\".",
"hate": 0.8547854359966223
},
{
"text": "It does become a problem when you have a system service developer who thinks the universe revolves around him, and nobody else matters, and people sending him bug-reports are annoyances that should be ignored rather than acknowledged and fixed. At that point, it's a problem. It looks like Greg has stepped in as a baby-sitter for Kay, and things are going to be fixed. And I'd really like to avoid adding hacky code to the kernel because of Kay's continued bad behavior, so I hope this works. But it's really sad that things like this get elevated to this kind of situation, and I personally find it annoying that it's always the same f*cking primadonna involved.",
"hate": 0.8554723792056538
},
{
"text": ".. And how the f*^& did you imagine that something like chrome would do that? You need massive amounts of privileges, and it's a total disaster in every single respect. Stop pushing crap. No, ptrace isn't wonderful, but your LSM+auditing idea is a billion times worse in all respects. ... THERE IS NO WAY IN HELL YOU CAN EVER FIX LSM+AUDIT TO BE USABLE! Stop bothering to even bring it up. It's dead, Jim.",
"hate": 0.8560065729226336
},
{
"text": "What the hell have you done with the commit messages? The first line is completely corrupted for those reverts, and as a result your own shortlog looks like crap and is completely misleading. ... presumably due to some horribly broken automation crap of yours that adds the \"[media]\" prefix or something. How did you not notice this when you sent the shortlog? Or even earlier? This is some serious sh*t, since it basically means that your log messages are very misleading, since the one-liner actually implies exactly the reverse of what the commit does. I unpulled this, because I think misleading commit messages is a serious problem, and basically *half* (and patch-wise, the bulk) of the commits in this queue are completely broken.",
"hate": 0.8560482082252309
},
{
"text": "That's a cop-out. ... See? It's stupid. It's wrong. It's *bad*.",
"hate": 0.8566992640799118
},
{
"text": "Not acceptable. ... Plase stop sending me untested crap that doesn't even compile cleanly!",
"hate": 0.8569026842796138
},
{
"text": "And you are making that excuse exactly *why*? ... Stop making excuses for bad behavior. Just admit that you guys screwed up rather than trying to soldier on. ... That's a f*cking disgrace. ... Stop making excuses for it. Really. It just makes you look even worse. ...",
"hate": 0.857176751784924
},
{
"text": "This code makes absolutely no sense.... So the code may end up *working*, but the comments in it are misleading, insane, and nonsensical. ...The comment is actively and entirely wrong. ... So the code looks insane to me. ...So in no case can that code make sense, as far as I can tell.",
"hate": 0.8577694243214468
},
{
"text": "This is all *COMPLETELY* wrong. ... Fix ARC, don't try to blame generic code. You should have asked yourself why only ARC saw this bug, when the code apparently works fine for everybody else!",
"hate": 0.8582253166579739
},
{
"text": "No way. ... In fact, just to prove how bad it is, YOU SCREWED IT UP YOURSELF. ... But the \"hacky workaround\" absolutely needs to be *automatic*. Because the \"driver writers need to get this subtle untestable thing right\" is *not* acceptable. That's the patch that Ming Lei did, and I refuse to have that kind of fragile crap in the kernel.",
"hate": 0.8589598049817351
},
{
"text": "Ugh. This patch is too ugly to live. ... I really detest debug code (or compiler warnings) that encourage people to write code that is *worse* than the code that causes the debug code or warning to trigger. It's fundamentally wrong when those \"fixes\" actually make the code less readable and maintainable in the long run.",
"hate": 0.859209414339482
},
{
"text": "Ugh, this is disgusting.",
"hate": 0.8596986686485475
},
{
"text": "Mauro, SHUT THE FUCK UP! It's a bug alright - in the kernel. How long have you been a maintainer? And you *still* haven't learnt the first rule of kernel maintenance? ... Shut up, Mauro. And I don't _ever_ want to hear that kind of obvious garbage and idiocy from a kernel maintainer again. Seriously. ...The fact that you then try to make *excuses* for breaking user space, and blaming some external program that *used* to work, is just shameful. It's not how we work. Fix your f*cking \"compliance tool\", because it is obviously broken. And fix your approach to kernel programming",
"hate": 0.8617790975244219
},
{
"text": "... I will not be pulling this tree at all. It's pure and utter shit, and I wonder how long (forever?) this has been going on. ...the thing that makes me go \"uhhuh, no way in *hell* should I pull this\" is that you have apparently totally broken all sign-offs. Avi, you ABSOLUTELY MUST NOT rebase other peoples commits. That's a total no-no. And one thing I notice when I look through the commits is that you have totally broken the Signed-off-by: series in the process, exactly because what you do is crap, crap, CRAP. ... That's simply not true in your tree. Maybe because you have rebased other peoples (Alexander's) commits? I see commits where the sign-off ends with Alexander, but then the committer is you. WTF? Fix your f*cking broken shit *now*. I'm not pulling crap like this. And it makes me unhappy to realize that this has probably happened a long time and I haven't even noticed. The whole \"you MUST NOT rebase other peoples commits\" is the thing I've been telling people for *years* now. Why the hell is it still going on?",
"hate": 0.8623463787609112
},
{
"text": "Yeah. Bloated, over-engineered, and stupid. ... Despite making the code slower, bigger, and buggier. I guess I'll fetch the git tree and see if they document this braindamage.. ...Oh well. What a cock-up. The code is insane in other ways too. ... I can't take it any more. That code is crazy.",
"hate": 0.8625105510296714
},
{
"text": "Ok, so things are somewhat calm, and I'm trying to take time off to see what's going on. And I'm not happy. ... Please don't call this thing a \"generic page table\". ... In other words, looking at this, I just go \"this is re-implementing existing models, and uses naming that is actively misleading\". I think it's actively horrible, in other words. ... I also find it absolutely disgusting how you use USE_SPLIT_PTE_PTLOCKS for this, which seems to make absolutely zero sense. ... I'm also looking at the \"locking\". It's insane. It's wrong, and doesn't have any serialization. ... Rik, the fact that you acked this just makes all your other ack's be suspect. Did you do it just because it was from Red Hat, or do you do it because you like seeing Acked-by's with your name? ...",
"hate": 0.862733574040647
},
{
"text": "But that's *BS*. You didn't actually listen to the main issue. Paul, why do you insist on this carries-a-dependency crap? It's broken. ... The \"carries a dependency\" model is broken. Get over it.... I gave an alternate model (the \"restrict\"), and you didn't seem to understand the really fundamental difference. ... So please stop arguing against that. Whenever you argue against that simple fact, you are arguing against sane compilers....",
"hate": 0.8630689004755419
},
{
"text": "Ugh. This is getting really ugly. ... because quite frankly, the whole spinunlock inlining logic is *already* unreadable, and you just made it worse.",
"hate": 0.8639408003469052
},
{
"text": "Whjat the f*ck is the logic there? Just fix the *obvious* breakage in BLKROSET. It's clearly what the code *intends* to do, it just didn't check for ENOIOCTLCMD. ... It's a DISEASE how you seem to think that \"we have ugly mistakes in the kernel, SO LET'S ADD ANOTHER ONE\". That's not how we do things. We *fix* things, instead of making things even *worse*. Stop this insanity from spreading, instead of making it spread more!",
"hate": 0.8645812553613907
},
{
"text": "No there isn't. Your \"action by the holder\" argument is pure and utter garbage, for a very simple and core reason: the *filesystem* doesn't know or care. ... ...Face it, your patch is broken. And it's *fundamentally* broken, which is why I'm so tired of your stupid ad-hoc hacks that cannot possibly work.",
"hate": 0.8647867861849572
},
{
"text": "Ugh. This is too ugly, it needs to die. ... Because this is unreadable.",
"hate": 0.8647894373012335