On Wed, Dec 14, 2022 at 04:30:48PM -0500, Peter Xu wrote: > On Wed, Dec 14, 2022 at 09:14:09AM +0000, Daniel P. Berrangé wrote: > > On Tue, Dec 13, 2022 at 04:38:46PM -0500, Peter Xu wrote: > > > From: "manish.mishra" <manish.mis...@nutanix.com> > > > > > > MSG_PEEK reads from the peek of channel, The data is treated as > > > unread and the next read shall still return this data. This > > > support is currently added only for socket class. Extra parameter > > > 'flags' is added to io_readv calls to pass extra read flags like > > > MSG_PEEK. > > > > > > Reviewed-by: Daniel P. Berrang?? <berra...@redhat.co > > > Suggested-by: Daniel P. Berrang?? <berra...@redhat.com > > > > The last letter of my name has been mangled - whatever tools used > > to pull in manish's patches seem to not be UTF-8 clean. > > > > Also the email addr isn't terminated, but that was pre-existing > > in manish's previous posting. > > I'll fix at least the latter in my next post, sorry. > > For the 1st one - I am still looking at what went wrong. > > Here from the web interfaces it all looks good (besides the wrong > ending..), e.g. on lore or patchew: > > https://lore.kernel.org/all/20221213213850.1481858-2-pet...@redhat.com/ > https://patchew.org/QEMU/20221213213850.1481858-1-pet...@redhat.com/20221213213850.1481858-2-pet...@redhat.com/ > > It also looks good with e.g. Gmail webclient. > > Then I digged into the email headers and I found that comparing to Manish's > original message, the patches I posted has one more line of "Content-type": > > Content-Type: text/plain; charset="utf-8" > Content-type: text/plain > https://patchew.org/QEMU/20221213213850.1481858-2-pet...@redhat.com/mbox > > While Manish's patch only has one line: > > Content-Type: text/plain; charset="utf-8" > > https://patchew.org/QEMU/20221123172735.25181-2-manish.mis...@nutanix.com/mbox
Don't trust what is shown by patchew, as that's been through many hops. The copy I receieved came directly to me via CC, so didn't hit mailman, nor patchew, and that *only* has "Content-type: text/plain". So the extra Content-type line with utf8 must have been added either by mailman or patchew. So it probably looks like a config problem in the tool you use to send the patches originally. With regards, Daniel -- |: https://berrange.com -o- https://www.flickr.com/photos/dberrange :| |: https://libvirt.org -o- https://fstop138.berrange.com :| |: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|
