From mboxrd@z Thu Jan 1 00:00:00 1970 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751372AbeAPQRQ (ORCPT + 1 other); Tue, 16 Jan 2018 11:17:16 -0500 Received: from mx0b-001b2d01.pphosted.com ([148.163.158.5]:48922 "EHLO mx0a-001b2d01.pphosted.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1750868AbeAPQRO (ORCPT ); Tue, 16 Jan 2018 11:17:14 -0500 Subject: Re: [PATCH 1/1] vfio-ccw: update documentation To: Cornelia Huck , bjsdjshi@linux.vnet.ibm.com, pmorel@linux.vnet.ibm.com, borntraeger@de.ibm.com Cc: linux-s390@vger.kernel.org, kvm@vger.kernel.org, linux-kernel@vger.kernel.org References: <20180115104353.27413-1-cohuck@redhat.com> <20180115104353.27413-2-cohuck@redhat.com> From: Halil Pasic Date: Tue, 16 Jan 2018 17:17:08 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.5.0 MIME-Version: 1.0 In-Reply-To: <20180115104353.27413-2-cohuck@redhat.com> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit X-TM-AS-GCONF: 00 x-cbid: 18011616-0012-0000-0000-000005A46223 X-IBM-AV-DETECTION: SAVI=unused REMOTE=unused XFE=unused x-cbparentid: 18011616-0013-0000-0000-0000191FD511 Message-Id: <5a99b7e2-3cce-c92d-f1d8-1469a334ff6d@linux.vnet.ibm.com> X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10432:,, definitions=2018-01-16_05:,, signatures=0 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 priorityscore=1501 malwarescore=0 suspectscore=0 phishscore=0 bulkscore=0 spamscore=0 clxscore=1011 lowpriorityscore=0 impostorscore=0 adultscore=0 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.0.1-1709140000 definitions=main-1801160226 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Return-Path: On 01/15/2018 11:43 AM, Cornelia Huck wrote: > The vfio-ccw documentation comes from the cover letter of the > original patch submission, which shows in some parts. Give it some > love; in particular: > > - Remove/rework statements that make sense in a cover letter, but not > in regular documentation. > - Fix some typos. > - Describe the current limitations in more detail. > > Signed-off-by: Cornelia Huck The changes look sane and definitively improve on what we have right now. Acked by me if you like. When we are at the limitations, the prefetch story seems to be still missing. IMHO it's the most important limitation. Regards, Halil