[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20151030213546.GL7716@linux.vnet.ibm.com>
Date: Fri, 30 Oct 2015 14:35:46 -0700
From: Nishanth Aravamudan <nacc@...ux.vnet.ibm.com>
To: David Miller <davem@...emloft.net>
Cc: hch@...radead.org, keith.busch@...el.com, aik@...abs.ru,
linux-kernel@...r.kernel.org, linux-nvme@...ts.infradead.org,
paulus@...ba.org, sparclinux@...r.kernel.org,
willy@...ux.intel.com, linuxppc-dev@...ts.ozlabs.org,
david@...son.dropbear.id.au
Subject: Re: [PATCH 0/5 v3] Fix NVMe driver support on Power with 32-bit DMA
On 29.10.2015 [18:49:55 -0700], David Miller wrote:
> From: Nishanth Aravamudan <nacc@...ux.vnet.ibm.com>
> Date: Thu, 29 Oct 2015 08:57:01 -0700
>
> > So, would that imply changing just the NVMe driver code rather than
> > adding the dma_page_shift API at all? What about
> > architectures that can support the larger page sizes? There is an
> > implied performance impact, at least, of shifting the IO size down.
> >
> > Sorry for the continuing questions -- I got lots of conflicting feedback
> > on the last series and want to make sure v4 is more acceptable.
>
> In the long term I would be very happy to see us having a real interface
> for this stuff, just my opinion...
Yep, I think I'll try and balance the two -- fix NVMe for now with a 4K
page size as suggested by Christoph, and then work on the more complete
API for the next merge.
Thanks,
Nish
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists