diff for duplicates of <2944633.ljab0sy3Dg@avalon> diff --git a/a/1.txt b/N1/1.txt index 0faf1be..ce30a80 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -5,7 +5,7 @@ On Tuesday 21 Feb 2017 06:20:58 Sodagudi Prasad wrote: > > It is not clear why KERNEL_DS was set explicitly here. In this path > video_usercopy() gets called and it -> copies the ?struct v4l2_buffer? struct to user space stack memory. +> copies the “struct v4l2_buffer” struct to user space stack memory. > > Can you please share reasons for setting to KERNEL_DS here? @@ -21,8 +21,8 @@ The ioctl code should be refactored to get rid of this hack. > static long do_video_ioctl(struct file *file, unsigned int cmd, unsigned > long arg) > { -> ? -> ? +> … +> … > > if (compatible_arg) > err = native_ioctl(file, cmd, (unsigned long)up); @@ -33,7 +33,7 @@ The ioctl code should be refactored to get rid of this hack. > err = native_ioctl(file, cmd, (unsigned long)&karg); > set_fs(old_fs); > } -> ? +> … > } > > On 2017-02-16 02:39, James Morse wrote: @@ -64,8 +64,8 @@ The ioctl code should be refactored to get rid of this hack. > >> static long do_video_ioctl(struct file *file, unsigned int cmd, > >> unsigned long arg) > >> { -> >> ? -> >> ? +> >> … +> >> … > >> > >> if (compatible_arg) > >> diff --git a/a/content_digest b/N1/content_digest index ca11a30..f86f136 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -1,10 +1,27 @@ "ref\0def87360266193184dc013a055ec3869@codeaurora.org\0" "ref\058A58162.2020101@arm.com\0" "ref\0568205ddc2e7af6a57a71b8c5cd47d68@codeaurora.org\0" - "From\0laurent.pinchart@ideasonboard.com (Laurent Pinchart)\0" - "Subject\0<Query> Looking more details and reasons for using orig_add_limit.\0" + "From\0Laurent Pinchart <laurent.pinchart@ideasonboard.com>\0" + "Subject\0Re: <Query> Looking more details and reasons for using orig_add_limit.\0" "Date\0Wed, 22 Feb 2017 21:53:08 +0200\0" - "To\0linux-arm-kernel@lists.infradead.org\0" + "To\0Sodagudi Prasad <psodagud@codeaurora.org>\0" + "Cc\0James Morse <james.morse@arm.com>" + mchehab@s-opensource.com + linux-media@vger.kernel.org + shijie.huang@arm.com + catalin.marinas@arm.com + will.deacon@arm.com + mark.rutland@arm.com + akpm@linux-foundation.org + sandeepa.s.prabhu@gmail.com + linux-arm-kernel@lists.infradead.org + linux-kernel@vger.kernel.org + hans.verkuil@cisco.com + sakari.ailus@linux.intel.com + tiffany.lin@mediatek.com + nick@shmanahar.org + shuah@kernel.org + " ricardo.ribalda@gmail.com\0" "\00:1\0" "b\0" "Hi Prasad,\n" @@ -14,7 +31,7 @@ "> \n" "> It is not clear why KERNEL_DS was set explicitly here. In this path\n" "> video_usercopy() gets called and it\n" - "> copies the ?struct v4l2_buffer? struct to user space stack memory.\n" + "> copies the \342\200\234struct v4l2_buffer\342\200\235 struct to user space stack memory.\n" "> \n" "> Can you please share reasons for setting to KERNEL_DS here?\n" "\n" @@ -30,8 +47,8 @@ "> static long do_video_ioctl(struct file *file, unsigned int cmd, unsigned\n" "> long arg)\n" "> {\n" - "> ?\n" - "> ?\n" + "> \342\200\246\n" + "> \342\200\246\n" "> \n" "> if (compatible_arg)\n" "> err = native_ioctl(file, cmd, (unsigned long)up);\n" @@ -42,7 +59,7 @@ "> err = native_ioctl(file, cmd, (unsigned long)&karg);\n" "> set_fs(old_fs);\n" "> }\n" - "> ?\n" + "> \342\200\246\n" "> }\n" "> \n" "> On 2017-02-16 02:39, James Morse wrote:\n" @@ -73,8 +90,8 @@ "> >> static long do_video_ioctl(struct file *file, unsigned int cmd,\n" "> >> unsigned long arg)\n" "> >> {\n" - "> >> ?\n" - "> >> ?\n" + "> >> \342\200\246\n" + "> >> \342\200\246\n" "> >> \n" "> >> if (compatible_arg)\n" "> >> \n" @@ -196,4 +213,4 @@ "\n" Laurent Pinchart -9a8e4c55bfd9691da6425bb29f7a542976beba24fb54c0eaa80af713146f56a8 +8f63ec0164c562ae64c431cdc2ffc51dcf7c73e722b9d9072691b928d9b8e7de
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.