diff for duplicates of <1474069064.10494.4.camel@linux.intel.com> diff --git a/a/1.txt b/N1/1.txt index e732d65..a6263dc 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -1,17 +1,17 @@ -On Fri, 2016-09-16@11:16 -0700, Andy Lutomirski wrote: +On Fri, 2016-09-16 at 11:16 -0700, Andy Lutomirski wrote: > As far as I can tell, there is basically nothing correct about this -> code.??It misinterprets npss (off-by-one).??It hardcodes a bunch of +> code. It misinterprets npss (off-by-one). It hardcodes a bunch of > power states, which is nonsense, because they're all just indices -> into a table that software needs to parse.??It completely ignores +> into a table that software needs to parse. It completely ignores > the distinction between operational and non-operational states. > And, until 4.8, if all of the above magically succeeded, it would > dereference a NULL pointer and OOPS. > > Since this code appears to be useless, just delete it. -Acked-by: Jay Freyensee <james_p_freyensee at linux.intel.com> +Acked-by: Jay Freyensee <james_p_freyensee@linux.intel.com> > -> Signed-off-by: Andy Lutomirski <luto at kernel.org> +> Signed-off-by: Andy Lutomirski <luto@kernel.org> > --- ->? +> diff --git a/a/content_digest b/N1/content_digest index 6bef824..2138df7 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -1,26 +1,32 @@ "ref\0cover.1474049701.git.luto@kernel.org\0" "ref\0714df91022b06c6799afb315962769f22b8ab9f4.1474049701.git.luto@kernel.org\0" - "From\0james_p_freyensee@linux.intel.com (J Freyensee)\0" - "Subject\0[PATCH v4 1/3] nvme/scsi: Remove power management support\0" + "From\0J Freyensee <james_p_freyensee@linux.intel.com>\0" + "Subject\0Re: [PATCH v4 1/3] nvme/scsi: Remove power management support\0" "Date\0Fri, 16 Sep 2016 16:37:44 -0700\0" + "To\0Andy Lutomirski <luto@kernel.org>" + Keith Busch <keith.busch@intel.com> + " Jens Axboe <axboe@fb.com>\0" + "Cc\0linux-nvme@lists.infradead.org" + Christoph Hellwig <hch@lst.de> + " linux-kernel@vger.kernel.org\0" "\00:1\0" "b\0" - "On Fri, 2016-09-16@11:16 -0700, Andy Lutomirski wrote:\n" + "On Fri, 2016-09-16 at 11:16 -0700, Andy Lutomirski wrote:\n" "> As far as I can tell, there is basically nothing correct about this\n" - "> code.??It misinterprets npss (off-by-one).??It hardcodes a bunch of\n" + "> code.\302\240\302\240It misinterprets npss (off-by-one).\302\240\302\240It hardcodes a bunch of\n" "> power states, which is nonsense, because they're all just indices\n" - "> into a table that software needs to parse.??It completely ignores\n" + "> into a table that software needs to parse.\302\240\302\240It completely ignores\n" "> the distinction between operational and non-operational states.\n" "> And, until 4.8, if all of the above magically succeeded, it would\n" "> dereference a NULL pointer and OOPS.\n" "> \n" "> Since this code appears to be useless, just delete it.\n" "\n" - "Acked-by: Jay Freyensee <james_p_freyensee at linux.intel.com>\n" + "Acked-by: Jay Freyensee <james_p_freyensee@linux.intel.com>\n" "\n" "> \n" - "> Signed-off-by: Andy Lutomirski <luto at kernel.org>\n" + "> Signed-off-by: Andy Lutomirski <luto@kernel.org>\n" "> ---\n" - >? + > -563342adf9da5ea00cbfa10e3b972c578f7a71af93de6601fab2158b21507395 +fc5d2055b0eb72a48bda71f7df5bdef5f1328d17d61e22cef95b9e8023a766ce
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.