Хороший пример лайвлока?

142

Я понимаю, что такое livelock, но мне было интересно, есть ли у кого-нибудь хороший пример этого кода? И под «основанным на коде» я не имею в виду «двух людей, пытающихся обойти друг друга в коридоре». Если я прочитаю это снова, то потеряю обед.

Алекс Миллер
источник
97
Как насчет компьютерной симуляции двух людей, пытающихся обойти друг друга в коридоре?
ИНФОРМАЦИЯ 1800
37
Будь проклят! Я потерял обед!
Alex Miller,
3
Странно уместно: seuss.wikia.com/wiki/The_Zax
NotMe
Родственная шутка для любопытных парней: codingarchitect.wordpress.com/2006/01/18/…
Jorjon
4
Два человека пытаются обойти друг друга в коридоре: gist.github.com/deepankarb/d2dd6f21bc49902376e614d3746b8965 : p
iceman

Ответы:

121

Вот очень простой пример livelock на Java, когда муж и жена пытаются съесть суп, но между ними остается только одна ложка. Каждый из супругов слишком вежлив и передаст ложку, если другой еще не ел.

public class Livelock {
    static class Spoon {
        private Diner owner;
        public Spoon(Diner d) { owner = d; }
        public Diner getOwner() { return owner; }
        public synchronized void setOwner(Diner d) { owner = d; }
        public synchronized void use() { 
            System.out.printf("%s has eaten!", owner.name); 
        }
    }

    static class Diner {
        private String name;
        private boolean isHungry;

        public Diner(String n) { name = n; isHungry = true; }       
        public String getName() { return name; }
        public boolean isHungry() { return isHungry; }

        public void eatWith(Spoon spoon, Diner spouse) {
            while (isHungry) {
                // Don't have the spoon, so wait patiently for spouse.
                if (spoon.owner != this) {
                    try { Thread.sleep(1); } 
                    catch(InterruptedException e) { continue; }
                    continue;
                }                       

                // If spouse is hungry, insist upon passing the spoon.
                if (spouse.isHungry()) {                    
                    System.out.printf(
                        "%s: You eat first my darling %s!%n", 
                        name, spouse.getName());
                    spoon.setOwner(spouse);
                    continue;
                }

                // Spouse wasn't hungry, so finally eat
                spoon.use();
                isHungry = false;               
                System.out.printf(
                    "%s: I am stuffed, my darling %s!%n", 
                    name, spouse.getName());                
                spoon.setOwner(spouse);
            }
        }
    }

    public static void main(String[] args) {
        final Diner husband = new Diner("Bob");
        final Diner wife = new Diner("Alice");

        final Spoon s = new Spoon(husband);

        new Thread(new Runnable() { 
            public void run() { husband.eatWith(s, wife); }   
        }).start();

        new Thread(new Runnable() { 
            public void run() { wife.eatWith(s, husband); } 
        }).start();
    }
}
Джереми Элборн
источник
6
Разве getOwnerметод тоже не нужно синхронизировать? Начиная с Effective Java, « синхронизация не имеет эффекта, если и чтение, и запись ».
Санхюн Ли
Разве ему не следует использовать Thread.join()вместо Thread.sleep(), потому что он хочет дождаться, пока супруга поест?
Solace
что мы должны сделать, чтобы преодолеть проблему блокировки в этом конкретном примере?
Тор
1
getOwnerМетод должен быть синхронизированы , поскольку даже если setOwnerсинхронизируется, это не гарантирует нить с помощью getOwner(или доступ поле ownerнепосредственно) будут видеть изменения , сделанные другим потоком , выполняющего setOwner. В этом видео это очень подробно объясняется: youtube.com/watch?v=WTVooKLLVT8
Тимофей
2
Вам не нужно использовать synchronized ключевое слово для setOwnerметода, потому что чтение и запись являются атомарными действиями для ссылочной переменной.
атикхалед
75

Если оставить в стороне ошибочные комментарии, известно, что один из примеров - это код, который пытается обнаруживать и обрабатывать ситуации взаимоблокировки. Если два потока обнаруживают взаимоблокировку и пытаются «отойти в сторону» друг для друга, они без особой осторожности застрянут в цикле, который всегда «отступает в сторону» и никогда не сможет двигаться вперед.

Под «отойти в сторону» я имею в виду, что они снимут блокировку и попытаются позволить другому заполучить ее. Мы можем представить себе ситуацию, когда это делают два потока (псевдокод):

// thread 1
getLocks12(lock1, lock2)
{
  lock1.lock();
  while (lock2.locked())
  {
    // attempt to step aside for the other thread
    lock1.unlock();
    wait();
    lock1.lock();
  }
  lock2.lock();
}

// thread 2
getLocks21(lock1, lock2)
{
  lock2.lock();
  while (lock1.locked())
  {
    // attempt to step aside for the other thread
    lock2.unlock();
    wait();
    lock2.lock();
  }
  lock1.lock();
}

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

Правильное решение - всегда соблюдать иерархию блокировок . Выберите порядок, в котором вы приобретаете замки, и придерживайтесь его. Например, если оба потока всегда получают lock1 перед lock2, то нет возможности взаимоблокировки.

1800 ИНФОРМАЦИЯ
источник
Да, я это понимаю. Я ищу реальный пример такого кода. Вопрос в том, что означает «отойти в сторону» и как это порождает такой сценарий.
Alex Miller
Я понимаю, что это надуманный пример, но возможно ли, что это может привести к блокировке? Разве не было бы намного более вероятным, что в конечном итоге откроется окно, в котором одна функция может захватить обе, из-за несоответствий во времени, когда потоки запускаются вслух, и когда они запланированы.
DubiousPusher
Хотя это не стабильный лайвлок, потому что они, очевидно, вырвутся из него в конце концов, я думаю, что он достаточно хорошо подходит под описание
ИНФОРМАЦИЯ 18
Отличный и содержательный пример.
Алецов
7

Поскольку нет ответа, помеченного как принятый, я попытался создать пример живой блокировки;

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

Итак, давайте сначала разберемся с постановкой проблемы;

Проблема с создателем файлов cookie

Есть несколько контейнеров для ингредиентов: ChocoPowederContainer , WheatPowderContainer . CookieMaker берет некоторое количество порошка из контейнеров для ингредиентов, чтобы испечь печенье . Если производитель файлов cookie обнаруживает, что контейнер пуст, он проверяет наличие другого контейнера, чтобы сэкономить время. И ждет, пока Filler заполнит необходимый контейнер. Есть наполнитель, который регулярно проверяет контейнер и наполняет некоторое количество, если это необходимо контейнеру.

Пожалуйста, проверьте полный код на github ;

Позвольте мне вкратце объяснить вам реализацию.

  • Я запускаю Filler как поток демона. Таким образом, он будет регулярно наполнять контейнеры. Чтобы сначала заполнить контейнер, он блокирует контейнер -> проверяет, нужен ли ему порошок -> заполняет его -> сигнализирует всем производителям, которые его ждут -> разблокирует контейнер.
  • Я создаю CookieMaker и устанавливаю, что он может запекать до 8 файлов cookie параллельно. И я запускаю 8 потоков, чтобы испечь печенье.
  • Каждый поток создателя создает 2 вызываемых подпотока для извлечения порошка из контейнеров.
  • вспомогательная резьба захватывает контейнер и проверяет, достаточно ли в нем порошка. Если нет, подождите некоторое время. Когда наполнитель заполняет контейнер, он берет порошок и разблокирует контейнер.
  • Теперь он выполняет другие действия, такие как приготовление смеси, выпечка и т. Д.

Заглянем в код:

CookieMaker.java

private Integer getMaterial(final Ingredient ingredient) throws Exception{
        :
        container.lock();
        while (!container.getIngredient(quantity)) {
            container.empty.await(1000, TimeUnit.MILLISECONDS);
            //Thread.sleep(500); //For deadlock
        }
        container.unlock();
        :
}

IngredientContainer.java

public boolean getIngredient(int n) throws Exception {
    :
    lock();
    if (quantityHeld >= n) {
        TimeUnit.SECONDS.sleep(2);
        quantityHeld -= n;
        unlock();
        return true;
    }
    unlock();
    return false;
}

Все идет нормально, пока Filler не заполнит контейнеры. Но если я забываю запустить наполнитель или наполнитель неожиданно уходит в отпуск, подпотоки продолжают менять свое состояние, позволяя другому производителю пойти и проверить контейнер.

Я также создал демон ThreadTracer, который следит за состояниями потоков и взаимоблокировками. Это вывод из консоли;

2016-09-12 21:31:45.065 :: [Maker_0:WAITING, Maker_1:WAITING, Maker_2:WAITING, Maker_3:WAITING, Maker_4:WAITING, Maker_5:WAITING, Maker_6:WAITING, Maker_7:WAITING, pool-7-thread-1:TIMED_WAITING, pool-7-thread-2:TIMED_WAITING, pool-8-thread-1:TIMED_WAITING, pool-8-thread-2:TIMED_WAITING, pool-6-thread-1:TIMED_WAITING, pool-6-thread-2:TIMED_WAITING, pool-5-thread-1:TIMED_WAITING, pool-5-thread-2:TIMED_WAITING, pool-1-thread-1:TIMED_WAITING, pool-3-thread-1:TIMED_WAITING, pool-2-thread-1:TIMED_WAITING, pool-1-thread-2:TIMED_WAITING, pool-4-thread-1:TIMED_WAITING, pool-4-thread-2:RUNNABLE, pool-3-thread-2:TIMED_WAITING, pool-2-thread-2:TIMED_WAITING]
2016-09-12 21:31:45.065 :: [Maker_0:WAITING, Maker_1:WAITING, Maker_2:WAITING, Maker_3:WAITING, Maker_4:WAITING, Maker_5:WAITING, Maker_6:WAITING, Maker_7:WAITING, pool-7-thread-1:TIMED_WAITING, pool-7-thread-2:TIMED_WAITING, pool-8-thread-1:TIMED_WAITING, pool-8-thread-2:TIMED_WAITING, pool-6-thread-1:TIMED_WAITING, pool-6-thread-2:TIMED_WAITING, pool-5-thread-1:TIMED_WAITING, pool-5-thread-2:TIMED_WAITING, pool-1-thread-1:TIMED_WAITING, pool-3-thread-1:TIMED_WAITING, pool-2-thread-1:TIMED_WAITING, pool-1-thread-2:TIMED_WAITING, pool-4-thread-1:TIMED_WAITING, pool-4-thread-2:TIMED_WAITING, pool-3-thread-2:TIMED_WAITING, pool-2-thread-2:TIMED_WAITING]
WheatPowder Container has 0 only.
2016-09-12 21:31:45.082 :: [Maker_0:WAITING, Maker_1:WAITING, Maker_2:WAITING, Maker_3:WAITING, Maker_4:WAITING, Maker_5:WAITING, Maker_6:WAITING, Maker_7:WAITING, pool-7-thread-1:TIMED_WAITING, pool-7-thread-2:TIMED_WAITING, pool-8-thread-1:TIMED_WAITING, pool-8-thread-2:TIMED_WAITING, pool-6-thread-1:TIMED_WAITING, pool-6-thread-2:TIMED_WAITING, pool-5-thread-1:TIMED_WAITING, pool-5-thread-2:TIMED_WAITING, pool-1-thread-1:TIMED_WAITING, pool-3-thread-1:TIMED_WAITING, pool-2-thread-1:TIMED_WAITING, pool-1-thread-2:TIMED_WAITING, pool-4-thread-1:TIMED_WAITING, pool-4-thread-2:TIMED_WAITING, pool-3-thread-2:TIMED_WAITING, pool-2-thread-2:RUNNABLE]
2016-09-12 21:31:45.082 :: [Maker_0:WAITING, Maker_1:WAITING, Maker_2:WAITING, Maker_3:WAITING, Maker_4:WAITING, Maker_5:WAITING, Maker_6:WAITING, Maker_7:WAITING, pool-7-thread-1:TIMED_WAITING, pool-7-thread-2:TIMED_WAITING, pool-8-thread-1:TIMED_WAITING, pool-8-thread-2:TIMED_WAITING, pool-6-thread-1:TIMED_WAITING, pool-6-thread-2:TIMED_WAITING, pool-5-thread-1:TIMED_WAITING, pool-5-thread-2:TIMED_WAITING, pool-1-thread-1:TIMED_WAITING, pool-3-thread-1:TIMED_WAITING, pool-2-thread-1:TIMED_WAITING, pool-1-thread-2:TIMED_WAITING, pool-4-thread-1:TIMED_WAITING, pool-4-thread-2:TIMED_WAITING, pool-3-thread-2:TIMED_WAITING, pool-2-thread-2:TIMED_WAITING]

Вы заметите, что подпотоки меняют свое состояние и ожидают.

Амит Кумар Гупта
источник
4

Реальный (хотя и без точного кода) пример - это два конкурирующих процесса с динамической блокировкой в ​​попытке исправить тупик SQL-сервера, причем каждый процесс использует один и тот же алгоритм ожидания-повтора для повторной попытки. Хотя это удача по времени, я видел, что это происходило на разных машинах с аналогичными характеристиками производительности в ответ на сообщение, добавленное в тему EMS (например, сохранение обновления одного графа объекта более одного раза), и невозможность управления порядок блокировки.

Хорошим решением в этом случае было бы наличие конкурирующих потребителей (предотвращение дублирования обработки как можно выше в цепочке путем разделения работы на несвязанные объекты).

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

Еще одно решение (по крайней мере, для SQL Server) - попробовать другой уровень изоляции (например, моментальный снимок).

Комплект
источник
2

Я закодировал пример прохождения 2 человек по коридору. Две нити будут избегать друг друга, как только поймут, что их направления совпадают.

public class LiveLock {
    public static void main(String[] args) throws InterruptedException {
        Object left = new Object();
        Object right = new Object();
        Pedestrian one = new Pedestrian(left, right, 0); //one's left is one's left
        Pedestrian two = new Pedestrian(right, left, 1); //one's left is two's right, so have to swap order
        one.setOther(two);
        two.setOther(one);
        one.start();
        two.start();
    }
}

class Pedestrian extends Thread {
    private Object l;
    private Object r;
    private Pedestrian other;
    private Object current;

    Pedestrian (Object left, Object right, int firstDirection) {
        l = left;
        r = right;
        if (firstDirection==0) {
            current = l;
        }
        else {
            current = r;
        }
    }

    void setOther(Pedestrian otherP) {
        other = otherP;
    }

    Object getDirection() {
        return current;
    }

    Object getOppositeDirection() {
        if (current.equals(l)) {
            return r;
        }
        else {
            return l;
        }
    }

    void switchDirection() throws InterruptedException {
        Thread.sleep(100);
        current = getOppositeDirection();
        System.out.println(Thread.currentThread().getName() + " is stepping aside.");
    }

    public void run() {
        while (getDirection().equals(other.getDirection())) {
            try {
                switchDirection();
                Thread.sleep(100);
            } catch (InterruptedException e) {}
        }
    }
} 
PoweredByRice
источник
2

C # версия кода jelbourn:

using System;
using System.Runtime.CompilerServices;
using System.Threading;
using System.Threading.Tasks;

namespace LiveLockExample
{
    static class Program
    {
        public static void Main(string[] args)
        {
            var husband = new Diner("Bob");
            var wife = new Diner("Alice");

            var s = new Spoon(husband);

            Task.WaitAll(
                Task.Run(() => husband.EatWith(s, wife)),
                Task.Run(() => wife.EatWith(s, husband))
                );
        }

        public class Spoon
        {
            public Spoon(Diner diner)
            {
                Owner = diner;
            }


            public Diner Owner { get; private set; }

            [MethodImpl(MethodImplOptions.Synchronized)]
            public void SetOwner(Diner d) { Owner = d; }

            [MethodImpl(MethodImplOptions.Synchronized)]
            public void Use()
            {
                Console.WriteLine("{0} has eaten!", Owner.Name);
            }
        }

        public class Diner
        {
            public Diner(string n)
            {
                Name = n;
                IsHungry = true;
            }

            public string Name { get; private set; }

            private bool IsHungry { get; set; }

            public void EatWith(Spoon spoon, Diner spouse)
            {
                while (IsHungry)
                {
                    // Don't have the spoon, so wait patiently for spouse.
                    if (spoon.Owner != this)
                    {
                        try
                        {
                            Thread.Sleep(1);
                        }
                        catch (ThreadInterruptedException e)
                        {
                        }

                        continue;
                    }

                    // If spouse is hungry, insist upon passing the spoon.
                    if (spouse.IsHungry)
                    {
                        Console.WriteLine("{0}: You eat first my darling {1}!", Name, spouse.Name);
                        spoon.SetOwner(spouse);
                        continue;
                    }

                    // Spouse wasn't hungry, so finally eat
                    spoon.Use();
                    IsHungry = false;
                    Console.WriteLine("{0}: I am stuffed, my darling {1}!", Name, spouse.Name);
                    spoon.SetOwner(spouse);
                }
            }
        }
    }
}
asostechnix
источник
1

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

boolean tryLockAll(Collection<Lock> locks) {
  boolean grabbedAllLocks = false;
  for(int i=0; i<locks.size(); i++) {
    Lock lock = locks.get(i);
    if(!lock.tryLock(5, TimeUnit.SECONDS)) {
      grabbedAllLocks = false;

      // undo the locks I already took in reverse order
      for(int j=i-1; j >= 0; j--) {
        lock.unlock();
      }
    }
  }
}

Я могу представить, что такой код будет проблематичным, поскольку у вас много потоков, которые сталкиваются и ожидают получения набора блокировок. Но я не уверен, что этот простой пример мне подходит.

Алекс Миллер
источник
2
чтобы это была livelock, вам понадобится другой поток, чтобы получить эти блокировки в другом порядке. Если все потоки используют tryLockAll()блокировки в locksодном и том же порядке, живой блокировки нет.
JaviMerino
0

Версия кода jelbourn для Python:

import threading
import time
lock = threading.Lock()

class Spoon:
    def __init__(self, diner):
        self.owner = diner

    def setOwner(self, diner):
        with lock:
            self.owner = diner

    def use(self):
        with lock:
            "{0} has eaten".format(self.owner)

class Diner:
    def __init__(self, name):
        self.name = name
        self.hungry = True

    def eatsWith(self, spoon, spouse):
        while(self.hungry):
            if self != spoon.owner:
                time.sleep(1) # blocks thread, not process
                continue

            if spouse.hungry:
                print "{0}: you eat first, {1}".format(self.name, spouse.name)
                spoon.setOwner(spouse)
                continue

            # Spouse was not hungry, eat
            spoon.use()
            print "{0}: I'm stuffed, {1}".format(self.name, spouse.name)
            spoon.setOwner(spouse)

def main():
    husband = Diner("Bob")
    wife = Diner("Alice")
    spoon = Spoon(husband)

    t0 = threading.Thread(target=husband.eatsWith, args=(spoon, wife))
    t1 = threading.Thread(target=wife.eatsWith, args=(spoon, husband))
    t0.start()
    t1.start()
    t0.join()
    t1.join()

if __name__ == "__main__":
    main()
nflacco
источник
Ошибки: в use () печать не используется и, что более важно, флаг голодания не установлен в значение False.
GDR
0

Я изменяю ответ @jelbourn. Когда один из них замечает, что другой голоден, он (она) должен отпустить ложку и дождаться другого уведомления, чтобы произошла блокировка.

public class LiveLock {
    static class Spoon {
        Diner owner;

        public String getOwnerName() {
            return owner.getName();
        }

        public void setOwner(Diner diner) {
            this.owner = diner;
        }

        public Spoon(Diner diner) {
            this.owner = diner;
        }

        public void use() {
            System.out.println(owner.getName() + " use this spoon and finish eat.");
        }
    }

    static class Diner {
        public Diner(boolean isHungry, String name) {
            this.isHungry = isHungry;
            this.name = name;
        }

        private boolean isHungry;
        private String name;


        public String getName() {
            return name;
        }

        public void eatWith(Diner spouse, Spoon sharedSpoon) {
            try {
                synchronized (sharedSpoon) {
                    while (isHungry) {
                        while (!sharedSpoon.getOwnerName().equals(name)) {
                            sharedSpoon.wait();
                            //System.out.println("sharedSpoon belongs to" + sharedSpoon.getOwnerName())
                        }
                        if (spouse.isHungry) {
                            System.out.println(spouse.getName() + "is hungry,I should give it to him(her).");
                            sharedSpoon.setOwner(spouse);
                            sharedSpoon.notifyAll();
                        } else {
                            sharedSpoon.use();
                            sharedSpoon.setOwner(spouse);
                            isHungry = false;
                        }
                        Thread.sleep(500);
                    }
                }
            } catch (InterruptedException e) {
                System.out.println(name + " is interrupted.");
            }
        }
    }

    public static void main(String[] args) {
        final Diner husband = new Diner(true, "husband");
        final Diner wife = new Diner(true, "wife");
        final Spoon sharedSpoon = new Spoon(wife);

        Thread h = new Thread() {
            @Override
            public void run() {
                husband.eatWith(wife, sharedSpoon);
            }
        };
        h.start();

        Thread w = new Thread() {
            @Override
            public void run() {
                wife.eatWith(husband, sharedSpoon);
            }
        };
        w.start();

        try {
            Thread.sleep(10000);
        } catch (InterruptedException e) {
            e.printStackTrace();
        }
        h.interrupt();
        w.interrupt();

        try {
            h.join();
            w.join();
        } catch (InterruptedException e) {
            e.printStackTrace();
        }
    }
}
И Чжан
источник
0
package concurrently.deadlock;

import static java.lang.System.out;


/* This is an example of livelock */
public class Dinner {

    public static void main(String[] args) {
        Spoon spoon = new Spoon();
        Dish dish = new Dish();

        new Thread(new Husband(spoon, dish)).start();
        new Thread(new Wife(spoon, dish)).start();
    }
}


class Spoon {
    boolean isLocked;
}

class Dish {
    boolean isLocked;
}

class Husband implements Runnable {

    Spoon spoon;
    Dish dish;

    Husband(Spoon spoon, Dish dish) {
        this.spoon = spoon;
        this.dish = dish;
    }

    @Override
    public void run() {

        while (true) {
            synchronized (spoon) {
                spoon.isLocked = true;
                out.println("husband get spoon");
                try { Thread.sleep(2000); } catch (InterruptedException e) {}

                if (dish.isLocked == true) {
                    spoon.isLocked = false; // give away spoon
                    out.println("husband pass away spoon");
                    continue;
                }
                synchronized (dish) {
                    dish.isLocked = true;
                    out.println("Husband is eating!");

                }
                dish.isLocked = false;
            }
            spoon.isLocked = false;
        }
    }
}

class Wife implements Runnable {

    Spoon spoon;
    Dish dish;

    Wife(Spoon spoon, Dish dish) {
        this.spoon = spoon;
        this.dish = dish;
    }

    @Override
    public void run() {
        while (true) {
            synchronized (dish) {
                dish.isLocked = true;
                out.println("wife get dish");
                try { Thread.sleep(2000); } catch (InterruptedException e) {}

                if (spoon.isLocked == true) {
                    dish.isLocked = false; // give away dish
                    out.println("wife pass away dish");
                    continue;
                }
                synchronized (spoon) {
                    spoon.isLocked = true;
                    out.println("Wife is eating!");

                }
                spoon.isLocked = false;
            }
            dish.isLocked = false;
        }
    }
}
Афанасий В.
источник