qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Justin Hibbits <chmeeedalf@gmail.com>
To: Laurent Vivier <laurent@vivier.eu>
Cc: qemu-trivial@nongnu.org, "Emilio G. Cota" <cota@braap.org>,
	qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [Qemu-trivial] Fix cacheline size retrieval on FreeBSD/PowerPC(64)
Date: Wed, 26 Jun 2019 19:15:25 -0500	[thread overview]
Message-ID: <CAHSQbTCUzR5WiykWGZOH1bvBPWx15_ZA-=7b17c6AnRK4kn3yA@mail.gmail.com> (raw)
In-Reply-To: <477ebe22-eddb-41ba-5811-4a70e639a89a@vivier.eu>

On Wed, Jun 26, 2019, 19:08 Laurent Vivier <laurent@vivier.eu> wrote:

> Le 27/06/2019 à 02:02, Justin Hibbits a écrit :
> >
> >
> > On Wed, Jun 26, 2019, 13:04 Justin Hibbits <chmeeedalf@gmail.com
> > <mailto:chmeeedalf@gmail.com>> wrote:
> >
> >     On Wed, 26 Jun 2019 18:47:42 +0200
> >     Laurent Vivier <laurent@vivier.eu <mailto:laurent@vivier.eu>> wrote:
> >
> >     > Le 26/06/2019 à 18:37, Justin Hibbits a écrit :
> >     > > On Wed, 26 Jun 2019 18:16:36 +0200
> >     > > Laurent Vivier <laurent@vivier.eu <mailto:laurent@vivier.eu>>
> wrote:
> >     > >
> >     > >> Le 26/06/2019 à 18:14, Laurent Vivier a écrit :
> >     > >>> Le 07/06/2019 à 20:56, Justin Hibbits a écrit :
> >     > >>>> The attached very trivial patch fixes a startup bug that
> prevents
> >     > >>>> at least Qemu 3.1 and later from working on FreeBSD/powerpc64.
> >     > >>>>
> >     > >>>> - Justin
> >     > >>>>
> >     > >>>
> >     > >>> Please don't send a patch in attachment but inlined in the
> message
> >     > >>> (you may use "git send-email" for that).
> >     > >>>
> >     > >>> This patch fixes "util: add cacheinfo" that has changed the
> type
> >     > >>> from unsigned to long.
> >     > >>>
> >     > >>> You can add the following line in the commit message:
> >     > >>>
> >     > >>> Fixes: b255b2c8a548 ("util: add cacheinfo")
> >     > >>>
> >     > >>> Reviewed-by: Laurent Vivier <laurent@vivier.eu
> >     <mailto:laurent@vivier.eu>>
> >     > >>>
> >     > >>
> >     > >> CC: author of b255b2c8a548 ("util: add cacheinfo")
> >     > >>
> >     > >> Thanks,
> >     > >> Laurent
> >     > >
> >     > > Hi Laurent,
> >     > >
> >     > > Sorry.  I had never used git send-email before, so wasn't
> >     > > comfortable with it.  I just updated the commit message with your
> >     > > feedback and used git send-email to submit the patch.  I hope
> >     > > everything went well.
> >     >
> >     > It seems not. I didn't receive it.
> >     >
> >     > Did you configure the SMTP server. See git-send-email(1):
> >     >
> >     >    Use gmail as the smtp server
> >     >
> >     >        To use git send-email to send your patches through the GMail
> >     > SMTP server, edit ~/.gitconfig to specify your account settings:
> >     >
> >     >            [sendemail]
> >     >                    smtpEncryption = tls
> >     >                    smtpServer = smtp.gmail.com <
> http://smtp.gmail.com>
> >     >                    smtpUser = yourname@gmail.com
> >     <mailto:yourname@gmail.com>
> >     >                    smtpServerPort = 587
> >     >
> >     >        If you have multifactor authentication setup on your gmail
> >     > account, you will need to generate an app-specific password for use
> >     > with git send-email. Visit
> >     >        https://security.google.com/settings/security/apppasswords
> to
> >     > create it.
> >     >
> >     >        Once your commits are ready to be sent to the mailing list,
> >     > run the following commands:
> >     >
> >     >            $ git format-patch --cover-letter -M origin/master -o
> >     > outgoing/ $ edit outgoing/0000-*
> >     >            $ git send-email outgoing/*
> >     >
> >     >        The first time you run it, you will be prompted for your
> >     > credentials. Enter the app-specific or your regular password as
> >     > appropriate. If you have credential helper configured (see
> >     > git-credential(1)), the password will be saved in the credential
> >     > store so you won’t have to type it the next time.
> >     >
> >     >        Note: the following perl modules are required
> Net::SMTP::SSL,
> >     >        MIME::Base64 and Authen::SASL
> >     >
> >     > Thanks,
> >     > Laurent
> >     >
> >     >
> >
> >     Hm, you're right.  Even after making the config changes and
> installing
> >     the necessary packages, I still have no luck with git send-email.
> Might
> >     take a bit to debug this.
> >
> >     - Justin
> >
> >
> > Sorry for the multiplicity, looks like Gmail forwarded it eventually,
> > but not until I tried several times debugging it.
>
> Not sure, I didn't receive any of them.
>
> Thanks,
> Laurent
>

Sigh, 4 copies ended up in my Gmail spam folder, so I thought it made it
through.

- Justin

>

  reply	other threads:[~2019-06-27  0:17 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-06-07 18:56 [Qemu-devel] Fix cacheline size retrieval on FreeBSD/PowerPC(64) Justin Hibbits
2019-06-26 16:14 ` [Qemu-devel] [Qemu-trivial] " Laurent Vivier
2019-06-26 16:16   ` Laurent Vivier
2019-06-26 16:37     ` Justin Hibbits
2019-06-26 16:47       ` Laurent Vivier
2019-06-26 18:04         ` Justin Hibbits
2019-06-27  0:02           ` Justin Hibbits
2019-06-27  0:08             ` Laurent Vivier
2019-06-27  0:15               ` Justin Hibbits [this message]
2019-07-01  8:54                 ` Laurent Vivier
2019-07-01 13:02                   ` Justin Hibbits
2019-07-01 13:17                     ` Laurent Vivier
2019-07-02 22:11                       ` Justin Hibbits
2019-07-01 14:08       ` Christophe de Dinechin

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to='CAHSQbTCUzR5WiykWGZOH1bvBPWx15_ZA-=7b17c6AnRK4kn3yA@mail.gmail.com' \
    --to=chmeeedalf@gmail.com \
    --cc=cota@braap.org \
    --cc=laurent@vivier.eu \
    --cc=qemu-devel@nongnu.org \
    --cc=qemu-trivial@nongnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).