lkml.org 
[lkml]   [2020]   [May]   [26]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH v3 03/18] nitro_enclaves: Define enclave info for internal bookkeeping
From
Date


On 26/05/2020 09:46, Greg KH wrote:
> On Tue, May 26, 2020 at 01:13:19AM +0300, Andra Paraschiv wrote:
>> +/* Nitro Enclaves (NE) misc device */
>> +extern struct miscdevice ne_miscdevice;
> Why does your misc device need to be in a .h file?
>
> Having the patch series like this (add random .h files, and then start
> to use them), is hard to review. Would you want to try to review a
> series written in this way?

The misc device is registered / unregistered while having the NE PCI
device probe / remove, as a dependency to actually having a PCI device
working to expose a misc device.

The way the codebase is split in files is mainly the ioctl logic / misc
device in one file and the PCI device logic in another file; thus not
have all the codebase in a single big file. Given the misc device
(un)register logic above, the misc device needs to be available to the
PCI device setup logic.

Andra




Amazon Development Center (Romania) S.R.L. registered office: 27A Sf. Lazar Street, UBC5, floor 2, Iasi, Iasi County, 700045, Romania. Registered in Romania. Registration number J22/2621/2005.
\
 
 \ /
  Last update: 2020-05-26 19:21    [W:0.104 / U:1.116 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site