Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
From:   Matt Sickler <Matt.Sickler@daktronics.com>
To:     Bharath Vedartham <linux.bhar@gmail.com>,
        "jhubbard@nvidia.com" <jhubbard@nvidia.com>,
        "ira.weiny@intel.com" <ira.weiny@intel.com>,
        "jglisse@redhat.com" <jglisse@redhat.com>,
        "gregkh@linuxfoundation.org" <gregkh@linuxfoundation.org>
CC:     "linux-mm@kvack.org" <linux-mm@kvack.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "devel@driverdev.osuosl.org" <devel@driverdev.osuosl.org>
Subject: RE: [PATCH v3] staging: kpc2000: Convert put_page to put_user_page*()
Thread-Topic: [PATCH v3] staging: kpc2000: Convert put_page to
 put_user_page*()
Thread-Index: AQHVPmzwE09u8sDYuEmu2VoulZlSaKbSax/w
Date:   Fri, 19 Jul 2019 20:59:02 +0000
Message-ID: <SN6PR02MB4016754FE1BB6200746281A2EECB0@SN6PR02MB4016.namprd02.prod.outlook.com>
References: <20190719200235.GA16122@bharath12345-Inspiron-5559>
In-Reply-To: <20190719200235.GA16122@bharath12345-Inspiron-5559>
Accept-Language: en-US
Content-Language: en-US
received-spf: None (protection.outlook.com: daktronics.com does not designate
 permitted sender hosts)
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-Network-Message-Id: b09fe611-239c-4c46-f80e-08d70c8bed98
X-MS-Exchange-CrossTenant-originalarrivaltime: 19 Jul 2019 20:59:02.1607
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: be88af81-0945-42aa-a3d2-b122777351a2
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: matt.sickler@daktronics.com
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SN6PR02MB4191
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

>From: Bharath Vedartham <linux.bhar@gmail.com>
>Changes since v2
>        - Added back PageResevered check as suggested by John Hubbard.
>
>The PageReserved check needs a closer look and is not worth messing
>around with for now.
>
>Matt, Could you give any suggestions for testing this patch?

Myself or someone else from Daktronics would have to do the testing since t=
he
hardware isn't really commercially available.  I've been toying with the id=
ea
of asking for a volunteer from the mailing list to help me out with this - =
I'd
send them some hardware and they'd do all the development and testing. :)
I still have to run that idea by Management though.

>If in-case, you are willing to pick this up to test. Could you
>apply this patch to this tree and test it with your devices?

I've been meaning to get to testing the changes to the drivers since upstre=
aming
them, but I've been swamped with other development.  I'm keeping an eye on =
the
mailing lists, so I'm at least aware of what is coming down the pipe.
I'm not too worried about this specific change, even though I don't really =
know
if the reserved check and the dirtying are even necessary.
It sounded like John's suggestion was to not do the PageReserved() check an=
d just
use put_user_pges_dirty() all the time.  John, is that incorrect?