From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from EUR04-VI1-obe.outbound.protection.outlook.com (EUR04-VI1-obe.outbound.protection.outlook.com [40.107.8.47]) by mx.groups.io with SMTP id smtpd.web12.3614.1592359759869120178 for ; Tue, 16 Jun 2020 19:09:20 -0700 Authentication-Results: mx.groups.io; dkim=pass header.i=@mysuse.onmicrosoft.com header.s=selector1-mysuse-onmicrosoft-com header.b=BwD9/yuK; spf=pass (domain: suse.com, ip: 40.107.8.47, mailfrom: glin@suse.com) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=XD489/LUv25d+ZG0rJEK6Cb3yaqG/7Z9T1q/ZbkDepxuDQsNN5vffMYHNsM8LDoBOFvZAjnPPLTttm1YNyZyK+Yix8BPXs6/0+B/2jUT9UGCKtT24gG3JUa60vzXruA3gX8BOz7kEvDXJ1H+7gyWfU0bpp+261hMK4cEL4K5ctZfdgdwHTWl378ejMSrUM5KPTn12tL4mv/WWSU7lXczGPqtS6EVrbczneYwZj457dZUSWJe56EdLW8cJDQsqn209Gq6mfvcTh5qMAR56TalP/BXxptb9dO9Hf2F3YZk93aUqn3rYrWNAYTpEAUvk30wmaMzQjFZK9p4l02TEiyh1Q== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=fGVknwBYi08ysICMG10qi99OeSTvvzaPIPw2QaqkNlE=; b=WI61Kz21iF7ktpLccG1Km65xzTqlZLUxbn2qbZnbjZo9/x5LPMgWjv2XeonT/2IoGc/y8dJOar2/YMyizRSa7X6ZaULpjBV2OoIqF3aRaPUitmNS4qyQ/5uiTSeTXHNvg7m5af8ViRa6R8dmAvFza34XloQWtrXU3C+McavLXRStA95vknCI3eR+hLNtUW/KC617vx8VRZ2NdgLko56MFz8UIOIJdvx8QLf5KCXIYXuvmBqDgtxVvoFjSHM5nO1wlwly6J4qOXeYcD83UjvKI72LxgUQpJ2n4DAu7644Dr4VKqUbtBn/S4NH0FVJCPdjl37sTADm8uH/UxwFavyiuQ== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=suse.com; dmarc=pass action=none header.from=suse.com; dkim=pass header.d=suse.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=mysuse.onmicrosoft.com; s=selector1-mysuse-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=fGVknwBYi08ysICMG10qi99OeSTvvzaPIPw2QaqkNlE=; b=BwD9/yuK8jxqSi7Yeojd7dcum0HPs+ek6Vj14nxBtrVN37/hGp+5kESqlE7/qZuQcB4V7j0wCeCVigoCda5RrTTOfEE3d6cCOFld7ktX4wJm7NWIJZVWr4FrQH563CqNfG/J38hm80Arfa1e7bdFbgnLq80AaCSmD/PxHmuZTmMkmc8whfRCV8izoO6giVqdtzwFemrNO6aZcidrUzlCrIMirhyj6SoJr2LbHgU2s8ZemBsofVsWJWMlpvENGyuJTP3pgPwZG+qzYwmrqtToj3KT8wkZ2f7GsN/0jPCIwh2Q0lgjaL0DDBZungOZ2gD/eDt4VCZ7iJuGhfZG7xCrVA== Authentication-Results: redhat.com; dkim=none (message not signed) header.d=none;redhat.com; dmarc=none action=none header.from=suse.com; Received: from AM0PR0402MB3809.eurprd04.prod.outlook.com (2603:10a6:208:10::30) by AM0PR0402MB3427.eurprd04.prod.outlook.com (2603:10a6:208:17::14) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3088.19; Wed, 17 Jun 2020 02:09:16 +0000 Received: from AM0PR0402MB3809.eurprd04.prod.outlook.com ([fe80::8806:715f:1409:8c6b]) by AM0PR0402MB3809.eurprd04.prod.outlook.com ([fe80::8806:715f:1409:8c6b%7]) with mapi id 15.20.3088.028; Wed, 17 Jun 2020 02:09:16 +0000 Date: Wed, 17 Jun 2020 10:09:08 +0800 From: "Gary Lin" To: Laszlo Ersek Cc: devel@edk2.groups.io, Jordan Justen , Ard Biesheuvel Subject: Re: [RFC PATCH 1/1] OvmfPkg: Introduce LSI 53C895A SCSI Controller Driver Message-ID: <20200617020908.GC18504@GaryWorkstation> References: <20200612100451.12832-1-glin@suse.com> <06577314-ae06-ed01-8c59-f533466efecc@redhat.com> In-Reply-To: <06577314-ae06-ed01-8c59-f533466efecc@redhat.com> X-ClientProxiedBy: AM0P190CA0017.EURP190.PROD.OUTLOOK.COM (2603:10a6:208:190::27) To AM0PR0402MB3809.eurprd04.prod.outlook.com (2603:10a6:208:10::30) Return-Path: glin@suse.com MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 Received: from GaryWorkstation (60.251.47.115) by AM0P190CA0017.EURP190.PROD.OUTLOOK.COM (2603:10a6:208:190::27) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3109.21 via Frontend Transport; Wed, 17 Jun 2020 02:09:14 +0000 X-Originating-IP: [60.251.47.115] X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: f35bf30d-7511-41db-4b69-08d812636ffb X-MS-TrafficTypeDiagnostic: AM0PR0402MB3427: X-Microsoft-Antispam-PRVS: X-MS-Oob-TLC-OOBClassifiers: OLM:10000; X-Forefront-PRVS: 04371797A5 X-MS-Exchange-SenderADCheck: 1 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: 5VZsuKZHTqbE3x2cSXA1RY9a9zjWchO3srE8dZ3/EWbzdCjeukbEt1J0EwHn6KQbQAID4OCDq5GUbOxn613cMqtz3SpfuQmg1KYxOixZOcHVyR7fIs3MfBGhdNB63gqo2LamvQLgjalgPAwAp/zWL7bhkmnlp6tDlWLZdjUx0ZCIoanEDFZvSjf/WV/ULE36Vn8R9qXYKShsLO0UXZkiCVZkMSKwK9n6hrTTzIYzoZ60IN3TuU2k8OrS0VoZhza2Ai76GKxteqgSCMeEMyr6XA3OEELvpm37jDVQF6afvfrib9LllR/VQEAXUusWCIbshRGjBXFtKSaJsVI9gCOs5ssQUk00AwzSp9TA9SJsO0aD7LFBzuUHVPoFR62Id2ld5YzK9GfGBMTOEkSO3MsYhg== X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:AM0PR0402MB3809.eurprd04.prod.outlook.com;PTR:;CAT:NONE;SFTY:;SFS:(366004)(6496006)(54906003)(5660300002)(83380400001)(1076003)(66476007)(956004)(66556008)(66946007)(16526019)(186003)(55016002)(33656002)(26005)(55236004)(9686003)(52116002)(6916009)(8936002)(498600001)(966005)(33716001)(4326008)(2906002)(6666004)(86362001)(53546011)(8676002);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData: ziFsW363Wh5o3nSyzabFUon9uVVXJrrMEfdoKnvWkpUkIRjcDfTyOUQlNWKzxC2yMycrIWHCzpIjl74fhSEPOjlayk0BB8xAhzgIPxCfl2CRa493VEYxVD7Sfakv0Mcz80NGO6mbTRJqIhSmomchCUISgIMLbPCgQtV2U1CwpD+T7vHLMj0Q1KcTLO+qqfK/Oz3iTqfvw9MM7LiKgLsQ2ABsmKSZAyjYYo9pBE6328J21p3R8xuIxi/kdg7AAbLv7kLWJx5SmLDGEbkwB0LkR0sSom14gV+mFz7RYm3mUnVMCOY2WK25QXelLnHbsPuyLZGHmu3+8NE+aCNGaTgcRHPqzC328CsChLslZ1O2WQw3GVp8LP7KaB/vKtNbwphXkPl2tZd9O+3OP+R6fZfpiHjht4UQELfo5/ZZ9PGElx1CmnpoxKfkOKdm6PzqWbnqr2owlIkXBQLqrKtMhrSDWNIT/IGNc1KED7tN3jyi9FQ= X-OriginatorOrg: suse.com X-MS-Exchange-CrossTenant-Network-Message-Id: f35bf30d-7511-41db-4b69-08d812636ffb X-MS-Exchange-CrossTenant-OriginalArrivalTime: 17 Jun 2020 02:09:16.6225 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: f7a17af6-1c5c-4a36-aa8b-f5be247aa4ba X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: pM84kOzee6hxdQehk64D3gLiHRMyrNh3l05nKNRcTk4IgwX6701SlKvkGGskFSwf X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM0PR0402MB3427 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline On Mon, Jun 15, 2020 at 01:27:22PM +0200, Laszlo Ersek wrote: > Hello Gary, > Hi Laszlo, > On 06/12/20 12:04, Gary Lin wrote: > > This commit introduces the driver for LSI 53C895A SCSI Controller > > which, so that OVMF can access the devices attached to the emulated > > "lsi" SCSI controller. > > > > Cc: Jordan Justen > > Cc: Laszlo Ersek > > Cc: Ard Biesheuvel > > Signed-off-by: Gary Lin > > --- > > Although lsi is now considered obsolete in QEMU, I wrote this driver > > mainly for learning the UEFI SCSI driver and ... FUN! The majority of > > the code is actually borrowed from VirtioScsci and MptScsi, and I > > mainly focus on LsiScsiPassThru(). > > > > If it's fine to add this driver into OvmfPkg, I'll start to split this > > patch into smaller pieces to make it easier to review. > > (1) Do we have an official deprecation notice for this SCSI controller > in QEMU? > > If we do, then (AIUI) the controller will be removed in QEMU in one or > two releases, so this code would become effectively dead in the mid > term. I wouldn't like to review and/or carry code that's soon to be > dead. > I just vaguely remember that virtio-scsi is the new default over lsi and it's not recommended to use lsi except for the old OS without virtio-scsi driver. > (2) If there is no official deprecation notice in QEMU, then I agree it > makes sense to include this driver. In that case, I have another > question: > > Do you intend this driver for production purposes? I.e., do you expect > users or "layered products" (libvirt, proxmox, openstack, ...) to use > this SCSI controller for some well-defined purpose? (The MPT SCSI and PV > SCSI drivers had a clear product-oriented modivation: > > https://edk2.groups.io/g/devel/message/55620 > http://mid.mail-archive.com/a96b6b74-c35d-e291-2122-9d77f1d5f89c@oracle.com > ) > > (2a) If this driver is not meant for a production environment, then > LSI_SCSI_ENABLE should be FALSE by default (and I'll do a lighter > review). > > (2b) If the driver is meant for production, then LSI_SCSI_ENABLE should > indeed be TRUE, and I'll have to be more diligent in reviewing this. > I kind of wonder if the any serious use for lsi+ovmf. If the OS is so old and only supports lsi, seabios probably serves it better. Anyway, I'll try to gather more information from my colleagues to see if they got any feedback about lsi from our customers. If not, I'll set LSI_SCSI_ENABLE to FALSE by default. > For either (2a) or (2b), please do split up the driver into smaller > patches, and please also add yourself to Maintainers.txt as the > designated reviewer of the new driver. > Sure. Will do that. Thanks, Gary Lin