1. C++ / Говнокод #19882

    −3

    1. 1
    for(;;){new int;}

    Запостил: GC1, 26 Апреля 2016

    Комментарии (43) RSS

    • ava Xom94ok 26.04.2016 16:41 # +8

      ну так прибери, хули на говнокодике сидишь!
      Ответить

    • ava Dummy00001 26.04.2016 17:48 # +13

      вот, пофиксил проблему производительности:
      for(;;){new int[128];}
      Ответить

      • ava Soul_re@ver 26.04.2016 18:25 # +6

        Пофиксил ещё больше
        throw bad_alloc;


        (В конце концов адресное пространство кончится и в любом случае исключение вылетит)
        Ответить

        • ava Dummy00001 26.04.2016 18:50 # +6

          результаты теста: FAIL. потому что еще за долго до конца адресного пространства закончится физическая память и своп.

          ЗЫ я по тупости к слову ГК скомпилил и запустил... я не просто так проблему производительности фиксил. я эту проблему видел: оригинальному коду нужно секунд 5 что бы 8ГБ памяти съесть. поэтому и фикс.
          Ответить

          • ava alexinzaz 26.04.2016 23:13 # +6

            не силен в крестах, но разве сборщик мусора не будет следом подчищать не используемые переменные?
            Ответить

            • ava dxd 26.04.2016 23:14 # +5

              Так толсто, что тонко.
              Ответить

            • ava kurwa 26.04.2016 23:35 # +9

              Так и есть. Приходит оом-киллер и собирает мусор.
              Ответить

              • ava bormand 27.04.2016 06:19 # +5

                Правда, иногда он приходит не к тому... Но это мелочи.
                Ответить

            • ava kerman 27.04.2016 01:07 # +2

              Как только цикл закончится, всё сразу будет убрано.
              Ответить

          • ava Soul_re@ver 26.04.2016 23:50 # −1

            Хм. В никсах вроде как память физически не выделяется пока её не потрогают. Можно терабайтами выделять.

            А в винде ты даже конца свопа можешь не дождаться из-за диких тормозов.
            Ответить

            • ava Soul_re@ver 26.04.2016 23:52 # −1

              И да, если кончатся и физическая память и своп должно исключение таки вылететь, если ось процесс не прибьёт раньше.
              Ответить

              • ava Dummy00001 27.04.2016 00:17 # +1

                и на самом деле. не инициализированая память занимает только виртуальную память, но не физическую.

                к слову. а вы когда-нибудь задумывались об организационных издержках динамического управления памяти? (ну это так, толстый намек.)
                Ответить

                • ava bormand 27.04.2016 06:22 # 0

                  Здесь просто выделение же, working set не такой и большой (только те странички, которые трогает аллокатор) Так что, в теории, должно просраться и ожить через несколько минут, даже в линухе...

                  А вот если все эти инты реально юзать...
                  Ответить

                  • ava Dummy00001 27.04.2016 10:46 # 0

                    попробуй просто сам динамический аллокатор написать, и потом посмотри сколько памяти будет трогатся - и писатся - при выделении памяти. не забывать: каждый отдельно выделеный int можно отдельно освобождать.
                    Ответить

                    • ava kurwa 27.04.2016 11:05 # +1 

                      const size_t PEKA = 64;
const size_t YOBA = 100500;
char *small_kucha = new char[PEKA * YOBA];
std::vector<bool> zanyato(YOBA, false);

void *malloc(size_t skolko) {
  if (skolko <= PEKA) {
     auto svobodno = std::find(zanyato, false);
     if (svobodno != zanyato.end()) {
       *svobodno = true;
       return small_kucha[PEKA * (svobodno - zanyato.begin())];
     }
  }
  return mmap(0, skolko, PROT_READ | PROT_WRITE, MAP_SHARED | MAP_ANONYMOUS, -1, 0);
}
                      Ответить

                      • ava Dummy00001 27.04.2016 11:11 # −1

                        хм. интересно. очень интересно. а free() где?
                        Ответить

                        • ava kurwa 27.04.2016 11:23 # +3 

                          void free(void *pamyat) {
  char *pamyat2 = static_cast<char*>(pamyat);
  if (pamyat2 >= small_kucha.data() && (pamyat2 - small_kucha.data()) < small_kucha.size()) {
    zanyato[(pamyat2 - small_kucha.data()) / PEKA] = false;
  } else {
    // надо сделать munmap
  }
}
                          Ответить

                        • ava roman-kashitsyn 27.04.2016 11:29 # +2

                          free() тривиален же.
                          Если указатель внутри small_kucha, рассчитать номер блока и поставить флажок. Иначе - munmap. Правда, для unmap нужно хранить длину.
                          void free(void* p) {
  if (!p) return;
  auto ptr = static_cast<char*>(p);
  auto L = begin(small_kucha), R = end(small_kucha);
  if (ptr >= L && ptr < R) {
    auto offset = (ptr - L) / PEKA;
    if (L + offset != ptr) {
      terminate("WUT?!");
    }
    zanyato[offset] = false;
  } else {
    munmap(ptr, length_of(ptr));
  }
}
                          Ответить

                          • ava Dummy00001 27.04.2016 11:36 # +1

                            > free() тривиален же.

                            лень думать.

                            > Правда, для unmap нужно хранить длину.

                            на линухе можно просто грепнуть мапы процесса что бы найти сколько памяти было мапнуто. между блоками как правило есть немапнутые странички как разграничители.
                            Ответить

                      • ava 3.14159265 27.04.2016 14:55 # +6

                        Нейминг знатный. Не хватает переменных bugurt, kokoko, ololo и pitux.
                        Ответить

                    • ava bormand 27.04.2016 18:59 # 0

                      > попробуй просто сам динамический аллокатор написать
                      Писал. На паттерне из топика он бы трогал только свежие странички. Тут же удалений никаких нету, дырок нету, оббегать нечего.
                      Ответить

            • ava kurwa 27.04.2016 00:33 # +6

              Ты уверен, что если выделять память интами, то аллокатор не потрогает большую часть страниц?
              Ответить

    • ava gost 27.04.2016 21:08 # +1

      Течёт.
      Ответить

    • ava Antervis 03.05.2016 10:08 # 0

      а вообще, надо for(;;) new int(0);
      Ответить

      • ava kurwa 03.05.2016 11:18 # +1

        Без скобочек пишут только пидоры.
        Ответить

        • ava bormand 03.05.2016 13:50 # 0

          И символ ктулху for(;;) вместо while (true) - тоже.
          Ответить

          • ava kurwa 03.05.2016 14:48 # +1

            Как-то раз тимлид исправил в моем коммите while (true) на for (;;). Я постеснялся спросить, зачем.
            Ответить

            • ava kurwa 03.05.2016 14:49 # +2

              Еще он часто без скобочек пишет. Получается, мой тимлид - пидор.
              Ответить

              • ava bormand 03.05.2016 14:52 # +1

                > Получается, мой тимлид - пидор.
                А если он ещё и читает ГК, то завтра ты ощутишь это на своей жопе...
                Ответить

                • ava kurwa 03.05.2016 15:12 # +1

                  К счастью, на работе никто не знает, что курва - это один из моих псевдонимов.
                  Ответить

                  • ava 3_14dar 03.05.2016 19:23 # 0

                    Кстати, почему никто не подкатывает к курве? У нее же ник - курва.
                    Ответить

                    • ava guesto 04.05.2016 14:22 # +3

                      Потому что мальчиков на этом сайте любишь только ты
                      Ответить

                      • ava 3_14dar 04.05.2016 17:35 # 0

                        Да нихуя, вон вася от пердака оторваться не может.
                        Ответить

            • ava kegdan 03.05.2016 19:29 # 0

              Больше символов - больше заплатят
              Ответить

            • ava kipar 03.05.2016 20:22 # 0

              Какой-то компилятор ругался что условие в цикле всегда true и рекомендовал for (;;). Но я просто варнинг отключил и пишу while(1).
              Ответить

        • ava Antervis 04.05.2016 06:31 # 0

          еще я ради пидоров скобочки не ставил
          Ответить

    Добавить комментарий