All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Paul E. McKenney" <paulmck@linux.vnet.ibm.com>
To: Akira Yokosawa <akiyks@gmail.com>
Cc: perfbook@vger.kernel.org
Subject: Re: [GIT PULL] SMPdesign: Use subscript in lockdeqhash figures
Date: Mon, 8 Aug 2016 09:42:45 -0700	[thread overview]
Message-ID: <20160808164245.GK3482@linux.vnet.ibm.com> (raw)
In-Reply-To: <60290e7c-1444-9064-d481-640ae3e5daa0@gmail.com>

On Tue, Aug 09, 2016 at 12:21:04AM +0900, Akira Yokosawa wrote:
> On 2016/08/07 17:15:28 -0700, Paul E. McKenney wrote:
> > On Mon, Aug 08, 2016 at 07:08:17AM +0900, Akira Yokosawa wrote:
> >> On 2016/08/07 11:18:39 -0700, Paul E. McKenney wrote:
> >>> On Sat, Aug 06, 2016 at 07:07:29PM +0900, Akira Yokosawa wrote:
> >>>> Hi Paul,
> >>>>
> >>>> This is somewhat related to the "using hyphens for minus signs" issue.
> >>>> In Figures 6.8 and 6.9, entry names such as "L-2" are used.
> >>>> The "-" here means minus, but it is *not* obvious at first glance if
> >>>> it means "L minus 2" or not.
> >>>>
> >>>> If we make "-2" a subscript, it would be much easier to see that
> >>>> it means "L whose sequence number is 'minus 2'".
> >>>> So, I tweaked the xfig sources of the figures to make those numbers
> >>>> in entries look like subscripts. I also altered the numbers in entry names
> >>>> mentioned in the corresponding text as subscripts (using math mode).
> >>>>
> >>>> I'm sending this as a pull request as the resulting patch became fairly
> >>>> large due to eps files.
> >>>>
> >>>> I hope you like there tweaks.
> >>>>
> >>>> Additional patch is a fix of entry numbers of Figure 6.9, which is
> >>>> actually 16 if my understanding of the figure is correct.
> >>>
> >>> I pulled both, thank you.
> >>>
> >>> Any thoughts on converting .fig files to .svg?  My approach has been to
> >>> import .fig files into inkscape as I need to modify them.  The results
> >>> are OK, but not perfect (for example, arrows in .fig become two objects
> >>> in .svg, the line and the arrowhead).
> >>
> >> So, I tried this.
> >> But by importing into inkscape, all the texts get the same font size...
> >> Unfortunately, font size specifier in .fig seems to be ignored.
> >> Or I might be missing something.
> > 
> > Hmmm...  I guess I have been getting lucky, working only with .fig files
> > with a single font size.
> > 
> >> BTW, now that we can use -jN option for building .pdf files, wouldn't it
> >> be a good idea to get rid of .eps files in the repository whose sources
> >> are .fig files by adding a ".fig -> .eps" rule in Makefile?
> >> Then the patch would have been much smaller.
> >>
> >> Let me know what you think.
> > 
> > Making that initial clone and build go faster makes a lot of sense to me!
> 
> Well, current build starts from .eps and if we remove .eps, then the build
> will start from .fig. So the build time should increase somewhat.
> Reduction of the patch size will be the point of doing this.

Fair point!

> I'll try and evaluate how much longer the first build would take.
> Give me a week or so. Ah, my environment has only a few cores so the
> evaluation result would be only a hint.

I do have a 128-core system, although it is shared, so use of more than
(say) 64 cores is unlikely to give reproducible results.  Just let me
know exactly what you want tested, preferably by sending me a simple
script that does what you want.

							Thanx, Paul


      reply	other threads:[~2016-08-08 16:42 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-08-06 10:07 [GIT PULL] SMPdesign: Use subscript in lockdeqhash figures Akira Yokosawa
2016-08-07 18:18 ` Paul E. McKenney
2016-08-07 22:08   ` Akira Yokosawa
2016-08-08  0:15     ` Paul E. McKenney
2016-08-08 15:21       ` Akira Yokosawa
2016-08-08 16:42         ` Paul E. McKenney [this message]

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=20160808164245.GK3482@linux.vnet.ibm.com \
    --to=paulmck@linux.vnet.ibm.com \
    --cc=akiyks@gmail.com \
    --cc=perfbook@vger.kernel.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.