diff for duplicates of <1485537668.4267.7.camel@sandisk.com> diff --git a/a/1.txt b/N1/1.txt index 8d3962a..0022c3a 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -1,16 +1,17 @@ On Fri, 2017-01-27 at 17:12 +0100, Christoph Hellwig wrote: > On Wed, Jan 25, 2017 at 10:15:55PM -0500, Martin K. Petersen wrote: > > +static void *alloc_request_size(gfp_t gfp_mask, void *data) -> > +> >=20 > > I like alloc_request_simple() but alloc_request_size() seems a bit > > contrived. _reserve? _extra? _special? Don't have any good suggestions, > > I'm afraid. -> +>=20 > Not that I'm a fan of _size, but I like the other suggestions even less. Hello Christoph and Martin, -How about using the function names alloc_full_request() / free_full_request() +How about using the function names alloc_full_request() / free_full_request= +() together with a comment that mentions that cmd_size is set by the LLD? -Bart. +Bart.= diff --git a/a/content_digest b/N1/content_digest index 9921712..7414fa7 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -7,30 +7,31 @@ "Date\0Fri, 27 Jan 2017 17:21:28 +0000\0" "To\0hch@lst.de <hch@lst.de>" " martin.petersen@oracle.com <martin.petersen@oracle.com>\0" - "Cc\0linux-block@vger.kernel.org <linux-block@vger.kernel.org>" - linux-scsi@vger.kernel.org <linux-scsi@vger.kernel.org> - snitzer@redhat.com <snitzer@redhat.com> - axboe@fb.com <axboe@fb.com> + "Cc\0linux-scsi@vger.kernel.org <linux-scsi@vger.kernel.org>" linux-raid@vger.kernel.org <linux-raid@vger.kernel.org> dm-devel@redhat.com <dm-devel@redhat.com> + linux-block@vger.kernel.org <linux-block@vger.kernel.org> + axboe@fb.com <axboe@fb.com> + snitzer@redhat.com <snitzer@redhat.com> " j-nomura@ce.jp.nec.com <j-nomura@ce.jp.nec.com>\0" "\00:1\0" "b\0" "On Fri, 2017-01-27 at 17:12 +0100, Christoph Hellwig wrote:\n" "> On Wed, Jan 25, 2017 at 10:15:55PM -0500, Martin K. Petersen wrote:\n" "> > +static void *alloc_request_size(gfp_t gfp_mask, void *data)\n" - "> > \n" + "> >=20\n" "> > I like alloc_request_simple() but alloc_request_size() seems a bit\n" "> > contrived. _reserve? _extra? _special? Don't have any good suggestions,\n" "> > I'm afraid.\n" - "> \n" + ">=20\n" "> Not that I'm a fan of _size, but I like the other suggestions even less.\n" "\n" "Hello Christoph and Martin,\n" "\n" - "How about using the function names alloc_full_request() / free_full_request()\n" + "How about using the function names alloc_full_request() / free_full_request=\n" + "()\n" "together with a comment that mentions that cmd_size is set by the LLD?\n" "\n" - Bart. + Bart.= -0cbcdb644acb6144c5c058b0acadca764b6ca60b2be3663fc33c56f9b252e459 +9695029d3b482a49056be054be40173908e7d9fb61a9997ab7213cee352fc20b
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.