Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <b3d5e596-adce-20b5-73cc-89c07917082f@nvidia.com>
Date:   Thu, 17 Mar 2022 19:58:25 -0700
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101
 Thunderbird/91.7.0
Subject: Re: [PATCH v2] mm/hmm/test: simplify hmm test code: use miscdevice
 instead of char dev
Content-Language: en-US
To:     =?UTF-8?Q?Mika_Penttil=c3=a4?= <mpenttil@redhat.com>,
        Jason Gunthorpe <jgg@ziepe.ca>
Cc:     linux-kernel@vger.kernel.org, linux-mm@kvack.org,
        apopple@nvidia.com, rcampbell@nvidia.com, vbabka@suse.cz
References: <20220311033050.22724-1-mpenttil@redhat.com>
 <20220314182439.GB64706@ziepe.ca>
 <d9b9c82e-4012-cf0a-d966-d9669a684a27@redhat.com>
 <20220315183922.GC64706@ziepe.ca>
 <8e836d75-97b0-d301-4d6a-92025e91cad5@redhat.com>
 <0f53c5ff-1711-861d-9a12-0634d4ab9fca@redhat.com>
 <20220317141539.GD64706@ziepe.ca>
 <10fb1261-f22f-8e21-4056-166c283e26f2@redhat.com>
From:   John Hubbard <jhubbard@nvidia.com>
In-Reply-To: <10fb1261-f22f-8e21-4056-166c283e26f2@redhat.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 8bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?bGtGbkNYdGhwdmhISG5FWmxMUXVuMlJzWE5lUVpTSi9qa0ZYOU11dmcxMlVz?=
 =?utf-8?B?REN1MkZtVUJ0eTJZN091dzMreHVobWxqUDNnV0xKMnpTQTlMQ1FwaEhhTlJL?=
 =?utf-8?B?ajRwb2hhV25ORldRQUVUOEUya2N2WHdtU2lWdjRCdTVWY3NnMzhuVk0zZis1?=
 =?utf-8?B?dnY3UWYrYkFrNk5FZGk2MldZSWNIT3VOZW1sSEVjSER0NWgvWC9uMVpmK0RV?=
 =?utf-8?B?akNHM25xM0NLeit0QmdEQUIxdzY3b3ZSS095eGV4YkRTMWVvS3VNcEFQTE51?=
 =?utf-8?B?NnNkOHJQRUgvMVhpQk54Szd5b2wzRnhDNzkrYysrMzdkenBUclQvWHpWaGFn?=
 =?utf-8?B?TTVwZjczVk51QUs2Z09nVEtwazM4bDIrdEpDTkxLQ2xwOHJVRzZQenVkZGtG?=
 =?utf-8?B?MU4zQUFSSXJRS2w2UFNuYlU4cEZtQ2syL1hrTWVNUnk1QS9kMVZZeFRXcjh5?=
 =?utf-8?B?QnRwOFVsWGY5WWpWRU42ZU5QTHVENEFvcktHWXNKcG9US1BDN3Naa0xEQzNn?=
 =?utf-8?B?OE51Uk8ra2dWZlJ2b1hTMnBkTEEyVnFmRWFJN0gxZUdpM2J6cnFRRUFvcTJM?=
 =?utf-8?B?ZEtkSk9xTUk0M3VCSVhmZFZ5OXExbVdDckZqYVE2NnZXTEhKK1k4TDQ0QmRM?=
 =?utf-8?B?ekFSMCtFOU1wNmJxTmgxVmYzTzlrcFdHZExiaWNLNkZjYmpqNHVOcHM3MWVt?=
 =?utf-8?B?MEZqRzI4SVlXdERDTFd1K25FeFNURElYL1ZTZG9ZdUl2dTRuS2JtbXN3Y1Bs?=
 =?utf-8?B?RWZ2Vm1vM3g2WVpLYjFBdllYSER0d3RHbWppdzhDdXJDU0M4NElERGlYTGYz?=
 =?utf-8?B?WnNoNGNhcDRyQXY5bEZVc0hreGlQUmI4QytPNE9PRGIvWVVoaExwS1ZNWUlu?=
 =?utf-8?B?NVZMYWZKQ0owOWNzKzJRQUROL2dLMURjWWg4b0Nmd25CNXJvRkEvUElDalh5?=
 =?utf-8?B?bjRKU1A3TTROa2V2NWhoYmNGcXU3UURROVVZLzkrR2txOXhZS2tFMnFiVXoy?=
 =?utf-8?B?ZlE0U1k2YnVIV1JGUkM3dnFwaUxoNGlEQUVDNlFaN1JSdWhFdGkzTk02Qk1N?=
 =?utf-8?B?VTkrRitpdHhuVTlhbjJlRFphaE84aTJFYnlTN09KeG0rTWUzVmh4Q0hMR0dQ?=
 =?utf-8?B?bzQ3RXA0Ni9Sc280aUtPcnpFcHBsZHl3SXdYRndGMWM1S0VQbkpDTG1uMDlR?=
 =?utf-8?B?ZjEzOVozbkV3cmVyRkdFeXhSbEMzUTZ6dXVVSlFKSVgyNmJLQ3lyY3lPTGxZ?=
 =?utf-8?B?YVkyZ25QL1RybDQ3ck9RWG41ajhyNzcrY0QyZjVibnNNZXNyK2VsRnpROWdN?=
 =?utf-8?B?TWtRdGg2TTlabHNkWVhYdnNaMzhNbkhLb1RwTytqcWF2YmxadCt3Nkd0V0hy?=
 =?utf-8?B?alZIU2sxSjF5ME1YV1MvUkdBN014SDM0cTRuZk02MUdYUk1ucEw3a0FDMklj?=
 =?utf-8?B?RUxJS2h4ZXA0VURLcUkrUnc4YXB5VXV2alJrMytWeFdTYWdMMXgrUXc1VEJk?=
 =?utf-8?B?QlV4WEJEbXZ3cXY2aDB2N1dUNnMzV2g3cGsrZmFIY1dGcVhjcXBibEFMRW5F?=
 =?utf-8?B?cjE0dko0SkJySVQ0ZzRBOHprK0E4UnBzVlZNbnpZUTN6dERBWWY5NGYzVENN?=
 =?utf-8?B?dUsxZ3N1QzFtdy85RGNhK0h1cTRkWkV0aXFrWmhYM091bkhWUkhuVVlxVmgr?=
 =?utf-8?B?TURWUCt5VTlPdWJtaldhakxXNUY0UjZzVGlNMndkSGdZNnkwRGJnTGFWT2lE?=
 =?utf-8?B?L0ZXV3VUWGhLYUhmUGZmak1WZitNT3gvWWJuaDh2TWxBRzh5cUtHZTlzMFdx?=
 =?utf-8?B?ZzhkTjVoMGptcEdIaVdoKzZoZXQxaVhrZ0xRZDBmdysvRkF5VG5zcnltWE1F?=
 =?utf-8?B?eG1NQ0dYWUpmbkJtNnZlRWJFOWQyenVpUVJQMFpIZ0xGajZ3NXlsTzJON3Fa?=
 =?utf-8?B?akdVUjB5bTVSYUhwS3Npd1FNaFRGSmVnZXNBYUFEOE5aWVZGVmlvTXN4SVkx?=
 =?utf-8?B?L1pYV3g2MENBPT0=?=
X-OriginatorOrg: Nvidia.com
X-MS-Exchange-CrossTenant-Network-Message-Id: d5788b68-3d3e-4af2-1999-08da088b2d2e
X-MS-Exchange-CrossTenant-AuthSource: BY5PR12MB4130.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 18 Mar 2022 02:58:27.9537
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 43083d15-7273-40c1-b7db-39efd9ccc17a
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: VW1WZBE4viztNaGhVV0BSd02jFQamQ1xdXJtTqZ1iy00cvCdzpykRz3+BHDCgK34MsFNG1pbTnTsMmhGYcZ49Q==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM6PR12MB3963
X-Spam-Status: No, score=-2.6 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH,
        DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,FORGED_SPF_HELO,
        NICE_REPLY_A,RCVD_IN_DNSWL_NONE,RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,
        SPF_NONE,T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no
        version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On 3/17/22 19:34, Mika Penttilä wrote:
> On 17.3.2022 16.15, Jason Gunthorpe wrote:
>> On Thu, Mar 17, 2022 at 08:58:52AM +0200, Mika Penttilä wrote:
>>> @@ -1225,7 +1232,11 @@ static int dmirror_device_init(struct dmirror_device
>>> *mdevice, int id)
>>>
>>>          cdev_init(&mdevice->cdevice, &dmirror_fops);
>>>          mdevice->cdevice.owner = THIS_MODULE;
>>> -       ret = cdev_add(&mdevice->cdevice, dev, 1);
>>> +       device_initialize(&mdevice->device);
>>> +       dev_set_name(&mdevice->device, "%s", dmirror_device_names[id]);
>>> +       mdevice->device.devt = dev;
>>> +
>>> +       ret = cdev_device_add(&mdevice->cdevice, &mdevice->device);
>>>          if (ret)
>>>                  return ret;
>>
>> Right, miscdev isn't that helpful in the end..
>>
>> Jason
>>
> 
> To wrap up, I could send a v3 formal patch with either this cdev way or the miscdev way.. Both 
> eliminate the user space /proc/devices parsing and mknod'ing. Jason brought up concerns using 
> miscdevice like this, although for this case it works correctly. miscdevice also provides a little 
> more cleanup and simpler code, but the difference is not huge. So what do people prefer?
> 

No strong preference from me either way, but I would like to bottom on
the potential problems with using misdevice. The fact that miscdevice is
misused in other drivers does not seem like a reason that it must be not
used here...Jason, after Mika's explanation, what's your latest
assessment?

In other words, given that the usage here is correct, is there some
reason that miscdevice is still a poor design fit? Is cdev_device_add()
a better choice here, for design reasons?

Also, is there any change that could or should be made to miscdevice,
that you have in mind?


thanks,
-- 
John Hubbard
NVIDIA