Re: [PATCH 0/2 REPOST] remove unneded irq save

2018-06-14 Thread Dan Williams
On Thu, Jun 14, 2018 at 7:30 AM, Sebastian Andrzej Siewior wrote: > On 2018-06-12 08:46:38 [-0700], Dan Williams wrote: >> On Tue, Jun 12, 2018 at 8:04 AM, John Garry wrote: >> >> We had this comment for 6 years or so and nothing happend. What makes >> >> you think that an updated version of

Re: [PATCH 0/2 REPOST] remove unneded irq save

2018-06-14 Thread Sebastian Andrzej Siewior
On 2018-06-12 08:46:38 [-0700], Dan Williams wrote: > On Tue, Jun 12, 2018 at 8:04 AM, John Garry wrote: > >> We had this comment for 6 years or so and nothing happend. What makes > >> you think that an updated version of that comment will motivate someone > >> to make change here in the near

Re: [PATCH 0/2 REPOST] remove unneded irq save

2018-06-12 Thread Dan Williams
On Tue, Jun 12, 2018 at 8:04 AM, John Garry wrote: > On 12/06/2018 15:31, Sebastian Andrzej Siewior wrote: >> >> On 2018-06-12 13:54:36 [+0100], John Garry wrote: >>> >>> +Dan >>> >>> On 11/06/2018 19:23, Sebastian Andrzej Siewior wrote: On 2018-06-11 18:12:55 [+0100], John Garry wrote:

Re: [PATCH 0/2 REPOST] remove unneded irq save

2018-06-12 Thread John Garry
On 12/06/2018 15:31, Sebastian Andrzej Siewior wrote: On 2018-06-12 13:54:36 [+0100], John Garry wrote: +Dan On 11/06/2018 19:23, Sebastian Andrzej Siewior wrote: On 2018-06-11 18:12:55 [+0100], John Garry wrote: On 11/06/2018 15:40, Sebastian Andrzej Siewior wrote: This is the repost of

Re: [PATCH 0/2 REPOST] remove unneded irq save

2018-06-12 Thread Sebastian Andrzej Siewior
On 2018-06-12 13:54:36 [+0100], John Garry wrote: > +Dan > > On 11/06/2018 19:23, Sebastian Andrzej Siewior wrote: > > On 2018-06-11 18:12:55 [+0100], John Garry wrote: > > > On 11/06/2018 15:40, Sebastian Andrzej Siewior wrote: > > > > This is the repost of the two patches I posted in earlier

Re: [PATCH 0/2 REPOST] remove unneded irq save

2018-06-12 Thread John Garry
+Dan On 11/06/2018 19:23, Sebastian Andrzej Siewior wrote: On 2018-06-11 18:12:55 [+0100], John Garry wrote: On 11/06/2018 15:40, Sebastian Andrzej Siewior wrote: This is the repost of the two patches I posted in earlier this month: - [PATCH 1/2] libsas: remove irq save in sas_ata_qc_issue()

Re: [PATCH 0/2 REPOST] remove unneded irq save

2018-06-11 Thread Sebastian Andrzej Siewior
On 2018-06-11 18:12:55 [+0100], John Garry wrote: > On 11/06/2018 15:40, Sebastian Andrzej Siewior wrote: > > This is the repost of the two patches I posted in earlier this month: > > > > - [PATCH 1/2] libsas: remove irq save in sas_ata_qc_issue() > > Received feedback but nothing really

Re: [PATCH 0/2 REPOST] remove unneded irq save

2018-06-11 Thread John Garry
On 11/06/2018 15:40, Sebastian Andrzej Siewior wrote: This is the repost of the two patches I posted in earlier this month: - [PATCH 1/2] libsas: remove irq save in sas_ata_qc_issue() Received feedback but nothing really changed. I explained that this is not about "local_irqsave() +

[PATCH 0/2 REPOST] remove unneded irq save

2018-06-11 Thread Sebastian Andrzej Siewior
This is the repost of the two patches I posted in earlier this month: - [PATCH 1/2] libsas: remove irq save in sas_ata_qc_issue() Received feedback but nothing really changed. I explained that this is not about "local_irqsave() + spin_lock()" *but* "local_irq_save() + spin_unlock()". This