Re: [PATCH] alpha: add udelay to io port paths

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

To: "Maciej W. Rozycki" <macro@xxxxxxxxxxxxxx>
Subject: Re: [PATCH] alpha: add udelay to io port paths
From: Sinan Kaya <Okaya@xxxxxxxxxx>
Date: Fri, 5 Apr 2019 14:34:32 -0400
Cc: Mikulas Patocka <mpatocka@xxxxxxxxxx>, Richard Henderson <rth@xxxxxxxxxxx>, Ivan Kokshaysky <ink@xxxxxxxxxxxxxxxxxxxx>, Arnd Bergmann <arnd@xxxxxxxx>, Linus Torvalds <torvalds@xxxxxxxxxxxxxxxxxxxx>, Sinan Kaya <okaya@xxxxxxxxxxxxxx>, Matt Turner <mattst88@xxxxxxxxx>, alpha <linux-alpha@xxxxxxxxxxxxxxx>
In-reply-to: <alpine.LFD.2.21.1904051927100.5650@eddie.linux-mips.org>
References: <alpine.LRH.2.02.1808161556450.13597@file01.intranet.prod.int.rdu2.redhat.com> <alpine.LRH.2.02.1902190844210.9111@file01.intranet.prod.int.rdu2.redhat.com> <CAK8P3a0zA91kLVsrWpsKWV_W0XnsOMmhdc4RV5BAgu9NcfhOLA@mail.gmail.com> <alpine.LFD.2.21.1902191652400.2486@eddie.linux-mips.org> <alpine.LRH.2.02.1902191234040.20738@file01.intranet.prod.int.rdu2.redhat.com> <alpine.LFD.2.21.1902191933170.2486@eddie.linux-mips.org> <alpine.LRH.2.02.1902201000370.18786@file01.intranet.prod.int.rdu2.redhat.com> <alpine.LFD.2.21.1902201526380.2486@eddie.linux-mips.org> <alpine.LRH.2.02.1904031244520.4480@file01.intranet.prod.int.rdu2.redhat.com> <alpine.LFD.2.21.1904042316320.5650@eddie.linux-mips.org> <alpine.LRH.2.02.1904050535001.5958@file01.intranet.prod.int.rdu2.redhat.com> <alpine.LFD.2.21.1904051748580.5650@eddie.linux-mips.org> <14889d8d-c9d7-019d-ade7-476d83f755f9@kernel.org> <alpine.LFD.2.21.1904051927100.5650@eddie.linux-mips.org>
User-agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:60.0) Gecko/20100101 Thunderbird/60.6.1

On 4/5/2019 2:28 PM, Maciej W. Rozycki wrote:

You basically can't remove the mb() after __ioread() and before
__iowrite() unless your architecture guarantees IO vs. memory ordering.

  I never suggested that; please check my example too.


Sounds good. Sorry for the noise.

I just wanted to make sure you are not removing mb() after
read(). I'm catching up here.

References:
- Alpha Avanti broken by 9ce8654323d69273b4977f76f11c9e2d345ab130
  - From: Mikulas Patocka
- Re: [PATCH] add delay between port write and port read
  - From: Mikulas Patocka
- Re: [PATCH] add delay between port write and port read
  - From: Arnd Bergmann
- Re: [PATCH] add delay between port write and port read
  - From: Maciej W. Rozycki
- Re: [PATCH] add delay between port write and port read
  - From: Mikulas Patocka
- Re: [PATCH] add delay between port write and port read
  - From: Maciej W. Rozycki
- Re: [PATCH] add delay between port write and port read
  - From: Mikulas Patocka
- Re: [PATCH] add delay between port write and port read
  - From: Maciej W. Rozycki
- [PATCH] alpha: add udelay to io port paths
  - From: Mikulas Patocka
- Re: [PATCH] alpha: add udelay to io port paths
  - From: Maciej W. Rozycki
- Re: [PATCH] alpha: add udelay to io port paths
  - From: Mikulas Patocka
- Re: [PATCH] alpha: add udelay to io port paths
  - From: Maciej W. Rozycki
- Re: [PATCH] alpha: add udelay to io port paths
  - From: Sinan Kaya
- Re: [PATCH] alpha: add udelay to io port paths
  - From: Maciej W. Rozycki

Prev by Date: Re: [PATCH] alpha: add udelay to io port paths
Next by Date: Re: [PATCH] alpha: add udelay to io port paths
Previous by thread: Re: [PATCH] alpha: add udelay to io port paths
Next by thread: Re: [PATCH] alpha: add udelay to io port paths
Index(es):
- Date
- Thread

[Index of Archives] [Netdev] [Linux Wireless] [Kernel Newbies] [Security] [Linux for Hams] [Netfilter] [Bugtraq] [Yosemite News] [MIPS Linux] [ARM Linux] [Linux RAID] [Linux Admin] [Samba]