Messages in this thread | | | From | "Elliott, Robert (Server Storage)" <> | Subject | RE: [PATCH 1/4] pmem: Initial version of persistent memory driver | Date | Tue, 4 Nov 2014 16:26:32 +0000 |
| |
> -----Original Message----- > From: Boaz Harrosh [mailto:boaz@plexistor.com] > Sent: Tuesday, 04 November, 2014 4:38 AM > To: Wilcox, Matthew R; Elliott, Robert (Server Storage); Ross > Zwisler; Jens Axboe; Nick Piggin; Kani, Toshimitsu; Knippers, Linda; > linux-fsdevel@vger.kernel.org; linux-kernel@vger.kernel.org; linux- > nvdimm@lists.01.org; Matthew Wilcox > Subject: Re: [PATCH 1/4] pmem: Initial version of persistent memory > driver > > On 11/03/2014 06:19 PM, Wilcox, Matthew R wrote: ... > > I wish you guys would actually review the correct code. > > In the actual good driver that has any shape of proper code all these > issue are gone. > > * config defaults gone, multiple-devices multiple-memory ranges fully > supported hot plug style. > * above shifts cruft completely gone it is left overs from brd.c and > its page usage. > * getgeo fixed to do what we realy want by the only application on earth > that still uses it, fdisk. All other partitioners do not call it at > all. > > Why are we reviewing dead code ? > > Cheers > Boaz
Ross, what's the status of Boaz' patches (available in git://git.open-osd.org/pmem.git)?
https://github.com/01org/prd.git doesn't include any of them yet.
| |