> -----Original Message----- > From: Konrad Rzeszutek Wilk [mailto:konrad.wilk@xxxxxxxxxx] > Sent: Friday, September 23, 2011 11:00 AM > To: Rose, Gregory V > Cc: Kirsher, Jeffrey T; Ian Campbell; davem@xxxxxxxxxxxxx; Jesse Barnes; > netdev@xxxxxxxxxxxxxxx; gospo@xxxxxxxxxx; linux-pci@xxxxxxxxxxxxxxx > Subject: Re: [net-next 1/8] pci: Add flag indicating device has been > assigned by KVM > > On Fri, Sep 23, 2011 at 10:04:59AM -0700, Rose, Gregory V wrote: > > > -----Original Message----- > > > From: Konrad Rzeszutek Wilk [mailto:konrad.wilk@xxxxxxxxxx] > > > Sent: Friday, September 23, 2011 9:43 AM > > > To: Kirsher, Jeffrey T > > > Cc: Rose, Gregory V; Ian Campbell; davem@xxxxxxxxxxxxx; Jesse Barnes; > > > netdev@xxxxxxxxxxxxxxx; gospo@xxxxxxxxxx; linux-pci@xxxxxxxxxxxxxxx > > > Subject: Re: [net-next 1/8] pci: Add flag indicating device has been > > > assigned by KVM > > > > > > On Fri, Sep 23, 2011 at 08:45:09AM -0700, Jeff Kirsher wrote: > > > > On Fri, 2011-09-23 at 08:11 -0700, Rose, Gregory V wrote: > > > > > > -----Original Message----- > > > > > > From: Ian Campbell [mailto:ijc@xxxxxxxxxxxxxx] > > > > > > Sent: Friday, September 23, 2011 8:04 AM > > > > > > To: Rose, Gregory V > > > > > > Cc: Kirsher, Jeffrey T; davem@xxxxxxxxxxxxx; > konrad.wilk@xxxxxxxxxx; > > > Jesse > > > > > > Barnes; netdev@xxxxxxxxxxxxxxx; gospo@xxxxxxxxxx; linux- > > > > > > pci@xxxxxxxxxxxxxxx > > > > > > Subject: RE: [net-next 1/8] pci: Add flag indicating device has > been > > > > > > assigned by KVM > > > > > > > > > > > > On Fri, 2011-09-23 at 07:41 -0700, Rose, Gregory V wrote: > > > > > > > > -----Original Message----- > > > > > > > > From: Ian Campbell [mailto:ijc@xxxxxxxxxxxxxx] > > > > > > > > Sent: Friday, September 23, 2011 12:28 AM > > > > > > > > To: Kirsher, Jeffrey T > > > > > > > > Cc: davem@xxxxxxxxxxxxx; konrad.wilk@xxxxxxxxxx; Jesse > Barnes; > > > Rose, > > > > > > > > Gregory V; netdev@xxxxxxxxxxxxxxx; gospo@xxxxxxxxxx; linux- > > > > > > > > pci@xxxxxxxxxxxxxxx > > > > > > > > Subject: Re: [net-next 1/8] pci: Add flag indicating device > has > > > been > > > > > > > > assigned by KVM > > > > > > > > > > > > > > > > I suppose by that measure the comment could be less KVM > > > specific: > > > > > > > > > + /* Provide indication device is assigned by KVM */ > > > > > > > > > + PCI_DEV_FLAGS_ASSIGNED = (__force pci_dev_flags_t) > 4, > > > > > > > > > > > > > > We can resubmit with a more generic comment, maybe this: > > > > > > > > > > > > > > /* Provide indication device is assigned by a Virtual Machine > > > Manager */ > > > > > > > > > > > > Sounds good to me. > > > > > > > > > > Dave, Jeff, > > > > > > > > > > Should I resubmit the patch or would it be more convenient to post > a > > > follow on patch that fixes up the comment? Either way is fine by me. > > > > > > > > > > - Greg > > > > > > > > > > > > > let's fix up the patch in my tree and add Jesse Barnes's ACK at the > same > > > > time. > > > > > > Were are the patches now? I am kind of curious to see the changes to > the > > > xen pciback. > > > > > > > > > > Version 2 of this patch with the modified comment was just posted to > netdev. Ian Campbell had mentioned that the Xen pciback driver should be > modified to use this flag but I don't believe any patches for it have been > created yet. > > http://marc.info/?i=20110729165446.GA6731@xxxxxxxxxxxx > I'm not sure who generated the patches to xen-pciback. It wasn't me and I couldn't tell from the links you provided who did generate that patch. I don't see it now though. - Greg -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
